Bug 1014695

Summary: Submission delegate support in bkr.client
Product: [Retired] Beaker Reporter: Miroslav Hradílek <mhradile>
Component: command lineAssignee: Dan Callaghan <dcallagh>
Status: CLOSED CURRENTRELEASE QA Contact: tools-bugs <tools-bugs>
Severity: medium Docs Contact:
Priority: medium    
Version: 0.14CC: aigao, asaha, bpeck, dcallagh, dowang, jstancek, qwan, tools-bugs
Target Milestone: 19.1Keywords: EasyFix, Patch
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
: 1215138 (view as bug list) Environment:
Last Closed: 2014-12-17 08:40:06 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Bug Depends On:    
Bug Blocks: 1215138    

Description Miroslav Hradílek 2013-10-02 14:40:49 UTC
Description of problem:

It seems that, currently, there is not support for submission-delegate (job atribute user) in bkr.client. I was expecting it to be here :( matching the server's new functionality.

class BeakerJob(BeakerBase):
    def __init__(self, *args, **kwargs):
8><-----
        if kwargs.get('retention_tag'):
            self.node.setAttribute('retention_tag', kwargs.get('retention_tag'))
        if kwargs.get('product'):
            self.node.setAttribute('product', kwargs.get('product'))
        if kwargs.get('job_group'):
            self.node.setAttribute('group', kwargs.get('job_group'))

something like should be sufficient:

        if kwargs.get('user'):
            self.node.setAttribute('user', kwargs.get('user'))

Version-Release number of selected component (if applicable):
beaker-client-0.14.1-1.fc18.noarch

How reproducible:
100%

Comment 2 Nick Coghlan 2013-10-03 01:49:00 UTC
Huh, I thought we added that along with the rest of the feature, but it appears not.

The new parameter should be called "--job-owner" (the fact we called the XML attribute "user" instead of "owner" was a design mistake)

Comment 4 Bill Peck 2013-11-21 15:21:44 UTC
can this be moved up to 0.14.4?  Seems a simple fix no?

Thanks

Comment 5 Nick Coghlan 2013-11-22 03:47:14 UTC
I agree it would be nice, but the regression in bug 1033032 just made life a little more interesting than I like :(

That said, I'm easily bought by Gerrit patches that come with tests and docs ;)

Comment 6 Dan Callaghan 2014-11-24 01:20:00 UTC
http://gerrit.beaker-project.org/3505

Comment 11 Dan Callaghan 2014-12-17 08:40:06 UTC
Beaker 19.1 is released.