Skip to content

Fixes #39683 - Scope host group Puppet class completion - #453

Open
jakduch wants to merge 1 commit into
theforeman:masterfrom
jakduch:fix/39683-hostgroup-puppetclass-completion
Open

Fixes #39683 - Scope host group Puppet class completion#453
jakduch wants to merge 1 commit into
theforeman:masterfrom
jakduch:fix/39683-hostgroup-puppetclass-completion

Conversation

@jakduch

@jakduch jakduch commented Aug 21, 2026

Copy link
Copy Markdown
Contributor

Summary

Fixes #39683 as a follow-up to foreman_puppet #450.

Host group Puppet class completion reused the host assignment scope, so it suggested classes assigned only to hosts. Pass the resource being completed to the Puppet class completer and use a dedicated host group assignment scope restricted to host groups visible in the current organization and location. Direct and inherited host group assignments and classes provided through config groups remain available.

Testing

  • Ruby syntax checks pass for all changed files
  • git diff --check passes
  • Regression coverage includes classes assigned through host group config groups and excludes classes assigned only to hosts
  • Full Rails tests are left to CI

AI usage disclosure

Per the community discussion on AI policy, the issue was investigated and the changes, tests, and PR wording were prepared with the assistance of Codex 5.6 Sol High. The resulting changes were reviewed before submitting. The commit also includes an Assisted-By trailer.

Use host group assignments when completing Puppet class values on the host group list. Keep direct, inherited, and config group assignments while excluding classes assigned only to hosts.

Assisted-By: Codex 5.6 Sol High
@jakduch
jakduch force-pushed the fix/39683-hostgroup-puppetclass-completion branch from de09128 to 7b3ce1c Compare August 21, 2026 12:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant