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

Avoid appending origin to branch name twice for checking out PA RCA #284

Conversation

kiranprakash154
Copy link
Contributor

@kiranprakash154 kiranprakash154 commented Sep 23, 2022

Signed-off-by: Kiran Prakash awskiran@amazon.com

Is your feature request related to a problem? Please provide an existing Issue # , or describe.

The Builds are failing with this error

* What went wrong:
Execution failed for task ':cloneRcaGitRepo'.
> org.eclipse.jgit.api.errors.RefNotFoundException: Ref origin/origin/2.x cannot be resolved

This adds safeguards against build triggers with a branch or without a branch.

Check List

  • New functionality includes testing.
    • All tests pass
  • New functionality has been documented.
    • New functionality has javadoc added
  • 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.

Signed-off-by: Kiran Prakash <awskiran@amazon.com>
@kiranprakash154 kiranprakash154 changed the title safeguard against appending origin twice Avoid appending origin to branch name twice for checking out PA RCA Sep 23, 2022
@kiranprakash154 kiranprakash154 self-assigned this Sep 23, 2022
@codecov-commenter
Copy link

codecov-commenter commented Sep 23, 2022

Codecov Report

Merging #284 (c5138fe) into main (bdb4e7f) will decrease coverage by 0.15%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##               main     #284      +/-   ##
============================================
- Coverage     71.84%   71.68%   -0.16%     
+ Complexity      368      367       -1     
============================================
  Files            44       44              
  Lines          2522     2522              
  Branches        171      171              
============================================
- Hits           1812     1808       -4     
- Misses          598      601       +3     
- Partials        112      113       +1     
Impacted Files Coverage Δ
...ormanceanalyzer/writer/EventLogQueueProcessor.java 55.95% <0.00%> (-4.77%) ⬇️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

sruti1312
sruti1312 previously approved these changes Sep 26, 2022
@kiranprakash154 kiranprakash154 marked this pull request as ready for review September 26, 2022 17:25
@kiranprakash154 kiranprakash154 requested a review from a team September 26, 2022 17:25
build.gradle Outdated Show resolved Hide resolved
dzane17
dzane17 previously approved these changes Sep 26, 2022
Signed-off-by: Kiran Prakash <awskiran@amazon.com>
@kiranprakash154 kiranprakash154 merged commit f3687f4 into opensearch-project:main Sep 26, 2022
@kiranprakash154 kiranprakash154 deleted the kp/avoid-adding-origin-twice branch September 26, 2022 19:17
kaushalmahi12 pushed a commit to kaushalmahi12/performance-analyzer that referenced this pull request Oct 25, 2022
…pensearch-project#284)

* safeguard against appending origin twice

Signed-off-by: Kiran Prakash <awskiran@amazon.com>
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.

None yet

5 participants