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 guides/logs 'xensource.log' specific config #301

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

cyrilchristin
Copy link

  • Added TIP information regarding changes to specific log rotation configuration for 'xensource.log'
  • Checkout upstream 'xen-api' pull #5507 / CA-389496

The above changes follow investigations of an 'unexpected' missing file /etc/cron.d/xapi-logrotate.cron on a new install.
We concluded this is normal/expected and is (probably) related to the update released in June 2024 for XCP-ng 8.2 LTS.
The upstream code changes were merged in March 2024 for 'xen-api' version > v24.12.0

Before submitting the pull request, you must agree with the following statements by checking both boxes with a 'x'.

  • "I accept that my contribution is placed under the CC BY-SA 2.0 license [1]."
  • "My contribution complies with the Developer Certificate of Origin [2]."

[1] https://creativecommons.org/licenses/by-sa/2.0/
[2] https://docs.xcp-ng.org/project/contributing/#developer-certificate-of-origin-dco

- Added TIP information regarding changes to specific log rotation configuration for 'xensource.log'
- Checkout upstream 'xen-api' pull #5507 / CA-389496

Signed-off-by: Cyril Christin <[email protected]>
@@ -24,7 +24,13 @@ Because a file must be rotated if a log exceeds 100 MiB, the `rsyslog` daemon is

## Specific config: `xensource.log`

:::tip No longer applicable since update released in June 2024 for XCP-ng 8.2 LTS.

The `xensource.log` file is still rotated but the `config+script+cron` files mentioned bellow are deleted during this update process.
Copy link
Member

Choose a reason for hiding this comment

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

s/bellow/below/

@stormi
Copy link
Member

stormi commented Dec 13, 2024

Thanks for your contribution! It's good, but if you want to make it even better, I think it would be best to write two separate sub sections, one for XCP-ng 8.2 prior to the June 2024 update, and one for current 8.2 + 8.3.

@stormi
Copy link
Member

stormi commented Jan 9, 2025

Hi! Will you submit a reviewed version of your commit?

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