Skip to content

Conversation

@jkroepke
Copy link

@jkroepke jkroepke commented Jul 14, 2021

Our company does not have the windows 10 ssh client installed. As user we not allowed to install such windows features. But we are able to provide ssh-keygen.exe on different ways. The ssh-keygen.exe command is inside our path, but the az cli forcefully look into C:\SysNative\OpenSSH.

This PR add a fallback on Windows systems, in case C:\SysNative\OpenSSH is not present, the ssh command from the PATH should be used.


This checklist is used to make sure that common guidelines for a pull request are followed.

General Guidelines

  • Have you run azdev style <YOUR_EXT> locally? (pip install azdev required)
  • Have you run python scripts/ci/test_index.py -q locally?

About Extension Publish

There is a pipeline to automatically build, upload and publish extension wheels.
Once your PR is merged into master branch, a new PR will be created to update src/index.json automatically.
The precondition is to put your code inside this repo and upgrade the version in the PR but do not modify src/index.json.

@jkroepke jkroepke requested a review from fengzhou-msft as a code owner July 14, 2021 14:33
@ghost ghost added the customer-reported Issues that are reported by GitHub users external to the Azure organization. label Jul 14, 2021
@ghost
Copy link

ghost commented Jul 14, 2021

Thank you for your contribution jkroepke! We will review the pull request and get back to you soon.

@jkroepke
Copy link
Author

Please let me know, if the code solution is fine. If yes, I take a look at the ITs

@yonzhan yonzhan requested review from Juliehzl, jiasli and zhoxing-ms and removed request for fengzhou-msft July 14, 2021 23:22
@yonzhan yonzhan added this to the Jul 2021 (2021-08-03) milestone Jul 14, 2021
@yonzhan
Copy link
Collaborator

yonzhan commented Jul 14, 2021

ssh

@jkroepke
Copy link
Author

Any update to this?

@jiasli jiasli requested review from N6UDP and removed request for Juliehzl, jiasli and zhoxing-ms September 13, 2021 01:51
@jkroepke
Copy link
Author

@jiasli @N6UDP any Feedback for me?

@jkroepke
Copy link
Author

jkroepke commented Jan 10, 2022

Hi, did we miss the milestone again?

@jkroepke
Copy link
Author

Superseeded by #4570

@jkroepke jkroepke closed this Apr 18, 2022
@jkroepke jkroepke deleted the patch-1 branch April 18, 2022 15:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

customer-reported Issues that are reported by GitHub users external to the Azure organization.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants