-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
Lotus Miner & Node v1.28.2 Release #12379
Comments
Scribbling down some notes while I go through this process and then we can consider. Keep in mind that this is a patch-release, and therefor a little outside the normal release flow:
This is only relevant in the case when shipping a normal release (i.e not a patch release). For patch releases this would be:
|
This is kind of in a weird spot, since we are asking for opening a PR - but unless you have any immediate changes to the RELEASE_ISSUE_TEMPLATE.md file you can´t open a PR. |
@rjan90 : not in the critical path of the release itself, but FYI concerning template improvements:
|
…lease) (#12400) * fix: lotus-miner: remove provecommit1 method (#12251) * remove provecommit1 * add changelog * update precommit and commit params * fix lint error * fix commit params * dep: f3: Update go-f3 to 0.0.6, enable it on mainnet (#12295) * Update go-f3 to 0.0.6 Signed-off-by: Jakub Sztandera <oss@kubuxu.com> * Enable F3 in passive configuration in mainnet config Signed-off-by: Jakub Sztandera <oss@kubuxu.com> * Add changelog Signed-off-by: Jakub Sztandera <oss@kubuxu.com> * add new butterfly assets --------- Signed-off-by: Jakub Sztandera <oss@kubuxu.com> Co-authored-by: Jennifer Wang <jiayingw703@gmail.com> * retract v1.28.0 * update v1.28.0 changelog and add v1.28.1 * Update CHANGELOG.md * wip - update f3 * don't convert bigint type We now use the same one in GPBFT. * update docs * fix wrong param name * update butterfy assets * update go-f3 * update changelog * update version * fix typo * Update CHANGELOG.md Co-authored-by: Steven Allen <steven@stebalien.com> * Update CHANGELOG.md Co-authored-by: Rod Vagg <rod@vagg.org> * Update CHANGELOG.md Co-authored-by: Rod Vagg <rod@vagg.org> * apply f3 patch * chore: bump versions and make gen/docsgen-cli chore: bump versions and make gen/docsgen-cli * chore: update v1.28.2 changelog chore: update v1.282. changelog * feat: f3: update go-f3 to 0.2.0 (#12390) * Update go-f3 to 0.2.0 Includes: - fix for excessive bandwidth usage - significant performance improvements - minor consensus fixes Signed-off-by: Jakub Sztandera <oss@kubuxu.com> * add changelog Signed-off-by: Jakub Sztandera <oss@kubuxu.com> * chore(f3): update to final released version --------- Signed-off-by: Jakub Sztandera <oss@kubuxu.com> Co-authored-by: Steven Allen <steven@stebalien.com> * fix!: sealer: handle initialisation error without panic storage/pipeline.NewPreCommitBatcher and storage/pipeline.New now have an additional error return to deal with errors arising from fetching the sealing config. * add breaking API upgrade warning to the ChangeLog * NewCommitBatcher now has an additional error return to deal with errors arising from fetching the sealing config. * fix: miner: Fix DDO pledge math (#12341) * Power is units of Space * Time so multiply by deal duration * fix: miner: Fix DDO pledge math * appease the changelog checker * Fix gen --------- Co-authored-by: zenground0 <ZenGround0@users.noreply.github.com> * chore: fix lint error - Updated the logging statement in `testOutOfGasError` to correctly reference `build.BlockGasLimit` instead of `buildconstants.BlockGasLimit`. * fix: update changelog to reference bandwidth issue ticket fix: update changelog to reference bandwidth issue ticket * Update CHANGELOG.md Co-authored-by: Steve Loeppky <biglep@filoz.org> * Update CHANGELOG.md * chore: make gen and make docsgen-cli Run `make gen` and `make docsgen-cli` --------- Signed-off-by: Jakub Sztandera <oss@kubuxu.com> Co-authored-by: LexLuthr <88259624+LexLuthr@users.noreply.github.com> Co-authored-by: Jakub Sztandera <oss@kubuxu.com> Co-authored-by: Jennifer Wang <jiayingw703@gmail.com> Co-authored-by: Jiaying Wang <42981373+jennijuju@users.noreply.github.com> Co-authored-by: Steven Allen <steven@stebalien.com> Co-authored-by: Rod Vagg <rod@vagg.org> Co-authored-by: aarshkshah1992 <aarshkshah1992@gmail.com> Co-authored-by: Łukasz Magiera <magik6k@users.noreply.github.com> Co-authored-by: zenground0 <ZenGround0@users.noreply.github.com> Co-authored-by: Steve Loeppky <biglep@filoz.org>
Feedback has been incorporated as part of #12378. Closing this release ticket now. |
Lotus Miner and Node v1.28.2 Release
😶🌫 Meta
🚢 Estimated shipping date
🪢 Dependencies for releases
Note
master
.Release Checklist
.Stable (non-RC)
✅ Release Patch Checklist
Before Release
release/v1.28.2
) fromrelease/v1.28.2
and make any further release-related changes to this branch.master
branch Version string updatesbuild/version.go
tovX.Y.Z+1-dev
.NodeBuildVersion
), a miner release (MinerBuildVersion
), or both.make gen && make docsgen-cli
before committing changes.build(vX.Y.Z+1): set initial version string
MinerBuildVersion
in master to v1.28.3-dev #12380Stable Release PR
build/version.go
to one NOT ending with '-rcX'.NodeBuildVersion
), a miner release (MinerBuildVersion
), or both.make gen && make docsgen-cli
to generate documentationgit log --oneline --graph vA.B.C..
, where A.B.C correspond to the previous release.gh
cli command looking at merged PRs into master and sorted by title to group similar areas: `gh pr list --repo filecoin-project/lotus --search "base:master merged:>YYYY-MM-DD" --json number,mergedAt,author,title | jq -r '.[] | [.number, mergedAt, .author.login, .title] | @TSV' | sort -k4build(vX.Y.Z): release vX.Y.Z
🚢 Estimated shipping date
tableTesting
Note
Link to any special steps for testing releases beyond ensuring CI is green. Steps can be inlined here or tracked elsewhere.
Post-Release
master
with a merge of therelease/vX.Y.Z
branch.version.go
(keep the-dev
version from master).Before RC1
with any improvements determined from this latest release iteration.❤️ Contributors
See the final release notes!
Leave a comment in this ticket!
The text was updated successfully, but these errors were encountered: