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

.github/workflows: replace probot-stale with equivalent github action #45230

Merged
merged 2 commits into from
Mar 23, 2022

Conversation

ljflores
Copy link
Contributor

@ljflores ljflores commented Mar 2, 2022

According to the stale bot code (https://github.com/ceph/ceph/blob/master/.github/stale.yml), PRs should become stale automatically after 60 days of inactivity. After 90 days, they should automatically close. This last part is not working properly; PRs are correctly marked stale, but they fail to close after 90 days.

We currently use probot-stale: https://github.com/probot/stale. According to several issues reported for this repository, there is a widespread problem of probot-stale not working properly in github projects (i.e. probot/stale#349). Some users recommend using https://github.com/actions/stale instead.

This PR proposes a switch from probot-stale to the GitHub stale action workflow.

Fixes: https://tracker.ceph.com/issues/54443
Signed-off-by: Laura Flores lflores@redhat.com

Checklist

  • Tracker (select at least one)
    • References tracker ticket
    • Very recent bug; references commit where it was introduced
    • New feature (ticket optional)
    • Doc update (no ticket needed)
    • Code cleanup (no ticket needed)
  • Component impact
    • Affects Dashboard, opened tracker ticket
    • Affects Orchestrator, opened tracker ticket
    • No impact that needs to be tracked
  • Documentation (select at least one)
    • Updates relevant documentation
    • No doc update is appropriate
  • Tests (select at least one)
Show available Jenkins commands
  • jenkins retest this please
  • jenkins test classic perf
  • jenkins test crimson perf
  • jenkins test signed
  • jenkins test make check
  • jenkins test make check arm64
  • jenkins test submodules
  • jenkins test dashboard
  • jenkins test dashboard cephadm
  • jenkins test api
  • jenkins test docs
  • jenkins render docs
  • jenkins test ceph-volume all
  • jenkins test ceph-volume tox

We currently use probot-stale (https://github.com/probot/stale) to mark stale PRs. According to several issues reported for this repository, there is a widespread problem of probot-state not working properly in github projects (i.e. probot/stale#349). Some users recommend using https://github.com/actions/stale instead.

Fixes: https://tracker.ceph.com/issues/54443
Signed-off-by: Laura Flores <lflores@redhat.com>
@ljflores ljflores requested a review from epuertat March 2, 2022 20:16
@github-actions github-actions bot added the CI Continuous Integration label Mar 2, 2022
@ljflores
Copy link
Contributor Author

ljflores commented Mar 2, 2022

jenkins test windows

@ljflores
Copy link
Contributor Author

ljflores commented Mar 2, 2022

jenkins test make check

@ljflores
Copy link
Contributor Author

ljflores commented Mar 2, 2022

jenkins test api

Copy link
Member

@epuertat epuertat left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! Thanks for taking care of these things @ljflores !

@ljflores
Copy link
Contributor Author

ljflores commented Mar 3, 2022

jenkins test make check

Copy link
Member

@neha-ojha neha-ojha left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@ljflores
Copy link
Contributor Author

ljflores commented Mar 5, 2022

jenkins test make check

@ljflores
Copy link
Contributor Author

ljflores commented Mar 7, 2022

@epuertat do you think any permissions need to be explicitly specified as seen here? https://github.com/actions/stale#recommended-permissions

permissions:
  issues: write
  pull-requests: write

I checked our other github action workflows and didn't see a need for it. But let me know what you think.

@ljflores
Copy link
Contributor Author

ljflores commented Mar 9, 2022

jenkins test windows

@ljflores ljflores requested a review from epuertat March 22, 2022 16:31
@ljflores
Copy link
Contributor Author

Merging this. I will keep tabs on it to make sure it is working properly as PRs become stale.

@ljflores ljflores merged commit ea290a4 into ceph:master Mar 23, 2022
@ljflores ljflores deleted the wip-github-actions-stale branch March 23, 2022 16:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI Continuous Integration
Projects
None yet
3 participants