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

Refactor repo to basic bash setup approach #25

Merged
merged 27 commits into from
Jun 13, 2024
Merged

Refactor repo to basic bash setup approach #25

merged 27 commits into from
Jun 13, 2024

Conversation

fe51
Copy link
Member

@fe51 fe51 commented May 9, 2024

Hello there,

This repo has been out of use for a very long time. However, we need to make our raspberrypi setup efficient. The Ansible approach is certainly relevant, but to get off to a solid start, we're going to restart here with a bash script setup.
(The ansible appproach is kept in the ansible-archive branch, as we'll probably come back to Ansible later for its flexibility, robustness and better debugging quality.

This PR introduces the following modifications:

  • deletes all Ansible related files
  • Add bash scripts setup approach
  • Documents the process in the readme
  • Updates contributing

It has been tested to setup a Pyronear station.

However :

  • Maybe "sudo apt update" and "sudo apt install docker-compose" are enough to install docker and makes curl -sSL https://get.docker.com | sh && sudo usermod -aG docker pi useless. It has to be tested soon.
  • About cron tasks, the update_script has been simplified, the suggested command in Notion was not working.

Happy to discuss it ! Feedback is welcome !

Copy link
Member

@MateoLostanlen MateoLostanlen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @fe51 !

Thanks for this PR! It's globally ok for me. A few changes though:

  • remove the ability to change PI_HOST
  • No need to redefine the wifi or add a common wifi for debug in connection sharing
  • no need to configure the pi on the command line, which will confuse people
  • add pi imager screen

RPI_INSTALLATION.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
setup_rpi.sh Outdated Show resolved Hide resolved
setup_rpi.sh Outdated Show resolved Hide resolved
setup_rpi.sh Show resolved Hide resolved
setup_rpi.sh Show resolved Hide resolved
setup_rpi.sh Show resolved Hide resolved
setup_rpi.sh Show resolved Hide resolved
setup_rpi.sh Outdated Show resolved Hide resolved
@fe51 fe51 requested a review from MateoLostanlen June 11, 2024 10:49
setup_rpi.sh Outdated Show resolved Hide resolved
@fe51 fe51 requested a review from MateoLostanlen June 11, 2024 13:52
.gitignore Outdated Show resolved Hide resolved
@MateoLostanlen MateoLostanlen self-requested a review June 13, 2024 08:58
Copy link
Member

@MateoLostanlen MateoLostanlen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I've just re-tested it and it works without a hitch, thanks for the PR! very happy to have it 😄

@fe51 fe51 merged commit 46214bb into master Jun 13, 2024
1 check passed
@fe51 fe51 deleted the setup_bash branch June 13, 2024 09:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants