-
Notifications
You must be signed in to change notification settings - Fork 718
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
[dev-v2.9] rancher-logging k8s 1.29 & 1.30 support #3908
[dev-v2.9] rancher-logging k8s 1.29 & 1.30 support #3908
Conversation
Validation steps
Ex:- longhorn-controller: repository: rancher/hardened-sriov-cni tag: v2.6.3-build20230913
|
Currently investigating 2 (seemingly related) issues with the chart on 2.9 alpha2. When trying to create a ClusterOutput:
And in the same process on the "Output buffer" tab:
|
We found an existing open issue in r/r that matches the issue I've encountered in my testing. rancher/rancher#38730 Because this is an existing issue and not technically an issue with installing the logging chart on 1.29 this is not going to be treated as a blocker for this PR. Once I rebase it and do a quick retest after that I will undraft it for review. @pankajsqe - You may want to keep this in mind for testing that UI creation of Output/ClusterOutput will have this issue until triaged and fixed by UI team. |
and tweak CRD version to match
f4a1866
to
c930d91
Compare
Validation steps
Ex:- longhorn-controller: repository: rancher/hardened-sriov-cni tag: v2.6.3-build20230913
|
@mallardduck @pankajsqe UI issue might be related to open API schema bugs mentionned in rancher/rancher#45157 or rancher/rancher#45158. |
Issue:
rancher/rancher#45356
rancher/rancher#45335
Problem
For Rancher 2.9.x we need to support K8S 1.29 and 1.30
Solution
Adjust the kubeVersion of the chart to support the window of K8S versions Rancher 2.9.X will support.
Testing
Logging should be able to install on 2.9.x build using 1.29 or 1.30 cluster.
Specifically after this is merged version
104.0.0-rc1
of logging will support 1.29 or 1.30 clusters.Engineering Testing
Manual Testing
Successfully installed on 1.29 and 1.30 downstream clusters.
Automated Testing
QA Testing Considerations
We found an existing open issue in r/r that matches the issue I've encountered in my testing. rancher/rancher#38730
Because this is an existing issue and not technically an issue with installing the logging chart on 1.29 this is not going to be treated as a blocker for this PR. Once I rebase it and do a quick retest after that I will undraft it for review.
Regressions Considerations
PSP related things are vestigial at this point - the minimum k8s version is 1.26.
However, unsure if we should begin removing them as they come from upstream.
Backporting considerations
Nope - only for Rancher 2.9.x