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

[AVM Module Issue]: desktop-virtualization/host-pool should expose "managementType" parameter #4485

Open
1 task done
hooverken opened this issue Feb 20, 2025 · 2 comments
Assignees
Labels
Class: Resource Module 📦 This is a resource module Needs: Triage 🔍 Maintainers need to triage still Type: AVM 🅰️ ✌️ Ⓜ️ This is an AVM related issue Type: Feature Request ➕ New feature or request

Comments

@hooverken
Copy link

Check for previous/existing GitHub issues

  • I have checked for previous/existing GitHub issues

Issue Type?

Feature Request

Module Name

avm/res/desktop-virtualization/host-pool

(Optional) Module Version

No response

Description

The current ARM definition for desktopVirtualization/hostpools includes a "managementType" property which can be set to "Automated" or "Standard". I am creating a Bicep template to deploy a host pool with a session host configuration and the deployment is throwing this error in the host-pool module but the parameter it wants isn't exposed. The linked doc says that the default is "automated" but apparently the module is setting it to "Standard" ...?

Requesting that this property be exposed in the module and also set to "Automated" instead of "Standard" by default.

Example:

11:45:34 AM - The deployment '85b0a762-03de-4727-9a37-b9da6cf70ada' failed with error(s). Showing 1 out of 1
| error(s). Status Message: ActivityId: 634b284a-152b-4711-b02c-fbdf5a23c0c9 Error: This operation cannot be
| performed on a hostpool with ManagementType Standard
. (Code:400) CorrelationId:
| 67aa71db-88a1-427d-87d3-364a6d9b79c7

(Optional) Correlation Id

67aa71db-88a1-427d-87d3-364a6d9b79c7

@hooverken hooverken added Needs: Triage 🔍 Maintainers need to triage still Type: AVM 🅰️ ✌️ Ⓜ️ This is an AVM related issue labels Feb 20, 2025
@github-project-automation github-project-automation bot moved this to Needs: Triage in AVM - Module Issues Feb 20, 2025

Important

The "Needs: Triage 🔍" label must be removed once the triage process is complete!

Tip

For additional guidance on how to triage this issue/PR, see the BRM Issue Triage documentation.

@avm-team-linter avm-team-linter bot added the Class: Resource Module 📦 This is a resource module label Feb 20, 2025
Copy link

@hooverken, thanks for submitting this issue for the avm/res/desktop-virtualization/host-pool module!

Important

A member of the @Azure/avm-res-desktopvirtualization-hostpool-module-owners-bicep or @Azure/avm-res-desktopvirtualization-hostpool-module-contributors-bicep team will review it soon!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Class: Resource Module 📦 This is a resource module Needs: Triage 🔍 Maintainers need to triage still Type: AVM 🅰️ ✌️ Ⓜ️ This is an AVM related issue Type: Feature Request ➕ New feature or request
Projects
Status: Needs: Triage
Development

No branches or pull requests

2 participants