Monday, 2017-07-17

*** zhangguoqing has joined #openstack-telemetry00:29
*** tovin07_ has joined #openstack-telemetry00:33
*** thorst has joined #openstack-telemetry00:59
*** thorst has quit IRC01:04
*** zhurong has joined #openstack-telemetry01:20
openstackgerritwenbin luo proposed openstack/ceilometer master: Remove the explanation of nonexistent parameter  https://review.openstack.org/48422001:20
openstackgerritTovin Seven proposed openstack/python-pankoclient master: Update documentation URLs  https://review.openstack.org/48379301:21
*** zhurong has quit IRC01:23
*** zhangguoqing has quit IRC01:34
openstackgerritHangdong Zhang proposed openstack/aodh master: Update aodh commands in doc  https://review.openstack.org/48422201:36
*** zhangguoqing has joined #openstack-telemetry01:37
*** lhx__ has joined #openstack-telemetry02:04
*** tovin07__ has joined #openstack-telemetry02:04
*** tovin07_ has quit IRC02:06
*** zhurong has joined #openstack-telemetry02:31
*** zhangguoqing has quit IRC02:54
*** zhangguoqing has joined #openstack-telemetry02:56
*** thorst has joined #openstack-telemetry03:01
*** dave-mccowan has quit IRC03:04
*** thorst has quit IRC03:05
*** tovin07 has joined #openstack-telemetry03:11
*** tovin07__ has quit IRC03:13
*** zhangguoqing has quit IRC03:35
*** links has joined #openstack-telemetry03:44
*** hoonetorg has quit IRC04:05
*** psachin has joined #openstack-telemetry04:18
*** zhangguoqing has joined #openstack-telemetry04:26
*** lhx__ has quit IRC04:28
*** zhangguo_ has joined #openstack-telemetry04:43
*** zhangguoqing has quit IRC04:45
*** thorst has joined #openstack-telemetry05:01
*** zhurong has quit IRC05:04
*** thorst has quit IRC05:06
*** zhurong has joined #openstack-telemetry05:28
*** rcernin_ has joined #openstack-telemetry06:00
openstackgerritHangdong Zhang proposed openstack/ceilometer master: Docs: update the path of "meters.yaml" file, and add the content of the feature: multiple meter definition files  https://review.openstack.org/48425706:15
*** Gautam has joined #openstack-telemetry06:48
*** thorst has joined #openstack-telemetry07:02
*** shardy has joined #openstack-telemetry07:06
*** shardy has quit IRC07:07
*** thorst has quit IRC07:07
*** shardy has joined #openstack-telemetry07:09
*** toddnni has quit IRC07:18
*** toddnni has joined #openstack-telemetry07:25
openstackgerritMerged openstack/ceilometer master: cap cpu_util  https://review.openstack.org/47594307:29
*** tesseract has joined #openstack-telemetry07:32
openstackgerritMehdi Abaakouk (sileht) proposed openstack/ceilometer stable/newton: cap cpu_util  https://review.openstack.org/48427407:46
openstackgerritHanxi Liu proposed openstack/ceilometer master: Docs: switch to openstackdocstheme  https://review.openstack.org/47675607:49
*** lhx__ has joined #openstack-telemetry07:49
*** Gautam has quit IRC07:49
*** Gautam has joined #openstack-telemetry07:50
*** Gautam has quit IRC07:55
*** Gautam has joined #openstack-telemetry07:56
openstackgerritHanxi Liu proposed openstack/panko master: Docs: switch to openstackdocstheme  https://review.openstack.org/47675808:08
openstackgerritBéla Vancsics proposed openstack/ceilometer master: Reduced the complexity of the SingleColumnValueFilter method  https://review.openstack.org/48428208:22
*** thorst has joined #openstack-telemetry08:27
*** thorst has quit IRC08:32
openstackgerritsudhir agarwal proposed openstack/aodh master: Replace six.iteritems() with .items()  https://review.openstack.org/48429008:45
*** tesseract has quit IRC08:50
*** tesseract has joined #openstack-telemetry08:53
openstackgerritTovin Seven proposed openstack/python-aodhclient master: Add OSprofiler support for Aodh client  https://review.openstack.org/48429509:01
openstackgerritTovin Seven proposed openstack/python-pankoclient master: Add Osprofiler support for Panko client  https://review.openstack.org/48429409:01
*** dixiaoli has joined #openstack-telemetry09:07
*** dixiaoli has quit IRC09:07
*** lhx__ has quit IRC09:17
tovin07hi sileht09:17
tovin07do you think that there is something wrong with aodh gate?09:18
tovin07for example http://logs.openstack.org/40/483840/1/check/gate-python-aodhclient-python35/4adbe6b/console.html09:20
jd_sileht: your backport is failing for good reason :(09:24
jd_tovin07: looks like there's a problem indeed09:25
jd_tovin07: all patches are failing?09:25
tovin07i saw some recent patches09:25
tovin07even with `just documentation` patch like this http://logs.openstack.org/40/483840/1/check/gate-python-aodhclient-python27-ubuntu-xenial/c15a044/console.html09:27
openstackgerritJulien Danjou proposed openstack/ceilometer stable/newton: cap cpu_util  https://review.openstack.org/48427409:27
openstackgerritMehdi Abaakouk (sileht) proposed openstack/python-aodhclient master: Fix gnocchi/pifpaf requirements  https://review.openstack.org/48430509:30
silehtjd_, tovin07 ^09:30
silehtthat should fix it09:30
silehtat least the py27 job :p09:30
tovin07oh great :v09:31
openstackgerritMerged openstack/aodh master: move class api reference into contributor docs  https://review.openstack.org/48395109:46
openstackgerritMerged openstack/aodh master: add missing sphinx extension  https://review.openstack.org/48395209:46
*** tovin07 has quit IRC09:52
*** zhangguo_ has quit IRC10:02
*** lhx__ has joined #openstack-telemetry10:05
openstackgerritHangdong Zhang proposed openstack/ceilometer master: Update Doc links because of doc-migration  https://review.openstack.org/48431410:06
openstackgerritHangdong Zhang proposed openstack/ceilometer master: Docs: update the path of "meters.yaml" and its new feature  https://review.openstack.org/48425710:13
openstackgerritJulien Danjou proposed openstack/python-aodhclient master: install uwsgi for gnocchi  https://review.openstack.org/48395010:25
*** thorst has joined #openstack-telemetry10:28
openstackgerritMerged openstack/python-aodhclient master: Fix gnocchi/pifpaf requirements  https://review.openstack.org/48430510:31
*** thorst has quit IRC10:32
openstackgerritJulien Danjou proposed openstack/python-pankoclient master: Update documentation URLs  https://review.openstack.org/48379310:35
openstackgerritMerged openstack/python-pankoclient master: Update documentation URLs  https://review.openstack.org/48379310:49
*** thorst has joined #openstack-telemetry11:08
*** thorst has quit IRC11:09
*** thorst has joined #openstack-telemetry11:16
openstackgerritMerged openstack/ceilometer master: tempest: Use primary user token  https://review.openstack.org/45965911:19
*** hoonetorg has joined #openstack-telemetry11:32
openstackgerritsudhir agarwal proposed openstack/aodh master: Replace six.iteritems() with .items()  https://review.openstack.org/48429011:35
*** zhangguoqing has joined #openstack-telemetry11:39
openstackgerritMerged openstack/ceilometer master: Remove the explanation of nonexistent parameter  https://review.openstack.org/48422011:42
*** zhangguoqing has quit IRC11:44
*** lhx__ has quit IRC12:07
*** flg has joined #openstack-telemetry12:07
*** Gautam_ has joined #openstack-telemetry12:08
*** Gautam has quit IRC12:10
*** donghao has joined #openstack-telemetry12:12
*** Gautam_ has quit IRC12:12
*** Gautam has joined #openstack-telemetry12:13
*** Gautam has quit IRC12:23
openstackgerritJulien Danjou proposed openstack/ceilometer stable/newton: cap cpu_util  https://review.openstack.org/48427412:32
*** lhx_ has joined #openstack-telemetry12:32
*** efoley has joined #openstack-telemetry12:48
*** catintheroof has joined #openstack-telemetry12:51
*** Jack_Iv has joined #openstack-telemetry12:52
*** Gautam has joined #openstack-telemetry13:08
*** gordc has joined #openstack-telemetry13:11
*** zhurong has quit IRC13:22
*** joadavis has quit IRC13:23
*** joadavis has joined #openstack-telemetry13:24
*** joadavis has quit IRC13:32
*** gordc has quit IRC13:35
*** gordc has joined #openstack-telemetry13:35
*** gordc has quit IRC13:35
openstackgerritMerged openstack/aodh master: Update URL home-page in documents according to document migration  https://review.openstack.org/48397713:36
*** links has quit IRC13:37
openstackgerritMerged openstack/aodh master: turn on warning-is-error for doc builds  https://review.openstack.org/48395313:42
openstackgerritHanxi Liu proposed openstack/python-aodhclient master: Docs: switch to openstackdocstheme  https://review.openstack.org/47675913:46
*** Gautam has quit IRC13:48
*** Gautam has joined #openstack-telemetry13:48
*** Gautam has quit IRC13:53
*** Gautam has joined #openstack-telemetry13:54
*** flg has quit IRC13:58
*** zhurong has joined #openstack-telemetry13:59
*** zhurong has quit IRC13:59
*** chlong has joined #openstack-telemetry14:05
*** chlong has quit IRC14:10
*** joadavis has joined #openstack-telemetry14:11
*** Jack_Iv has quit IRC14:12
*** gordc has joined #openstack-telemetry14:28
openstackgerritMerged openstack/ceilometer stable/newton: cap cpu_util  https://review.openstack.org/48427414:31
*** Gautam has quit IRC14:32
*** joadavis has quit IRC14:34
silehtEmilienM, you lie to me !14:39
EmilienMsileht: lol14:39
*** joadavis has joined #openstack-telemetry14:40
EmilienMsileht: what did I do again14:40
silehtEmilienM, I'm waiting to next dolerean build for openstack-ceilometer14:40
EmilienMsileht: why?14:40
silehtEmilienM, you tell me 3 hour, and my change have been merged 4 hour ago14:40
EmilienMwhat14:40
silehtEmilienM, I'm looking here: https://trunk.rdoproject.org/centos7-master-head/report.html14:40
EmilienMlet me check14:40
EmilienMsileht: https://dashboards.rdoproject.org/rdo-dev14:41
silehtEmilienM, I want this in the rpm: https://review.openstack.org/#/c/459659/14:41
EmilienMlet me check the version, a sec14:41
EmilienMsileht: the repo used by puppet CI is here: https://trunk.rdoproject.org/centos7-master/puppet-passed-ci/14:41
EmilienMsileht: it seems like we have it !14:42
EmilienMsileht: I did recheck on https://review.openstack.org/#/c/410865 to double check that we have it, and we'll merge it today14:42
silehtEmilienM, good :)14:43
*** psachin has quit IRC14:48
*** rwsu has joined #openstack-telemetry15:06
*** pradk has joined #openstack-telemetry15:22
*** chlong has joined #openstack-telemetry15:36
*** chlong has quit IRC15:38
*** NostawRm has joined #openstack-telemetry15:43
*** chlong has joined #openstack-telemetry15:44
*** lhx_ has quit IRC15:46
EmilienMsileht: ok so I confirm we are deploying the package with the patch but tempest is now failing: http://logs.openstack.org/65/410865/81/check/gate-puppet-openstack-integration-4-scenario001-tempest-centos-7/ff9aaa2/console.html#_2017-07-17_15_37_38_32297215:55
silehtEmilienM, I will looks into that tomorrow, thx for pointing my the puppet yum repo, it will save me some times15:55
EmilienMsileht: http://logs.openstack.org/65/410865/81/check/gate-puppet-openstack-integration-4-scenario001-tempest-centos-7/ff9aaa2/logs/repolist.txt.gz15:57
EmilienMsileht: keep in mind we use AFS mirrors in our gate - these mirrors won't work for you locally15:57
*** hoonetorg has quit IRC15:57
EmilienMsileht: but you can find the URL suffix that we use from trunk.rdo15:58
*** ddyer has quit IRC16:02
*** ddyer has joined #openstack-telemetry16:09
*** rcernin_ has quit IRC16:19
NostawRmAm I missing something about why ceilometer won't get disk.usage for network devices when using libvirt? I haven't been able to find anything on it but 'virsh domblkinfo' works on the network drive for a ceph backed instance I'm testing with16:31
NostawRmI'm betting its going to be right in front of my face and I'm not seeing it.16:31
gordcNostawRm: what version of ceilometer you using?16:34
NostawRmwell, for checking to see where it should be pulling the disk usage from, I grabbed the master16:35
gordcNostawRm: we recently changed master so it doesn't poll everything. https://github.com/openstack/ceilometer/blob/0972df767d6bb7922f2aaf3f23514e7345f1c19b/etc/ceilometer/polling.yaml16:35
gordcyou'll need to change it to include disk.usage (if your polling.yaml looks like 6)16:36
gordcs/6/^/16:36
*** swamireddy has quit IRC16:40
*** hoonetorg has joined #openstack-telemetry16:40
NostawRmgordc, just found the one actually being used is 8.0.1, but what I was looking at is here at line 198, https://github.com/openstack/ceilometer/blob/8.0.1/ceilometer/compute/virt/libvirt/inspector.py , or 132 in master16:50
NostawRmright now we're polling *16:50
NostawRmand it doesn't even try to get the info with device type 'network' but 'virsh domblkinfo' does seem to get it, and there was a note somewhere that I'm having to find again that said it just required libvirt greater than a certain version16:52
NostawRmah, the note wasn't something necessarily relevant. it just says for 'disk.* support' but some of it is working, like  read/write bytes/requests16:54
*** hoonetorg has quit IRC16:54
gordchmm.. we do seem to be getting disk.usage in gate: http://logs.openstack.org/70/483670/1/check/gate-telemetry-dsvm-integration-ceilometer-ubuntu-xenial/3455046/logs/screen-ceilometer-acompute.txt.gz#_2017-07-17_15_19_12_14916:56
gordcmaybe add some logging to check if it's being triggered?16:57
*** hoonetorg has joined #openstack-telemetry17:03
*** chlong has quit IRC17:03
*** donghao has quit IRC17:05
NostawRmmeh in my logs I just have the 'Inspection disk usage of network disk {instance_id} unsupported by libvirt' warnings, but this was just for some end user graphing, not strictly necessary17:12
*** harlowja has joined #openstack-telemetry17:17
*** chlong has joined #openstack-telemetry17:19
*** hoonetorg has quit IRC17:34
*** efoley has quit IRC17:38
*** hoonetorg has joined #openstack-telemetry17:54
*** hoonetorg has quit IRC17:59
*** tesseract has quit IRC18:01
*** rcernin has joined #openstack-telemetry18:26
*** vishwanathj has quit IRC18:38
*** vishwanathj has joined #openstack-telemetry18:39
*** rwsu has quit IRC18:39
*** rwsu has joined #openstack-telemetry18:55
*** harlowja has quit IRC19:32
openstackgerritMehdi Abaakouk (sileht) proposed openstack/ceilometer stable/newton: use domainListGetStats to get cputime  https://review.openstack.org/48446019:35
openstackgerritMehdi Abaakouk (sileht) proposed openstack/ceilometer stable/newton: Use vcpu.x.time and vcpu.x.wait values in libvirt inspector  https://review.openstack.org/48446119:35
*** dave-mccowan has joined #openstack-telemetry19:44
*** boris-42__ has joined #openstack-telemetry19:47
boris-42__someone around?19:48
boris-42__jd_ hi there are you still involved with Ceilometer?)19:48
gordcboris-42__: i would just ask. or ask on ML.19:52
boris-42__gordc: hi there19:53
boris-42__gordc: http://logs.openstack.org/04/484304/5/check/gate-rally-dsvm-neutron-telemetry-ubuntu-xenial-nv/5a92ebc/rally-plot/results.html.gz#/CeilometerResource.get_tenant_resources/failures19:53
boris-42__so we started seeing this in our Rally gates19:53
boris-42__we didn't change Ceilometer plugins for a while19:55
gordcboris-42__: what is this test doing?19:55
gordcwhen did it start failing?19:56
gordcwe haven't touched ceilometer api for a long long time i want to say.19:56
boris-42__https://github.com/openstack/rally/blob/master/rally/plugins/openstack/scenarios/ceilometer/resources.py#L61-L7519:57
boris-42__so this is the scenario that fails19:57
*** shardy has quit IRC19:58
boris-42__https://github.com/openstack/rally/blob/master/rally/plugins/openstack/context/ceilometer/samples.py#L121-L15019:58
boris-42__this is happening before scenario is executed19:59
boris-42__this is the method https://github.com/openstack/rally/blob/master/rally/plugins/openstack/scenarios/ceilometer/utils.py#L2820:00
boris-42__that creates samples20:00
gordcyou know when it started to fail?20:03
boris-42__I can try to figure it out20:03
boris-42__http://logs.openstack.org/63/481963/6/check/gate-rally-dsvm-neutron-telemetry-ubuntu-xenial-nv/692f1cf/rally-plot/results.html.gz#/CeilometerResource.get_tenant_resources20:05
boris-42__so this was on Jul 1120:06
boris-42__half iterations passed20:06
boris-42__http://logs.openstack.org/23/482323/2/check/gate-rally-dsvm-neutron-telemetry-ubuntu-xenial-nv/a2eb729/rally-plot/results.html.gz#/CeilometerResource.get_tenant_resources here a bit before Jul 1020:07
boris-42__as well half passed20:07
gordcboris-42__: this is probably the only patch of signifcance last week https://review.openstack.org/#/c/47583820:07
boris-42__Jul 520:08
boris-42__https://review.openstack.org/#/c/469127/20:08
boris-42__passed everything20:08
gordci don't really see how that patch would impact your tests though.20:08
boris-42__Jul 7 failures20:08
boris-42__so in between Jul 5 and 720:09
boris-42__gordc: can i make revert for testing purpouse?20:09
boris-42__I'll put depends on20:09
boris-42__and we can see20:09
gordcif you want.20:10
gordcthe patch i mentioned is from July 11... not Jul5th though20:10
gordcwhat does your test do? just push a list of samples and do query on them?20:11
boris-42__so actually https://review.openstack.org/#/c/475535/20:11
boris-42__only one patch20:11
boris-42__that matches the conditions in ceilometer20:11
boris-42__heh that's really sad that you are not running Rally20:12
gordcwell you're testing a deprecated api...20:12
gordcbut yeah, feel free to test reverting that patch.20:12
gordcboris-42__: just asking again, your test is using ceilometer polling or it pushes samples via ceilometer-api?20:13
boris-42__https://github.com/openstack/rally/blob/master/rally/plugins/openstack/scenarios/ceilometer/utils.py#L41420:16
boris-42__@gordc this is regular ceilometer client20:16
boris-42__so I think it uses API20:16
*** dave-mccowan has quit IRC20:17
gordcweird. i don't think it should even touch https://review.openstack.org/#/c/475535/ then...20:17
gordc*shrugs* give it a try with revert.20:17
*** chlong has quit IRC20:27
*** chlong has joined #openstack-telemetry20:32
*** sridhar_ram has joined #openstack-telemetry20:33
vkmcgordc, hey! by default gnocchi is not initialized with resources defined here https://github.com/openstack/ceilometer/blob/master/ceilometer/gnocchi_client.py20:37
vkmcright?20:37
vkmcwhat are the next steps?20:37
gordcvkmc: run ceilometer-upgrade20:41
vkmcgordc, I did that20:41
gordcvkmc: then it should have created the resources20:42
gordcin gnocchi20:42
gordcwhat is the output from gnocchi-upgrade?20:42
gordcerrr. ceilometer-upgrade20:42
*** vishwana_ has joined #openstack-telemetry20:43
vkmchmm so I see the list of resource-type list20:44
vkmcsomething else to add to docs maybe20:44
vkmcI assume that now if notifications are triggered then I should see those in gnocchi metric list20:45
gordchopefully20:45
gordcyou mean manila meters?20:46
vkmcyes20:46
*** vishwanathj has quit IRC20:46
gordckk, yeah, i haven't actually verified myself (i don't have manila in my environment)20:46
*** vishwana_ has quit IRC20:50
vkmcyeah, it's a bit odd.. cannot actually see them showing up20:53
vkmcthis is in the latest deployment20:53
gordcmaybe try adding logging20:53
gordci imagine you could add log here to see you actually receive event from manila: https://github.com/openstack/ceilometer/blob/master/ceilometer/meter/notifications.py#L22820:54
*** vint_bra has joined #openstack-telemetry20:54
vkmck k20:55
vkmccool20:55
vkmcwe are missing this patch set, probably doesn't have anything to do but it's missing a +220:55
vkmchttps://review.openstack.org/#/c/478907/20:55
*** harlowja has joined #openstack-telemetry20:56
gordcthat's independent of metrics.20:58
gordci'll probably just +A a bunch of patches soon if jd_ or sileht don't review.20:58
vkmck21:02
vkmcthanks for looking into it :)21:03
*** thorst has quit IRC21:10
*** gordc has quit IRC21:24
*** chlong has quit IRC21:26
*** rcernin has quit IRC21:33
*** dave-mccowan has joined #openstack-telemetry21:34
openstackgerritMerged openstack/python-aodhclient master: Update documentation URLs  https://review.openstack.org/48384021:43
*** thorst has joined #openstack-telemetry22:10
*** vint_bra has quit IRC22:13
*** thorst has quit IRC22:15
*** pradk has quit IRC22:30
openstackgerritMerged openstack/ceilometer master: Add event definition for manila  https://review.openstack.org/47890722:34
*** rwsu has quit IRC22:34
*** rwsu has joined #openstack-telemetry22:36
*** thorst has joined #openstack-telemetry22:41
*** thorst has quit IRC22:43
*** catintheroof has quit IRC23:07
*** chlong has joined #openstack-telemetry23:24

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