Skip to content

docs: clarify create_machines (SiteExplorerConfig) config parameter#1678

Merged
poroh merged 1 commit into
NVIDIA:mainfrom
kurotych:clarify-create-machines-param
May 18, 2026
Merged

docs: clarify create_machines (SiteExplorerConfig) config parameter#1678
poroh merged 1 commit into
NVIDIA:mainfrom
kurotych:clarify-create-machines-param

Conversation

@kurotych
Copy link
Copy Markdown
Contributor

Description

The create_machines config parameter is poorly documented. Updated the description. Please correct me if description is inaccurate.

Type of Change

  • Add - New feature or capability
  • Change - Changes in existing functionality
  • Fix - Bug fixes
  • Remove - Removed features or deprecated functionality
  • Internal - Internal changes (refactoring, tests, docs, etc.)

Related Issues (Optional)

Breaking Changes

  • This PR contains breaking changes

Testing

  • Unit tests added/updated
  • Integration tests added/updated
  • Manual testing performed
  • No testing required (docs, internal refactor, etc.)

Additional Notes

@kurotych kurotych requested a review from a team as a code owner May 14, 2026 13:23
@copy-pr-bot
Copy link
Copy Markdown

copy-pr-bot Bot commented May 14, 2026

This pull request requires additional validation before any workflows can run on NVIDIA's runners.

Pull request vetters can view their responsibilities here.

Contributors can view more details about this message here.

Signed-off-by: Anatolii Kurotych <akurotych@gmail.com>
@kurotych kurotych force-pushed the clarify-create-machines-param branch from 83315ac to 85242a5 Compare May 14, 2026 13:24
@kurotych kurotych changed the title docs: clarify create_machines (SiteExplorerConfig) behavior docs: clarify create_machines (SiteExplorerConfig) config parameter May 14, 2026
@poroh
Copy link
Copy Markdown
Contributor

poroh commented May 18, 2026

/ok to test 85242a5

@poroh poroh enabled auto-merge (squash) May 18, 2026 16:45
@poroh poroh merged commit dff971a into NVIDIA:main May 18, 2026
40 checks passed
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.

3 participants