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

feat(kayenta): Add Flag to Disable Canary Config UI Editing #627

Merged

Conversation

stevenlandrydd
Copy link
Contributor

@stevenlandrydd stevenlandrydd commented May 15, 2024

Adds a "disableConfigEdit" flag into the "Canary" settings of deck which will make the config edit portions of the canary config ui read only for all configs if the flag is set to true. This feature is intended for cases where the canary configs are being managed via code and deployed/updated via api calls to prevent drift and issues arising from manually made canary changes.

Images below show the state of the edit panels with the flag enabled.

image image

@spinnakerbot
Copy link

The following commits need their title changed:

  • 67fcc9c: feat:(kayenta) add ability to disable all canary editing in ui

Please format your commit title into the form:

<type>(<scope>): <subject>, e.g. fix(kubernetes): address NPE in status check

This allows us to easily generate changelogs & determine semantic version numbers when cutting releases. You can read more about commit conventions here.

@stevenlandrydd stevenlandrydd changed the title Feat: Flag to Disable Canary UI Editing Feat: Add Flag to Disable Canary UI Editing May 16, 2024
@stevenlandrydd stevenlandrydd changed the title Feat: Add Flag to Disable Canary UI Editing Feat(kayenta): Add Flag to Disable Canary UI Editing May 16, 2024
@stevenlandrydd stevenlandrydd changed the title Feat(kayenta): Add Flag to Disable Canary UI Editing feat(kayenta): Add Flag to Disable Canary UI Editing May 16, 2024
@stevenlandrydd stevenlandrydd changed the title feat(kayenta): Add Flag to Disable Canary UI Editing feat(kayenta): Add Flag to Disable Canary Config UI Editing May 16, 2024
@jasonmcintosh jasonmcintosh merged commit dd6b985 into spinnaker:master May 16, 2024
4 checks passed
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.

4 participants