Feedback on portal UI deployment

[danycontre]

Creating this issue to consolidate feedback on the custom portal UI:

https://portal.azure.com/#blade/Microsoft_Azure_CreateUIDef/CustomDeploymentBlade/uri/https%3A%2F%2Fraw.githubusercontent.com%2FAzure%2Favdaccelerator%2Fem-customui%2F01-avd-framework.json/uiFormDefinitionUri/https%3A%2F%2Fraw.githubusercontent.com%2FAzure%2Favdaccelerator%2Fem-customui%2FPortalDefinitions%2F01-landingzone.json

[danycontre]

Some initial comments:

• New scaling tab: There should be an scaling tab to let you you choose if using start vm on connect or scaling plan (once it is GA).
• Components tab: depending on the automation option selected the custom role should be created if it doesn't exist and the RBAC permissions assignment created, maybe we could show a message saying this will be done, but don't see a reason for the option since deploying the role is mandatory for the feature to work. (this could be moved to the scaling tap if we create it).
• Components tab: we should have an option to deploy an empty application group for remote apps.
• Host pool settings tab, we could add and advanced option with a text box field to enter specific RDP properties
• Idenity: don't see an option for authentication (ADDS or AADDS), for the MVP we could just show a message about it on the network section. on the network section stating that the subnet hosting the session hosts needs to have line of sight to the identity service (ADDS or AADDS).
• Network: when selecting existing vNet, we should show a message stating that the subnet hosting the session hosts needs to have line of sight to the identity service (ADDS or AADDS).
• Network: when selecting to create a new vNet we should ask for vNet where ADDS or AADDS exist to create peering (we could remove the new vNet option on the MVP).

[danycontre]

Thanks Erick, agree with your comments on getting some things in the backlog and maybe deliver them in the future as add-on modules.

• For the host pool RDP settings wasn't able to find an option in the portal UI to enter custom properties in a text box (i could be looking in the wrong place).
• Agree network peering will require RBAC setup that will be tricky to manage, so for the MVP we could just tell the user that it will be required prior deploying session hosts.

[edm-ms]

Third tab:

The 3 radio buttons at the top.

[danycontre]

Hey Erick, when i said custom RDP properties text box i was referring to something similar to what we can see in the host pool properties:

Not something we really need to have specially for the MVP, but know of multiple customers that already have the string for things they want to set as RDP properties.

[edm-ms]

Those are the same properties. What you choose in those boxes will create what you see here. If we need to add another box to copy/paste properties we could expose it though.

[danycontre]

It would be nice to have the option, though not really a critical thing, better to work on the top ones and get this one on the backlog for future updates