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

fix(google): allow SpEL in stack/detail in deploy stage #7105

Merged
merged 2 commits into from
Jun 11, 2019

Conversation

maggieneterval
Copy link
Contributor

Closes spinnaker/spinnaker#4324

  • You can use SpEL in stack/detail fields in Deploy stages, but you cannot use SpEL in stack/detail fields in ad-hoc Deploy/Clone operations. We set command.viewState.templatingEnabled = true in deploy state initialization methods shared by other providers but not by GCE, so previously we were incorrectly disallowing SpEL in stack/detail fields in Deploy stages.

@maggieneterval
Copy link
Contributor Author

@plumpy Not sure why I can't add you as a reviewer 😭

@maggieneterval maggieneterval merged commit 92492c4 into spinnaker:master Jun 11, 2019
@maggieneterval maggieneterval deleted the fix-gce-validation branch June 11, 2019 20:38
@maggieneterval
Copy link
Contributor Author

@spinnakerbot cherry-pick 1.14

@spinnakerbot
Copy link
Contributor

Cherry pick successful: #7106

christopherthielen added a commit to christopherthielen/deck that referenced this pull request Jul 3, 2019
cdd6f23 chore(package): Just Update Prettier™
5cc4146 perf(google): avoid unnecessary fetching and filtering of gce images (spinnaker#7115)
92492c4 fix(google): allow SpEL in stack/detail in deploy stage (spinnaker#7105)
94dc56d fix(google): Help text on GCE load balancer type selection screen (spinnaker#7044)
christopherthielen added a commit that referenced this pull request Jul 3, 2019
cdd6f23 chore(package): Just Update Prettier™
5cc4146 perf(google): avoid unnecessary fetching and filtering of gce images (#7115)
92492c4 fix(google): allow SpEL in stack/detail in deploy stage (#7105)
94dc56d fix(google): Help text on GCE load balancer type selection screen (#7044)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[GCE/Deck] Unable to use parameters in Detail field
3 participants