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

Bug #1334035 reported by Francis Ginther on 2014-06-24
14
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Ubuntu CI Engine
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'

Francis Ginther (fginther) wrote :
Ursula Junque (ursinha) on 2014-06-25
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
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  Edit
Everyone can see this information.

Duplicates of this bug

Other bug subscribers