Skip to content

[Backport 2.x] Remove endpoint_suffix dependency on account key#2761

Closed
opensearch-trigger-bot[bot] wants to merge 1 commit into2.xfrom
backport/backport-2485-to-2.x
Closed

[Backport 2.x] Remove endpoint_suffix dependency on account key#2761
opensearch-trigger-bot[bot] wants to merge 1 commit into2.xfrom
backport/backport-2485-to-2.x

Conversation

@opensearch-trigger-bot
Copy link
Copy Markdown
Contributor

Backport 365e07c from #2485

Signed-off-by: Raphael Lopez <raphael.lopez@aiven.io>
(cherry picked from commit 365e07c)
@opensearch-trigger-bot opensearch-trigger-bot bot requested a review from a team as a code owner April 5, 2022 00:13
@opensearch-ci-bot
Copy link
Copy Markdown
Collaborator

✅   Gradle Check success 3afcb28
Log 4169

Reports 4169

@dblock
Copy link
Copy Markdown
Member

dblock commented Apr 5, 2022

@raphlopez I am afraid you'll have to manually backport this, see DCO error (not sure what the right way to fix that is)

@tlfeng
Copy link
Copy Markdown
Contributor

tlfeng commented Apr 6, 2022

Hi @dblock , looks like @nknize found a way to fix the DCO check in this situation, which is adding an additional line of sign-off. (#2654 (comment)) 😄

Can be achieved by either one:

  1. Adding a line of sign-off by another person.
  2. Adding a line of the required sign-off that requested by the DCO App (in this case: Signed-off-by: Raphael Lopez <85206537+raphlopez@users.noreply.github.com>)

image

@dblock
Copy link
Copy Markdown
Member

dblock commented Apr 6, 2022

@tlfeng Feel free to do this and push an update to this auto-PR, but I'd prefer the author of the original PR ensure that things get backported everywhere needed.

@raphlopez
Copy link
Copy Markdown
Contributor

raphlopez commented Apr 7, 2022

I created a couple separate PRs to manually backport to 2.x and 2.0:

Seems the DCO check already passed for both of them. Let me know if I did that correctly or if there's anything I need to change.

@tlfeng
Copy link
Copy Markdown
Contributor

tlfeng commented Apr 7, 2022

@raphlopez Thanks for your time in creating the 2 PRs!
You've done correctly. Sorry about the trouble with the DCO check, the application uses a weird criteria to check the author name and email for cherry-picked commits. 😅
I will close this PR.

@tlfeng tlfeng closed this Apr 7, 2022
@peterzhuamazon peterzhuamazon deleted the backport/backport-2485-to-2.x branch July 8, 2022 18:48
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