Ability to (de)serialize NetworkConfigBuilder

[leruaa]

Closes #873

[leruaa]

Still need to make a bunch of things (De)Serializable. Added some #[serde(skip)] for now.

[codecov-commenter]

Codecov Report

Merging #897 (07105ab) into main (e9792c1) will increase coverage by 0.05%.
The diff coverage is 65.38%.

@@            Coverage Diff             @@
##             main     #897      +/-   ##
==========================================
+ Coverage   73.93%   73.99%   +0.05%     
==========================================
  Files         288      288              
  Lines       30786    30782       -4     
==========================================
+ Hits        22763    22777      +14     
+ Misses       8023     8005      -18     

Flag	Coverage Δ
unit-tests	73.99% <65.38%> (+0.05%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files	Coverage Δ
bin/reth/src/config.rs	59.61% <0.00%> (ø)
crates/net/network/src/session/config.rs	80.58% <0.00%> (-2.42%)	⬇️
crates/net/network/src/config.rs	59.78% <63.63%> (-1.48%)	⬇️
crates/net/discv4/src/config.rs	56.02% <100.00%> (ø)
crates/net/dns/src/config.rs	100.00% <100.00%> (ø)
crates/net/dns/src/tree.rs	83.13% <100.00%> (ø)
crates/net/nat/src/lib.rs	63.12% <100.00%> (ø)
crates/net/network/src/test_utils/testnet.rs	83.03% <100.00%> (ø)
crates/net/network/src/transactions.rs	22.22% <100.00%> (ø)
crates/primitives/src/forkid.rs	96.25% <100.00%> (ø)
... and 4 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

[gakonst]

Looks good so far

[leruaa]

Made some progress, some things to notice:

• I've added skip on ban_list field on Discv4Config because Instant is not serializable
• executor field is still skipped, and I noticed executor() is neved called on the builder. Do I need to better handle this field?
• Same thing for block_import, the corresponding method was never called on the builder, So I didn't added the parameter to build() and just create a NetworkConfig with ProofOfStakeBlockImport::default(). Is that ok?

Thanks in advance for your feedbacks

[mattsse]

ty,

some early pointers.

[gakonst]

@leruaa this is looking great -just a bunch of merge conflicts to address

[mattsse]

lgtm