Skip to content
This repository has been archived by the owner on Nov 15, 2023. It is now read-only.

ci: always run runtime migration checks #7422

Merged
merged 13 commits into from
Jul 24, 2023

Conversation

liamaharon
Copy link
Contributor

@liamaharon liamaharon commented Jun 22, 2023

Runtime migration checks are much more stable now, FRAME team would like to get to a point where they are non-optional on every PR.

We can start by running the checks on every PR, and once paritytech/substrate#14311 is merged and we fix some recently introduced issues get them all green.

Once they've been green for a while and we're confident in their stability we can disable allow_failure.

blocked by

@liamaharon liamaharon requested review from a team and chevdor as code owners June 22, 2023 16:36
@paritytech-ci paritytech-ci requested a review from a team June 22, 2023 16:36
@liamaharon liamaharon added A0-please_review Pull request needs code review. B0-silent Changes should not be mentioned in any release notes C1-low PR touches the given topic and has a low impact on builders. labels Jun 22, 2023
@liamaharon liamaharon marked this pull request as draft June 22, 2023 16:56
@liamaharon liamaharon marked this pull request as ready for review June 22, 2023 17:08
@liamaharon liamaharon marked this pull request as draft June 22, 2023 17:21
@liamaharon liamaharon removed request for a team and chevdor June 22, 2023 17:25
@liamaharon liamaharon added A3-in_progress Pull request is in progress. No review needed at this stage. S4-blocked PR/Issue is blocked, see comments for further information. and removed A0-please_review Pull request needs code review. labels Jun 22, 2023
@liamaharon liamaharon marked this pull request as ready for review June 26, 2023 14:01
@paritytech-ci paritytech-ci requested review from a team June 26, 2023 14:02
@liamaharon liamaharon added A0-please_review Pull request needs code review. and removed A3-in_progress Pull request is in progress. No review needed at this stage. S4-blocked PR/Issue is blocked, see comments for further information. labels Jun 26, 2023
scripts/ci/gitlab/pipeline/check.yml Outdated Show resolved Hide resolved
@paritytech-ci paritytech-ci requested a review from a team June 26, 2023 14:27
scripts/ci/gitlab/pipeline/check.yml Outdated Show resolved Hide resolved
Co-authored-by: Alexander Samusev <41779041+alvicsam@users.noreply.github.com>
Copy link
Member

@ggwpez ggwpez left a comment

Choose a reason for hiding this comment

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

LGTM

@ggwpez
Copy link
Member

ggwpez commented Jul 24, 2023

The rococo wont turn green before .43 is applied, or?
I guess we can merge then and remove the allow-failure afterwards?

@liamaharon
Copy link
Contributor Author

bot merge

@paritytech-processbot paritytech-processbot bot merged commit 5e7addf into master Jul 24, 2023
@paritytech-processbot paritytech-processbot bot deleted the liam-always-run-runtime-migration-ci branch July 24, 2023 15:10
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A0-please_review Pull request needs code review. B0-silent Changes should not be mentioned in any release notes C1-low PR touches the given topic and has a low impact on builders.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants