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

Rename bitwarden rs to vaultwarden #297

Merged
merged 4 commits into from Dec 27, 2021
Merged

Rename bitwarden rs to vaultwarden #297

merged 4 commits into from Dec 27, 2021

Conversation

RlndVt
Copy link
Contributor

@RlndVt RlndVt commented Nov 18, 2021

Fixes #296.

E:
Expanded to also fix #298 .

@RlndVt RlndVt marked this pull request as ready for review November 18, 2021 15:03
@FroggyFlox FroggyFlox added the needs review Test install, function, on / off behaviour, all links / info. label Nov 18, 2021
@phillxnet
Copy link
Member

@RlndVt & @Hooverdan96 Well done on resolving the related arm issue re #298.

Are we now just awaiting @Hooverdan96 's suggestions in that issue re:

#298 (comment)

So, I think, once the description in #297 is updated to include the available architecture language, then this could likely be closed.

The only architectures we are interested in are x86_64 & ARM64 really. The v7 arms are way under-powered for Rockstor and btrfs.

@RlndVt do you mind updating this pr's original text to list it's (welcome in this case) scope creep? I.e. it looks like it also fixes the arm compatibility/description (with edits) issue so another "Fixes" line could be added for example. All helps with assessing the state as this one is getting a little confusing with it's 3 associated issues :). All good.

Cheers. Almost there on this one by the looks of it. And thanks again for all you efforts. It will be good to get this one done and dusted.

Update references to use Bitwarden-rs has been renamed to Vaultwarden

bitwarden-rs->vaultwarden name change in root.json
Update and rename vaultwarden-alpine.json to vaultwarden.json .

Fixes #298

Update root.json to match vaultwarden tag change
@RlndVt
Copy link
Contributor Author

RlndVt commented Nov 30, 2021

Squashed the root.json commits as they shouldn´t be considered separate from the rest.

vaultwarden.json Outdated Show resolved Hide resolved
@FroggyFlox
Copy link
Member

Hi @RlndVt ,

First, please accept my apologies for taking so long to have a proper look at this. I've tested it on amd64 and it works as intended. I'll try to test it on an arm64 machine later today to make sure that works as well but in the meantime I thought I would make a suggestion to add this multi-architecture compatibility. Feel free to commit that suggestion when you get a chance.

Thanks again for submitting this PR and keeping this Rock-on up-to-date!

@FroggyFlox
Copy link
Member

@RlndVt ,

I just tested on both amd64 and arm64 architectures and it works like a charm. I created a self-signed certificate in the share to be used as "vaultwarden certificates":

openssl req -newkey rsa:2048 -nodes -keyout key.pem -x509 -days 365 -out certs.pem

... then installed the Rock-on and it all works as intended:

  • installs without error
  • webUI is accessible
  • restarts normally
  • restarts automatically and without error after a reboot
  • can be uninstalled and re-installed without a problem

I think we are ready to merge this in as soon as we have the little edits related to the explicit mention of the multi-arch compatibility in the Rock-on "description" field.

Thanks a lot!

RlndVt and others added 2 commits December 26, 2021 07:44
Co-authored-by: FroggyFlox <30297881+FroggyFlox@users.noreply.github.com>
Vaultwarden docker images tagged with 'latest' are built on Debian not Alpine.
@FroggyFlox
Copy link
Member

Thanks for the additional edits, @RlndVt !

I'll merge this now.

@FroggyFlox FroggyFlox merged commit 0c674ae into rockstor:master Dec 27, 2021
@RlndVt RlndVt deleted the Rename_bitwarden-rs_to_vaultwarden branch December 27, 2021 13:49
@FroggyFlox
Copy link
Member

This has now been published. Thanks again @RlndVt !

@FroggyFlox FroggyFlox removed the needs review Test install, function, on / off behaviour, all links / info. label Dec 27, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Vaultwarden alpine is not available for arm64 Update Bitwarden_rs refernces to Vaultwarden
3 participants