*** diogogmt has quit IRC | 00:10 | |
*** ljxiash has joined #openstack-telemetry | 00:18 | |
*** annasort has joined #openstack-telemetry | 00:18 | |
*** ljxiash has quit IRC | 00:23 | |
*** ildikov has quit IRC | 00:48 | |
*** ildikov has joined #openstack-telemetry | 00:52 | |
*** cheneydc has joined #openstack-telemetry | 01:08 | |
openstackgerrit | liusheng proposed openstack/python-aodhclient: WIP: Add composite rule alarm support in aodhclient https://review.openstack.org/284022 | 01:18 |
---|---|---|
*** julim has joined #openstack-telemetry | 01:26 | |
*** thorst has joined #openstack-telemetry | 01:28 | |
*** thorst has quit IRC | 01:34 | |
*** thorst has joined #openstack-telemetry | 01:34 | |
openstackgerrit | liusheng proposed openstack/python-aodhclient: WIP: Add composite rule alarm support in aodhclient https://review.openstack.org/284022 | 01:35 |
*** thorst has quit IRC | 01:39 | |
*** ljxiash has joined #openstack-telemetry | 01:44 | |
*** Liuqing has joined #openstack-telemetry | 01:51 | |
r-mibu | pradk, gordc, hi - i would like to tempest code to migrate code into aodh in this cycle, but let me check the issue first | 01:57 |
*** diogogmt has joined #openstack-telemetry | 02:05 | |
*** liamji has joined #openstack-telemetry | 02:33 | |
*** Ephur has quit IRC | 02:36 | |
openstackgerrit | liusheng proposed openstack/python-aodhclient: WIP: Add composite rule alarm support in aodhclient https://review.openstack.org/284022 | 02:42 |
*** achatterjee has joined #openstack-telemetry | 02:45 | |
*** links has joined #openstack-telemetry | 02:47 | |
openstackgerrit | liusheng proposed openstack/aodh: Add releasenote for composite alarm feature https://review.openstack.org/284505 | 03:03 |
*** liusheng has left #openstack-telemetry | 03:04 | |
*** boris-42 has quit IRC | 03:04 | |
*** liusheng has joined #openstack-telemetry | 03:04 | |
*** diogogmt has quit IRC | 03:23 | |
*** diogogmt has joined #openstack-telemetry | 03:26 | |
*** links has quit IRC | 03:34 | |
*** Liuqing has quit IRC | 03:37 | |
*** ljxiash has quit IRC | 03:38 | |
*** Liuqing has joined #openstack-telemetry | 03:38 | |
*** ljxiash has joined #openstack-telemetry | 03:38 | |
*** ljxiash has quit IRC | 03:43 | |
*** diogogmt has quit IRC | 03:43 | |
*** chlong_ has quit IRC | 03:47 | |
*** chlong_ has joined #openstack-telemetry | 03:49 | |
*** sanjana_ has joined #openstack-telemetry | 04:07 | |
openstackgerrit | gordon chung proposed openstack/gnocchi: fix migration from v1.3 to v2.0 https://review.openstack.org/284528 | 04:21 |
*** Liuqing has quit IRC | 04:21 | |
*** Liuqing has joined #openstack-telemetry | 04:22 | |
*** ljxiash has joined #openstack-telemetry | 04:28 | |
*** links has joined #openstack-telemetry | 04:29 | |
*** ljxiash has quit IRC | 04:34 | |
*** ljxiash has joined #openstack-telemetry | 04:34 | |
*** ljxiash has quit IRC | 04:39 | |
*** yprokule has joined #openstack-telemetry | 04:50 | |
*** Liuqing has quit IRC | 05:08 | |
*** Liuqing has joined #openstack-telemetry | 05:08 | |
*** belmoreira has joined #openstack-telemetry | 05:13 | |
*** ljxiash has joined #openstack-telemetry | 05:14 | |
*** _nadya_ has joined #openstack-telemetry | 06:20 | |
liamji | liusheng: it seems that the jenkins fails again? | 06:30 |
liusheng | liamji: :( | 06:31 |
liusheng | did you find any hint ? | 06:31 |
liamji | http://logs.openstack.org/34/277434/13/check/gate-tempest-dsvm-ceilometer-mongodb-full/6ffedbd/console.html | 06:32 |
liamji | ERROR! Specified hosts options do not match any hosts | 06:32 |
*** liamji has quit IRC | 06:36 | |
*** liamji has joined #openstack-telemetry | 06:36 | |
liamji | liusheng: according to the log file, it has some authority problem to create the user. | 06:42 |
*** dave-mccowan has quit IRC | 06:45 | |
liusheng | liamji: create user? what's the related log | 06:46 |
*** ljxiash has quit IRC | 06:47 | |
*** Liuqing has quit IRC | 06:53 | |
*** belmoreira has quit IRC | 06:53 | |
*** ljxiash has joined #openstack-telemetry | 06:55 | |
*** _nadya_ has quit IRC | 07:30 | |
liamji | :liusheng 2016-02-25 06:54:37.024 | + die_if_not_set 627 swift_tenant_test1 'Failure creating swift_tenant_test1' | 07:34 |
liamji | http://logs.openstack.org/14/280514/10/check/gate-tempest-dsvm-ceilometer-mongodb-full/f6b67d9/logs/devstack-early.txt.gz | 07:35 |
*** ljxiash_ has joined #openstack-telemetry | 07:35 | |
*** ljxiash has quit IRC | 07:36 | |
openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/gnocchi: Add some resource types tests https://review.openstack.org/270419 | 07:36 |
openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/gnocchi: Implements resource attribute number https://review.openstack.org/270091 | 07:36 |
openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/gnocchi: Implements resource attribute bool https://review.openstack.org/270418 | 07:36 |
openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/gnocchi: Don't create Ceilometer resource types by default. https://review.openstack.org/270322 | 07:36 |
openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/gnocchi: Move legacy Ceilometer resource into indexer. https://review.openstack.org/270266 | 07:36 |
*** ljxiash_ has quit IRC | 07:49 | |
*** belmoreira has joined #openstack-telemetry | 07:51 | |
*** ljxiash has joined #openstack-telemetry | 07:52 | |
*** belmoreira has quit IRC | 07:58 | |
*** belmoreira has joined #openstack-telemetry | 08:00 | |
*** belmoreira has quit IRC | 08:06 | |
*** boris-42 has joined #openstack-telemetry | 08:51 | |
*** _nadya_ has joined #openstack-telemetry | 09:08 | |
*** yassine__ has joined #openstack-telemetry | 09:10 | |
*** ildikov has quit IRC | 09:15 | |
*** liamji has quit IRC | 09:38 | |
*** ljxiash has quit IRC | 09:40 | |
*** ljxiash has joined #openstack-telemetry | 09:40 | |
*** ljxiash_ has joined #openstack-telemetry | 09:44 | |
*** efoley has joined #openstack-telemetry | 09:44 | |
*** ljxiash has quit IRC | 09:45 | |
openstackgerrit | liusheng proposed openstack/python-aodhclient: Add composite rule alarm support in aodhclient https://review.openstack.org/284022 | 09:52 |
*** r-mibu has quit IRC | 09:57 | |
*** ildikov has joined #openstack-telemetry | 10:01 | |
*** cheneydc has quit IRC | 10:02 | |
*** efoley has quit IRC | 10:06 | |
*** efoley_ has joined #openstack-telemetry | 10:06 | |
*** r-mibu has joined #openstack-telemetry | 10:08 | |
*** ljxiash has joined #openstack-telemetry | 10:08 | |
*** ljxiash_ has quit IRC | 10:10 | |
openstackgerrit | Ilya Tyaptin proposed openstack/ceilometer: Add an update interval to compute discovery https://review.openstack.org/284322 | 10:17 |
*** ljxiash has quit IRC | 10:21 | |
*** ljxiash has joined #openstack-telemetry | 10:27 | |
*** ljxiash has quit IRC | 10:32 | |
*** pcaruana has joined #openstack-telemetry | 10:42 | |
*** cdent has joined #openstack-telemetry | 10:45 | |
*** belmoreira has joined #openstack-telemetry | 10:49 | |
*** achatterjee has quit IRC | 10:52 | |
*** efoley__ has joined #openstack-telemetry | 11:02 | |
*** efoley_ has quit IRC | 11:04 | |
*** cdent has quit IRC | 11:16 | |
*** efoley__ has quit IRC | 11:46 | |
*** cdent has joined #openstack-telemetry | 11:53 | |
*** thorst has joined #openstack-telemetry | 12:01 | |
*** Liuqing has joined #openstack-telemetry | 12:03 | |
*** idegtiarov has joined #openstack-telemetry | 12:07 | |
*** Liuqing has quit IRC | 12:16 | |
*** ljxiash has joined #openstack-telemetry | 12:40 | |
*** diogogmt has joined #openstack-telemetry | 12:46 | |
*** gordc has joined #openstack-telemetry | 12:54 | |
*** pcaruana has quit IRC | 12:55 | |
*** dave-mccowan has joined #openstack-telemetry | 12:56 | |
* cdent has transcended cattle | 12:57 | |
*** efoley__ has joined #openstack-telemetry | 12:57 | |
openstackgerrit | Ilya Tyaptin proposed openstack/ceilometer: Fix the typo in the gnocchiclient exception https://review.openstack.org/284704 | 12:59 |
ityaptin | cdent, sileht, jd__ ^^^ | 12:59 |
sileht | ityaptin, done | 13:00 |
ityaptin | sileht: Thanks! | 13:01 |
gordc | ityaptin: we need to bump requirements. if i remember correctly, that spelling mistake was in gnocchiclient originally. | 13:02 |
*** ildikov has quit IRC | 13:03 | |
*** efoley_ has joined #openstack-telemetry | 13:08 | |
*** pcaruana has joined #openstack-telemetry | 13:09 | |
*** diogogmt has quit IRC | 13:10 | |
*** pradk has joined #openstack-telemetry | 13:10 | |
*** pradk has quit IRC | 13:10 | |
*** efoley__ has quit IRC | 13:11 | |
openstackgerrit | ZhiQiang Fan proposed openstack/ceilometer: Add validation for polling_namespaces option https://review.openstack.org/284710 | 13:16 |
ityaptin | gordc: you think we should to add support of the NamedMetricAreadyExists and NamedMetricAlreadyExists ? | 13:26 |
ityaptin | Or only change requirements? | 13:26 |
*** links has quit IRC | 13:26 | |
*** liamji has joined #openstack-telemetry | 13:30 | |
gordc | ityaptin: we can probably just change requirements. seems easier and more productive | 13:31 |
*** nicodemus_ has joined #openstack-telemetry | 13:31 | |
ityaptin | gordc: I agree | 13:35 |
gordc | ityaptin: cool cool | 13:36 |
openstackgerrit | Ilya Tyaptin proposed openstack/ceilometer: Fix the typo in the gnocchiclient exception https://review.openstack.org/284704 | 13:40 |
ityaptin | gordc: ^^^ | 13:40 |
ityaptin | sileht: ^^^ | 13:40 |
sileht | ityaptin, it seems I have broken backward compat with 2.2.0 | 13:42 |
sileht | ityaptin, perhaps I should put back the mis-spelling exception | 13:42 |
openstackgerrit | Igor Degtiarov proposed openstack/ceilometer: bracketer event transformer https://review.openstack.org/266488 | 13:44 |
gordc | sileht: *shrugs* i think it's ok. ( because i +2 the change in client.lol) | 13:45 |
sileht | gordc, lol | 13:46 |
*** datravis has joined #openstack-telemetry | 13:46 | |
sileht | gordc, I'm sure I'm the guys that have done the mis-spelling | 13:46 |
*** datravis has left #openstack-telemetry | 13:46 | |
gordc | no comment :P | 13:46 |
*** yassine__ has quit IRC | 13:49 | |
openstackgerrit | Igor Degtiarov proposed openstack/ceilometer: bracketer event transformer https://review.openstack.org/266488 | 13:49 |
*** ildikov has joined #openstack-telemetry | 13:50 | |
ityaptin | sileht: Maybe you'll create an additional exception with mis-spelling in the gnocchiclient ? | 13:56 |
ityaptin | So, it will support backward compatibility, but allows to use a word with right spelling? | 13:57 |
sileht | ityaptin, yes | 13:57 |
*** ddieterly has joined #openstack-telemetry | 14:00 | |
*** jasonamyers has left #openstack-telemetry | 14:01 | |
ityaptin | sileht: :D I thought that you want to rename current exception to mis-spelled. I was in slowpoke mode) | 14:01 |
*** efoley_ has quit IRC | 14:01 | |
*** efoley_ has joined #openstack-telemetry | 14:07 | |
*** links has joined #openstack-telemetry | 14:07 | |
*** yassine__ has joined #openstack-telemetry | 14:24 | |
*** datravis has joined #openstack-telemetry | 14:33 | |
*** pradk has joined #openstack-telemetry | 14:42 | |
*** jwcroppe has joined #openstack-telemetry | 14:44 | |
liamji | gordc: Hi | 14:52 |
gordc | liamji: hi | 14:53 |
gordc | whatsup? | 14:53 |
*** rbak has joined #openstack-telemetry | 14:54 | |
liamji | gordc: some UT cases seem to fail. And it leads to the jenkins fails. | 14:54 |
liamji | http://logs.openstack.org/34/277434/13/check/gate-ceilometer-python27/2c16ebf/console.html | 14:54 |
gordc | liamji: it's related to latest gnocchiclient | 14:55 |
gordc | it's sileht's fault. :) | 14:56 |
sileht | always | 14:56 |
liamji | gordc: okey. The fix code is ready now? | 14:56 |
gordc | liamji: https://review.openstack.org/#/c/284704/2 | 14:57 |
gordc | that should fix it | 14:57 |
gordc | ityaptin: we need to drop the requirement for now i think. it won't pass requiremetns checks | 14:57 |
liamji | gordc: thanks. I will recheck my change set after the fix is merged. :) | 14:58 |
gordc | liamji: your patch is fine for Mitaka (the polling part at least) | 14:58 |
*** diogogmt has joined #openstack-telemetry | 14:59 | |
*** rbak_ has joined #openstack-telemetry | 15:00 | |
liamji | gordc: okey, thanks :) | 15:00 |
gordc | short meetings over at openstack-meeting | 15:00 |
ityaptin | gordc: Did requirements job failed? | 15:01 |
gordc | ityaptin: http://logs.openstack.org/04/284704/2/check/gate-ceilometer-requirements/601ad03/console.html#_2016-02-25_13_57_30_338 | 15:02 |
*** llu-laptop has joined #openstack-telemetry | 15:02 | |
*** rbak has quit IRC | 15:03 | |
*** llu-laptop has quit IRC | 15:03 | |
ityaptin | ack. So, I will update my patch. Should I set an update to the global-requirements or somebody else will do it? | 15:04 |
*** PsionTheory has joined #openstack-telemetry | 15:04 | |
*** llu-laptop has joined #openstack-telemetry | 15:04 | |
gordc | ityaptin: you can do it if you'd like :) | 15:05 |
*** llu-laptop has quit IRC | 15:05 | |
*** rickyrem has joined #openstack-telemetry | 15:08 | |
ityaptin | :) ok | 15:08 |
openstackgerrit | Ilya Tyaptin proposed openstack/ceilometer: Fix the typo in the gnocchiclient exception https://review.openstack.org/284704 | 15:09 |
*** llu-laptop has joined #openstack-telemetry | 15:10 | |
*** links has quit IRC | 15:12 | |
*** llu-laptop has quit IRC | 15:13 | |
*** ddieterly has quit IRC | 15:13 | |
*** liusheng_ has joined #openstack-telemetry | 15:14 | |
*** davidsha_ has joined #openstack-telemetry | 15:14 | |
*** KrishR has joined #openstack-telemetry | 15:17 | |
*** yprokule has quit IRC | 15:19 | |
*** efoley_ has quit IRC | 15:24 | |
*** PsionTheory has quit IRC | 15:26 | |
*** ddieterly has joined #openstack-telemetry | 15:47 | |
openstackgerrit | Merged openstack/aodh: Add composite rule alarm API support https://review.openstack.org/257722 | 15:48 |
*** ddieterly has quit IRC | 15:48 | |
*** ddieterl_ has joined #openstack-telemetry | 15:48 | |
*** liusheng_ has quit IRC | 15:58 | |
gordc | sileht: m-3 is next week | 16:00 |
sileht | gordc, m-3 == feature freeze ? | 16:00 |
gordc | yep | 16:00 |
sileht | gordc, which day ? | 16:00 |
gordc | we can keep adding features but we should have a good reason | 16:00 |
gordc | sileht: any day next week | 16:01 |
gordc | but again, assume gate is broken next week and nothing will merge | 16:01 |
gordc | we start tagging monday-thursady | 16:01 |
gordc | everyone usually does wednesday/thursday | 16:01 |
sileht | gordc, I don't known how will be the code change, so lets see how my works in going-on and what the gate health to take a decision | 16:02 |
sileht | gordc, on Monday I should have a idea on how the patch will be big | 16:02 |
gordc | sileht: ack. we can always make batching FFE if we think it's important | 16:02 |
gordc | but yeah, let me know | 16:02 |
sileht | I don't like that | 16:02 |
gordc | don't like FFE? | 16:03 |
sileht | yes | 16:03 |
sileht | originally I have thougth that bachting will be for next release, not mikita | 16:03 |
sileht | but we have release Gnocchi 2.0, so we can write the code now :) | 16:03 |
gordc | sileht: kk. let's assume defer unless you have it ready on monday | 16:03 |
sileht | that's sound reasonable | 16:04 |
gordc | idegtiarov_: we want to keep discussing online? or maybe mailing list? | 16:04 |
gordc | sileht: cool cool. thanks | 16:04 |
*** davidsha_ has quit IRC | 16:05 | |
*** diogogmt has quit IRC | 16:08 | |
*** datravis has left #openstack-telemetry | 16:11 | |
nicodemus_ | hello gordc | 16:13 |
openstackgerrit | Xia Linjuan proposed openstack/ceilometer: Enable the Load Balancer v2 for notification https://review.openstack.org/281777 | 16:14 |
gordc | nicodemus_: hey, whatsup? | 16:20 |
nicodemus_ | gordc: just wanted to let you know that your patch (https://review.openstack.org/#/c/284528/) worked, I just used it to upgrade our large deployment | 16:21 |
gordc | nicodemus_: oh nice! | 16:21 |
gordc | nicodemus_: i wasn't exactly sure, i needed to dig in to code to figure out what we were writing/storing in 1.3.x | 16:22 |
gordc | i'm not entirely sure that's the correct approach still but i think it does handle the migration part | 16:23 |
gordc | sileht: i don't know if you have any ideas. re: gnocchi storage migration. | 16:23 |
sileht | gordc, I have though that gnocchi-upgrade already do that | 16:25 |
sileht | gordc, it does | 16:25 |
gordc | it doesn't. it was a trick.lol | 16:25 |
sileht | nicodemus_, did you run gnocchi-upgrade before start metricd after the upgrade | 16:26 |
gordc | https://review.openstack.org/#/c/284528/ | 16:26 |
sileht | gordc, https://github.com/openstack/gnocchi/blob/master/gnocchi/storage/_carbonara.py#L310 | 16:27 |
nicodemus_ | sileht: yes, I had to run gnocchi-upgrade. So far the API is working without having to hack what I described here https://bugs.launchpad.net/gnocchi/+bug/1548367 | 16:27 |
openstack | Launchpad bug 1548367 in Gnocchi "Gnocchi-upgrade fails when moving to latest master" [High,Triaged] | 16:27 |
gordc | sileht: right. but if you look at TimeSerieArchive, it uses AggregatedTimeSerie. | 16:27 |
*** belmoreira has quit IRC | 16:27 | |
sileht | gordc, we do not support online migration | 16:27 |
nicodemus_ | once i finish upgrading the remaining APIs, I'll upgrade metricd (it's stopped now) | 16:28 |
sileht | http://gnocchi.xyz/install.html#upgrading | 16:28 |
gordc | so basically the test we have is writing a TimeSerieArchive of v2.0 AggregatedTimeSerie and then migrating it to V2.0 AggregatedTimeSerie | 16:28 |
sileht | touching carbonara.py looks not a good idea | 16:29 |
gordc | but in reality, it's a TimeSerieArchive of v1.3 AggregatedTimeSerie | 16:29 |
sileht | lol very good test ! | 16:29 |
gordc | yeah, and v1.3 -> v2.0 AggregatedTimeSerie has different fields/data | 16:29 |
sileht | gordc, _check_for_metric_upgrade must be fixed in this case | 16:30 |
gordc | yeah, i can basically move my hack to replace https://github.com/openstack/gnocchi/blob/master/gnocchi/storage/_carbonara.py#L310 | 16:30 |
gordc | sileht: that should do the same thing. | 16:30 |
gordc | sileht: i think big concern right now is there is no way to tell what is a v1.3 datapoint vs v2.0. | 16:32 |
gordc | right now i'm just basing it on fact we're missing 'first_timestamp' attribute | 16:32 |
sileht | gordc, before all aggregates was in object backend object, now each have an object | 16:34 |
sileht | in/in one | 16:34 |
sileht | aggregates/granularity | 16:35 |
gordc | all in one timeseriearchive you mean | 16:35 |
gordc | the aggregatedtimeserie format also changed from v1.3 and that's why i have the check for 'first_timestamp' | 16:37 |
gordc | i probably don't need it but i think we ever change format again, we might need a way to figure out what version it is | 16:37 |
sileht | gordc,agreed that will help a lot | 16:38 |
sileht | gordc, at least to bail out when something unexpected is read from the backend | 16:39 |
gordc | sileht: i'll try moving my hack into the upgrade method for now. i'm not sure it's completely right... | 16:39 |
gordc | but it's more right than now.lol | 16:39 |
sileht | gordc, I'm starting to see the issue | 16:40 |
sileht | we expect 'carbonara.TimeSerieArchive.unserialize(data)' to always return the new format | 16:40 |
gordc | yeah. i had to hack the test to write old format | 16:42 |
gordc | and then unserialise will explode | 16:42 |
gordc | my fix is good for v1.3->v2.0 but if there's another migration, i'm not sure how that will work. unless we tag and keep changing _check_for_metric_upgrade method every time we change format | 16:44 |
sileht | gordc, I think you just have to move your code in AggregatedTimeSerie.from_dict | 16:46 |
sileht | gordc, in fact we have two things to upgrade the object format (granularity split for panda measures) and the AggregatedTimeSerie description | 16:47 |
sileht | gordc, storage/_carbonara.py must handle the first one, | 16:47 |
sileht | gordc, I think it's ok to handle the seconds one in carbonara.py | 16:48 |
nicodemus_ | one question... after upgrading metricd to 2.0.1.dev1, when I start it it displays all config options and then freezes there, with no error message nor anything and it's not processing). Where should I start looking? | 16:48 |
nicodemus_ | I don't see it attempting to connect to the database (sniffing packets on the net interface) | 16:49 |
gordc | nicodemus_: um... that's with debugging turned on? | 16:51 |
nicodemus_ | gordc: yes | 16:51 |
gordc | nicodemus_: from this log, http://logs.openstack.org/54/282454/1/gate/gate-telemetry-dsvm-integration-gnocchi/88688ae/logs/screen-gnocchi-metricd.txt.gz | 16:53 |
gordc | what line does your metricd stop at? | 16:53 |
gordc | sileht: i'm not entirely sure how to split first and second parts | 16:54 |
nicodemus_ | gordc: this is the entire log output: http://paste.openstack.org/show/488226/ | 16:55 |
gordc | hmm... i wonder if that's some driver loading issues | 16:56 |
sileht | gordc, the old format was having block_size and back_window fields, are they useless to build the new format ? | 16:57 |
*** liamji has quit IRC | 16:58 | |
nicodemus_ | gordc: on this host I was using 1.3.1.dev280, I pulled master, uninstalled prev version with pip, installed the requirements, updated api-paste.ini and then installed gnocchi | 16:58 |
gordc | sileht: did we actually write block_size/back_window? i built the v1.3 format based on https://github.com/openstack/gnocchi/blob/stable/1.3/gnocchi/carbonara.py#L411 | 16:59 |
gordc | which has https://github.com/openstack/gnocchi/blob/stable/1.3/gnocchi/carbonara.py#L291-L296 | 16:59 |
sileht | gordc, https://github.com/openstack/gnocchi/blob/stable/1.3/gnocchi/carbonara.py#L291 | 16:59 |
sileht | https://github.com/openstack/gnocchi/blob/stable/1.3/gnocchi/carbonara.py#L217 | 17:00 |
sileht | https://github.com/openstack/gnocchi/blob/stable/1.3/gnocchi/carbonara.py#L129 | 17:00 |
sileht | gordc, we was using inheritance for that, so each sub class was adding thing into the dict | 17:00 |
gordc | AggregatedTimeSerie doesn't inherit BoundTimeSerie | 17:00 |
gordc | (i don't know the diff between both) | 17:01 |
sileht | oh right sorry | 17:01 |
gordc | no worries. i'm sort of guessing right now which is why i'm not sure what right fix is. | 17:02 |
sileht | gordc, so i would said, just move your fix into AggregatedTimeSerie.from_dict | 17:02 |
gordc | sileht: shoud i keep it in _check_for_metric_upgrade? from_dict i believe is called even when not migrating | 17:03 |
*** thorst is now known as thorst_afk | 17:04 | |
sileht | gordc, I think this is not a problem | 17:04 |
*** liusheng has quit IRC | 17:05 | |
*** liusheng has joined #openstack-telemetry | 17:05 | |
gordc | sileht: ack. i'll try that. | 17:06 |
*** cdent has quit IRC | 17:09 | |
*** _nadya_ has quit IRC | 17:10 | |
*** thorst_afk is now known as thorst | 17:12 | |
openstackgerrit | gordon chung proposed openstack/gnocchi: fix migration from v1.3 to v2.0 https://review.openstack.org/284528 | 17:14 |
gordc | nicodemus_: what did you upgrade in api-paste.ini (although it shouldn't matter) | 17:15 |
gordc | nicodemus_: maybe run ps -ef | grep metricd and kill all workers and try again? | 17:16 |
nicodemus_ | gordc: here are the two files: http://paste.openstack.org/show/488230/ | 17:16 |
nicodemus_ | gordc: I tried... changed backend to file just to see what happened, same behavior | 17:17 |
sileht | gordc, oh I just got why you put it in TimeSerieArchive... | 17:18 |
sileht | gordc, I think your first version was OK, and you don't have to check 'first_timestamp' it wasn't exists before | 17:19 |
sileht | sorry for my slow mind | 17:20 |
* sileht should stop working today | 17:20 | |
gordc | sileht: :) no worries. | 17:22 |
gordc | i guess they bothe work whether it's in timeseriearchive vs aggregatedtimeserie | 17:23 |
gordc | it's actually less code in aggregatedtimeserie. https://review.openstack.org/284528 | 17:23 |
*** KrishR has quit IRC | 17:27 | |
*** julim has quit IRC | 17:28 | |
nicodemus_ | gordc: would gnocchi-metricd 1.3.5.dev6 work if the API and the data were already migrated to 2.0.1? | 17:28 |
*** yassine__ has quit IRC | 17:30 | |
*** yassine__ has joined #openstack-telemetry | 17:31 | |
*** cdent has joined #openstack-telemetry | 17:35 | |
*** rbak__ has joined #openstack-telemetry | 17:35 | |
*** cdent has quit IRC | 17:36 | |
*** cdent has joined #openstack-telemetry | 17:36 | |
*** rbak_ has quit IRC | 17:38 | |
*** jwcroppe has quit IRC | 17:38 | |
*** jwcroppe has joined #openstack-telemetry | 17:39 | |
nicodemus_ | gordc: I got metricd to work, the indexer url was mysql:// instead of mysql+pymysql://. But sadly, metricd is stacktracing for each new measure: http://paste.openstack.org/show/488235/ | 17:42 |
*** jwcroppe has quit IRC | 17:43 | |
gordc | nicodemus_: and you migrated db? | 17:44 |
gordc | nicodemus_: i need to drop for a bit, going to eat and head to office. will be back later. | 17:44 |
*** gordc has quit IRC | 17:44 | |
nicodemus_ | gordc: yes, with the db migrated. Ok, talk to you later! | 17:45 |
*** ljxiash has quit IRC | 17:49 | |
*** jwcroppe has joined #openstack-telemetry | 17:58 | |
*** rbak__ has quit IRC | 18:05 | |
*** lsmola has quit IRC | 18:08 | |
*** julim has joined #openstack-telemetry | 18:10 | |
*** rickyrem has quit IRC | 18:13 | |
*** rbak has joined #openstack-telemetry | 18:15 | |
*** Ephur has joined #openstack-telemetry | 18:16 | |
*** annasort has quit IRC | 18:19 | |
*** Ephur has quit IRC | 18:20 | |
*** rbak has quit IRC | 18:21 | |
*** rbak has joined #openstack-telemetry | 18:21 | |
*** AndreiaKumpera has quit IRC | 18:25 | |
*** KrishR has joined #openstack-telemetry | 18:43 | |
*** gordc has joined #openstack-telemetry | 18:47 | |
openstackgerrit | Steve Lewis proposed openstack/ceilometer: Docs: Configure meters/events dispatch separately https://review.openstack.org/283282 | 18:51 |
nicodemus_ | is it possible that gnocchi-upgrade no longer works with any version prior to 2.0? | 18:55 |
gordc | nicodemus_: which version? | 18:56 |
nicodemus_ | just cloned 1.3, and gnocchi-upgrade tries to import "upgrade" from gnocchi.cli | 18:57 |
gordc | nicodemus_: i'm not sure there was any data format changes within 1.x | 18:57 |
nicodemus_ | but upgrade is on cli.py on 2.0, not 1.3 | 18:57 |
nicodemus_ | I'm trying to have gnocchi back up, so until metricd is able to resume processing metricd I wanted to re-deploy in 1.3 | 18:58 |
gordc | gnocchi upgrade doesn't exist in 1.3 | 18:58 |
nicodemus_ | and use gnocchi-upgrade command to create a new DB | 18:58 |
gordc | in 1.3 the command is gnocchi-dbsync | 18:58 |
gordc | i believe prior to 2.0, we only migrated indexer(sql) data. | 18:59 |
gordc | storage(ceph|swift|file) data didn't change. | 18:59 |
nicodemus_ | if I installed from git, where should I have gnocchi-dbsync? Because it's nowhere on the filesystem | 18:59 |
gordc | it should enable gnocchi-dbsync if you install | 19:00 |
gordc | same way it enabled gnocchi-upgrade when you installed 2.x | 19:01 |
nicodemus_ | is gnocchi-dbsync it available for download at any location? | 19:01 |
*** yassine__ has quit IRC | 19:01 | |
nicodemus_ | I only have gnocchi-upgrade :( | 19:01 |
nicodemus_ | with gnocchi (1.3.5.dev6) | 19:02 |
gordc | did you pip uninstall gnocchi? | 19:04 |
nicodemus_ | yes | 19:04 |
gordc | you shouldn't have gnocchi-upgrade if you actually removed it | 19:04 |
nicodemus_ | I just uninstalled and I still have it, let me delete it by hand | 19:06 |
gordc | i ususally just keep running pip uninstall until it says doesn't exist. i find sometimes i have multiple copies installed | 19:07 |
nicodemus_ | you were right... had to run pip uninstall again O.o | 19:09 |
gordc | brute force always works best | 19:10 |
*** rbak_ has joined #openstack-telemetry | 19:20 | |
*** rbak has quit IRC | 19:20 | |
*** rickyrem has joined #openstack-telemetry | 19:22 | |
nicodemus_ | gordc: I promise I won't keep bugging you much more (at least for today :) with the 2.0 deploy w/ migrated db, metricd is still showing the overflowerror | 19:25 |
gordc | nicodemus_: can you patch the system again. http://paste.openstack.org/show/488255/ | 19:33 |
*** _nadya_ has joined #openstack-telemetry | 19:35 | |
openstackgerrit | Merged openstack/ceilometer: Fix the typo in the gnocchiclient exception https://review.openstack.org/284704 | 19:35 |
*** ddieterl_ has quit IRC | 19:37 | |
*** ddieterly has joined #openstack-telemetry | 19:39 | |
*** rickyrem has quit IRC | 19:40 | |
nicodemus_ | gordc: here's the log extract: http://paste.openstack.org/show/488258/ | 19:42 |
*** rickyrem has joined #openstack-telemetry | 19:43 | |
gordc | can you take a larger chunk before and after | 19:43 |
gordc | just so i can figure out which value is actually throwing error | 19:43 |
nicodemus_ | sure, I'm filtering the previous 10 lines and the 50 lines after like so: tail -f gnocchi-metricd.log | grep -A 50 -B 10 "Error processing new measures" | 19:44 |
nicodemus_ | would 100 before and 100 after be enough? | 19:44 |
gordc | sure | 19:44 |
*** ddieterl_ has joined #openstack-telemetry | 19:44 | |
nicodemus_ | gordc: here's the new log: http://paste.openstack.org/show/488260/ | 19:47 |
*** rickyrem has quit IRC | 19:47 | |
gordc | hmm.. i wonder how the hell it's getting DatetimeIndex instead of a timestamp | 19:49 |
*** ddieterl_ is now known as ddieterly_ | 19:49 | |
gordc | what happens if you re-run gnocchi-upgrade | 19:49 |
nicodemus_ | gordc: one thing I didn't mention, I made a change on the archive policy | 19:50 |
*** krotscheck is now known as krotscheck_dcm | 19:50 | |
gordc | hmm.. i don't think that should be an issue assuming it was done through api | 19:51 |
nicodemus_ | gordc: I changed the first definition on the default policy, from "points: 1440, granularity: 0:01:00, timespan: 1 day" to "points: 288, granularity: 0:05:00, timespan: 1 day" | 19:51 |
nicodemus_ | gordc: but didn't empty the pool or anything | 19:52 |
nicodemus_ | gordc: let me try re-running gnocchi-upgrae | 19:52 |
openstackgerrit | ZhiQiang Fan proposed openstack/ceilometer: Add validation for polling_namespaces option https://review.openstack.org/284710 | 19:53 |
gordc | nicodemus_: i have a meeting with manager. i'll biab. if you can, can you include the above comments in the bug report | 19:56 |
nicodemus_ | sure, thanks a lot! | 19:56 |
*** ljxiash has joined #openstack-telemetry | 20:01 | |
*** ddieterl_ has joined #openstack-telemetry | 20:03 | |
*** ddieterly has quit IRC | 20:05 | |
*** ddieterly_ has quit IRC | 20:05 | |
*** ljxiash has quit IRC | 20:05 | |
*** annasort has joined #openstack-telemetry | 20:15 | |
*** thorst is now known as thorst_afk | 20:38 | |
*** jwcroppe has quit IRC | 20:40 | |
*** jwcroppe has joined #openstack-telemetry | 20:40 | |
*** jwcroppe has quit IRC | 20:45 | |
*** Ephur has joined #openstack-telemetry | 20:47 | |
*** _nadya_ has quit IRC | 20:52 | |
*** thorst_afk is now known as thorst | 20:53 | |
*** ddieterl_ has quit IRC | 21:02 | |
*** vishwanathj has quit IRC | 21:04 | |
*** ddieterly has joined #openstack-telemetry | 21:04 | |
openstackgerrit | Merged openstack/python-ceilometerclient: Trivial: remove `None` as a redundant argument to dict.get() https://review.openstack.org/282699 | 21:07 |
*** ddieterly has quit IRC | 21:09 | |
*** ildikov has quit IRC | 21:16 | |
*** annasort has quit IRC | 21:25 | |
*** annasort has joined #openstack-telemetry | 21:28 | |
*** thorst is now known as thorst_afk | 21:39 | |
*** thorst_afk has quit IRC | 21:43 | |
*** thorst_afk has joined #openstack-telemetry | 21:43 | |
*** annasort has quit IRC | 21:44 | |
*** annasort has joined #openstack-telemetry | 21:50 | |
*** thorst_afk has quit IRC | 21:52 | |
gordc | nicodemus_: i'm not sure i'll get around to looking at your issue today. did gnocchi-upgrade do anything? | 21:53 |
*** ddieterly has joined #openstack-telemetry | 21:53 | |
nicodemus_ | gordc: not much, at least nothing different :( | 21:53 |
gordc | nicodemus_: kk. will try to look tomorrow | 21:54 |
nicodemus_ | gordc: np, thanks! :) | 21:54 |
*** nicodemus_ has quit IRC | 21:56 | |
*** julim has quit IRC | 22:04 | |
*** pradk has quit IRC | 22:09 | |
*** pradk has joined #openstack-telemetry | 22:09 | |
*** annasort has quit IRC | 22:43 | |
*** thorst_afk has joined #openstack-telemetry | 22:49 | |
*** thorst_afk has quit IRC | 22:57 | |
*** cdent has quit IRC | 23:09 | |
gordc | cool everything is broken. time to go home | 23:29 |
*** gordc has quit IRC | 23:29 | |
*** pradk has quit IRC | 23:39 | |
*** ddieterly has quit IRC | 23:47 | |
*** thorst_afk has joined #openstack-telemetry | 23:55 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!