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

Initial cherry-picks for 1.4.2 #1875

Merged
merged 4 commits into from
Jul 17, 2018
Merged

Conversation

zjs
Copy link
Member

@zjs zjs commented Jul 3, 2018

Cherry-pick:

@zjs zjs self-assigned this Jul 3, 2018
@zjs zjs requested a review from hickeng July 3, 2018 18:09
reasonerjt and others added 4 commits July 17, 2018 12:57
…mware#1630) (vmware#1875)

After ca.crt and token.properties are copied to Harbor's directory the owner changed to root.
This became a problem after Harbor stop running chown in container.
This commit change the owner to 10000:10000 after it's copied.

(cherry picked from commit d42c6d9)
The builder OVF used to construct the OVA has hard-coded disk sizes.
Correct these to match the final size of the VMDKs to avoid warnings
during the OVA deployment process.

Refactor to pass image data from build-main to build-disks to avoid
duplication of image names.

(cherry picked from commit 62bccb8)
…e#1875)

As described in installer/docs/BUILD.md, our release process involves
several steps: a tag build, a staging build, and a release build.

Fully connecting these steps with automation is undesirable; we don't
want to stage all tag builds and we want to manually inspect staged
builds before deciding to release them.

However, requiring users to manually assemble the appropriate command
for the next step of the process is error prone. Instead, we output a
script containing the next command for them to copy and run.

(cherry picked from commit 840ed86)
…#1875)

As documented in installer/docs/RELEASE.md, our process is to include
filesize and checksum information when publishing a release on GitHub.

Output that information in a structured way during the build process
to ensure that accurate information can be easily posted.

Include this output for all published builds to allow integrity to be
verified in all cases.

(cherry picked from commit 4459076)
@zjs zjs force-pushed the releases/1.4.2 branch from 6c77067 to eeee81b Compare July 17, 2018 05:01
@zjs zjs merged commit 01961f2 into vmware:releases/1.4.2 Jul 17, 2018
zjs pushed a commit that referenced this pull request Jul 17, 2018
…1630) (#1875)

After ca.crt and token.properties are copied to Harbor's directory the owner changed to root.
This became a problem after Harbor stop running chown in container.
This commit change the owner to 10000:10000 after it's copied.

(cherry picked from commit d42c6d9)
zjs added a commit that referenced this pull request Jul 17, 2018
The builder OVF used to construct the OVA has hard-coded disk sizes.
Correct these to match the final size of the VMDKs to avoid warnings
during the OVA deployment process.

Refactor to pass image data from build-main to build-disks to avoid
duplication of image names.

(cherry picked from commit 62bccb8)
zjs added a commit that referenced this pull request Jul 17, 2018
As described in installer/docs/BUILD.md, our release process involves
several steps: a tag build, a staging build, and a release build.

Fully connecting these steps with automation is undesirable; we don't
want to stage all tag builds and we want to manually inspect staged
builds before deciding to release them.

However, requiring users to manually assemble the appropriate command
for the next step of the process is error prone. Instead, we output a
script containing the next command for them to copy and run.

(cherry picked from commit 840ed86)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants