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] Use registry version check on main #125495

Merged
merged 1 commit into from
Feb 14, 2022

Conversation

joshdover
Copy link
Contributor

Summary

Related to #125409

We currently have special logic in place that removes the kibana.version parameter from requests to EPR when running on the main branch. This makes testing logic around compatibility inflexible and leads to developers running different logic than we use in production.

This removes this logic in preference of the xpack.fleet.developer.disableRegistryVersionCheck config that was added previously. I did consider making this default to true in when running Kibana in development, but decided against it in order to facilitate using the production behavior by default and making the other logic more explicit.

This does not affect production behavior at all.

@joshdover joshdover added release_note:skip Skip the PR/issue when compiling release notes Team:Fleet Team label for Observability Data Collection Fleet team v8.1.0 v8.2.0 v8.0.1 labels Feb 14, 2022
@joshdover joshdover requested a review from a team as a code owner February 14, 2022 12:42
@elasticmachine
Copy link
Contributor

Pinging @elastic/fleet (Team:Fleet)

@joshdover joshdover enabled auto-merge (squash) February 14, 2022 14:07
@joshdover joshdover added the auto-backport Deprecated - use backport:version if exact versions are needed label Feb 14, 2022
@kibana-ci
Copy link
Collaborator

💚 Build Succeeded

Metrics [docs]

✅ unchanged

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

@joshdover joshdover merged commit 9278294 into elastic:main Feb 14, 2022
kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Feb 14, 2022
@kibanamachine
Copy link
Contributor

💔 Some backports could not be created

Status Branch Result
8.1
8.0 Backport failed because of merge conflicts

You might need to backport the following PRs to 8.0:
- [Fleet] Re-enable registry version check (#120045)

How to fix

Re-run the backport manually:

node scripts/backport --pr 125495

Note: Successful backport PRs will be merged automatically after passing CI.

Questions ?

Please refer to the Backport tool documentation

kibanamachine added a commit that referenced this pull request Feb 14, 2022
(cherry picked from commit 9278294)

Co-authored-by: Josh Dover <[email protected]>
joshdover added a commit to kibanamachine/kibana that referenced this pull request Feb 16, 2022
@joshdover
Copy link
Contributor Author

Handling 8.0 backport in #125672

kibanamachine added a commit that referenced this pull request Feb 16, 2022
…ilable in registry (#125525) (#125672)

* [Fleet] Avoid breaking setup when compatible package is not available in registry (#125525)

(cherry picked from commit 928638e)

* [Fleet] Use registry version check on main (#125495)

* [Fleet] Get package info should not store the whole package (#123509)

Co-authored-by: Josh Dover <[email protected]>
Co-authored-by: Nicolas Chaulet <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
auto-backport Deprecated - use backport:version if exact versions are needed release_note:skip Skip the PR/issue when compiling release notes Team:Fleet Team label for Observability Data Collection Fleet team v8.0.1 v8.1.0 v8.2.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants