Fuel for OpenStack

[osci] re-implement comments to LP bugs

Bug #1439138 reported by Dmitry Mescheryakov
24
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Fuel for OpenStack
Fix Committed
Medium
Max Rasskazov
Fuel for OpenStack 6.1

Bug Description

Right now when somebody creates a new change request in review.fuel-infra.org or updates existing one, "OSCI Robot (oscirobot)" creates two comments in corresponding LP bug, for example:
https://bugs.launchpad.net/mos/+bug/1332058/comments/22
https://bugs.launchpad.net/mos/+bug/1332058/comments/23

We need current commenting to be removed and a new one implemented instead. The new commenting should leave at maximum two comments in LP bug per change request in Gerrit:
 * when change request is created
 * when change request is merged

Please use this bug as etalon:
https://bugs.launchpad.net/keystone/+bug/1407661

Specifically, that comment for new change request:
https://bugs.launchpad.net/keystone/+bug/1407661/comments/1

and that one for merged change request:
https://bugs.launchpad.net/keystone/+bug/1407661/comments/2

It would be nice if you copy the format fully.

Tags: osci
Max Rasskazov (mrasskazov)
Changed in fuel:
assignee: Fuel OSCI Team (fuel-osci) → Max Rasskazov (mrasskazov)
status: New → In Progress
Revision history for this message
Max Rasskazov (mrasskazov) wrote :

Partial fix proposed: https://review.fuel-infra.org/#/c/5161/

Revision history for this message
Max Rasskazov (mrasskazov) wrote :

https://review.fuel-infra.org/#/c/5728

Revision history for this message
Max Rasskazov (mrasskazov) wrote :

http://osci-jenkins.srt.mirantis.net:8080/job/gerrit_event_handler/

Revision history for this message
Fuel Devops McRobotson (fuel-devops-robot) wrote : Fix proposed to infra/obs (autobuild-master)

Fix proposed to branch: autobuild-master
Change author: Max Rasskazov <email address hidden>
Review: https://review.fuel-infra.org/5938

Revision history for this message
OSCI Robot (oscirobot) wrote :

Fix proposed to branch: autobuild-master
Review: https://review.fuel-infra.org/5938

Revision history for this message
Fuel Devops McRobotson (fuel-devops-robot) wrote : Fix proposed to infra/obs (feature/mox)

Fix proposed to branch: feature/mox
Change author: Max Rasskazov <email address hidden>
Review: https://review.fuel-infra.org/5943

Revision history for this message
OSCI Robot (oscirobot) wrote :

Fix proposed to branch: feature/mox
Review: https://review.fuel-infra.org/5943

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

Fix proposed to branch: fuel-infra-packages
Change author: Max Rasskazov <email address hidden>
Review: https://review.fuel-infra.org/5944

Revision history for this message
OSCI Robot (oscirobot) wrote :

Fix proposed to branch: fuel-infra-packages
Review: https://review.fuel-infra.org/5944

Revision history for this message
Fuel Devops McRobotson (fuel-devops-robot) wrote : Fix proposed to infra/obs (new-flow)

Fix proposed to branch: new-flow
Change author: Max Rasskazov <email address hidden>
Review: https://review.fuel-infra.org/5946

Revision history for this message
OSCI Robot (oscirobot) wrote :

Fix proposed to branch: new-flow
Review: https://review.fuel-infra.org/5946

Revision history for this message
Fuel Devops McRobotson (fuel-devops-robot) wrote : Fix proposed to infra/obs (stable/6.0)

Fix proposed to branch: stable/6.0
Change author: Max Rasskazov <email address hidden>
Review: https://review.fuel-infra.org/5947

Revision history for this message
OSCI Robot (oscirobot) wrote :

Fix proposed to branch: stable/6.0
Review: https://review.fuel-infra.org/5947

Revision history for this message
Fuel Devops McRobotson (fuel-devops-robot) wrote : Fix proposed to infra/obs (stable/6.0-updates)

Fix proposed to branch: stable/6.0-updates
Change author: Max Rasskazov <email address hidden>
Review: https://review.fuel-infra.org/5948

Revision history for this message
OSCI Robot (oscirobot) wrote :

Fix proposed to branch: stable/6.0-updates
Review: https://review.fuel-infra.org/5948

Revision history for this message
Fuel Devops McRobotson (fuel-devops-robot) wrote : Fix merged to infra/obs (stable/6.0-updates)

Reviewed: https://review.fuel-infra.org/5948
Submitter: Dmitry Burmistrov <email address hidden>
Branch: stable/6.0-updates

Commit: cc4e1b562305d55754d1cc423a19788c8b41d839
Author: Max Rasskazov <email address hidden>
Date: Fri Apr 17 13:38:46 2015

Commenting of LP bugs is disabled

Change-Id: Iabe3fd3dcb9e17831d6c11c3f3728bfa3e243930
Partial-Bug: #1439138

Revision history for this message
OSCI Robot (oscirobot) wrote :

Reviewed: https://review.fuel-infra.org/5948
Committed: https://review.fuel-infra.org/gitweb?p=infra/obs.git;a=commitdiff;h=cc4e1b562305d55754d1cc423a19788c8b41d839
Submitter: Dmitry Burmistrov
Branch: stable/6.0-updates

commit cc4e1b562305d55754d1cc423a19788c8b41d839
Author: \"Max Rasskazov\" <email address hidden>

Commenting of LP bugs is disabled

Change-Id: Iabe3fd3dcb9e17831d6c11c3f3728bfa3e243930
Partial-Bug: #1439138

Revision history for this message
Fuel Devops McRobotson (fuel-devops-robot) wrote : Fix merged to infra/obs (new-flow)

Reviewed: https://review.fuel-infra.org/5946
Submitter: Dmitry Burmistrov <email address hidden>
Branch: new-flow

Commit: c2f1c55a8a3ffedaf7c31d006e0ed1d0f8230f3c
Author: Max Rasskazov <email address hidden>
Date: Fri Apr 17 13:30:10 2015

Commenting of LP bugs is disabled

Change-Id: Iabe3fd3dcb9e17831d6c11c3f3728bfa3e243930
Partial-Bug: #1439138

Revision history for this message
OSCI Robot (oscirobot) wrote :

Reviewed: https://review.fuel-infra.org/5946
Committed: https://review.fuel-infra.org/gitweb?p=infra/obs.git;a=commitdiff;h=c2f1c55a8a3ffedaf7c31d006e0ed1d0f8230f3c
Submitter: Dmitry Burmistrov
Branch: new-flow

commit c2f1c55a8a3ffedaf7c31d006e0ed1d0f8230f3c
Author: \"Max Rasskazov\" <email address hidden>

Commenting of LP bugs is disabled

Change-Id: Iabe3fd3dcb9e17831d6c11c3f3728bfa3e243930
Partial-Bug: #1439138

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

Reviewed: https://review.fuel-infra.org/5944
Submitter: Dmitry Burmistrov <email address hidden>
Branch: fuel-infra-packages

Commit: dd84617f09597cb7c703c4380ab7b9c6495ea2f4
Author: Max Rasskazov <email address hidden>
Date: Fri Apr 17 13:27:29 2015

Commenting of LP bugs is disabled

Change-Id: Iabe3fd3dcb9e17831d6c11c3f3728bfa3e243930
Partial-Bug: #1439138

Revision history for this message
Fuel Devops McRobotson (fuel-devops-robot) wrote : Fix merged to infra/obs (feature/mox)

Reviewed: https://review.fuel-infra.org/5943
Submitter: Dmitry Burmistrov <email address hidden>
Branch: feature/mox

Commit: d9f01a492a7415941d00a51baa0be06f16ef498f
Author: Max Rasskazov <email address hidden>
Date: Fri Apr 17 13:25:31 2015

Commenting of LP bugs is disabled

Change-Id: Iabe3fd3dcb9e17831d6c11c3f3728bfa3e243930
Partial-Bug: #1439138

