Wednesday, 2014-08-20

openstackgerritlifeless proposed a change to stackforge/monasca-agent: Remove hardcoded version.  https://review.openstack.org/11546802:01
*** cindyo has quit IRC02:31
*** sandywalsh has quit IRC03:05
*** sandywalsh has joined #openstack-monasca03:06
*** garyh has quit IRC04:09
*** garyh has joined #openstack-monasca04:49
*** openstackgerrit has quit IRC10:10
*** tongli has joined #openstack-monasca12:42
*** cindyo has joined #openstack-monasca14:15
*** dschroeder has joined #openstack-monasca14:49
*** aviau has quit IRC14:53
*** rhochmuth has joined #openstack-monasca15:03
*** rhochmuth has left #openstack-monasca15:04
*** rhochmuth has joined #openstack-monasca15:04
*** aviau has joined #openstack-monasca15:08
*** openstackgerrit has joined #openstack-monasca15:12
cindyoproposing this change: https://review.openstack.org/11567615:31
cindyodon’t know why openstackgerrit didn’t notify the room15:31
cindyotongli: looking at openstack-infra room I see they removed the zuul comments.  this seems to be affecting our room notificaitons15:35
tonglihmmm. just this channel or all channels?15:36
cindyothey removed all comments.  not sure how you hooked in our room notifications15:36
tonglihmmm, we still have openstackstatus and openstackgerrit in this channel.15:37
tongliit should work.15:37
cindyomy last review didn’t get seen15:38
tongliposted review comments or submitted a patch set?15:38
cindyoa new review15:38
tongliI do not think review comments get on irc channel.15:38
cindyook, only commits to master?15:38
tongliI believe it is only the merge, patch submit.15:38
tongliyeah.15:39
tongliI do not think review comments get on the channel. that will be a lot of flood, I think.15:39
cindyotongli: the initial review only used to get posted, see the one from lifeless above, it has not been approved15:40
cindyoif we can live with just merges that’s ok, it means you might not get a chance to review it15:40
tonglihmmm, I do not actually know there is something there for review comments to be posted (notified ) in the channel.15:41
cindyoi guess if you want to be involved in the view you can sign up to be notified by your email15:42
cindyoin the review15:42
tonglimerges and patch set submission should be ok.15:42
tonglithat is right.15:42
tongliif you reveiw a patch set, then you will be receiving the emails when there is new review comments.15:42
openstackgerritA change was merged to stackforge/python-monascaclient: use keystone endpoint when api url is not input  https://review.openstack.org/11567615:52
*** aviau has quit IRC15:59
*** aviau has joined #openstack-monasca16:12
openstackgerritTim Kuhlman proposed a change to stackforge/monasca-notification: Move from using zookeeper to track partition offsets to kafka  https://review.openstack.org/11570516:49
*** rhochmuth has quit IRC17:11
*** ChanServ sets mode: +v openstackstatus18:07
*** cindyo has quit IRC18:12
*** cindyo has joined #openstack-monasca19:08
*** robbybb has joined #openstack-monasca19:37
openstackgerritA change was merged to stackforge/monasca-agent: Updating metric names to match new format  https://review.openstack.org/11510222:01
cindyosandywalsh: do you typically have a pipeline per unique rule set? The rule set example has trigger_name uuid, and request_id trigger def22:11
openstackgerritTim Kuhlman proposed a change to stackforge/monasca-persister: Add a tar.gz to the package step  https://review.openstack.org/11581322:54
*** dschroeder has quit IRC23:08
*** robbybb111 has joined #openstack-monasca23:45
openstackgerritlifeless proposed a change to stackforge/monasca-agent: Remove hardcoded version  https://review.openstack.org/11546823:47
*** robbybb has quit IRC23:50
*** openstackgerrit has quit IRC23:54
*** cindyo has quit IRC23:56
*** openstackgerrit has joined #openstack-monasca23:58

Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!