Skip to content

Conversation

fanquake
Copy link
Member

@fanquake fanquake commented Jun 19, 2024

@fanquake fanquake added this to the 27.2 milestone Jun 19, 2024
@DrahtBot
Copy link
Contributor

DrahtBot commented Jun 19, 2024

The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.

Code Coverage

For detailed information about the code coverage, see the test coverage report.

Reviews

See the guideline for information on the review process.

Type Reviewers
ACK willcl-ark, stickies-v

If your review is incorrectly listed, please react with 👎 to this comment and the bot will ignore it on the next update.

@maflcko
Copy link
Member

maflcko commented Jun 19, 2024

CI note: If you want the Asan task, you'll have to backport it from GHA

m3dwards and others added 4 commits June 19, 2024 12:48
Allows IPV6 functional tests to run inside the container

Github-Pull: bitcoin#30193
Rebased-From: 4b527fa
Moving it from Cirrus CI so it can be easier to maintain and used by forks

Github-Pull: bitcoin#30193
Rebased-From: 9eea51d
See: miniupnp/miniupnp@c0a50ce

The return value of 2 now indicates:
"A valid connected IGD has been found but its IP address is reserved (non routable)"

We continue to ignore any return value other than 1.

Github-Pull: bitcoin#30283
Rebased-From: 8acdf66
@fanquake fanquake force-pushed the more_27_backports branch from cb67470 to d80a6d8 Compare June 19, 2024 11:53
@fanquake fanquake marked this pull request as ready for review June 24, 2024 10:26
@fanquake fanquake force-pushed the more_27_backports branch from d80a6d8 to b3093eb Compare June 24, 2024 10:38
Copy link
Member

@willcl-ark willcl-ark left a comment

Choose a reason for hiding this comment

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

ACK b3093eb

Manually compared each backported commit with its original version.

Copy link
Contributor

@stickies-v stickies-v left a comment

Choose a reason for hiding this comment

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

ACK b3093eb

All backports are clean, except:

  • 0d524b1 backported from 9eea51d: win64-native CI steps Clone fuzz corpus and RUun fuzz binaries aren't backported, they are part of #29774

=====================

Bitcoin Core version 27.1 is now available from:
Bitcoin Core version 27.x is now available from:
Copy link
Contributor

Choose a reason for hiding this comment

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

I know we usually do it this way, but what's the harm in just making this (and other references, except for the git tag) 27.2rc1 right away?

Copy link
Member Author

Choose a reason for hiding this comment

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

The next release might not be 27.2. It could be 27.1.1 etc.

Copy link
Contributor

Choose a reason for hiding this comment

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

Sure, but that seems to be an undocumented exception? Since 0.10.0, it seems we only have 3 (in the root dir) releases where we we had a non-zero patch number. We also don't mention patch versions in our documentation.

Keeping the git tag to .x makes sense because it gives us the flexibility to release as a minor or a patch version, i.e. we can still easily update the release notes to 27.1.1 if necessary. It just seems like currently the code/doc updates just add a bit of unnecessary churn? Anyway, no biggie either way.

Copy link
Member Author

Choose a reason for hiding this comment

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

Ok. Lets follow up with ways we might want to streamline this for 28.x.

@fanquake fanquake merged commit b6440f2 into bitcoin:27.x Jun 24, 2024
@fanquake fanquake deleted the more_27_backports branch June 24, 2024 12:36
@bitcoin bitcoin locked and limited conversation to collaborators Jun 24, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants