Merged
Conversation
fidencio
force-pushed
the
topic/tests-add-a-way-to-test
branch
from
2d1c2f2 to
8f2a495
Compare
fidencio
force-pushed
the
topic/tests-add-a-way-to-test
branch
from
8f2a495 to
31cf1c1
Compare
fidencio
force-pushed
the
topic/tests-add-a-way-to-test
branch
from
31cf1c1 to
9e49de7
Compare
fidencio
force-pushed
the
topic/tests-add-a-way-to-test
branch
8 times, most recently
from
d5e2348 to
2904667
Compare
fidencio
force-pushed
the
topic/tests-add-a-way-to-test
branch
14 times, most recently
from
63c33a3 to
2d656b3
Compare
fidencio
force-pushed
the
topic/tests-add-a-way-to-test
branch
from
2d656b3 to
fb19652
Compare
fidencio
force-pushed
the
topic/tests-add-a-way-to-test
branch
11 times, most recently
from
74df2bb to
cea33b9
Compare
fidencio
force-pushed
the
topic/tests-add-a-way-to-test
branch
from
cea33b9 to
0daf265
Compare
fidencio
force-pushed
the
topic/tests-add-a-way-to-test
branch
from
0daf265 to
53362dd
Compare
fidencio
force-pushed
the
topic/tests-add-a-way-to-test
branch
from
53362dd to
e897196
Compare
fidencio
force-pushed
the
topic/tests-add-a-way-to-test
branch
2 times, most recently
from
0abfa82 to
3d39fa7
Compare
Following the work I've done on kata-containers side, let's use what we added there ... but make those proper actions instead of just scripts. This will help us (and hopefully other repos as well) to test things out in all the supported k8s flavours we have interest in. Signed-off-by: Fabiano Fidêncio <ffidencio@nvidia.com>
Although not used yet, this will help us immensely to test development versions of Kata Containers, and catch issues on our integration as soon as possible. Let's add a new action to ensure we don't accidentaly add / push the CI specific dependency as part of the release. The action itself was heavily vibe coded using Cursor AI. Signed-off-by: Fabiano Fidêncio <ffidencio@nvidia.com>
Those actions have been vibe generated, although still heavily based on what we currently have on kata-containers / confidential-containers' oeprator. Signed-off-by: Fabiano Fidêncio <ffidencio@nvidia.com>
Let's make sure we test all the k8s flavours we're interested in, on every opened PR, and ensure no regressions are added. CRI-O is being skipped for now as kata-containers' helm chart doesn't support configuring CRI-O for TEE runtime classes. Hopefully Red Hat folks will be able to collaborate with such support at some point. Signed-off-by: Fabiano Fidêncio <ffidencio@nvidia.com>
fidencio
force-pushed
the
topic/tests-add-a-way-to-test
branch
from
3d39fa7 to
c73a87b
Compare
This file contains hidden or 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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
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.
This commit is huge, I know it, and I am sorry about it.
What we're doing here is basically:
Last but not least, we're adding a new action to ensure that our test dependency is never ever commited to our Chart.lock.
Majority of the tests were vibe generated using Cursor AI based on what I personally wrote for kata-containers.