watch_ppa: add support to watch the uploaded package series, not only the default one

Bug #1334035 reported by Francis Ginther
14
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Ubuntu CI Engine
New
High
Ursula Junque

Bug Description

bsbuilder only checks for packages on saucy series. When a package targeted to another series is uploaded, it will check in the wrong series, never find the package and fail. We need to add support to watch_ppa to watch the proper package series.

branch source builder with revision 607, when processing a precise package, failed twice with different errors, as below:

'message_list' is not defined in bsbuilder

[Log snippet:]
[2014-06-24 21:26:20,223] bsbuilder:INFO:upload_list: [{'files': [u'hello_2.8-4ubuntu3_source.changes', u'hello_2.8-4ubuntu3.debian.tar.gz', u'hello_2.8-4ubuntu3.dsc'], 'resource': u'/api/v1/fullsubticket/4/', 'name': u'hello', 'version': u'2.8-4ubuntu3', 'location': '/tmp/tmpe6Ukp3', 'id': 4, 'architecture': 'any'}]
[2014-06-24 21:26:32,584] bsbuilder:INFO:subticket_status: [{'status': 210, 'step': 100, 'version': u'2.8-4ubuntu3', 'resource': u'/api/v1/fullsubticket/4/', 'name': u'hello', 'artifacts': None, 'step_text': 'Queued', 'message': None, 'status_text': 'Not started', 'id': 4}]
[2014-06-24 21:26:32,593] bsbuilder:INFO:Going to sleep for 10
...
[2014-06-24 21:46:18,059] bsbuilder:INFO:subticket_status: [{'status': 210, 'step': 100, 'version': u'2.8-4ubuntu3', 'resource': u'/api/v1/fullsubticket/4/', 'name': u'hello', 'artifacts': None, 'step_text': 'Queued', 'message': None, 'status_text': 'Not started', 'id': 4}]
[2014-06-24 21:46:18,059] bsbuilder:INFO:Going to sleep for 10
[2014-06-24 21:46:32,517] bsbuilder:ERROR:Unexpected exception occurred
Traceback (most recent call last):
  File "/srv/ci-airline-bsb-worker/ci-utils/ci_utils/amqp_worker.py", line 198, in _on_message
    amqp_cb, ret = self.handle_request(params, logger)
  File "./branch-source-builder/bsbuilder/run_worker.py", line 126, in handle_request
    upload_list)
  File "/srv/ci-airline-bsb-worker/branch-source-builder/watch_ppa.py", line 346, in watch_ppa
    message_list.append(MISSING_SOURCE_MESSAGE.format(
NameError: global name 'message_list' is not defined

and

'SubTicketWorkflowStepStatus' has no attribute 'WAITING' in bsbuilder

[log snippet:]
[2014-06-24 20:23:25,306] bsbuilder:INFO:upload_list: [{'files': [u'hello_2.8-4ubuntu3_source.changes', u'hello_2.8-4ubuntu3.debian.tar.gz', u'hello_2.8-4ubuntu3.dsc'], 'resource': u'/api/v1/fullsubticket/2/', 'name': u'hello', 'version': u'2.8-4ubuntu3', 'location': '/tmp/tmpp3_gK3', 'id': 2, 'architecture': 'any'}]
[2014-06-24 20:23:35,740] bsbuilder:INFO:subticket_status: [{'status': 210, 'step': 100, 'version': u'2.8-4ubuntu3', 'resource': u'/api/v1/fullsubticket/2/', 'name': u'hello', 'artifacts': None, 'step_text': 'Queued', 'message': None, 'status_text': 'Not started', 'id': 2}]
[2014-06-24 20:23:35,749] bsbuilder:INFO:Going to sleep for 10
...
[2014-06-24 20:43:30,833] bsbuilder:INFO:subticket_status: [{'status': 210, 'step': 100, 'version': u'2.8-4ubuntu3', 'resource': u'/api/v1/fullsubticket/2/', 'name': u'hello', 'artifacts': None, 'step_text': 'Queued', 'message': None, 'status_text': 'Not started', 'id': 2}]
[2014-06-24 20:43:30,833] bsbuilder:INFO:Going to sleep for 10
[2014-06-24 20:43:44,738] bsbuilder:ERROR:Unexpected exception occurred
Traceback (most recent call last):
  File "/srv/ci-airline-bsb-worker/ci-utils/ci_utils/amqp_worker.py", line 198, in _on_message
    amqp_cb, ret = self.handle_request(params, logger)
  File "./branch-source-builder/bsbuilder/run_worker.py", line 126, in handle_request
    upload_list)
  File "/srv/ci-airline-bsb-worker/branch-source-builder/watch_ppa.py", line 340, in watch_ppa
    if package['status'] == SubTicketWorkflowStepStatus.WAITING.value:
AttributeError: type object 'SubTicketWorkflowStepStatus' has no attribute 'WAITING'

Revision history for this message
Francis Ginther (fginther) wrote :
Ursula Junque (ursinha)
summary: - 'message_list' is not defined in bsbuilder
+ watch_ppa: add support to watch the uploaded package series, not only
+ the default one
description: updated
tags: added: airline bsbuilder
Revision history for this message
Paul Larson (pwlars) wrote :

Assigning to Ursula since she seems to be working on this in the cli right now

Changed in uci-engine:
importance: Undecided → High
assignee: nobody → Ursula Junque (ursinha)
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Duplicates of this bug

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.