Merge branch 'cargo-lock' into 'main'

Cargo.lock: update outdated deps

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

Cargo: relax dep on env_logger to accept 0.11

We don't need anything beyond the basic API, unlikely to break even on
new "major" releases.

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

Cargo.lock: update outdated deps

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

Merge branch 'static-bugfixes' into 'main'

ci: add missing -L flag

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

ci: add missing -L flag

For some reason neither cross-rs nor pkg-config tells rustc that cross
libs do live in the sysroot.  Only a problem for static linking, since
that's the only use-case we have besides dlopen.

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!63

Cargo.lock: update outdated deps

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

freebsd/netlink: use new libc branch following 0.2

Update of crate rustix now wants libc 0.2.152, which lives on the
libc-0.2 branch while the main libc branch sails towards 0.3.  To be
able to continue building until the stuff we need gets released as
0.3, without causing our PRs to become unmergeable, a new
freebsd-netlink-0.2 was necessary.

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

Merge branch 'ci-freebsd-netlink' into 'main'

ci: add a build job for freebsd13 with netlink support

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

Drop old devscripts/freebsd-fetch-package.sh

Since 88d4faf7764dc581700c5192b76cf85a450add95 and
7caebf0cd73e5bb6c0ddc0d111a377bb44d49186 there is not reason to keep
this script any more.

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

ci: add a build job for freebsd13 with netlink support

Still waiting on the libc PR to land, so we need unofficial branches for
now.

We don't make it a release job because of the unofficial status of those
branches, but we still provide the executable as an artifact for
convenience.

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

Merge branch 'ci-code-checks' into 'main'

Add CI jobs for code checks

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

Apply a few select formatting fixes from fmt

Most of the rest mostly hampers readability.

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

ci: new check job for clippy

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

ci: separate test-builds from release-builds, as being "just checks"

We'll want to have more checks, and they will naturally fit in that new
stage, rather than delaying test builds with a separate "check" stage.

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

Apply suggestions from clippy for code simplification

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

ci: make sure to update freebsd package list before trying to install

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

Merge branch 'doc-cleanup-vif' into 'main'

Document removal of stale network info

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

Document removal of stale network info

Merge branch 'gtn-cleanup-vif' into 'main'

Removing all vif entries from xenstore at startup #12

Closes #12

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

Removing all vif entries from xenstore at startup

When interfaces reported by Xenstore are suppressed while the service is
not running, these data remain. To avoid this we start by removing all
entries from previous agent. Entries that are still valid will be
repopulated immediatly.

Fixes: #12

Merge branch 'freebsd-12-eol' into 'main'

ci: restore FreeBSD builds, now with 13.2

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

Revert "ci: ignore FreeBSD build failures until new container is ready"

Now that the FreeBSD jbs can run again...

This reverts commit 9619b77b712e35afb3ed752b9c560bbca08da294.

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

ci: use official package-installing scripts for freebsd

Note that devscripts/freebsd-fetch-package.sh was written for the CI,
and never worked from Cross.toml.  Switch everything to the
slightly-improved-and-still-undocumented version of the scripts
shipped in the container itself.

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

ci: switch away from EOL'd FreeBSD 12.4 to 13.2

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!57

Cargo.lock: update outdated deps

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

Merge branch 'readme-update' into 'main'

Update README with what's next

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

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>

Update README with what's next

Signed-off-by: Olivier Lambert <olivier.lambert@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>