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

DietPi-Software | vaultwarden: Switch to self-hosted pre-compiled binaries #5638

Merged
merged 12 commits into from Jul 23, 2022

Conversation

MichaIng
Copy link
Owner

No description provided.

@MichaIng MichaIng added this to the v8.7 milestone Jul 21, 2022
@MichaIng MichaIng force-pushed the vaultwarden branch 8 times, most recently from 4901a12 to b807d6c Compare July 21, 2022 20:48
@MichaIng MichaIng changed the title DietPi-Software | vaultwarden: Add GitHub workflow and build scripts DietPi-Software | vaultwarden: Switch to self-hosted pre-compiled binaries Jul 23, 2022
@MichaIng MichaIng linked an issue Jul 23, 2022 that may be closed by this pull request
- DietPi-Software | vaultwarden: Add GitHub workflow and build scripts
- DietPi-Software | vaultwarden: Try to fix  failing build in emulated container by installing rustup into tmpfs
- DietPi-Software | vaultwarden: Increase size of build tmpfs, add debug code to check for success and actually unsed memory, and fail alreaedy in container step if the package was not generated
- DietPi-Software | vaultwarden: Within the DietPi container, /tmp is a sufficient large tmpfs, also fix check for built package outside of container
- DietPi-Software | vaultwarden: Fix libSSL dependency on Bullseye
- DietPi-Software | vaultwarden: Use $G_HW_ARCH_NAME as package architecture suffix
- DietPi-Software | vaultwarden: Syntax
- DietPi-Software_builds | Assure that builds start after DietPi-PostBoot finished, to reduce interferance and assure that .hw_model (DietPi-PreBoot) was crearted already so that G_HW_ARCH etc exist
- DietPi-Software | vaultwarden: Some build fixes
- DietPi-Software | Amiberry: Remove the $arch variable from build script, simply use $(dpkg --print-architecture) instead. Also bump version string suffix since LibSDL2_image/ttf versions have been bumped
- Fix drop-in service config
@Joulinar
Copy link
Collaborator

Joulinar commented Jul 23, 2022

something doesnt seems to fit on ARMv6 RPi1

Set vaultwarden userdata owner ...
Configuring vaultwarden systemd service ...
Created symlink /etc/systemd/system/multi-user.target.wants/vaultwarden.service → /lib/systemd/system/vaultwarden.service.
Processing triggers for libc-bin (2.31-13+rpt2+rpi1+deb11u2) ...
ldconfig: /lib/arm-linux-gnueabihf/libyaml-0.so.2 is not an ELF file - it has the wrong magic bytes at the start.

ldconfig: /lib/arm-linux-gnueabihf/libyaml-0.so.2.0.6 is not an ELF file - it has the wrong magic bytes at the start.

of course service did not start

Jul 23 23:07:58 DietPi1 systemd[1]: Started vaultwarden (DietPi).
Jul 23 23:07:59 DietPi1 systemd[1]: vaultwarden.service: Main process exited, code=killed, status=4/ILL
Jul 23 23:07:59 DietPi1 systemd[1]: vaultwarden.service: Failed with result 'signal'.
Jul 23 23:08:04 DietPi1 systemd[1]: vaultwarden.service: Scheduled restart job, restart counter is at 1.
Jul 23 23:08:04 DietPi1 systemd[1]: Stopped vaultwarden (DietPi).

@MichaIng
Copy link
Owner Author

MichaIng commented Jul 23, 2022

Hmm, that file is not part of the package, does it exist and look otherwise fine?

ls -l /usr/lib/arm-linux-gnueabihf/libyaml-0.so.2.0.6
apt install --reinstall libyaml-0-2

Strange that it is required, it wasn't installed on the VMs I tested this on already. Probably it isn't vaultwarden itself, but ldconfig which failed, so that available libraries weren't fully linked or so?

@Joulinar
Copy link
Collaborator

strange thing was, that vaultwarden triggert installation of bash and libc6

Note, selecting 'vaultwarden' instead of './vaultwarden_armv6l.deb'
The following additional packages will be installed:
  bash libc6

@MichaIng
Copy link
Owner Author

Yes, intentionally those are dependencies, but how is it possible that libc6 is missing?

@Joulinar
Copy link
Collaborator

the executable seems to be not fitting

root@DietPi1:/tmp# /opt/vaultwarden/vaultwarden
Illegal instruction
root@DietPi1:/tmp#

@MichaIng
Copy link
Owner Author

MichaIng commented Jul 23, 2022

It installs fine on my RPi Zero W, but the binary fails with Illegal instruction. Looks like despite the workaround for ARMv6/7 builds, the build in emulated environment has still issues 😞. Testing on RPi 2 now.

But I'll merge it now regardless. Want to push the beta.

... okay at least on RPi 2 it works, so only ARMv6 left as an issue. Actually I wonder whether this ever worked, if I ever tested compiling it on RPi Zero W? Will try tomorrow... although we already know the issue with the swap file. Probably we simply need to disable it for ARMv6.

@Joulinar
Copy link
Collaborator

again a software title not working on armv6 😢
But yeah it should not have been working before as well.

- DietPi-Software | vaultwarden: Fix uninstall
@MichaIng MichaIng merged commit db62afe into dev Jul 23, 2022
@MichaIng MichaIng deleted the vaultwarden branch July 23, 2022 21:53
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.

error compiling vaultwarden
2 participants