[Flaky test] skip FQDN integration test

[AndersonQ]

What does this PR do?

Disable the flaky FQDN integration test

Why is it important?

CI needs to be green

Checklist

• My code follows the style guidelines of this project
• [x] I have commented my code, particularly in hard-to-understand areas
• [ ] I have made corresponding changes to the documentation
• [ ] I have made corresponding change to the default configuration files
• [ ] I have added tests that prove my fix is effective or that my feature works
• [ ] I have added an entry in ./changelog/fragments using the changelog tool
• [ ] I have added an integration test or an E2E test

Related issues

• Relates [Flaky Test] Integration/e2e tests installing the agent randomly fail #3154

Questions to ask yourself

• How are we going to support this in production?
• How are we going to measure its adoption?
• How are we going to debug this?
• What are the metrics I should take care of?
• ...

[elasticmachine]

Pinging @elastic/elastic-agent (Team:Elastic-Agent)

[ycombinator]

LGTM.

[elasticmachine]

💚 Build Succeeded

the below badges are clickable and redirect to their specific view in the CI or DOCS

Expand to view the summary

Build stats

• Start Time: 2023-08-08T11:43:03.441+0000

• Duration: 25 min 2 sec

Test stats 🧪

Test	Results
Failed	0
Passed	6209
Skipped	31
Total	6240

💚 Flaky test report

Tests succeeded.

🤖 GitHub comments

Expand to view the GitHub comments

To re-run your PR in the CI, just comment with:

• /test : Re-trigger the build.

• /package : Generate the packages.

• run integration tests : Run the Elastic Agent Integration tests.

• run end-to-end tests : Generate the packages and run the E2E Tests.

• run elasticsearch-ci/docs : Re-trigger the docs validation. (use unformatted text in the comment!)

[pierrehilbert]

12:14:31 Error: test TestFQDN first statement must be a function call to define.Require
12:14:31 make: *** [Makefile:46: check-ci] Error 1

[ycombinator]

12:14:31 Error: test TestFQDN first statement must be a function call to define.Require 12:14:31 make: *** [Makefile:46: check-ci] Error 1

Good catch. The t.Skip should be added after the define.Require. For example, see

elastic-agent/testing/integration/enroll_test.go

Lines 26 to 38 in dfd9554

	func TestEnrollAndLog(t *testing.T) {
	info := define.Require(t, define.Requirements{
	OS: []define.OS{
	{Type: define.Linux},
	},
	Stack: &define.Stack{},
	Local: false,
	Sudo: true,
	})
	t.Logf("got namespace: %s", info.Namespace)
	t.Skip("Test is flaky; see https://github.com/elastic/elastic-agent/issues/3081")
	suite.Run(t, &EnrollRunner{requirementsInfo: info})
	}

[elasticmachine]

🌐 Coverage report

Name	Metrics % (covered/total)	Diff
Packages	98.734% (78/79)	👍
Files	66.549% (189/284)	👍
Classes	65.399% (344/526)	👍
Methods	52.258% (1076/2059)	👍
Lines	38.314% (12309/32127)	👍 0.016
Conditionals	100.0% (0/0)	💚