-
Notifications
You must be signed in to change notification settings - Fork 19
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
Switch workflows to new EKS runners #1362
Comments
# Goal The goal of this PR is to switch Verify PR workflow to the new EKS runners upon successful testing. Part of #1362
While testing release workflow on the new EKS runners, I’ve run into this error in “Build Deterministic WASM” job that also exists on main and will pop-up next time someone cuts a release:
The failed job is “Build Deterministic WASM” : https://github.com/LibertyDSNP/frequency/actions/runs/4853427916/jobs/8650492538 We need #1449 to be resolved first. Also, per @saraswatpuneet and @wilwade, upgrading the polkadot version should include updating rust and toolchain if needed. |
# Goal The goal of this PR is switch Merge PR workflow to the new EKS runners. Part of #1362
Another error that may be resolved by #1449: The “Install Subwasm” step currently fails in CI Release workflow with this error: error[E0658]: sym operands for inline assembly are unstable Here is the failed job: https://github.com/LibertyDSNP/frequency/actions/runs/5006563355/jobs/8972114967 |
Ran test |
Per @wilwade, he will attempt to fix this error on main. Waiting until then to re-run test release. |
# Goal The goal of this PR is to upgrade the polkadotjs tools for metadata comparison. Closes #1230 This also includes PR #1601 for expediency. So also part of #1362 Ended up needing to run each one in a docker container with a new docker network. This allowed everything to talk correctly. # Checklist - [x] Tested with: make start + docker run --rm --platform=linux/amd64 --pull always jacogr/polkadot-js-tools:0.55.3 metadata wss://1.rpc.frequency.xyz ws://host.docker.internal:9944 - [x] Tested with test release Running: https://github.com/LibertyDSNP/frequency/actions/runs/5282806513 --------- Co-authored-by: Dmitri <4452412+demisx@users.noreply.github.com>
Created https://www.pivotaltracker.com/story/show/185631909 story to update runner labels. Waiting on @saisomepalle to respond. Need to coordinate the switch together with #1636. |
# Goal The goal of this PR is to update runner labels to their long-term stable versions. Part of #1362
This issue has been completed. |
# Goal The goal of this PR is to upgrade the polkadotjs tools for metadata comparison. Closes #1230 This also includes PR #1601 for expediency. So also part of #1362 Ended up needing to run each one in a docker container with a new docker network. This allowed everything to talk correctly. # Checklist - [x] Tested with: make start + docker run --rm --platform=linux/amd64 --pull always jacogr/polkadot-js-tools:0.55.3 metadata wss://1.rpc.frequency.xyz ws://host.docker.internal:9944 - [x] Tested with test release Running: https://github.com/LibertyDSNP/frequency/actions/runs/5282806513 --------- Co-authored-by: Dmitri <4452412+demisx@users.noreply.github.com>
# Goal The goal of this PR is to update runner labels to their long-term stable versions. Part of #1362
The new runners have been successfully tested with Verify PR workflow. We need to initiate the switch.
Tasks:
v1
label to the legacy runners update runner labels #1636testing, v2
->build, v2
label update runner labels #1636Ask devOps to disable legacy runnersStill needed for couple of jobsBlocked by:
The text was updated successfully, but these errors were encountered: