Merge branch 'freebsd-ignore-errors' into 'main'

ci: ignore FreeBSD build failures until new container is ready

See merge request xen-project/xen-guest-agent!56

ci: ignore FreeBSD build failures until new container is ready

Updating the cross-rs container needs some time, let's not stop
things here just because that work is not ready yet.

Signed-off-by: Yann Dirson <yann.dirson@vates.fr>

Merge branch 'cargo-lock' into 'main'

Cargo.lock: update outdated deps

See merge request xen-project/xen-guest-agent!53

Cargo.lock: update outdated deps

Signed-off-by: Yann Dirson <yann.dirson@vates.fr>

Merge branch 'ci-cross-aarch64' into 'main'

WIP ci: cross-build Linux aarch64 binary

See merge request xen-project/xen-guest-agent!52

ci: cross-build Linux aarch64 binary

Testbuilding only, as we don't have an obvious use-case to support.

Builds against glibc in a Debian chroot to be able to check
the xenstore build feature.

ci: more consistency on parallel specification

Merge branch 'ci-images-full-path' into 'main'

ci: use full path for container images

See merge request xen-project/xen-guest-agent!50

Merge branch 'ci-cross-windows' into 'main'

Windows CI support

See merge request xen-project/xen-guest-agent!51

ci: cross-build Windows binary

Mostly a PoC, since we're missing crucial bits:
- Xenstore works differently on WIndows, and we don't have a crate
  supporting it yet
- pnet will not build with mingw, needs msvc, and it seems there are
  things to be done before we can use that

Use panic!() not abort!()

Hopefully we won't really need that safeguard, but it's still nice to be
able to compile non non-unix platforms.

Signed-off-by: Yann Dirson <yann.dirson@vates.fr>

ci: use full path for container images

Much easier to copy-paste them on a podman command-line, alias files
are not always up to date.

Signed-off-by: Yann Dirson <yann.dirson@vates.fr>

Merge branch 'ci-no-scheduled-pkg-source' into 'main'

ci: don't run source packaging when scheduled

See merge request xen-project/xen-guest-agent!49

ci: don't run source packaging when scheduled

Signed-off-by: Yann Dirson <yann.dirson@vates.fr>

Merge branch 'scheduled-freebsd' into 'main'

ci/freebsd: use proper before_script template

See merge request xen-project/xen-guest-agent!48

ci/freebsd: use proper before_script template

Fixes 27fb1ae87b512efc2c55657108fa27531a26f3ea, should really
investigate why I was not able to test this scheduled job on the PR
first <sigh>.

Signed-off-by: Yann Dirson <yann.dirson@vates.fr>

Merge branch 'scheduled-freebsd' into 'main'

scheduled ci jobs for FreeBSD

See merge request xen-project/xen-guest-agent!47

ci: add a scheduled job for checking freebsd build

Signed-off-by: Yann Dirson <yann.dirson@vates.fr>

ci: resync un/scheduled Linux job definitions

Signed-off-by: Yann Dirson <yann.dirson@vates.fr>

Merge branch 'ci-fix-scheduled' into 'main'

ci/deb: don't attempt to run deploy job when scheduled

See merge request xen-project/xen-guest-agent!46

ci/deb: don't attempt to run deploy job when scheduled

This caused a "yaml error" since dependencies were already disabled in
scheduled jobs.

Signed-off-by: Yann Dirson <yann.dirson@vates.fr>

Merge branch 'post-release-0.3.0' into 'main'

Bump version to 0.4.0-dev

See merge request xen-project/xen-guest-agent!45

Bump version to 0.4.0-dev

Signed-off-by: Yann Dirson <yann.dirson@vates.fr>

Merge branch 'release-0.3.0' into 'main'

Release v0.3.0

See merge request xen-project/xen-guest-agent!44

Release version 0.3.0

Textual clarification and fixes

Signed-off-by: Yann Dirson <yann.dirson@vates.fr>

Merge branch 'deb-file-index' into 'main'

ci/deb: provide uncompressed Contents file

See merge request xen-project/xen-guest-agent!43

ci/deb: provide uncompressed Contents file

The uncompressed *checksum* is needed.

Signed-off-by: Yann Dirson <yann.dirson@vates.fr>

Merge branch 'ci-package-repo' into 'main'

ci/pkg: fix handing of git-describe version for deb and rpm

See merge request xen-project/xen-guest-agent!42

ci/rpm: replace - with + in git-describe version

Signed-off-by: Yann Dirson <yann.dirson@vates.fr>

ci/deb: don't attempt package renaming if version did not need mangling

Signed-off-by: Yann Dirson <yann.dirson@vates.fr>

