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

Update systemd configs to start at boot time #343

Merged
merged 1 commit into from Dec 18, 2020
Merged

Update systemd configs to start at boot time #343

merged 1 commit into from Dec 18, 2020

Conversation

woneill
Copy link
Contributor

@woneill woneill commented Dec 15, 2020

The current configurations won't start vouch-proxy after a system reboot

@bnfinet
Copy link
Member

bnfinet commented Dec 15, 2020

@woneill thanks for the contribution

Is there a reason not to use the default runlevel?

@woneill
Copy link
Contributor Author

woneill commented Dec 16, 2020

@bnfinet To be honest, I'm not sure! I changed it based on the other systemd configs I saw on our system.

I looked into what the difference is between default.target and multi-user.target was. It sounds like default.target is usually an alias for either graphical.target or multi-user.target so that change might not be needed.

@bnfinet
Copy link
Member

bnfinet commented Dec 16, 2020

@woneill thanks for checking..

I'm not sure who might be running VP on their laptop but I bet they're out there. :)

With that adjustment I'm happy to merge.

bnfinet added a commit to simongottschlag/vouch-proxy that referenced this pull request Dec 18, 2020
@bnfinet bnfinet merged commit 46c8a84 into vouch:master Dec 18, 2020
@woneill
Copy link
Contributor Author

woneill commented Dec 19, 2020

@bnfinet Thanks for the merge! I'm sorry I misunderstood that you were asking for me to update the runlevel- I've come across Xwindows running on rackmounted servers so I don't discount what kinds of crazy things people do :)

@woneill woneill deleted the systemd branch December 19, 2020 23:36
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.

None yet

2 participants