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>

Merge branch 'cargo-lock' into 'main'

Cargo.lock: update outdated deps

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

Cargo.lock: update outdated deps

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

Merge branch 'ci-everycommit' into 'main'

ci: in merge requests, build every commit (on Linux only)

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

ci: let everycommit before_script be also timed and collapsed-by-default

Consistency, nitpicking, and playing with my new toy ;)

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

ci: in merge requests, build every commit (on Linux only)

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

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

ci: mark packaging jobs as not-scheduled, to fix scheduled jobs

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

ci: mark packaging jobs as not-scheduled, to fix scheduled jobs

Without it, scheduled jobs want to run those jobs, but cannot since their
dependencies are exluded as not-scheduled.

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

Merge branch 'binary-packaging' into 'main'

Binary packaging

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

ci: get version for RPM package from Cargo.toml

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

ci: build RPM packages

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

ci: get version for DEB package from Cargo.toml

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

ci: build DEB packages

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

Initial DEB packaging using prebuilt binaries

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

Initial RPM packaging using prebuilt binaries

This leaves a FIXME note to update later when the article gets
published, but nothing to block this branch.

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

Merge branch 'cli-logging' into 'main'

Control logging from command line

Closes #11

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

cli: add --loglevel control (fixes: #11)

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

cli: add --stderr flag to help debugging (part of #11)

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

env_logger: set default log level to "info", consistently with syslog

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

Improve description of release process

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

Merge branch 'cargo-lock' into 'main'

Cargo.lock update

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

Cargo.lock: update outdated deps

Includes rolling back from yanked version of "cc" crate.

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

Cargo.lock: sync crate version

5978771c72d02c109e304108a9df2828dd69a18f should have included this.

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

Merge branch 'ci-freebsd' into 'main'

Cross-build support for FreeBSD binaries through Docker

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

ci: cross-build FreeBSD 12 binary

This uses cross-rs container image as a base, for its
already-functional toolchain.

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

ci: use our script to fetch freebsd packages

Apparently much more reliable.

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

Cross-build support for FreeBSD binaries through Docker

This requires a bleeding-edge cross-rs and a few workarounds.

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

Merge branch 'ci' into 'main'

CI improvements and release artifacts

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

ci: add a job for release build

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

ci: add flags variable, be more verbose

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

ci: build on a Debian 10 image

This is the oldest supported Debian release, we'll use it to generate
binaries with widest compatibility.

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

Fix dev version

"0.2.0-dev" sorts before "0.2.0" in SemVer.

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

Document release process

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

Include a copy of the AGPL license.

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

Cargo.lock: update outdated deps

Merge branch 'xenstore-0.5' into 'main'

Cargo.toml: allow xenstore-rs 0.5

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

Cargo.toml: allow xenstore-rs 0.5

Retain compatibility with 0.4, but deactivate all features, since we are
not making any use of async_watch.

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

Cargo.lock: update outdated deps

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

Merge branch 'fix-formatting-errors' into 'main'

Fix error messages not getting formatted

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

Fix error messages not getting formatted

Errors that never happened, bugs that did not get caught, including
messages that did not get updated accross refactorings, and types not
implementing std::fmt::Debug.

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

Merge branch 'freebsd-memory' into 'main'

Add a custom memory collector for FreeBSD (#7)

Closes #7

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

Add a custom memory collector for FreeBSD (fixes: #7)

This collects metrics through sysctl, and compute available memory as
the FreeBSD port of xe-guest-utilities (shell-based) 6.0.2 did.

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

Propagate errors from get_available_kb()

It took me some time to identify a proper syntax to propagate the
error from a match branch, now we can be more specific about what we
ignore.  Now log a warning once in the "Unsupported" case (when
fetching the total memory), and let other memory-collector errors
cause a fatal error.

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

README: small update of 'current state' section

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

Merge branch 'xenstore-0.4' into 'main'

Switch to new xenstore-rs 0.4, update outdated deps

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

Switch to new xenstore-rs 0.4

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 'unpin-xenstore-sys' into 'main'

Unpin xenstore-sys and update deps

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

Unpin xenstore-sys to allow latest 0.1.4

0.1.4 relaxes the build-dep on buildgen for compatibility with latest
versions.

Reverts fe26b6de06d5e9ffa0566658b1f64bccdc0515fe.

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 'logging' into 'main'

Proper logging using syslog on unix (closes:#4)

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

Send logs to syslog on unix, else to stdout (closes:#4)

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

Bump to xenstore-rs 0.3.4 to avoid buggy 0.3.3

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

Use log crate instead of println for user feedback

Merge branch 'pin-xenstore-sys' into 'main'

Explicit pinning of xenstore-sys

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

Cargo.lock: update outdated deps

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

Explicit pinning of xenstore-sys

Explicit reference to xenstore-sys, to allow to run `cargo update`
without bringing back older version of bindgen and its vulnerable
dependencies.

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

Merge branch 'xenstore-0.3.3' into 'main'

Switch to xenstore-rs official release (fixes #6)

Closes #6

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

Switch to xenstore-rs official release (fixes #6)

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

Merge branch 'cargo-lock' into 'main'

update outdated deps

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

Cargo.lock: update outdated deps

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

Bump version to 0.2.0-dev after release

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

Merge pull request #6 from ydirson/dependabot/cargo/rustix-0.38.19

build(deps): bump rustix from 0.38.14 to 0.38.19

build(deps): bump rustix from 0.38.14 to 0.38.19

Bumps [rustix](https://github.com/bytecodealliance/rustix) from 0.38.14 to 0.38.19.
- [Release notes](https://github.com/bytecodealliance/rustix/releases)
- [Commits](https://github.com/bytecodealliance/rustix/compare/v0.38.14...v0.38.19)

---
updated-dependencies:
- dependency-name: rustix
  dependency-type: indirect
...

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

Prepare for 0.2.0

xenstore_std: show more details about unparsed os version

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

Merge branch 'use-crate-version' into 'main'

xenstore schemas: use crate version from Cargo.toml

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

xenstore schemas: use crate version from Cargo.toml

* "std" schema will now advertise as "1.0.0-proto-0.1.0"
* "rfc" schema just spares a duplication of the version string

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