-
Notifications
You must be signed in to change notification settings - Fork 3k
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
Run e2e performance regression tests on merge into main
via AWS Device farm
#12320
Merged
Merged
Changes from all commits
Commits
Show all changes
109 commits
Select commit
Hold shift + click to select a range
b857f73
Apply some logging tweaks
AndrewGable 106a88b
Add TestSpec
AndrewGable 9ce17a6
Tweak TestSpec to a working solution
AndrewGable b06342f
Tweak artifacts
AndrewGable 7bec64d
Move e2e to tests folder and remove unused code
AndrewGable 6c255fb
Try making run from GitHub actions
AndrewGable 1631bea
Use correct version
AndrewGable cb30060
Use `workflow_dispatch` as well
AndrewGable 30d1c33
Use macos for Bundle
AndrewGable 579141e
Delete unused workflows to clean up my inbox
AndrewGable 478ee01
Run on push
AndrewGable 48c9f8c
Set up ruby and include device pool
AndrewGable 574b8fb
Tweak android build settings
AndrewGable 84b94e4
Add debug lines
AndrewGable 90f41cc
Correct env file location
AndrewGable 57dc2d7
Tweak two configs to pass tests
AndrewGable 1d3a54b
Remove debug
AndrewGable 0633815
Tweak install path for adb
AndrewGable a57301e
Show all files
AndrewGable 6b53c5d
Add find
AndrewGable 278350d
Try to fix file paths
AndrewGable 1a95f0a
Try to grab artifacts from device farm
AndrewGable 3e9deb0
Try to see if this improves perf of build
AndrewGable 4ad6c44
Test cache
AndrewGable 4bd55e1
Try to use straight npm ci
AndrewGable 1ffc8d0
Try different path for artifacts
AndrewGable 665296a
Try one last path
AndrewGable 540f36b
Clean up TestSpec
AndrewGable 2048bd7
Try to save to `WORKING_DIRECTORY`
AndrewGable f090796
Add debug lines
AndrewGable 3b5ad5a
Try to leave as a comment
AndrewGable e472470
Add \
AndrewGable e3e4456
Remove debug
AndrewGable 01ea314
Try to use ./
AndrewGable 6711db7
Try ubuntu
AndrewGable 827de9e
More debug
AndrewGable 3bba04e
Escape $
AndrewGable 5e0bf35
Fix env too
AndrewGable b2a7bb6
Last fix?
AndrewGable ebd543c
Use quotes?
AndrewGable 9c3c4c5
Use multi line env variable
AndrewGable f985b4f
Try a different comment action
AndrewGable 47cdafe
Clean up debug
AndrewGable fa5049f
Try to build two APKs
AndrewGable 68b2c1a
Tweak checkout command
AndrewGable f39aa10
Merge branch 'main' into andrew-aws-df
AndrewGable e861d70
Update file names
AndrewGable 79aea8e
Add param
AndrewGable 862c7fb
Merge branch 'main' into andrew-aws-df
AndrewGable 0f6f593
Current -> Compare
AndrewGable 7d002b1
Try to use npm install from GitHub runner
AndrewGable 7baa3e5
Debug
AndrewGable 1e91780
Try npm-bundle
AndrewGable 47b9069
More debug
AndrewGable 543aed0
Testing bundle in different spot
AndrewGable 8c22a84
Try a debug
AndrewGable a19e5aa
Undo debug
AndrewGable 11e298f
More debug
AndrewGable a7e89bb
Use custom fork to try to fix bug
AndrewGable c9a1dbc
Use zip instead of bundle
AndrewGable 95da732
Clean up after runs
AndrewGable ae7278b
Install correct node and do not switch branches
AndrewGable 2a86245
Quick debug test
AndrewGable 1fae6ec
Undo debug
AndrewGable 31220d7
Fix path
AndrewGable eb7da36
Checkout preivous branch
AndrewGable 5b314b0
Remove debug lines
AndrewGable 573c24b
Quiet zip and use npmrc file
AndrewGable 899ea8e
Undo removal of other workflows
AndrewGable 7c7cb99
Filter outliers via different method
AndrewGable 769b8ed
Bump formatting to 3 digits
AndrewGable b1d95ca
Reduce test runs by half, see how long it takes
AndrewGable 5e4f62b
Undo reducing number of runs
AndrewGable 8bd22d8
Use large runner
AndrewGable dbdc9fd
Only zip up required e2e tests
AndrewGable ff9f04f
Update TestSpec
AndrewGable 62ca8de
Merge branch 'main' into andrew-aws-df
AndrewGable 4d2b888
Merge branch 'main' into andrew-aws-df
AndrewGable 17c693e
Make a zip directory
AndrewGable 7b72a08
fix path
AndrewGable 9a34e71
Add recursive flag
AndrewGable ba07fed
Fix another path
AndrewGable 5cd62d9
Tweak commands now that we have a stripped down test
AndrewGable 6e5853c
Remove fork
AndrewGable d1bc36b
Fix path again and more debug
AndrewGable 8caab9a
Try to install underscore elsewhere
AndrewGable d6700e6
Remove some debug lines
AndrewGable 46b2a1e
Fix lint
AndrewGable 0255d3f
Fix lint
AndrewGable 911864a
Try to fix lint more
AndrewGable 82cadee
Try one last thing for linting
AndrewGable a493465
Move back to runs-on
AndrewGable 1d7de1d
Skip linting specific workflow due to bug
AndrewGable 11568bf
Forgot $
AndrewGable 789e695
Merge branch 'main' into andrew-aws-df
AndrewGable 32e12b8
Re-add checkout
AndrewGable a9ca824
Use my branch
AndrewGable 5381ce1
Add test for fail/pass
AndrewGable 08202e4
Tweak syntax
AndrewGable e333c47
Adjust test
AndrewGable 5d63147
Fix lint and remove unused file
AndrewGable 98e836f
Checkout latest release as baseline
AndrewGable 1db38a8
Add "
AndrewGable 92a52ad
One more test
AndrewGable 90b9ed9
More testing
AndrewGable 372aa2e
Remove test file
AndrewGable e679eef
Move checkout back
AndrewGable e11332d
Merge branch 'main' into andrew-aws-df
AndrewGable df5905f
Apply peer review
AndrewGable File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,4 @@ | ||
# See https://github.com/rhysd/actionlint/blob/main/docs/config.md | ||
self-hosted-runner: | ||
labels: | ||
- ubuntu-20.04-64core |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file was deleted.
Oops, something went wrong.
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -93,4 +93,4 @@ storybook-static | |
.jest-cache | ||
|
||
# E2E test reports | ||
e2e/.results/ | ||
tests/e2e/.results/ |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file was deleted.
Oops, something went wrong.
This file was deleted.
Oops, something went wrong.
This file was deleted.
Oops, something went wrong.
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
File renamed without changes.
File renamed without changes.
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,26 @@ | ||
version: 0.1 | ||
|
||
phases: | ||
install: | ||
commands: | ||
# Install correct version of node | ||
- export NVM_DIR=$HOME/.nvm | ||
- . $NVM_DIR/nvm.sh | ||
- nvm install 16.15.1 | ||
- nvm use 16.15.1 | ||
|
||
# Reverse ports using AWS magic | ||
- PORT=4723 | ||
- IP_ADDRESS=$(ip -4 addr show eth0 | grep -Po "(?<=inet\s)\d+(\.\d+){3}") | ||
- reverse_values="{\"ip_address\":\"$IP_ADDRESS\",\"local_port\":\"$PORT\",\"remote_port\":\"$PORT\"}" | ||
- "curl -H \"Content-Type: application/json\" -X POST -d \"$reverse_values\" http://localhost:31007/reverse_forward_tcp" | ||
- adb reverse tcp:$PORT tcp:$PORT | ||
|
||
test: | ||
commands: | ||
- cd zip | ||
- npm install underscore | ||
- node e2e/testRunner.js -- --skipInstallDeps --skipBuild | ||
|
||
artifacts: | ||
- $WORKING_DIRECTORY |
File renamed without changes.
File renamed without changes.
File renamed without changes.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I see what you mean about this action being pointless now ... annoying that local composite actions from the same repo don't have access to secrets.