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

Update provision_aks.sh (#320) #321

Merged
merged 1 commit into from
May 12, 2020
Merged

Conversation

BSamodien
Copy link
Contributor

  • Update provision_aks.sh

Update AKS cluster version from 1.13.11 to 1.15.7

  • Update install_jenkins.sh

Update jenkins for 1.15.7 kubectl

  • Update Dockerfile

Update proctor container for 1.15.7 kubectl

Purpose

  • ...

Does this introduce a breaking change?

[ ] Yes
[ ] No

Pull Request Type

What kind of change does this Pull Request introduce?

[ ] 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

* Update provision_aks.sh

Update AKS cluster version from 1.13.11 to 1.15.7

* Update install_jenkins.sh

Update jenkins for 1.15.7 kubectl

* Update Dockerfile

Update proctor container for 1.15.7 kubectl
@dtzar
Copy link
Contributor

dtzar commented May 12, 2020

1.15.10 is latest default, but 1.16.7 is GA and I'd recommend bumping to this instead to buy more time until 1.16.x expires.

Copy link
Contributor

@rguthriemsft rguthriemsft left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If this were a longer term fix I would agree with David but it's only short term.

@rguthriemsft rguthriemsft merged commit 477d490 into master May 12, 2020
@dtzar
Copy link
Contributor

dtzar commented May 12, 2020

Ok @rguthriemsft why not at least do 1.15.10?

@dtzar dtzar deleted the openhack_v1_updatek8s branch May 12, 2020 20:11
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.

4 participants