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

Dependency updates & CI fixes #98

Merged
merged 18 commits into from
Sep 1, 2023

Conversation

shaneutt
Copy link
Member

@shaneutt shaneutt commented Aug 30, 2023

Some of our dependencies got a bit out of date, and included some breaking changes. This updates all controlplane and dataplane dependencies, makes necessary adjustments, and gets CI working right again.

Also includes:

  • some general improvements to the Makefile
  • lint fixes

@shaneutt shaneutt force-pushed the shaneutt/ci-fixes-20923-08-30 branch from 2d78ffb to 3baf4f6 Compare August 30, 2023 19:42
@shaneutt shaneutt force-pushed the shaneutt/ci-fixes-20923-08-30 branch 2 times, most recently from 1507b75 to 12b18ca Compare August 31, 2023 15:10
@astoycos astoycos force-pushed the shaneutt/ci-fixes-20923-08-30 branch 2 times, most recently from e20b67b to c94b66c Compare August 31, 2023 17:54
bump deps so build-ebpf doesn't fail + bump all deps that had
more recent versions, like tonic clap etc.

make sure to run `cargo install bpf-linker` locally to ensure
that its up to date.

use the `take_map` aya API to take ownership of the CONNTRACK map before
passing it to our "api-server".

Signed-off-by: astoycos <[email protected]>
These must be some tonic automatic updates with
the new dep.

Signed-off-by: astoycos <[email protected]>
@astoycos astoycos force-pushed the shaneutt/ci-fixes-20923-08-30 branch 2 times, most recently from b81c31e to 66af020 Compare August 31, 2023 18:00
@shaneutt shaneutt marked this pull request as ready for review August 31, 2023 19:27
@shaneutt shaneutt enabled auto-merge (rebase) August 31, 2023 19:27
Copy link
Member

@astoycos astoycos left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LTGM

@shaneutt shaneutt merged commit ab4411e into kubernetes-sigs:main Sep 1, 2023
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants