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

dockerLogin step with less verbose output #691

Merged
merged 1 commit into from
Aug 6, 2020
Merged

dockerLogin step with less verbose output #691

merged 1 commit into from
Aug 6, 2020

Conversation

v1v
Copy link
Member

@v1v v1v commented Jul 31, 2020 •
edited
Loading

What does this PR do?

Less verbose output, for instance

10:29:57  + set +x
10:29:57  /usr/bin/host
10:29:57  /usr/bin/dig

Why is it important?

It does not provide any value though

Test

image

@v1v v1v self-assigned this Jul 31, 2020
@v1v v1v added the automation label Jul 31, 2020
@v1v v1v requested a review from a team July 31, 2020 10:16
@v1v v1v marked this pull request as ready for review July 31, 2020 10:17
@botelastic botelastic bot added the groovy label Jul 31, 2020
@apmmachine
Copy link
Contributor

💚 Build Succeeded

Pipeline View Test View Changes Artifacts preview

Expand to view the summary

Build stats

  • Build Cause: [Pull request #691 opened]

  • Start Time: 2020-07-31T10:16:14.803+0000

  • Duration: 15 min 5 sec

Test stats 🧪

Test Results
Failed 0
Passed 746
Skipped 9
Total 755

@v1v v1v merged commit 9ec7d4f into elastic:master Aug 6, 2020
@v1v v1v deleted the feature/less-verbose branch August 6, 2020 06:35
v1v added a commit that referenced this pull request Aug 6, 2020
@v1v
Merge remote-tracking branch 'upstream/master' into feature/2.0-beats
e9e7c1c 
* upstream/master:
  dockerLogin step with less verbose output (#691)
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@mdelapenya mdelapenya mdelapenya approved these changes

@cachedout cachedout cachedout approved these changes

Assignees

@v1v v1v

Labels
automation groovy
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@v1v @apmmachine @cachedout @mdelapenya