search

neilpeterson / azure-blueprints-pipeline-tasks

MIT License
17 stars 16 forks source link

Include logic to make assignment file optional #44

Open neilpeterson opened 4 years ago

neilpeterson commented 4 years ago

Currently, an assignment JSON file is required. Looking at the docs for new-azblueprintassignment it appears an assignment file is not required. Update the assignment logic to reflect.

https://docs.microsoft.com/en-us/powershell/module/az.blueprint/new-azblueprintassignment?view=azps-2.8.0

stefanpetter commented 4 years ago

It seems like you added this already in commit #41

neilpeterson commented 4 years ago

Thanks for the note @stefanpetter.

What I am considering here is removing the need to have an assignment file. Looking that the new-azblueprintassignment command, I see these three options:

 [-SecureStringParameter <Hashtable>]
 [-ResourceGroupParameter <Hashtable>]
 [-Parameter <Hashtable>]

So instead of providing a json file, one could provide a hash table holding the assignment values (parameter values).

I'm not sure what the priority is on this. If you have an opinion, I would love to hear it.

Thanks again.