openstack-k8s-operators / openstack-operator

Meta Operator for OpenStack
https://openstack-k8s-operators.github.io/openstack-operator/
Apache License 2.0
27 stars 76 forks source link

[ovn-controller] Don't create ovn-controller if nicMappings empty #1070

Closed averdagu closed 5 days ago

averdagu commented 2 weeks ago

If nicMappings is empty on the ovn/ovn-controller section don't spawn the ovn-controller as it won't do anything.

Resolves: OSPRH-7463

averdagu commented 2 weeks ago

Functional tests should be fixed now, working on kuttl

softwarefactory-project-zuul[bot] commented 1 week ago

Build failed (check pipeline). Post recheck (without leading slash) to rerun all jobs. Make sure the failure cause has been resolved before you rerun jobs.

https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/0e2392c13d1f483390eef56e38e4d6f8

:heavy_check_mark: openstack-k8s-operators-content-provider SUCCESS in 2h 05m 33s :heavy_check_mark: podified-multinode-edpm-deployment-crc SUCCESS in 1h 12m 42s :x: cifmw-crc-podified-edpm-baremetal RETRY_LIMIT in 21m 00s :heavy_check_mark: openstack-operator-tempest-multinode SUCCESS in 1h 30m 41s

softwarefactory-project-zuul[bot] commented 1 week ago

Build failed (check pipeline). Post recheck (without leading slash) to rerun all jobs. Make sure the failure cause has been resolved before you rerun jobs.

https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/f42d2751c89b4141bf6830db4e957e92

:heavy_check_mark: openstack-k8s-operators-content-provider SUCCESS in 2h 00m 28s :heavy_check_mark: podified-multinode-edpm-deployment-crc SUCCESS in 1h 13m 23s :heavy_check_mark: cifmw-crc-podified-edpm-baremetal SUCCESS in 1h 25m 33s :x: openstack-operator-tempest-multinode FAILURE in 1h 07m 07s

softwarefactory-project-zuul[bot] commented 1 week ago

Build failed (check pipeline). Post recheck (without leading slash) to rerun all jobs. Make sure the failure cause has been resolved before you rerun jobs.

https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/b9ab338f13f64df895e2ffd52b58ad6f

:heavy_check_mark: openstack-k8s-operators-content-provider SUCCESS in 1h 45m 00s :heavy_check_mark: podified-multinode-edpm-deployment-crc SUCCESS in 1h 16m 31s :x: cifmw-crc-podified-edpm-baremetal FAILURE in 27m 37s :x: openstack-operator-tempest-multinode FAILURE in 1h 08m 55s

averdagu commented 1 week ago

recheck

averdagu commented 1 week ago

/retest

openshift-ci[bot] commented 1 week ago

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: averdagu, bshephar

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

The pull request process is described here

Needs approval from an approver in each of these files: - ~~[OWNERS](https://github.com/openstack-k8s-operators/openstack-operator/blob/main/OWNERS)~~ [bshephar] Approvers can indicate their approval by writing `/approve` in a comment Approvers can cancel approval by writing `/approve cancel` in a comment
booxter commented 5 days ago

/lgtm