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

improve package-release.sh by using ENVS variable #2385

Conversation

joefitzgerald
Copy link
Contributor

Signed-off-by: Joe Fitzgerald jfitzgerald@vmware.com

What this PR does / why we need it

This PR improves the build script to remove duplication in package-release.sh by leveraging the ENVS variable set in the Makefile.

  • pass ENVS to package-release.sh
  • remove duplicate instructions for each architecture, and instead loop through each element in ENVS

Details for the Release Notes (PLEASE PROVIDE)

improve package-release.sh by leveraging the ENVS variable during builds

Which issue(s) this PR fixes

N/A

Describe testing done for PR

$ make release

Special notes for your reviewer

N/A

- pass ENVS to package-release.sh
- remove duplicate instructions for each architecture, and instead loop through each element in ENVS

Signed-off-by: Joe Fitzgerald <jfitzgerald@vmware.com>
@joefitzgerald joefitzgerald requested a review from a team as a code owner October 28, 2021 04:18
@github-actions github-actions bot added the owner/release-eng Work executed by VMware release engineering team label Oct 28, 2021
Signed-off-by: Joe Fitzgerald <jfitzgerald@vmware.com>
Copy link
Contributor

@davidvonthenen davidvonthenen left a comment

Choose a reason for hiding this comment

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

Also, please check to make sure that make release-docker works since that is what the final release process uses.

Signed-off-by: Joe Fitzgerald <jfitzgerald@vmware.com>
Signed-off-by: Joe Fitzgerald <jfitzgerald@vmware.com>
@davidvonthenen davidvonthenen merged commit f6deb87 into vmware-tanzu:main Oct 28, 2021
@joefitzgerald joefitzgerald deleted the joefitzgerald/improve-package-release branch October 28, 2021 18:28
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
cla-not-required owner/release-eng Work executed by VMware release engineering team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants