apollo_infra: return ReloadHandle from configure_tracing

[nadin-Starkware]

No description provided.

[reviewable-StarkWare]

This change is 

[nadin-Starkware]

• apollo_infra: return ReloadHandle from configure_tracing #10083 👈 (View in Graphite)
• main-v0.14.1

This stack of pull requests is managed by Graphite. Learn more about stacking.

[Itay-Tsabary-Starkware]

Reviewable status: 0 of 1 files reviewed, 1 unresolved discussion

---

crates/apollo_infra/src/trace_util.rs line 24 at r2 (raw file):

    "reqwest",
    "yamux",
];

If we set the default to be Info (which makes a lot of sense) then this becomes redundant. I suggest removing it.

Code quote:

// Crates we always keep at INFO regardless of operator-supplied spec.
const QUIET_LIBS: &[&str] = &[
    "alloy_provider",
    "alloy_transport_http",
    "alloy_rpc_client",
    "futures-util",
    "hickory-proto",
    "hyper",
    "hyper_util",
    "h2",
    "libp2p",
    "libp2p-gossipsub",
    "multistream_select",
    "netlink_proto",
    "reqwest",
    "yamux",
];

[Itay-Tsabary-Starkware]

Reviewable status: 0 of 1 files reviewed, 2 unresolved discussions (waiting on @nadin-Starkware)

---

crates/apollo_infra/src/trace_util.rs line 70 at r2 (raw file):

        .await;

    reload_handle.clone()

I'd like to make sure a clone can be used to trigger the reloading process.

Code quote:

 reload_handle.clone()