-
Notifications
You must be signed in to change notification settings - Fork 82
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
Update openstack-k8s-operators #732
Conversation
Build failed (check pipeline). Post https://review.rdoproject.org/zuul/buildset/ac6ec47c1e544c758e62e43986c65c00 ✔️ openstack-k8s-operators-content-provider SUCCESS in 2h 33m 48s |
8015480
to
81e8378
Compare
Merge Failed. This change or one of its cross-repo dependencies was unable to be automatically merged with the current state of its repository. Please rebase the change and upload a new patchset. |
/test openstack-operator-build-deploy-kuttl keystone-db-sync failed, no logs |
recheck |
/test openstack-operator-build-deploy-kuttl |
Build failed (check pipeline). Post https://review.rdoproject.org/zuul/buildset/66cbbba3b45c4de2bfd057d6b77004c1 ✔️ openstack-k8s-operators-content-provider SUCCESS in 3h 19m 27s |
recheck |
/test openstack-operator-build-deploy-kuttl pod pending for more than 1h0m0s: containers have not started in 1h0m0.003549592s: test: \n* Container test is not ready with reason ImagePullBackOff and message Back-off pulling |
It's required with tls enabled, [1] already enabled tls by default so need to include the install-certs service before the services requiring the certs. The issue should be visible once dataplane-operator is updated[2] [1] openstack-k8s-operators/dataplane-operator#754 [2] openstack-k8s-operators/openstack-operator#732
It's required with tls enabled, [1] already enabled tls by default so need to include the install-certs service before the services requiring the certs. The issue should be visible once dataplane-operator is updated[2] or tlsEnabled flag is explicitly set in the nodeset. [1] openstack-k8s-operators/dataplane-operator#754 [2] openstack-k8s-operators/openstack-operator#732
Build failed (check pipeline). Post https://review.rdoproject.org/zuul/buildset/fcf149901e5c4dec83cf49d03a0770a1 ✔️ openstack-k8s-operators-content-provider SUCCESS in 4h 45m 39s |
It's required with tls enabled, [1] already enabled tls by default so need to include the install-certs service before the services requiring the certs. The issue should be visible once dataplane-operator is updated[2] or tlsEnabled flag is explicitly set in the nodeset. Also update nova-custom-ceph dataplane service to include missing tls params. [1] openstack-k8s-operators/dataplane-operator#754 [2] openstack-k8s-operators/openstack-operator#732
It's required with tls enabled, [1] already enabled tls by default so need to include the missing certs. The issue should be visible once dataplane-operator is updated[2] or tlsEnabled flag is explicitly set in the nodeset. [1] openstack-k8s-operators/dataplane-operator#754 [2] openstack-k8s-operators/openstack-operator#732
81e8378
to
632aed8
Compare
Build failed (check pipeline). Post https://review.rdoproject.org/zuul/buildset/075a7afbd1614254aafb2400b2ad7c2f ✔️ openstack-k8s-operators-content-provider SUCCESS in 1h 56m 23s |
It's required with tls enabled, [1] already enabled tls by default so need to include the install-certs service before the services requiring the certs. The issue should be visible once dataplane-operator is updated[2] or tlsEnabled flag is explicitly set in the nodeset. Also update nova-custom-ceph dataplane service to include missing tls params. [1] openstack-k8s-operators/dataplane-operator#754 [2] openstack-k8s-operators/openstack-operator#732
7c9319f
to
eaaf85f
Compare
Build failed (check pipeline). Post https://review.rdoproject.org/zuul/buildset/c11b55c23d694e648cdbade14f25468e ✔️ openstack-k8s-operators-content-provider SUCCESS in 2h 36m 45s |
It's required with tls enabled, [1] already enabled tls by default so need to include the missing certs. The issue should be visible once dataplane-operator is updated[2] or tlsEnabled flag is explicitly set in the nodeset. [1] openstack-k8s-operators/dataplane-operator#754 [2] openstack-k8s-operators/openstack-operator#732
The live migration test failures are real, fails as triggered by tls-e enable with dataplane-operator:- |
eaaf85f
to
cefcbb5
Compare
dropped dataplane-operator bump from this PR, can be fixed/updated in seperate PR |
/test openstack-operator-build-deploy-kuttl Unable to establish connection to https://keystone-public-openstack.apps.oko-07.oooci.ccitredhat.com/v3/services?: |
It's required with tls enabled, [1] already enabled tls by default so need to include the install-certs service before the services requiring the certs. The issue should be visible once dataplane-operator is updated[2] or tlsEnabled flag is explicitly set in the nodeset. Also update nova-custom-ceph dataplane service to include missing tls params. [1] openstack-k8s-operators/dataplane-operator#754 [2] openstack-k8s-operators/openstack-operator#732
Yes, openstack-k8s-operators/edpm-ansible#612. Will update the dataplane operator with #727 |
Build failed (check pipeline). Post https://review.rdoproject.org/zuul/buildset/f8c7dfd6842b42c0bfa74725eae2fb1f ✔️ openstack-k8s-operators-content-provider SUCCESS in 2h 28m 24s |
Now dataplane adoption job failed with:- in previous run it passed so shouln't be specific to this patch |
recheck this is a know intermittent issue being troubleshooted separately.
|
last few runs in different PRs fails similarly ihttps://review.rdoproject.org/zuul/builds?job_name=cifmw-data-plane-adoption-osp-17-to-extracted-crc&project=openstack-k8s-operators/openstack-operator
|
So far I'm seeing it 1 in 5 runs in openstack-k8s-operators/data-plane-adoption#370 But honestly I'm in the dark why it is failing intermittently. |
Looking better now, should we approve and merge? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/lgtm
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: openstack-k8s-ci-robot, stuggi 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 |
This PR contains the following updates:
073a542
->29d27af
c504b07
->407c91f
17495fd
->d33c69b
7477c1f
->b48a63e
3a7330c
->cacd7b5
7cb8f24
->0ac0a3e
f2ff3c9
->54f145c
f2ff3c9
->54f145c
f2ff3c9
->54f145c
f2ff3c9
->54f145c
f2ff3c9
->54f145c
f2ff3c9
->54f145c
9f02757
->74e2f3c
40cf3e6
->75b048d
f275109
->743a139
6eaf5be
->a2d67e4
8b4e3c4
->a8b0124
1003b1a
->a61ca86
20c4db0
->7708664
30c4c43
->f1aeebd
fae0c72
->69252e9
1c9229f
->f8cecba
3172ebe
->ab779da
Configuration
📅 Schedule: Branch creation - "every weekend" in timezone America/New_York, Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.
This PR has been generated by Renovate Bot.