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

Csp adapter release v3.0.1 #3524

Merged
merged 3 commits into from
Feb 27, 2024
Merged

Conversation

mbelur
Copy link
Contributor

@mbelur mbelur commented Feb 27, 2024

##Issue:
rancher/rancher#43729

##Problem:
Support for k8s 1.28

##Testing:

scenario 1 (upgrade scenario):

Start with an EKS cluster (k8s version 1.27), install rancher 2.8.2 with csp-adapter 3.0.0
Upgrade to locally built rancher with cspAdapterMinVersion: 103.0.1+up3.0.1
Update charts url repo
Upgrade the csp adapter to 103.0.0+up3.0.1    
Upgraded EKS cluster to k8s 1.28 in aws
Validated the support config and licence entitlement functionality. Created a downstream EKS cluster with 21 nodes (with 1 entitlement) to trigger out-of-compliance message. Scaled down the cluster to 17 nodes, validated that the out-of-compliance message is no longer displayed. Scaled back up to 21 nodes, validated the out-of-compliance message popped up.

scenario 2 (fresh install):

On an EKS cluster (k8s version 1.28), install locally built rancher with cspAdapterMinVersion: 103.0.1+up3.0.1
Validated the support config and licence entitlement functionality. Created a downstream EKS cluster with 21 nodes (with 1 entitlement) to trigger out-of-compliance message. Scaled down the cluster to 17 nodes, validated that the out-of-compliance message is no longer displayed. Scaled back up to 21 nodes, validated the out-of-compliance message popped up.
Downgrade to 103.0.0+up3.0.0
Upgrade to 103.0.1+up3.0.1
Uninstall

Copy link

Validation steps

  • Ensure all container images have repository and tag on the same level to ensure that all container images are included in rancher-images.txt which are used by airgap customers.
  Ex:-
    longhorn-controller:
      repository: rancher/hardened-sriov-cni
      tag: v2.6.3-build20230913
  
  • Add a 👍 (thumbs up) reaction to this comment once done. CI won't pass without this reaction to the github-action bot's latest validation comment.
  • Approve the PR to run the CI check.

@mbelur mbelur requested a review from a team February 27, 2024 03:48
Copy link

Validation steps

  • Ensure all container images have repository and tag on the same level to ensure that all container images are included in rancher-images.txt which are used by airgap customers.
  Ex:-
    longhorn-controller:
      repository: rancher/hardened-sriov-cni
      tag: v2.6.3-build20230913
  
  • Add a 👍 (thumbs up) reaction to this comment once done. CI won't pass without this reaction to the github-action bot's latest validation comment.
  • Approve the PR to run the CI check.

Copy link

@guangyee guangyee left a comment

Choose a reason for hiding this comment

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

lgtm

@mbelur mbelur merged commit 311b612 into rancher:dev-v2.8 Feb 27, 2024
6 checks passed
lucasmlp pushed a commit that referenced this pull request Mar 24, 2024
lucasmlp pushed a commit that referenced this pull request Mar 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants