Skip to content

DLPX-96879 Disable Influxdb default package#863

Open
dbshah12 wants to merge 2 commits into
developfrom
dlpx/pr/dbshah12/0b9a0001-b8d3-4fa7-aa23-23a650d87b5c
Open

DLPX-96879 Disable Influxdb default package#863
dbshah12 wants to merge 2 commits into
developfrom
dlpx/pr/dbshah12/0b9a0001-b8d3-4fa7-aa23-23a650d87b5c

Conversation

@dbshah12

@dbshah12 dbshah12 commented Apr 2, 2026

Copy link
Copy Markdown
Contributor

Problem

The influxdb2 package ships with a default influxdb.service systemd unit that starts automatically when the package is installed. The performance-diagnostics package (delphix/performance-diagnostics#119) declares influxdb2 as a runtime dependency and provides its own custom service wrapper (delphix-influxdb.service) that handles InfluxDB startup along with one-time initialization (org/bucket/token setup). If the default influxdb.service is not disabled, it will auto-start on install/boot and conflict with the custom service.

Solution

Add influxdb.service to the list of services that are disabled and masked at appliance build time in the appliance-build.virtualization-common Ansible role. This follows the same pattern used for other services (e.g., nginx.service, postgresql.service, telegraf.service) that are installed by packages but managed by Delphix-specific service wrappers.

This ensures that when influxdb2 is installed via the performance-diagnostics package, the default upstream service does not auto-start and the delphix-influxdb.service unit remains the sole entry point for managing InfluxDB on the appliance.

Testing Done

Covered by the ab-pre-push run in the dependent PR: delphix/performance-diagnostics#119.

@dbshah12 dbshah12 force-pushed the dlpx/pr/dbshah12/0b9a0001-b8d3-4fa7-aa23-23a650d87b5c branch from 44c429f to d233f80 Compare April 2, 2026 07:24
@dbshah12 dbshah12 self-assigned this Apr 2, 2026
@dbshah12 dbshah12 requested a review from nealquigley April 2, 2026 07:30
@dbshah12 dbshah12 marked this pull request as ready for review April 2, 2026 07:30

@VenkatanadhanG VenkatanadhanG 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.

Do we need to wait for https://github.com/delphix/performance-diagnostics/pull/119/changes before merging this.

Changes looks good.

@dbshah12 dbshah12 force-pushed the dlpx/pr/dbshah12/0b9a0001-b8d3-4fa7-aa23-23a650d87b5c branch from d233f80 to 7efa1b6 Compare April 2, 2026 08:46

@ShibasishDelphix ShibasishDelphix 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.

LGTM, I think we need to merge https://github.com/delphix/performance-diagnostics/pull/119/changes first though.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants