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

chore: update mysten-infra and fastcrypto #5854

Merged
merged 1 commit into from
Nov 8, 2022
Merged

chore: update mysten-infra and fastcrypto #5854

merged 1 commit into from
Nov 8, 2022

Conversation

joyqvq
Copy link
Contributor

@joyqvq joyqvq commented Nov 4, 2022

No description provided.

@joyqvq joyqvq force-pushed the pointer-update branch 3 times, most recently from 8ff0604 to 302e73f Compare November 7, 2022 21:02
@joyqvq joyqvq marked this pull request as ready for review November 7, 2022 21:03
@joyqvq joyqvq requested review from benr-ml and jonas-lj November 7, 2022 21:03
@joyqvq joyqvq force-pushed the pointer-update branch 2 times, most recently from 1738843 to 78280ef Compare November 7, 2022 22:01
Copy link
Contributor

@huitseeker huitseeker left a comment

Choose a reason for hiding this comment

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

Thanks!

expression: committee
---
{
"authorities": {
"i9DpjC/kZbDd57csSg7qWFMF3uTECO2jBD3M5BOXU8T7AebTrMO+9D5IbnUI8N7lFhmjEiKFhwUOlFGbx1jkdP212PR/XvnayHxDaJuMkytxACBnshi9TrbZiZP10tqL": {
"ku8PylNwBUMzkshpDSeMnbV25O8rPkUFDPcqAyeEXztH02eOq6WemzFMEsi1p0LrBvKBvTLpwLwHhkCatYptihcO75iZRjtFkYwkAlv0GmQQyHZ+xO+UYweLQiYdFHtf": {
Copy link
Contributor Author

Choose a reason for hiding this comment

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

turns out this Rng change caused the config to change. it's been a while for me to remember, does this have any downstream impact on narwhal config other than updating this snapshot? cc @mwtian

Copy link
Contributor

Choose a reason for hiding this comment

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

I think nothing else needs to happen. Even if a genesis is needed, it should happen as part of the release process.

@joyqvq joyqvq force-pushed the pointer-update branch 3 times, most recently from 6a9a1a9 to 4dc0036 Compare November 8, 2022 01:09
@joyqvq joyqvq enabled auto-merge (squash) November 8, 2022 02:04
@joyqvq joyqvq merged commit 05d2fad into main Nov 8, 2022
@joyqvq joyqvq deleted the pointer-update branch November 8, 2022 02:16
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.

3 participants