Skip to content

feat(openstack-sync): add generic extra_status support to the sync framework - #2363

Open
haseebsyed12 wants to merge 1 commit into
mainfrom
oso-neutronsubnetpoolsyncs
Open

haseebsyed12 wants to merge 1 commit into
mainfrom
oso-neutronsubnetpoolsyncs

Conversation

@haseebsyed12

@haseebsyed12 haseebsyed12 commented Sep 21, 2026

Copy link
Copy Markdown
Contributor

What does this change do?

Used it for NeutronSubnetPool prefixes to report each Nautobot prefix on NeutronSubnetPool status

Generated CR in cluster

apiVersion: neutron.understack.rackspace.net/v1alpha1
kind: NeutronSubnetPool
metadata:
  annotations:
    argocd.argoproj.io/tracking-id: >-
      site-dev-openstack-sync-plugins:neutron.understack.example.com/NeutronSubnetPool:openstack/servicenet-ip4
    link.argocd.argoproj.io/external-link: >-
      https://nautobot.dev.understack.example.com/ipam/prefixes/b86d8eba-ef24-4c4c-b254-b5fac75014d7/
  creationTimestamp: '2026-09-18T17:20:25Z'
  generation: 1
  labels:
    app.kubernetes.io/component: neutron-subnet-pools
    app.kubernetes.io/name: openstack-sync-plugins
    app.kubernetes.io/part-of: openstack-sync
  name: servicenet-ip4
  namespace: openstack
  resourceVersion: '523126350'
  uid: c5d9daba-d650-493a-a6a0-46ab82dfddfc
  selfLink: >-
    /apis/neutron.understack.example.com/v1alpha1/namespaces/openstack/neutronsubnetpools/servicenet-ip4
status:
  conditions:
    - lastTransitionTime: '2026-09-18T17:20:43Z'
      message: Successfully reconciled subnet pool
      observedGeneration: 1
      reason: Reconciled
      status: 'True'
      type: Ready
  lastSyncTime: '2026-09-21T09:29:48Z'
  message: Successfully reconciled subnet pool
  observedGeneration: 1
  prefixes:
    - cidr: 10.100.4.0/22
      id: b86d8eba-ef24-4c4c-b254-b5fac75014d7
      url: >-
        https://nautobot.dev.understack.example.com/ipam/prefixes/b86d8eba-ef24-4c4c-b254-b5fac75014d7/
  syncStatus: Synced
spec:
  address_scope:
    create: true
    name: servicenet-ip4
    shared: true
  cloudCredentialsRef:
    cloudName: understack
    secretName: topsecret
  default_prefix_length: 27
  is_default: false
  maximum_prefix_length: 30
  minimum_prefix_length: 26
  name: servicenet-ip4
  nautobot:
    api_version: '2.0'
    prefix_refs:
      - id: b86d8eba-ef24-4c4c-b254-b5fac75014d7
    require:
      namespace: Rackspace
      status: Active
      tags:
        - openstack-subnet-pool
      type: pool
    tokenSecretRef:
      key: token
      secretName: nautobot-token
    url: https://nautobot.dev.understack.example.com
  shared: true
  tags:
    - managed-by-openstack-sync

Upgrade impact

  • This change requires operator action to upgrade. If checked, add the
    upgrade-impact label and a release note: run scriv create from the
    repository root and describe the required action in the generated
    changelog.d/ file. See RELEASING.md.

Operator action means anything a deployment has to do beyond a normal resync:
deploy repo or values changes, new or removed secrets, enabling or disabling a
component, or a manual one-time step.

@github-actions

github-actions Bot commented Sep 21, 2026

Copy link
Copy Markdown

Python test results

454 tests   454 ✅  3s ⏱️
  1 suites    0 💤
  1 files      0 ❌

Results for commit 9c8535a.

♻️ This comment has been updated with latest results.

@haseebsyed12 haseebsyed12 changed the title feat(openstack-sync): report each Nautobot prefix on NeutronSubnetPool status feat(openstack-sync): add generic extra_status support to the sync framework Sep 21, 2026
@haseebsyed12
haseebsyed12 requested a review from a team September 21, 2026 09:56
@haseebsyed12
haseebsyed12 marked this pull request as ready for review September 21, 2026 09:56
@haseebsyed12
haseebsyed12 force-pushed the oso-neutronsubnetpoolsyncs branch 3 times, most recently from a650c0f to 7edb4db Compare September 21, 2026 12:55
@haseebsyed12
haseebsyed12 force-pushed the oso-neutronsubnetpoolsyncs branch from 7edb4db to 9c8535a Compare September 21, 2026 18:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants