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

Add nightly CI check against latest substrate. #335

Merged
merged 16 commits into from
Nov 30, 2021
Merged

Add nightly CI check against latest substrate. #335

merged 16 commits into from
Nov 30, 2021

Conversation

jsdw
Copy link
Collaborator

@jsdw jsdw commented Nov 29, 2021

Rel #334

uses: actions-rs/[email protected]
with:
command: test
args: --all-targets --workspace
Copy link
Member

Choose a reason for hiding this comment

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

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Ooh, great idea! I'll look at that :)

@jsdw
Copy link
Collaborator Author

jsdw commented Nov 29, 2021

I realised that the reason that the doc step was not finishing was because the tokio1 features was missing from async-std and causing a panic that seemed to block the build step from progressing (I'm not yet sure why it didn't just bail!). That's fixed now.

Edit: Making sure to kill the substrate process on panic (with a custom drop impl) seems to have resolved the issue of panics blocking completion)

@jsdw jsdw changed the title Add nightly CI check against latest substrate. PRs to run against selected substrate Add nightly CI check against latest substrate. Nov 30, 2021
@jsdw
Copy link
Collaborator Author

jsdw commented Nov 30, 2021

Just FYI: this issue was opened to address the question of "which substrate binary to pin to for PR runs": https://github.com/paritytech/ci_cd/issues/272

Copy link
Contributor

@ascjones ascjones left a comment

Choose a reason for hiding this comment

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

LGTM

@jsdw jsdw marked this pull request as ready for review November 30, 2021 14:55
@jsdw jsdw merged commit 4ca50ef into master Nov 30, 2021
@jsdw jsdw deleted the jsdw-nightly-ci branch November 30, 2021 15:01
0623forbidden pushed a commit to DEIPworld/substrate-subxt that referenced this pull request Feb 15, 2022
* Add nightly check and use specific version (currently the same) in PR checks

* wrong var name, whoops

* Create GH issue if nightly run fails

* need tokio feature for RpcClient; match dependency in root Cargo.toml.

* Ensure substrate command is properly cleaned up in the event of a panic, to prevent blocking

* increase retry limit again

* fmt

* Tweak text for nightly-run-failed issue template

* Run nightly-CI every 5mins for testing

* I clearly dont use cron enough

* Put back cron timings to 8am daily

* break a test

* revert nightly workflow again (doesnt run off master) and testing rust workflow

* more testing..

* undo testing things; seems to work

* tidy up; substrate url at top of both CI files
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants