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

Change default AVI_CONTROLLER_VERSION to 20.1.3. #2931

Conversation

HanFa
Copy link
Contributor

@HanFa HanFa commented Jul 13, 2022

What this PR does / why we need it

We cannot use the newer 20.1.7 Avi controller version yet since the downstream testbed is still using the older version 20.1.3. This version mismatch will cause the applied ADC to be rejected by its admission webhook since the ako-operator cannot talk to the Avi controller (400 invalid version).

It reverts the changes by https://github.com/vmware-tanzu/tanzu-framework/pull/2827/files here.

Which issue(s) this PR fixes

Fixes # #2932

Describe testing done for PR

Release note

Change default AVI_CONTROLLER_VERSION to 20.1.3.

PR Checklist

  • Squash the commits into one or a small number of logical commits
  • Use good commit messages
  • Ensure PR contains terms all contributors can understand and links all contributors can access

Additional information

Special notes for your reviewer

@HanFa HanFa requested review from a team as code owners July 13, 2022 21:14
@github-actions
Copy link

Cluster Generation A/B Results:
https://storage.googleapis.com/tkg-clustergen/2931/20220713212624/clustergen.diff.txt
Author/reviewers:
Please review to verify that the effects on the generated cluster configurations are exactly what the PR intended, and give a thumbs-up if so.

@vijaykatam vijaykatam added the ok-to-merge PRs should be labelled with this before merging label Jul 13, 2022
@HanFa HanFa merged commit 5e8076d into vmware-tanzu:main Jul 14, 2022
pgandigesang pushed a commit to pgandigesang/tanzu-framework that referenced this pull request Jul 15, 2022
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Labels
cla-not-required ok-to-merge PRs should be labelled with this before merging
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants