Skip to content
This repository has been archived by the owner on Nov 9, 2020. It is now read-only.

Enables ssh via Go for Windows OpenSSH Server #1640

Closed
wants to merge 1 commit into from

Conversation

venilnoronha
Copy link
Contributor

This PR enables ssh via Go for the Windows OpenSSH server. This will aid the development of e2e tests for the Windows plugin.

This commit enables ssh via Go for the Windows OpenSSH Server.
Copy link
Contributor

@shaominchen shaominchen left a comment

Choose a reason for hiding this comment

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

LGTM.


// +build !runoncewin

package ssh
Copy link
Contributor

Choose a reason for hiding this comment

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

let's not introduce "_" with this PR ... tests/* we have tried to avoid introducing that unless it is necessary.


// Exposes util to invoke remote commands on windows hosts via ssh.

// +build runoncewin
Copy link
Contributor

Choose a reason for hiding this comment

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

If I understand correctly, you are controlling to pick up during windows run. I would suggest to use different name avoid using "test" related tags here.

@venilnoronha
Copy link
Contributor Author

@shuklanirdesh82 I've addressed your comments in fbe4a2c under #1643. I've re-created this PR from my private repo, so closing this now.

@shuklanirdesh82 shuklanirdesh82 deleted the ssh-win.venilnoronha branch September 1, 2017 21:59
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants