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

Location on server of webpage push #11

Closed
pbitzer opened this issue Sep 3, 2021 · 3 comments · Fixed by #18
Closed

Location on server of webpage push #11

pbitzer opened this issue Sep 3, 2021 · 3 comments · Fixed by #18
Assignees
Milestone

Comments

@pbitzer
Copy link
Contributor

pbitzer commented Sep 3, 2021

Right now, the location on the server of where the webpages are pushed is in src/website/mjolnir-website.lektorproject, and this is in version control. This value needs to be different for each sensor, so we need to find a way to have a way to specify the location differently for each sensor site.

@CAM-Gerlach CAM-Gerlach self-assigned this Sep 3, 2021
@CAM-Gerlach CAM-Gerlach added this to the v0.3.0 milestone Sep 3, 2021
@CAM-Gerlach
Copy link
Member

Yep, fully decoupling the config from the code is the main thing in work for v0,3.0. In particular, website.py and the lektorproject will be configurable outside of the code, falling back to the current defaults if anything isn't specified. It'll use Brokkr's existing config management system, so you can set most things at the system level and then any unit-specific overrides on the device, and I'd add a small feature that will autofill the unit name and number in standard placeholders in the lektorproject so it should be usable out of the box with no additional per-unit config. This will also be needed for the Sindri instance running on the VPS.

@CAM-Gerlach
Copy link
Member

Fixed in deployed v0.3.0a1 with the config being pulled out to mjolnir-hamma and automatically pushing to the correct subdir determined by the Brokkr unit number; PR incoming. In v0.3.0a2, Sindri won't need to run on the Pis at all, just once on the server.

@CAM-Gerlach
Copy link
Member

Resolved by #18

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants