Skip to content

Setup inventory view using oneshot quadlet#32

Open
ehelms wants to merge 1 commit into
masterfrom
use-quadlet-inventory-view
Open

Setup inventory view using oneshot quadlet#32
ehelms wants to merge 1 commit into
masterfrom
use-quadlet-inventory-view

Conversation

@ehelms
Copy link
Copy Markdown
Member

@ehelms ehelms commented Jul 7, 2025

No description provided.

},
'Service' => {
'Type' => 'oneshot',
'RemainAfterExit' => 'true',
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could this cause issues if the view would need to be updated?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes. But the same problem exists with migrations and the services in general. There is the open question of "how does Puppet know it needs to restart services?". In our upgrade tooling, we have worked around this complication by ensuring we stop all services during upgrade. Then the installer will start all the services.

end

describe service('iop-core-host-inventory-create-view') do
it { is_expected.to be_enabled }
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Unrelated: Do you happen to have a link to documentation that describes the be_enable and be_running predicates. I wasn't able to find any. :-/

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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.

2 participants