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

[EPM] update registry path structure #61621

Merged
merged 6 commits into from
Mar 30, 2020

Conversation

neptunian
Copy link
Contributor

#61447

elastic/package-registry#300, changes the path structure from {package}-{version} to {package}/{version}. This PR updates our requests to the registry. This depends on elastic/package-registry#300 being merged at the same time to prevent breaking the app.

@neptunian neptunian added v8.0.0 release_note:skip Skip the PR/issue when compiling release notes Feature:EPM Fleet team's Elastic Package Manager (aka Integrations) project v7.8 labels Mar 27, 2020
@neptunian neptunian requested a review from jfsiii March 27, 2020 13:38
@neptunian neptunian self-assigned this Mar 27, 2020
@elasticmachine
Copy link
Contributor

Pinging @elastic/ingest-management (Feature:EPM)

@neptunian neptunian requested a review from ruflin March 27, 2020 13:42
Copy link
Member

@ruflin ruflin left a comment

Choose a reason for hiding this comment

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

Tested this locally with the new registry and seems to work as expected.

@neptunian neptunian marked this pull request as ready for review March 27, 2020 15:33
@neptunian neptunian requested a review from a team March 27, 2020 15:33
@neptunian
Copy link
Contributor Author

@elasticmachine merge upstream

@neptunian
Copy link
Contributor Author

@elasticmachine merge upstream

@kibanamachine
Copy link
Contributor

💚 Build Succeeded

History

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

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.

Looks good to me

@neptunian neptunian merged commit 5042a0d into elastic:master Mar 30, 2020
neptunian added a commit to neptunian/kibana that referenced this pull request Mar 30, 2020
* update registry path structure

* temporarily remove fleet tests

Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com>
spalger pushed a commit that referenced this pull request Mar 30, 2020
* update registry path structure

* temporarily remove fleet tests

Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com>

Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com>
spalger pushed a commit that referenced this pull request Mar 31, 2020
* update registry path structure

* temporarily remove fleet tests

Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com>

Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com>
(cherry picked from commit 760ef4c)
@spalger
Copy link
Contributor

spalger commented Mar 31, 2020

7.x/7.8: 760ef4c
7.7: 5937e97

@rayafratkina rayafratkina removed the v7.8 label Apr 6, 2020
@jen-huang jen-huang added Team:Fleet Team label for Observability Data Collection Fleet team and removed v7.7.0 labels Apr 9, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature:EPM Fleet team's Elastic Package Manager (aka Integrations) project release_note:skip Skip the PR/issue when compiling release notes Team:Fleet Team label for Observability Data Collection Fleet team v7.8.0 v8.0.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants