Jason Wallace
Jason Wallace
Part of https://github.com/tiny-pilot/ansible-role-tinypilot-pro/issues/119 Dependent on https://github.com/tiny-pilot/ansible-role-tinypilot/pull/213 This PR upgrades Ansible from `2.9.13` to `2.10.7`. For context, please review https://github.com/tiny-pilot/ansible-role-tinypilot/pull/213 before reviewing this PR. This PR has been [successfully built here](https://app.circleci.com/pipelines/github/tiny-pilot/tinypilot/2433/workflows/63eea8e9-810a-4e65-a67d-70f87a5f2c6c)....
Resolves https://github.com/tiny-pilot/tinypilot/issues/1027 # Testing procedure ## Test update path from an existing TinyPilot installation Follow these [install instructions](https://gist.github.com/jdeanwallace/9f293965f404a995c507e52afda1d39e). ### Bugs 1. The `/opt/tinypilot-updater` directory is orphaned after an update. https://github.com/tiny-pilot/tinypilot/pull/1060...
Part of https://github.com/tiny-pilot/tinypilot/issues/1027 This PR can be merge anytime after merging https://github.com/tiny-pilot/tinypilot/pull/1046 because we already reference this branch directly when [building the bundle file](https://github.com/tiny-pilot/tinypilot/blob/d30f55dbd572dceee7c44570148e6dd9e968094e/bundler/create-bundle#L18).
Part of https://github.com/tiny-pilot/tinypilot-pro/issues/596
Update Overhaul: Remove the use of the TinyPilot git repo in favor of the TinyPilot Debian package.
After having merged both https://github.com/tiny-pilot/tinypilot/pull/1046 and https://github.com/tiny-pilot/ansible-role-tinypilot/pull/214, we can remove the use of git for TinyPilot code. For example: - [ ] https://github.com/tiny-pilot/ansible-role-tinypilot/blob/19100f2991a9afef8d9a42bdef6a4298545f3c6b/tasks/main.yml#L124-L135 - [x] https://github.com/tiny-pilot/ansible-role-tinypilot/blob/19100f2991a9afef8d9a42bdef6a4298545f3c6b/files/collect-debug-logs#L68-L70
While reviewing https://github.com/tiny-pilot/tinypilot/pull/1666, @jotaen4tinypilot [suggested](https://codeapprove.com/pr/tiny-pilot/tinypilot/1666#thread-a5a85265-2127-401c-a118-e39d5ad83625) we suppress some of the Flask logging when running Flask in CLI commands. Example of logs to suppress: ```bash $ FLASK_APP=app/main.py flask cli streaming-mode 2023-10-30...
Related https://github.com/tiny-pilot/style-guides/issues/5 Fix any shell variables that don't comply with our [casing style for constant and non-constant variables](https://github.com/tiny-pilot/style-guides/blob/7112fc6291104bc19ee81619f4c672682e7f08e2/README.md#variable-names).
This issue has been recreated from https://github.com/tiny-pilot/ansible-role-tinypilot/issues/229 Sometimes, customers file support requests saying that TinyPilot doesn't work on their device, and it takes a few back-and-forths to discover that they're...
Related https://github.com/tiny-pilot/tinypilot/issues/1711 This is a follow-up PR to a [review comment](https://codeapprove.com/pr/tiny-pilot/tinypilot/1778#thread-4d8c230a-afdd-4361-8e21-0a7be15efb06) asking for more code comments in our `enable-wifi-ap` and `disable-wifi-ap` scripts.
As discovered [here](https://github.com/tiny-pilot/tinypilot/pull/1898#issuecomment-3096484819), it seems like Python package's can have different structures depending on how they were packaged/built. For example, on my macOS development machine, installing `MarkupSafe==3.0.2`, results in the...