Skip to content

Add new update data plane service CRs into openstack-operator#1478

Merged
openshift-merge-bot[bot] merged 1 commit into
openstack-k8s-operators:mainfrom
ciecierski:update-split-services
Jun 12, 2025
Merged

Add new update data plane service CRs into openstack-operator#1478
openshift-merge-bot[bot] merged 1 commit into
openstack-k8s-operators:mainfrom
ciecierski:update-split-services

Conversation

@ciecierski

@ciecierski ciecierski commented Jun 4, 2025

Copy link
Copy Markdown
Contributor

Add services for edpm_update_services and edpm_update_system playbooks.[1]
For update-services EDPMServiceType add logic to indicate that minor update
is complete post service deployment, mirroring the behavior of
'update' EDPMServiceType.

Functional test for new update-service is the same as for previous
update EDPMServiceType.

OSPRH-16789

[1]openstack-k8s-operators/edpm-ansible#962

@openshift-ci openshift-ci Bot requested review from lewisdenny and stuggi June 4, 2025 08:06

@jistr jistr 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.

We should also make sure the OpenStackVersion resource considers the update finished when the update-services service has deployed. I think we need to edit this spot to make it work both with update and with update-services but not sure if there are more places to edit:

if service.Spec.EDPMServiceType != "update" {

@softwarefactory-project-zuul

Copy link
Copy Markdown

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/0d4d4bf2916a4ab59d4d1efb7b535c09

✔️ openstack-k8s-operators-content-provider SUCCESS in 3h 27m 35s
✔️ podified-multinode-edpm-deployment-crc SUCCESS in 1h 24m 32s
✔️ cifmw-crc-podified-edpm-baremetal SUCCESS in 1h 38m 21s
✔️ adoption-standalone-to-crc-ceph-provider SUCCESS in 3h 09m 15s
openstack-operator-tempest-multinode FAILURE in 1h 48m 59s

@ciecierski ciecierski force-pushed the update-split-services branch from 8e307ad to 13ae2b6 Compare June 5, 2025 14:23
@ciecierski ciecierski marked this pull request as ready for review June 5, 2025 14:26
@openshift-ci openshift-ci Bot requested review from jpodivin and rebtoor June 5, 2025 14:26
@softwarefactory-project-zuul

Copy link
Copy Markdown

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/849eb6c0a04f45bfa00e23d6cae29806

✔️ openstack-k8s-operators-content-provider SUCCESS in 3h 26m 51s
✔️ podified-multinode-edpm-deployment-crc SUCCESS in 1h 22m 00s
✔️ cifmw-crc-podified-edpm-baremetal SUCCESS in 1h 39m 16s
✔️ adoption-standalone-to-crc-ceph-provider SUCCESS in 3h 10m 52s
openstack-operator-tempest-multinode FAILURE in 1h 41m 13s

@ciecierski ciecierski force-pushed the update-split-services branch from 13ae2b6 to 5a0868a Compare June 5, 2025 19:56
@softwarefactory-project-zuul

Copy link
Copy Markdown

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/5d3cfa78ca8f40988b613e806a50e38b

✔️ openstack-k8s-operators-content-provider SUCCESS in 3h 32m 14s
podified-multinode-edpm-deployment-crc FAILURE in 1h 38m 14s
✔️ cifmw-crc-podified-edpm-baremetal SUCCESS in 1h 36m 21s
✔️ adoption-standalone-to-crc-ceph-provider SUCCESS in 3h 13m 41s
openstack-operator-tempest-multinode FAILURE in 1h 57m 11s

@ciecierski ciecierski force-pushed the update-split-services branch from 5a0868a to 0508784 Compare June 6, 2025 07:28
@jistr

jistr commented Jun 6, 2025

Copy link
Copy Markdown
Contributor

/lgtm

@openshift-ci openshift-ci Bot added the lgtm label Jun 6, 2025
@ciecierski ciecierski requested review from dprince and slagle and removed request for jpodivin and lewisdenny June 6, 2025 13:16

@jistr jistr 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.

My review was still showing as "changes requested" but i already gave a LGTM.

@slagle slagle 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.

add logic to indicate that minor update
is complete post service deployment, mirroring the behavior of
'update' EDPMServiceType.

This isn't entirely accurate in that it doesn't mirror the behavior, it's actually a change in behavior and what is being saved on the Status. Previously, we knew when both a service and system update was completed as represented by the deployed OpenStackVersion on the Status. Now we don't, so I think we need to add something in a followup.

Approving with this understanding.

@openshift-ci

openshift-ci Bot commented Jun 11, 2025

Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: ciecierski, dprince, jistr, slagle

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

@jistr

jistr commented Jun 11, 2025

Copy link
Copy Markdown
Contributor

The new procedure won't track the state of system update, as using our system update procedure will be optional. Admins will be free to use a different way to keep their operating system up to date, if they'd like. They operating system version will have to fit within what we consider supported.

@ciecierski

Copy link
Copy Markdown
Contributor Author

/retest-required

Add services for edpm_update_services and edpm_update_system playbooks.[1]
For update-services EDPMServiceType add logic to indicate that minor update
is complete post service deployment, mirroring the behavior of
'update' EDPMServiceType.

Functional test for new update-service is the same as for previous
update EDPMServiceType.

[1]openstack-k8s-operators/edpm-ansible#962
@ciecierski ciecierski force-pushed the update-split-services branch from 0508784 to 5dfbabd Compare June 11, 2025 18:50
@openshift-ci openshift-ci Bot removed the lgtm label Jun 11, 2025
@softwarefactory-project-zuul

Copy link
Copy Markdown

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/47630ebb3f8d41be99aab925a0014539

✔️ openstack-k8s-operators-content-provider SUCCESS in 1h 48m 37s
✔️ podified-multinode-edpm-deployment-crc SUCCESS in 1h 12m 44s
✔️ cifmw-crc-podified-edpm-baremetal SUCCESS in 1h 32m 57s
adoption-standalone-to-crc-ceph-provider FAILURE in 1h 31m 15s
✔️ openstack-operator-tempest-multinode SUCCESS in 1h 25m 00s

@ciecierski

Copy link
Copy Markdown
Contributor Author

recheck

@dprince

dprince commented Jun 12, 2025

Copy link
Copy Markdown
Contributor

/lgtm

@openshift-ci openshift-ci Bot added the lgtm label Jun 12, 2025
@openshift-merge-bot openshift-merge-bot Bot merged commit fdf9869 into openstack-k8s-operators:main Jun 12, 2025
8 checks passed
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