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

[cherry-pick][v1.10] Remove container-builder-env section and add pr-containers action #5770

Merged
merged 2 commits into from
Jan 18, 2023

Conversation

blackpiglet
Copy link
Contributor

@blackpiglet blackpiglet commented Jan 16, 2023

  • Add PR container build action, which will not push the Velero image.
  • Add GOARM parameter in Restic docker builder section in Dockerfile.
  • Remove the container-builder-env section from Makefile.

Signed-off-by: Xun Jiang blackpiglet@gmail.com

Thank you for contributing to Velero!

Please add a summary of your change

Does your change fix a particular issue?

Fixes #(issue)

Please indicate you've done the following:

  • Accepted the DCO. Commits without the DCO will delay acceptance.
  • Created a changelog file or added /kind changelog-not-required as a comment on this pull request.
  • Updated the corresponding documentation in site/content/docs/main.

Signed-off-by: Xun Jiang <blackpiglet@gmail.com>
@blackpiglet blackpiglet added the kind/changelog-not-required PR does not require a user changelog. Often for docs, website, or build changes label Jan 16, 2023
@blackpiglet blackpiglet self-assigned this Jan 16, 2023
Signed-off-by: Xun Jiang <blackpiglet@gmail.com>
@blackpiglet blackpiglet removed the kind/changelog-not-required PR does not require a user changelog. Often for docs, website, or build changes label Jan 18, 2023
@blackpiglet blackpiglet changed the title [cherry-pick][v1.10] Remove container-builder-env section. [cherry-pick][v1.10] Remove container-builder-env section and add pr-containers action Jan 18, 2023
@codecov-commenter
Copy link

codecov-commenter commented Jan 18, 2023

Codecov Report

Merging #5770 (bc7d1d0) into release-1.10 (62287da) will not change coverage.
The diff coverage is n/a.

@@              Coverage Diff              @@
##           release-1.10    #5770   +/-   ##
=============================================
  Coverage         40.85%   40.85%           
=============================================
  Files               239      239           
  Lines             20663    20663           
=============================================
  Hits               8442     8442           
  Misses            11604    11604           
  Partials            617      617           

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@qiuming-best qiuming-best merged commit fa162a3 into vmware-tanzu:release-1.10 Jan 18, 2023
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