Skip to content
This repository has been archived by the owner on Nov 15, 2023. It is now read-only.

Authority-discovery no longer publishes non-global IP addresses #8643

Merged
3 commits merged into from
Apr 20, 2021

Conversation

tomaka
Copy link
Contributor

@tomaka tomaka commented Apr 20, 2021

The authority discover no longer publishes non-global IP addresses on the DHT, unless the user is running a local/dev chain or if --discover-local is passed.

polkadot companion: paritytech/polkadot#2908

@tomaka tomaka added A0-please_review Pull request needs code review. B5-clientnoteworthy C1-low PR touches the given topic and has a low impact on builders. labels Apr 20, 2021
client/authority-discovery/src/lib.rs Outdated Show resolved Hide resolved
Co-authored-by: Andronik Ordian <write@reusable.software>
@tomaka
Copy link
Contributor Author

tomaka commented Apr 20, 2021

bot merge

@ghost
Copy link

ghost commented Apr 20, 2021

Error: #8643 is not mergeable

@tomaka
Copy link
Contributor Author

tomaka commented Apr 20, 2021

@parity-processbot Why?

@tomaka
Copy link
Contributor Author

tomaka commented Apr 20, 2021

bot merge

@ghost
Copy link

ghost commented Apr 20, 2021

Trying merge.

@ghost
Copy link

ghost commented Apr 20, 2021

Bot will approve on the behalf of @tomaka, since they are a team lead, in an attempt to reach the minimum approval count

@ghost ghost merged commit 681f188 into paritytech:master Apr 20, 2021
@tomaka
Copy link
Contributor Author

tomaka commented Apr 20, 2021

🤷

@tomaka tomaka deleted the audi-non-global branch April 20, 2021 13:43
@ordian
Copy link
Member

ordian commented Apr 20, 2021

cc @joao-paulo-parity ^^^

@joao-paulo-parity
Copy link
Contributor

In that case it was Github saying the PR was not mergeable, not the bot's rules.

I think we can clarify that error message by telling where it came from.

This pull request was closed.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A0-please_review Pull request needs code review. C1-low PR touches the given topic and has a low impact on builders.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants