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 toggling templating on a service, and azure resource group config #691

Merged

Conversation

michaeljguarino
Copy link
Member

@michaeljguarino michaeljguarino commented Feb 16, 2024

Summary

Need the latter for velero setup. Also realized service secret fetching is very effectively cacheable, so doing that.

Test Plan

regression

Checklist

  • If required, I have updated the Plural documentation accordingly.
  • I have added tests to cover my changes.
  • I have added a meaningful title and summary to convey the impact of this PR to a user.
  • I have added relevant labels to this PR to help with categorization for release notes.

@michaeljguarino michaeljguarino added the enhancement New feature or request label Feb 16, 2024
@michaeljguarino michaeljguarino requested a review from a team February 16, 2024 00:36
Copy link

linear bot commented Feb 16, 2024

Copy link
Contributor

github-actions bot commented Feb 16, 2024

Visit the preview URL for this PR (updated for commit 0272815):

https://pluralsh-console--pr691-mjg-eng-1766-add-api-xidg6a3b.web.app

(expires Fri, 23 Feb 2024 05:43:58 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: dd1ffa0705acc6ef7d6db370e6bd6fc390e945ce

@michaeljguarino michaeljguarino force-pushed the mjg/eng-1766-add-api-support-for-liquid-template-disabling branch from 791bf75 to 9a1b9c2 Compare February 16, 2024 02:58
…roup config

Need the latter for velero setup.  Also realized service secret fetching is very effectively cacheable, so doing that.
@michaeljguarino michaeljguarino force-pushed the mjg/eng-1766-add-api-support-for-liquid-template-disabling branch from 9a1b9c2 to 8332d7a Compare February 16, 2024 03:13
* external token only for admins
* k8s queries on mgmt cluster (OSS cluster) require operator role
@maciaszczykm maciaszczykm merged commit cbbcbbd into master Feb 16, 2024
11 checks passed
@maciaszczykm maciaszczykm deleted the mjg/eng-1766-add-api-support-for-liquid-template-disabling branch February 16, 2024 08:41
snikch pushed a commit to snikch/plural-console that referenced this pull request Apr 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants