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: suffix ci k8s namespaces with "-ci" #1055

Merged
merged 2 commits into from
Dec 9, 2024

Conversation

3u13r
Copy link
Member

@3u13r 3u13r commented Dec 6, 2024

We invoke different e2e tests with the same nix command. This nix command already defines flags that are therefore shared between all current (and future) e2e tests. Let's make it easy to introduce new shared flags while maintaining the possibility that some tests define their own additional flags.

Also introduce a new namespace suffix flag.

@3u13r 3u13r added the no changelog PRs not listed in the release notes label Dec 6, 2024
@3u13r 3u13r force-pushed the euler/feat/ci/suffix-ci-k8s-namespace-with-ci branch 3 times, most recently from f96384f to b74cba6 Compare December 6, 2024 19:42
@3u13r
Copy link
Member Author

3u13r commented Dec 6, 2024

The e2e test aks runtime check seems to be broken on main: https://github.com/edgelesssys/contrast/actions/runs/12179300514/job/33971147711

@3u13r 3u13r force-pushed the euler/feat/ci/suffix-ci-k8s-namespace-with-ci branch from b74cba6 to 7866155 Compare December 6, 2024 20:04
@3u13r 3u13r marked this pull request as ready for review December 6, 2024 20:05
@3u13r 3u13r requested a review from burgerdev as a code owner December 6, 2024 20:05
@3u13r
Copy link
Member Author

3u13r commented Dec 6, 2024

In the regression test testregression-2a2a5cbe-ci/mysql-centos-5f9875b787-8blvs reports
failed to unpack usr/libexec/mysqld into /run/kata-containers/image/layers/sha256_58bde37520d48c550fe5dba6a67740d181f8140ecdf387371bb92a7f6fae126f/usr/libexec/mysqld
No space left on device (os error 28)

e2e/internal/contrasttest/contrasttest.go Outdated Show resolved Hide resolved
@@ -79,7 +79,8 @@ jobs:
--image-replacements workspace/just.containerlookup \
--namespace-file workspace/e2e.namespace \
--platform ${{ inputs.platform }} \
--skip-undeploy="${{ inputs.skip-undeploy && 'true' || 'false' }}"
--skip-undeploy="${{ inputs.skip-undeploy && 'true' || 'false' }}" \
Copy link
Contributor

Choose a reason for hiding this comment

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

fyi, @miampf is removing --skip-undeploy in #1046.

@3u13r 3u13r force-pushed the euler/feat/ci/suffix-ci-k8s-namespace-with-ci branch from 7866155 to 20453a5 Compare December 9, 2024 11:45
@3u13r 3u13r merged commit 7386451 into main Dec 9, 2024
15 of 19 checks passed
@3u13r 3u13r deleted the euler/feat/ci/suffix-ci-k8s-namespace-with-ci branch December 9, 2024 13:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
no changelog PRs not listed in the release notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants