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

NO-TICKET: Fix era cleanup. #1038

Merged
merged 2 commits into from
Mar 1, 2021
Merged

NO-TICKET: Fix era cleanup. #1038

merged 2 commits into from
Mar 1, 2021

Conversation

afck
Copy link
Contributor

@afck afck commented Mar 1, 2021

The code accidentally cleaned up the current era instead of the old one.

The code accidentally cleaned up the current era instead of the old one.
@afck
Copy link
Contributor Author

afck commented Mar 1, 2021

bors r+

@bors
Copy link
Contributor

bors bot commented Mar 1, 2021

Build failed:

@afck
Copy link
Contributor Author

afck commented Mar 1, 2021

As discussed, we ignore RUSTSEC-2020-0146 for now.

@afck
Copy link
Contributor Author

afck commented Mar 1, 2021

bors r+

@afck
Copy link
Contributor Author

afck commented Mar 1, 2021

I created NDRS-956 for RUSTSEC-2020-0146.

@bors
Copy link
Contributor

bors bot commented Mar 1, 2021

Build succeeded:

@bors bors bot merged commit 7003c6e into casper-network:master Mar 1, 2021
@afck afck deleted the evidence_only branch March 1, 2021 16:32
@piotr-dziubecki piotr-dziubecki added this to the Mainnet milestone Mar 4, 2021
bors bot added a commit that referenced this pull request Mar 4, 2021
1052: NDRS-955: Make tests include unbonding and dropping eras. r=afck a=afck

This changes the configuration used in the reactor tests so that the 5-era tests cause eras to become unbonded and dropped in the era validator.

This would have made the tests catch the bug fixed in #1038.

https://casperlabs.atlassian.net/browse/NDRS-955

1064: OP-1882: refactor itst scenarios r=goral09 a=TomVasile

Moves common functions used in itst scenarios into a common sourceable script.

Co-authored-by: Andreas Fackler <andreas@casperlabs.io>
Co-authored-by: TomVasile <43349666+TomVasile@users.noreply.github.com>
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.

None yet

3 participants