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

air-gap script should support customized tkg-bom tag #1367

Merged
merged 1 commit into from
Dec 14, 2021

Conversation

ZihanJiang96
Copy link
Contributor

@ZihanJiang96 ZihanJiang96 commented Dec 14, 2021

What this PR does / why we need it

  1. the current airgap script only downloads the tkg-bom image whose tag is equal to the tanzu-cli's version. we need a new way (a new env var TKG_BOM_IMAGE_TAG) to specify the tkg-bom tag we want to download and migrate.

  2. since we added some new tanzu-cli related packages to the tkg-bom, the airgap script needs to use the correct imgpkg copy flag to migrate them.

@ZihanJiang96 ZihanJiang96 requested a review from a team as a code owner December 14, 2021 00:58
@vmwclabot
Copy link

@ZihanJiang96, you must sign our contributor license agreement before your changes are merged. Click here to sign the agreement. If you are a VMware employee, read this for further instruction.

@anujc25
Copy link
Contributor

anujc25 commented Dec 14, 2021

@ZihanJiang96 can you please check this link for CLA requirements? https://cla.vmware.com/employee

@vmwclabot
Copy link

@ZihanJiang96, VMware has approved your signed contributor license agreement.

Copy link
Contributor

@anujc25 anujc25 left a comment

Choose a reason for hiding this comment

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

Looks good

@anujc25 anujc25 added area/docs ok-to-merge PRs should be labelled with this before merging labels Dec 14, 2021
@ZihanJiang96
Copy link
Contributor Author

please merge :) Thanks!

Copy link
Contributor

@vuil vuil left a comment

Choose a reason for hiding this comment

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

lgtm, thanks.

@vuil vuil merged commit f7f4662 into vmware-tanzu:main Dec 14, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
area/docs 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