Revision history for this message
OSCI Robot (oscirobot) wrote : Fix merged to infra/obs (fuel-infra-packages)

Reviewed: https://review.fuel-infra.org/5944
Committed: https://review.fuel-infra.org/gitweb?p=infra/obs.git;a=commitdiff;h=dd84617f09597cb7c703c4380ab7b9c6495ea2f4
Submitter: Dmitry Burmistrov
Branch: fuel-infra-packages

commit dd84617f09597cb7c703c4380ab7b9c6495ea2f4
Author: \"Max Rasskazov\" <email address hidden>

Commenting of LP bugs is disabled

Change-Id: Iabe3fd3dcb9e17831d6c11c3f3728bfa3e243930
Partial-Bug: #1439138

Revision history for this message
OSCI Robot (oscirobot) wrote : Fix merged to infra/obs (feature/mox)

Reviewed: https://review.fuel-infra.org/5943
Committed: https://review.fuel-infra.org/gitweb?p=infra/obs.git;a=commitdiff;h=d9f01a492a7415941d00a51baa0be06f16ef498f
Submitter: Dmitry Burmistrov
Branch: feature/mox

commit d9f01a492a7415941d00a51baa0be06f16ef498f
Author: \"Max Rasskazov\" <email address hidden>

Commenting of LP bugs is disabled

Change-Id: Iabe3fd3dcb9e17831d6c11c3f3728bfa3e243930
Partial-Bug: #1439138

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

Reviewed: https://review.fuel-infra.org/5938
Submitter: Dmitry Burmistrov <email address hidden>
Branch: autobuild-master

Commit: 52a4053f948690d5515a127664250c6a1fcea157
Author: Max Rasskazov <email address hidden>
Date: Fri Apr 17 13:06:37 2015

Commenting of LP bugs is disabled

Change-Id: Iabe3fd3dcb9e17831d6c11c3f3728bfa3e243930
Partial-Bug: #1439138

Revision history for this message
OSCI Robot (oscirobot) wrote :

Reviewed: https://review.fuel-infra.org/5938
Committed: https://review.fuel-infra.org/gitweb?p=infra/obs.git;a=commitdiff;h=52a4053f948690d5515a127664250c6a1fcea157
Submitter: Dmitry Burmistrov
Branch: autobuild-master

commit 52a4053f948690d5515a127664250c6a1fcea157
Author: \"Max Rasskazov\" <email address hidden>

Commenting of LP bugs is disabled

Change-Id: Iabe3fd3dcb9e17831d6c11c3f3728bfa3e243930
Partial-Bug: #1439138

Revision history for this message
Fuel Devops McRobotson (fuel-devops-robot) wrote : Fix merged to infra/obs (stable/6.0)

Reviewed: https://review.fuel-infra.org/5947
Submitter: Dmitry Burmistrov <email address hidden>
Branch: stable/6.0

Commit: 64abd3ad1ab88fce4da2367cefd6cdd02b72d39d
Author: Max Rasskazov <email address hidden>
Date: Fri Apr 17 13:33:33 2015

Commenting of LP bugs is disabled

Change-Id: Iabe3fd3dcb9e17831d6c11c3f3728bfa3e243930
Partial-Bug: #1439138

Revision history for this message
OSCI Robot (oscirobot) wrote :

Reviewed: https://review.fuel-infra.org/5947
Committed: https://review.fuel-infra.org/gitweb?p=infra/obs.git;a=commitdiff;h=64abd3ad1ab88fce4da2367cefd6cdd02b72d39d
Submitter: Dmitry Burmistrov
Branch: stable/6.0

commit 64abd3ad1ab88fce4da2367cefd6cdd02b72d39d
Author: \"Max Rasskazov\" <email address hidden>

Commenting of LP bugs is disabled

Change-Id: Iabe3fd3dcb9e17831d6c11c3f3728bfa3e243930
Partial-Bug: #1439138

Max Rasskazov (mrasskazov)
Changed in fuel:
status: In Progress → Fix Committed
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

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