Merge branch 'ci-package-repo' into 'main'

ci: fix apk-add invocation

See merge request xen-project/xen-guest-agent!41

ci: fix apk-add invocation

Really fixes breakage from e651f8fae5c49bb1301a9b38fb047eba78d941dc.

So much for trusting search results on CLI syntax :/

Signed-off-by: Yann Dirson <yann.dirson@vates.fr>

Merge branch 'ci-package-repo' into 'main'

ci: make sure env-setup job has git available when needed

See merge request xen-project/xen-guest-agent!40

ci: make sure env-setup job has git available when needed

Fixes breakage from e651f8fae5c49bb1301a9b38fb047eba78d941dc.

CI in PR kinda sucks when using main-specific actions :/

Signed-off-by: Yann Dirson <yann.dirson@vates.fr>

Merge branch 'ci-package-repo' into 'main'

Make package repo usable

See merge request xen-project/xen-guest-agent!39

ci: use git-describe to set version for main branch builds

This will ensure not all builds share the same version string, and let
package update mechanism do their job.

Signed-off-by: Yann Dirson <yann.dirson@vates.fr>

ci: rename VERSION to TILDEVERSION for less ambiguity

Signed-off-by: Yann Dirson <yann.dirson@vates.fr>

ci: set VERSION and UPSTREAMVERSION once in env-setup

Similarly to UPDATE_CHANNEL, we want a more complex heuristic to set
it in the end.

Signed-off-by: Yann Dirson <yann.dirson@vates.fr>

ci: use "release" and "testing" update channels for tags

Signed-off-by: Yann Dirson <yann.dirson@vates.fr>

ci: setup packaging update channel once

Prepares introduction of more complex choice of update channel.

Signed-off-by: Yann Dirson <yann.dirson@vates.fr>

ci: activate pipelines for tags

Those are intended to produce the release artifacts.

Signed-off-by: Yann Dirson <yann.dirson@vates.fr>

ci/deb: remove unused dotenv report

Leftover from abandoned work.

Signed-off-by: Yann Dirson <yann.dirson@vates.fr>

ci: disable curl progress meter for more readability

Signed-off-by: Yann Dirson <yann.dirson@vates.fr>

Merge branch 'cargo-lock' into 'main'

bump dependencies

See merge request xen-project/xen-guest-agent!38

Cargo.lock: update outdated deps

Signed-off-by: Yann Dirson <yann.dirson@vates.fr>

Merge branch 'more-ci-artifacts' into 'main'

CI source artifact, package repositories

See merge request xen-project/xen-guest-agent!37

ci/deb: post-process Packages to use branch-name as component

This avoids the problem seen with HTTP 400 errors.

The sources.list entry for latest dev snapshot will be:

 deb [trusted=yes] https://gitlab.com/api/v4/projects/xen-project%252Fxen-guest-agent/packages/generic/deb-amd64/ main/

Signed-off-by: Yann Dirson <yann.dirson@vates.fr>

ci/deb: publish APT repository as a gitlab "generic package"

This ought to be used as sources.list entry to access the repo, but
for some reason `apt-get update` gets HTTP 400 on Packages file,
whereas curl fetches it all right.

 deb [trusted=yes] https://gitlab.com/api/v4/projects/xen-project%252Fxen-guest-agent/packages/generic/deb-amd64/ci/ ./

Signed-off-by: Yann Dirson <yann.dirson@vates.fr>

ci/deb: let deb job generate files for an APT repository

For some reason, while we can fetch the artifact files with curl,
apt-get gets told they do not exist.

Signed-off-by: Yann Dirson <yann.dirson@vates.fr>

ci: add source-tarball generation

Signed-off-by: Yann Dirson <yann.dirson@vates.fr>

ci: use "pkg-" prefix for packaging jobs

Signed-off-by: Yann Dirson <yann.dirson@vates.fr>

build(deps): bump github/codeql-action from 2 to 3

Bumps [github/codeql-action](https://github.com/github/codeql-action) from 2 to 3.
- [Release notes](https://github.com/github/codeql-action/releases)
- [Changelog](https://github.com/github/codeql-action/blob/main/CHANGELOG.md)
- [Commits](https://github.com/github/codeql-action/compare/v2...v3)

---
updated-dependencies:
- dependency-name: github/codeql-action
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>

ci/rpm: avoid confusing nesting of quotation marks

Although valid shell syntax, this confuses some editors.

Signed-off-by: Yann Dirson <yann.dirson@vates.fr>

doc/release: acknowledge that CI builds are now there

Signed-off-by: Yann Dirson <yann.dirson@vates.fr>

Fix typo in --help

Signed-off-by: Yann Dirson <yann.dirson@vates.fr>

Merge branch 'netlink-packet-route-0.18' into 'main'

netlink: upgrade to netlink-packet-route 0.18

See merge request xen-project/xen-guest-agent!36

netlink: upgrade to netlink-packet-route 0.18

Merge branch 'cargo-lock' into 'main'

Cargo.lock: update outdated deps

See merge request xen-project/xen-guest-agent!35

Cargo.lock: update outdated deps

Signed-off-by: Yann Dirson <yann.dirson@vates.fr>

Include VIF bugfixes in relnotes

Signed-off-by: Yann Dirson <yann.dirson@vates.fr>

Merge branch 'relnotes' into 'main'

Introduce a file for release notes

See merge request xen-project/xen-guest-agent!31

Introduce a CHANGELOG.md file for release notes

Signed-off-by: Yann Dirson <yann.dirson@vates.fr>

Merge branch 'index-iface-by-number' into 'main'

Index network interfaces by their numeric index, not by their name

Closes #13

See merge request xen-project/xen-guest-agent!23

netlink: handle iface renaming

Signed-off-by: Yann Dirson <yann.dirson@vates.fr>

net: prepare interface cache for name changing

While name changes do not necessarily have external impact (eg.
xenstore_schema_std does not expose those names), they are necessary
on Linux to know whether an iface is a VIF.  This requires
NetInterface objects in the cache to be mutable.

Signed-off-by: Yann Dirson <yann.dirson@vates.fr>

netlink: properly behave if link event has no MAC

Should never happen, but not a reason to have bad code.

And we can do that without worrying, because now we have separate Add/RmIface
events.

Signed-off-by: Yann Dirson <yann.dirson@vates.fr>

net: separate add/rm of interface and add/rm of MAC address

The current code would do nothing on interface removal, as we only
emit a `RmMac` event, with which xenstore_schema_std does nothing (we did
not add them in the first place).

An explicit `RmIface` allows for the removal of the full iface tree in
Xenstore, since there is no need to expose a mostly-empty tree.

Signed-off-by: Yann Dirson <yann.dirson@vates.fr>

netlink: prepare to generate more NetEvent's from a RtnlMessage

This will allow to separate handling of interfaces and MAC addresses.

Signed-off-by: Yann Dirson <yann.dirson@vates.fr>

Drop unix_helper

We finally don't have a need for separately querying for an interface name
from its index, since we don't need to fetch this information every time a
packet arrives, netlink already gives us everything we need.

Signed-off-by: Yann Dirson <yann.dirson@vates.fr>

pnet: keep a note on InterfaceInfo::name

Storage of interface name in pnet::InterfaceInfo would seem redundant
now, but we still need it stored temporarily, and this still looks
like the best place to do so.

Signed-off-by: Yann Dirson <yann.dirson@vates.fr>

net: keep a cache of network interfaces

Collects in a central place a map of iface indices, their names, and their
toolstack interface if any.

Finally replaces add_vif_info() calls with get_toolstack_interface()
called once to populate the cache.

Finally the RmIp events can be applied when an interface disappears, which
essentially fixes #13.

Though this makes a new problem apparent: recreating the VIF sometimes
does not get caught on Linux, when the iface is created as eth0 and
then renamed by udev as enX0 in a subsequent Newlink message: since we
are now only attempting to identify the toolkit-interface once, if
when we try to access the /sys node for eth0 the renaming already
occured the toolkit-interface is frozen as None.

Signed-off-by: Yann Dirson <yann.dirson@vates.fr>

netlink: refactor private funcs as methods

While there is no use for `self` today in there, we're going to need to
refer to an iface_cache in next commits, which gets much simplified if we
never have to borrow across method boundaries.

Signed-off-by: Yann Dirson <yann.dirson@vates.fr>

pnet: reword network_view as current_addresses

Makes wording consistent with addresses_cache, easier reading.

Signed-off-by: Yann Dirson <yann.dirson@vates.fr>

pnet: reword cache as addresses_cache

We're going to have an interface cache too, let's be clear.

Signed-off-by: Yann Dirson <yann.dirson@vates.fr>

pnet: reword NetworkView as AddressesState

Prepares renaming of "cache" member.

Signed-off-by: Yann Dirson <yann.dirson@vates.fr>

net: rework toolstack-interface detection

Detection of toolstack-interface (and not only VIF) does not have to
be done on every NetEvent, but only once.  To prepare for this,
introduce a new functional API that can be used instead of mutating a
NetEvent - this does not change call sites, which still make use of
the legacy API.

Signed-off-by: Yann Dirson <yann.dirson@vates.fr>

netlink: use name from NLA in Link events

We were always calling `if_indextoname()` to get the interface name when
receiving a New/DelLink message.  Just use the data we already received
instead.

On NewLink messages we just received the interface name, so we don't need
to make another query to get the information; on DelLink the sycall cannot
even give us this information any more.

Note at this point we still use "" to denote the lack of a name - it
seems unlikely to happen there anyway, and will be made cleaner next.

Signed-off-by: Yann Dirson <yann.dirson@vates.fr>

xenstore_schema_rfc: propose using iface number as key

This is more stable than names, and requires less bookkeeping in the
agent.

Signed-off-by: Yann Dirson <yann.dirson@vates.fr>

xenstore_schema_std: index the iface address cache with OS index

Iface name is not always part of a Netlink message, as opposed to the
OS index of the interface, and looking up the name following reception of
an interface removal notification just does not work.

All supported and prospective platforms seem to have a concept of
interface OS index, so this should cause no problem and be more efficient.

Signed-off-by: Yann Dirson <yann.dirson@vates.fr>

pnet: index NetworkView by interface index, not name

Current structure made us fill NetInterface with a dummy interface
number, and we want to use interface number in xenstore_schema_std to
fix #13.

Iface name becomes part of the associated data, bundle it into
InterfaceInfo for now (it will get redundant with introduction of a more
global interface cache).

Signed-off-by: Yann Dirson <yann.dirson@vates.fr>

pnet: use a NetworkView type alias to reduce redundancy

Signed-off-by: Yann Dirson <yann.dirson@vates.fr>

Merge branch 'cargo-lock' into 'main'

Cargo.lock: update outdated deps

See merge request xen-project/xen-guest-agent!34

Cargo.lock: update outdated deps

Signed-off-by: Yann Dirson <yann.dirson@vates.fr>

Merge branch 'ci-everycommit-catch-failures' into 'main'

ci: fix everycommit regression

See merge request xen-project/xen-guest-agent!29

ci: make sure collapsed-section footer is always printed

In case of error the lack of footer would fool gitlab log display into
thinking error summary from git-rebase was part of the collapsible section,
which is confusing to the reader.

Signed-off-by: Yann Dirson <yann.dirson@vates.fr>

ci: fix handling of build errors

When using the "! command" shell syntax, $? returns the status of the
negated command, so the script was reporting success in all cases.

Just avoid using that syntax.

#TestsNeedTesting

Signed-off-by: Yann Dirson <yann.dirson@vates.fr>

ci: use stderr not stdout in build wrapper

The "Executing ..." lines from gitlab-ci occasionally got captured before
the footer of previous collapsed section, so hidden when said section is
collapsed.

Signed-off-by: Yann Dirson <yann.dirson@vates.fr>

Merge branch 'general-improvements-from-mr23' into 'main'

General improvements from !23

See merge request xen-project/xen-guest-agent!28

pnet: use or_insert_with to avoid useless allocations in nominal path

Signed-off-by: Yann Dirson <yann.dirson@vates.fr>

xenstore_schema_std: refuse publish_netevent calls with no toolstack iface

This is already filtered at a more general level, in main.

Signed-off-by: Yann Dirson <yann.dirson@vates.fr>

xenstore_schema_std: more idiomatic String to &str conversion

Signed-off-by: Yann Dirson <yann.dirson@vates.fr>

publisher: let compiler tell us of missing enum values

Signed-off-by: Yann Dirson <yann.dirson@vates.fr>

net: add some debug-level traces

Should help digging precisely why a given event gets dropped.

Signed-off-by: Yann Dirson <yann.dirson@vates.fr>

netlink: remove redundant traces

* separately logging header+nlas just duplicates with less info the
  full message log
* the full message itself is already available from netlink crate, as now
  described in the README

Signed-off-by: Yann Dirson <yann.dirson@vates.fr>

README: update usage doc to include recently-added logging

Signed-off-by: Yann Dirson <yann.dirson@vates.fr>

Merge branch 'ci-everycommit-nofail-on-wip' into 'main'

ci: don't fail an `everycommit` pipeline when the failure is on a WIP commit

See merge request xen-project/xen-guest-agent!27

ci: don't fail the pipeline when a WIP commit does not build

ci: avoid useless "collapsed" flag on "section_end"

Signed-off-by: Yann Dirson <yann.dirson@vates.fr>

ci: use a dedicated script to isolate complexity of running commands

We're making the build logic more complicated, better have a real
script file to read it in good conditions.

Will also less the log look better, showing build args instead of printf
before the collapsed section.

Signed-off-by: Yann Dirson <yann.dirson@vates.fr>