Skip to content

Test for the existence of the private key file.#2779

Merged
derekbekoe merged 1 commit intoAzure:masterfrom
brendandburns:ssh1
Apr 10, 2017
Merged

Test for the existence of the private key file.#2779
derekbekoe merged 1 commit intoAzure:masterfrom
brendandburns:ssh1

Conversation

@brendandburns
Copy link
Copy Markdown
Member

Improve error handling to hopefully help resolve:

#2403

See meta-parimiko discussion here:

paramiko/paramiko#521
paramiko/paramiko#387

@codecov-io
Copy link
Copy Markdown

Codecov Report

Merging #2779 into master will decrease coverage by <.01%.
The diff coverage is 0%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #2779      +/-   ##
==========================================
- Coverage   62.87%   62.86%   -0.01%     
==========================================
  Files         480      480              
  Lines       25807    25811       +4     
  Branches     3905     3907       +2     
==========================================
  Hits        16226    16226              
  Misses       8572     8572              
- Partials     1009     1013       +4
Impacted Files Coverage Δ
...re-cli-acs/azure/cli/command_modules/acs/custom.py 35.02% <0%> (-0.29%) ⬇️
...dback/azure/cli/command_modules/feedback/custom.py 34.69% <0%> (ø) ⬆️
src/azure-cli-core/azure/cli/core/util.py 68.99% <0%> (ø) ⬆️
...-cli-role/azure/cli/command_modules/role/custom.py 19.28% <0%> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update fa855de...27246c0. Read the comment docs.

@derekbekoe derekbekoe merged commit 80a7b9a into Azure:master Apr 10, 2017
00Kai0 pushed a commit to 00Kai0/azure-cli that referenced this pull request Apr 7, 2021
Triggered by Azure CLI Extensions Release Pipeline - ADO_BUILD_ID=651004

Last commit: Azure/azure-cli-extensions@f6ad444
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants