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

[CI] Fix permissions should not fail #18899

Merged
merged 1 commit into from
Jun 2, 2020
Merged

Conversation

v1v
Copy link
Member

@v1v v1v commented Jun 2, 2020

What does this PR do?

Since #18847 was merged there are some errors when fixing the permissions in some MacOSX. This particular step should not fail the build as it's just intended to help with the cleaning up of the workspaces in the static workers.

Why is it important?

Avoid any misleading with steps that are not required to catch the error.

Screenshots

image

image

@v1v v1v self-assigned this Jun 2, 2020
@v1v v1v added the automation label Jun 2, 2020
@botelastic botelastic bot added the needs_team Indicates that the issue/PR needs a Team:* label label Jun 2, 2020
@v1v v1v requested a review from a team June 2, 2020 14:35
@botelastic botelastic bot added Team:Automation Label for the Observability productivity team and removed needs_team Indicates that the issue/PR needs a Team:* label labels Jun 2, 2020
@v1v v1v marked this pull request as ready for review June 2, 2020 14:37
@v1v v1v merged commit 6369b78 into elastic:master Jun 2, 2020
v1v added a commit to v1v/beats that referenced this pull request Jun 2, 2020
v1v added a commit to v1v/beats that referenced this pull request Jun 2, 2020
v1v added a commit to v1v/beats that referenced this pull request Jun 2, 2020
…-stage-level

* upstream/master:
  [CI] Fix permissions should not fail (elastic#18899)
  Revert "Allow the Docker image to be run with a random user id (elastic#12905)" (elastic#18872)
  Add new fields to HAProxy module of Metricbeat (elastic#18523)
  Avoid duplicate names in dynamic_templates (elastic#18849)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
automation Team:Automation Label for the Observability productivity team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants