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

[mergify] close stalled PRs with the bump automation#1852

Merged
v1v merged 4 commits into
elastic:masterfrom
v1v:feature/mergify-stalled-bumps
Dec 1, 2021
Merged

[mergify] close stalled PRs with the bump automation#1852
v1v merged 4 commits into
elastic:masterfrom
v1v:feature/mergify-stalled-bumps

Conversation

@v1v
Copy link
Copy Markdown
Member

@v1v v1v commented Nov 30, 2021

What

  1. Delete open PRs older than 3 days on a daily basis if they are created by the apmmachine
  2. Auto-merge PRs if they are created by the apmmachine are part of the automated bump process

Why

  • Obsoleted bumps are not needed.
  • Some Automated PRs don't change the docker-compose.yml files anymore.

@v1v v1v requested a review from a team November 30, 2021 10:32
@v1v v1v self-assigned this Nov 30, 2021
@mergify
Copy link
Copy Markdown
Contributor

mergify Bot commented Nov 30, 2021

This pull request does not have a backport label. Could you fix it @v1v? 🙏
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 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 Nov 30, 2021
Comment thread .mergify.yml
- check-success=beats-ci/e2e-testing/pr-merge
- label=automation
- files~=docker-compose\.yml$
- author=apmmachine
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.

For instance:

#1848 matched the previous behaviour of chjanges in the docker-compose.yml but #1849 somehow is only touching one file

This author filtering will allow the merge for the above-mentioned PRs.

Comment thread .mergify.yml
- "#check-success>0"
- schedule=Mon-Mon 06:00-10:00[Europe/Paris]
#- created-at<3 days ago
- created-at<2 days ago
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.

Unrelated, but commented, so this will ensure PRs older than 2 days will be affected

@elasticmachine
Copy link
Copy Markdown
Contributor

elasticmachine commented Nov 30, 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-12-01T12:51:06.929+0000

  • Duration: 4 min 3 sec

  • Commit: ffa1475

Test stats 🧪

Test Results
Failed 0
Passed 19
Skipped 0
Total 19

💚 Flaky test report

Tests succeeded.

🤖 GitHub comments

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

  • /test : Re-trigger the build.

@v1v v1v merged commit bd363a9 into elastic:master Dec 1, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

backport-skip Skip notification from the automated backport with mergify

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants