Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Pre-release updates #441

Open
wants to merge 10 commits into
base: main
Choose a base branch
from
Open

Pre-release updates #441

wants to merge 10 commits into from

Conversation

atenart
Copy link
Contributor

@atenart atenart commented Nov 4, 2024

Including the upgrade to a newer version of libbpf/-sys/-rs now that the upstream fix is in the latest release.

Also update pyo3 to its latest version, which comes with multiple API changes.

@atenart atenart added the run-functional-tests Request functional tests to be run by CI label Nov 4, 2024
@atenart atenart added this to the v1.5 milestone Nov 4, 2024
@atenart atenart force-pushed the at/updates branch 6 times, most recently from ae12b77 to f658cf8 Compare November 28, 2024 17:01
Signed-off-by: Antoine Tenart <[email protected]>
'pyo3' is listed as an extra feature dependency to the 'python' one.
However retis-event does not have a 'pyo3' feature. It's not a way to
select the 'pyo3' crate either as this is done in the 'python-embed' and
'python-lib' features. The 'python' feature is just an underlying one
for shared behavior between the other more specialize Python features.

Remove it.

Signed-off-by: Antoine Tenart <[email protected]>
Signed-off-by: Antoine Tenart <[email protected]>
Signed-off-by: Antoine Tenart <[email protected]>
Lots of API changes as described here,
https://pyo3.rs/v0.23.0/migration.html

Signed-off-by: Antoine Tenart <[email protected]>
For now keep rust-pcap 1.3.

Signed-off-by: Antoine Tenart <[email protected]>
- Downgrade cargo-platform.
- Downgrade the Cargo.lock version.

Signed-off-by: Antoine Tenart <[email protected]>
Data inside complex enums can't be directly accessed in the Python
representation of the events. The only way was to use the dict
representation, which prevents from using the event fields directly.
This made the enum fields to stand out, including the whole OvS event.

Instead do a little dance with serde and allow in Python things like:

  print(e['skb'].ip.version.v6.flow_label)

This does not change the JSON representation.

Closes #437.

Suggested-by: Adrian Moreno <[email protected]>
Signed-off-by: Antoine Tenart <[email protected]>
A new macro, BINDGEN_DEF, is introduced to avoid having to define those
constants twice. It provides a transparent way to define a constant
value which can be shared between eBPF and Rust through bindgen.

Signed-off-by: Antoine Tenart <[email protected]>
Signed-off-by: Antoine Tenart <[email protected]>
@atenart atenart changed the title [WIP] Pre-release updates Pre-release updates Dec 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
run-functional-tests Request functional tests to be run by CI
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant