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

Add OVN IC CI #1338

Merged
merged 1 commit into from
Aug 15, 2023
Merged

Add OVN IC CI #1338

merged 1 commit into from
Aug 15, 2023

Conversation

dfarrell07
Copy link
Member

@submariner-bot
Copy link

🤖 Created branch: z_pr1338/dfarrell07/ovn_ic_ci_pr

@dfarrell07 dfarrell07 requested a review from aswinsuryan August 14, 2023 21:43
Copy link
Contributor

@aswinsuryan aswinsuryan left a comment

Choose a reason for hiding this comment

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

Worked fine, using=ovn-ic , Thanks

@nyechiel
Copy link
Member

@mkolesnik can you please review?

.github/workflows/testing.yml Outdated Show resolved Hide resolved
scripts/shared/lib/clusters_kind Outdated Show resolved Hide resolved
scripts/shared/lib/clusters_kind Outdated Show resolved Hide resolved
scripts/shared/lib/clusters_kind Outdated Show resolved Hide resolved
scripts/shared/lib/clusters_kind Show resolved Hide resolved
scripts/shared/lib/clusters_kind Show resolved Hide resolved
scripts/shared/lib/clusters_kind Outdated Show resolved Hide resolved
scripts/shared/lib/clusters_kind Show resolved Hide resolved
@dfarrell07 dfarrell07 force-pushed the ovn_ic_ci_pr branch 2 times, most recently from 685ec28 to 42ce623 Compare August 15, 2023 14:27
@nyechiel nyechiel requested a review from mkolesnik August 15, 2023 14:44
Fixes: submariner-io/enhancements#209
Co-authored-by: Mike Kolesnik <[email protected]>
Signed-off-by: Daniel Farrell <[email protected]>
Copy link
Collaborator

@mkolesnik mkolesnik left a comment

Choose a reason for hiding this comment

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

LGTM, but based on #1297 (comment) perhaps you want to always rebuild OVN images, at least until they fix it..

It doesn't have to be part of this patch but since you're already changing this area might as well..
If not, then it should probably be done either way after this gets merged.

@dfarrell07
Copy link
Member Author

Let's do it after we rebase the kind 0.20 bump on to this and test both the rebuilt image and the old image.

@dfarrell07 dfarrell07 merged commit 2fce546 into submariner-io:devel Aug 15, 2023
@submariner-bot
Copy link

🤖 Closed branches: [z_pr1338/dfarrell07/ovn_ic_ci_pr]

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
No open projects
Status: Done
Development

Successfully merging this pull request may close these issues.

Add CI job for OVN IC
6 participants