*** zhurong has quit IRC | 00:07 | |
*** thorst has joined #openstack-telemetry | 00:29 | |
*** thorst has quit IRC | 00:37 | |
*** vint_bra has joined #openstack-telemetry | 00:41 | |
*** vint_bra has quit IRC | 00:41 | |
*** vint_bra has joined #openstack-telemetry | 00:50 | |
*** cheneydc has joined #openstack-telemetry | 00:55 | |
*** cheneydc has quit IRC | 00:57 | |
*** zhurong has joined #openstack-telemetry | 01:09 | |
*** thorst has joined #openstack-telemetry | 01:14 | |
*** thorst has quit IRC | 01:15 | |
*** thorst has joined #openstack-telemetry | 01:16 | |
openstackgerrit | Ha Van Tu proposed openstack/ceilometer: Refactor Ceilometer resource API https://review.openstack.org/372190 | 01:17 |
---|---|---|
*** thorst has quit IRC | 01:24 | |
*** xiaozhuangqing has joined #openstack-telemetry | 01:36 | |
*** hongbin has joined #openstack-telemetry | 01:40 | |
*** hongbin has left #openstack-telemetry | 01:40 | |
*** zhurong has quit IRC | 01:43 | |
*** zhurong has joined #openstack-telemetry | 01:44 | |
*** xiaozhuangqing has quit IRC | 01:53 | |
*** xiaozhuangqing has joined #openstack-telemetry | 02:11 | |
*** thorst has joined #openstack-telemetry | 02:22 | |
*** fguillot is now known as fguillot_afk | 02:24 | |
*** fguillot_afk has quit IRC | 02:26 | |
*** thorst has quit IRC | 02:29 | |
*** zhurong has quit IRC | 02:39 | |
*** liuhanxi has joined #openstack-telemetry | 02:41 | |
*** zhurong has joined #openstack-telemetry | 02:43 | |
*** xiaozhuangqing_ has joined #openstack-telemetry | 02:48 | |
*** xiaozhuangqing has quit IRC | 02:48 | |
*** xiaozhuangqing_ is now known as xiaozhuangqing | 02:48 | |
*** vint_bra has quit IRC | 02:57 | |
openstackgerrit | liusheng proposed openstack/ceilometer: Clean the deprecated non-mitric related code https://review.openstack.org/345837 | 03:09 |
*** liusheng has joined #openstack-telemetry | 03:18 | |
*** thorst has joined #openstack-telemetry | 03:26 | |
*** thorst has quit IRC | 03:34 | |
*** hongbin has joined #openstack-telemetry | 04:04 | |
*** jwcroppe has joined #openstack-telemetry | 04:16 | |
*** thorst has joined #openstack-telemetry | 04:32 | |
*** davidlenwell has quit IRC | 04:37 | |
*** thorst has quit IRC | 04:39 | |
*** liuhanxi has quit IRC | 04:47 | |
*** davidlenwell has joined #openstack-telemetry | 04:49 | |
*** pcaruana has quit IRC | 05:05 | |
*** thorst has joined #openstack-telemetry | 05:37 | |
*** ljxiash has joined #openstack-telemetry | 05:41 | |
*** liamji has joined #openstack-telemetry | 05:42 | |
*** thorst has quit IRC | 05:45 | |
*** liuhanxi has joined #openstack-telemetry | 05:57 | |
*** rcernin has joined #openstack-telemetry | 06:03 | |
*** pcaruana has joined #openstack-telemetry | 06:17 | |
*** yprokule has joined #openstack-telemetry | 06:28 | |
openstackgerrit | liusheng proposed openstack/ceilometer: Clean the deprecated non-mitric related code https://review.openstack.org/345837 | 06:38 |
*** adriant has quit IRC | 06:38 | |
*** thorst has joined #openstack-telemetry | 06:42 | |
*** sorrison has joined #openstack-telemetry | 06:44 | |
*** thorst has quit IRC | 06:49 | |
*** jwcroppe has quit IRC | 07:21 | |
*** jwcroppe has joined #openstack-telemetry | 07:41 | |
*** jwcroppe_ has joined #openstack-telemetry | 07:44 | |
*** jwcroppe has quit IRC | 07:46 | |
*** thorst has joined #openstack-telemetry | 07:47 | |
*** amoralej|off is now known as amoralej | 07:51 | |
*** thorst has quit IRC | 07:55 | |
*** kong has quit IRC | 08:01 | |
*** shardy has joined #openstack-telemetry | 08:04 | |
openstackgerrit | Julien Danjou proposed openstack/gnocchi: devstack: rename werkzeug to simple https://review.openstack.org/372294 | 08:09 |
*** yassine has joined #openstack-telemetry | 08:11 | |
*** yassine is now known as Guest63247 | 08:11 | |
*** tasdomas has quit IRC | 08:14 | |
*** tasdomas has joined #openstack-telemetry | 08:18 | |
*** yprokule_ has joined #openstack-telemetry | 08:19 | |
*** yprokule has quit IRC | 08:20 | |
*** yprokule_ is now known as yprokule | 08:20 | |
*** jwcroppe_ has quit IRC | 08:24 | |
*** efoley has joined #openstack-telemetry | 08:27 | |
*** efoley_ has joined #openstack-telemetry | 08:29 | |
*** efoley has quit IRC | 08:32 | |
*** jwcroppe has joined #openstack-telemetry | 08:32 | |
openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/gnocchi: ceph: make computed measures read async https://review.openstack.org/370898 | 08:40 |
openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/gnocchi: ceph: make computed measures write async https://review.openstack.org/370897 | 08:40 |
openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/gnocchi: ceph: move out of xattr completely https://review.openstack.org/370896 | 08:40 |
openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/gnocchi: Don't mock ceph https://review.openstack.org/371301 | 08:40 |
openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/gnocchi: track the metric locked time https://review.openstack.org/371300 | 08:40 |
openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/gnocchi: Make some gabbi tests aware of ceph https://review.openstack.org/371302 | 08:40 |
openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/gnocchi: Add simple upgrade tests https://review.openstack.org/368412 | 08:40 |
openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/gnocchi: ceph: rename optional extra names https://review.openstack.org/372308 | 08:40 |
openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/gnocchi: Fix postgresql migration script https://review.openstack.org/372309 | 08:40 |
openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/gnocchi: ceph: make computed measures read async https://review.openstack.org/370898 | 08:47 |
openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/gnocchi: ceph: make computed measures write async https://review.openstack.org/370897 | 08:47 |
openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/gnocchi: ceph: rename optional extra names https://review.openstack.org/372308 | 08:47 |
openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/gnocchi: ceph: move out of xattr completely https://review.openstack.org/370896 | 08:47 |
openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/gnocchi: Fix postgresql migration script https://review.openstack.org/372309 | 08:47 |
openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/gnocchi: Don't mock ceph https://review.openstack.org/371301 | 08:47 |
openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/gnocchi: track the metric locked time https://review.openstack.org/371300 | 08:47 |
openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/gnocchi: Make some gabbi tests aware of ceph https://review.openstack.org/371302 | 08:47 |
openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/gnocchi: Add simple upgrade tests https://review.openstack.org/368412 | 08:47 |
*** thorst has joined #openstack-telemetry | 08:51 | |
openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/gnocchi: ceph: make computed measures read async https://review.openstack.org/370898 | 08:55 |
openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/gnocchi: ceph: make computed measures write async https://review.openstack.org/370897 | 08:55 |
*** thorst has quit IRC | 08:58 | |
openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/gnocchi: ceph: make computed measures read async https://review.openstack.org/370898 | 09:02 |
openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/gnocchi: ceph: make computed measures write async https://review.openstack.org/370897 | 09:02 |
openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/gnocchi: ceph: move out of xattr completely https://review.openstack.org/370896 | 09:02 |
openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/gnocchi: Fix postgresql migration script https://review.openstack.org/372309 | 09:02 |
openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/gnocchi: Don't mock ceph https://review.openstack.org/371301 | 09:02 |
openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/gnocchi: track the metric locked time https://review.openstack.org/371300 | 09:02 |
openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/gnocchi: Make some gabbi tests aware of ceph https://review.openstack.org/371302 | 09:02 |
openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/gnocchi: Add simple upgrade tests https://review.openstack.org/368412 | 09:02 |
*** liuhanxi has quit IRC | 09:03 | |
openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/gnocchi: ceph: make computed measures read async https://review.openstack.org/370898 | 09:09 |
openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/gnocchi: ceph: make computed measures write async https://review.openstack.org/370897 | 09:09 |
openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/gnocchi: ceph: move out of xattr completely https://review.openstack.org/370896 | 09:09 |
openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/gnocchi: Fix postgresql migration script https://review.openstack.org/372309 | 09:09 |
openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/gnocchi: Don't mock ceph https://review.openstack.org/371301 | 09:09 |
openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/gnocchi: track the metric locked time https://review.openstack.org/371300 | 09:09 |
openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/gnocchi: Make some gabbi tests aware of ceph https://review.openstack.org/371302 | 09:09 |
openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/gnocchi: Add simple upgrade tests https://review.openstack.org/368412 | 09:09 |
*** jwcroppe has quit IRC | 09:16 | |
*** jwcroppe has joined #openstack-telemetry | 09:23 | |
*** yprokule_ has joined #openstack-telemetry | 09:30 | |
*** yprokule has quit IRC | 09:31 | |
*** yprokule_ is now known as yprokule | 09:31 | |
*** kong has joined #openstack-telemetry | 09:49 | |
*** donghao has joined #openstack-telemetry | 09:50 | |
*** hoonetorg has quit IRC | 09:56 | |
*** hoonetorg has joined #openstack-telemetry | 09:57 | |
*** zhurong has quit IRC | 10:00 | |
*** Guest63247 has quit IRC | 10:00 | |
*** ljxiash has quit IRC | 10:02 | |
*** ljxiash has joined #openstack-telemetry | 10:03 | |
*** ljxiash has quit IRC | 10:07 | |
*** thorst has joined #openstack-telemetry | 10:08 | |
openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/gnocchi: ceph: make computed measures read async https://review.openstack.org/370898 | 10:09 |
openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/gnocchi: ceph: make computed measures write async https://review.openstack.org/370897 | 10:09 |
openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/gnocchi: ceph: rename optional extra names https://review.openstack.org/372308 | 10:09 |
openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/gnocchi: ceph: move out of xattr completely https://review.openstack.org/370896 | 10:09 |
openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/gnocchi: Fix postgresql migration script https://review.openstack.org/372309 | 10:09 |
openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/gnocchi: Don't mock ceph https://review.openstack.org/371301 | 10:09 |
openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/gnocchi: track the metric locked time https://review.openstack.org/371300 | 10:09 |
openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/gnocchi: Make some gabbi tests aware of ceph https://review.openstack.org/371302 | 10:09 |
openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/gnocchi: Add simple upgrade tests https://review.openstack.org/368412 | 10:09 |
*** jwcroppe has quit IRC | 10:10 | |
*** jwcroppe has joined #openstack-telemetry | 10:14 | |
*** cdent has joined #openstack-telemetry | 10:18 | |
*** andreykurilin has joined #openstack-telemetry | 10:19 | |
*** openstackstatus has quit IRC | 10:28 | |
openstackgerrit | Julien Danjou proposed openstack/gnocchi: Fix PostgreSQL migration script with resource_type_state_enum https://review.openstack.org/372309 | 10:32 |
*** thorst has quit IRC | 10:35 | |
*** liuhanxi has joined #openstack-telemetry | 10:41 | |
*** yassine has joined #openstack-telemetry | 10:54 | |
*** yassine is now known as Guest22675 | 10:54 | |
*** ljxiash has joined #openstack-telemetry | 10:55 | |
*** jwcroppe has quit IRC | 10:55 | |
*** ljxiash has quit IRC | 11:00 | |
*** yarkot1 has quit IRC | 11:00 | |
*** jwcroppe has joined #openstack-telemetry | 11:02 | |
*** smurke has quit IRC | 11:04 | |
*** mpotdar has quit IRC | 11:04 | |
*** yarkot has quit IRC | 11:04 | |
*** pshedimb has quit IRC | 11:04 | |
*** mpotdar has joined #openstack-telemetry | 11:10 | |
*** dhague has joined #openstack-telemetry | 11:15 | |
*** gzhai has joined #openstack-telemetry | 11:16 | |
*** yarkot has joined #openstack-telemetry | 11:20 | |
*** yarkot1 has joined #openstack-telemetry | 11:20 | |
*** hoonetorg has quit IRC | 11:23 | |
*** gzhai1 has joined #openstack-telemetry | 11:28 | |
*** gzhai has quit IRC | 11:29 | |
*** cdent has quit IRC | 11:33 | |
*** gordc has joined #openstack-telemetry | 11:39 | |
*** jwcroppe has quit IRC | 11:41 | |
*** thorst has joined #openstack-telemetry | 11:42 | |
*** ljxiash has joined #openstack-telemetry | 11:45 | |
*** thorst has quit IRC | 11:46 | |
*** jwcroppe has joined #openstack-telemetry | 11:49 | |
*** hoonetorg has joined #openstack-telemetry | 11:56 | |
*** cdent has joined #openstack-telemetry | 11:56 | |
*** hoonetorg has quit IRC | 11:57 | |
*** hoonetorg has joined #openstack-telemetry | 11:58 | |
*** shardy is now known as shardy_lunch | 12:01 | |
openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/gnocchi: ceph: make computed measures read async https://review.openstack.org/370898 | 12:02 |
openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/gnocchi: ceph: make computed measures write async https://review.openstack.org/370897 | 12:02 |
openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/gnocchi: ceph: move out of xattr completely https://review.openstack.org/370896 | 12:02 |
*** liamji has quit IRC | 12:02 | |
*** amoralej is now known as amoralej|lunch | 12:03 | |
openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/gnocchi: ceph: make computed measures read async https://review.openstack.org/370898 | 12:06 |
openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/gnocchi: ceph: make computed measures write async https://review.openstack.org/370897 | 12:06 |
openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/gnocchi: ceph: move out of xattr completely https://review.openstack.org/370896 | 12:06 |
openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/gnocchi: Don't mock ceph https://review.openstack.org/371301 | 12:06 |
*** jwcroppe has quit IRC | 12:07 | |
*** jwcroppe has joined #openstack-telemetry | 12:08 | |
openstackgerrit | Merged openstack/gnocchi: devstack: rename werkzeug to simple https://review.openstack.org/372294 | 12:11 |
*** vint_bra has joined #openstack-telemetry | 12:22 | |
gzhai1 | gordc:hi | 12:24 |
*** jwcroppe has quit IRC | 12:26 | |
gordc | gzhai1: hi, whatsup? | 12:26 |
gzhai1 | wanna discuss event-alarm timeout patch. | 12:27 |
gzhai1 | https://review.openstack.org/#/c/272028/5 | 12:27 |
gordc | sure | 12:27 |
gzhai1 | gordc: you suggest use separated queue for 'alarm.timeout.*' event. | 12:28 |
gzhai1 | but multiple queue introduce locking issue. | 12:28 |
gzhai1 | say one queue for normal event, another queue for 'alarm.timeout.end' | 12:29 |
gzhai1 | if required event comes in queue 1, and alarm.timeout.end in queue2, both wanna operate same alarm. | 12:29 |
gzhai1 | one want set it to "ALARM", another want "OK". | 12:30 |
gordc | gzhai1: do you have a picture of your workflow in your proposal? i think it's slightly confusing right now. | 12:31 |
gzhai1 | ok. | 12:31 |
*** jwcroppe has joined #openstack-telemetry | 12:31 | |
gordc | gzhai1: the reason i suggested two queues is that if a timeout alarm is sent, it has to wait behind a backlog of other events when in reality, the alarm is already timed out. | 12:32 |
gordc | but yeah, i think we need to put some pictures up or something. it's all very ambiguous right now. | 12:32 |
gordc | maybe create docs first and then code. | 12:32 |
gzhai1 | I describe it in the spec | 12:33 |
gzhai1 | But may be not clear enough. | 12:33 |
gzhai1 | I'm writing a simple description for you. | 12:33 |
jd__ | and for me? :) | 12:33 |
gzhai1 | say. event-alarm with 'event1' and 'timeout'=10. | 12:34 |
gzhai1 | if event1 comes, become ALARM | 12:34 |
gordc | jd__: never for you | 12:34 |
gordc | gzhai1: maybe move the spec to aodh docs | 12:34 |
gordc | i already mentioned in my other review that your code doesn't actually match your spec. | 12:34 |
*** fguillot has joined #openstack-telemetry | 12:34 | |
gzhai1 | gordc: it's one new state. we can use existed one, which is more clear. | 12:35 |
jd__ | it's just that I wrote several times my concern about the general design and since I did not have any argument against mine I -2 the patch now :( | 12:35 |
gzhai1 | we also start a process for timeout calculation for 10s and send out 'alarm.timeout.end' event. | 12:35 |
jd__ | maybe I'm wrong and all but I have no clue | 12:36 |
gzhai1 | jd__: I'm aware that you have concerns about the event for timeout. | 12:37 |
gzhai1 | gordc: so the final alarm state depends on which event comes first. | 12:37 |
gordc | gzhai1: same concerns as jd__ ... also, it's been almost 1 year since we wrote the spec so i'm not sure anyone actually remembers anything from original | 12:37 |
jd__ | I admit I slept a bit since then | 12:38 |
*** david-lyle has joined #openstack-telemetry | 12:38 | |
gzhai1 | jd__: how do you define the period of the worker? | 12:39 |
gordc | gzhai1: the event queue listener is threaded. it's kind of hard to say what came first if you let 64 threads race to process events. | 12:39 |
gzhai1 | gordc: if they come closely, it could be "ALARM" or "OK". but they are in sequence and no race condition | 12:40 |
jd__ | gzhai1: it's just a SQL request in the end, so you can run it every N seconds as far as I can tell | 12:41 |
gzhai1 | jd__: event-alarm is event-driven, which is different from normal alarm. Periodic worker doesn't work | 12:41 |
jd__ | O_o | 12:42 |
jd__ | it… it can work | 12:42 |
gzhai1 | jd__: say if you define 60s as period. But we only have one alarm with 10000s timeout? what a waste. | 12:42 |
jd__ | you add a field "last seen" to your alarm in the db and you update it? | 12:42 |
gzhai1 | furthermore, if you define 60s, but I want a 10s timeout, how to achieve it. | 12:43 |
gordc | gzhai1: they are in sequence in the queue but when they get pulled off they are racing. | 12:43 |
*** david-lyle has quit IRC | 12:43 | |
gzhai1 | gordc: you handle them one by one. | 12:44 |
gordc | gzhai1: i'd prefer you do docs first, you can summarise your spec if you want. the main issue is the design is hard to follow and it the spec is old. | 12:44 |
gordc | gzhai1: you don't. it's hidden to you, but oslo.messaging is threaded. it's not pulling one at a time. | 12:44 |
gordc | it's pulling 64 at a time. or something like that. | 12:44 |
gzhai1 | yes. | 12:45 |
jd__ | sileht: gordc why are you snobing https://review.openstack.org/#/c/171764/ ? :) | 12:45 |
gzhai1 | but aodh code handle them one by one, right? | 12:45 |
sileht | jd__, because I would verify that mysql 5.6 is shipped in newton osp, but I still haven't checked | 12:46 |
gordc | gzhai1: yes, but it'll be very, very, very slow. | 12:46 |
gzhai1 | then handler pull all the event, the use them one by one to trigger the evaluator. | 12:46 |
jd__ | sileht: ha good call, I think I asked that to pradk back then but I can ask yurii | 12:46 |
gordc | jd__: why are there so many rechecks?lol | 12:47 |
jd__ | gordc: because it needed MySQL 5.6 (Xenial) | 12:47 |
gzhai1 | gord: slow is the reality. | 12:47 |
*** fguillot is now known as fguillot_afk | 12:47 | |
jd__ | gordc: so I had to change -infra :) | 12:47 |
gordc | gzhai1: well. if it's slow, it sort of defeats purpose of implementing it? | 12:47 |
*** fguillot_afk has quit IRC | 12:48 | |
gordc | jd__: word... i'll try to look later today. | 12:48 |
gzhai1 | gordc: if many required event comes, it's probably slow to trigger the alarm, right? | 12:48 |
gzhai1 | then it's sort of defeats the purpose of event-alarm? | 12:48 |
gzhai1 | timeout event is just same priority as other event. | 12:48 |
gordc | gzhai1: maybe? we shouldnt' be implementing it if we know already it's unusable. | 12:48 |
*** shardy_lunch is now known as shardy | 12:49 | |
gzhai1 | gordc: it's not unusable. it's probably as slow/fast as current design. | 12:50 |
gzhai1 | use multiple queue need add extra locking, and probably make it slower. | 12:50 |
gordc | gzhai1: i'd say write it in doc first since you'll need to do that anyways. this way it'll help explain design to everyone | 12:51 |
gordc | right now it seems like the idea is it either: 1) is slow or 2) is racing | 12:51 |
*** fguillot has joined #openstack-telemetry | 12:52 | |
gzhai1 | gordc: ok | 12:52 |
gordc | gzhai1: yeah, sorry. it seems right now only you understands what the workflow is. better to write it down for everyone (in non-spec form). | 12:53 |
*** cdent has quit IRC | 12:57 | |
sileht | jd, it looks like rdo/osp use mariadb, and mariadb support prceise datetime since 5.3 | 12:58 |
sileht | and 10.X is shipped I think we are good ! | 12:58 |
jd__ | sileht: awesome | 12:58 |
*** vishwanathj has joined #openstack-telemetry | 13:04 | |
*** gzhai1 has quit IRC | 13:06 | |
*** jwcroppe has quit IRC | 13:12 | |
*** fguillot is now known as fguillot_afk | 13:17 | |
*** thorst_ has joined #openstack-telemetry | 13:17 | |
*** ljxiash has quit IRC | 13:19 | |
*** ljxiash has joined #openstack-telemetry | 13:20 | |
openstackgerrit | Hanxi Liu proposed openstack/ceilometer: Fix UnicodeEncodeError in Ceilometer polling https://review.openstack.org/368418 | 13:20 |
*** vint_bra has quit IRC | 13:23 | |
*** zhurong has joined #openstack-telemetry | 13:23 | |
*** thorst_ has quit IRC | 13:24 | |
*** donghao has quit IRC | 13:26 | |
*** liuhanxi has quit IRC | 13:26 | |
*** vint_bra has joined #openstack-telemetry | 13:27 | |
openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/gnocchi: ceph: make computed measures read async https://review.openstack.org/370898 | 13:30 |
openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/gnocchi: ceph: make computed measures write async https://review.openstack.org/370897 | 13:30 |
openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/gnocchi: ceph: move out of xattr completely https://review.openstack.org/370896 | 13:30 |
openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/gnocchi: Don't mock ceph https://review.openstack.org/371301 | 13:30 |
openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/gnocchi: ceph: fix python3 issue https://review.openstack.org/372494 | 13:30 |
*** amoralej|lunch is now known as amoralej | 13:31 | |
*** kong has quit IRC | 13:41 | |
*** vint_bra has quit IRC | 13:48 | |
*** vint_bra has joined #openstack-telemetry | 13:50 | |
*** liamji has joined #openstack-telemetry | 13:52 | |
*** liuhanxi has joined #openstack-telemetry | 13:54 | |
sileht | jd__, replacing the rados mocking with real ceph have show up a ton of python 3 bugs ... | 13:55 |
openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/gnocchi: ceph: make computed measures read async https://review.openstack.org/370898 | 13:57 |
openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/gnocchi: ceph: make computed measures write async https://review.openstack.org/370897 | 13:57 |
openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/gnocchi: ceph: move out of xattr completely https://review.openstack.org/370896 | 13:57 |
openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/gnocchi: Don't mock ceph https://review.openstack.org/371301 | 13:57 |
openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/gnocchi: ceph: fix python3 issue https://review.openstack.org/372494 | 13:57 |
jd__ | sileht: nice :) | 13:58 |
EmilienM | sileht: yes, we ship MariaDB 10.x since Mitaka | 14:00 |
*** vint_bra has quit IRC | 14:01 | |
*** rbak has joined #openstack-telemetry | 14:01 | |
*** vint_bra has joined #openstack-telemetry | 14:04 | |
*** thorst_ has joined #openstack-telemetry | 14:13 | |
*** fguillot has joined #openstack-telemetry | 14:15 | |
*** ljxiash_ has joined #openstack-telemetry | 14:25 | |
openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/gnocchi: ceph: make computed measures read async https://review.openstack.org/370898 | 14:26 |
openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/gnocchi: ceph: make computed measures write async https://review.openstack.org/370897 | 14:26 |
openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/gnocchi: ceph: move out of xattr completely https://review.openstack.org/370896 | 14:26 |
openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/gnocchi: Don't mock ceph https://review.openstack.org/371301 | 14:26 |
openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/gnocchi: ceph: fix python3 issue https://review.openstack.org/372494 | 14:26 |
*** ljxiash has quit IRC | 14:28 | |
*** ljxiash_ has quit IRC | 14:31 | |
*** thorst_ has quit IRC | 14:33 | |
*** thorst_ has joined #openstack-telemetry | 14:34 | |
*** openstackgerrit has quit IRC | 14:37 | |
*** openstackgerrit has joined #openstack-telemetry | 14:38 | |
*** david-lyle has joined #openstack-telemetry | 14:40 | |
*** thorst_ has quit IRC | 14:42 | |
*** thorst_ has joined #openstack-telemetry | 14:43 | |
*** david-lyle has quit IRC | 14:45 | |
*** cdent has joined #openstack-telemetry | 14:47 | |
openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/gnocchi: ceph: make computed measures read async https://review.openstack.org/370898 | 14:55 |
openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/gnocchi: ceph: make computed measures write async https://review.openstack.org/370897 | 14:55 |
openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/gnocchi: ceph: move out of xattr completely https://review.openstack.org/370896 | 14:55 |
openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/gnocchi: Don't mock ceph https://review.openstack.org/371301 | 14:55 |
openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/gnocchi: carbonara-drivers: elapsed can be zero https://review.openstack.org/372555 | 14:55 |
*** pcaruana has quit IRC | 15:07 | |
sileht | arf our gate is broken by tempest now :( | 15:25 |
*** cdent has quit IRC | 15:26 | |
*** pshedimb has joined #openstack-telemetry | 15:27 | |
*** rcernin has quit IRC | 15:30 | |
*** thorst_ has quit IRC | 15:31 | |
*** thorst_ has joined #openstack-telemetry | 15:31 | |
*** david-lyle has joined #openstack-telemetry | 15:32 | |
*** fguillot has quit IRC | 15:32 | |
openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/gnocchi: ceph: make computed measures read async https://review.openstack.org/370898 | 15:35 |
openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/gnocchi: ceph: make computed measures write async https://review.openstack.org/370897 | 15:35 |
openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/gnocchi: ceph: move out of xattr completely https://review.openstack.org/370896 | 15:35 |
openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/gnocchi: Don't mock ceph https://review.openstack.org/371301 | 15:35 |
openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/gnocchi: track the metric locked time https://review.openstack.org/371300 | 15:35 |
openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/gnocchi: carbonara-drivers: elapsed can be zero https://review.openstack.org/372555 | 15:35 |
openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/gnocchi: ceph: fix python3 issue https://review.openstack.org/372494 | 15:35 |
openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/gnocchi: Add simple upgrade tests https://review.openstack.org/368412 | 15:35 |
openstackgerrit | Julien Danjou proposed openstack/aodh: sqlalchemy: use DATETIME(fsp=6) rather than DECIMAL https://review.openstack.org/372586 | 15:36 |
openstackgerrit | Hanxi Liu proposed openstack/ceilometer: Replace import objects instead of modules https://review.openstack.org/372587 | 15:36 |
openstackgerrit | Hanxi Liu proposed openstack/ceilometer: Remove import objects and use import modules https://review.openstack.org/372587 | 15:38 |
*** Jeffrey4l_ has left #openstack-telemetry | 15:46 | |
sileht | jd__, any idea for http://logs.openstack.org/97/370897/19/check/gate-gnocchi-dsvm-functional-file-mysql-ubuntu-xenial/6fa2852/logs/devstacklog.txt.gz#_2016-09-19_15_11_33_661 | 15:52 |
*** Guest22675 has quit IRC | 15:53 | |
*** cdent has joined #openstack-telemetry | 15:53 | |
openstackgerrit | Darren Hague proposed openstack/ceilometer: Batching of event & meter dispatching over HTTP https://review.openstack.org/372596 | 15:53 |
sileht | jd__, a good example of why gerrit workflow based on rebase is better than pull request merge: | 15:54 |
sileht | jd__, https://github.com/sileht/cotyledon/commit/640743402b4edeabea484e07396ef7d7c37deed7 | 15:55 |
*** zhurong has quit IRC | 15:55 | |
sileht | jd__, https://github.com/sileht/cotyledon/commit/d50ee9cb385469a5e4193ae901ea33c3c1badaf7 | 15:55 |
jd__ | lol | 15:55 |
jd__ | ok | 15:55 |
sileht | jd__, both commits do the same fix | 15:55 |
sileht | jd__, git show two diffs | 15:55 |
sileht | jd__, but the result is the same | 15:55 |
jd__ | sileht: it seems glance is not enabled in tempest at first glance | 15:55 |
jd__ | (pun not intended) | 15:56 |
sileht | is it a rythme joke ? | 15:56 |
sileht | :) | 15:56 |
*** cdent has quit IRC | 15:56 | |
jd__ | sileht: seems like ff865d625cbb70e858034b06fa494cafc381cd49 just got merged | 15:57 |
jd__ | and that might be the culprit | 15:57 |
jd__ | sileht: it happens on all patches? | 15:58 |
sileht | jd__, and because this is a merge, I didn't see the same commit order on Github and locally ... | 15:58 |
sileht | jd__, at least on my gnocchi ceph branch | 15:58 |
sileht | jd__, I will recheck | 15:58 |
jd__ | sileht: https://review.openstack.org/#/c/372603/ | 15:59 |
jd__ | you're welcome | 16:00 |
*** xiaozhuangqing has quit IRC | 16:01 | |
*** rwsu has joined #openstack-telemetry | 16:14 | |
liuhanxi | gordc, llu, are you online? | 16:15 |
*** thorst_ has quit IRC | 16:18 | |
*** thorst_ has joined #openstack-telemetry | 16:18 | |
gordc | liuhanxi: yes | 16:19 |
gordc | whatsup? | 16:19 |
liuhanxi | gordc, would you give some advice about this patch? | 16:19 |
liuhanxi | https://review.openstack.org/#/c/368418/ | 16:19 |
gordc | liuhanxi: i can't verify your bug but what's the id that is causing failure? | 16:23 |
gordc | are you putting in your own id? | 16:23 |
liuhanxi | what you mean about id? | 16:24 |
*** thorst_ has quit IRC | 16:25 | |
*** thorst_ has joined #openstack-telemetry | 16:26 | |
liuhanxi | gordc, could you expain it? | 16:28 |
gordc | liuhanxi: your bug. it is failing to encode one of the ids? | 16:28 |
liuhanxi | gordc, I use some non-ASCII characters to test the unicode case. | 16:29 |
*** liamji has quit IRC | 16:32 | |
liuhanxi | If use just id follow u'\u***', there will cause the error. | 16:32 |
liuhanxi | gordc, the key convert to str will cause the problem when the key is a unicode. | 16:35 |
liuhanxi | https://github.com/openstack/ceilometer/blob/master/ceilometer/utils.py#L240 | 16:35 |
gordc | i'm just wondering where we even allow you to pass in unicode | 16:35 |
liuhanxi | gordc, I think the UnicodeEncodeError is caused by the unicode | 16:37 |
liuhanxi | And the unit test has no unicode string to test the problem | 16:38 |
*** yprokule has quit IRC | 16:40 | |
*** dhague has quit IRC | 16:41 | |
*** thorst_ has quit IRC | 16:41 | |
*** thorst_ has joined #openstack-telemetry | 16:48 | |
*** rcernin has joined #openstack-telemetry | 16:51 | |
*** yarkot1 has quit IRC | 16:57 | |
openstackgerrit | Hanxi Liu proposed openstack/python-ceilometerclient: Replace assertEqual(None, ...) with assertIsNone(...) https://review.openstack.org/372571 | 17:00 |
*** thorst_ has quit IRC | 17:05 | |
*** thorst_ has joined #openstack-telemetry | 17:11 | |
*** thorst_ has quit IRC | 17:16 | |
*** Tamayo has joined #openstack-telemetry | 17:18 | |
openstackgerrit | Merged openstack/ceilometer: Remove import objects and use import modules https://review.openstack.org/372587 | 17:20 |
*** liuhanxi has quit IRC | 17:24 | |
*** efoley_ has quit IRC | 17:28 | |
*** Tamayo has quit IRC | 17:28 | |
*** harlowja has joined #openstack-telemetry | 17:30 | |
*** rbak has quit IRC | 17:30 | |
*** rbak has joined #openstack-telemetry | 17:32 | |
*** amoralej is now known as amoralej|off | 17:46 | |
*** jwcroppe has joined #openstack-telemetry | 17:51 | |
*** jwcroppe_ has joined #openstack-telemetry | 17:52 | |
*** shardy has quit IRC | 17:52 | |
*** jwcropp__ has joined #openstack-telemetry | 17:53 | |
*** jwcroppe has quit IRC | 17:56 | |
*** jwcroppe_ has quit IRC | 17:57 | |
*** liusheng has quit IRC | 18:40 | |
*** liusheng has joined #openstack-telemetry | 18:40 | |
*** jwcropp__ has quit IRC | 18:43 | |
*** jwcroppe has joined #openstack-telemetry | 19:13 | |
*** r-mibu has quit IRC | 19:17 | |
*** davidlenwell has quit IRC | 19:17 | |
*** r-mibu has joined #openstack-telemetry | 19:17 | |
*** davidlenwell has joined #openstack-telemetry | 19:29 | |
*** gordc has quit IRC | 19:34 | |
*** jwcroppe has quit IRC | 19:43 | |
*** jwcroppe has joined #openstack-telemetry | 19:49 | |
*** thorst_ has joined #openstack-telemetry | 19:54 | |
*** thorst_ has quit IRC | 20:10 | |
*** thorst_ has joined #openstack-telemetry | 20:25 | |
*** rwsu has quit IRC | 20:36 | |
*** jwcroppe has quit IRC | 20:44 | |
*** thorst_ has quit IRC | 20:54 | |
*** jwcroppe has joined #openstack-telemetry | 20:55 | |
*** jwcroppe has quit IRC | 21:11 | |
*** kong has joined #openstack-telemetry | 21:17 | |
*** vint_bra has quit IRC | 21:32 | |
*** jwcroppe has joined #openstack-telemetry | 21:36 | |
*** r-daneel has joined #openstack-telemetry | 21:39 | |
*** vishwanathj has quit IRC | 21:43 | |
*** jwcroppe has quit IRC | 21:48 | |
*** harlowja has quit IRC | 21:51 | |
*** larainema has quit IRC | 21:51 | |
*** tobasco has quit IRC | 21:51 | |
*** tobasco has joined #openstack-telemetry | 21:51 | |
*** larainema has joined #openstack-telemetry | 21:53 | |
*** rwsu has joined #openstack-telemetry | 21:53 | |
*** harlowja has joined #openstack-telemetry | 21:56 | |
*** jwcroppe has joined #openstack-telemetry | 22:06 | |
*** adriant has joined #openstack-telemetry | 22:09 | |
*** rcernin has quit IRC | 22:27 | |
*** jwcroppe has quit IRC | 22:40 | |
openstackgerrit | Prudhvi Rao Shedimbi proposed openstack/ceilometer: Compute agent can poll tx and rx errors and drops https://review.openstack.org/368225 | 22:43 |
openstackgerrit | Dharini Chandrasekar proposed openstack/ceilometer: Compute agent can poll tx and rx errors and drops https://review.openstack.org/368225 | 22:48 |
*** jwcroppe has joined #openstack-telemetry | 22:51 | |
*** jwcroppe has quit IRC | 22:56 | |
*** rbak has quit IRC | 23:01 | |
*** jwcroppe has joined #openstack-telemetry | 23:06 | |
*** jwcroppe has quit IRC | 23:11 | |
*** r-daneel has quit IRC | 23:11 | |
*** fguillot has joined #openstack-telemetry | 23:12 | |
*** thorst_ has joined #openstack-telemetry | 23:14 | |
*** adriant has quit IRC | 23:23 | |
*** thorst_ has quit IRC | 23:25 | |
*** jwcroppe has joined #openstack-telemetry | 23:29 | |
openstackgerrit | Prudhvi Rao Shedimbi proposed openstack/ceilometer: Compute agent can poll tx and rx errors and drops https://review.openstack.org/368225 | 23:31 |
*** jwcroppe has quit IRC | 23:41 | |
*** openstackgerrit has quit IRC | 23:42 | |
*** zhurong has joined #openstack-telemetry | 23:43 | |
*** openstackgerrit has joined #openstack-telemetry | 23:47 | |
*** jwcroppe has joined #openstack-telemetry | 23:48 | |
*** openstackgerrit has quit IRC | 23:51 | |
*** openstackgerrit has joined #openstack-telemetry | 23:55 | |
*** jwcroppe has quit IRC | 23:57 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!