Also allow --hostname in az ssh config

[jacquerie-meritis]

Similarly to az ssh vm, az ssh config should also accept using the --hostname form of the argument as well as --ip. See:

azure-cli-extensions/src/ssh/azext_ssh/_params.py

Lines 11 to 12 in 5f48738

	c.argument('ssh_ip', options_list=['--ip', '--hostname'],
	help='The public (or reachable private) IP address (or hostname) of the VM')

And:

azure-cli-extensions/src/ssh/azext_ssh/_params.py

Line 27 in 5f48738

c.argument('ssh_ip', options_list=['--ip'], help='The public IP address (or hostname) of the VM')

---

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?

For new extensions:

• My extension description/summary conforms to the Extension Summary Guidelines.

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.

[ghost]

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

[jacquerie-meritis]

For completeness, I can also report that this crashes:

$ azdev style ssh
Traceback (most recent call last):
  File "/home/jacquerie/.local/bin/azdev", line 8, in <module>
    sys.exit(main())
  File "/home/jacquerie/.local/lib/python3.8/site-packages/azdev/__main__.py", line 39, in main
    config_dir=get_azdev_config_dir())
  File "/home/jacquerie/.local/lib/python3.8/site-packages/azdev/utilities/config.py", line 24, in get_azdev_config_dir
    _, env_name = os.path.splitdrive(get_env_path())
  File "/usr/lib/python3.8/posixpath.py", line 134, in splitdrive
    p = os.fspath(p)
TypeError: expected str, bytes or os.PathLike object, not NoneType

I didn't investigate further because I doubt such a small change can really be violating the style guide anyway...

[yonzhan]

ssh