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

[release-v2.0] main: Use backported addrmgr updates. #3316

Merged
merged 2 commits into from
May 24, 2024

Conversation

davecgh
Copy link
Member

@davecgh davecgh commented May 24, 2024

This updates the 2.0 release branch to use the latest version of the addrmgr module which includes an update to give unattempted addresses a chance at being selected.

In particular, the following updated module version is used:

Note that it also cherry picks all of the commits included in updates to the addrmgr/v2 module to ensure they are also included in the release branch even though it is not strictly necessary since go.mod has been updated to require the new addrmgr/v2.0.4 release and thus will pull in the new code. However, from past experience, not having code backported to modules available in the release branch too leads to headaches for devs building from source in their local workspace with overrides such as those in go.work.

dajohi and others added 2 commits May 24, 2024 13:25
Subtracting an unset time.Time value yields a large number, resulting
in a chance value of 1, and will never be less than the random value.
This updates the 2.0 release branch to use the latest version of the
addrmgr module which includes an update to give unattempted addresses a
chance at being selected.

In particular, the following updated module version is used:

- github.com/decred/dcrd/addrmgr/[email protected]
@davecgh davecgh added this to the 2.0.1 milestone May 24, 2024
@davecgh davecgh merged commit 3fce5ee into decred:release-v2.0 May 24, 2024
2 checks passed
@davecgh davecgh deleted the rel20_addrmgr_backports branch May 24, 2024 18:37
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