Skip to content

fix(pkgbuild): enable service on upgrade to recreate WantedBy symlink#118

Merged
M0Rf30 merged 1 commit into
mainfrom
CO-3797
May 29, 2026
Merged

fix(pkgbuild): enable service on upgrade to recreate WantedBy symlink#118
M0Rf30 merged 1 commit into
mainfrom
CO-3797

Conversation

@M0Rf30

@M0Rf30 M0Rf30 commented May 29, 2026

Copy link
Copy Markdown
Contributor

The postinst scriptlets only ran 'systemctl enable' on fresh install, so nodes upgraded across the PR #96 [Install] target change (carbonio-appserver.target -> service-discover.target) never got the new WantedBy symlink under service-discover.target.wants/. The target then failed to pull the service at boot (CO-3797).

Enable on upgrade too (yum: $1 -ge 1; apt: drop -z $2 guard) so the symlink is always reconciled against the current unit.

The postinst scriptlets only ran 'systemctl enable' on fresh install,
so nodes upgraded across the PR #96 [Install] target change
(carbonio-appserver.target -> service-discover.target) never got the
new WantedBy symlink under service-discover.target.wants/. The target
then failed to pull the service at boot (CO-3797).

Enable on upgrade too (yum: $1 -ge 1; apt: drop -z $2 guard) so the
symlink is always reconciled against the current unit.
@sonarqube-zextras

Copy link
Copy Markdown

@gjed gjed requested a review from Copilot May 29, 2026 12:30
@M0Rf30 M0Rf30 merged commit be75277 into main May 29, 2026
3 of 4 checks passed
@M0Rf30 M0Rf30 deleted the CO-3797 branch May 29, 2026 12:31

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

@ZxBot

ZxBot commented May 29, 2026

Copy link
Copy Markdown
Contributor

🎉 This PR is included in version 0.7.1 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

@ZxBot ZxBot added the released label May 29, 2026
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.

4 participants