| *** xmltok has joined #openstack-ceilometer | 00:10 | |
| *** ssakhamuri has joined #openstack-ceilometer | 00:42 | |
| *** stevelle_ is now known as stevelle | 01:00 | |
| *** Qiming has joined #openstack-ceilometer | 01:23 | |
| *** Qiming has quit IRC | 01:37 | |
| *** jaypipes has quit IRC | 01:41 | |
| *** nosnos has joined #openstack-ceilometer | 01:42 | |
| *** jaypipes has joined #openstack-ceilometer | 01:42 | |
| *** gordc has joined #openstack-ceilometer | 01:44 | |
| *** sbfox has joined #openstack-ceilometer | 01:45 | |
| *** shakamunyi has quit IRC | 02:01 | |
| openstackgerrit | Ren Qiaowei proposed a change to openstack/ceilometer: XenAPI support: Changes for cpu_util https://review.openstack.org/112453 | 02:13 |
|---|---|---|
| openstackgerrit | Ren Qiaowei proposed a change to openstack/ceilometer: XenAPI support: Memory Usage https://review.openstack.org/114867 | 02:16 |
| *** gordc has quit IRC | 02:24 | |
| *** sdake has quit IRC | 02:24 | |
| *** julim has joined #openstack-ceilometer | 02:30 | |
| openstackgerrit | Ren Qiaowei proposed a change to openstack/ceilometer: XenAPI support: Update measurements documentation https://review.openstack.org/114874 | 02:33 |
| *** gordc has joined #openstack-ceilometer | 02:34 | |
| *** gordc has quit IRC | 02:34 | |
| *** shakamunyi has joined #openstack-ceilometer | 02:34 | |
| *** harlowja is now known as harlowja_away | 02:35 | |
| *** julim has quit IRC | 02:37 | |
| *** shakamunyi has quit IRC | 02:41 | |
| *** shakamunyi has joined #openstack-ceilometer | 02:42 | |
| *** alexpilotti has quit IRC | 02:48 | |
| *** sbfox has quit IRC | 02:52 | |
| *** sbfox has joined #openstack-ceilometer | 03:14 | |
| *** sbfox1 has joined #openstack-ceilometer | 03:15 | |
| *** sbfox has quit IRC | 03:15 | |
| *** deepthi has joined #openstack-ceilometer | 03:25 | |
| *** cmyster has joined #openstack-ceilometer | 03:34 | |
| *** cmyster has joined #openstack-ceilometer | 03:34 | |
| *** sdake has joined #openstack-ceilometer | 03:36 | |
| *** deepthi has quit IRC | 03:39 | |
| *** ccrouch has joined #openstack-ceilometer | 03:46 | |
| *** deepthi has joined #openstack-ceilometer | 03:47 | |
| *** ccrouch has quit IRC | 04:20 | |
| *** deepthi has quit IRC | 05:20 | |
| *** ssakhamuri has quit IRC | 05:26 | |
| *** sbfox1 has quit IRC | 05:31 | |
| *** deepthi has joined #openstack-ceilometer | 05:43 | |
| *** yatin has joined #openstack-ceilometer | 05:46 | |
| *** idegtiarov has joined #openstack-ceilometer | 05:52 | |
| openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/ceilometer: Imported Translations from Transifex https://review.openstack.org/115508 | 06:09 |
| *** ildikov_ has quit IRC | 06:18 | |
| openstackgerrit | Lianhao Lu proposed a change to openstack/ceilometer: Added new hardware inspector interface https://review.openstack.org/108938 | 06:26 |
| *** shakamunyi has quit IRC | 06:43 | |
| openstackgerrit | Ren Qiaowei proposed a change to openstack/ceilometer: XenAPI support: Changes for networking metrics https://review.openstack.org/114868 | 06:43 |
| openstackgerrit | Ren Qiaowei proposed a change to openstack/ceilometer: XenAPI support: Disk rates https://review.openstack.org/114873 | 06:44 |
| openstackgerrit | Ren Qiaowei proposed a change to openstack/ceilometer: XenAPI support: Update measurements documentation https://review.openstack.org/114874 | 06:44 |
| *** shakamunyi has joined #openstack-ceilometer | 06:44 | |
| *** ildikov has joined #openstack-ceilometer | 06:46 | |
| *** ifarkas has joined #openstack-ceilometer | 06:47 | |
| *** shakamunyi has quit IRC | 06:47 | |
| *** shakamunyi has joined #openstack-ceilometer | 06:48 | |
| *** ildikov_ has joined #openstack-ceilometer | 06:48 | |
| openstackgerrit | Lianhao Lu proposed a change to openstack/ceilometer: Rebase hardware pollsters to use new inspector interface https://review.openstack.org/109207 | 06:49 |
| *** ildikov has quit IRC | 06:49 | |
| *** ildikov has joined #openstack-ceilometer | 06:52 | |
| *** ildikov_ has quit IRC | 06:53 | |
| *** idegtiarov has quit IRC | 07:10 | |
| *** _nadya_ has joined #openstack-ceilometer | 07:13 | |
| *** _nadya_ has quit IRC | 07:16 | |
| *** sdake has quit IRC | 07:17 | |
| *** idegtiarov has joined #openstack-ceilometer | 07:23 | |
| *** sdake has joined #openstack-ceilometer | 07:35 | |
| *** sdake has quit IRC | 08:08 | |
| *** ildikov_ has joined #openstack-ceilometer | 08:09 | |
| *** ildikov has quit IRC | 08:12 | |
| *** taplax has quit IRC | 08:17 | |
| *** sdake has joined #openstack-ceilometer | 08:19 | |
| *** sdake_ has joined #openstack-ceilometer | 08:20 | |
| *** shakamunyi has quit IRC | 08:25 | |
| openstackgerrit | Igor Degtiarov proposed a change to openstack/ceilometer: Improving of configuration.rst https://review.openstack.org/115543 | 08:30 |
| *** IvanBerezovskiy has joined #openstack-ceilometer | 08:32 | |
| *** Dafna has quit IRC | 08:33 | |
| *** Dafna has joined #openstack-ceilometer | 08:34 | |
| *** yatin has quit IRC | 08:43 | |
| *** Longgeek has joined #openstack-ceilometer | 08:46 | |
| *** sdake has quit IRC | 08:47 | |
| *** Longgeek has quit IRC | 08:48 | |
| *** _nadya_ has joined #openstack-ceilometer | 08:48 | |
| *** Longgeek has joined #openstack-ceilometer | 08:48 | |
| openstackgerrit | Nejc Saje proposed a change to openstack/ceilometer: Update tox.ini pep8 config to ignore i18n functions https://review.openstack.org/115550 | 08:52 |
| *** ifarkas has quit IRC | 08:52 | |
| *** ifarkas has joined #openstack-ceilometer | 08:54 | |
| openstackgerrit | Nejc Saje proposed a change to openstack/ceilometer: Central agent work-load partitioning https://review.openstack.org/113549 | 08:58 |
| *** yatin has joined #openstack-ceilometer | 09:00 | |
| nsaje_ | jd__, sileht, ildikov_ : can you guys check out https://review.openstack.org/#/c/115550/ ? tox pep8 fix for the new translation functions from Oslo that several incoming patches use | 09:04 |
| sileht | nsaje_, done | 09:05 |
| ildikov_ | nsaje_: done | 09:05 |
| nsaje_ | wow, such fast, so speed :) | 09:06 |
| DinaBelova | nsaje_, I see these changes in your central agent HA change as well... that looks as a mistake? | 09:06 |
| nsaje_ | DinaBelova: I included them so they will pass the check, since this patch isn't merged yet | 09:07 |
| DinaBelova | a-ha, ok :) btw, it looks like it'll be merged really soon :) | 09:07 |
| nsaje_ | DinaBelova: I'll take it out if I need another patch for central agent, otherwise they'll get ignored at the rebase | 09:07 |
| nsaje_ | *merge | 09:08 |
| *** taplax has joined #openstack-ceilometer | 09:08 | |
| DinaBelova | nsaje_, okay, got it | 09:08 |
| *** yatin has quit IRC | 09:39 | |
| *** taplax_ has joined #openstack-ceilometer | 09:47 | |
| openstackgerrit | Sanja Nosan proposed a change to openstack/ceilometer: Fix typos in transformer docstrings https://review.openstack.org/115559 | 09:55 |
| openstackgerrit | Igor Degtiarov proposed a change to openstack/ceilometer: Improving of configuration.rst https://review.openstack.org/115543 | 10:04 |
| *** openstackgerrit has quit IRC | 10:10 | |
| *** yatin has joined #openstack-ceilometer | 10:16 | |
| *** jaypipes has quit IRC | 10:39 | |
| *** jaypipes has joined #openstack-ceilometer | 10:40 | |
| *** nosnos has quit IRC | 10:57 | |
| *** ildikov_ has quit IRC | 11:10 | |
| *** alexpilotti has joined #openstack-ceilometer | 11:18 | |
| *** cdent has joined #openstack-ceilometer | 11:27 | |
| *** cmyster has quit IRC | 11:37 | |
| *** idegtiarov has quit IRC | 11:43 | |
| *** ildikov has joined #openstack-ceilometer | 11:44 | |
| *** idegtiarov has joined #openstack-ceilometer | 11:44 | |
| *** alexpilotti has quit IRC | 11:48 | |
| *** _nadya_ has quit IRC | 11:58 | |
| *** ccrouch has joined #openstack-ceilometer | 11:59 | |
| *** deepthi has quit IRC | 12:02 | |
| *** alexpilotti has joined #openstack-ceilometer | 12:03 | |
| * cdent takes a walk | 12:08 | |
| *** cdent has quit IRC | 12:08 | |
| *** alexpilotti has quit IRC | 12:16 | |
| *** alexpilotti has joined #openstack-ceilometer | 12:37 | |
| *** cdent has joined #openstack-ceilometer | 12:39 | |
| *** tongli has joined #openstack-ceilometer | 12:42 | |
| *** gordc has joined #openstack-ceilometer | 12:42 | |
| *** cdent has quit IRC | 12:43 | |
| *** cdent_ has joined #openstack-ceilometer | 12:43 | |
| *** yatin has quit IRC | 13:00 | |
| *** ildikov has quit IRC | 13:04 | |
| *** thomasem has joined #openstack-ceilometer | 13:04 | |
| *** julim has joined #openstack-ceilometer | 13:09 | |
| *** dmsimard_away is now known as dmsimard | 13:18 | |
| *** _nadya_ has joined #openstack-ceilometer | 13:26 | |
| *** theanalyst_ is now known as theanalyst | 13:26 | |
| *** pradk has joined #openstack-ceilometer | 13:48 | |
| *** rbowen has joined #openstack-ceilometer | 13:50 | |
| *** rbowen has quit IRC | 13:53 | |
| *** zul has joined #openstack-ceilometer | 13:54 | |
| *** Longgeek has quit IRC | 13:55 | |
| *** Longgeek has joined #openstack-ceilometer | 13:56 | |
| *** zul has quit IRC | 14:01 | |
| *** cdent_ is now known as cdent | 14:02 | |
| *** deepthi has joined #openstack-ceilometer | 14:04 | |
| *** zul has joined #openstack-ceilometer | 14:14 | |
| *** zul has quit IRC | 14:15 | |
| *** zul has joined #openstack-ceilometer | 14:17 | |
| *** ccrouch has quit IRC | 14:17 | |
| gordc | pradk: regarding the duplicate meter issue. i think it's really because the discovery is defined in a misleading place in pipeline.yaml | 14:19 |
| *** ildikov has joined #openstack-ceilometer | 14:19 | |
| *** deepthi has quit IRC | 14:19 | |
| *** zul has quit IRC | 14:20 | |
| pradk | gordc, yea i was debugging the issue yesterday and realized the same.. will have a patch in a bit i think | 14:21 |
| gordc | pradk: ok. you planning on changing how we define discovery in pipeline or something smaller? | 14:21 |
| *** zul has joined #openstack-ceilometer | 14:21 | |
| gordc | or bigger. | 14:22 |
| *** ccrouch has joined #openstack-ceilometer | 14:22 | |
| pradk | you mean in the source right? | 14:22 |
| *** rwsu has quit IRC | 14:24 | |
| gordc | right. well right now we define discovery in a specific source... but the discovery ends up being more global (affecting all sources) | 14:24 |
| gordc | if i understand correctly... | 14:24 |
| *** rbowen has joined #openstack-ceilometer | 14:25 | |
| *** _nadya_ has quit IRC | 14:29 | |
| *** nadya has joined #openstack-ceilometer | 14:29 | |
| *** nadya is now known as Guest42262 | 14:30 | |
| pradk | gordc, yea the patch i was trying was to make the discovery load per source.. but i'm not entirely confident about it, as its breaking a bunch of tests which has a different assumption. | 14:30 |
| pradk | gordc, was thinking if i should wait for eglynn to get back.. | 14:30 |
| pradk | gordc, do you have something working already? | 14:31 |
| gordc | pradk: sounds like a good idea. unless other cores here have thoughts? | 14:31 |
| gordc | i haven't work on anything yet. just been doing reviews | 14:31 |
| gordc | other cores/contributors i should. ^ | 14:32 |
| *** ifarkas has quit IRC | 14:34 | |
| *** rwsu has joined #openstack-ceilometer | 14:37 | |
| *** zul has quit IRC | 14:37 | |
| *** zul has joined #openstack-ceilometer | 14:41 | |
| gordc | DinaBelova: just checking, are you looking into why telemetry tempest tests return Conflict: An object with that identifier already exists? | 14:45 |
| *** ifarkas has joined #openstack-ceilometer | 14:46 | |
| *** Guest42262 is now known as _nadya_ | 14:47 | |
| *** zul has quit IRC | 14:49 | |
| *** aviau has quit IRC | 14:53 | |
| *** ildikov has quit IRC | 14:59 | |
| *** sbfox has joined #openstack-ceilometer | 15:00 | |
| *** shakamunyi has joined #openstack-ceilometer | 15:00 | |
| *** sbfox has quit IRC | 15:00 | |
| *** sbfox has joined #openstack-ceilometer | 15:01 | |
| *** ifarkas has quit IRC | 15:02 | |
| *** sbfox has quit IRC | 15:07 | |
| *** sbfox has joined #openstack-ceilometer | 15:08 | |
| *** aviau has joined #openstack-ceilometer | 15:08 | |
| *** deepthi has joined #openstack-ceilometer | 15:09 | |
| *** openstackgerrit has joined #openstack-ceilometer | 15:12 | |
| *** rbowen has quit IRC | 15:13 | |
| *** Longgeek_ has joined #openstack-ceilometer | 15:14 | |
| openstackgerrit | Julien Danjou proposed a change to openstack/ceilometer: Switch to oslotest https://review.openstack.org/115679 | 15:15 |
| openstackgerrit | A change was merged to openstack/ceilometer: Fix typos in transformer docstrings https://review.openstack.org/115559 | 15:16 |
| *** Longgeek has quit IRC | 15:17 | |
| *** mrmoje has joined #openstack-ceilometer | 15:21 | |
| openstackgerrit | Julien Danjou proposed a change to openstack/ceilometer: Switch to oslotest https://review.openstack.org/115679 | 15:23 |
| openstackgerrit | Julien Danjou proposed a change to openstack/ceilometer: Remove oslo middleware https://review.openstack.org/115681 | 15:28 |
| *** sdake has joined #openstack-ceilometer | 15:31 | |
| *** zul has joined #openstack-ceilometer | 15:35 | |
| openstackgerrit | Julien Danjou proposed a change to openstack/ceilometer: Switch to oslo.utils https://review.openstack.org/115214 | 15:36 |
| *** deepthi has quit IRC | 15:37 | |
| *** ildikov has joined #openstack-ceilometer | 15:40 | |
| *** chuck_ has joined #openstack-ceilometer | 15:42 | |
| *** sbfox has quit IRC | 15:43 | |
| *** zul has quit IRC | 15:45 | |
| openstackgerrit | Igor Degtiarov proposed a change to openstack/ceilometer: Adding comparison operators in query for event traits https://review.openstack.org/112582 | 15:51 |
| *** sbfox has joined #openstack-ceilometer | 15:54 | |
| *** sbfox has quit IRC | 15:54 | |
| *** sbfox has joined #openstack-ceilometer | 15:54 | |
| *** atan8 has joined #openstack-ceilometer | 15:58 | |
| openstackgerrit | A change was merged to openstack/ceilometer: Add upgrading alarm storage in dbsync https://review.openstack.org/115596 | 15:59 |
| *** aviau has quit IRC | 15:59 | |
| *** sdake has quit IRC | 16:06 | |
| *** IvanBerezovskiy has left #openstack-ceilometer | 16:07 | |
| *** aviau has joined #openstack-ceilometer | 16:12 | |
| *** k4n0 has quit IRC | 16:25 | |
| *** idegtiarov has quit IRC | 16:31 | |
| *** pradk has quit IRC | 16:36 | |
| *** ildikov has quit IRC | 16:39 | |
| *** mrmoje has quit IRC | 16:55 | |
| *** pradk has joined #openstack-ceilometer | 17:02 | |
| *** idegtiarov has joined #openstack-ceilometer | 17:03 | |
| *** alexpilotti has quit IRC | 17:12 | |
| *** chuck_ has quit IRC | 17:15 | |
| *** zul has joined #openstack-ceilometer | 17:18 | |
| *** harlowja_away is now known as harlowja | 17:19 | |
| *** _nadya_ has quit IRC | 17:21 | |
| openstackgerrit | Eric Pendergrass proposed a change to openstack/ceilometer: Placeholder commit for WIP https://review.openstack.org/115717 | 17:26 |
| *** sbfox has quit IRC | 17:27 | |
| *** sbfox has joined #openstack-ceilometer | 17:27 | |
| *** alexpilotti has joined #openstack-ceilometer | 17:42 | |
| *** sbfox has quit IRC | 17:42 | |
| *** ccrouch has quit IRC | 17:42 | |
| *** claudiub has joined #openstack-ceilometer | 17:50 | |
| *** zul has quit IRC | 17:51 | |
| *** _nadya_ has joined #openstack-ceilometer | 17:52 | |
| *** sbfox has joined #openstack-ceilometer | 17:57 | |
| *** _nadya_ has quit IRC | 18:02 | |
| *** _nadya_ has joined #openstack-ceilometer | 18:02 | |
| *** rbowen has joined #openstack-ceilometer | 18:09 | |
| *** PsionTheory has joined #openstack-ceilometer | 18:14 | |
| *** harlowja has quit IRC | 18:20 | |
| *** harlowja_ has joined #openstack-ceilometer | 18:20 | |
| *** clarkb has joined #openstack-ceilometer | 18:27 | |
| clarkb | hello I am bumping up against https://bugs.launchpad.net/ceilometer/+bug/1359049 and wondering if anyone knows why the code loads the driver for sqlalchemy. Can we just let sqlalchemy do that for itself? | 18:28 |
| clarkb | if not, is the solution to split the scheme on + if + in scheme and load scheme.split('+')[1] as the driver? | 18:29 |
| gordc | clarkb: we use stevedore so we know which driver to load (sql, mongo, hbase, db2)... | 18:31 |
| clarkb | gordc: right but it doesn't work properly | 18:31 |
| gordc | unfortunately it seems it's not smart enough to figure out unique sql combinations | 18:31 |
| clarkb | gordc: because if you specify something like mysql+pymysql or mysql+mysqldb or postgres+psycopg it fails | 18:32 |
| gordc | yeah... is that the typical combination it's specified in? ie. mysql/postgresl+<adapter>? | 18:33 |
| clarkb | gordc: yup | 18:33 |
| gordc | clarkb: if it's specified in some standardised way, i'm sure it's an easy tweak on ceilometer's end | 18:34 |
| openstackgerrit | Phil Neal proposed a change to openstack/ceilometer: Document the standard for PaaS service notifications https://review.openstack.org/113396 | 18:34 |
| gordc | cool cool | 18:34 |
| clarkb | gordc: sqlalchemy documents the combos and if you leave out the + it uses a default driver | 18:34 |
| clarkb | which I think just happens to work for mysql and postgres out of chance | 18:34 |
| gordc | clarkb: i see... i can take a look at it. unless you feel like diving in to ceilo code. | 18:34 |
| clarkb | gordc: I am happy if others want to look at it, I was willing to look too but I will definitely be slower | 18:35 |
| gordc | clarkb: deferred to you! | 18:35 |
| gordc | o | 18:35 |
| *** zul has joined #openstack-ceilometer | 18:35 | |
| clarkb | kk I will ping if I get lost | 18:35 |
| gordc | clarkb: awesome. thanks for helping out :) | 18:36 |
| *** mrmoje has joined #openstack-ceilometer | 18:36 | |
| gordc | clarkb: not trying to force work on you. just hinting... not forcing. :) but yeah, if anything, feel free to defer back to me. | 18:38 |
| clarkb | yup no problem | 18:39 |
| clarkb | do you think we could have sqlalchemy laod the driver when it is sql and let stevedore do it for the other stores? | 18:39 |
| clarkb | or would it be better to keep using stevedore and haev it understand how to load with a + in the scheme? | 18:39 |
| gordc | clarkb: hmm.. good question. my initial thought was just to parse for +, attempt to match against the generic defined mysql/postgres definition we have in setup... | 18:43 |
| gordc | and then let sqla figure out how to fully parse it... | 18:43 |
| gordc | but i'm not sure if there's precedence elsewhere... | 18:44 |
| *** idegtiarov has quit IRC | 18:46 | |
| clarkb | no best I could tell ceilometer is the only project doing this | 18:47 |
| gordc | clarkb: taking a look at marconi.. i know they also have sql/mongo options... it's could also be a bug there too. | 18:48 |
| *** idegtiarov has joined #openstack-ceilometer | 18:49 | |
| gordc | yeah, doesn't seem like marconi does anything to handle it. | 18:52 |
| *** mrmoje has quit IRC | 18:52 | |
| gordc | i would probably go with your first option. avoids the complexity of having ceilometer figure out correct mappings. | 18:53 |
| *** sdake has joined #openstack-ceilometer | 18:56 | |
| *** sdake has quit IRC | 18:56 | |
| *** sdake has joined #openstack-ceilometer | 18:56 | |
| *** Fabio_ has joined #openstack-ceilometer | 18:56 | |
| *** rbowen has quit IRC | 18:57 | |
| openstackgerrit | Igor Degtiarov proposed a change to openstack/ceilometer: [Hbase] Adds new functions that is redesign keys for data storing in HBase https://review.openstack.org/115277 | 18:59 |
| *** zul has quit IRC | 19:05 | |
| *** shardy is now known as shardy_afk | 19:06 | |
| *** cdent_ has joined #openstack-ceilometer | 19:07 | |
| *** cdent has quit IRC | 19:07 | |
| *** cdent_ is now known as cdent | 19:07 | |
| *** zul has joined #openstack-ceilometer | 19:17 | |
| *** claudiub has quit IRC | 19:21 | |
| *** pradk is now known as prad | 19:26 | |
| *** amalagon has quit IRC | 19:28 | |
| *** _nadya_ has quit IRC | 19:35 | |
| *** mrutkows has joined #openstack-ceilometer | 19:53 | |
| *** _nadya_ has joined #openstack-ceilometer | 19:54 | |
| *** harlowja_ is now known as harlowja_away | 19:58 | |
| *** _nadya_ has quit IRC | 19:59 | |
| *** rbowen has joined #openstack-ceilometer | 20:01 | |
| *** sbfox1 has joined #openstack-ceilometer | 20:08 | |
| *** sbfox has quit IRC | 20:08 | |
| *** zul has quit IRC | 20:09 | |
| openstackgerrit | Dolph Mathews proposed a change to openstack/pycadf: add federated credential https://review.openstack.org/115771 | 20:18 |
| *** rbowen has quit IRC | 20:20 | |
| *** idegtiarov has quit IRC | 20:22 | |
| clarkb | gordc: ok finally poking at it a bit more. reading the generated ceilometer config file the comments say conf.datatase.connection is a sqlalchemy connection string. Is this used for the other drivers that you listed? | 20:23 |
| clarkb | looks like the tests tests a log:// url so it must be | 20:24 |
| clarkb | and the comment is wrong | 20:24 |
| gordc | clarkb: it's use for all drivers. if i remember correctly, it's becasue oslodb defines a connection string which we used to avoid having multiple config options | 20:25 |
| gordc | all drivers will use conf.database.connection. (even though the comment seems sql specific) side effect of having oslodb's generic name but sql specific purpose | 20:26 |
| clarkb | thats :/ | 20:29 |
| *** Longgeek_ has quit IRC | 20:31 | |
| gordc | clarkb: we use the same config option... i guess you have a preference to use have a highlevel db config option and a connection opt for each backend? it's never been brought up as a concern i guess but it could be done. | 20:34 |
| clarkb | no I just want the documentation to be correct | 20:35 |
| clarkb | and it sounds like it being wrong is expected | 20:35 |
| gordc | clarkb: oh. i wasn't implying that. we're working on docs now. | 20:36 |
| gordc | can you link the string you're referring to? | 20:36 |
| clarkb | gordc: when you generate the sample config | 20:36 |
| clarkb | # The SQLAlchemy connection string used to connect to the database (string value) | 20:37 |
| clarkb | is the comment | 20:37 |
| *** PsionTheory has quit IRC | 20:37 | |
| *** zul has joined #openstack-ceilometer | 20:37 | |
| gordc | ah i see. i'll make a note of it. | 20:37 |
| gordc | ildiko is working on docs so i'll let her know tomorrow when she's online. | 20:37 |
| clarkb | cool | 20:38 |
| clarkb | I am being pulled in 5 directions today so just trying to wrap myself around how all of this works. next step is stevedore | 20:38 |
| gordc | clarkb: np. i assumed you had other items which is why i added disclaimer that you could defer back. | 20:39 |
| clarkb | yup no problem (I am learnign too so its good) | 20:39 |
| gordc | clarkb: that's what i was hoping for :) | 20:40 |
| gordc | let me know if you have stevedore questions... or go straight to the source (dhellmann) | 20:40 |
| *** cdent has quit IRC | 20:47 | |
| *** sbfox1 has quit IRC | 20:49 | |
| *** sbfox has joined #openstack-ceilometer | 20:49 | |
| *** mrmoje has joined #openstack-ceilometer | 20:50 | |
| *** sdake_ has quit IRC | 20:50 | |
| *** zul has quit IRC | 20:58 | |
| *** zul has joined #openstack-ceilometer | 20:58 | |
| clarkb | ok so it looks like stevedore looks for impl_foo files in the namespace path? I am guessing that maybe what we really need to do in that case is split scheme on + and take the first element of that list and lookup that name | 21:00 |
| clarkb | but I don't see impl_mysql instead it is impl_sqlalchemy so that can't be quite right | 21:01 |
| clarkb | ah entry points in setup.cfg? | 21:02 |
| gordc | clarkb: iiuc so stevedore leverages the namespaces you define in setup.cfg... https://github.com/openstack/ceilometer/blob/master/setup.cfg#L189-L196 | 21:02 |
| gordc | yeah. you got it | 21:02 |
| clarkb | yup thats it | 21:02 |
| clarkb | ok in that case I think I may just do the hacky thing of split on + and take the first element | 21:03 |
| gordc | yeah. if it's pretty much standardised and repeatable, then it seems like a safe bet. | 21:03 |
| *** jasondotstar has joined #openstack-ceilometer | 21:05 | |
| gordc | you'll probably need to do your hacky thing here: https://github.com/openstack/ceilometer/blob/master/ceilometer/storage/__init__.py | 21:05 |
| clarkb | ya | 21:05 |
| clarkb | I can do it in get_connection() | 21:05 |
| gordc | sounds right to me. | 21:06 |
| *** sbfox has quit IRC | 21:10 | |
| *** sbfox has joined #openstack-ceilometer | 21:10 | |
| openstackgerrit | A change was merged to openstack/ceilometer: Triple double-quoted strings should be used for docstrings https://review.openstack.org/115629 | 21:16 |
| openstackgerrit | Dolph Mathews proposed a change to openstack/pycadf: add federated credential https://review.openstack.org/115771 | 21:16 |
| *** harlowja_away is now known as harlowja_ | 21:17 | |
| openstackgerrit | Clark Boylan proposed a change to openstack/ceilometer: Handle sqlalchemy connection strings with drivers https://review.openstack.org/115787 | 21:20 |
| clarkb | gordc: ^ so thats not actually tesetd locally because tox yelled at me that I didn't have mongodb | 21:21 |
| *** sbfox has quit IRC | 21:23 | |
| *** sbfox has joined #openstack-ceilometer | 21:23 | |
| *** thomasem has quit IRC | 21:25 | |
| gordc | clarkb: yeah... that i think is known.. i can't remember if anyone has ever opened a bug against it (since most people using/developing ceilometer use mongo) | 21:26 |
| *** prad has quit IRC | 21:33 | |
| openstackgerrit | Eric Pendergrass proposed a change to openstack/ceilometer: Placeholder commit for WIP https://review.openstack.org/115717 | 21:42 |
| harlowja_ | qq, whats up with https://github.com/openstack/ceilometer/blob/2014.1.2/test-requirements.txt#L8 ? | 21:44 |
| harlowja_ | all of swift is needed to test ceilometer :-/ | 21:44 |
| gordc | harlowja_: ... yeah... i should probably check up on that patch: https://bugs.launchpad.net/ceilometer/+bug/1285388 | 21:46 |
| openstackgerrit | Dolph Mathews proposed a change to openstack/pycadf: define the project goal https://review.openstack.org/115796 | 21:46 |
| harlowja_ | gordc cool, thx, was wondering what that was about | 21:46 |
| *** sdake has quit IRC | 21:47 | |
| gordc | harlowja_: np. always good to have comments to improve on. | 21:47 |
| gordc | i'll try to get around to that patch later today | 21:48 |
| harlowja_ | gordc np, we are just building packages for ceilometer and had to bypass trying to build the packages of its dependencies listed in test-requirements | 21:48 |
| harlowja_ | https://github.com/stackforge/anvil/commit/b42f60c51c9cc3d | 21:48 |
| gordc | harlowja_: ah i see. i was under impression the test-requirements didn't affect packaging... wrong assumption on my part. | 21:49 |
| harlowja_ | gordc ya, depends on the packager i guess | 21:49 |
| gordc | harlowja_: cool cool. got it | 21:49 |
| clarkb | gordc: my change looks like it worked, but the test isn't quite right | 21:51 |
| *** julim has quit IRC | 21:52 | |
| clarkb | apparently getting a connection instantiates it | 21:52 |
| clarkb | and my notarealdriver driver failed :) | 21:52 |
| * clarkb tries a thing | 21:54 | |
| gordc | hmm.. i'll take a quick look. | 21:54 |
| openstackgerrit | Clark Boylan proposed a change to openstack/ceilometer: Handle sqlalchemy connection strings with drivers https://review.openstack.org/115787 | 21:54 |
| clarkb | according to sqlalchemy docs it should fall back on the built in sqlite driver if pysqlite2 is not present. So that should test the code path with the +driver without any other changes | 21:55 |
| clarkb | gordc: https://jenkins06.openstack.org/job/gate-ceilometer-python27/1002/console will let us know | 21:56 |
| gordc | running patch locally... i get the feeling jenkins will beat me. | 21:58 |
| *** harlowja_ has quit IRC | 21:59 | |
| gordc | yep beat me. :( | 21:59 |
| clarkb | :( doesn't look like that fixed it | 21:59 |
| clarkb | I guess the fallback doesn't work when you specify it explicitly | 21:59 |
| clarkb | oh gah its pysqlite not pysqlite2 | 22:00 |
| clarkb | gordc: can you test it with s/pysqlite2/pysqlite/ ? | 22:01 |
| gordc | sure | 22:01 |
| clarkb | awesome thank you | 22:01 |
| *** harlowja has joined #openstack-ceilometer | 22:02 | |
| gordc | clarkb: works locally for me | 22:03 |
| clarkb | gordc: awesome, I will push that up | 22:03 |
| openstackgerrit | Clark Boylan proposed a change to openstack/ceilometer: Handle sqlalchemy connection strings with drivers https://review.openstack.org/115787 | 22:03 |
| gordc | clarkb: cool cool. will review | 22:03 |
| *** mrutkows has quit IRC | 22:09 | |
| *** mrmoje has quit IRC | 22:11 | |
| clarkb | gordc: https://review.openstack.org/#/c/115612/1 apparently ihar has already written a patch to fix this? | 22:16 |
| clarkb | gordc: I am happy with either as they seem to be mostly equivalent | 22:16 |
| clarkb | however ihar's patch doesn't test that this works with sqlalchemyn but meh | 22:17 |
| gordc | wow. i totally missed that from this morning. | 22:17 |
| gordc | i apologise dude. | 22:17 |
| gordc | but yeah you're right. it's not actually testing anything real. | 22:18 |
| openstackgerrit | Eric Pendergrass proposed a change to openstack/python-ceilometerclient: Verify alarm found before modifying https://review.openstack.org/110345 | 22:22 |
| clarkb | gordc: I should've checked too :) ihar is working on similar stuff trying to get mysqlconnector working. I am trying to convince them to just use pymysql :) | 22:22 |
| clarkb | and I think with these ceilometer changes pymysql will just work | 22:23 |
| gordc | clarkb: now i feel bad for wasting your time.lol... i'm just going to comment on his patch to improve test. i guess i'll need to accept ihars if he fixes it since he came first. | 22:23 |
| clarkb | gordc: thats fine and no problem. I learned about stevedore today :) | 22:24 |
| clarkb | ihar can just take my test too if that helps | 22:24 |
| *** jasondotstar has quit IRC | 22:24 | |
| gordc | clarkb: cool cool. i'll just redirect it to yours. or maybe someone will just quick approve yours in the morning.. i'll let fate decide. | 22:25 |
| *** jasondotstar has joined #openstack-ceilometer | 22:29 | |
| *** dmsimard is now known as dmsimard_away | 22:35 | |
| *** sbfox has quit IRC | 22:37 | |
| *** jasondotstar has quit IRC | 22:39 | |
| *** gordc has quit IRC | 22:41 | |
| *** _nadya_ has joined #openstack-ceilometer | 23:00 | |
| *** _nadya_ has quit IRC | 23:04 | |
| *** zul has quit IRC | 23:10 | |
| *** ondergetekende has quit IRC | 23:13 | |
| *** shakamunyi has quit IRC | 23:13 | |
| *** ondergetekende has joined #openstack-ceilometer | 23:17 | |
| *** dmsimard_away is now known as dmsimard | 23:18 | |
| *** sbfox has joined #openstack-ceilometer | 23:21 | |
| *** dmsimard is now known as dmsimard_away | 23:21 | |
| openstackgerrit | gordon chung proposed a change to openstack/ceilometer: normalise resource data https://review.openstack.org/111313 | 23:23 |
| *** alexpilotti has quit IRC | 23:34 | |
| *** taplax has quit IRC | 23:46 | |
| *** underyx|off has quit IRC | 23:46 | |
| *** taplax has joined #openstack-ceilometer | 23:47 | |
| *** ilyashakhat_ has joined #openstack-ceilometer | 23:47 | |
| *** underyx|off has joined #openstack-ceilometer | 23:48 | |
| *** underyx|off is now known as underyx | 23:49 | |
| *** ilyashakhat has quit IRC | 23:49 | |
| *** dhellmann has quit IRC | 23:49 | |
| *** dhellmann has joined #openstack-ceilometer | 23:50 | |
| *** openstackgerrit has quit IRC | 23:54 | |
| *** rbowen has joined #openstack-ceilometer | 23:55 | |
| *** openstackgerrit has joined #openstack-ceilometer | 23:57 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!