-
Notifications
You must be signed in to change notification settings - Fork 39
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
JSON array as parameter input #36
Comments
This issue is marked default for generating issues report. |
This issue is stale because it has been open for 7 days with no activity. |
@bishal-pdMSFT We are running into issues, when trying to do this. Due to the |
This issue is marked default for generating issues report. |
This issue is stale because it has been open for 7 days with no activity. |
Hi @marvinbuss, will it be possible to share the exact parameter input. Are you passing a json or a json string something like this |
Hi @t-dedah, I am passing a JSON string as input (e.g. |
This issue is marked default for generating issues report. |
@marvinbuss I am little confused. What secrets are you trying to pass as parameters? shouldn't we use azure login task for that? Also I consoled out Will it be possible to share repro steps for the issue? |
Please try this workflow, then you will see the issue:
|
This issue is marked default for generating issues report. |
This issue is marked default for generating issues report. |
@t-dedah No this is not the behavior of az cli. The issue is, that we are storing a JSON as secret (AZURE_CREDENTIALS). If you remove the reference to it, everything will run just fine. The issue is Azure/login. The Action needs to accept individual secrets rather than a json. This requires an improvment. |
This issue is marked default for generating issues report. |
This issue is stale because it has been open for 7 days with no activity. |
This issue is idle because it has been open for 14 days with no activity. |
Attempt to single quote database server name from example in Azure/arm-deploy#36. Confirmed that parameter spelling is correct. Confirmed that only valid characters for a SQL Server Server Name are lowercase letters, numbers, and dashes.
Hi @gummigroda Can you please try this?
|
This is the reason why it will not work. If you use the new Workload Identity Federation, this issue will most likely be solved, as you are not storing a JSON as secret then. |
This issue is idle because it has been open for 14 days with no activity. |
This issue is stale because it has been open for 7 days with no activity. |
I think you have to put the line between quotes, did you try understanding?
And better to use https://docs.microsoft.com/en-us/azure/active-directory/develop/workload-identity-federation |
When googling "azure/arm-deploy json array parameter" this issue is the first hit. After reading the comments above, I'm none the wiser. I've tried escaping quotes, encapsulating the whole thing with a single quote and without escapes, and... No go.
Result: Which leads me to believe I should not have escaped the double quotes. So... Change that so that Error: ERROR: Failed to parse string as JSON: Wait... What?!? gh variable list clearly disagrees. "maxSizeBytes" is definitively quoted. |
@vRune4 A shot in the dark, but here's what I would try: Setting the variable:
Using the variable:
|
Quoting the value is one of the things I tried. It expands into:
(the last parameter is a secret)
If the error message is to be believed, the double quote goes AWOL somewhere along the way. (I am reworking this to single parameters now) |
Hello.
Is it possible to use a JSON array as a parameter value in the parameters input?
For example
parameters: param1=
Thank you.
The text was updated successfully, but these errors were encountered: