-
Notifications
You must be signed in to change notification settings - Fork 2k
Create more PVs in the ovn-operator kuttl test #71153
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
base: master
Are you sure you want to change the base?
Conversation
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: slawqo 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:
Approvers can indicate their approval by writing |
|
@slawqo, Interacting with pj-rehearseComment: Once you are satisfied with the results of the rehearsals, comment: |
Synchronization of the OVN dbs with Neutron is pretty heavy and manual operation it is benefical to not delete those OVN databases when OVNDBCluster's PODs are deleted. To achieve that this patch removes `OwnerReference` attribute from the PVCs created for the OVNDBClusters so that they are not cleaned by the OpenShift's garbage collector and it also changes PersistentVolumeClaimRetentionPolicy when deleted and when scaled to the same `Retain` policy. Depends-On: openshift/release#71153 Closes: #OSPRH-19930 Signed-off-by: Slawek Kaplonski <[email protected]>
With [1] PVCs which are created for the OVN databases aren't cleaned automatically thus we should have more than 12 PVs created in the kuttl job so that each test will use different PVs and there will be no old data on the used PV. [1] openstack-k8s-operators/ovn-operator#511 Related: #OSPRH-19930 Signed-off-by: Slawek Kaplonski <[email protected]>
6ca6704 to
60293ef
Compare
|
/pj-rehearse ovn-operator-build-deploy-kuttl |
|
@slawqo: now processing your pj-rehearse request. Please allow up to 10 minutes for jobs to trigger or cancel. |
|
Change looks good to me, waiting for jobs to finish to give approve |
|
[REHEARSALNOTIFIER]
Interacting with pj-rehearseComment: Once you are satisfied with the results of the rehearsals, comment: |
|
@slawqo: job(s): ovn-operator-build-deploy-kuttl either don't exist or were not found to be affected, and cannot be rehearsed |
|
@slawqo: The following tests failed, say
Full PR test history. Your PR dashboard. Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here. |
With [1] PVCs which are created for the OVN databases aren't cleaned automatically thus we should have more than 12 PVs created in the kuttl job so that each test will use different PVs and there will be no old data on the used PV.
[1] openstack-k8s-operators/ovn-operator#511
Related: #OSPRH-19930