Skip to content

cloudbeat artifact path fix#288

Merged
ph merged 2 commits intoelastic:mainfrom
oren-zohar:cloudbeat-artifact-path-fix
Apr 4, 2022
Merged

cloudbeat artifact path fix#288
ph merged 2 commits intoelastic:mainfrom
oren-zohar:cloudbeat-artifact-path-fix

Conversation

@oren-zohar
Copy link
Copy Markdown
Contributor

What does this PR do?

Fixes cloudbeat artifact path.

Why is it important?

Since cloudbeat is in a separated repo from the other beats, so is its artifact. In order for the beat to pull successfully cloudbeat artifact, the path should be cloudbeat, and not beats/cloudbeat.

Checklist

  • My code follows the style guidelines of this project
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • I have made corresponding change to the default configuration files
  • I have added tests that prove my fix is effective or that my feature works
  • I have added an entry in CHANGELOG.next.asciidoc or CHANGELOG-developer.next.asciidoc.

@oren-zohar oren-zohar added the bug Something isn't working label Apr 4, 2022
@mergify
Copy link
Copy Markdown
Contributor

mergify bot commented Apr 4, 2022

This pull request does not have a backport label. Could you fix it @oren-zohar? 🙏
To fixup this pull request, you need to add the backport labels for the needed
branches, such as:

  • backport-v./d./d./d is the label to automatically backport to the 8./d branch. /d is the digit

NOTE: backport-skip has been added to this pull request.

@elasticmachine
Copy link
Copy Markdown
Contributor

elasticmachine commented Apr 4, 2022

💚 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 preview

Expand to view the summary

Build stats

  • Start Time: 2022-04-04T11:03:22.053+0000

  • Duration: 20 min 38 sec

🤖 GitHub comments

To re-run your PR in the CI, just comment with:

  • /test : Re-trigger the build.

  • /package : Generate the packages.

  • run integration tests : Run the Elastic Agent Integration tests.

  • run end-to-end tests : Generate the packages and run the E2E Tests.

  • run elasticsearch-ci/docs : Re-trigger the docs validation. (use unformatted text in the comment!)

@oren-zohar oren-zohar marked this pull request as draft April 4, 2022 10:59
@oren-zohar oren-zohar marked this pull request as ready for review April 4, 2022 11:18
@oren-zohar oren-zohar requested a review from ph April 4, 2022 11:18
@ph ph added the backport-v8.3.0 Automated backport with mergify label Apr 4, 2022
@ph ph merged commit 545f6ff into elastic:main Apr 4, 2022
mergify bot pushed a commit that referenced this pull request Apr 4, 2022
* cloudbeat artifact path fix

(cherry picked from commit 545f6ff)
@oren-zohar oren-zohar deleted the cloudbeat-artifact-path-fix branch April 4, 2022 16:21
ph pushed a commit that referenced this pull request Apr 4, 2022
* cloudbeat artifact path fix

(cherry picked from commit 545f6ff)

Co-authored-by: Oren Zohar <85433724+oren-zohar@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport-v8.2.0 backport-v8.3.0 Automated backport with mergify bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants