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

[Backport 2.x] React to changes in ActionListener and ActionFuture from #9082 #467

Merged
merged 1 commit into from
Aug 14, 2023

Conversation

opensearch-trigger-bot[bot]
Copy link

Backport 6abaa3c from #464

* React to changes in ActionListener and ActionFuture from #9082

Signed-off-by: Craig Perkins <[email protected]>

* Checkin all files

Signed-off-by: Craig Perkins <[email protected]>

* LockServiceIT

Signed-off-by: Craig Perkins <[email protected]>

---------

Signed-off-by: Craig Perkins <[email protected]>
(cherry picked from commit 6abaa3c)
@joshpalis
Copy link
Member

Seems the related changes upstream have not been backported yet : opensearch-project/OpenSearch#9082. Will wait to re-run checks until the changes are in 2.x

@kaituo kaituo merged commit f510461 into 2.x Aug 14, 2023
@github-actions github-actions bot deleted the backport/backport-464-to-2.x branch August 14, 2023 20:51
@cwperks
Copy link
Member

cwperks commented Aug 15, 2023

This may need to be temporarily reverted until core backports opensearch-project/OpenSearch#9082 to 2.x. @derek-ho What PRs is this blocking?

@joshpalis
Copy link
Member

This may need to be temporarily reverted until core backports opensearch-project/OpenSearch#9082 to 2.x. @derek-ho What PRs is this blocking?

Backport PR has been created opensearch-project/OpenSearch#9352, perhaps its best to wait for this to get merged in instead of reverting. @cwperks wdyt

@joshpalis
Copy link
Member

Tagging this related autocut issue for the 2.10.0 build failure : #470

@cwperks
Copy link
Member

cwperks commented Aug 15, 2023

We need to publish new maven snapshots. Seeing if I can do that from github now.

@cwperks
Copy link
Member

cwperks commented Aug 15, 2023

The maven snapshots have been published with this change included.

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