docs: document pi-manager-X sister repo with templates#44
Open
MateoLostanlen wants to merge 3 commits into
Open
docs: document pi-manager-X sister repo with templates#44MateoLostanlen wants to merge 3 commits into
MateoLostanlen wants to merge 3 commits into
Conversation
Scope the host_vars and group_vars ignore patterns so the template subdirs can be tracked, and add worked examples covering every file the private pi-manager-X sister repo is expected to provide. vars.vault.yml templates ship as plain YAML with CHANGE_ME placeholders so the expected keys are visible; the in-file header reminds to ansible-vault encrypt them before commit.
Add a 'sister repo' section showing the expected tree, point readers to the inventory/group_vars/host_vars templates, and explain the ansible-vault encrypt workflow before commit.
- Use wifi_connections in templates (the only shape consumed by the wifi role); drop the unused wifi_ssid/wifi_password pattern. - Drop dockerhub_username from the vault template (username is hardcoded to pyronear in roles/servers/tasks/main.yml). - Replace mediamtx_server_ip: 0.0.0.0 with a CHANGE_ME placeholder. - Readme: clarify that prepare only copies inventory/host_vars/ group_vars; .vault_passwrd and id_rsa are referenced in place via VAULT_PASSWORD_FILE/SSH_PRIVATE_KEY_FILE in .env.
fe51
requested changes
May 29, 2026
fe51
left a comment
fe51
left a comment
Member
There was a problem hiding this comment.
Hi Matéo, thanks for the PR, a good idea. We will need to keep it up to date now :)
Just a question before accepting your changes, see comment.
| S3_REGION: gra | ||
|
|
||
| # Public hostnames served by the platform-react server. | ||
| platform_react_url_backend: https://alertapi.pyronear.org |
Member
There was a problem hiding this comment.
are those url wanted to be exposed ? ( same comment elsewhere whenever they're present)
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The sister repo holding inventory and secrets is private, so this repo had no visible record of what it must contain. Adding worked examples and a Readme section so a new operator can stand up a
pi-manager-Xfrom scratch.host_vars/template/{engine,pi_zero,alert_server,annotation_server}/and fills the missinggroup_vars/template/{envdev,envprod,pi_zero,...}files.vars.vault.ymlfiles ship as plain YAML withCHANGE_MEplaceholders; an in-file header reminds toansible-vault encryptbefore commit.host_vars/group_varsignore patterns in.gitignoreso the newtemplate/subdirs are trackable (the previoushost_vars/rule excluded the directory entirely, making any!exception inert).make preparecopy paths, and noting that.vault_passwrd/id_rsaare referenced in place via.envrather than copied.