[OpenStack-Infra] Freescale CI is allowed one comment per patchset - please acknowledge

trinath.somanchi at freescale.com trinath.somanchi at freescale.com
Fri Mar 7 05:07:36 UTC 2014


Hi Anita-

Thank you for the guidance. 

Will monitor the CI as per your comments.

Thanks a lot for helping me improve the CI.

--
Trinath Somanchi - B39208
trinath.somanchi at freescale.com | extn: 4048


-----Original Message-----
From: Anita Kuno [mailto:anteaya at anteaya.info] 
Sent: Thursday, March 06, 2014 8:17 PM
To: Somanchi Trinath-B39208; Michael Still
Cc: openstack-infra at lists.openstack.org; Mark McClain
Subject: Re: [OpenStack-Infra] Freescale CI is allowed one comment per patchset - please acknowledge

On 03/06/2014 01:29 AM, trinath.somanchi at freescale.com wrote:
> Hi Michael-
> 
> Thank you for the comment.
> 
> Do we need to improve our CI, further ..? Can post your suggestions/comments on the same.
> 
> 
> 
> --
> Trinath Somanchi - B39208
> trinath.somanchi at freescale.com | extn: 4048
> 
> 
> -----Original Message-----
> From: mikalstill at gmail.com [mailto:mikalstill at gmail.com] On Behalf Of 
> Michael Still
> Sent: Thursday, March 06, 2014 11:53 AM
> To: Anita Kuno
> Cc: Somanchi Trinath-B39208; openstack-infra at lists.openstack.org; Mark 
> McClain
> Subject: Re: [OpenStack-Infra] Freescale CI is allowed one comment per 
> patchset - please acknowledge
> 
> Anita,
> 
> here's a report of recent (last seven days) Freescale CI votes. It doesn't include the review revision that caused your initial concern, because that revision lasted less than the required three hours for my reporting. I hope it helps regardless.
> 
> http://www.rcbops.com/gerrit/reports/Freescale_CI-cireport.html
> 
> Cheers,
> Michael
Okay, let's go through this report:
Starting with this series of patches:

68142: patchset 17 - 1 comment, 1.5 hours ago
       patchset 16 - 2 comments, 2.5 hours ago
       patchset 15 - 2 comments, March 5

70700: patchset 4 - 2 comments, March 5

74156: patchset 18 - 4 comments, March 5,
       patchset 17 - 8 comments, March 5

75253: patchset 11 - 2 comments, March 5,
       patchset 10 - 2 comments, March 5,
       patchset 9 - 2 comments, March 5

75861: patchset 17 - 2 comments, March 5

76670: patchset 6 - 2 comments, March 5
       patchset 5 - 2 comments, March 5

I think that is enough to demonstrate that this statement is not a true reflection of the status of your system when I emailed you:
"Its that one patch which got many comments. Rest all patches got a single comment."

Please provide factual information about the status of your system in future.

Also as can be seen by the only single comment on a patchset of the patchsets I examined (patchset 68142, 17), it is possible that you are now commenting only once per patchset as of an hour or so ago.

Please continue to monitor the activity of your system and adjust it accordingly.

Thank you, Trianth,
Anita.


> 
> On Thu, Mar 6, 2014 at 1:21 PM, Anita Kuno <anteaya at anteaya.info> wrote:
>> On 03/05/2014 09:10 PM, trinath.somanchi at freescale.com wrote:
>>> Hi Anita-
>>>
>>> Its that one patch which got many comments. Rest all patches got a single comment.
>>>
>>> Let me check on what might have happened.
>>>
>>> Will review the system on this ..
>>>
>>> Thank you for the email.
>>>
>>> --
>>> Trinath
>> Thank you, Trianth.
>>
>> Please review other recent patches your system has reviewed as well.
>>
>> Please post your findings from your investigation into this patch.
>>
>> Thank you,
>> Anita.
>>>
>>> ________________________________________
>>> From: Anita Kuno <anteaya at anteaya.info>
>>> Sent: Thursday, March 06, 2014 3:59 AM
>>> To: Somanchi Trinath-B39208; openstack-infra at lists.openstack.org;
>>> Mark McClain
>>> Subject: Freescale CI is allowed one comment per patchset - please 
>>> acknowledge
>>>
>>> Hello Trinath:
>>>
>>> All 3rd party ci systems are allowed one comment per patchset.
>>>
>>> Right now Freescale CI is commenting more than once per patchset:
>>> https://review.openstack.org/#/c/73575/
>>>
>>> I count 8 Freescale CI comments on patchset 21 for patch 73575. This 
>>> creates noise for developers.
>>>
>>> Please post one comment per patchset (or recheck) which is posted 
>>> after your testing is completed with your test results, link to the 
>>> test logs and contact information for your system.
>>>
>>> If you cannot make this change immediately, please shut down 
>>> commenting on your system until you can make the change.
>>>
>>> Remember you can vote on the sandbox repo:
>>> http://git.openstack.org/cgit/openstack-dev/sandbox/ as much as you 
>>> want to ensure your system is configured in the way you are 
>>> requested. You can post links to your system comments in the sandbox 
>>> repo in the irc channel (#openstack-neutron on freenode) to ensure 
>>> your system is compliant with this request.
>>>
>>> Please contact me on irc if you are unclear in what you are being 
>>> asked to do. My irc nickname is anteaya.
>>>
>>> Thank you,
>>> Anita.
>>>
>>>
>>
>>
>> _______________________________________________
>> OpenStack-Infra mailing list
>> OpenStack-Infra at lists.openstack.org
>> http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-infra
> 
> 
> 
> --
> Rackspace Australia
> 
> 






More information about the OpenStack-Infra mailing list