Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Fleet] Re-enable registry version check #120045

Merged
merged 5 commits into from
Dec 2, 2021

Conversation

jen-huang
Copy link
Contributor

@jen-huang jen-huang commented Dec 1, 2021

Summary

Resolves #117107. Put back the registry Kibana version check. The check is still skipped for main branch for development purposes - this is unchanged.

Backport PR: #120049
^ This was done manually as the 8.0 branch is the one that would actually be impacted by this :)

@jen-huang jen-huang added v8.0.0 release_note:skip Skip the PR/issue when compiling release notes Team:Fleet Team label for Observability Data Collection Fleet team auto-backport Deprecated - use backport:version if exact versions are needed v8.1.0 labels Dec 1, 2021
@jen-huang jen-huang self-assigned this Dec 1, 2021
@jen-huang jen-huang removed the auto-backport Deprecated - use backport:version if exact versions are needed label Dec 1, 2021
@kibana-ci
Copy link
Collaborator

💚 Build Succeeded

Metrics [docs]

✅ unchanged

History

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

cc @jen-huang

@jen-huang jen-huang added the backport:skip This commit does not require backporting label Dec 2, 2021
@jen-huang jen-huang marked this pull request as ready for review December 2, 2021 01:13
@jen-huang jen-huang requested review from a team as code owners December 2, 2021 01:13
@elasticmachine
Copy link
Contributor

Pinging @elastic/fleet (Team:Fleet)

@jen-huang jen-huang changed the title [WIP][Fleet] Re-enable registry version check [Fleet] Re-enable registry version check Dec 2, 2021
Copy link
Contributor

@juliaElastic juliaElastic left a comment

Choose a reason for hiding this comment

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

LGTM

@botelastic botelastic bot added the Team:APM All issues that need APM UI Team support label Dec 2, 2021
@elasticmachine
Copy link
Contributor

Pinging @elastic/apm-ui (Team:apm)

@@ -13,7 +13,7 @@ import { packageRegistryPort } from './ftr_config';
import { FtrProviderContext } from './ftr_provider_context';

export const dockerImage =
'docker.elastic.co/package-registry/distribution@sha256:13d9996dd24161624784704e080f5f5b7f0ef34ff0d9259f8f05010ccae00058';
'docker.elastic.co/package-registry/distribution@sha256:de952debe048d903fc73e8a4472bb48bb95028d440cba852f21b863d47020c61';
Copy link
Member

@sorenlouv sorenlouv Dec 2, 2021

Choose a reason for hiding this comment

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

Would it make sense to share this sha between fleet and apm so we won't forget to update it here next time it's bumped in fleet?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I opened #120235 for this

@jen-huang jen-huang merged commit 2c4dfde into elastic:main Dec 2, 2021
@jen-huang jen-huang deleted the fix/epr-version-check branch December 2, 2021 16:01
TinLe pushed a commit to TinLe/kibana that referenced this pull request Dec 22, 2021
* Re-enable registry version check

* Update registry image

* Use dockerized registry for base x-pack api integration and functional tests

* Revert "Use dockerized registry for base x-pack api integration and functional tests"

This reverts commit 2fd4ec1.
@sorenlouv
Copy link
Member

@jen-huang Did you mean to backport this to 8.0? Afaict it was never backported but the labels suggests it should be.

@jen-huang
Copy link
Contributor Author

@sqren It was backported as a manual PR in #120049 (due to making sure CI passes in 8.0 before merging this one into main).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport:skip This commit does not require backporting release_note:skip Skip the PR/issue when compiling release notes Team:APM All issues that need APM UI Team support Team:Fleet Team label for Observability Data Collection Fleet team v8.0.0 v8.1.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Fleet] Re-enable registry version check
5 participants