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 tmate session to debug runner #9728

Merged
merged 3 commits into from
Sep 27, 2021
Merged

add tmate session to debug runner #9728

merged 3 commits into from
Sep 27, 2021

Conversation

HotThoughts
Copy link
Contributor

@HotThoughts HotThoughts commented Sep 27, 2021

Proposed changes:
Add a step right after backend's test. If the test fails, setup a tmate session, which allows us ssh into it and debug directly from there.

@HotThoughts HotThoughts added status:all-tests-required All check filters are set to true, and all tests are run and removed status:all-tests-required All check filters are set to true, and all tests are run labels Sep 27, 2021
@HotThoughts HotThoughts marked this pull request as ready for review September 27, 2021 16:01
@HotThoughts
Copy link
Contributor Author

HotThoughts commented Sep 27, 2021

@koernerfelicia Instead of setting up a new instance, I find a way to ssh into GitHub's runners. Would you interested to try it? I tested it and successfully got into the runner.

How to get in?
Check out the "Setup tmate session" step, you should a ssh command printed in the log like this SSH: ssh [email protected], for example, here.

Then open your terminal, ssh into it by ssh [email protected] (the username changes each time).

@koernerfelicia koernerfelicia merged commit b9efd30 into RasaHQ:tf-2.5-final Sep 27, 2021
@HotThoughts HotThoughts deleted the debug-windows-runner branch September 27, 2021 16:22
koernerfelicia pushed a commit that referenced this pull request Sep 28, 2021
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.

2 participants