openstack-k8s-operators / edpm-ansible

External Dataplane Management Ansible Playbooks
https://openstack-k8s-operators.github.io/edpm-ansible/
Apache License 2.0
9 stars 65 forks source link

Skip removal of services related to swift storage #668

Closed jpodivin closed 3 months ago

jpodivin commented 3 months ago

Swift storage services are removed by a separate procedure later during adoption process. Therefore it is undesirable to remove them together with other services originally deployed by tripleO.

https://issues.redhat.com/browse/OSPRH-7432

openshift-ci[bot] commented 3 months ago

Skipping CI for Draft Pull Request. If you want CI signal for your change, please convert it to an actual PR. You can still manually trigger a test run with /test all

openshift-ci[bot] commented 3 months ago

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: jpodivin

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/edpm-ansible/blob/main/OWNERS)~~ [jpodivin] Approvers can indicate their approval by writing `/approve` in a comment Approvers can cancel approval by writing `/approve cancel` in a comment
jistr commented 3 months ago

/lgtm Thanks! I think having the literal skip list is probably better than trying to do multiple skip wildcard matches.

softwarefactory-project-zuul[bot] commented 3 months 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://review.rdoproject.org/zuul/buildset/379aa5a366d746c88c2aa6108aedb580

:heavy_check_mark: openstack-k8s-operators-content-provider SUCCESS in 2h 34m 38s :x: podified-multinode-edpm-deployment-crc FAILURE in 2h 17m 05s :heavy_check_mark: cifmw-crc-podified-edpm-baremetal SUCCESS in 1h 50m 44s :heavy_check_mark: edpm-ansible-molecule-edpm_bootstrap SUCCESS in 6m 28s :heavy_check_mark: edpm-ansible-molecule-edpm_podman SUCCESS in 5m 09s :heavy_check_mark: edpm-ansible-molecule-edpm_module_load SUCCESS in 4m 37s :heavy_check_mark: edpm-ansible-molecule-edpm_kernel SUCCESS in 9m 23s :heavy_check_mark: edpm-ansible-molecule-edpm_libvirt SUCCESS in 9m 26s :heavy_check_mark: edpm-ansible-molecule-edpm_nova SUCCESS in 9m 03s :heavy_check_mark: edpm-ansible-molecule-edpm_frr SUCCESS in 6m 17s :heavy_check_mark: edpm-ansible-molecule-edpm_iscsid SUCCESS in 4m 46s :heavy_check_mark: edpm-ansible-molecule-edpm_ovn_bgp_agent SUCCESS in 10m 13s :heavy_check_mark: edpm-ansible-molecule-edpm_ovs SUCCESS in 5m 39s :heavy_check_mark: edpm-ansible-molecule-edpm_tripleo_cleanup SUCCESS in 3m 58s

jpodivin commented 3 months ago

Ok...but, why? Can you update the commit message?

It's a draft. I'm not even sure it's going to work properly yet. But I will put the context in there when I am.

jpodivin commented 3 months ago

/recheck

jpodivin commented 3 months ago

recheck

softwarefactory-project-zuul[bot] commented 3 months ago

This change depends on a change that failed to merge.

Change https://github.com/openstack-k8s-operators/ci-framework/pull/1815 is needed.

jpodivin commented 3 months ago

retest

jistr commented 3 months ago

recheck

softwarefactory-project-zuul[bot] commented 3 months 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://review.rdoproject.org/zuul/buildset/eead798c07154d35a844ba3c5710722a

:heavy_check_mark: openstack-k8s-operators-content-provider SUCCESS in 2h 30m 20s :x: podified-multinode-edpm-deployment-crc FAILURE in 2h 10m 45s :x: cifmw-crc-podified-edpm-baremetal FAILURE in 2h 05m 16s :heavy_check_mark: edpm-ansible-molecule-edpm_bootstrap SUCCESS in 6m 18s :heavy_check_mark: edpm-ansible-molecule-edpm_podman SUCCESS in 5m 14s :heavy_check_mark: edpm-ansible-molecule-edpm_module_load SUCCESS in 4m 37s :heavy_check_mark: edpm-ansible-molecule-edpm_kernel SUCCESS in 9m 20s :heavy_check_mark: edpm-ansible-molecule-edpm_libvirt SUCCESS in 8m 26s :heavy_check_mark: edpm-ansible-molecule-edpm_nova SUCCESS in 8m 50s :heavy_check_mark: edpm-ansible-molecule-edpm_frr SUCCESS in 6m 24s :heavy_check_mark: edpm-ansible-molecule-edpm_iscsid SUCCESS in 4m 35s :heavy_check_mark: edpm-ansible-molecule-edpm_ovn_bgp_agent SUCCESS in 6m 46s :heavy_check_mark: edpm-ansible-molecule-edpm_ovs SUCCESS in 5m 03s :heavy_check_mark: edpm-ansible-molecule-edpm_tripleo_cleanup SUCCESS in 3m 57s

jpodivin commented 3 months ago

recheck

jistr commented 3 months ago

/lgtm