Skip to content
This repository was archived by the owner on Sep 29, 2026. It is now read-only.

docs(configuration): require hostnames of at least 2 characters - #70

Closed
GhostofGoes wants to merge 1 commit into
sandialabs:mainfrom
GhostofGoes:docs-hostname-min-length
Closed

GhostofGoes wants to merge 1 commit into
sandialabs:mainfrom
GhostofGoes:docs-hostname-min-length

Conversation

@GhostofGoes

Copy link
Copy Markdown
Contributor

Description

Document the new 2-character minimum for topology node hostnames. VyOS, Vyatta, and Windows reject 1-character hostnames, so phenix now rejects them at validation time.

  • configuration.md: change the hostname rule from 1–63 to 2–63 characters and link the VyOS and Microsoft references.
  • schema.md: update the reproduced minimega_node hostname schema to match.

Related Issues/PRs

Type of Change

  • Bugfix (fix)
  • Feature (feat)
  • Documentation (docs)
  • Refactor (refactor)
  • Chore (CI, build, dependencies, etc.) (chore)
  • Other (please describe):

Checklist

  • This PR conforms to the process detailed in the Contributing Guide.
  • I have included no proprietary/sensitive information in my code or the PR.
  • I have commented my code, particularly in hard-to-understand areas.
  • I have made corresponding changes to the documentation.
  • I have tested my code (describe below).

Testing

prek hooks pass. The site was not built locally.

Additional Notes

Merge together with sandialabs/sceptre-phenix#429.

🤖 Generated with Claude Code

phenix now rejects 1-character node hostnames because VyOS, Vyatta, and
Windows do not accept them. Update the hostname rules and the schema
reference to match, and link the VyOS and Microsoft references.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
cmulk pushed a commit to sandialabs/sceptre-phenix that referenced this pull request Sep 29, 2026
phenix now rejects 1-character node hostnames because VyOS, Vyatta, and
Windows do not accept them. Update the hostname rules and the schema
reference to match, and link the VyOS and Microsoft references.

Migrated from sandialabs/sceptre-phenix-docs#70.

Refs: #429

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
@cmulk cmulk closed this Sep 29, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants