Skip to content

Issue 7832 - remove instance monitor directory when server is removed - #7839

Merged
mreynolds389 merged 1 commit into
389ds:mainfrom
mreynolds389:rmeove_monitor_file
Sep 16, 2026
Merged

mreynolds389 merged 1 commit into
389ds:mainfrom
mreynolds389:rmeove_monitor_file

Conversation

@mreynolds389

@mreynolds389 mreynolds389 commented Sep 11, 2026 •

Copy link
Copy Markdown
Contributor

Description:

The /run/dirsrv/slapd-INSTANCE.monitor/ directory is not removed when the instance is uninstalled.

relates: #7832

Summary by Sourcery

Remove leftover monitor runtime directories during Directory Server instance cleanup.

Bug Fixes:

  • Remove the instance monitor runtime directory when a Directory Server instance is uninstalled.
  • Extend removal tests to verify that the monitor directory is deleted.

Description:

The /run/dirsrv/slapd-INSTANCE.monitor/ directory is not removed when the
instance is uninstalled.

relates: 389ds#7832

Reviewed by: ?

@sourcery-ai sourcery-ai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Hey - I've reviewed your changes and they look great!


Sourcery is free for open source - if you like our reviews please consider sharing them ✨

@tbordaz tbordaz left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@mreynolds389
mreynolds389 merged commit 682269c into 389ds:main Sep 16, 2026
234 of 246 checks passed
@mreynolds389
mreynolds389 deleted the rmeove_monitor_file branch September 16, 2026 12:40
mreynolds389 added a commit that referenced this pull request Sep 16, 2026
…#7839)

Description:

The /run/dirsrv/slapd-INSTANCE.monitor/ directory is not removed when the
instance is uninstalled.

relates: #7832

Reviewed by: tbordaz(Thanks!)
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