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

[Question/Feedback]: Unable to deploy via pipeline using ubuntu-latest #64

Closed
1 task done
ac-matthews opened this issue Nov 2, 2023 · 4 comments · Fixed by #79
Closed
1 task done

[Question/Feedback]: Unable to deploy via pipeline using ubuntu-latest #64

ac-matthews opened this issue Nov 2, 2023 · 4 comments · Fixed by #79
Assignees
Labels
Pattern: ALZ 🚁 Issues / PR's related to the ALZ Pattern question Further information is requested

Comments

@ac-matthews
Copy link

Check for previous/existing GitHub issues

  • I have checked for previous/existing GitHub issues

Description

Hi,

I'm having an issue deploying AMBA as part of our ADO pipeline. When using image version 'Ubuntu-Latest' , I get a script failed message saying the parameter file was unable to be parsed. However when using 'Windows-Latest' the deployment works just fine. There are no errors with the param file and the pipeline stage is basically identical to the example provided in the repo. Any advice on this would be greatly appreciated, cheers!

Screenshot 2023-11-02 142413

@ac-matthews ac-matthews added the question Further information is requested label Nov 2, 2023
@philrice
Copy link

@ac-matthews - I had this issue and for me I could resolve it by changing the the az deployment line in the pipeline yaml , so that the --parameters was using the full URI to the params file in github (in the same way the template URI does).
The default pipeline yaml had a relative path for the parameters file in windows format :

inlineScript: | az deployment mg create --template-uri https://raw.githubusercontent.com/Azure/azure-monitor-baseline-alerts/main/patterns/alz/alzArm.json --location $(location) --management-group-id $(ManagementGroupPrefix) --parameters .\patterns\alz\alzArm.param.json

You could also try switch the backslashes for forward slashes, as that will likely fix it for Ubuntu (but break for Windows)

@paulgrimley
Copy link
Contributor

paulgrimley commented Nov 10, 2023

Apologies for the delay in response , we had discussed this internally earlier in the week and given you had a workaround we reduced the priority as we are driving a few enhancements to AMBA needing focus, adding @arjenhuitema to comment but also thanks @philrice for your input also!

@ac-matthews
Copy link
Author

Many thanks @philrice the forward slashes worked :) will be sticking to ubuntu for our pipelines so that will sort it for now!

Also no problem @paulgrimley, will keep and eye out for any other fixes relating to this issue

@arjenhuitema arjenhuitema linked a pull request Nov 27, 2023 that will close this issue
6 tasks
@arjenhuitema
Copy link
Contributor

As @philrice said, forward slashes or URI of the parameter files are the options the solve this. thanks Phil! A PR to change to forward slashes is created waiting to be merged.

@Brunoga-MS Brunoga-MS added the Pattern: ALZ 🚁 Issues / PR's related to the ALZ Pattern label Dec 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Pattern: ALZ 🚁 Issues / PR's related to the ALZ Pattern question Further information is requested
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants