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 Qubes staging standalone VM due to conflict with workstation VM #5097

Closed
emkll opened this issue Jan 20, 2020 · 2 comments · Fixed by #5099
Closed

Rename Qubes staging standalone VM due to conflict with workstation VM #5097

emkll opened this issue Jan 20, 2020 · 2 comments · Fixed by #5099

Comments

@emkll
Copy link
Contributor

emkll commented Jan 20, 2020

Description

Initially reported by @zenmonkeykstop

We have introduced changes in freedomofpress/securedrop-workstation#407 that rename sd-svs to sd-app. As a result, the Qubes staging environment instructions will no longer work if the SecureDrop workstation is provisioned on the host.

Expected Behavior

Qubes VMs should have a unique name

Actual Behavior

sd-app refers to both the AppVM that is running the SecureDrop Client, as well as the SecureDrop Application server for the qubes-staging scenario.

Comments

We should update the qubes-staging docs to rename the sd-app to another name (e.g. sd-app-staging)

@eloquence
Copy link
Member

eloquence commented Jan 20, 2020

I'd recommend:

  • sd-staging-app and sd-staging-mon for the staging VMs
  • sd-staging-app-base and sd-staging-mon-base for their templates
  • sd-staging-base for the base template.

Not normally a fan of multiple hyphens but for development I think it's OK to break that convention. The reason to avoid starting with sd-app is to reduce developer confusion/frustration when navigating menus etc. The sd-staging- prefix sets everything related to the staging env apart from the workstation VMs.

@zenmonkeykstop zenmonkeykstop self-assigned this Jan 21, 2020
@zenmonkeykstop
Copy link
Contributor

zenmonkeykstop commented Jan 21, 2020

Yikes, I'd already redone this as sd-appserver, sd-monserver, but tbh the sd-staging prefix makes more sense. Will fix it up and submit a PR

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

Successfully merging a pull request may close this issue.

3 participants