Skip to content

[CI] docker login for linux only#24030

Merged
v1v merged 1 commit intoelastic:masterfrom
v1v:feature/docker-login-for-linux-only
Feb 15, 2021
Merged

[CI] docker login for linux only#24030
v1v merged 1 commit intoelastic:masterfrom
v1v:feature/docker-login-for-linux-only

Conversation

@v1v
Copy link
Copy Markdown
Member

@v1v v1v commented Feb 13, 2021

What does this PR do?

As a consequence of #23854 then it's required the login for Linux workers to interact with the packaging correctly. But it's not required for other platforms such as darwin and windows.

Why is it important?

Docker login in Darwin is not trivial since it requires some docker-machine context, some Darwin workers got no issues regarding the login since it seems the docker-machine context is correct, but some of the CI Darwin workers got some issues.

This should fix the issues of broken builds in the master and 7.x branches regarding those docker login issues.

Workers

The below workers have been disabled until this fix is not merged for master and 7.x branches.

@v1v v1v self-assigned this Feb 13, 2021
@botelastic botelastic Bot added the needs_team Indicates that the issue/PR needs a Team:* label label Feb 13, 2021
@elasticmachine
Copy link
Copy Markdown
Contributor

💚 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 #24030 opened

  • Start Time: 2021-02-13T15:08:28.470+0000

  • Duration: 80 min 34 sec

  • Commit: 3111609

Test stats 🧪

Test Results
Failed 0
Passed 45843
Skipped 4762
Total 50605

Trends 🧪

Image of Build Times

Image of Tests

💚 Flaky test report

Tests succeeded.

Expand to view the summary

Test stats 🧪

Test Results
Failed 0
Passed 45843
Skipped 4762
Total 50605

@andresrc andresrc added the Team:Automation Label for the Observability productivity team label Feb 14, 2021
@botelastic botelastic Bot removed the needs_team Indicates that the issue/PR needs a Team:* label label Feb 14, 2021
@v1v v1v marked this pull request as ready for review February 15, 2021 09:06
@v1v v1v requested a review from a team as a code owner February 15, 2021 09:06
@v1v v1v merged commit ee0d74a into elastic:master Feb 15, 2021
@v1v v1v deleted the feature/docker-login-for-linux-only branch February 15, 2021 15:06
v1v added a commit to v1v/beats that referenced this pull request Feb 15, 2021
v1v added a commit that referenced this pull request Feb 16, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Team:Automation Label for the Observability productivity team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants