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

Add support for https catalog repo #215

Merged
merged 2 commits into from
Dec 17, 2024
Merged

Add support for https catalog repo #215

merged 2 commits into from
Dec 17, 2024

Conversation

simu
Copy link
Member

@simu simu commented Dec 17, 2024

Because component-argocd can't use Vault references directly (due to needing to be able to be synced to setup the Vault integration during cluster bootstrap), we simply provide a parameter which contains the name of an externally managed secret in the syn namespace.

This secret must have keys username and password which will be configured as the credentials for the cluster catalog if the catalog URL starts with https://.

Checklist

  • The PR has a meaningful title. It will be used to auto-generate the
    changelog.
    The PR has a meaningful description that sums up the change. It will be
    linked in the changelog.
  • PR contains a single logical change (to build a better changelog).
  • Update the documentation.
  • Categorize the PR by adding one of the labels:
    bug, enhancement, documentation, change, breaking, dependency
    as they show up in the changelog.

@simu simu added the enhancement New feature or request label Dec 17, 2024
simu added 2 commits December 17, 2024 11:31
Because component-argocd can't use Vault references directly (due to
needing to be able to be synced to setup the Vault integration during
cluster bootstrap), we simply provide a parameter which contains the
name of an externally managed secret in the `syn` namespace.

This secret must have keys `username` and `password` which will be
configured as the credentials for the cluster catalog if the catalog URL
starts with `https://`.
Rendered from template version: main (ff9d5a8)
@simu simu force-pushed the feat/https-catalog branch from e25e52a to e4d3111 Compare December 17, 2024 10:31
@simu simu marked this pull request as ready for review December 17, 2024 10:43
@simu simu requested review from zugao and a team December 17, 2024 10:43
Copy link

@zugao zugao left a comment

Choose a reason for hiding this comment

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

LGTM, thanks for your effort

Copy link

github-actions bot commented Dec 17, 2024

🚀 This PR has been released as v8.8.0

Triggering workflows Release

🛠️ Auto tagging enabled with label bump:minor

@simu simu merged commit 8c31821 into master Dec 17, 2024
16 checks passed
@simu simu deleted the feat/https-catalog branch December 17, 2024 12:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bump:minor enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants