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

[AUTO] Incremented version to 2.14.1. #653

Merged
merged 2 commits into from
Jun 4, 2024
Merged

Conversation

opensearch-trigger-bot[bot]
Copy link
Contributor

I've noticed that a new tag 2.14.0.0 was pushed, and incremented the version from 2.14.0 to 2.14.1.

@dbwiddis
Copy link
Member

dbwiddis commented Jun 4, 2024

Hi, maintainers! (@lezzago @qreshi @bowenlan-amzn @rishabhmaurya @getsaurabh02 @eirsep @sbcd90 @AWSHurneyt @engechas @riysaxen-amzn @jowg-amazon)

There is a possibility we may cut a 2.14.1 release due to opensearch-project/OpenSearch#13927 in which case your downstream dependencies, and their downstream dependencies, need this version bump merged.

Please merge this PR.

It's pretty fast to re-spin a GitHub Action test with a single mouse click if it's within 30 days but these version bump PRs are often lingering here well past that point, which requires more time for downstream maintainers to address these bump commits. I maintain a repo 2 dependencies down from this one so I have to nag two upstream repos to get this all done.

Merging this version bump commit is a checklist item on #626 which has not been completed. Downstream dependencies are unable to complete these checklist items either because of this upstream dependency. For KTLO downstream plugins, this lack of version bump has caused additional follow-on impacts with scripts merging elsewhere assuming new commits between versions.

All of this is a plea for this repo to be more proactive in merging these version bump PRs so I don't keep posting these nag comments. :)

@AWSHurneyt AWSHurneyt merged commit 8f7d13f into 2.14 Jun 4, 2024
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants