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

Fix Janus installation during molecule execution (test) #236

Closed
jotaen4tinypilot opened this issue Oct 31, 2022 · 0 comments · Fixed by #240
Closed

Fix Janus installation during molecule execution (test) #236

jotaen4tinypilot opened this issue Oct 31, 2022 · 0 comments · Fixed by #240

Comments

@jotaen4tinypilot
Copy link
Contributor

Currently, the Janus debian package can only be installed on armhf systems. Our test environment on CircleCI has a different architecture, which prevents us from installing Janus during the molecule execution (i.e. Ansible’s “unit tests”).

See #234 for a discussion of the problem and potential approaches to solve this. For now, we decided to move forward with a pretty ugly hack, but we should probably come back and solve this properly at some point.

Depending on the eventual fix, this ticket should/could be moved to the uStreamer Ansible repo.

jdeanwallace added a commit that referenced this issue Dec 5, 2022
Part of tiny-pilot/ansible-role-ustreamer#77
Dependent on
tiny-pilot/ansible-role-ustreamer#78
Resolves #236

### Notes:
1. I had to make the majority of changes in
tiny-pilot/ansible-role-ustreamer#78 to get this
branch to build. I still need to [remove this ustreamer version
pin](https://github.com/tiny-pilot/ansible-role-tinypilot/blob/a6889cbbb4aa3b9bcee1cce8cccc1d7f78a36c63/meta/requirements.yml#L3).
1. It turns out that ustreamer fails to compile `WITH_JANUS=1` even on
Debian Buster (not just Raspberry Pi OS Buster). So I [forced a legacy
uStreamer version on Debian
Buster](https://github.com/tiny-pilot/ansible-role-tinypilot/blob/a6889cbbb4aa3b9bcee1cce8cccc1d7f78a36c63/tasks/ustreamer.yml#L14-L15)
too.

<a data-ca-tag
href="https://codeapprove.com/pr/tiny-pilot/ansible-role-tinypilot/240"><img
src="https://codeapprove.com/external/github-tag-allbg.png" alt="Review
on CodeApprove" /></a>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant