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.

Sign up for GitHub

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

Jump to bottom

Enable OVN IC CI #2760

Closed
wants to merge 1 commit into from
Closed

Enable OVN IC CI #2760

wants to merge 1 commit into from

Conversation

dfarrell07
Copy link
Member

This will be removed once we're ready to default to OVN IC.

Relates-to: submariner-io/enhancements#209

@dfarrell07 dfarrell07 self-assigned this Aug 16, 2023
@submariner-bot
Copy link
Contributor

🤖 Created branch: z_pr2760/dfarrell07/ovn_ic_ci_pr
🚀 Full E2E won't run until the "ready-to-test" label is applied. I will add it automatically once the PR has 2 approvals, or you can add it manually.

@dfarrell07 dfarrell07 added the ready-to-test When a PR is ready for full E2E testing label Aug 16, 2023
@dfarrell07
Copy link
Member Author

There seems to be an issue.

OVN IC:

2023-08-16T19:10:21.9479323Z Cluster settings::
2023-08-16T19:10:21.9480183Z   broker - 'cluster1'
2023-08-16T19:10:21.9480498Z   clusters - 'cluster1' 'cluster2'
2023-08-16T19:10:21.9480779Z   cni - ([cluster2]="" [cluster1]="" )

Reference OVN:

2023-08-16T19:10:20.0918047Z Cluster settings::
2023-08-16T19:10:20.0918681Z   broker - 'cluster1'
2023-08-16T19:10:20.0918983Z   clusters - 'cluster1' 'cluster2'
2023-08-16T19:10:20.0919324Z   cni - ([cluster2]="ovn" [cluster1]="ovn" )

@dfarrell07 dfarrell07 marked this pull request as draft August 16, 2023 20:53
@mkolesnik
Copy link
Contributor

Seems the shipyard support is not loading the settings - https://github.com/submariner-io/submariner-operator/actions/runs/5882867770/job/15954300255?pr=2760#step:3:299

It should be cheeked why when using=ovn-ic it doesnt load them

@stale
Copy link

stale bot commented Sep 17, 2023

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the wontfix This will not be worked on label Sep 17, 2023
@stale stale bot removed the wontfix This will not be worked on label Sep 20, 2023
@tpantelis
Copy link
Contributor

Is this still relevant?

@github-actions GitHub Actions
Copy link

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further
activity occurs. Thank you for your contributions.

@github-actions github-actions bot added the stale label Nov 18, 2023
@dfarrell07
Enable OVN IC CI
3c2c079 
This will be removed once we're ready to default to OVN IC.

Relates-to: submariner-io/enhancements#209
Signed-off-by: Daniel Farrell <[email protected]>
@github-actions github-actions bot removed the stale label Nov 23, 2023
@github-actions GitHub Actions
Copy link

github-actions bot commented Dec 8, 2023

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further
activity occurs. Thank you for your contributions.

@github-actions github-actions bot added the stale label Dec 8, 2023
@github-actions github-actions bot closed this Dec 15, 2023
@submariner-bot
Copy link
Contributor

🤖 Closed branches: [z_pr2760/dfarrell07/ovn_ic_ci_pr]

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@aswinsuryan aswinsuryan aswinsuryan approved these changes

@mkolesnik mkolesnik Awaiting requested review from mkolesnik mkolesnik is a code owner

@Oats87 Oats87 Awaiting requested review from Oats87 Oats87 is a code owner

@skitt skitt Awaiting requested review from skitt skitt is a code owner

@sridhargaddam sridhargaddam Awaiting requested review from sridhargaddam sridhargaddam is a code owner

@tpantelis tpantelis Awaiting requested review from tpantelis tpantelis is a code owner

@vthapar vthapar Awaiting requested review from vthapar vthapar will be requested when the pull request is marked ready for review vthapar is a code owner

Assignees

@dfarrell07 dfarrell07

Labels
automation ready-to-test When a PR is ready for full E2E testing stale
Projects
No open projects
Submariner 0.17
Status: Done
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@dfarrell07 @submariner-bot @mkolesnik @tpantelis @aswinsuryan