Skip to content

[8.x] [Connector] Checkout to right version (#200623)#200637

Merged
kibanamachine merged 1 commit intoelastic:8.xfrom
kibanamachine:backport/8.x/pr-200623
Nov 18, 2024
Merged

[8.x] [Connector] Checkout to right version (#200623)#200637
kibanamachine merged 1 commit intoelastic:8.xfrom
kibanamachine:backport/8.x/pr-200623

Conversation

@kibanamachine
Copy link
Contributor

Backport

This will backport the following commits from main to 8.x:

Questions ?

Please refer to the Backport tool documentation

## Summary

When running `connectors` from sources let's checkout to the right git
tag. If version is not provided use `main` branch as default value.

If version is provided use git tag e.g. `v8.16.0` to checkout to right
version of connectors repo.

This fixes the bug when running `9.0` (main) connectors against <9.0
stack what causes issues.

Similar logic to how we handle versioning docker images
https://github.com/elastic/kibana/blob/main/x-pack/plugins/enterprise_search/public/applications/enterprise_search_content/components/connector_detail/deployment.tsx#L204

(cherry picked from commit f6ba99d)
@kibanamachine kibanamachine added the backport This PR is a backport of another PR label Nov 18, 2024
@kibanamachine kibanamachine enabled auto-merge (squash) November 18, 2024 19:13
@kibanamachine kibanamachine merged commit e2c8fa8 into elastic:8.x Nov 18, 2024
@elasticmachine
Copy link
Contributor

💚 Build Succeeded

Metrics [docs]

Async chunks

Total size of all lazy-loaded chunks that will be downloaded as the user navigates the app

id before after diff
enterpriseSearch 2.6MB 2.6MB +74.0B

cc @jedrazb

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

Labels

backport This PR is a backport of another PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants