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

etcd-e2d-test:rename e2e-arm64 file and runs it on every pull request #16912

Closed
wants to merge 14 commits into from

Conversation

mingli103
Copy link
Contributor

@mingli103 mingli103 commented Nov 10, 2023

KubeconNA 2023 Contribfest issue #16893 . Recently in #16801 we introduced on demand github actions runners for the arm64 platform. Having on demand runner infrastructure in place means we should now have enough capacity to begin running arm64 tests for every pull request. Currently we have:
.github/workflows/e2e-arm64-template.yaml - Shared template .github/workflows/e2e-arm64-nightly.yaml - Runs template against both release-3.5 and main branches nightly. Moving forward we can just rename .github/workflows/e2e-arm64-template.yaml to .github/workflows/e2e-arm64.yaml and delete the other file. We can then just make the template file a standard workflow that will run on pull request.

Please read https://github.com/etcd-io/etcd/blob/main/CONTRIBUTING.md#contribution-flow.

Copy link
Member

@jmhbnz jmhbnz left a comment

Choose a reason for hiding this comment

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

Thanks for working on this @mingli103 - A suggestion below about how we handle the branching 🙏🏻

@@ -20,7 +15,7 @@ jobs:
steps:
- uses: actions/checkout@b4ffde65f46336ab88eb53be808477a3936bae11 # v4.1.1
with:
ref: "${{ inputs.etcdBranch }}"
ref: "main"
Copy link
Member

Choose a reason for hiding this comment

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

Rather than hardcoding main I suggest we remove the with: completely for actions/checkout so we use the default behaviour of only a single commit being fetched for the ref/SHA that triggered the workflow.

This will allow us to safely backport these changes to release-3.5 branch.

@mingli103 mingli103 requested a review from jmhbnz November 11, 2023 02:30
@@ -1,11 +1,6 @@
---
name: Reusable Arm64 E2E Workflow
Copy link
Member

Choose a reason for hiding this comment

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

Just noticed we should probably align this name with the existing e2e per pull request workflow which is E2E: https://github.com/etcd-io/etcd/blob/main/.github/workflows/e2e.yaml#L2

So we could make this E2E-Arm64.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Make sense.

@mingli103 mingli103 requested a review from jmhbnz November 11, 2023 05:09
Copy link
Member

@jmhbnz jmhbnz left a comment

Choose a reason for hiding this comment

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

LGTM on green - Thanks @mingli103

@jmhbnz
Copy link
Member

jmhbnz commented Nov 13, 2023

Hey @mingli103 - Can you please rebase and force push this branch? We had some flaky tests recently and if you can rebase onto latest commits from main your branch will get some fixes so tests will hopefully pass.

Recently in etcd-io#16801 we introduced on demand github actions runners for the arm64 platform.
Having on demand runner infrastructure in place means we should now have enough capacity to begin running arm64 tests for every pull request.
Currently we have:
.github/workflows/e2e-arm64-template.yaml - Shared template
.github/workflows/e2e-arm64-nightly.yaml - Runs template against both release-3.5 and main branches nightly.
Moving forward we can just rename .github/workflows/e2e-arm64-template.yaml to .github/workflows/e2e-arm64.yaml and delete the other file. We can then just make the template file a standard workflow that will run on pull request.

Signed-off-by: Ming Li <mli103hawk@gmail.com>
we use the default behaviour of only a single commit being fetched for the ref/SHA that triggered the workflow.

Signed-off-by: Ming Li <mli103hawk@gmail.com>
Align this name with the existing e2e per pull request workflow

Signed-off-by: Ming Li <mli103hawk@gmail.com>
@mingli103 mingli103 marked this pull request as draft November 15, 2023 22:06
@mingli103 mingli103 marked this pull request as ready for review November 15, 2023 22:42
mingli103 added a commit to mingli103/etcd that referenced this pull request Nov 15, 2023
…tcd-io#16912

KubeconNA 2023 Contribfest issue etcd-io#16893 . Recently in etcd-io#16801 we introduced on demand github actions runners for the arm64 platform. Having on demand runner infrastructure in place means we should now have enough capacity to begin running arm64 tests for every pull request. Currently we have:
.github/workflows/e2e-arm64-template.yaml - Shared template
.github/workflows/e2e-arm64-nightly.yaml - Runs template against both
release-3.5 and main branches nightly. Moving forward we can just rename
.github/workflows/e2e-arm64-template.yaml to
.github/workflows/e2e-arm64.yaml and delete the other file. We can then
just make the template file a standard workflow that will run on pull
request.

Signed-off-by: Ming Li <mli103hawk@gmail.com>
serathius and others added 10 commits November 18, 2023 10:10
Signed-off-by: Marek Siarkowicz <siarkowicz@google.com>
…0 to 2.18.1

Signed-off-by: Sharath Sivakumar <mailssr9@gmail.com>
Signed-off-by: Sharath Sivakumar <mailssr9@gmail.com>
Signed-off-by: Sharath Sivakumar <mailssr9@gmail.com>
Signed-off-by: Sharath Sivakumar <mailssr9@gmail.com>
Signed-off-by: Sharath Sivakumar <mailssr9@gmail.com>

Co-authored-by: Benjamin Wang <wachao@vmware.com>
Missing binary, fixes golangci-lint command not found on prow jobs

Signed-off-by: Etienne Simard <esimard@ripple.com>
Signed-off-by: Benjamin Wang <benjamin.ahrtr@gmail.com>
…-action

Signed-off-by: Benjamin Wang <benjamin.ahrtr@gmail.com>
…stall-golangci-lint

Signed-off-by: Benjamin Wang <benjamin.ahrtr@gmail.com>
Signed-off-by: arjunmalhotra1 <am2cj@virginia.edu>
@mingli103 mingli103 marked this pull request as draft November 18, 2023 16:11
mingli103 added a commit to mingli103/etcd that referenced this pull request Nov 18, 2023
…tcd-io#16912

KubeconNA 2023 Contribfest issue etcd-io#16893 . Recently in etcd-io#16801 we introduced on demand github actions runners for the arm64 platform. Having on demand runner infrastructure in place means we should now have enough capacity to begin running arm64 tests for every pull request. Currently we have:
.github/workflows/e2e-arm64-template.yaml - Shared template
.github/workflows/e2e-arm64-nightly.yaml - Runs template against both
release-3.5 and main branches nightly. Moving forward we can just rename
.github/workflows/e2e-arm64-template.yaml to
.github/workflows/e2e-arm64.yaml and delete the other file. We can then
just make the template file a standard workflow that will run on pull
request.

Signed-off-by: Ming Li <mli103hawk@gmail.com>
@jmhbnz
Copy link
Member

jmhbnz commented Nov 23, 2023

Hey @mingli103 - It looks like this pr is rewriting history. Can you please retry the rebase so your pr only includes your single commit?

@mingli103 mingli103 closed this Nov 23, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

7 participants