Added --properties-allowed switch and retries to test.

[ryanrolds]

This is part of reducing cardinality. The JUnit XML from Ginkgo and gotestsum contains arbitrary properties we don't want pushed to our metrics system.

• Adds new flag that allows defining an allowed list of properties
• Added retry package and used to improve test reliability
• Updates junit-go to latest version

[mdelapenya]

Let's rebase and resolve conflicts first 🙏

[ryanrolds]

I've rebased and resolved conflicts. This was the branch that originally included the retry changes. Are you wanting those removed?

[mdelapenya]

I think the branch still has conflicts, probably gotten from the retry, so yeah, let's remove them by now, as they sleep was not needed anymore with the wait.ForFile strategy in the tests

[mdelapenya]

Let's keep the wait strategy do its job here.

[ryanrolds]

The conflicts are from the PR that was merged after I resolved the conflicts from the earlier merge.

The test is much faster without the 30s wait. I will resolve the conflicts in the morning.

[ryanrolds]

I tried it without the retry and I randomly get:

=== RUN   Test_Main_SampleXML
    main_test.go:187: could not get the collector port: port "4317/tcp" not found
2024/12/20 07:43:31 failed to upload metrics: failed to exit idle mode: dns resolver: missing port after port-separator colon
2024/12/20 07:43:31 traces export: failed to exit idle mode: dns resolver: missing port after port-separator colon
    main_test.go:236: expected 2 JSONs, got 1 - []
--- FAIL: Test_Main_SampleXML (5.62s)
panic: runtime error: index out of range [1] with length 1 [recovered]
	panic: runtime error: index out of range [1] with length 1

[ryanrolds]

I've resolved the merge conflicts and am switching the port retry to use wait.

[mdelapenya]

LGTM, thanks!