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 06_configuring_for_eager_failover.mdx #4785

Merged
merged 1 commit into from
Sep 11, 2023

Conversation

tureba
Copy link
Contributor

@tureba tureba commented Sep 7, 2023

The safest way for end users to edit service files is using systemctl edit. Alternatively, we can instruct them on how to edit the overrides files in /etc/systemd/system/<unitname>.d/override.conf. But creating a new file and .include shouldn't have been an option. Moreover, the .include feature has been removed from recent systemd versions, so it must be changed before it starts erroring out in the customer's hands.

The safest way for end users to edit service files is using `systemctl edit`. Alternatively, we can instruct them on how to edit the overrides files in `/etc/systemd/system/<unitname>.d/override.conf`. But creating a new file and `.include` shouldn't have been an option. Moreover, the `.include` feature has been removed from recent systemd versions, so it must be changed before it starts erroring out in the customer's hands.
Copy link
Contributor

@EFM-Bobby EFM-Bobby left a comment

Choose a reason for hiding this comment

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

Looks good to me. The example uses efm 4.5, which is fine -- doesn't have to be the latest. But I'll update it when I make the changes for the next release so it doesn't look like this section has been forgotten.

@drothery-edb drothery-edb merged commit 7c3b72e into EnterpriseDB:develop Sep 11, 2023
1 check passed
@tureba tureba deleted the patch-12 branch November 10, 2023 15: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.

3 participants