Skip to content

Conversation

@ydshieh
Copy link
Collaborator

@ydshieh ydshieh commented Oct 31, 2023

What does this PR do?

Now our CIs run on AWS runners, and they are spun up when a action is triggered and removed afterward, it doesn't make sense to check the runner status as before (where the runners existed forever).

@ydshieh ydshieh requested a review from LysandreJik October 31, 2023 07:56
runner_failed = True if runner_env_status is not None and runner_env_status != "success" else False
# runner_not_available = True if runner_status is not None and runner_status != "success" else False
# runner_failed = True if runner_env_status is not None and runner_env_status != "success" else False
# Let's keep the lines regardig runners' status (we might be able to use them again in the future)
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Just in case :-)

Comment on lines +779 to +780
runner_not_available = False
runner_failed = False
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

set to False (so no failure)

@HuggingFaceDocBuilderDev
Copy link

HuggingFaceDocBuilderDev commented Oct 31, 2023

The documentation is not available anymore as the PR was closed or merged.

Copy link
Member

@LysandreJik LysandreJik left a comment

Choose a reason for hiding this comment

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

Sounds good, LGTM!

@ydshieh ydshieh merged commit 9dc4ce9 into main Oct 31, 2023
@ydshieh ydshieh deleted the remove_runner_check branch October 31, 2023 10:59
EduardoPach pushed a commit to EduardoPach/transformers that referenced this pull request Nov 19, 2023
Disable runner check

Co-authored-by: ydshieh <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants