Skip to content

feat(oso): dynamically loads credentials from named secret at runtime - #2220

Merged
cardoe merged 1 commit into
mainfrom
correct-cloud-names
Aug 17, 2026
Merged

feat(oso): dynamically loads credentials from named secret at runtime#2220
cardoe merged 1 commit into
mainfrom
correct-cloud-names

Conversation

@haseebsyed12

@haseebsyed12 haseebsyed12 commented Aug 17, 2026

Copy link
Copy Markdown
Contributor

What does this change do?

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.

@cardoe

cardoe commented Aug 17, 2026

Copy link
Copy Markdown
Contributor

I'd honestly rather let the operator have the ability to read the data it needs and pass in a secretRef with the resource so that it can utilize the correct one.

@haseebsyed12
haseebsyed12 requested a review from a team August 17, 2026 17:33
@haseebsyed12 haseebsyed12 changed the title fix: renaming to assemble one clouds.yaml containing all uniquely-named cloud entries into a single secret feat(oso): dynamically loads credentials from named secret at runtime Aug 17, 2026
@haseebsyed12
haseebsyed12 marked this pull request as ready for review August 17, 2026 18:53
Comment thread python/openstack-sync/openstack_sync/utils.py
@cardoe
cardoe added this pull request to the merge queue Aug 17, 2026
Merged via the queue into main with commit aa1d482 Aug 17, 2026
67 checks passed
@cardoe
cardoe deleted the correct-cloud-names branch August 17, 2026 20:09
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.

3 participants