Jira (PUP-10527) Add beaker test to ensure puppet-agent service executes in the background

18 views
Skip to first unread message

Josh Cooper (Jira)

unread,
May 21, 2020, 2:17:03 PM5/21/20
to puppe...@googlegroups.com
Josh Cooper created an issue
 
Puppet / Task PUP-10527
Add beaker test to ensure puppet-agent service executes in the background
Issue Type: Task Task
Assignee: Unassigned
Created: 2020/05/21 11:16 AM
Priority: Normal Normal
Reporter: Josh Cooper

Commit https://github.com/puppetlabs/puppet/commit/7d8b0d37664e1bdf274bfa04ccf7ef5e5ab425eb#diff-c6a03ccca02fa0c4815d03fba3017602L192-R179 disable the job that runs the daemonized agent. As a result, puppet would never run when started as a service on *nix platforms. Had this gotten out we would have had to yank the release as daemonized agents would never run, so you couldn't use puppet to fix/update them.

Fortunately we noticed and fixed it before release (https://github.com/puppetlabs/puppet/commit/58c02cceda56025623ad5bc227579a9aaeddfc80) but we have some gaps in test coverage. It looks like our service test is getting skipped due to the medium tag, https://github.com/puppetlabs/puppet/blob/58c02cceda56025623ad5bc227579a9aaeddfc80/acceptance/tests/resource/service/puppet_service_management.rb#L3, though I don't think that would have been sufficient to detect this issue. We either need a new test that verifies the service can be started and run in the background or update the existing test. The success or failure of the run could be detected by the last run summary file that the agent produces or ensuring the master receives a report for that node.

/cc Mihai Buzgau, John Duarte, Christopher Thorn

Add Comment Add Comment
 
This message was sent by Atlassian Jira (v8.5.2#805002-sha1:a66f935)
Atlassian logo

Josh Cooper (Jira)

unread,
May 22, 2020, 11:06:04 AM5/22/20
to puppe...@googlegroups.com
Josh Cooper commented on Task PUP-10527
 
Re: Add beaker test to ensure puppet-agent service executes in the background

That should have caught the issue which makes me wonder why it didn’t? Does the test not run nightly?

I also agree that it’d be better to test this in PA pipeline.

Mihai Buzgau (Jira)

unread,
May 26, 2020, 10:46:03 AM5/26/20
to puppe...@googlegroups.com

Christopher Thorn (Jira)

unread,
May 26, 2020, 12:37:03 PM5/26/20
to puppe...@googlegroups.com
Christopher Thorn commented on Task PUP-10527
 
Re: Add beaker test to ensure puppet-agent service executes in the background

Those tests get ran whenever there is a SCM change to PE, puppet_agent module, or pe_acceptance_tests. So if the puppet-agent got promoted it should have ran. 

Mihai Buzgau (Jira)

unread,
May 27, 2020, 4:30:04 AM5/27/20
to puppe...@googlegroups.com

Mihai Buzgau (Jira)

unread,
May 27, 2020, 4:34:03 AM5/27/20
to puppe...@googlegroups.com

John Duarte (Jira)

unread,
Jun 2, 2020, 5:27:03 PM6/2/20
to puppe...@googlegroups.com

Mihai Buzgau (Jira)

unread,
Jun 10, 2020, 4:54:04 AM6/10/20
to puppe...@googlegroups.com
Mihai Buzgau updated an issue
Change By: Mihai Buzgau
Sprint: NW - 2020-06-10 , NW - 2020-06-24

Gabriel Nagy (Jira)

unread,
Jun 16, 2020, 8:51:03 AM6/16/20
to puppe...@googlegroups.com

Ciprian Badescu (Jira)

unread,
Jul 8, 2020, 9:04:03 AM7/8/20
to puppe...@googlegroups.com
Reply all
Reply to author
Forward
0 new messages