*** thumpba has quit IRC | 00:29 | |
*** thumpba has joined #openstack-telemetry | 00:55 | |
*** zhangguoqing has quit IRC | 01:23 | |
openstackgerrit | Ha Van Tu proposed openstack/ceilometer: Refactor Ceilometer alarm API https://review.openstack.org/369985 | 01:59 |
---|---|---|
openstackgerrit | Ha Van Tu proposed openstack/ceilometer: Refactor Ceilometer event API https://review.openstack.org/370510 | 01:59 |
*** zhurong has joined #openstack-telemetry | 02:01 | |
*** thumpba has quit IRC | 02:08 | |
*** catintheroof has joined #openstack-telemetry | 02:27 | |
*** fguillot has quit IRC | 02:29 | |
*** thumpba has joined #openstack-telemetry | 02:35 | |
*** vishwanathj has quit IRC | 02:39 | |
*** zhurong has quit IRC | 02:45 | |
*** zhurong has joined #openstack-telemetry | 02:49 | |
*** zhurong has quit IRC | 03:13 | |
*** thumpba has quit IRC | 03:40 | |
*** thumpba has joined #openstack-telemetry | 03:40 | |
*** thumpba has quit IRC | 03:58 | |
*** thumpba has joined #openstack-telemetry | 03:58 | |
*** thumpba has quit IRC | 03:58 | |
*** thumpba has joined #openstack-telemetry | 03:59 | |
*** thumpba has quit IRC | 03:59 | |
*** thumpba has joined #openstack-telemetry | 03:59 | |
*** thumpba has quit IRC | 04:00 | |
*** donghao has joined #openstack-telemetry | 04:06 | |
*** dave-mccowan has quit IRC | 04:08 | |
*** yprokule has joined #openstack-telemetry | 04:33 | |
*** rcernin has quit IRC | 04:37 | |
*** harlowja has quit IRC | 04:45 | |
*** donghao has quit IRC | 04:49 | |
*** vishwanathj has joined #openstack-telemetry | 04:59 | |
*** vishwanathj has quit IRC | 05:00 | |
*** adriant has quit IRC | 05:40 | |
*** rcernin has joined #openstack-telemetry | 05:43 | |
*** AJaeger has left #openstack-telemetry | 05:43 | |
sileht | jd__, looks like only v2 endpoint is enabled and client configuration force usage of v3 | 05:57 |
*** thumpba has joined #openstack-telemetry | 06:01 | |
*** thumpba has quit IRC | 06:06 | |
*** rcernin has quit IRC | 06:14 | |
*** rcernin has joined #openstack-telemetry | 06:19 | |
*** pcaruana has joined #openstack-telemetry | 06:23 | |
*** thorst_ has joined #openstack-telemetry | 06:34 | |
*** dhellmann has quit IRC | 06:39 | |
*** dhellmann has joined #openstack-telemetry | 06:42 | |
*** david-lyle_ has joined #openstack-telemetry | 06:59 | |
*** david-lyle has quit IRC | 07:00 | |
*** amoralej|off is now known as amoralej | 07:19 | |
*** julian1 has quit IRC | 07:19 | |
*** julian1 has joined #openstack-telemetry | 07:20 | |
openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/gnocchi: Allow to retry to delete a resource https://review.openstack.org/370783 | 07:22 |
openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/gnocchi: ceph: make computed measures read async https://review.openstack.org/370898 | 07:23 |
openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/gnocchi: ceph: make computed measures write async https://review.openstack.org/370897 | 07:23 |
openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/gnocchi: ceph: move out of xattr completely https://review.openstack.org/370896 | 07:23 |
openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/gnocchi: Add simple upgrade tests https://review.openstack.org/368412 | 07:23 |
openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/gnocchi: track the metric locked time https://review.openstack.org/371300 | 07:23 |
openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/gnocchi: Don't mock ceph https://review.openstack.org/371301 | 07:23 |
openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/gnocchi: Make some gabbi tests aware of ceph https://review.openstack.org/371302 | 07:23 |
sileht | jd__, my code was right yesterday, the bench run too, but the thing measured is not comparable with async and sync IO | 07:25 |
*** david-lyle has joined #openstack-telemetry | 07:28 | |
sileht | jd__, because the 'Computed new metric' take IO time in consideration when IO are sync and when IO are async everything returns instantly (and wait later after the computation) | 07:28 |
*** david-lyle_ has quit IRC | 07:29 | |
sileht | jd__, I have added a new time tracking log that measure the metric lock duration, this is more comparable to bench carbonara + backend IOs | 07:30 |
sileht | jd__, ASYNC increase thing a bit, but not so much finally :( | 07:31 |
*** thorst_ has quit IRC | 07:31 | |
jd__ | sileht: ok :) so you have to wait() or something after write? | 07:33 |
sileht | jd__, I call a wait method on ceph driver just before releasing the lock | 07:33 |
*** zhurong has joined #openstack-telemetry | 07:34 | |
sileht | jd__, My bench is on a one ceph node cluster, that not terrible to test my change | 07:39 |
*** kong has quit IRC | 07:41 | |
openstackgerrit | Javier Peña proposed openstack/aodh: Fix locale builds https://review.openstack.org/371319 | 07:51 |
*** shardy has joined #openstack-telemetry | 07:58 | |
*** jkraj has joined #openstack-telemetry | 08:01 | |
*** thumpba has joined #openstack-telemetry | 08:01 | |
*** openstackgerrit has quit IRC | 08:03 | |
*** openstackgerrit has joined #openstack-telemetry | 08:04 | |
openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/gnocchi: ceph: make computed measures read async https://review.openstack.org/370898 | 08:07 |
openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/gnocchi: ceph: make computed measures write async https://review.openstack.org/370897 | 08:07 |
openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/gnocchi: ceph: move out of xattr completely https://review.openstack.org/370896 | 08:07 |
openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/gnocchi: Don't mock ceph https://review.openstack.org/371301 | 08:07 |
openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/gnocchi: Make some gabbi tests aware of ceph https://review.openstack.org/371302 | 08:07 |
openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/gnocchi: ceph: make computed measures read async https://review.openstack.org/370898 | 08:08 |
openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/gnocchi: ceph: make computed measures write async https://review.openstack.org/370897 | 08:08 |
openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/gnocchi: ceph: move out of xattr completely https://review.openstack.org/370896 | 08:08 |
openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/gnocchi: Don't mock ceph https://review.openstack.org/371301 | 08:08 |
openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/gnocchi: track the metric locked time https://review.openstack.org/371300 | 08:08 |
openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/gnocchi: Make some gabbi tests aware of ceph https://review.openstack.org/371302 | 08:08 |
sileht | jd__, is that ok to pifpaf for ceph instead of the mock class for testing ? https://review.openstack.org/371301 | 08:09 |
*** cdent has joined #openstack-telemetry | 08:10 | |
* sileht dislike mock and the current mock start to be big | 08:11 | |
sileht | jd__, on stable branch the keystone issue is a requirement thing: http://logs.openstack.org/34/371334/1/check/gate-gnocchi-dsvm-functional-file-mysql-ubuntu-trusty/f3359de/logs/screen-key-a.txt.gz#_2016-09-16_08_07_31_969 | 08:14 |
*** khushbu has joined #openstack-telemetry | 08:16 | |
sileht | I don't understand gnocchi requires 3.3, other 3.5, requirement repo install 3.8 and keystone fail to start because someones ask for 3.16 | 08:19 |
sileht | jd__, gnocchi on 2.2 requires oslo.db 4.8 (that require oslo.utils >= 3.15) when other requires oslo.db 4.7 (that requires oslo.utils 3.5) | 08:21 |
sileht | jd__, does this really requires 4.8 ? https://github.com/openstack/gnocchi/commit/5551a276 | 08:24 |
sileht | that's looks require for 'DBNonExistentTable' | 08:26 |
sileht | I dunno what to do | 08:27 |
*** thumpba has quit IRC | 08:30 | |
*** shardy has quit IRC | 08:40 | |
*** donghao has joined #openstack-telemetry | 08:50 | |
*** efoley has joined #openstack-telemetry | 09:03 | |
openstackgerrit | Merged openstack/aodh: Fix locale builds https://review.openstack.org/371319 | 09:08 |
openstackgerrit | Merged openstack/ceilometer: update docs to show Telemetry projects https://review.openstack.org/369622 | 09:12 |
openstackgerrit | Merged openstack/ceilometer: Update the compute node service_credentials parameters https://review.openstack.org/367698 | 09:13 |
*** cdent has quit IRC | 09:13 | |
openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/gnocchi: ceph: make computed measures read async https://review.openstack.org/370898 | 09:13 |
openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/gnocchi: ceph: make computed measures write async https://review.openstack.org/370897 | 09:13 |
openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/gnocchi: ceph: move out of xattr completely https://review.openstack.org/370896 | 09:13 |
openstackgerrit | Merged openstack/gnocchi: lower processor job polling and handle queuing https://review.openstack.org/370960 | 09:14 |
openstackgerrit | Merged openstack/gnocchi: doc: remove the rolling upgrade documentation https://review.openstack.org/371002 | 09:14 |
openstackgerrit | Merged openstack/gnocchi: carbonara: retrieve unaggregated timeserie only once for upgrade https://review.openstack.org/371001 | 09:14 |
*** khushbu has quit IRC | 09:18 | |
openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/gnocchi: ceph: make computed measures read async https://review.openstack.org/370898 | 09:20 |
openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/gnocchi: ceph: make computed measures write async https://review.openstack.org/370897 | 09:20 |
openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/gnocchi: ceph: move out of xattr completely https://review.openstack.org/370896 | 09:20 |
*** khushbu_ has joined #openstack-telemetry | 09:23 | |
*** shardy has joined #openstack-telemetry | 09:26 | |
*** cdent has joined #openstack-telemetry | 09:31 | |
jd__ | sileht: fuck… | 09:36 |
jd__ | sileht: so Newton is based on oslodb. 4.7? | 09:37 |
jd__ | ah no it's mitaka | 09:37 |
jd__ | sileht: we should just test 2.2 with Newton? | 09:37 |
jd__ | sileht: that was gordc plan | 09:38 |
openstackgerrit | Julien Danjou proposed openstack/ceilometer: add note regarding pipeline_processing_queues option https://review.openstack.org/365072 | 09:50 |
jd__ | sileht: maybe https://review.openstack.org/#/c/371151/ if you have an opinion would save us some recheck… | 09:51 |
*** zhurong has quit IRC | 10:02 | |
*** thorst_ has joined #openstack-telemetry | 10:12 | |
openstackgerrit | Merged openstack/ceilometer: Refactor Ceilometer alarm API https://review.openstack.org/369985 | 10:25 |
openstackgerrit | Merged openstack/ceilometer: Refactor Ceilometer event API https://review.openstack.org/370510 | 10:25 |
*** thumpba has joined #openstack-telemetry | 10:27 | |
*** thumpba has quit IRC | 10:32 | |
*** achatterjee has quit IRC | 10:35 | |
*** kong has joined #openstack-telemetry | 10:38 | |
*** thorst_ has quit IRC | 10:42 | |
*** khushbu_ has quit IRC | 10:43 | |
*** thorst_ has joined #openstack-telemetry | 10:47 | |
*** thorst_ has quit IRC | 10:52 | |
openstackgerrit | Merged openstack/ceilometer: add note regarding pipeline_processing_queues option https://review.openstack.org/365072 | 10:57 |
openstackgerrit | Merged openstack/gnocchi: swift: bump swiftclient dependency to 3.1.0 and set retries=0 https://review.openstack.org/367436 | 10:57 |
*** thorst_ has joined #openstack-telemetry | 10:59 | |
*** zhurong has joined #openstack-telemetry | 11:01 | |
*** thorst_ has quit IRC | 11:22 | |
*** cdent has quit IRC | 11:23 | |
*** thorst_ has joined #openstack-telemetry | 11:24 | |
*** thorst_ has quit IRC | 11:28 | |
*** thorst_ has joined #openstack-telemetry | 11:33 | |
*** thorst_ has quit IRC | 11:38 | |
*** catintheroof has quit IRC | 11:39 | |
*** amoralej is now known as amoralej|lunch | 12:08 | |
*** liusheng has quit IRC | 12:11 | |
*** liusheng has joined #openstack-telemetry | 12:11 | |
*** thorst_ has joined #openstack-telemetry | 12:14 | |
*** thorst_ has quit IRC | 12:14 | |
*** thorst_ has joined #openstack-telemetry | 12:20 | |
*** pradk has joined #openstack-telemetry | 12:23 | |
*** fguillot has joined #openstack-telemetry | 12:24 | |
*** catintheroof has joined #openstack-telemetry | 12:27 | |
*** thorst_ has quit IRC | 12:28 | |
*** thorst_ has joined #openstack-telemetry | 12:29 | |
sileht | jd__, I'm a bit lazy of moving everything to Newton, this have never be tested, perhaps other thing will break | 12:31 |
jd__ | sileht: Mitaka? | 12:31 |
sileht | jd__, why not just copy the exception with oslo.db < 4.8 ? | 12:31 |
sileht | jd__, something like that: https://review.openstack.org/#/c/371500/ | 12:36 |
*** fguillot has quit IRC | 12:40 | |
*** gordc has joined #openstack-telemetry | 12:42 | |
*** khushbu_ has joined #openstack-telemetry | 12:44 | |
*** thumpba has joined #openstack-telemetry | 12:44 | |
*** yprokule_ has joined #openstack-telemetry | 12:46 | |
*** yprokule has quit IRC | 12:48 | |
*** yprokule_ is now known as yprokule | 12:48 | |
*** cdent has joined #openstack-telemetry | 12:49 | |
*** thorst_ has quit IRC | 12:55 | |
*** vishwanathj has joined #openstack-telemetry | 12:55 | |
*** thorst_ has joined #openstack-telemetry | 12:56 | |
*** thorst_ has quit IRC | 13:02 | |
*** thorst_ has joined #openstack-telemetry | 13:04 | |
*** khushbu_ has quit IRC | 13:05 | |
*** dave-mccowan has joined #openstack-telemetry | 13:07 | |
openstackgerrit | Julien Danjou proposed openstack/ceilometer: agentbase: remove flaky test https://review.openstack.org/371151 | 13:08 |
*** thumpba has quit IRC | 13:17 | |
*** rbak has joined #openstack-telemetry | 13:19 | |
*** amoralej|lunch is now known as amoralej | 13:27 | |
*** thorst_ has quit IRC | 13:32 | |
*** fguillot has joined #openstack-telemetry | 13:32 | |
*** paw has quit IRC | 13:32 | |
jd__ | sileht: does not seem to work :( | 13:34 |
jd__ | sileht: we tested 2.2 with newton-master at the time so I'd say it'd work? | 13:34 |
gordc | jd__: we can only gate 2.2 on newton i believe. | 13:37 |
jd__ | gordc: sileht want to gate on Mitaka | 13:37 |
jd__ | fight! | 13:37 |
gordc | lol | 13:37 |
*** thorst_ has joined #openstack-telemetry | 13:37 | |
gordc | i don't think you can. the upper-constraints non-sense will block you. | 13:38 |
*** yprokule_ has joined #openstack-telemetry | 13:38 | |
sileht | jd__, ! https://github.com/openstack-infra/project-config/commit/779f18ab2183bd9bb99d78e6a3424d1fedd5127e | 13:38 |
gordc | upper constraints in mitaka doesn't match the tooz we use in 2.2 | 13:38 |
sileht | ok if tooz is also concerned ... | 13:38 |
gordc | i tried to bump requirements | 13:38 |
gordc | they told me that's not possible even though the requirement has no real upper cap | 13:39 |
*** yprokule has quit IRC | 13:39 | |
*** yprokule_ is now known as yprokule | 13:39 | |
gordc | somethign about tooz we use is newton tooz. | 13:39 |
jd__ | I wish I never put tooz in oslo | 13:40 |
jd__ | I was young and a believer | 13:40 |
gordc | i do blame you | 13:40 |
jd__ | u can | 13:41 |
jd__ | it was before I met you… | 13:41 |
jd__ | the sky was blue and the clouds white | 13:41 |
jd__ | no rain on the forecast | 13:41 |
gordc | hahah, now it's darkness | 13:42 |
*** thorst_ has quit IRC | 13:42 | |
jd__ | so much disturbance in the force | 13:42 |
gordc | i'm sure that's what rest of community sees when they see us. 'f, it's those telemetry folks again making trouble' | 13:42 |
jd__ | I am pretty sure of it too | 13:43 |
*** thorst_ has joined #openstack-telemetry | 13:45 | |
jd__ | gordc: sileht: I suggest we put effort on giving up on devstack | 13:46 |
jd__ | rather than fix that | 13:46 |
jd__ | stable/3.0 is coming shortly, and that's the one we'll really maintain | 13:46 |
*** thorst_ has quit IRC | 13:46 | |
gordc | jd__: i tried to add threading to the batch api. it helps maybe 10-15% if i post 20metric+60measures batches... if i post 20metrics+720measures batches, no difference. | 13:46 |
jd__ | gordc: and if you push 1 measure per batch? that's a very common case | 13:47 |
gordc | jd__: you mean give up on devstack completely? | 13:47 |
gordc | that's EmilienM's dream | 13:47 |
sileht | I replace devstack by pifpaf often now | 13:48 |
gordc | jd__: i'll try 1 measures. but you would think 720 measure would be biggest gain since that' a bigger file. i guess https://github.com/openstack/gnocchi/blob/master/gnocchi/storage/_carbonara.py#L327 is cpu bound? | 13:48 |
jd__ | sileht: that's a probably what we should focus on | 13:49 |
jd__ | gordc: yes | 13:49 |
jd__ | gordc: for Gnocchi at least | 13:49 |
jd__ | gordc: haha | 13:49 |
jd__ | gordc: you talk about that line and I reply with: | 13:49 |
jd__ | (wait for it) | 13:49 |
jd__ | (wait for it) | 13:51 |
jd__ | (wait a little more for it) | 13:51 |
gordc | what if i just drop right now | 13:51 |
sileht | out of this body barney ! | 13:51 |
*** nicodemus_ has joined #openstack-telemetry | 13:51 | |
openstackgerrit | Julien Danjou proposed openstack/gnocchi: carbonara: implement compressed format for BoundTimeSerie https://review.openstack.org/370849 | 13:52 |
openstackgerrit | Julien Danjou proposed openstack/gnocchi: carbonara: replace msgpack encoding with struct for new measures https://review.openstack.org/371550 | 13:52 |
jd__ | thaaaat ^^^^ | 13:52 |
jd__ | …DARY! LEGENDARY! | 13:52 |
* jd__ . o O (maybe I mixed things up a bit) | 13:52 | |
sileht | Did we keep code from 2.2 in 3.0 on storage side ? | 13:53 |
sileht | Or are we rewritte everything :) | 13:53 |
gordc | lol | 13:53 |
gordc | i was just thinking that... we should just delete upgrade code. | 13:54 |
gordc | no chance it works | 13:54 |
*** thorst_ has joined #openstack-telemetry | 13:54 | |
gordc | jd__: isn't msgpack fast? it's suppose to be fast. | 13:56 |
gordc | is the problem the datetime stuff or msgpack as well? | 13:56 |
*** thorst_ has quit IRC | 13:57 | |
*** fguillot has quit IRC | 13:57 | |
*** dhague has joined #openstack-telemetry | 13:57 | |
jd__ | gordc: msgpackutils encodes datetime objects in a dict with hours=X, minutes=3, etc… | 13:57 |
jd__ | so your 16 bytes timestamp is actually… I don't know, 80 bytes? :) | 13:58 |
jd__ | gordc: I hope the upgrade code works… we're gonna use/need it -_- | 13:58 |
*** thorst_ has joined #openstack-telemetry | 13:58 | |
dhague | jd__: Does my reply about eventlet/monkeypatching on https://review.openstack.org/#/c/366639/6 make sense? | 13:59 |
openstackgerrit | Julien Danjou proposed openstack/gnocchi: storage: increase default processing delays https://review.openstack.org/371561 | 14:00 |
jd__ | dhague: it seems | 14:00 |
dhague | sorry for the confusion - I'm also against eventlet, we just needed those 2 lines to stop things falling over because of eventlet use elsewhere :-( | 14:01 |
jd__ | yeah no my bad :) | 14:01 |
jd__ | #teamkilleventlet | 14:02 |
dhague | :-) | 14:02 |
jd__ | this is a eventlet free zone, you're safe with us | 14:02 |
jd__ | :D | 14:02 |
*** thorst_ has quit IRC | 14:04 | |
*** paw has joined #openstack-telemetry | 14:04 | |
openstackgerrit | Merged openstack/ceilometer: agentbase: remove flaky test https://review.openstack.org/371151 | 14:09 |
*** thorst_ has joined #openstack-telemetry | 14:09 | |
*** thorst_ has quit IRC | 14:12 | |
jd__ | Ceilometer Newton RC1 on https://review.openstack.org/371570 | 14:12 |
gordc | jd__: regular msgpack doesn't do that expansion i don't think. was there a reason we had msgpackutils? | 14:13 |
jd__ | gordc: regular msgpack does not know how to serialize datetime so that's why we picked msgpackutils back then | 14:14 |
jd__ | if I had any clue people will use Gnocchi on production at large scale 2 years ago I probably have done things differently… | 14:14 |
jd__ | :P | 14:14 |
*** thorst_ has joined #openstack-telemetry | 14:15 | |
gordc | lol, kk | 14:15 |
*** thorst_ has quit IRC | 14:16 | |
gordc | but i guess in theory we could convert to datetime to unix_time and use msgpack? | 14:16 |
gordc | i like our use of struct but just wondering if we're applying it everywhere with good reason. (and i'm also not awake enough yet to look at more struct magic.) | 14:17 |
gordc | we should make a seriespack lib. | 14:18 |
jd__ | gordc: msgpack is handy to serialize complex things (dict, arrays, etc) especially when mixed; here we just have an array of long/double, struct is the fastest | 14:20 |
jd__ | and smallest | 14:20 |
jd__ | I did not use lz4 because I don't think it's worth it here | 14:20 |
gordc | kk, yeah, that makes sense to me. good to see we thinking same :) | 14:20 |
*** thorst_ has joined #openstack-telemetry | 14:22 | |
EmilienM | sileht, devstack, pippafpouf its all the same | 14:22 |
EmilienM | nobody use it in production | 14:22 |
EmilienM | and developpers keep thinking their code is awesome because it works in devstack | 14:22 |
sileht | EmilienM, yeah but with pifpaf you can run in it in prodution, everything is scrashed when service stop | 14:23 |
*** thorst_ has quit IRC | 14:23 | |
EmilienM | are you testing ipv6, ssl in your gate? | 14:24 |
sileht | EmilienM, we don't test ourself we have user for that ;p | 14:26 |
EmilienM | that's what I'm saying | 14:27 |
EmilienM | we don't live in the same world | 14:27 |
sileht | EmilienM, I known we write bugs, you discover them, we write new bugs to hide them | 14:28 |
*** zhurong has quit IRC | 14:28 | |
sileht | EmilienM, more seriously I don't think we enable this in Gate, we run the default | 14:29 |
*** ryanpetrello has joined #openstack-telemetry | 14:30 | |
ryanpetrello | jd__ cdent `tox -e gabbi` seems busted for me in ceilometer HEAD | 14:30 |
ryanpetrello | https://travis-ci.org/pecan/pecan/jobs/160446344 | 14:30 |
ryanpetrello | looks like perhaps an import order thing | 14:30 |
ryanpetrello | I found a similar issue from a few years back: https://bugs.launchpad.net/ceilometer/+bug/1233139 | 14:31 |
openstack | Launchpad bug 1233139 in Ceilometer "Ceilometer nosetests fail " [Low,Fix released] - Assigned to Julien Danjou (jdanjou) | 14:31 |
EmilienM | sileht: don't worry, it's general in OpenStack | 14:31 |
gordc | jd__: i'm going to test the POST format change first. before the BoundTimeseries format. unless you think i should switch order because it's not ready or boundseries in higher priority | 14:32 |
cdent | ryanpetrello: hmmm. lemme see if I can replicate | 14:32 |
openstackgerrit | Davanum Srinivas (dims) proposed openstack/ceilometer: Update reno for stable/newton https://review.openstack.org/371585 | 14:33 |
*** thorst_ has joined #openstack-telemetry | 14:37 | |
EmilienM | sileht: it's funny to see that the deployement tool to test OpenStack in CI is the one that nobody use in production | 14:37 |
sileht | EmilienM, pifpaf have been designed to not be able to run full integration tests, but only to quickly have a backend setuped | 14:38 |
sileht | EmilienM, but for devstack I agree, integration should be done with something else than bash ! | 14:38 |
gordc | i tried 20metric+1measure batch, 108.58ms vs 114.42ms(no thread) but stdev overlaps so nothing important | 14:41 |
openstackgerrit | gordon chung proposed openstack/gnocchi: WIP enable threading of batched writes https://review.openstack.org/371589 | 14:46 |
jd__ | gordc: whatever, I want both :p | 14:47 |
jd__ | gordc: ok, so IIUC it's not such a big change that we want to reenable threads? | 14:47 |
gordc | i don't think it is. not in current state. | 14:48 |
jd__ | ryanpetrello: yeah likely an import problem caused by this good old CONF global variable #stateoftheart | 14:48 |
jd__ | gordc: /me nods | 14:48 |
*** fguillot has joined #openstack-telemetry | 14:48 | |
gordc | i'm just posting it for now while i test your other stuff. | 14:48 |
gordc | or if someone spots something wrong with code. | 14:49 |
gordc | jd__: there a reason the two format patches are dependent? just cause they will probably conflict separately? | 14:50 |
*** kong has quit IRC | 14:51 | |
jd__ | gordc: yeah just to avoid any kind of conflict | 14:53 |
jd__ | gordc: but functionnally they are not linked | 14:54 |
cdent | ryanpetrello: if you run tox -e functional, does it all work (I'm trying that now, but my puter is being slow) | 14:55 |
*** dhague has quit IRC | 14:56 | |
openstackgerrit | Julien Danjou proposed openstack/gnocchi: carbonara: replace msgpack encoding with struct for new measures https://review.openstack.org/371550 | 15:00 |
cdent | ryanpetrello: my guess is something has gone wrong with global conf | 15:07 |
openstackgerrit | Julien Danjou proposed openstack/gnocchi: storage: increase default processing delays https://review.openstack.org/371561 | 15:07 |
openstackgerrit | Julien Danjou proposed openstack/gnocchi: devstack-gate: do not create legacy resources https://review.openstack.org/371609 | 15:08 |
*** jkraj has quit IRC | 15:13 | |
*** rcernin has quit IRC | 15:15 | |
*** thorst_ has quit IRC | 15:16 | |
openstackgerrit | Julien Danjou proposed openstack/ceilometer: gabbi: import pipeline_cfg_file option before using it https://review.openstack.org/371613 | 15:16 |
jd__ | cdent: ryanpetrello: does that help ^^^ ? | 15:16 |
*** jkraj has joined #openstack-telemetry | 15:19 | |
cdent | jd__: did it move? | 15:20 |
cdent | jd__: or to put it another way: why did it stop working? | 15:20 |
jd__ | cdent: probably because it worked by chance and some import moved around | 15:21 |
jd__ | this whole CONF global var is sh** | 15:21 |
cdent | truly | 15:21 |
*** yprokule has quit IRC | 15:22 | |
*** thorst_ has joined #openstack-telemetry | 15:22 | |
*** thorst_ has quit IRC | 15:24 | |
*** thorst_ has joined #openstack-telemetry | 15:27 | |
*** jkraj has quit IRC | 15:30 | |
*** khushbu_ has joined #openstack-telemetry | 15:42 | |
openstackgerrit | Merged openstack/ceilometer: Update reno for stable/newton https://review.openstack.org/371585 | 15:43 |
cdent | ryanpetrello, jd__: something's weird. now with jd__'s change there failures in pecan looking for a controller | 15:44 |
cdent | I can't dig into this at the moment in any quick fashion: I'm at a conference and will be moving soon. | 15:45 |
*** thorst_ has quit IRC | 16:03 | |
*** donghao has quit IRC | 16:08 | |
*** thorst_ has joined #openstack-telemetry | 16:11 | |
*** thorst_ has quit IRC | 16:11 | |
*** efoley has quit IRC | 16:13 | |
*** thorst_ has joined #openstack-telemetry | 16:16 | |
*** cdent has quit IRC | 16:16 | |
openstackgerrit | Merged openstack/gnocchi: Allow to retry to delete a resource https://review.openstack.org/370783 | 16:27 |
*** agireud has quit IRC | 16:49 | |
*** agireud has joined #openstack-telemetry | 16:51 | |
*** thorst_ has quit IRC | 17:12 | |
*** amoralej is now known as amoralej|off | 17:24 | |
*** khushbu_ has quit IRC | 17:25 | |
*** rvasilets___ has joined #openstack-telemetry | 17:25 | |
openstackgerrit | Julien Danjou proposed openstack/gnocchi: carbonara: replace msgpack encoding with struct for new measures https://review.openstack.org/371550 | 17:28 |
openstackgerrit | Julien Danjou proposed openstack/gnocchi: carbonara: implement compressed format for BoundTimeSerie https://review.openstack.org/370849 | 17:28 |
*** dave-mcc_ has joined #openstack-telemetry | 18:12 | |
*** harlowja has joined #openstack-telemetry | 18:13 | |
*** dave-mccowan has quit IRC | 18:16 | |
*** jwcroppe has quit IRC | 18:18 | |
*** nicodemus_ has quit IRC | 18:22 | |
*** chlong_ has quit IRC | 18:27 | |
*** nicodemus_ has joined #openstack-telemetry | 18:34 | |
*** nicodemus_ has quit IRC | 19:02 | |
*** nicodemus_ has joined #openstack-telemetry | 19:16 | |
*** r-mibu has quit IRC | 19:16 | |
*** r-mibu has joined #openstack-telemetry | 19:17 | |
*** david-lyle has quit IRC | 19:30 | |
*** david-lyle has joined #openstack-telemetry | 19:30 | |
*** david-lyle has quit IRC | 19:43 | |
pradk | sileht, seeing this http://paste.openstack.org/show/580246/ | 19:48 |
*** fguillot has quit IRC | 19:49 | |
*** thorst_ has joined #openstack-telemetry | 19:52 | |
pshedimb | gordc, stevelle, llu: I'm working on https://review.openstack.org/#/c/368225/ | 19:52 |
pshedimb | And I made changes in pollsters | 19:52 |
pshedimb | pollster files, inspector files | 19:52 |
pshedimb | I'm not sure why but collector is failing | 19:52 |
pshedimb | AttributeError: 'DatabaseDispatcher' object has no attribute 'record_metering_data' | 19:54 |
pshedimb | When I add a debug point in collector.py, no error is showing up | 19:54 |
pshedimb | And debug point is also not being reached | 19:54 |
pshedimb | Which is strange | 19:55 |
pshedimb | Any idea what's happening | 19:55 |
pshedimb | ? | 19:55 |
*** thorst_ has quit IRC | 19:56 | |
*** openstackstatus has quit IRC | 19:58 | |
stevelle | pshedimb: confirming that is strange, "object has no attribute" | 19:59 |
*** thorst_ has joined #openstack-telemetry | 19:59 | |
*** openstackstatus has joined #openstack-telemetry | 20:00 | |
*** ChanServ sets mode: +v openstackstatus | 20:00 | |
pshedimb | stevelle, yeah. and surprising thing is when i add a pdb point before the method call, it works fine withour any error | 20:01 |
pshedimb | and pdb point is never reached :( | 20:01 |
stevelle | pshedimb: where is your point | 20:02 |
stevelle | my only thought right now is that you have tweaked your ceilometer.conf to direct meters to an event database dispatcher | 20:03 |
*** dave-mcc_ has quit IRC | 20:11 | |
*** AndyWojo has quit IRC | 20:16 | |
*** AndyWojo has joined #openstack-telemetry | 20:17 | |
pshedimb | Debug point before line 159 in collector.py | 20:19 |
pshedimb | I didn't make any kind of change in ceilometer.conf file | 20:19 |
pshedimb | stevelle, should I send you my changes? | 20:20 |
gordc | which test is failing? | 20:21 |
*** larainema has quit IRC | 20:24 | |
stevelle | pshedimb: ^ and you can publish a new patch set and mark it -1 Workflow so you can get more help | 20:32 |
*** paw has quit IRC | 20:33 | |
openstackgerrit | Prudhvi Rao Shedimbi proposed openstack/ceilometer: Compute agent can poll tx and rx errors and drops https://review.openstack.org/368225 | 20:57 |
pshedimb | stevelle, gordc llu: Published a patch | 21:01 |
*** Guest31676 has quit IRC | 21:17 | |
gordc | pshedimb: i don't know why you would get a record_metering_data error since your code doesn't touch it | 21:21 |
gordc | you do have a bad merge though. | 21:21 |
gordc | you should rebase on master | 21:21 |
*** vishwanathj has quit IRC | 21:22 | |
gordc | git checkout master && git pull | 21:22 |
pshedimb | gordc, I think I found my error | 21:32 |
pshedimb | Small typo | 21:32 |
gordc | pshedimb: the toughest mistake the find.lol good catch | 21:33 |
pshedimb | hahaa. Thanks to smurke to find the typo | 21:34 |
pshedimb | And thanks to gordc and stevelle as well | 21:34 |
stevelle | pshedimb: it almost always is a small typo :) | 21:35 |
pshedimb | I'll ping if I have some other issues :D | 21:35 |
stevelle | good catch smurke | 21:35 |
pshedimb | stevelle, hahaa | 21:35 |
smurke | :) | 21:36 |
*** paw has joined #openstack-telemetry | 21:39 | |
gordc | cool cool. will look later. heading offline. | 21:41 |
*** gordc has quit IRC | 21:41 | |
*** rbak has quit IRC | 21:56 | |
*** nicodemus_ has quit IRC | 22:29 | |
*** catintheroof has quit IRC | 22:31 | |
*** david-lyle has joined #openstack-telemetry | 22:48 | |
*** fguillot has joined #openstack-telemetry | 23:21 | |
*** rbak has joined #openstack-telemetry | 23:27 | |
*** rvasilets___ has quit IRC | 23:27 | |
*** rbak has quit IRC | 23:39 | |
*** fguillot is now known as fguillot_afk | 23:48 | |
*** fguillot_afk is now known as fguillot | 23:48 | |
*** zhurong has joined #openstack-telemetry | 23:53 | |
*** rbak has joined #openstack-telemetry | 23:54 | |
*** zhurong has quit IRC | 23:54 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!