Skip to content

Add ovs and ovn images - #58

Merged
openshift-merge-bot[bot] merged 1 commit into
openstack-k8s-operators:mainfrom
karelyatin:OSPRH-34494
Aug 17, 2026
Merged

Add ovs and ovn images#58
openshift-merge-bot[bot] merged 1 commit into
openstack-k8s-operators:mainfrom
karelyatin:OSPRH-34494

Conversation

@karelyatin

Copy link
Copy Markdown
Contributor

This PR adds below container images:-

  • ovn-base
  • ovn-sb-db-server
  • ovn-nb-db-server
  • ovn-northd
  • ovn-controller

Resolves: https://redhat.atlassian.net/browse/OSPRH-34494

@karelyatin

Copy link
Copy Markdown
Contributor Author

Validated built images in https://gateway-cloud-softwarefactory.apps.ocp.cloud.ci.centos.org/zuul/t/rdoproject.org/build/76844ee4b6a648ebad01f89fc37bec81
3 tempest test failures but those test and config specific

@amoralej

Copy link
Copy Markdown
Contributor

I'd prefer to get #71 in first which includes the changes in build.sh and rebase on top.

@amoralej

Copy link
Copy Markdown
Contributor

I'd prefer to get #71 in first which includes the changes in build.sh and rebase on top.

It will require to change the COPY commands i think as #71 uses the base project as build base for consistency with the rest of projects. I think that will help for portability of this content out of this repo (i.e. konflux environments)

@amoralej amoralej left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I'd like this to be rebased on #71

@slawqo slawqo left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

LGTM for me

@karelyatin

Copy link
Copy Markdown
Contributor Author

I'd like this to be rebased on #71

@amoralej #71 looks outdated now with recent merges. Can you get that cleaned/merged for rebasing this.
Additionally can you point me what exactly required for COPY commands, /me will handle that along with the rebase.

@amoralej

Copy link
Copy Markdown
Contributor

I'd like this to be rebased on #71

@amoralej #71 looks outdated now with recent merges. Can you get that cleaned/merged for rebasing this. Additionally can you point me what exactly required for COPY commands, /me will handle that along with the rebase.

Yes, I'm working on it. I'll let you know when done.

Comment thread containers/ovn/ovn-base/Containerfile Outdated
Comment thread containers/ovn/ovn-controller/Containerfile Outdated
Comment thread containers/ovn/ovn-nb-db-server/Containerfile Outdated
Comment thread containers/ovn/ovn-northd/Containerfile Outdated
Comment thread containers/ovn/ovn-sb-db-server/Containerfile Outdated
Comment thread build.sh Outdated
This PR adds below container images:-
- ovn-base
- ovn-sb-db-server
- ovn-nb-db-server
- ovn-northd
- ovn-controller

Resolves: https://redhat.atlassian.net/browse/OSPRH-34494
Signed-off-by: Yatin Karel <ykarel@redhat.com>
@openshift-ci openshift-ci Bot added the lgtm label Aug 17, 2026
@karelyatin
karelyatin requested a review from rebtoor August 17, 2026 12:30

@rebtoor rebtoor left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

/approve
/lgtm

@openshift-ci

openshift-ci Bot commented Aug 17, 2026

Copy link
Copy Markdown

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: rebtoor

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@openshift-merge-bot
openshift-merge-bot Bot merged commit 6286bd3 into openstack-k8s-operators:main Aug 17, 2026
6 checks passed
@karelyatin
karelyatin deleted the OSPRH-34494 branch August 17, 2026 13:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants