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 steps for forwarding system logs to use new syntax in rsyslog.conf #2005

Merged
merged 1 commit into from
Oct 18, 2024

Conversation

nauaneed
Copy link
Contributor

@nauaneed nauaneed commented Sep 8, 2024

Updated steps for forwarding of system logs to use new rsyslog syntax.

Without this, rsyslogd would report error during parsing file /etc/rsyslog.conf

Ref: https://docs.redhat.com/en/documentation/red_hat_enterprise_linux/6/html/deployment_guide/sec-using_the_new_syntax_for_rsyslog_queues#sec-Using_the_New_Syntax_for_rsyslog_queues

Updated steps for forwarding of system logs to use new rsyslog syntax.
Without this, rsyslogd would report `error during parsing file /etc/rsyslog.conf`

Ref: https://docs.redhat.com/en/documentation/red_hat_enterprise_linux/6/html/deployment_guide/sec-using_the_new_syntax_for_rsyslog_queues#sec-Using_the_New_Syntax_for_rsyslog_queues

Signed-off-by: Navaneet Villodi <[email protected]>
@nauaneed nauaneed changed the title Update steps for forwarding system logs to use new syntax for rsyslog.conf Update steps for forwarding system logs to use new syntax in rsyslog.conf Sep 8, 2024
Copy link

github-actions bot commented Sep 8, 2024

Test Results

14 files  ±0  14 suites  ±0   6s ⏱️ ±0s
41 tests ±0  41 ✅ ±0  0 💤 ±0  0 ❌ ±0 
54 runs  ±0  54 ✅ ±0  0 💤 ±0  0 ❌ ±0 

Results for commit baafee3. ± Comparison against base commit 8e59d8c.

@nauaneed
Copy link
Contributor Author

nauaneed commented Sep 8, 2024

The previous workflow run was unsuccessful due to an issue reported by the codespell linter. Now, added the fix suggested by codespell linter.

Copy link
Contributor

@martin-g martin-g left a comment

Choose a reason for hiding this comment

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

Rocky 9.x and openEuler 22.03 LTS SP3 provide RSyslog 8.x, so the change looks good to me!

@nauaneed
Copy link
Contributor Author

Hmm, so commit 8b02d0d from yesterday introduces a merge conflict. Let me remove 2d3cc8b to fix this.

Copy link

A friendly reminder that this PR had no activity for 30 days.

@adrianreber adrianreber merged commit d731775 into openhpc:3.x Oct 18, 2024
19 of 20 checks passed
@adrianreber
Copy link
Member

Thanks.

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

Successfully merging this pull request may close these issues.

3 participants