Skip to content
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

PTEUDO-2183: Add new status condition for dbc with implicitly versioning #399

Merged
merged 3 commits into from
Jan 27, 2025

Conversation

bfabricio
Copy link
Contributor

@bfabricio bfabricio commented Jan 24, 2025

  • adds status which indicates that a deprecated implied version is in use

Test result

apiVersion: persistance.atlas.infoblox.com/v1
kind: DatabaseClaim
metadata:
  annotations:
    kubectl.kubernetes.io/last-applied-configuration: |
      {"apiVersion":"persistance.atlas.infoblox.com/v1","kind":"DatabaseClaim","metadata":{"annotations":{},"name":"fbottega-sample-database","namespace":"fbottega"},"spec":{"appId":"some-app","class":"fbottega","databaseName":"sample-db","dsnName":"dsn.txt","minStorageGB":20,"secretName":"sample-secret-name","type":"postgres","userName":"sample-user"}}
  creationTimestamp: "2024-12-17T11:35:44Z"
  finalizers:
  - databaseclaims.persistance.atlas.infoblox.com/finalizer
  generation: 2
  name: fbottega-sample-database
  namespace: fbottega
  resourceVersion: "2865790067"
  uid: b0360c5c-84ab-4c1a-b58d-e0d9d0a10f11
spec:
  appId: some-app
  class: fbottega
  databaseName: sample-db
  dbVersion: ""
  dsnName: dsn.txt
  enableReplicationRole: false
  enableSuperUser: false
  minStorageGB: 20
  secretName: sample-secret-name
  shape: ""
  type: postgres
  useExistingSource: false
  userName: sample-user
status:
  activeDB:
    DbState: ready
    connectionInfo:
      databaseName: sample-db
      port: "5432"
      sslMode: require
      userName: sample-user_a
    connectionUpdatedAt: "2025-01-27T14:00:09Z"
    dbCreateAt: "2025-01-27T14:00:09Z"
    dbversion: "15"
    minStorageGB: 20
    shape: db.t4g.medium
    type: postgres
    userUpdatedAt: "2025-01-27T14:00:09Z"
  conditions:
  - lastTransitionTime: "2024-12-17T11:36:44Z"
    message: Database successfully synchronized and ready for use
    observedGeneration: 2
    reason: Available
    status: "True"
    type: Ready
  - lastTransitionTime: "2025-01-27T14:00:09Z"
    message: Database is provisioned
    observedGeneration: 2
    reason: Available
    status: "True"
    type: Synced
  - lastTransitionTime: "2025-01-25T20:35:36Z"
    message: No database version specified
    observedGeneration: 2
    reason: Available
    status: "True"
    type: NeedsMigrate
  newDB: {}
  oldDB: {}

@bfabricio bfabricio marked this pull request as ready for review January 24, 2025 16:57
@bfabricio bfabricio requested review from drewwells and leandrorichardtoledo and removed request for drewwells January 24, 2025 16:57
Co-authored-by: Drew Wells <dwells@infoblox.com>
@bfabricio bfabricio merged commit b41fb22 into main Jan 27, 2025
3 checks passed
@bfabricio bfabricio deleted the PTEUDO-2183 branch January 27, 2025 23:43
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