With the very recent changes to systemd we "lost" our LD_PRELAOD of jemalloc. We need to add it back to the new custom drop-in file
As Viktor pointed out, we should not preload jemalloc in case of ASAN build. Therefore, I would suggest to have a separate drop-in file doing this, which would then be installed only when necessary (similarly to xsan.conf, but in the other case).
Also, seems like *-initconfig files are not necessary after the recent cleanup either. Changes along the patch attached should fix this. <img alt="0001-Drop-initconfig-files-in-case-of-SystemD.patch" src="/389-ds-base/issue/raw/files/6800f970c9923cccfe042d0ac2d9d88c12c10a5608e0209bf1dcdecace5b388d-0001-Drop-initconfig-files-in-case-of-SystemD.patch" />
Metadata Update from @mhonek: - Custom field origin adjusted to None - Custom field reviewstatus adjusted to None
I also have a working patch for this(needs some cleanup though), but preloading jemalloc must be the default behavior.
@mhonek do we still need those init files for non-systemd systems? It's going to complicate the spec file otherwise.
Filed https://pagure.io/389-ds-base/pull-request/50501 We need to get this addressed one way or another, and sooner than later. I filed a PR but its just one of many ways we could solve this.
Commit 7466be3 relates to this ticket
Metadata Update from @mreynolds: - Issue close_status updated to: fixed - Issue status updated to: Closed (was: Open)
Commit 76c0e75 relates to this ticket
389-ds-base is moving from Pagure to Github. This means that new issues and pull requests will be accepted only in 389-ds-base's github repository.
This issue has been cloned to Github and is available here: - https://github.com/389ds/389-ds-base/issues/3483
If you want to receive further updates on the issue, please navigate to the github issue and click on subscribe button.
subscribe
Thank you for understanding. We apologize for all inconvenience.
Metadata Update from @spichugi: - Issue close_status updated to: wontfix (was: fixed)
Log in to comment on this ticket.