semaphoreui / semaphore

Modern UI and powerful API for Ansible, Terraform, OpenTofu, PowerShell and other DevOps tools.
https://semaphoreui.com
MIT License
10.65k stars 1.07k forks source link

fix: ensure environment variables are passed correctly in integrations #2352

Open ronzyfonzy opened 1 month ago

ronzyfonzy commented 1 month ago

This fix is connected to the discussion I opened up in General

fiftin commented 1 month ago

t.Environment.JSON isn't environment variables. This JSON used in --extra-vars parameter of ansible-playbook.

ronzyfonzy commented 1 month ago

Well then you must absolutely modify your docs where it states differently

https://github.com/semaphoreui/semaphore-docs/blob/fe6c8b493b5d7d47c9a0eceb8f738986372be6d1/user-guide/integrations.md?plain=1#L21