Skip to content
This repository has been archived by the owner on Apr 5, 2023. It is now read-only.

Document separation of concerns between ansible and app repos #189

Open
jotaen4tinypilot opened this issue Mar 17, 2022 · 2 comments
Open
Labels
documentation Improvements or additions to documentation medium

Comments

@jotaen4tinypilot
Copy link
Contributor

It’s not always 100% clear where functionality should go, either to the ansible repo or the app one. So for example:

  • When do we create privileged scripts and implement the logic around it in a shell script, vs. when do we just add a sudoer’s entry for a specific command and implement the other logic in Python?
  • How do we go about repetition between the two repos, e.g. when certain parameters are needed in both repositories? Example: configfs paths for mass storage, that we have both in the app and in ansible.
  • Does the fact that we have mock scripts for local development justify single-line privileged scripts, only for the sake of mocking them out? Or is it okay to just use debug toggles for that? What is our general approach towards dev-prod-parity?

It’s probably hard to come up with a definite answer, but at least we could try writing up some guidelines that help us make consistent decisions.

PRs/Issues during which that discussion came up already:

@mtlynch mtlynch added documentation Improvements or additions to documentation medium labels Mar 17, 2022
@jotaen4tinypilot
Copy link
Contributor Author

We could consider putting this one on hold for now, since we might change or reconsider things when working on the upcoming update mechanism. At least we should keep the possibility of changes in mind, before putting effort into documenting the current design.

@mtlynch
Copy link
Contributor

mtlynch commented May 24, 2022

That's true. I'll push this until next release.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
documentation Improvements or additions to documentation medium
Projects
None yet
Development

No branches or pull requests

2 participants