Don't throw exceptions from ~_single_instance_hook

[Alan-Jowett]

Resolves: #2511

Description

C++ destructors are noexcept by definition. Throwing from a destructor is a noexcept violation which results in abort. Avoid this by making best effort to cleanup in destructors.

Testing

CI/CD

Documentation

No.

[codecov]

Codecov Report

Merging #2514 (fa4deee) into main (667a948) will decrease coverage by 0.12%.
The diff coverage is 33.33%.

@@            Coverage Diff             @@
##             main    #2514      +/-   ##
==========================================
- Coverage   84.06%   83.95%   -0.12%     
==========================================
  Files         155      155              
  Lines       28847    28849       +2     
==========================================
- Hits        24251    24219      -32     
- Misses       4596     4630      +34     

Impacted Files	Coverage Δ
tests/end_to_end/helpers.h	89.75% <33.33%> (-1.57%)	⬇️

... and 10 files with indirect coverage changes

[saxena-anurag]

Though unrelated to changes in this PR, it looks like detach() has a bug: It is throwing an exception when ebpf_link_detach() returns EBPF_SUCCESS.

    detach()
    {
        if (link_object != nullptr) {
            if (ebpf_link_detach(link_object) == EBPF_SUCCESS) {
                throw std::runtime_error("ebpf_link_detach failed");
            }
            ebpf_link_close(link_object);
            link_object = nullptr;
        }
    }