Skip to content

arm64: add initial build-tools job with bootstap image #4187

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 3 commits into from
Jul 29, 2022
Merged

arm64: add initial build-tools job with bootstap image #4187

merged 3 commits into from
Jul 29, 2022

Conversation

howardjohn
Copy link
Member

This sets up the initial build. We need a bootstrap image at first,
since we have a circular dependency

@istio-testing istio-testing added the do-not-merge/work-in-progress Block merging of a PR because it isn't ready yet. label Jul 28, 2022
@istio-testing
Copy link
Collaborator

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

@istio-testing istio-testing added size/S Denotes a PR that changes 10-29 lines, ignoring generated files. needs-rebase Indicates a PR needs to be rebased before being merged size/M Denotes a PR that changes 30-99 lines, ignoring generated files. and removed size/S Denotes a PR that changes 10-29 lines, ignoring generated files. labels Jul 28, 2022
@howardjohn
arm64: add initial build-tools job with bootstap image
b8243aa 
This sets up the initial build. We need a bootstrap image at first,
since we have a circular dependency
@howardjohn
add presubmit as well
04c1913
@howardjohn howardjohn force-pushed the arch/tools-optional branch from 167b39d to 04c1913 Compare July 29, 2022 17:14
@howardjohn howardjohn marked this pull request as ready for review July 29, 2022 17:14
@howardjohn howardjohn requested a review from a team as a code owner July 29, 2022 17:14
@istio-testing istio-testing removed do-not-merge/work-in-progress Block merging of a PR because it isn't ready yet. needs-rebase Indicates a PR needs to be rebased before being merged labels Jul 29, 2022
@howardjohn
Copy link
Member Author

/retest

@istio-testing istio-testing merged commit 09bc33e into istio:master Jul 29, 2022
@istio-testing
Copy link
Collaborator

@howardjohn: Updated the job-config configmap in namespace default at cluster default using the following files:

  • key istio.tools.master.gen.yaml using file prow/cluster/jobs/istio/tools/istio.tools.master.gen.yaml

In response to this:

This sets up the initial build. We need a bootstrap image at first,
since we have a circular dependency

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
1 more reviewer

@stewartbutler stewartbutler stewartbutler approved these changes

Reviewers whose approvals may not affect merge requirements
Assignees
No one assigned
Labels
size/M Denotes a PR that changes 30-99 lines, ignoring generated files.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@howardjohn @istio-testing @stewartbutler