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

Documenting usage of overriding Default Distribution Download URL with custom URL by setting external property customDistributionUrl #125

Merged

Conversation

Rishikesh1159
Copy link
Member

@Rishikesh1159 Rishikesh1159 commented Feb 24, 2022

Signed-off-by: Rishikesh1159 [email protected]

Description

This PR will allow developers to override Default Distribution Download URL with custom URL by setting "customDistributionUrl". This will help to pull artifacts from custom location for testing during release process.

Issues Resolved

#126

Check List

  • Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

…h custom URL by setting external property customDistributionUrl

Signed-off-by: Rishikesh1159 <[email protected]>
@Rishikesh1159 Rishikesh1159 requested a review from a team February 24, 2022 18:49
@dblock
Copy link
Member

dblock commented Mar 3, 2022

We rolled this back, right? Let us know when this can be merged @Rishikesh1159.

@Rishikesh1159
Copy link
Member Author

Yes we rolled this back. Sure @dblock I will let you know once I open a PR and get it merged in OpenSearch repo, then we can merge this.

@saratvemulapalli
Copy link
Member

@Rishikesh1159 we merged the changes to 2.0.
Is this ready for review?

@Rishikesh1159
Copy link
Member Author

@Rishikesh1159 we merged the changes to 2.0. Is this ready for review

@saratvemulapalli small change was needed from last commit. Made it now, it is ready for review.

@dblock
Copy link
Member

dblock commented Apr 20, 2022

@Rishikesh1159 should I merge this? I remember something about this change being reverted ...

@Rishikesh1159
Copy link
Member Author

Yes @dblock please merge it. After reverting it was fixed and merged changes with this PR :opensearch-project/OpenSearch#2420

@dblock dblock merged commit ba853c2 into opensearch-project:main Apr 22, 2022
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.

3 participants