Skip to content
This repository was archived by the owner on Nov 16, 2023. It is now read-only.

Added maxUnavailable setting in the deployments#123

Merged
rguthriemsft merged 2 commits intomasterfrom
update/maxUnavailable
Jan 9, 2019
Merged

Added maxUnavailable setting in the deployments#123
rguthriemsft merged 2 commits intomasterfrom
update/maxUnavailable

Conversation

@dcaro
Copy link
Copy Markdown
Contributor

@dcaro dcaro commented Dec 4, 2018

Purpose

Adding the maxUnavailable setting on the helm chart for the APIs.

  • ...

Does this introduce a breaking change?

[ ] Yes
[X] No

Pull Request Type

What kind of change does this Pull Request introduce?

[X] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Documentation content changes
[ ] Other... Please describe:

How to Test

  • Get the code
git clone [repo-address]
cd [repo-name]
git checkout [branch-name]
npm install
  • Test the code

What to Check

Verify that the following are valid

  • ...

Other Information

@dcaro dcaro changed the title [WIP] added maxUnavailable setting in the deployments Added maxUnavailable setting in the deployments Jan 5, 2019
@dcaro dcaro requested a review from rguthriemsft January 5, 2019 04:21
@rguthriemsft rguthriemsft merged commit 4ac8aee into master Jan 9, 2019
@dtzar dtzar deleted the update/maxUnavailable branch April 6, 2019 15:59
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants