Revert "Default fleet policy removal"

[ph]

Reverts #1157

Reverting we will rework it to make it non-breaking.

[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: 2022-04-01T14:21:03.899+0000

• Duration: 12 min 51 sec

Test stats 🧪

Test	Results
Failed	0
Passed	271
Skipped	0
Total	271

🤖 GitHub comments

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

• /test : Re-trigger the build.

[ph]

test this please

[ph]

[2022-04-01T13:17:50.735Z]     enrollment_api_key_integration_test.go:113:   []model.EnrollmentApiKey{
[2022-04-01T13:17:50.735Z]           	{
[2022-04-01T13:17:50.735Z]           		ESDocument: model.ESDocument{
[2022-04-01T13:17:50.735Z]         - 			Id:      "c93fmvc2jgjfauojood0",
[2022-04-01T13:17:50.735Z]         + 			Id:      "c93fmvc2jgjfauojooe0",
[2022-04-01T13:17:50.735Z]           			Version: 0,
[2022-04-01T13:17:50.735Z]           			SeqNo:   0,
[2022-04-01T13:17:50.735Z]           		},
[2022-04-01T13:17:50.735Z]           		Active:    true,
[2022-04-01T13:17:50.735Z]           		ApiKey:    "d2JndlFIWUJJUVVxWDVia2NJTV86X0d6ZmljZGNTc1d4R1otbklrZFFRZw==",
[2022-04-01T13:17:50.735Z]         - 		ApiKeyId:  "c93fmvc2jgjfauojoodg",
[2022-04-01T13:17:50.735Z]         + 		ApiKeyId:  "c93fmvc2jgjfauojooeg",
[2022-04-01T13:17:50.735Z]           		CreatedAt: "2022-04-01T13:17:49Z",
[2022-04-01T13:17:50.735Z]           		ExpireAt:  "",
[2022-04-01T13:17:50.735Z]           		... // 3 identical fields
[2022-04-01T13:17:50.735Z]           	},
[2022-04-01T13:17:50.735Z]           	{
[2022-04-01T13:17:50.735Z]           		ESDocument: model.ESDocument{
[2022-04-01T13:17:50.735Z]         - 			Id:      "c93fmvc2jgjfauojooe0",
[2022-04-01T13:17:50.735Z]         + 			Id:      "c93fmvc2jgjfauojood0",
[2022-04-01T13:17:50.735Z]           			Version: 0,
[2022-04-01T13:17:50.735Z]           			SeqNo:   0,
[2022-04-01T13:17:50.735Z]           		},
[2022-04-01T13:17:50.735Z]           		Active:    true,
[2022-04-01T13:17:50.735Z]           		ApiKey:    "d2JndlFIWUJJUVVxWDVia2NJTV86X0d6ZmljZGNTc1d4R1otbklrZFFRZw==",
[2022-04-01T13:17:50.735Z]         - 		ApiKeyId:  "c93fmvc2jgjfauojooeg",
[2022-04-01T13:17:50.735Z]         + 		ApiKeyId:  "c93fmvc2jgjfauojoodg",
[2022-04-01T13:17:50.735Z]           		CreatedAt: "2022-04-01T13:17:49Z",
[2022-04-01T13:17:50.735Z]           		ExpireAt:  "",
[2022-04-01T13:17:50.735Z]           		... // 3 identical fields
[2022-04-01T13:17:50.735Z]           	},
[2022-04-01T13:17:50.735Z]           }
[2022-04-01T13:17:50.735Z]         
[2022-04-01T13:17:50.735Z] --- FAIL: TestSearchEnrollmentAPIKeyByPolicyID (1.10s)

The ci was hiding the real reason.. again :(

[ph]

I think its either the query or the expectation are wrong in this integration test, I am not familiar with the logic but this could potentially hide a bug, I am going to retrigger this test and create an issue to investigate it.

[ph]

/test

[ph]

Please ignore lint, we are reverting changes the linter will yield at us.

[narph]

compared with initial PR , LGTM

[ph]

@Mergifyio requeue

[mergify]

requeue

❌ This pull request head commit has not been previously disembarked from queue.

[mdelapenya]

@Mergifyio refresh

[mergify]

refresh

✅ Pull request refreshed

[mdelapenya]

@Mergifyio backport 8.2

[mergify]

backport 8.2

✅ Backports have been created

• #1290 Revert "Default fleet policy removal" (backport #1287) has been created for branch 8.2