Skip to content

Add dt-sharded deployment topology with per-service galera, rabbitmq and memcached#752

Merged
centosinfra-prod-github-app[bot] merged 1 commit into
openstack-k8s-operators:mainfrom
lmiccini:dt-sharded
May 18, 2026
Merged

Add dt-sharded deployment topology with per-service galera, rabbitmq and memcached#752
centosinfra-prod-github-app[bot] merged 1 commit into
openstack-k8s-operators:mainfrom
lmiccini:dt-sharded

Conversation

@lmiccini

@lmiccini lmiccini commented May 6, 2026

Copy link
Copy Markdown
Contributor

New deployment topology for testing per-service database, message bus,
and cache sharding. Each OpenStack service gets a dedicated galera
cluster, a dedicated rabbitmq cluster, and a dedicated memcached
instance, all with replicas:1.

Co-Authored-By: Claude Opus 4.6 noreply@anthropic.com

Depends-on: openstack-k8s-operators/ci-framework#3907

@openshift-ci openshift-ci Bot requested review from fultonj and raukadah May 6, 2026 18:08
@lmiccini lmiccini added the do-not-merge Changes are not ready to be merged label May 7, 2026
@lmiccini lmiccini changed the title Add dt-sharded deployment topology with per-service galera and rabbitmq Add dt-sharded deployment topology with per-service galera, rabbitmq and memcached May 7, 2026
@lmiccini lmiccini force-pushed the dt-sharded branch 5 times, most recently from 9aebc5b to cb811be Compare May 7, 2026 09:21
@centosinfra-prod-github-app

Copy link
Copy Markdown
Contributor

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://gateway-cloud-softwarefactory.apps.ocp.cloud.ci.centos.org/zuul/t/rdoproject.org/buildset/de1249963edc407c9d84a0881d3b3292

✔️ noop SUCCESS in 0s
rhoso-architecture-validate-dt-sharded FAILURE in 3m 43s

@lmiccini

lmiccini commented May 7, 2026

Copy link
Copy Markdown
Contributor Author

recheck

@lmiccini lmiccini force-pushed the dt-sharded branch 5 times, most recently from d92188d to ec43cbe Compare May 7, 2026 12:24
@centosinfra-prod-github-app

Copy link
Copy Markdown
Contributor

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://gateway-cloud-softwarefactory.apps.ocp.cloud.ci.centos.org/zuul/t/rdoproject.org/buildset/040a18c2189049b7aadbe188d747d9ef

✔️ noop SUCCESS in 0s
rhoso-architecture-validate-dt-sharded FAILURE in 3m 57s

@lmiccini lmiccini force-pushed the dt-sharded branch 2 times, most recently from 6cb857f to 4aff600 Compare May 8, 2026 13:34
@centosinfra-prod-github-app

Copy link
Copy Markdown
Contributor

This change depends on a change that failed to merge.

Change openstack-k8s-operators/ci-framework#3907 is needed.

@lmiccini

lmiccini commented May 9, 2026

Copy link
Copy Markdown
Contributor Author

recheck

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

/lgtm

Will wait for others to review if needed

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

This looks good @lmiccini I just found one thing below.

Comment thread dt/dt-sharded/kustomization.yaml Outdated
replicas: 1
secret: osp-secret
storageRequest: 5Gi
openstack-horizon:

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.

It looks like openstack-horizon is created but not used.

I created the control plane but I don't see any reference to openstack-horizon

[johfulto@laptop architecture{dt-sharded}]$ grep databaseInstance /tmp/cr-review/dt-sharded-cp.yaml
      databaseInstance: openstack-barbican
      databaseInstance: openstack-cinder
      databaseInstance: openstack-designate
      databaseInstance: openstack-glance
      databaseInstance: openstack
      databaseInstance: openstack
      databaseInstance: openstack-keystone
      databaseInstance: openstack
      databaseInstance: openstack-neutron
      databaseInstance: openstack-octavia
      databaseInstance: openstack-placement
          databaseInstance: openstack
[johfulto@laptop architecture{dt-sharded}]$ 

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks John, dropping that db as it's not required indeed, not sure how it got in there.

… and memcached

New deployment topology for testing per-service database, message bus,
and cache sharding. Each OpenStack service gets a dedicated galera
cluster, a dedicated rabbitmq cluster, and a dedicated memcached
instance, all with replicas:1.

Galera databases: openstack-barbican, openstack-cinder,
openstack-designate, openstack-glance, openstack-horizon,
openstack-keystone, openstack-neutron, openstack-octavia,
openstack-nova, openstack-placement.

Per-service rabbitmq clusters: rabbitmq-barbican, rabbitmq-cinder,
rabbitmq-designate, rabbitmq-glance, rabbitmq-keystone,
rabbitmq-neutron, rabbitmq-octavia. Nova cells use the standard rabbitmq
(cell0/notifications) and rabbitmq-cell1 (cell1).

Per-service memcached instances: memcached-keystone, memcached-glance,
memcached-cinder, memcached-neutron, memcached-horizon, memcached-nova.
Services without memcachedInstance support (barbican, designate, octavia,
placement) are excluded.

Enabled services: barbican, cinder, designate, glance, horizon,
keystone, neutron, nova, octavia.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@fultonj

fultonj commented May 18, 2026

Copy link
Copy Markdown
Contributor

/approval
/lgtm

@abays

abays commented May 18, 2026

Copy link
Copy Markdown
Contributor

recheck-gate

@openshift-ci

openshift-ci Bot commented May 18, 2026

Copy link
Copy Markdown

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: fultonj, lmiccini

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

@centosinfra-prod-github-app

Copy link
Copy Markdown
Contributor

@centosinfra-prod-github-app centosinfra-prod-github-app Bot merged commit 5cafc8f into openstack-k8s-operators:main May 18, 2026
9 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.

3 participants