Disables collectorConnectionDiesThenReconnects because it is flaky

[MadVikingGod]

This disables a test that has become more and more problematic. It was originally reported in #1527, and has gotten progressively worse.

I would recommend we reenable if there is a PR that fixes it, or when #2329 is accepted.

[MrAlias]

Can we also add skips in these locations?

opentelemetry-go/exporters/otlp/otlptrace/otlptracegrpc/client_test.go

Line 166 in 249accd

func TestNew_collectorConnectionDiesThenReconnectsWhenInRestMode(t *testing.T) {

opentelemetry-go/exporters/otlp/otlpmetric/otlpmetricgrpc/client_test.go

Line 170 in 249accd

func TestNewExporter_collectorConnectionDiesThenReconnectsWhenInRestMode(t *testing.T) {

and

opentelemetry-go/exporters/otlp/otlpmetric/otlpmetricgrpc/client_test.go

Line 493 in 249accd

func TestNewExporter_collectorConnectionDiesThenReconnects(t *testing.T) {

[MadVikingGod]

I most defantly can add it there. I was just starting with the one that was consistently failing. To see if that made the other fail as well.

[MrAlias]

I most defantly can add it there. I was just starting with the one that was consistently failing. To see if that made the other fail as well.

From what I've seen they all fail at an equal rate.

[evantorrie]

Yes, I've fought these tests more than I care to...

[MrAlias]

/easycla

[MrAlias]

I was able to resolve the EasyCLA issues with #2374 in #2378 by getting rid of all the commits made though GitHub.

• Rebase to main git rebase upstream/main
• Force a collapse of all changes in the branch to a single commit

  git reset $(git merge-base main $(git branch --show-current))
  git add .
  git commit
  

• Finally, force pushing

[MadVikingGod]

phew, it passed the easy CLA

[MrAlias]

phew, it passed the easy CLA

I don't want to mess things up by hitting the "Update branch" button. Can you rebase this to the latest on main?

[codecov]

Codecov Report

Merging #2373 (cdc6e99) into main (a65d50a) will increase coverage by 0.0%.
The diff coverage is n/a.

@@          Coverage Diff          @@
##            main   #2373   +/-   ##
=====================================
  Coverage   73.8%   73.8%           
=====================================
  Files        175     175           
  Lines      12436   12436           
=====================================
+ Hits        9185    9189    +4     
+ Misses      3015    3012    -3     
+ Partials     236     235    -1     

Impacted Files	Coverage Δ
...s/otlp/otlptrace/internal/connection/connection.go	14.6% <0.0%> (-1.6%)	⬇️
sdk/metric/sdk.go	81.6% <0.0%> (+1.4%)	⬆️
sdk/metric/refcount_mapped.go	100.0% <0.0%> (+20.0%)	⬆️

[evantorrie]

Yay!