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

Add action to label and close stalled PRs #8178

Merged
merged 1 commit into from
Jun 21, 2023

Conversation

andrross
Copy link
Member

Description

This action labels PRs as "stalled" after 30 days, then closes them 7 days later if no follow up.

The following choices here were all fairly arbitrary and open for discussion:

  • When to run: every morning, Pacific Time
  • What label to use: stalled, because it already exists in the repo but appears to be unused
  • Inactivity period to label as stalled: 30 days
  • Period after stalled before closing: 7 days

Related Issues

Resolves #8093

Check List

  • New functionality includes testing.
    • All tests pass
  • New functionality has been documented.
    • New functionality has javadoc added
  • Commits are signed per the DCO using --signoff
  • Commit changes are listed out in CHANGELOG.md file (See: Changelog)

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

@github-actions
Copy link
Contributor

Gradle Check (Jenkins) Run Completed with:

@codecov
Copy link

codecov bot commented Jun 20, 2023

Codecov Report

Merging #8178 (117618c) into main (5dce570) will decrease coverage by 0.20%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##               main    #8178      +/-   ##
============================================
- Coverage     70.93%   70.73%   -0.20%     
+ Complexity    56633    56555      -78     
============================================
  Files          4719     4719              
  Lines        267555   267555              
  Branches      39206    39206              
============================================
- Hits         189778   189258     -520     
- Misses        61721    62345     +624     
+ Partials      16056    15952     -104     

see 482 files with indirect coverage changes

.github/workflows/stalled.yml Outdated Show resolved Hide resolved
.github/workflows/stalled.yml Show resolved Hide resolved
Signed-off-by: Andrew Ross <andrross@amazon.com>
@github-actions
Copy link
Contributor

Gradle Check (Jenkins) Run Completed with:

  • RESULT: UNSTABLE ❕
  • TEST FAILURES:
      2 org.opensearch.index.shard.RemoteStoreRefreshListenerTests.classMethod
      1 org.opensearch.search.backpressure.SearchBackpressureIT.testSearchShardTaskCancellationWithHighCpu
      1 org.opensearch.indices.replication.SegmentReplicationIT.testScrollCreatedOnReplica
      1 org.opensearch.index.shard.RemoteStoreRefreshListenerTests.testRefreshSuccessAfterFailureInFirstAttemptAfterSnapshotAndMetadataUpload

@andrross andrross merged commit 9c5c6eb into opensearch-project:main Jun 21, 2023
@andrross andrross deleted the stalled-prs branch June 21, 2023 23:18
gaiksaya pushed a commit to gaiksaya/OpenSearch that referenced this pull request Jun 21, 2023
Signed-off-by: Andrew Ross <andrross@amazon.com>
Signed-off-by: Sayali Gaikawad <gaiksaya@amazon.com>
imRishN pushed a commit to imRishN/OpenSearch that referenced this pull request Jun 27, 2023
Signed-off-by: Andrew Ross <andrross@amazon.com>
Signed-off-by: Rishab Nahata <rnnahata@amazon.com>
shiv0408 pushed a commit to Gaurav614/OpenSearch that referenced this pull request Apr 25, 2024
Signed-off-by: Andrew Ross <andrross@amazon.com>
Signed-off-by: Shivansh Arora <hishiv@amazon.com>
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.

Automatically close draft PRs after a period of inactivity
3 participants