Packaging CI jobs should register in CR before start

Bug #1513425 reported by Dmitry Mescheryakov
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Fuel for OpenStack
Fix Committed
Medium
Fuel CI

Bug Description

Right now, it does not happen and as a result it leaves wrong impression that CI is finished, while it is not. For example see
https://review.fuel-infra.org/#/c/13649/

Example of good behaviour: mos-infra-ci leaves the following comment on start
------------------------------------------------
Patch Set 1: -Verified
Build started (check pipeline) http://status.mosi.mirantis.net/zuul/
gate-oslo.messaging-pep8 STARTED
gate-oslo.messaging-python27 STARTED
------------------------------------------------
As a result, gate-oslo.messaging-pep8 and gate-oslo.messaging-python27 jobs appear in CI table when they are running and a user understands that CI is not finished yet.

The same should be done for all other CI jobs, specifically for 8.0-pkg-pipeline-ubuntu and 8.0-pkg-pipeline-centos

Another bad example is comment from Pkgs Jenkins:
------------------------------------------------
Patch Set 1: -Verified
Starting pkg-build-mos jobs. http://packaging-ci-gate.infra.mirantis.net/
------------------------------------------------
It is bad because the comment it done in wrong format and as a result, it does not make 'pkg-build-mos' appear in the CI table

description: updated
Changed in fuel:
milestone: none → 8.0
importance: Undecided → Medium
status: New → Confirmed
tags: added: area-ci
Revision history for this message
Alexander Evseev (aevseev) wrote :

I tried patch ('Report jobs that will run immediately' by Dmitry Teselkin) used by Infa CI - it not works with new Zuul (Infra uses 2.0.1, Packaging uses 2.1.0):

2015-11-11 09:25:04,595 ERROR zuul.IndependentPipelineManager: Exception while reporting start:
Traceback (most recent call last):
  File "/usr/local/lib/python2.7/dist-packages/zuul/scheduler.py", line 1096, in reportStart
    for job in self.pipeline.getJobs(change):
  File "/usr/local/lib/python2.7/dist-packages/zuul/model.py", line 123, in getJobs
    if not item.live:
AttributeError: 'Change' object has no attribute 'live'

So this feature needs more Zuul investigation.

Revision history for this message
Fuel Devops McRobotson (fuel-devops-robot) wrote : Fix merged to fuel-infra-packages/zuul (master)

Reviewed: https://review.fuel-infra.org/17179
Submitter: Andrey Nikitin <email address hidden>
Branch: master

Commit: f8b0a2a12a30d5487378b9839032bd49640fc2a7
Author: Alexander Evseev <email address hidden>
Date: Thu Feb 18 12:55:40 2016

Start zuul services via Upstart configs

 * Replace start scripts by upstart configuration files
 * Use tilde expansion instead of environment variable HOME (not set by
   Upstart)
 * Update patch for dealing with merge-commit changes (containing the
   only file '/COMMIT_MSG')
 * Add patch to report QUEUED state
 * Fix permissions for directory /etc/zuul/layout

All patches used in package:
 * https://review.openstack.org/215553
 * https://review.openstack.org/239943
 * https://review.openstack.org/246433
 * https://review.openstack.org/247383
 * https://review.openstack.org/272967
 * https://review.openstack.org/246226 (Merged)

Change-Id: Id4dfb6a39f41c0d3c9c28a82b5302b1239e1634d
Closes-Bug: #1539480
Closes-Bug: #1513425

Changed in fuel:
status: Confirmed → Fix Committed
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

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