Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

confining on structured facts isn't optimal #1170

Closed
bastelfreak opened this issue Aug 7, 2024 · 2 comments
Closed

confining on structured facts isn't optimal #1170

bastelfreak opened this issue Aug 7, 2024 · 2 comments
Labels
triaged This ticket has been accepted to our backlog and migrated internally

Comments

@bastelfreak
Copy link

bastelfreak commented Aug 7, 2024 •
edited
Loading

Describe the Change You Would Like

https://www.puppet.com/docs/puppet/8/custom_facts.html#configuring_facts-confining-facts suggests:

confine 'os' do |os|
  os['family'] == 'RedHat'
end

but I think this is overly complicated and should be shortened:

confine 'os.family' == 'RedHat'
@lisarobertson lisarobertson added the triaged This ticket has been accepted to our backlog and migrated internally label Aug 9, 2024
@github-actions GitHub Actions
Copy link

github-actions bot commented Aug 9, 2024

Migrated issue to PUPDOC-5681

@mtaggart13
Copy link

Issue will be fixed in docs scheduled for release on Sept 10th.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
triaged This ticket has been accepted to our backlog and migrated internally
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@bastelfreak @lisarobertson @mtaggart13