Skip to content

Ignore deprecated release tag from manifests#647

Merged
jsoriano merged 11 commits intoelastic:mainfrom
jsoriano:release-deprecation
Feb 11, 2022
Merged

Ignore deprecated release tag from manifests#647
jsoriano merged 11 commits intoelastic:mainfrom
jsoriano:release-deprecation

Conversation

@jsoriano
Copy link
Copy Markdown
Member

@jsoriano jsoriano commented Jan 14, 2022

Stop showing information about the release tag of packages, to be deprecated in elastic/package-spec#256.

Use the prerelease parameter when searching for packages, related to elastic/package-registry#785.

Part of elastic/package-spec#225

@jsoriano jsoriano added the Team:Ecosystem Label for the Packages Ecosystem team label Jan 14, 2022
@jsoriano jsoriano self-assigned this Jan 14, 2022
@jsoriano jsoriano marked this pull request as draft January 14, 2022 17:10
go.mod Outdated
sigs.k8s.io/yaml v1.2.0 // indirect
)

replace github.com/elastic/package-spec v1.3.0 => github.com/jsoriano/package-spec v0.0.0-20220112095620-98e0d531845d
Copy link
Copy Markdown
Member Author

@jsoriano jsoriano Jan 14, 2022

Choose a reason for hiding this comment

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

This change won't be included.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I understand that this PR is blocked on the other one: elastic/package-spec#256?

@elasticmachine
Copy link
Copy Markdown
Collaborator

elasticmachine commented Jan 14, 2022

💚 Build Succeeded

the below badges are clickable and redirect to their specific view in the CI or DOCS
Pipeline View Test View Changes Artifacts preview preview

Expand to view the summary

Build stats

  • Start Time: 2022-02-11T09:50:19.252+0000

  • Duration: 27 min 39 sec

Test stats 🧪

Test Results
Failed 0
Passed 525
Skipped 0
Total 525

🤖 GitHub comments

To re-run your PR in the CI, just comment with:

  • /test : Re-trigger the build.

@jsoriano jsoriano force-pushed the release-deprecation branch from 5cf33c8 to b719f91 Compare January 14, 2022 18:15
@jsoriano jsoriano force-pushed the release-deprecation branch from b719f91 to 0d43aab Compare January 14, 2022 18:16
@jsoriano jsoriano marked this pull request as ready for review January 19, 2022 18:34
Copy link
Copy Markdown
Contributor

@mtojek mtojek left a comment

Choose a reason for hiding this comment

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

Code changes LGTM, just left questions about dependent issues.

go.mod Outdated
sigs.k8s.io/yaml v1.2.0 // indirect
)

replace github.com/elastic/package-spec v1.3.0 => github.com/jsoriano/package-spec v0.0.0-20220112095620-98e0d531845d
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I understand that this PR is blocked on the other one: elastic/package-spec#256?

// SearchOptions specify the query parameters without the package name for the search API
type SearchOptions struct {
Experimental bool `url:"experimental"`
Prerelease bool `url:"prerelease"`
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

... oh, and on the EPR release.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Yes, I think this will be the last piece to merge.

Copy link
Copy Markdown
Contributor

@mtojek mtojek left a comment

Choose a reason for hiding this comment

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

Approved, but considering the order of other pull requests :)

@jsoriano
Copy link
Copy Markdown
Member Author

jsoriano commented Feb 8, 2022

Testing with integrations repo in elastic/integrations#2651

@jsoriano jsoriano force-pushed the release-deprecation branch from 6c6de82 to 21b91cb Compare February 10, 2022 10:57
@jsoriano
Copy link
Copy Markdown
Member Author

Added changes to implement in elastic-package status the same conventions for release labels than the ones decided for Kibana (elastic/kibana#122973).

@mtojek could you please take another look?

@jsoriano jsoriano requested a review from mtojek February 10, 2022 11:17
Copy link
Copy Markdown
Contributor

@mtojek mtojek left a comment

Choose a reason for hiding this comment

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

LGTM

@jsoriano
Copy link
Copy Markdown
Member Author

/test

@jsoriano jsoriano merged commit f661b6f into elastic:main Feb 11, 2022
@jsoriano jsoriano deleted the release-deprecation branch February 11, 2022 10:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Team:Ecosystem Label for the Packages Ecosystem team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants