Skip to content

CI: use the withAzureCredentials step#1240

Merged
v1v merged 1 commit intoelastic:masterfrom
v1v:feature/use-credentials-azure
Mar 24, 2021
Merged

CI: use the withAzureCredentials step#1240
v1v merged 1 commit intoelastic:masterfrom
v1v:feature/use-credentials-azure

Conversation

@v1v
Copy link
Copy Markdown
Member

@v1v v1v commented Mar 23, 2021

IMPORTANT it's required a new release of the existing apm-pipeline-library to be able to use the new step

What

Enable azure credentials for running the tests on linux and windows.

Issues

Requires elastic/apm-pipeline-library#1048

@v1v v1v requested review from a team March 23, 2021 11:38
@v1v v1v self-assigned this Mar 23, 2021
@ghost
Copy link
Copy Markdown

ghost commented Mar 23, 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: Started by user Victor Martinez

  • Start Time: 2021-03-23T23:53:41.716+0000

  • Duration: 59 min 52 sec

  • Commit: 063f266

Test stats 🧪

Test Results
Failed 0
Passed 18759
Skipped 80
Total 18839

Trends 🧪

Image of Build Times

Image of Tests

Copy link
Copy Markdown
Contributor

@russcam russcam left a comment

Choose a reason for hiding this comment

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

LGTM

@v1v v1v marked this pull request as ready for review March 24, 2021 12:25
@v1v
Copy link
Copy Markdown
Member Author

v1v commented Mar 24, 2021

Credentials are configured:

image

image

@v1v v1v merged commit e540d53 into elastic:master Mar 24, 2021
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