Thursday, 2017-08-03

*** http_GK1wmSU has joined #openstack-telemetry00:17
*** http_GK1wmSU has left #openstack-telemetry00:17
*** rwsu has quit IRC00:38
*** zhurong has joined #openstack-telemetry00:46
*** ddyer has quit IRC00:53
*** ddyer has joined #openstack-telemetry00:57
*** dixiaoli has joined #openstack-telemetry00:58
*** gongysh has joined #openstack-telemetry01:30
*** thorst_afk has quit IRC01:37
*** lhx__ has joined #openstack-telemetry01:42
*** joadavis has quit IRC01:43
*** abhi89 has joined #openstack-telemetry01:50
openstackgerritliuwei proposed openstack/ceilometer master: Add meters for libvirt: disk.device.read.latency, disk.device.write.latency  https://review.openstack.org/48991001:51
openstackgerritxianbin xie proposed openstack/ceilometer master: fix cpu_util precision is too precise  https://review.openstack.org/49027801:58
*** abhi89 has quit IRC01:59
*** joadavis has joined #openstack-telemetry02:48
*** dikonoor has joined #openstack-telemetry02:54
*** abhi89 has joined #openstack-telemetry03:10
*** masuberu has joined #openstack-telemetry03:18
*** masber has quit IRC03:22
*** dikonoo has joined #openstack-telemetry03:24
*** dikonoor has quit IRC03:28
*** dixiaoli has quit IRC03:28
*** thorst_afk has joined #openstack-telemetry03:38
*** thorst_afk has quit IRC03:52
*** links has joined #openstack-telemetry03:54
*** gongysh has quit IRC04:15
*** dikonoo has quit IRC04:57
*** dikonoo has joined #openstack-telemetry05:12
openstackgerritxianbin xie proposed openstack/ceilometer master: fix cpu_util precision is too precise  https://review.openstack.org/49027805:42
*** lhx__ has quit IRC05:46
*** thorst_afk has joined #openstack-telemetry05:49
*** psachin has joined #openstack-telemetry05:51
*** dims has quit IRC05:52
*** lhx__ has joined #openstack-telemetry05:53
*** thorst_afk has quit IRC05:53
*** dims has joined #openstack-telemetry05:54
*** larainema has quit IRC05:56
*** tovin07 has quit IRC06:02
*** dikonoo has quit IRC06:02
*** hoonetorg has quit IRC06:12
*** dikonoor has joined #openstack-telemetry06:14
*** rcernin has joined #openstack-telemetry06:19
*** hoonetorg has joined #openstack-telemetry06:25
*** pcaruana has joined #openstack-telemetry06:48
*** tesseract has joined #openstack-telemetry07:17
*** thorst_afk has joined #openstack-telemetry07:22
*** yprokule has joined #openstack-telemetry07:25
*** thorst_afk has quit IRC07:27
openstackgerritDebo Zhang proposed openstack/ceilometer master: Update API link in README  https://review.openstack.org/49037107:39
*** tovin07_ has joined #openstack-telemetry07:45
*** larainema has joined #openstack-telemetry07:54
*** efoley has joined #openstack-telemetry08:02
*** sileht has joined #openstack-telemetry08:12
*** openstackgerrit has quit IRC08:18
*** flg has joined #openstack-telemetry08:43
*** flg_ has joined #openstack-telemetry08:44
*** dims has quit IRC09:04
*** dims has joined #openstack-telemetry09:08
*** thorst_afk has joined #openstack-telemetry09:23
*** dikonoor has quit IRC09:25
*** thorst_afk has quit IRC09:28
*** dikonoor has joined #openstack-telemetry09:52
*** tovin07_ has quit IRC10:09
*** dikonoor has quit IRC10:25
*** zhurong has quit IRC10:38
*** abhi89 has quit IRC10:41
*** dikonoor has joined #openstack-telemetry10:55
*** dixiaoli has joined #openstack-telemetry10:59
*** dixiaoli has quit IRC11:04
*** openstackgerrit has joined #openstack-telemetry11:04
openstackgerritOpenStack Proposal Bot proposed openstack/aodh master: Imported Translations from Zanata  https://review.openstack.org/48472011:04
*** dikonoor has quit IRC11:53
*** thorst_afk has joined #openstack-telemetry12:03
*** catintheroof has joined #openstack-telemetry12:36
*** zhurong has joined #openstack-telemetry12:49
*** gordc has joined #openstack-telemetry12:50
*** abhi89 has joined #openstack-telemetry13:06
*** links has quit IRC13:06
*** lhx__ has quit IRC13:08
*** psachin has quit IRC13:13
*** dikonoor has joined #openstack-telemetry13:22
*** psachin has joined #openstack-telemetry13:24
*** lhx_ has joined #openstack-telemetry13:30
*** dikonoor has quit IRC13:34
*** dikonoor has joined #openstack-telemetry13:35
openstackgerritMerged openstack/ceilometer master: Update and replace http with https for doc links in ceilometer  https://review.openstack.org/48832413:59
*** dikonoor has quit IRC14:03
*** zhurong has quit IRC14:17
openstackgerritMerged openstack/ceilometer master: vmware: ignore -1 values  https://review.openstack.org/48265914:19
*** thorst_afk has quit IRC14:30
*** thorst_afk has joined #openstack-telemetry14:32
*** thorst_afk has quit IRC14:37
*** thorst_afk has joined #openstack-telemetry14:42
*** thorst_afk has quit IRC15:00
*** thorst_afk has joined #openstack-telemetry15:01
*** yprokule has quit IRC15:20
akrzosgordc: yeah to change 3.0 scheduling to hash based looks a like a lot of work15:35
akrzosneed to access coordination driver to know how many members15:35
akrzosand not even sure if it would apply to the other drivers then15:36
akrzosdo you think it's worth it?15:36
*** psachin has quit IRC16:01
*** thorst_afk has quit IRC16:08
*** rcernin has quit IRC16:12
*** dave-mccowan has joined #openstack-telemetry16:14
*** thorst_afk has joined #openstack-telemetry16:15
gordcakrzos: sorry, was afk.16:16
gordcpersonally, don't think it's worth it. especially since we have v4 which address this stuff already.16:16
*** jmlowe has quit IRC16:20
*** vishwana_ has joined #openstack-telemetry16:20
*** vishwanathj has quit IRC16:24
*** lhx__ has joined #openstack-telemetry16:40
*** donghao has joined #openstack-telemetry16:41
*** jmlowe has joined #openstack-telemetry16:41
*** lhx_ has quit IRC16:43
*** efoley has quit IRC16:49
*** pcaruana has quit IRC16:54
*** rcernin has joined #openstack-telemetry17:02
*** donghao has quit IRC17:24
akrzosgordc: got it so you think generall with the current sort all, then basiclly throw away thats good enough for that patch?17:28
gordcakrzos: i guess that's the most inline with what exists.17:33
gordcakrzos: i think you'll still have that memory issue though.17:33
akrzosyeah i can share what the memory looks like under stress17:34
gordci think it's the fact that 3.0 pulls in entire list that's causing memory issue. which is why we stopped it in 3..117:34
akrzosthat was a previous issue though?17:34
gordcakrzos: let me dig. it was jmlowe who ran into it.17:34
gordc(i think(17:35
gordc)17:35
gordci will say, by sorting, it does make it less likely we'll have the backlog grow as big.17:35
jmlowehuh what?17:35
gordci'm naming names17:35
akrzoslet me grab a quick screen scrape to show you17:36
jmloweentirely possible, I tend to break things17:36
gordcakrzos: this was discussion: http://eavesdrop.openstack.org/irclogs/%23openstack-telemetry/%23openstack-telemetry.2017-01-27.log.html#t2017-01-27T16:57:4217:38
akrzoshttps://snapshot.raintank.io/dashboard/snapshot/SgKcct5xym8a4ZeSY9LFYVO8CuPKdWFC17:39
gordci don't think we have the original issue. it was a timeout issue.17:39
akrzosso let me explain that graph17:39
akrzosi have the sorting patch on 8/2 180017:40
akrzosi switched back to the old method17:40
akrzos8/3 11ish17:40
akrzosbasicly when you see the measures grow17:40
akrzosturn it back on17:40
akrzosand boom backlog drops to a steady state17:40
gordcyeah, it's because some of the metrics are unprocessed because we only chunk from front of alphanumerically sorted result17:41
akrzosyes17:41
gordcso i guess you're not hitting the timeout that https://github.com/gnocchixyz/gnocchi/commit/cd8d85e9d5505667802f8a17db5fa361521714a4 addresses17:42
akrzosso sorting prevents that unbounded growth on the untouched metrics due to how the work is distributed17:42
akrzosnow i'd love to argue lets just change how the keys are named but you mentioned basiclly how data is pulled you need the metric uuid first17:43
akrzosso changing names is not an option17:43
akrzosi'm gonna push that to a whole lot more instances17:44
akrzosbut it's limited in size17:44
akrzosi migth be able to get some time on a bigger cluster to prove this patch out17:44
akrzosbut basiclly your issues then become how slow ceilometer collector is on getting the data into the backlog17:44
gordcakrzos: well i know your patch works. because that's what we tried to fix in v417:44
akrzoseven after we fixed that for batching17:44
gordcmy main concern was that if you have very long processing_delay value, you'll run into a scenario where you're sorting millions of records potentially17:45
akrzosyeah i can see that scenario17:46
akrzostechnically if you have several million keys in that omap your probably in a world of hurt already17:47
akrzoswhich is why gnocchi v4 is better17:47
akrzosmultiple omaps17:47
gordcyeah. true.17:47
gordcyou'll probably need to apply your patch against 3.1 as well.17:48
gordcbut yeah, you've convinced me with "several million keys in that omap your probably in a world of hurt already"17:49
*** tesseract has quit IRC17:49
akrzosi can investigate memory a bit more17:50
akrzosmaybe there is a different use case with how gnocchi would be setup that i'm not aware of that would allow a much larger backlog?17:51
gordcall my benchmarks have metricd aggregating constantly just to test max throughput17:52
akrzosmy other thought is letting a backlog grow a bit, and processing is more samples processed per interval for each metrics which is probably more efficent than just processing the latest sample17:53
gordcbut i imagine most deployments will have more relaxed processing_delay17:53
akrzosbut at the expense of delay17:54
gordcakrzos: like modifying the TASKS_PER_WORKER param?17:54
gordcbiab. going to grab lunch.17:56
akrzosok no worries17:56
akrzosmy opinion is we need a two fold approach17:56
akrzosexpose TASKS_PER_WORKER and essentially cap backlog growth by sorting17:56
*** lhx__ has quit IRC18:17
*** vint_bra1 has joined #openstack-telemetry18:20
gordcyeah, both could work. but we also have v4.x :p18:20
openstackgerritSamuel Pilla proposed openstack/ceilometer master: Add bandit to ceilometer tests  https://review.openstack.org/49062118:30
*** abhi89 has quit IRC18:32
*** masber has joined #openstack-telemetry18:57
*** masuberu has quit IRC18:59
*** vint_bra1 has quit IRC19:17
*** vint_bra has joined #openstack-telemetry19:18
*** harlowja has quit IRC19:57
*** flg_ has quit IRC20:39
*** flg has quit IRC20:39
*** thorst_afk has quit IRC21:10
*** thorst_afk has joined #openstack-telemetry21:12
*** joadavis has quit IRC21:12
*** harlowja has joined #openstack-telemetry21:15
*** thorst_afk has quit IRC21:17
*** jmlowe has quit IRC21:18
*** efoley has joined #openstack-telemetry21:20
*** joadavis has joined #openstack-telemetry21:26
*** thorst_afk has joined #openstack-telemetry21:30
*** thorst_afk has quit IRC21:34
*** thorst_afk has joined #openstack-telemetry21:50
*** thorst_afk has quit IRC21:55
*** gordc has quit IRC22:05
*** vint_bra has quit IRC22:37
*** dave-mccowan has quit IRC22:51
*** rcernin has quit IRC22:58
*** thorst_afk has joined #openstack-telemetry23:26
*** jmlowe has joined #openstack-telemetry23:27
*** thorst_afk has quit IRC23:32

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