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.

Sign up for GitHub

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

Jump to bottom

feat(core/pipeline): Add the ability to disable base os options #8851

Merged
merged 2 commits into from
Jan 20, 2021
Merged

feat(core/pipeline): Add the ability to disable base os options #8851

merged 2 commits into from
Jan 20, 2021

Conversation

caseyhebebrand
Copy link
Contributor

For outdated baseOsOptions that have not yet been fully deprecated. This prevents future adoption of the base OS, while keeping the UI from throwing an error for any bake stages that still utilize the disabled option(s).

@caseyhebebrand caseyhebebrand merged commit 5e45b85 into spinnaker:master Jan 20, 2021
@caseyhebebrand caseyhebebrand deleted the disable-image-selection branch January 20, 2021 18:27
caseyhebebrand added a commit that referenced this pull request Jan 21, 2021
@caseyhebebrand
chore(core): publish core@0.0.542
5b0ca27 
b0139a6 feat(core/loadBalancer): Filter by load balancer type (#8850)
5e45b85 feat(core/pipeline): Add the ability to disable base os options (#8851)
dde8513 fix(core/pipelines): break and wrap long comments text (#8748)
e78115e fix(API): encodeURIComponent for each path() element (#8646)
@caseyhebebrand caseyhebebrand mentioned this pull request Jan 21, 2021
Merged
mergify bot pushed a commit that referenced this pull request Jan 21, 2021
@caseyhebebrand @mergify
chore(core): publish core@0.0.542
35e0131 
b0139a6 feat(core/loadBalancer): Filter by load balancer type (#8850)
5e45b85 feat(core/pipeline): Add the ability to disable base os options (#8851)
dde8513 fix(core/pipelines): break and wrap long comments text (#8748)
e78115e fix(API): encodeURIComponent for each path() element (#8646)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@vigneshm vigneshm vigneshm approved these changes

@alanmquach alanmquach Awaiting requested review from alanmquach

Assignees
No one assigned
Labels
target-release/1.25
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@caseyhebebrand @vigneshm @spinnakerbot