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

CI: Add workflow for v1.13 #38

Merged
merged 2 commits into from
Mar 13, 2024
Merged

CI: Add workflow for v1.13 #38

merged 2 commits into from
Mar 13, 2024

Conversation

tommyp1ckles
Copy link
Collaborator

No description provided.

@tommyp1ckles tommyp1ckles changed the title [wip] add workflow for v1.13 CI: Add workflow for v1.13 Mar 11, 2024
v1.14 had some breaking changes to configuration, namely simplification of the kpr configuration and changes for compatibility for l7 and encryption.

To make things easier, this introduces v1.13 as a seperate workflow to avoid having special case logic between v1.13 and more recent versions.

Upon releasing v1.16 we can remove this workflow and just use conformance.yaml.

Signed-off-by: Tom Hadlaw <[email protected]>
Also, adds missing flag to enable sysdumps upon failure.

Signed-off-by: Tom Hadlaw <[email protected]>
@tommyp1ckles tommyp1ckles marked this pull request as ready for review March 12, 2024 01:02
Copy link
Collaborator

@PhilipSchmid PhilipSchmid left a comment

Choose a reason for hiding this comment

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

LGTM. Thanks!

Just one question: Do you think we should build a retry around the cilium-cli install command? -> https://github.com/isovalent/terraform-aws-talos/actions/runs/8237765455/job/22543525926?pr=38#step:7:30

@tommyp1ckles
Copy link
Collaborator Author

@PhilipSchmid yeah good idea, there's some issues with the CI right now due to cleanups failing so ill address and such issues in a seperate PR.

@tommyp1ckles tommyp1ckles merged commit 3755bb5 into main Mar 13, 2024
64 of 65 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants