Skip to content

Conversation

joamaki
Copy link
Contributor

@joamaki joamaki commented May 23, 2025

Backport of #39679

The pkg/loadbalancer/experimental and pkg/ciliumenvoyconfig/exp_test.go tests were marked to be skipped as these older versions had some flakes in them and they're not relevant at all to be run in the v1.17 branch.


This pulls in the new Hive version that has a safer API for jobs: cilium/hive#51.

Code is fixed as both Hive and StateDB had breaking API changes.

@joamaki joamaki added the backport/1.17 This PR represents a backport for Cilium 1.17.x of a PR that was merged to main. label May 23, 2025
@joamaki
Copy link
Contributor Author

joamaki commented May 29, 2025

/test

[ upstream commit a719911 ]
[ backporter note: disabled the experimental tests as they're not relevant to
  run for v1.17 and these old versions were flaky ]

This pulls in the new Hive version that has a safer API for jobs:
cilium/hive#51.

Code is fixed as both Hive and StateDB had breaking API changes.

Signed-off-by: Jussi Maki <jussi@isovalent.com>
@joamaki joamaki force-pushed the pr/joamaki/v1.17-bump-hive-statedb branch from e82f367 to f449216 Compare June 2, 2025 08:52
@joamaki
Copy link
Contributor Author

joamaki commented Jun 2, 2025

/test

@joamaki joamaki marked this pull request as ready for review June 4, 2025 09:09
@joamaki joamaki requested a review from a team as a code owner June 4, 2025 09:09
@maintainer-s-little-helper maintainer-s-little-helper bot added the ready-to-merge This PR has passed all tests and received consensus from code owners to merge. label Jun 4, 2025
@tklauser tklauser added this pull request to the merge queue Jun 4, 2025
Merged via the queue into cilium:v1.17 with commit 65f9b77 Jun 4, 2025
61 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport/1.17 This PR represents a backport for Cilium 1.17.x of a PR that was merged to main. ready-to-merge This PR has passed all tests and received consensus from code owners to merge.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants