Skip to content
This repository was archived by the owner on Oct 28, 2024. It is now read-only.

Add withAzureCredentials step#1048

Merged
v1v merged 2 commits intoelastic:masterfrom
v1v:feature/azure-credentials
Mar 23, 2021
Merged

Add withAzureCredentials step#1048
v1v merged 2 commits intoelastic:masterfrom
v1v:feature/azure-credentials

Conversation

@v1v
Copy link
Copy Markdown
Member

@v1v v1v commented Mar 23, 2021

What does this PR do?

Add new step to interact with azure

Why is it important?

Facilitate the consumer's pipelines.

Related issues

Closes #ISSUE

@v1v v1v added the enhancement New feature or request label Mar 23, 2021
@v1v v1v requested a review from russcam March 23, 2021 11:31
@v1v v1v self-assigned this Mar 23, 2021
@botelastic botelastic bot added the groovy label 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: Pull request #1048 updated

  • Start Time: 2021-03-23T13:16:13.934+0000

  • Duration: 33 min 58 sec

  • Commit: 5ea3dba

Test stats 🧪

Test Results
Failed 0
Passed 1081
Skipped 2
Total 1083

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 1081
Skipped 2
Total 1083

error 'withAzureCredentials: was not possible to get authentication info'
}
dir(path) {
writeFile file: credentialsFile, text: """
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

can we set the permissions of the file to 0600?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

writeFile does not support permissions yet (https://issues.jenkins.io/browse/JENKINS-47747) I'm a bit reluctant to add a sh/bat step only for that. Do you see any issues with the existing approach?

Co-authored-by: Ivan Fernandez Calvo <kuisathaverat@users.noreply.github.com>
@v1v v1v merged commit 02634ad into elastic:master Mar 23, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

enhancement New feature or request groovy

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants