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 UBI source URL #5506

Merged
merged 2 commits into from
Jun 22, 2021
Merged

Fix UBI source URL #5506

merged 2 commits into from
Jun 22, 2021

Conversation

jmlrt
Copy link
Member

@jmlrt jmlrt commented Jun 21, 2021

Motivation/summary

This commit fix the source URL for UBI image to ensure that it stays
consistent with the URL generated in
https://artifacts.elastic.co/reports/dependencies/dependencies-current.html

Related issues

None

This commit fix the source URL for UBI image to ensure that it stays
consistent with the URL generated in
https://artifacts.elastic.co/reports/dependencies/dependencies-current.html
@jmlrt jmlrt requested a review from axw June 21, 2021 12:35
@apmmachine
Copy link
Contributor

apmmachine commented Jun 21, 2021

💚 Build Succeeded

the below badges are clickable and redirect to their specific view in the CI or DOCS
Pipeline View Test View Changes Artifacts preview

Expand to view the summary

Build stats

  • Build Cause: Pull request #5506 updated

  • Start Time: 2021-06-21T13:48:38.604+0000

  • Duration: 37 min 4 sec

  • Commit: a56df5a

Test stats 🧪

Test Results
Failed 0
Passed 6122
Skipped 120
Total 6242

Trends 🧪

Image of Build Times

Image of Tests

Copy link
Member

@axw axw left a comment

Choose a reason for hiding this comment

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

LGTM. Assuming this will need to go into 7.14, can you please label v7.14.0 to kick off an automated backport?

@jmlrt
Copy link
Member Author

jmlrt commented Jun 21, 2021

Thanks @axw 👍🏻
This should also eventually go into 7.13.3, should I also create a v7.13.3 label?

@jmlrt jmlrt added the v7.14.0 label Jun 21, 2021
@axw
Copy link
Member

axw commented Jun 22, 2021

@jmlrt perhaps not very intuitively, v7.13.0 is the label to use for backporting to the next 7.13.x release.

@axw axw added the v7.13.0 label Jun 22, 2021
@axw axw merged commit bd52909 into elastic:master Jun 22, 2021
mergify bot pushed a commit that referenced this pull request Jun 22, 2021
This commit fix the source URL for UBI image to ensure that it stays
consistent with the URL generated in
https://artifacts.elastic.co/reports/dependencies/dependencies-current.html

(cherry picked from commit bd52909)
mergify bot pushed a commit that referenced this pull request Jun 22, 2021
This commit fix the source URL for UBI image to ensure that it stays
consistent with the URL generated in
https://artifacts.elastic.co/reports/dependencies/dependencies-current.html

(cherry picked from commit bd52909)
axw pushed a commit that referenced this pull request Jun 22, 2021
This commit fix the source URL for UBI image to ensure that it stays
consistent with the URL generated in
https://artifacts.elastic.co/reports/dependencies/dependencies-current.html

(cherry picked from commit bd52909)

Co-authored-by: Julien Mailleret <[email protected]>
axw pushed a commit that referenced this pull request Jun 22, 2021
This commit fix the source URL for UBI image to ensure that it stays
consistent with the URL generated in
https://artifacts.elastic.co/reports/dependencies/dependencies-current.html

(cherry picked from commit bd52909)

Co-authored-by: Julien Mailleret <[email protected]>
@jmlrt jmlrt deleted the fix-ubi-source-url branch June 22, 2021 07:40
stuartnelson3 pushed a commit to stuartnelson3/apm-server that referenced this pull request Jun 22, 2021
This commit fix the source URL for UBI image to ensure that it stays
consistent with the URL generated in
https://artifacts.elastic.co/reports/dependencies/dependencies-current.html
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants