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

build: release: v1.28.2 #12392

Merged
merged 12 commits into from
Aug 15, 2024
Merged

build: release: v1.28.2 #12392

merged 12 commits into from
Aug 15, 2024

Conversation

jennijuju
Copy link
Member

@jennijuju jennijuju commented Aug 15, 2024

@beck-8
Copy link
Contributor

beck-8 commented Aug 15, 2024

Hopefully v1.28.2 will completely fix the pledge issue together.

@MikeH1999
Copy link

Is it already available?

rjan90 and others added 3 commits August 15, 2024 13:22
chore: bump versions and make gen/docsgen-cli
chore: update v1.282. changelog
* Update go-f3 to 0.2.0

Includes:
 - fix for excessive bandwidth usage
 - significant performance improvements
 - minor consensus fixes

Signed-off-by: Jakub Sztandera <[email protected]>

* add changelog

Signed-off-by: Jakub Sztandera <[email protected]>

* chore(f3): update to final released version

---------

Signed-off-by: Jakub Sztandera <[email protected]>
Co-authored-by: Steven Allen <[email protected]>
aarshkshah1992 and others added 6 commits August 15, 2024 19:20
storage/pipeline.NewPreCommitBatcher and storage/pipeline.New now have an additional
error return to deal with errors arising from fetching the sealing config.
error return to deal with errors arising from fetching the sealing config.
* 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 <[email protected]>
- Updated the logging statement in `testOutOfGasError` to correctly reference `build.BlockGasLimit` instead of `buildconstants.BlockGasLimit`.
fix: update changelog to reference bandwidth issue ticket
@rjan90 rjan90 marked this pull request as ready for review August 15, 2024 17:37
@rjan90 rjan90 requested review from Stebalien and magik6k August 15, 2024 17:37
CHANGELOG.md Outdated Show resolved Hide resolved
@BigLep
Copy link
Member

BigLep commented Aug 15, 2024

I think we should also make clear if we're going to backport additional fixes around pledging like what was proposed in #12386

@rjan90 rjan90 mentioned this pull request Aug 15, 2024
23 tasks
Co-authored-by: Steve Loeppky <[email protected]>
CHANGELOG.md Outdated Show resolved Hide resolved
@jennijuju
Copy link
Member Author

I think we should also make clear if we're going to backport additional fixes around pledging like what was proposed in #12386

IIUC we won't?

@BigLep
Copy link
Member

BigLep commented Aug 15, 2024

I think we should also make clear if we're going to backport additional fixes around pledging like what was proposed in #12386

I think I missed some of the details about backporting and what has been done is good. #12395, which was backported, was informed by #12386

I don't see any issues at a high level. Thanks for handling.

@jennijuju jennijuju merged commit e2a77b4 into release/v1.28.2 Aug 15, 2024
78 checks passed
@jennijuju jennijuju deleted the jen/f3patch branch August 15, 2024 22:41
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.

9 participants