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

Wait for Tetragon's images exist before run test #2401

Merged
merged 1 commit into from
May 5, 2024
Merged

Wait for Tetragon's images exist before run test #2401

merged 1 commit into from
May 5, 2024

Conversation

Trung-DV
Copy link
Contributor

@Trung-DV Trung-DV commented May 2, 2024

Currently, our six runners have to wait for Tetragon's images to be available before pulling. Therefore, I am moving this step to the preparation job to save time.

image

@Trung-DV Trung-DV requested review from willfindlay and a team as code owners May 2, 2024 10:13
@jrfastab jrfastab added release-note/ci This PR makes changes to the CI. release-note/minor This PR introduces a minor user-visible change labels May 3, 2024
@jrfastab
Copy link
Contributor

jrfastab commented May 3, 2024

@Trung-DV looks like a nice win to me please resolve the conflicts thanks.

Copy link
Contributor

@willfindlay willfindlay left a comment

Choose a reason for hiding this comment

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

Looks great, thank you!

@willfindlay
Copy link
Contributor

@Trung-DV just needs a rebase

@Trung-DV Trung-DV requested a review from willfindlay May 4, 2024 01:33
@jrfastab jrfastab merged commit 5fbcdae into cilium:main May 5, 2024
37 checks passed
@jrfastab
Copy link
Contributor

jrfastab commented May 5, 2024

thanks.

@Trung-DV Trung-DV deleted the pr/Trung-DV/wait_tetragon_images branch May 5, 2024 05:32
@kkourt kkourt removed the release-note/minor This PR introduces a minor user-visible change label Sep 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release-note/ci This PR makes changes to the CI.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants