Skip to content

fix(systemctl): use product-key for service and container names in shared docs#7171

Open
jstirnaman wants to merge 3 commits into
masterfrom
fix-systemctl-product-key
Open

fix(systemctl): use product-key for service and container names in shared docs#7171
jstirnaman wants to merge 3 commits into
masterfrom
fix-systemctl-product-key

Conversation

@jstirnaman
Copy link
Copy Markdown
Contributor

Summary

Six systemctl commands and two Docker container names in shared admin docs used hardcoded names that don't match either product:

  • `shared/influxdb3-admin/upgrade.md` — 4 instances of `systemctl ... influxdb3` (bare). The packaged service name is `influxdb3-core` or `influxdb3-enterprise`; bare `influxdb3` is not a valid unit name on either product, so these commands as written would fail on a real install.
  • `shared/influxdb3-admin/backup-restore.md` — 2 instances of `systemctl ... influxdb3` and 2 instances of `docker ... influxdb3-core` in the file-system restore example. The Docker name was hardcoded to the Core convention; on the Enterprise rendering this would point to a non-existent container.

Replace with `influxdb3-{{< product-key >}}` so each rendered page (Core / Enterprise) emits the correct name. This is the same convention already used by the install page's systemd section.

Verified against the packaged unit file at `.circleci/packages/influxdb3/fs/lib/systemd/system/influxdb3-enterprise.service` (v3.9.1).

Refs influxdata/DAR#650 (the reporter flagged this inconsistency in the issue body).

Test plan

  • `yarn test:codeblocks` on the changed shared files renders cleanly.
  • On the Core-rendered upgrade and backup-restore pages, the systemctl commands show `influxdb3-core`.
  • On the Enterprise-rendered upgrade and backup-restore pages, the systemctl commands show `influxdb3-enterprise`.

…ared docs

The packaged systemd unit names are influxdb3-core and influxdb3-enterprise
(verified against the v3.9.1 unit file at
.circleci/packages/influxdb3/fs/lib/systemd/system/influxdb3-enterprise.service).
Several systemctl commands in shared admin docs used the bare name
"influxdb3", which is not a valid unit on either product. The
file-system restore example in backup-restore.md also hard-coded
"influxdb3-core" as the Docker container name in fallbacks; on the
Enterprise rendering the conventional name is influxdb3-enterprise.

Replace the bare/hard-coded names with the {{< product-key >}}
shortcode so each rendered page emits the correct name. This is the
same convention already used in the install page's systemd section.

Refs influxdata/DAR#650
@jstirnaman jstirnaman requested a review from a team as a code owner April 29, 2026 21:25
@jstirnaman jstirnaman requested review from sanderson and removed request for a team April 29, 2026 21:25
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 29, 2026

Vale Style Check Results

Metric Count
Errors 0
Warnings 4
Suggestions 0
Warnings (4)
File Line Rule Message
content/influxdb3/core/admin/backup-restore.md 18 InfluxDataDocs.WordList Use 'administrator' instead of 'admin'.
content/influxdb3/core/admin/upgrade.md 17 InfluxDataDocs.WordList Use 'administrator' instead of 'admin'.
content/influxdb3/enterprise/admin/backup-restore.md 20 InfluxDataDocs.WordList Use 'administrator' instead of 'admin'.
content/influxdb3/enterprise/admin/upgrade.md 19 InfluxDataDocs.WordList Use 'administrator' instead of 'admin'.

Check passed

@github-actions github-actions Bot added product:shared Shared content across products product:v3-monolith InfluxDB 3 Core and Enterprise (single-node / clusterable) labels Apr 29, 2026
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 29, 2026

PR Preview Action v1.4.8
🚀 Deployed preview to https://influxdata.github.io/docs-v2/pr-preview/pr-7171/
on branch gh-pages at 2026-05-01 22:15 UTC

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 29, 2026

📦 PR Preview — Preview Bot

Status Details
Result ✅ DEPLOYED
Preview View preview
Pages 4 page(s) deployed
Build time 72s
Last updated 2026-05-01 22:15:24 UTC
Pages included in this preview
  • /influxdb3/enterprise/admin/backup-restore/
  • /influxdb3/core/admin/backup-restore/
  • /influxdb3/enterprise/admin/upgrade/
  • /influxdb3/core/admin/upgrade/

Preview auto-deploys on push. Will be cleaned up when PR closes.

@jstirnaman jstirnaman added source:feedback From user feedback source:dar Generated by DAR pipeline (issue analysis) root-cause:missing-test root-cause:product-drift labels May 1, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

product:shared Shared content across products product:v3-monolith InfluxDB 3 Core and Enterprise (single-node / clusterable) root-cause:missing-test root-cause:product-drift source:dar Generated by DAR pipeline (issue analysis) source:feedback From user feedback

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants