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

[CPREQ-9284] k8s and 3rd party update #682

Merged
merged 7 commits into from
Aug 23, 2024
Merged

Conversation

nikhil1697
Copy link
Collaborator

@nikhil1697 nikhil1697 commented Aug 21, 2024

Description

New kubernetes version v1.28.12, v1.29.7 & v1.30.3 should be supported
New plugins versions should be adopted.
*

Solution

Adopt latest release of kubernetes, calico, local-path-provisioner and nginx-ingress-controller

kubernetes version: v1.28.12, v1.29.7 & v1.30.3
calico: v3.28.1
nginx-ingress-controller: v1.11.1
local-path-provisioner: v0.0.27

How to apply

Provide steps how to apply on top previous Kubemarine version to execute on existing clusters

Test Cases

Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration.

TestCase 1

Steps:

  1. Kubemarine install

Results:

Before After
Not applicable Success
  1. Kubemarine upgrade

Results:

Before After
Not applicable Success

Checklist

  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • Integration CI passed
  • Unit tests. If Yes list of new/changed tests with brief description
  • There is no merge conflicts

Unit tests

Indicate new or changed unit tests and what they do, if any.

@koryaga koryaga requested review from Imadzuma and alexarefev August 22, 2024 12:42
@koryaga koryaga self-assigned this Aug 22, 2024
@koryaga koryaga changed the title CPREQ-9284_Third_party_update [CPREQ-9284] k8s and 3rd party update Aug 22, 2024
@koryaga koryaga marked this pull request as ready for review August 22, 2024 12:43
Copy link
Collaborator

@alexarefev alexarefev left a comment

Choose a reason for hiding this comment

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

I'm not sure that we need Calico v3.28.0 original YAMLs

@nikhil1697
Copy link
Collaborator Author

I'm not sure that we need Calico v3.28.0 original YAMLs

v3.28.0 was added in the previous commit but later v3.28.1 was added and the original YAMLs was left without removing

@alexarefev
Copy link
Collaborator

It's not needed, since it not used in the compatibility map

@koryaga koryaga merged commit 27dd2c6 into main Aug 23, 2024
45 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants