Remove IPv6-only on IPv6 only infra test #768
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Initially, those tests were skipped on main because of Cilium LB limitations. We now moved to Metallb for this and therefore can enable it everywhere.
|
seems like snap has some issues with IPv6-only LXD containers on 24.04. I'll investigate. |
bschimke95
changed the title
|
Turned out that the issue was not 24.04 but rather the ipv6 connectivity, which we cannot support on the runners: actions/runner-images#668 This only popped up on 24.04 because on the previous versions |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Initially, those tests were skipped on main because of Cilium LB limitations. We now moved to Metallb for this and therefore can enable it everywhere.
Note that we also remove the
test_ipv6_on_ipv6_only_infrabecause of limitations on the github runners: actions/runner-images#668