http://gerrit.beaker-project.org/#/c/1205/

Tests broken:

FAIL: test_change_retention_tag (bkr.inttest.server.selenium.test_jobs.JobAttributeChange)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/home/dcallagh/work/beaker/IntegrationTests/src/bkr/inttest/server/selenium/test_jobs.py", line 464, in test_change_retention_tag
    self.wait_and_try(lambda: self.assert_(sel.is_text_present("Tag has been updated")), wait_time=10)
  File "/home/dcallagh/work/beaker/IntegrationTests/src/bkr/inttest/server/selenium/__init__.py", line 50, in wait_and_try
    f()
  File "/home/dcallagh/work/beaker/IntegrationTests/src/bkr/inttest/server/selenium/test_jobs.py", line 464, in <lambda>
    self.wait_and_try(lambda: self.assert_(sel.is_text_present("Tag has been updated")), wait_time=10)
AssertionError: 

Server traceback is:

2012-07-13 13:58:33,870 cherrypy.msg INFO HTTP: Page handler: <bound method Jobs.update of <bkr.server.jobs.Jobs object at 0x2ed2110>>
Traceback (most recent call last):
[...]
  File "/home/dcallagh/work/beaker/Server/bkr/server/jobs.py", line 739, in update
    returns.update(Utility.update_task_product(job,retention_tag_id,product_id))
  File "/home/dcallagh/work/beaker/Server/bkr/server/job_utilities.py", line 38, in update_task_product
    return cls.check_retentiontag_job(job, retentiontag)
  File "/home/dcallagh/work/beaker/Server/bkr/server/job_utilities.py", line 76, in check_retentiontag_job
    if new_retentiontag.requires_product() != bool(the_product):
AttributeError: 'int' object has no attribute 'requires_product'

http://gerrit.beaker-project.org/#/c/1213/

Trying to set a product while leaving the retention tag at 'scratch' gives this unhelpful error:

<Fault 1: "<class 'bkr.common.bexceptions.BeakerException'>:'Job 16475 not updated: Unknown reason'">

Should we try and make the message more descriptive?

(In reply to comment #6)
> Trying to set a product while leaving the retention tag at 'scratch' gives
> this unhelpful error:
> 
> <Fault 1: "<class 'bkr.common.bexceptions.BeakerException'>:'Job 16475 not
> updated: Unknown reason'">
> 
> Should we try and make the message more descriptive?

Setting the retention tag back to scratch without unsetting the product gives a much better error message:

<Fault 1: "<class 'bkr.common.bexceptions.BeakerException'>:'Job 16475 not updated: Incompatible product and tags'">

http://gerrit.beaker-project.org/#/c/1215/

Beaker 0.9.1 has been released.