Skip to content

Conversation

@cardoe
Copy link
Contributor

@cardoe cardoe commented Oct 17, 2025

Firstly this takes the OpenStack automation account docs and code renaming it to "automation account" from "service account" to avoid confusion as suggested by @nicholaskuechler. Then create an automation account "infrasetup" per site which can be used by each of the services as a post deployment. Then moved the octavia and neutron jobs into their respective kustomize and standardized their behavior. Created some checks in the ansible to ensure that we are able to communicate with the environment correctly. Set resource limits on the jobs so that they don't run away from us and set labels so that ArgoCD can track it successfully.

@cardoe cardoe requested review from a team, mfencik, nicholaskuechler and skrobul October 17, 2025 17:23
@cardoe cardoe force-pushed the automation-accounts branch from 93127a0 to 0154e57 Compare October 17, 2025 17:30
@cardoe cardoe requested a review from haseebsyed12 October 17, 2025 19:07
Rename from service accounts to automation accounts to avoid confusion
with OpenStack service accounts.
This account has read/write permissions on the baremetal project so that
we can use it to configure various services and load any data that is
necessary for the site to work.
This task just checks that we can authenticate to OpenStack.
Add security contexts. Add hook config so that ArgoCD sees the job as
completing and cleans it up. Add resource limits. Switch to using the
infrasetup user and loading the keystone URL from the site metadata.
Instead of attempting to detect when a deployment happens of neutron,
just run the post deployment job after we deploy neutron each time. This
job should be idempotent so this should be successful each time.
Due to the way that ExternalSecrets can update the Secret, its possible
this will trigger with an empty password set so we need to skip that.
The nautobot sync job is hitting the time limit so remove it for now and
increase how much resources it gets.
@cardoe cardoe force-pushed the automation-accounts branch from 0154e57 to 95eba6c Compare October 17, 2025 19:07
@cardoe cardoe added this pull request to the merge queue Oct 17, 2025
Merged via the queue into main with commit 7653f6d Oct 17, 2025
32 checks passed
@cardoe cardoe deleted the automation-accounts branch October 17, 2025 21:23
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