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

Fix link to GCP upload in build scans #48248

Merged
merged 3 commits into from
Oct 21, 2019

Conversation

alpar-t
Copy link
Contributor

@alpar-t alpar-t commented Oct 18, 2019

Also changed the name of the link to make it less confusing

Also changed the name of the link to make it less confusing
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-core-infra (:Core/Infra/Build)

Copy link
Contributor

@mark-vieira mark-vieira left a comment

Choose a reason for hiding this comment

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

One comment regarding naming, otherwise LGTM.

@@ -18,8 +18,8 @@ buildScan {
tag 'CI'
tag System.getenv('JOB_NAME')
link 'Jenkins Build', System.getenv('BUILD_URL')
link 'GCP Upload',
"https://console.cloud.google.com/storage/elasticsearch-ci-artifacts/jobs/${jobName}/${buildNumber}.tar.bz2"
link 'GCP Logs',
Copy link
Contributor

Choose a reason for hiding this comment

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

I'm inclined to call this something more like "Build Logs" or "Additional build logs". The "GCP" bit here is irrelevant. If these were stored in S3 it would make no functional difference.

@alpar-t
Copy link
Contributor Author

alpar-t commented Oct 21, 2019

@elasticmachine update branch

1 similar comment
@alpar-t
Copy link
Contributor Author

alpar-t commented Oct 21, 2019

@elasticmachine update branch

@alpar-t alpar-t merged commit 59d5d3e into elastic:master Oct 21, 2019
alpar-t added a commit that referenced this pull request Oct 21, 2019
* Fix link to GCP upload in build scans

Also changed the name of the link to make it less confusing

* Change the name
alpar-t added a commit that referenced this pull request Oct 21, 2019
* Fix link to GCP upload in build scans

Also changed the name of the link to make it less confusing

* Change the name
alpar-t added a commit that referenced this pull request Oct 21, 2019
* Fix link to GCP upload in build scans

Also changed the name of the link to make it less confusing

* Change the name
alpar-t added a commit that referenced this pull request Oct 22, 2019
* Fix link to GCP upload in build scans

Also changed the name of the link to make it less confusing

* Change the name
@alpar-t alpar-t deleted the fix-gcp-upload-link branch November 11, 2019 09:38
@mark-vieira mark-vieira added the Team:Delivery Meta label for Delivery team label Nov 11, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants