contrib/seeds: bump MIN_BLOCKS = 687249 & update #22529

No description provided.

Without bump MIN_BLOCKS = 687249

https://github.com/RandyMcMillan/bitcoin/commit/50a40e7d57287649c7959ff32a2c11f120b0490b

I think MIN_BLOCKS should be in another PR. It's a simple enough review.

Sorry to pick your brain, but I have a few questions regarding changes to the list of included peers. Maybe they have already been answered somewhere.

1. It's quite hard to verify on github that chainparams_seed_main matches nodes_main.txt. Is there an easy way for the reviewers to do this?

2. Is there any way to verify that the list of peers being added are peers worth adding? I think in order to be included in the core list, they must somehow be superior to peers that are being advertised normally via the network from seed nodes. Just a few criteria I can think of:

• Uptime
• First-seen time
• Services running
• How fast can the node serve blocks
• Forwarding transactions
• Possible suspicious behavior, eg. only advertises a small set of nodes
• Recent or old version of core, or some other implementation
• etc.

Thank you!

Thanks, however this step of the release process is still being performed by @laanwj, and I don't think that's changing just yet. This was pointed out to you the last time you opened a PR to make a seeds related change (#20190).