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

Upgrade k8s patch version for CI #39

Merged
merged 1 commit into from
Oct 14, 2021
Merged

Upgrade k8s patch version for CI #39

merged 1 commit into from
Oct 14, 2021

Conversation

miseyu
Copy link
Contributor

@miseyu miseyu commented Oct 14, 2021 •
edited
Loading

Why

https://github.com/scalar-labs/helm-charts/runs/3867651052?check_suite_focus=true

Error updating Endpoint Slices for Service default/scalardl-7q0a7l77to-metrics: failed to update scalardl-7q0a7l77to-metrics-j9sqw EndpointSlice for Service default/scalardl-7q0a7l77to-metrics: Operation cannot be fulfilled on endpointslices.discovery.k8s.io "scalardl-7q0a7l77to-metrics-j9sqw": the object has been modified; please apply your changes to the latest version and try again       3m46s        1       scalardl-7q0a7l77to-metrics.16adc279d21070aa

The error on the CI of helm-charts seems to be an error that can happen depending on the version of k8s.
1.19 version seems to be solved by making it 19.9+, so I'll try to raise the version of k8s to run in CI.

kubernetes/kubernetes#92928

In addition, we have updated the patch versions of other k8s versions.

EKS

https://docs.aws.amazon.com/eks/latest/userguide/kubernetes-versions.html

AKS

$ az aks get-versions --location eastus --output table
KubernetesVersion    Upgrades
-------------------  -----------------------
1.22.1(preview)      None available
1.21.2               1.22.1(preview)
1.21.1               1.21.2, 1.22.1(preview)
1.20.9               1.21.1, 1.21.2
1.20.7               1.20.9, 1.21.1, 1.21.2
1.19.13              1.20.7, 1.20.9
1.19.11              1.19.13, 1.20.7, 1.20.9

$ az aks get-versions --location eastasia --output table
KubernetesVersion    Upgrades
-------------------  -----------------------
1.22.1(preview)      None available
1.21.2               1.22.1(preview)
1.21.1               1.21.2, 1.22.1(preview)
1.20.9               1.21.1, 1.21.2
1.20.7               1.20.9, 1.21.1, 1.21.2
1.19.13              1.20.7, 1.20.9
1.19.11              1.19.13, 1.20.7, 1.20.9

Kind

$ curl  https://hub.docker.com/v2/repositories/kindest/node/tags/?page_size=25\&page=1\&name=v1.20 | jq -r '.results[].name'
v1.20.7
v1.20.2
v1.20.0

$ curl  https://hub.docker.com/v2/repositories/kindest/node/tags/?page_size=25\&page=1\&name=v1.19 | jq -r '.results[].name'
v1.19.11
v1.19.7
v1.19.4
v1.19.3
v1.19.1
v1.19.0

$ curl  https://hub.docker.com/v2/repositories/kindest/node/tags/?page_size=25\&page=1\&name=v1.18 | jq -r '.results[].name'
v1.18.19
v1.18.15
v1.18.8
v1.18.6
v1.18.4
v1.18.2
v1.18.0

@miseyu miseyu changed the title Upgrade k8s version for CI WIP: Upgrade k8s version for CI Oct 14, 2021
@miseyu miseyu self-assigned this Oct 14, 2021
@miseyu miseyu added the bugfix label Oct 14, 2021
@miseyu miseyu changed the title WIP: Upgrade k8s version for CI Upgrade k8s version for CI Oct 14, 2021
@miseyu miseyu changed the title Upgrade k8s version for CI Upgrade k8s patch version for CI Oct 14, 2021
feeblefakie
feeblefakie approved these changes Oct 14, 2021
View reviewed changes
Copy link
Contributor

@feeblefakie feeblefakie left a comment

Choose a reason for hiding this comment

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

LGTM! Thank you!

@feeblefakie feeblefakie merged commit 9e8b6a0 into main Oct 14, 2021
@feeblefakie feeblefakie deleted the fix/k8s-version branch October 14, 2021 02:48
scalar-boney
scalar-boney reviewed Oct 14, 2021
View reviewed changes
Copy link
Contributor

@scalar-boney scalar-boney left a comment

Choose a reason for hiding this comment

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

LGTM!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@scalar-boney scalar-boney scalar-boney left review comments

@feeblefakie feeblefakie feeblefakie approved these changes

Assignees

@miseyu miseyu

Labels
bugfix
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@miseyu @feeblefakie @scalar-boney