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

add journal-remote settings #483

Closed
wants to merge 2 commits into from

Conversation

trefzer
Copy link
Contributor

@trefzer trefzer commented Aug 13, 2024

WIP, DO NOT MERGE YET

Pull Request (PR) description

add possibility to manage journal-remote (central log server)

@trefzer trefzer force-pushed the add_journal_remote branch 3 times, most recently from 210b804 to 8777432 Compare August 13, 2024 13:30
@trefzer trefzer marked this pull request as draft August 13, 2024 13:48
Reason: equal options in journal-upload and journal-remote
@@ -1,2 +1,3 @@
---
systemd::nspawn_package: 'systemd-container'
systemd::journal_remote::package_name: 'systemd-journal-remote'
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Would be good to add an acceptance test to keep confirming all the package names are correct.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

right, bu was not able to create a succesfull test. I assume, I cannot start systemd-journal-remote service in the environment. I'm giving up now ;(
(see #482 for the commit trying ;))

@trefzer
Copy link
Contributor Author

trefzer commented Aug 14, 2024

I will include this in merge request #482 because of several merge conflicts and the possibility to use same spec tests.

@trefzer trefzer closed this Aug 14, 2024
@trefzer trefzer deleted the add_journal_remote branch August 14, 2024 12:54
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.

2 participants