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

chore: bump elastic-package to v0.25.0 #1721

Merged
merged 1 commit into from
Oct 26, 2021

Conversation

mdelapenya
Copy link
Contributor

@mdelapenya mdelapenya commented Oct 26, 2021

What does this PR do?

It bumps elastic-package to 0.25.0

Why is it important?

See #1720

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 run the Unit tests (make unit-test), and they are passing locally
  • I have run the End-2-End tests for the suite I'm working on, and they are passing locally
  • I have noticed new Go dependencies (run make notice in the proper directory)

Author's Checklist

  • [ ]

Related issues

@mergify
Copy link
Contributor

mergify bot commented Oct 26, 2021

This pull request is now in conflict. Could you fix it @mdelapenya? 🙏
To fixup this pull request, you can check out it locally. See documentation: https://help.github.com/articles/checking-out-pull-requests-locally/

git fetch upstream
git checkout -b bump-elastic-package upstream/bump-elastic-package
git merge upstream/master
git push upstream bump-elastic-package

@mergify
Copy link
Contributor

mergify bot commented Oct 26, 2021

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

  • v./d./d./d is the label to automatically backport to the 7./d branch. /d is the digit
    NOTE: backport-skip has been added to this pull request.

@mergify mergify bot added the backport-skip Skip notification from the automated backport with mergify label Oct 26, 2021
@mdelapenya mdelapenya changed the base branch from master to 7.16 October 26, 2021 08:33
@mdelapenya mdelapenya requested a review from a team October 26, 2021 08:33
@mdelapenya
Copy link
Contributor Author

Sending to 7.16 to verify the resolution. If CI passes we should:

  • forward port to master
  • backport to 7.15

@elasticmachine
Copy link
Contributor

elasticmachine commented Oct 26, 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 preview

Expand to view the summary

Build stats

  • Start Time: 2021-10-26T08:48:38.837+0000

  • Duration: 39 min 30 sec

  • Commit: 4bd98a9

Test stats 🧪

Test Results
Failed 0
Passed 98
Skipped 0
Total 98

💚 Flaky test report

Tests succeeded.

🤖 GitHub comments

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

  • /test : Re-trigger the build.

@mdelapenya
Copy link
Contributor Author

In https://beats-ci.elastic.co/blue/organizations/jenkins/e2e-tests%2Fe2e-testing-mbp/detail/PR-1721/2/pipeline/219 we can see that the fix is effective, and the runtime deps are provisioned by elastic-package:

Screenshot 2021-10-26 at 11 17 25

@mdelapenya mdelapenya added v7.15.0 and removed backport-skip Skip notification from the automated backport with mergify labels Oct 26, 2021
@mdelapenya mdelapenya self-assigned this Oct 26, 2021
@mdelapenya mdelapenya merged commit fa79597 into elastic:7.16 Oct 26, 2021
mdelapenya added a commit to mdelapenya/e2e-testing that referenced this pull request Oct 26, 2021
mergify bot pushed a commit that referenced this pull request Oct 26, 2021
mdelapenya added a commit that referenced this pull request Oct 26, 2021
(cherry picked from commit 1d213ca)

Co-authored-by: Manuel de la Peña <[email protected]>
@mdelapenya mdelapenya deleted the bump-elastic-package branch October 27, 2021 08:49
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants