Add Fixed-Rate Polling Specifier To ReportCollector

[tyurek]

To test this, I added .args(["--set-fixed-polling-ms", "100"]) before line 93 in the file containing test_hybrid(). I also verified that it fails with bad arguments

Test ran with cargo test --package ipa-core --test hybrid --no-default-features --features "cli compact-gate web-app real-world-infra test-fixture relaxed-dp multi-threading" -- test_hybrid --exact

I should say this usually passes... I ran it once (with the argument specified) and got

Unable to create query!: FailedHttpRequest { dest: "localhost:64473", status: 500, reason: "500 Internal Server Error: One or more peers rejected the request: [(ShardIndex(3), ShardError { shard_index: ShardIndex(0), source: ConnectError { dest: \"localhost:64493\", inner: hyper_util::client::legacy::Error(Connect, Custom { kind: Other, error: Custom { kind: InvalidData, error: InvalidMessage(InvalidContentType) } }) } })]" }

But I don't think that's due to any incorrect logic in this PR

[codecov]

Codecov Report

Attention: Patch coverage is 0% with 11 lines in your changes missing coverage. Please review.

Project coverage is 93.27%. Comparing base (4f112ad) to head (53b6198).
Report is 1 commits behind head on main.

Files with missing lines	Patch %	Lines
ipa-core/src/cli/playbook/hybrid.rs	0.00%	7 Missing ⚠️
ipa-core/src/bin/report_collector.rs	0.00%	4 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #1492      +/-   ##
==========================================
- Coverage   93.29%   93.27%   -0.03%     
==========================================
  Files         237      237              
  Lines       43510    43518       +8     
==========================================
- Hits        40592    40590       -2     
- Misses       2918     2928      +10     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[akoshelev]

is Option<u64> better to represent this value being not set?

[tyurek]

oh, yeah, probably

[akoshelev]

Setting interval in millis is not great but probably the best we can do without polling extra crates. Humans tend to make typo errors when entering large numbers and if we want to poll every 5 mins, 300000 vs 30000 makes a big difference

[tyurek]

Setting interval in millis is not great but probably the best we can do without polling extra crates. Humans tend to make typo errors when entering large numbers and if we want to poll every 5 mins, 300000 vs 30000 makes a big difference

Are we expecting to have large polling intervals like that?