Skip to content

Commit 964f9cd

Browse files
author
Ryan Lymburner
authored
Add E2E Merge Queue Support (aws#707)
1 parent 6b14571 commit 964f9cd

File tree

4 files changed

+14
-9
lines changed

4 files changed

+14
-9
lines changed

.github/PULL_REQUEST_TEMPLATE.md

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -56,4 +56,12 @@ from users switching to the new release, include the string "action required".
5656
5757
```
5858

59+
**Do all end-to-end tests successfully pass when running `make e2e-test`?**:
60+
<!--
61+
Please provide a snippet of a successful `make e2e-test` run to confirm.
62+
-->
63+
```
64+
65+
```
66+
5967
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

.github/workflows/e2e-test.yaml

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -3,10 +3,8 @@ permissions:
33
id-token: write
44
contents: read
55
on:
6-
# TODO: Find a secure enough way to run e2e tests from PR on-demand.
7-
push:
8-
branches:
9-
- main
6+
merge_group:
7+
branches: [ "main" ]
108
workflow_dispatch:
119
jobs:
1210
e2e-test:

.github/workflows/presubmit.yaml

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,9 +1,7 @@
11
name: presubmit
22
on:
3-
push:
4-
branches:
5-
- main
63
pull_request:
4+
branches: [ "main" ]
75
workflow_dispatch:
86
jobs:
97
golangci-lint:

test/suites/integration/ram_share_test.go

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -52,10 +52,11 @@ var _ = Describe("RAM Share", Ordered, func() {
5252
secondaryTestRoleArn = os.Getenv("SECONDARY_ACCOUNT_TEST_ROLE_ARN")
5353

5454
retryer := awsClient.DefaultRetryer{
55-
MinThrottleDelay: 1 * time.Second,
56-
MinRetryDelay: 1 * time.Second,
55+
MinThrottleDelay: 500 * time.Millisecond,
56+
MinRetryDelay: 500 * time.Millisecond,
5757
MaxThrottleDelay: 5 * time.Second,
5858
MaxRetryDelay: 5 * time.Second,
59+
NumMaxRetries: 5,
5960
}
6061

6162
primarySess := session.Must(session.NewSession(&aws.Config{

0 commit comments

Comments
 (0)