Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Login to Linux VMs with AAD as alternative to SSH #137

Open
nimojg-microsoft opened this issue May 29, 2018 · 3 comments
Open

Login to Linux VMs with AAD as alternative to SSH #137

nimojg-microsoft opened this issue May 29, 2018 · 3 comments
Labels
enhancement New feature or request Priority 2 (maybe one day) Pull Requests welcome, but review will be deprioritized for other items. Best to discuss first.

Comments

@nimojg-microsoft
Copy link
Contributor

Looks like it'd require the install of some VM extensions = documented here: https://docs.microsoft.com/en-us/azure/virtual-machines/linux/login-using-aad

Note the feature is currently in preview and shouldn't be used with production VMs

@SorraTheOrc
Copy link
Contributor

Apart from debug purposes there is almost no reason to log into the cluster. Those who are debugging issues almost certainly are comfortable with Linux and thus SSH keys. Am I missing an important use case for this one?

@SorraTheOrc SorraTheOrc added Priority 2 (maybe one day) Pull Requests welcome, but review will be deprioritized for other items. Best to discuss first. enhancement New feature or request labels May 30, 2018
@hosungsmsft
Copy link

@rgardler -- No, I don't think so. I agree with you.

@hosungsmsft
Copy link

hosungsmsft commented May 30, 2018

Just wanted to point out that this AAD extension still requires an admin user either with an SSH key or a password (in which case an SSH key is preferred definitely. See https://docs.microsoft.com/en-us/azure/virtual-machines/linux/login-using-aad#create-a-linux-virtual-machine), so this AAD extension doesn't replace the SSH key requirement, but just adds capability to log in with an AD account, which may not be desirable for other reasons. The SSH key (or password) requirement for an admin account is an Azure CRP (Compute Resource Provider) API requirement that can't be replaced with this AAD extension, so that's why.

I'm personally inclined not to add this option to the templates. If users want/need, they should do this after the initial deployment.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request Priority 2 (maybe one day) Pull Requests welcome, but review will be deprioritized for other items. Best to discuss first.
Projects
None yet
Development

No branches or pull requests

3 participants