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

workload_gitops_bootstrap - add health check and wait for all gitops apps #8960

Closed
wants to merge 13 commits into from

Conversation

newgoliath
Copy link
Collaborator

  • Feature Pull Request

@newgoliath newgoliath requested a review from juliaaano January 11, 2025 04:18
Copy link
Collaborator

@juliaaano juliaaano left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I love this PR. Thank you so much for creating it.

@newgoliath
Copy link
Collaborator Author

newgoliath commented Jan 12, 2025

@juliaaano I added code to copy data from labeled configmaps to agnosticd_user_info.

in a configmap: .data.users_json is a json string of agnosticd_user_info formatted user data.
All other data in a configmap is expected to by yaml.

Example helm implementation of configmaps: https://github.com/newgoliath/rosa_uplift/blob/main/uplift/templates/userinfo_configmap.yaml

The agnosticd code I just added to the workload:

  • first it copies all the non-users_json data into agnosticd_user_info, assuming it's yaml data.
  • then it creates a variable data_users_json which is a recursive json merge of all configmaps' data.user_json
  • finally it adds all this merged data into agnosticd_user_info

@newgoliath newgoliath requested a review from juliaaano January 12, 2025 20:36
@juliaaano
Copy link
Collaborator

Splitting this PR in two:

#8962
#8964

@juliaaano juliaaano closed this Jan 13, 2025
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