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

Simplify wwctl server command and logging #1229

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

anderbubble
Copy link
Collaborator

@anderbubble anderbubble commented May 24, 2024

Description of the Pull Request (PR):

  • Always run in the foreground
  • Always log to stdout/err (unless syslog is requested)
  • Update systemd to reload with a direct signal
  • Omit any pidfile for "warewulfd"

Issues affected

Before submitting a PR, make sure you have done the following:

- Always run in the foreground
- Always log to stdout/err (unless syslog is requested)
- Update systemd to reload with a direct signal
- Omit any pidfile for "warewulfd"

Signed-off-by: Jonathon Anderson <janderson@ciq.com>
@anderbubble anderbubble added this to the v4.6.0 milestone May 24, 2024
- Collapse SEND and RECV logs into INFO.
- Add /etc/default/warewulfd

Closes warewulf#725

Signed-off-by: Jonathon Anderson <janderson@ciq.com>
@anderbubble anderbubble marked this pull request as ready for review May 24, 2024 23:14
@anderbubble anderbubble changed the title Simplify wwctl server Simplify wwctl server command and logging Jun 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
1 participant