Skip to content

Conversation

vvnginx
Copy link
Contributor

@vvnginx vvnginx commented Aug 29, 2025

Proposed changes

Add Directory storage requirements. Also, dependency requirements for NIM, NGINX and Clickhouse in offline mode

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.

@vvnginx vvnginx requested a review from a team as a code owner August 29, 2025 11:14
@github-actions github-actions bot added documentation Improvements or additions to documentation product/nim Issues related to NGINX Instance Manager labels Aug 29, 2025
Copy link
Contributor

@mjang mjang left a comment

Choose a reason for hiding this comment

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

Fix for format issues. Example, unless you use bullets, the build appears like this:

Screenshot 2025-08-29 at 6 50 39 AM

Copy link
Contributor

@mjang mjang left a comment

Choose a reason for hiding this comment

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

Approving from a TW PoV based on recent changes

@mjang
Copy link
Contributor

mjang commented Aug 29, 2025

Merging with one other approval, since other writers are either gone for the weekend or on PTO

@mjang mjang merged commit f744d7a into nginx:main Aug 29, 2025
7 checks passed
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/nim Issues related to NGINX Instance Manager

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants