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

v0.38.4-RC1 #6249

Merged
merged 3 commits into from
May 21, 2020
Merged

v0.38.4-RC1 #6249

merged 3 commits into from
May 21, 2020

Conversation

alexanderbez
Copy link
Contributor

closes: #6006


For contributor use:

  • Targeted PR against correct branch (see CONTRIBUTING.md)
  • Linked to Github issue with discussion and accepted design OR link to spec that describes this work.
  • Code follows the module structure standards.
  • Wrote unit and integration tests
  • Updated relevant documentation (docs/) or specification (x/<module>/spec/)
  • Added relevant godoc comments.
  • Added a relevant changelog entry to the Unreleased section in CHANGELOG.md
  • Re-reviewed Files changed in the Github PR explorer

* Fix IncrementSequenceDecorator

* Update PR #

* Update godoc

* Add TestIncrementSequenceDecorator test
@codecov
Copy link

codecov bot commented May 19, 2020

Codecov Report

Merging #6249 into rc1/v0.38.4 will decrease coverage by 2.97%.
The diff coverage is 37.50%.

@@               Coverage Diff               @@
##           rc1/v0.38.4    #6249      +/-   ##
===============================================
- Coverage        54.37%   51.39%   -2.98%     
===============================================
  Files              313      336      +23     
  Lines            19035    20551    +1516     
===============================================
+ Hits             10350    10563     +213     
- Misses            7888     9185    +1297     
- Partials           797      803       +6     

@tac0turtle
Copy link
Member

Do you want to pull in the github action changes so we can run long sims here?

@alexanderbez
Copy link
Contributor Author

@marbar3778 add it to the meta-issue.

@tac0turtle
Copy link
Member

how long do the long sims usually run? it was run for 6 hours here, is that in line with before

@alexanderbez
Copy link
Contributor Author

how long do the long sims usually run? it was run for 6 hours here, is that in line with before

Depends on the infra. Back when we had dedicated AWS machines that we used via a slackbot, a few hours. Via Github Actions, no clue...

@tac0turtle
Copy link
Member

hmm github only allows 6 hours for an action, we may have to think of a way around this.


### Bug Fixes

* (x/auth) [\#5950](https://github.com/cosmos/cosmos-sdk/pull/5950) Fix `IncrementSequenceDecorator` to use is `IsReCheckTx` instead of `IsCheckTx` to allow account sequence incrementing.
Copy link
Collaborator

Choose a reason for hiding this comment

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

is this the only entry we need to add?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes -- see #6006. There were only two PRs added to this release.

@alexanderbez
Copy link
Contributor Author

hmm github only allows 6 hours for an action, we may have to think of a way around this.

We can either pay for infra or we can bump down the number of seeds/blocks we execute.

@tac0turtle
Copy link
Member

was the number of seeds/blocks chosen arbitrarily? we can reduce and split the seeds across two jobs

@alexanderbez
Copy link
Contributor Author

TBH, I don't recall how the long multi-seed sim works. Take a look at the Makefile target and see what it does. I'm pretty sure we just execute a bash script with a pre-configured set of seeds.

@alexanderbez
Copy link
Contributor Author

That being said, we can still release here and look into this separately.

@alessio
Copy link
Contributor

alessio commented May 20, 2020

https://github.com/cosmos/tools/blob/master/cmd/runsim/main.go#L32 - that's the default list that can be overridden by a comma-separated list of seeds passed via the -Seeds flag. For more info, please see runsim -h.

@alexanderbez alexanderbez merged commit ca5cbff into rc1/v0.38.4 May 21, 2020
@alexanderbez alexanderbez deleted the rc0/v0.38.4 branch May 21, 2020 13:54
@alessio
Copy link
Contributor

alessio commented May 21, 2020

Please remember to use git tag -a :)

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.

5 participants