[Testing] Setup SSH tests to cluster (internal testing only)#100
Closed
JoeZiminski wants to merge 6 commits into
Closed
[Testing] Setup SSH tests to cluster (internal testing only)#100JoeZiminski wants to merge 6 commits into
JoeZiminski wants to merge 6 commits into
Conversation
…n load. Do this next
JoeZiminski
force-pushed
the
generalising_ssh_tests
branch
from
6e278f6 to
ede0ff9
Compare
Closed
JoeZiminski
changed the title
JoeZiminski
changed the title
Member
|
@JoeZiminski is this PR still useful? It's nearly two years old! |
Member
Author
|
2 years 😱 sadly it is still on the TODO alongisde #208, but I'm hoping to include in next release 🙏 |
7 tasks
Member
Author
|
On further consideration, I don't think there is much point in these tests if we are already testing the ssh connection on linux using docker. Will close! |
Closed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This PR mixes up two new required types of testing
The image-route will be implemented in another PR. This PR will be stripped of image-related things and when we extend our internal testing can be used as a base. But it is unlikely this PR will be merged into main.