Skip to content

Use package registry lite image for fleet tests#147139

Merged
jsoriano merged 5 commits intoelastic:mainfrom
jsoriano:fleet-tests-use-package-registry-lite
Dec 13, 2022
Merged

Use package registry lite image for fleet tests#147139
jsoriano merged 5 commits intoelastic:mainfrom
jsoriano:fleet-tests-use-package-registry-lite

Conversation

@jsoriano
Copy link
Member

@jsoriano jsoriano commented Dec 6, 2022

Summary

Lite image is much smaller that the production image, and should be enough for testing.

This change was already applied in #146809, but reverted, testing it again.

@jsoriano jsoriano self-assigned this Dec 6, 2022
@jsoriano jsoriano requested a review from a team as a code owner December 6, 2022 20:03
@botelastic botelastic bot added the Team:Fleet Team label for Observability Data Collection Fleet team label Dec 6, 2022
@elasticmachine
Copy link
Contributor

Pinging @elastic/fleet (Team:Fleet)

@jsoriano jsoriano added the release_note:skip Skip the PR/issue when compiling release notes label Dec 6, 2022
@jsoriano
Copy link
Member Author

jsoriano commented Dec 6, 2022

Umm, may failure be related to the update in bundled packages? #147085 🤔

@jsoriano
Copy link
Member Author

jsoriano commented Dec 7, 2022

@elasticmachine merge upstream

@criamico
Copy link
Contributor

criamico commented Dec 7, 2022

Umm, may failure be related to the update in bundled packages? #147085 🤔

The warning comes from here. Maybe that configuration is missing in the lite image but was there in the production one?

@nchaulet
Copy link
Member

nchaulet commented Dec 8, 2022

Looks like the test fail because there is no entry in the registry for the security_detection_engine package and it fall back to look for bundled package that are not configured here, I just created a PR for that https://github.com/elastic/package-storage-infra

@nchaulet
Copy link
Member

nchaulet commented Dec 9, 2022

@elasticmachine merge upstream

@nchaulet
Copy link
Member

@elasticmachine merge upstream

@mrodm
Copy link
Contributor

mrodm commented Dec 12, 2022

@elasticmachine merge upstream

@kibana-ci
Copy link

💚 Build Succeeded

Metrics [docs]

Unknown metric groups

ESLint disabled in files

id before after diff
osquery 1 2 +1

ESLint disabled line counts

id before after diff
enterpriseSearch 19 21 +2
fleet 60 66 +6
osquery 109 115 +6
securitySolution 445 451 +6
total +20

Total ESLint disabled count

id before after diff
enterpriseSearch 20 22 +2
fleet 69 75 +6
osquery 110 117 +7
securitySolution 521 527 +6
total +21

History

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

cc @jsoriano

Copy link
Member

@nchaulet nchaulet left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport:skip This PR does not require backporting release_note:skip Skip the PR/issue when compiling release notes Team:Fleet Team label for Observability Data Collection Fleet team v8.7.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants