Skip to content

nic: plm cli options - #2196

Open
vepatel wants to merge 6 commits into
nic-release-5.6from
nic/plm-options-args
Open

nic: plm cli options#2196
vepatel wants to merge 6 commits into
nic-release-5.6from
nic/plm-options-args

Conversation

@vepatel

@vepatel vepatel commented Aug 10, 2026

Copy link
Copy Markdown
Contributor

Proposed changes

Checklist

Before sharing this pull request, I completed the following checklist:

Footnotes

  1. Potentially sensitive information includes personally identify information (PII), authentication credentials, and live URLs. Refer to the style guide for guidance about placeholder content.

@github-actions github-actions Bot added documentation Improvements or additions to documentation product/nic Issues related to NGINX Ingress Controller labels Aug 10, 2026
@github-actions

Copy link
Copy Markdown

Deploy Preview will be available once build job completes!

Name Link
😎 Deploy Preview https://frontdoor-test-docs.nginx.com/previews/docs/2196/

@vepatel
vepatel marked this pull request as ready for review August 10, 2026 19:04
@vepatel
vepatel requested a review from a team as a code owner August 10, 2026 19:04
@vepatel vepatel changed the title feat: plm cli options nic: plm cli options Aug 10, 2026

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

Approved

Comment thread content/nic/configuration/global-configuration/command-line-arguments.md Outdated
Comment thread content/nic/configuration/global-configuration/command-line-arguments.md Outdated
Comment thread content/nic/configuration/global-configuration/command-line-arguments.md Outdated
Comment thread content/nic/configuration/global-configuration/command-line-arguments.md Outdated
Comment thread content/nic/configuration/global-configuration/command-line-arguments.md Outdated
Comment thread content/nic/install/helm/parameters.md Outdated
Comment thread content/nic/configuration/policy-resource.md Outdated
Comment thread content/nic/configuration/policy-resource.md Outdated
Comment thread content/nic/install/helm/parameters.md Outdated
Comment thread content/nic/install/helm/parameters.md Outdated
Co-authored-by: yar <y82@users.noreply.github.com>
Comment thread content/nic/configuration/global-configuration/command-line-arguments.md Outdated
Comment thread content/nic/configuration/global-configuration/command-line-arguments.md Outdated
Comment thread content/nic/configuration/global-configuration/command-line-arguments.md Outdated
Co-authored-by: yar <y82@users.noreply.github.com>
Comment on lines +482 to +484
<a name="cmdoption-plm-storage-url"></a>

### -plm-storage-url

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.

There is a way to do this without HTML (and we very much prefer not using HTML)

Suggested change
<a name="cmdoption-plm-storage-url"></a>
### -plm-storage-url
### -plm-storage-url {#cmdoption-plm-storage-url}

By default Hugo adds anchors to every header. If not custom anchor is added, ### -plm-storage-url will create the #-plm-storage-url anchor automatically.

Apply the same structure to the other anchors

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

can we do it as a separate PR since it involved unrelated changes?

Comment thread content/nic/configuration/global-configuration/command-line-arguments.md Outdated
vepatel and others added 2 commits August 12, 2026 17:59
Co-authored-by: Jon Torre <78599298+JTorreG@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation product/nic Issues related to NGINX Ingress Controller

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants