Skip to content

docs: fix incorrect Foreman examples in NodeClassifier docs#131

Merged
slauger merged 1 commit intomainfrom
docs/26-foreman-incompatibility
Mar 16, 2026
Merged

docs: fix incorrect Foreman examples in NodeClassifier docs#131
slauger merged 1 commit intomainfrom
docs/26-foreman-incompatibility

Conversation

@slauger
Copy link
Owner

@slauger slauger commented Mar 16, 2026

Summary

  • Remove misleading Foreman examples that suggested direct compatibility with the NodeClassifier CRD
  • Move Foreman into a "Known Limitations" section explaining the incompatibility (two-step ENC protocol with separate facts upload)
  • Link to docs: Foreman integration guide for NodeClassifier CRD #26 for details and planned workarounds
  • Update authentication table and Config example to not reference Foreman

Test plan

  • Review rendered markdown for correct links and formatting

Replace misleading Foreman examples that suggested direct compatibility
with a note explaining that Foreman is not directly compatible with the
NodeClassifier CRD due to its two-step ENC protocol (facts upload +
classification). Link to #26 for details and planned workarounds.
@slauger slauger merged commit c899f8d into main Mar 16, 2026
11 of 12 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.

1 participant