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

Update localdev platformworkloadidentityrolesets #3901

Merged

Conversation

tsatam
Copy link
Collaborator

@tsatam tsatam commented Oct 11, 2024

Which issue this PR addresses:

Fixes no Jira

What this PR does / why we need it:

  • Add secretLocation for all identities (required to put down CCO secrets in-cluster)
  • Update MachineAPI to use correct service account (machine-api-controllers)
  • Prefix all service account identifiers with 'system:serviceaccount:'

Test plan for issue:

  • Used this new definition locally for MIWI cluster installation. While complete installation is not yet successful, this configuration has allowed CCO secrets to be put down in the correct locations in-cluster, as well as for all federated identity credentials to have the correct identifiers.

Is there any documentation that needs to be updated for this PR?

No

How do you know this will function as expected in production?

Non-production change. This PR will perform the equivalent changes to production configuration: https://msazure.visualstudio.com/AzureRedHatOpenShift/_git/RP-Config/pullrequest/11037151

- Add secretLocation for all identities (required to put down CCO secrets in-cluster)
- Update MachineAPI to use correct service account (machine-api-controllers)
- Prefix all service account identifiers with 'system:serviceaccount:'
@tsatam
Copy link
Collaborator Author

tsatam commented Oct 11, 2024

/azp run ci

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@tsatam tsatam merged commit d52d69d into master Oct 11, 2024
18 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
chainsaw Pull requests or issues owned by Team Chainsaw loki
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants