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

[hotfix] Miscellaneous fixes on GHA workflows #3839

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

yuxiqian
Copy link
Contributor

@yuxiqian yuxiqian commented Jan 7, 2025

This PR contains some pretty trivial fixes on our GHA workflows:

  • Changes PR stale cycle to 120d / 60d

Now, our actions/stale workflow closes inactive PRs with a 60 / 30 days cycle, that is to say, 60 days' inactive to label it "Stale" and 30 more days to close it. But it turns out to be too eager and has accidentally closed many useful PRs.

This change has been discussed with other maintainers.

  • Update outdated "Labeler"

Labeler can automatically detect what files have been touched, and categorize PRs with suitable labels, based on a predefined ruleset. It's just a routine update on the rule definition.

  • Run nightly CI on a more "random" time instead of 00:00 UTC

There are really too much CI workflows scheduled to run on UTC midnight that Apache's download server could not handle (like this). Changing it to another "arbitrary" time to reduce such failure.

@github-actions github-actions bot added the build label Jan 7, 2025
@yuxiqian
Copy link
Contributor Author

yuxiqian commented Jan 7, 2025

ping @leonardBang

@yuxiqian yuxiqian force-pushed the misc/workflow-update branch from 2520f5e to 303a6e9 Compare January 8, 2025 02:56
@leonardBang
Copy link
Contributor

Thanks @yuxiqian for considering the newly-added maxcompute label, nice work

@yuxiqian
Copy link
Contributor Author

Seems MaxCompute unit tests are failing, but didn't get exposed since it never runs before.
Could @dingxin-tech please take a look?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants