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

[QA] [Code Coverage] Add Three Dot Compare Url #70525

Merged
merged 2 commits into from
Jul 2, 2020

Conversation

wayneseymour
Copy link
Member

Summary

Grab the previous sha from the downloadPrevious fn,
pass it through to the ingestion,
and process it in the itemizeVcs fn.

In order to provide the three dot compare url,
this pr uploads the last sha to gcp.
Then on subsequent ci runs, it downloads the "old" previous
and uses in the three dot url.

pass it through to the ingestion,
and process it in the itemizeVcs fn.
@wayneseymour wayneseymour added v8.0.0 release_note:skip Skip the PR/issue when compiling release notes test-coverage issues & PRs for improving code test coverage backport:skip This commit does not require backporting labels Jul 1, 2020
@wayneseymour wayneseymour self-assigned this Jul 1, 2020
@wayneseymour
Copy link
Member Author

wayneseymour commented Jul 1, 2020

@kibanamachine
Copy link
Contributor

💚 Build Succeeded

Build metrics

✅ unchanged

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

@wayneseymour wayneseymour marked this pull request as ready for review July 2, 2020 01:01
@wayneseymour wayneseymour requested a review from a team as a code owner July 2, 2020 01:01

''', title)

def previous = sh(script: 'cat downloaded_previous.txt', label: '### Capture Previous Sha', returnStdout: true).trim()
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You can just do def previous = readFile(file: 'downloaded_previous.txt'). Shelling out to cat works too, but there's a built-in step for this

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I knew you'd have an insight on this! Thanks B!

Copy link
Member

@dmlemeshko dmlemeshko left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@wayneseymour wayneseymour merged commit 12460b4 into master Jul 2, 2020
@wayneseymour wayneseymour deleted the qa-coverage-previous-sha branch July 2, 2020 16:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport:skip This commit does not require backporting release_note:skip Skip the PR/issue when compiling release notes test-coverage issues & PRs for improving code test coverage v8.0.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants