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

Mega: End-to-end test the TinyPilot Debian package install #1691

Closed
5 tasks
mtlynch opened this issue Nov 27, 2023 · 1 comment
Closed
5 tasks

Mega: End-to-end test the TinyPilot Debian package install #1691

mtlynch opened this issue Nov 27, 2023 · 1 comment
Labels
enhancement New feature or request extra-large

Comments

@mtlynch
Copy link
Contributor

mtlynch commented Nov 27, 2023

One of the things we lost when we vanquished Ansible from our install process (#1596) was that Ansible tested the idempotence of a TinyPilot install.

Currently, changes to our installer are error-prone and time-consuming to test (see regression of #1615). It's worth investing in some e2e testing scripts to make sure the install works as we expect.

Notes

The rough idea I have in mind is:

  1. Create a Dockerfile that inherits from debian:bullseye and makes whatever changes to make our install work (e.g., we'll probably have to populate a /boot/config.txt file with dummy data)
  2. Build that Dockerfile in CI and run it on an ARM instance
  3. Run a script that runs through different install scenarios to verify that our install does what we expect

Things to test

  • Verify that changes to config files have been made (and verify that expected lines appear exactly once)
    • e.g. to /boot/config.txt and /etc/modules
  • Verify that tinypilot/ustreamer user accounts are created
  • Verify that TC358743-based installs work as expected
  • Verify that MS2109-based installs work as expected
  • Verify that install, uninstall, reinstall has the same effect as single install (prevent regression of Prevent tinypilot.postinst from appending to Janus config #1615)
@mtlynch mtlynch added enhancement New feature or request extra-large labels Nov 27, 2023
@mtlynch
Copy link
Contributor Author

mtlynch commented Nov 29, 2023

Forgot we already had this in #1659. Closing as dupe.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request extra-large
Projects
None yet
Development

No branches or pull requests

1 participant