*** svilgelm has quit IRC | 00:01 | |
*** dims_ has joined #openstack-oslo | 00:02 | |
*** dimsum__ has quit IRC | 00:04 | |
*** adam_g has quit IRC | 00:09 | |
*** adam_g has joined #openstack-oslo | 00:10 | |
*** adam_g has quit IRC | 00:10 | |
*** adam_g has joined #openstack-oslo | 00:10 | |
*** mriedem has quit IRC | 00:43 | |
*** salv-orlando has joined #openstack-oslo | 00:46 | |
*** dims_ has quit IRC | 00:50 | |
*** dimsum__ has joined #openstack-oslo | 00:53 | |
*** salv-orlando has quit IRC | 00:55 | |
*** jamielennox is now known as jamielennox|away | 01:08 | |
*** haigang has joined #openstack-oslo | 01:10 | |
*** achanda has joined #openstack-oslo | 01:25 | |
*** dimsum__ has quit IRC | 01:25 | |
*** dimsum__ has joined #openstack-oslo | 01:28 | |
*** haigang has quit IRC | 01:35 | |
*** haigang has joined #openstack-oslo | 01:36 | |
*** salv-orlando has joined #openstack-oslo | 01:54 | |
*** dimsum__ has quit IRC | 02:02 | |
*** salv-orlando has quit IRC | 02:06 | |
*** svilgelm has joined #openstack-oslo | 03:04 | |
*** rossella_s has quit IRC | 03:11 | |
*** alejandrito has quit IRC | 03:12 | |
*** ccrouch has left #openstack-oslo | 03:25 | |
*** harlowja_at_home has joined #openstack-oslo | 03:33 | |
*** jecarey has joined #openstack-oslo | 03:33 | |
openstackgerrit | Joshua Harlow proposed openstack/taskflow: Show intermediary compilation(s) when BLATHER is enabled https://review.openstack.org/208185 | 03:35 |
---|---|---|
openstackgerrit | Joshua Harlow proposed openstack/taskflow: Provide and use a item -> subgraph extraction routine https://review.openstack.org/208183 | 03:44 |
openstackgerrit | Joshua Harlow proposed openstack/taskflow: Allow provided flow to be empty https://review.openstack.org/208279 | 03:49 |
*** salv-orlando has joined #openstack-oslo | 03:52 | |
*** salv-orlando has quit IRC | 03:58 | |
*** salv-orl_ has joined #openstack-oslo | 04:00 | |
*** salv-orl_ has quit IRC | 04:04 | |
*** svilgelm has quit IRC | 04:16 | |
*** jecarey has quit IRC | 04:17 | |
*** boris-42 has joined #openstack-oslo | 04:18 | |
*** amotoki has joined #openstack-oslo | 04:26 | |
*** harlowja_at_home has quit IRC | 04:32 | |
*** Kennan has quit IRC | 04:35 | |
*** Kennan has joined #openstack-oslo | 04:36 | |
*** jamielennox|away is now known as jamielennox | 04:38 | |
*** harlowja_at_home has joined #openstack-oslo | 04:49 | |
*** salv-orlando has joined #openstack-oslo | 05:03 | |
*** salv-orlando has quit IRC | 05:06 | |
*** salv-orlando has joined #openstack-oslo | 05:11 | |
*** LimorStotland has joined #openstack-oslo | 05:14 | |
*** harlowja_at_home has quit IRC | 05:30 | |
*** ig0r_ has joined #openstack-oslo | 05:33 | |
*** harlowja has quit IRC | 05:44 | |
*** ig0r_ has quit IRC | 05:53 | |
*** ig0r_ has joined #openstack-oslo | 05:56 | |
*** haigang has quit IRC | 06:03 | |
*** haigang has joined #openstack-oslo | 06:04 | |
*** Kennan has quit IRC | 06:09 | |
*** pcaruana has quit IRC | 06:09 | |
*** Kennan has joined #openstack-oslo | 06:11 | |
*** slogan621 has quit IRC | 06:12 | |
*** svilgelm has joined #openstack-oslo | 06:13 | |
*** haigang has quit IRC | 06:16 | |
*** haigang has joined #openstack-oslo | 06:20 | |
*** saggi has joined #openstack-oslo | 06:32 | |
saggi | Hi, been trying to get an RPC server running. It gets the message the client sends when I use cast but not when I use call. It's very simple and looks quite similar to all the examples I can find online. Did someone hit that kind of thing and can suggest a solution? | 06:37 |
openstackgerrit | Merged openstack/taskflow: Give the GC more of a break with regard to cycles https://review.openstack.org/200370 | 06:56 |
*** browne1 has quit IRC | 07:04 | |
*** e0ne has joined #openstack-oslo | 07:06 | |
*** boris-42 has quit IRC | 07:10 | |
*** achanda has quit IRC | 07:13 | |
*** achanda has joined #openstack-oslo | 07:17 | |
*** e0ne has quit IRC | 07:21 | |
*** salv-orlando has quit IRC | 07:22 | |
*** svilgelm has quit IRC | 07:26 | |
*** ihrachyshka has joined #openstack-oslo | 07:30 | |
*** svilgelm has joined #openstack-oslo | 07:32 | |
*** achanda has quit IRC | 07:36 | |
*** pcaruana has joined #openstack-oslo | 07:40 | |
*** salv-orlando has joined #openstack-oslo | 07:48 | |
*** e0ne has joined #openstack-oslo | 07:52 | |
*** e0ne has quit IRC | 07:55 | |
*** jamielennox is now known as jamielennox|away | 07:57 | |
*** svilgelm has quit IRC | 07:58 | |
*** svilgelm has joined #openstack-oslo | 08:03 | |
*** ihrachyshka has quit IRC | 08:04 | |
*** subscope has joined #openstack-oslo | 08:10 | |
*** pblaho has joined #openstack-oslo | 08:14 | |
*** svilgelm_ has joined #openstack-oslo | 08:16 | |
*** svilgelm has quit IRC | 08:16 | |
*** yassine has joined #openstack-oslo | 08:20 | |
openstackgerrit | Marian Horban proposed openstack/oslo.log: FastWatchedFileHandler class was added https://review.openstack.org/208366 | 08:25 |
openstackgerrit | Dmitry Tantsur proposed openstack/oslo.db: Add JSON-encoded types for sqlalchemy https://review.openstack.org/207460 | 08:25 |
openstackgerrit | Dmitry Tantsur proposed openstack/oslo.db: Add JSON-encoded types for sqlalchemy https://review.openstack.org/207460 | 08:27 |
*** ihrachyshka has joined #openstack-oslo | 08:36 | |
*** ozamiatin has joined #openstack-oslo | 09:06 | |
*** e0ne has joined #openstack-oslo | 09:08 | |
*** amotoki has quit IRC | 09:38 | |
*** amotoki has joined #openstack-oslo | 09:50 | |
*** ozamiatin has quit IRC | 09:59 | |
*** salv-orlando has quit IRC | 09:59 | |
*** sabari- has joined #openstack-oslo | 10:06 | |
*** ndipanov has joined #openstack-oslo | 10:07 | |
*** Kennan has quit IRC | 10:07 | |
*** sabari has quit IRC | 10:08 | |
*** LimorStotland has quit IRC | 10:08 | |
*** HenryG has quit IRC | 10:08 | |
*** LimorStotland has joined #openstack-oslo | 10:08 | |
*** sabari- is now known as sabari | 10:08 | |
*** HenryG has joined #openstack-oslo | 10:08 | |
*** Kennan has joined #openstack-oslo | 10:09 | |
*** e0ne has quit IRC | 10:16 | |
*** e0ne has joined #openstack-oslo | 10:17 | |
*** svilgelm_ has quit IRC | 10:20 | |
*** svilgelm has joined #openstack-oslo | 10:21 | |
*** amotoki has quit IRC | 10:22 | |
*** svilgelm has quit IRC | 10:29 | |
*** dimsum__ has joined #openstack-oslo | 10:34 | |
*** haigang has quit IRC | 10:44 | |
*** haigang has joined #openstack-oslo | 10:48 | |
*** e0ne has quit IRC | 10:50 | |
*** pcaruana has quit IRC | 10:57 | |
*** ozamiatin has joined #openstack-oslo | 11:00 | |
*** claudiub has joined #openstack-oslo | 11:03 | |
*** cdent has joined #openstack-oslo | 11:08 | |
*** pcaruana has joined #openstack-oslo | 11:14 | |
*** svilgelm has joined #openstack-oslo | 11:16 | |
*** dims_ has joined #openstack-oslo | 11:18 | |
*** dimsum__ has quit IRC | 11:20 | |
*** svilgelm has quit IRC | 11:26 | |
*** svilgelm has joined #openstack-oslo | 11:26 | |
*** ganso_ has joined #openstack-oslo | 11:26 | |
*** jaypipes has joined #openstack-oslo | 11:27 | |
*** dims_ has quit IRC | 11:54 | |
*** dimsum__ has joined #openstack-oslo | 11:55 | |
*** claudiub has quit IRC | 11:55 | |
*** dimsum__ has quit IRC | 12:08 | |
*** dimsum__ has joined #openstack-oslo | 12:22 | |
*** e0ne has joined #openstack-oslo | 12:24 | |
*** dims_ has joined #openstack-oslo | 12:27 | |
*** dims__ has joined #openstack-oslo | 12:29 | |
*** dimsum__ has quit IRC | 12:29 | |
*** dimsum__ has joined #openstack-oslo | 12:32 | |
*** dims_ has quit IRC | 12:32 | |
*** dims__ has quit IRC | 12:34 | |
*** dims_ has joined #openstack-oslo | 12:45 | |
openstackgerrit | Florian Wilhelm proposed openstack-dev/pbr: Fix retrieval of commit data and most recent tag. https://review.openstack.org/194954 | 12:45 |
*** dimsum__ has quit IRC | 12:46 | |
*** dims_ has quit IRC | 12:53 | |
*** dimsum__ has joined #openstack-oslo | 12:54 | |
*** dimsum__ is now known as dims | 12:55 | |
*** browne has joined #openstack-oslo | 12:57 | |
openstackgerrit | Dmitry Tantsur proposed openstack/oslo.db: Add JSON-encoded types for sqlalchemy https://review.openstack.org/207460 | 12:58 |
*** jaosorior has joined #openstack-oslo | 12:59 | |
*** e0ne has quit IRC | 13:08 | |
openstackgerrit | Timofey Durakov proposed openstack/taskflow: Attribute parrents_required introduced for task class https://review.openstack.org/208468 | 13:08 |
*** e0ne has joined #openstack-oslo | 13:10 | |
*** browne has quit IRC | 13:13 | |
*** kgiusti has joined #openstack-oslo | 13:13 | |
*** haigang has quit IRC | 13:17 | |
*** boris-42 has joined #openstack-oslo | 13:22 | |
*** zzzeek has joined #openstack-oslo | 13:24 | |
*** ig0r_ has quit IRC | 13:26 | |
*** bapalm has joined #openstack-oslo | 13:27 | |
*** ig0r__ has joined #openstack-oslo | 13:33 | |
*** lascii is now known as alaski | 13:39 | |
-openstackstatus- NOTICE: The Gerrit service on review.openstack.org has been restarted in an attempt to improve performance. | 13:40 | |
*** sdake has joined #openstack-oslo | 13:41 | |
*** ig0r__ has quit IRC | 13:44 | |
*** superdan is now known as dansmith | 13:47 | |
*** h00327910__ has quit IRC | 13:48 | |
*** bknudson has quit IRC | 13:49 | |
*** browne has joined #openstack-oslo | 13:50 | |
*** bapalm_ has joined #openstack-oslo | 13:51 | |
*** sigmavirus24_awa is now known as sigmavirus24 | 13:54 | |
*** bapalm has quit IRC | 13:54 | |
*** mriedem has joined #openstack-oslo | 14:03 | |
*** ccrouch has joined #openstack-oslo | 14:05 | |
*** ccrouch has left #openstack-oslo | 14:05 | |
*** browne has quit IRC | 14:08 | |
*** ccrouch has joined #openstack-oslo | 14:11 | |
*** ccrouch has left #openstack-oslo | 14:11 | |
*** prad has joined #openstack-oslo | 14:11 | |
*** prad has quit IRC | 14:11 | |
*** zz_dimtruck is now known as dimtruck | 14:11 | |
*** bknudson has joined #openstack-oslo | 14:13 | |
*** ig0r__ has joined #openstack-oslo | 14:14 | |
*** sdake has quit IRC | 14:16 | |
*** sdake has joined #openstack-oslo | 14:24 | |
*** prad has joined #openstack-oslo | 14:36 | |
*** jecarey has joined #openstack-oslo | 14:37 | |
openstackgerrit | Boris Bobrov proposed openstack/oslo.db: Upgrade and downgrade based on revision existence https://review.openstack.org/201265 | 14:37 |
*** achanda has joined #openstack-oslo | 14:53 | |
*** mtanino has joined #openstack-oslo | 14:54 | |
*** barra204 has joined #openstack-oslo | 14:57 | |
*** achanda has quit IRC | 15:04 | |
*** ig0r__ has quit IRC | 15:09 | |
*** barra204 has quit IRC | 15:09 | |
*** pcaruana has quit IRC | 15:13 | |
*** thumpba has joined #openstack-oslo | 15:14 | |
*** shakamunyi has joined #openstack-oslo | 15:14 | |
*** shakamunyi is now known as barra204 | 15:16 | |
*** harlowja_at_home has joined #openstack-oslo | 15:21 | |
*** edleafe has joined #openstack-oslo | 15:22 | |
*** barra204 has quit IRC | 15:26 | |
*** barra204 has joined #openstack-oslo | 15:40 | |
zzzeek | harlowja_at_home: I built out RAFT right here: https://bitbucket.org/zzzeek/haalchemy/src/45771a7f3ccba9d2a62ebe981df54b09311b22dc/haalchemy/service/raft.py?at=master :) works great. there’s no good Python raft implemenations right now, the ones that are out there are crap | 15:41 |
*** mriedem has quit IRC | 15:46 | |
harlowja_at_home | zzzeek, i heard jd__ was making one :-P | 15:47 |
zzzeek | DLM == distributed lock manager ? | 15:47 |
harlowja_at_home | none of the ones @ https://raftconsensus.github.io/#implementations are good yet in python? | 15:47 |
harlowja_at_home | zzzeek, something like that, raft imho is greater than a DLM, but sure | 15:48 |
zzzeek | there were only 2 last i looked and they were explicitly “not production ready” and they were all over the place | 15:48 |
harlowja_at_home | using raft/zookeeper for DLM is just like a side-effect of there architecture (not the main feature) | 15:48 |
zzzeek | harlowja_at_home: it seems like the point of tooz is to provide patterns like this ? | 15:48 |
harlowja_at_home | that was the goal | 15:48 |
zzzeek | harlowja_at_home: id maybe not use the database for it, just have daemons running and share state, thats how raft works | 15:49 |
harlowja_at_home | yup | 15:49 |
harlowja_at_home | pick one of the stable-raft impls, or zookeeper, and move on | 15:49 |
harlowja_at_home | thats what these systems do (or DLM is 10% of what they can do) | 15:49 |
jd__ | zzzeek: yassine is working on one too https://github.com/ylamgarchal/chillaxd | 15:50 |
harlowja_at_home | ya | 15:50 |
harlowja_at_home | jd__, he should get that listed on https://raftconsensus.github.io/#implementations | 15:50 |
yassine | harlowja_at_home, hiya :), yea i will | 15:51 |
harlowja_at_home | might as well :) get the free PR | 15:51 |
*** openstackgerrit_ has joined #openstack-oslo | 15:54 | |
*** rpodolyaka1 has joined #openstack-oslo | 15:55 | |
*** wendar has quit IRC | 15:57 | |
*** wendar has joined #openstack-oslo | 15:57 | |
dims | 3 mins to oslo meeting | 15:58 |
* zzzeek brims with excitement ! | 16:01 | |
harlowja_at_home | :-P | 16:01 |
*** ccrouch has joined #openstack-oslo | 16:04 | |
*** ccrouch has left #openstack-oslo | 16:04 | |
*** tsekiyama has joined #openstack-oslo | 16:06 | |
*** ig0r_ has joined #openstack-oslo | 16:08 | |
*** dguitarbite has quit IRC | 16:08 | |
*** sdake has quit IRC | 16:16 | |
*** mriedem has joined #openstack-oslo | 16:16 | |
*** sdake has joined #openstack-oslo | 16:21 | |
*** browne has joined #openstack-oslo | 16:21 | |
*** yamahata has quit IRC | 16:25 | |
openstackgerrit | Merged openstack/oslo.db: Add JSON-encoded types for sqlalchemy https://review.openstack.org/207460 | 16:26 |
*** jdandrea has joined #openstack-oslo | 16:29 | |
*** harlowja_at_home has quit IRC | 16:33 | |
*** ozamiatin has quit IRC | 16:35 | |
*** exploreshaifali has joined #openstack-oslo | 16:35 | |
*** ihrachyshka has quit IRC | 16:36 | |
*** e0ne has quit IRC | 16:38 | |
*** sdake has quit IRC | 16:43 | |
*** rpodolyaka1 has quit IRC | 16:45 | |
*** browne has quit IRC | 16:45 | |
*** rpodolyaka1 has joined #openstack-oslo | 16:46 | |
*** sdake has joined #openstack-oslo | 16:47 | |
*** ig0r_ has quit IRC | 16:49 | |
*** rpodolyaka1 has quit IRC | 16:50 | |
*** exploreshaifali has quit IRC | 16:53 | |
*** ig0r_ has joined #openstack-oslo | 16:54 | |
*** ig0r_ has quit IRC | 16:55 | |
yassine | jd__, https://github.com/raftconsensus/raftconsensus.github.io/pull/29 done | 16:56 |
*** ozamiatin has joined #openstack-oslo | 17:02 | |
*** harlowja has joined #openstack-oslo | 17:03 | |
*** ig0r_ has joined #openstack-oslo | 17:04 | |
*** dguitarbite has joined #openstack-oslo | 17:04 | |
*** pcaruana has joined #openstack-oslo | 17:06 | |
openstackgerrit | Moshe Levi proposed openstack/oslo.versionedobjects: Add validation on target_version arg when calling obj_to_primitive https://review.openstack.org/207998 | 17:16 |
*** ozamiatin has quit IRC | 17:21 | |
*** e0ne has joined #openstack-oslo | 17:22 | |
*** ig0r_ has quit IRC | 17:25 | |
*** salv-orlando has joined #openstack-oslo | 17:25 | |
*** achanda_ has joined #openstack-oslo | 17:26 | |
*** browne has joined #openstack-oslo | 17:30 | |
*** yassine has quit IRC | 17:31 | |
*** e0ne has quit IRC | 17:31 | |
*** ig0r_ has joined #openstack-oslo | 17:33 | |
*** e0ne has joined #openstack-oslo | 17:42 | |
*** ig0r_ has quit IRC | 17:44 | |
*** yamahata has joined #openstack-oslo | 17:53 | |
*** salv-orl_ has joined #openstack-oslo | 17:59 | |
*** salv-orlando has quit IRC | 18:01 | |
*** openstackgerrit_ has quit IRC | 18:07 | |
*** achanda has joined #openstack-oslo | 18:14 | |
*** achanda_ has quit IRC | 18:16 | |
*** rpodolyaka1 has joined #openstack-oslo | 18:17 | |
*** harlowja has quit IRC | 18:21 | |
*** Kennan has quit IRC | 18:21 | |
*** harlowja has joined #openstack-oslo | 18:21 | |
*** ozamiatin has joined #openstack-oslo | 18:26 | |
*** Kennan has joined #openstack-oslo | 18:29 | |
*** ozamiatin has quit IRC | 18:36 | |
dims | zigo: rabbitmq 3.5.4 was shipped on July 23rd. | 18:41 |
*** shakamunyi has joined #openstack-oslo | 18:46 | |
*** ihrachyshka has joined #openstack-oslo | 18:58 | |
bknudson | when I run oslo-incubator update.sh, looks like it copies over tools/colorizer.py and tools/install_venv_common.py, but not tools/install_venv.py | 18:59 |
harlowja | someone i don't think likes tooz :-P | 18:59 |
bknudson | so wondering if keystone is supposed to get tools/install_venv.py from oslo-incubator, too? | 18:59 |
dims | dhellmann: you are too kind :) (response to Justin) | 18:59 |
dhellmann | bknudson: is it listed in the config file for update.sh? I know that file grew out of sync quite a bit, so you might not want the copy in the incubator | 19:00 |
bknudson | or maybe keystone shouldn't even have install_venv? | 19:00 |
harlowja | dhellmann i give u medal of honor :-P | 19:00 |
bknudson | dhellmann: keystone actually removed openstack.conf | 19:00 |
*** yamahata has quit IRC | 19:01 | |
*** yamahata has joined #openstack-oslo | 19:01 | |
dhellmann | bknudson: ah, so if you have those tools you're syncing you may want to put it back and just list the scripts | 19:01 |
bknudson | openstack-common.conf | 19:01 |
dhellmann | harlowja: heh | 19:02 |
bknudson | oh, we do have it but it only has those 2 scripts. | 19:02 |
bknudson | So maybe we were supposed to also have install_venv.spy | 19:02 |
dhellmann | bknudson: yeah, I'd look at the differences carefully, but it sounds like you're just missing that entry | 19:03 |
dhellmann | my point is, though, it might be on purpose | 19:03 |
bknudson | dhellmann: I don't know what this is for... doesn't seem useful since tox takes care of creating venv | 19:06 |
dhellmann | bknudson: a lot of projects moved away from that script, so if the keystone devs don't use it you may not need it | 19:06 |
*** exploreshaifali has joined #openstack-oslo | 19:07 | |
openstackgerrit | Sergey Vilgelm proposed openstack/oslo.policy: [WIP] Add parse_rules method the the Rules class https://review.openstack.org/208617 | 19:10 |
*** e0ne has quit IRC | 19:11 | |
*** svilgelm has quit IRC | 19:14 | |
openstackgerrit | Joshua Harlow proposed openstack/taskflow: Cache atom name -> actions and provide accessor function https://review.openstack.org/198211 | 19:20 |
*** rpodolyaka1 has quit IRC | 19:24 | |
*** sdake has quit IRC | 19:28 | |
openstackgerrit | Joshua Harlow proposed openstack/taskflow: Separate revert/execute failure into different outcome constants https://review.openstack.org/199848 | 19:35 |
*** rpodolyaka1 has joined #openstack-oslo | 19:52 | |
openstackgerrit | Joshua Harlow proposed openstack/taskflow: Allow provided flow to be empty https://review.openstack.org/208279 | 20:01 |
*** sdake has joined #openstack-oslo | 20:01 | |
*** sdake_ has joined #openstack-oslo | 20:10 | |
*** rjaiswal has joined #openstack-oslo | 20:10 | |
harlowja | dims https://review.openstack.org/208634 if u get some time | 20:10 |
harlowja | thx :) | 20:13 |
*** sdake has quit IRC | 20:13 | |
dims | harlowja: could you please log one for oslo.log's trace functionality? | 20:15 |
harlowja | ? | 20:15 |
harlowja | meaning change default for oslotest? | 20:15 |
dims | log.trace() should be in the oslo.log released today | 20:15 |
harlowja | ah | 20:16 |
harlowja | k | 20:16 |
harlowja | gotcha | 20:16 |
harlowja | sure | 20:16 |
dims | thanks | 20:16 |
harlowja | dims https://review.openstack.org/208636 | 20:18 |
harlowja | all done | 20:18 |
dims | sdague: dhellmann: log.trace() stuff in requirements when you get a chance please - https://review.openstack.org/#/c/208636/ | 20:19 |
sdague | dims: +2 | 20:19 |
sdague | also, thanks a bunch for shepharding all that through | 20:19 |
dhellmann | dims: +2a | 20:20 |
dims | thanks to both of you! :) and harlowja too | 20:20 |
harlowja | cool | 20:21 |
* harlowja is still waiting for a response from the tooz email, haha, got my popcorn ready :-P | 20:21 | |
dhellmann | harlowja: +2a for oslotest, too | 20:21 |
dhellmann | oh, hmm, wrong josh :-) | 20:21 |
harlowja | idk, think right one? | 20:21 |
* dhellmann needs new glasses, | 20:21 | |
dhellmann | yeah, I misread that somehow | 20:22 |
dims | :) | 20:22 |
harlowja | there is only one josh | 20:22 |
harlowja | jeeeezsh | 20:22 |
dhellmann | heh | 20:22 |
* harlowja tells myself that every morning | 20:22 | |
openstackgerrit | Joshua Harlow proposed openstack/debtcollector: Add a moved function helper https://review.openstack.org/196338 | 20:22 |
harlowja | bknudson fine with https://review.openstack.org/#/c/206253/ ; doesn't look like there is a better way really, https://github.com/GrahamDumpleton/wrapt/issues/44 can be done, but idk, don't like that | 20:24 |
harlowja | descriptors + arguments are a pita, lol | 20:24 |
bknudson | harlowja: "If you really understand decorators and descriptors, its not really that much of a pain in the ass. " -- ouch! | 20:24 |
dhellmann | dims: what do you think about the oslo.policy fixture vs. public API question? | 20:24 |
harlowja | bknudson :) | 20:24 |
harlowja | bknudson i get that, its just not elegant imho | 20:25 |
harlowja | varying levels of ass pain , some can take more than others i guess | 20:25 |
* harlowja is now going to be quiet, before i get in trouble, lol | 20:25 | |
bknudson | harlowja: so you had to re-implement @property ? | 20:26 |
harlowja | bknudson basically | 20:26 |
dims | dhellmann: fixture if it works seems better than API change. need to dig into it | 20:26 |
harlowja | bknudson but property doesn't take arguments, so its more complicated than @property | 20:26 |
harlowja | (some or no arguments) | 20:27 |
dhellmann | dims: yeah, that's where I'm leaning, too, esp. after morganfainberg's response | 20:27 |
harlowja | which is why '__call__' and '__init__' are weird in that review | 20:27 |
*** svilgelm has joined #openstack-oslo | 20:27 | |
harlowja | (for some defintion of weird) | 20:27 |
bknudson | harlowja: if you change the order then the deprecation decorator isn't going to know if it's a property or a regular function, so it can't output the message | 20:29 |
bknudson | so you'd need a different decorator or an argument to the decorator so it knows. | 20:29 |
bknudson | or it would have to put some kind of hook in so it could get called later and update itself... | 20:29 |
morganfainberg | dhellmann: ++ | 20:29 |
morganfainberg | dhellmann: it seems silly to directly expose the parser | 20:30 |
dhellmann | morganfainberg: yeah | 20:30 |
bknudson | I think the parser should be available... otherwise the only way to get a policy is to read a file | 20:31 |
morganfainberg | dhellmann: like i said, i am not opposed to changing that stance, but that is just where we started from. | 20:32 |
morganfainberg | If it is only ever used in tests - im inclined to say fixture | 20:32 |
morganfainberg | If there is a real use case outside of tests, great we can expose the parser as public | 20:32 |
*** ig0r_ has joined #openstack-oslo | 20:36 | |
*** alexpilotti has joined #openstack-oslo | 20:37 | |
dhellmann | bknudson: where else would an application get policy settings? | 20:40 |
bknudson | dhellmann: the policy.json contents? they could hardcode it in their codebase if they want. | 20:43 |
dhellmann | bknudson: I think we would want to provide a better API than passing a blob of DSL for that, though, wouldn't we? | 20:45 |
dhellmann | bknudson: even if it's just passing something other than a filename to the enforcer constructor | 20:45 |
bknudson | dhellmann: I don't know what the better API would look like. Only thing I'd think of is having enforcer take either a filename or a file-like object. | 20:47 |
dhellmann | bknudson: a list of rule instances? | 20:48 |
bknudson | dhellmann: I'm worried that anything that we choose other than a blob would wind up constraining the implementation. | 20:49 |
*** rpodolyaka1 has quit IRC | 20:50 | |
*** ig0r_ has quit IRC | 20:50 | |
*** ig0r__ has joined #openstack-oslo | 20:50 | |
dhellmann | bknudson: how so? | 20:52 |
bknudson | I guess we could always version it so it's a list of Rule today and then an instance of something else later. | 20:52 |
dhellmann | or we can have multiple factories for creating enforcers in different ways. Or allow an enforcer to have no data at first, and then provide APIs for populating it separately. | 20:53 |
bknudson | dhellmann: I think we already have this functionality -- http://git.openstack.org/cgit/openstack/oslo.policy/tree/oslo_policy/policy.py#n247 | 20:54 |
bknudson | you can Rules.from_dict() and Enforcer(rules=) | 20:54 |
dhellmann | ok, there we go then | 20:54 |
dhellmann | ah, load_json() does what nova seems to want to do | 20:55 |
bknudson | it's calling parse_rule | 20:55 |
dhellmann | yeah | 20:55 |
*** thumpba has quit IRC | 20:56 | |
dhellmann | so nova could do global_enforcer.set_rules(Rules.load_json(some_variable_with_json_text)) | 20:57 |
*** bapalm_ has quit IRC | 21:00 | |
*** ozamiatin has joined #openstack-oslo | 21:05 | |
openstackgerrit | Merged openstack/oslo.messaging: Ensures that some assumptions are true. https://review.openstack.org/192437 | 21:06 |
*** kgiusti has left #openstack-oslo | 21:06 | |
*** ndipanov has quit IRC | 21:12 | |
*** ig0r__ has quit IRC | 21:21 | |
*** ozamiatin has quit IRC | 21:28 | |
*** mriedem is now known as mriedem_away | 21:31 | |
*** svilgelm has quit IRC | 21:49 | |
*** cdent has quit IRC | 21:52 | |
*** jecarey has quit IRC | 21:56 | |
*** cdent has joined #openstack-oslo | 22:00 | |
*** cdent has quit IRC | 22:06 | |
*** dimtruck is now known as zz_dimtruck | 22:19 | |
*** thumpba has joined #openstack-oslo | 22:19 | |
*** thumpba has quit IRC | 22:29 | |
*** thumpba has joined #openstack-oslo | 22:41 | |
*** thumpba has quit IRC | 22:41 | |
*** zz_dimtruck is now known as dimtruck | 22:48 | |
*** shakamunyi has quit IRC | 22:51 | |
*** dims_ has joined #openstack-oslo | 22:51 | |
*** dims has quit IRC | 22:52 | |
*** dims has joined #openstack-oslo | 22:53 | |
*** ihrachyshka has quit IRC | 22:55 | |
*** bknudson has quit IRC | 22:56 | |
*** dims_ has quit IRC | 22:56 | |
*** salv-orlando has joined #openstack-oslo | 22:59 | |
*** salv-orl_ has quit IRC | 23:02 | |
*** jaosorior has quit IRC | 23:04 | |
*** dimtruck is now known as zz_dimtruck | 23:19 | |
*** sigmavirus24 is now known as sigmavirus24_awa | 23:24 | |
*** dims has quit IRC | 23:29 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!