| *** sdake_ has joined #openstack-oslo | 00:00 | |
| *** sdake has quit IRC | 00:03 | |
| *** sdake_ is now known as sdake | 00:06 | |
| openstackgerrit | Joshua Harlow proposed openstack/futurist: Allow providing customized scheduling strategies https://review.openstack.org/191163 | 00:09 |
|---|---|---|
| openstackgerrit | Joshua Harlow proposed openstack/futurist: Allow providing customized scheduling strategies https://review.openstack.org/191163 | 00:10 |
| openstackgerrit | lifeless proposed openstack-dev/pbr: Strip markers from test requirements. https://review.openstack.org/193372 | 00:19 |
| *** amotoki_ has joined #openstack-oslo | 00:23 | |
| *** tsekiyam_ has joined #openstack-oslo | 00:27 | |
| *** amotoki_ has quit IRC | 00:28 | |
| *** tsekiyama has quit IRC | 00:30 | |
| openstackgerrit | Joshua Harlow proposed openstack/tooz: Ensure lock(s) acquire/release returns boolean values https://review.openstack.org/193373 | 00:30 |
| *** tsekiyam_ has quit IRC | 00:31 | |
| *** mtanino has quit IRC | 00:31 | |
| *** dims has joined #openstack-oslo | 00:39 | |
| *** sdake_ has joined #openstack-oslo | 00:41 | |
| *** sheeprine has quit IRC | 00:42 | |
| *** sdake has quit IRC | 00:44 | |
| *** dims has quit IRC | 00:47 | |
| *** dims has joined #openstack-oslo | 00:47 | |
| *** sheeprine has joined #openstack-oslo | 00:48 | |
| *** zzzeek has joined #openstack-oslo | 00:52 | |
| *** zzzeek has quit IRC | 00:52 | |
| *** openstack has joined #openstack-oslo | 00:55 | |
| *** arnaud___ has quit IRC | 00:59 | |
| openstackgerrit | Joshua Harlow proposed openstack/futurist: Allow providing customized scheduling strategies https://review.openstack.org/191163 | 01:03 |
| openstackgerrit | Joshua Harlow proposed openstack/futurist: Allow providing customized scheduling strategies https://review.openstack.org/191163 | 01:08 |
| *** mriedem_afk is now known as mriedem | 01:09 | |
| *** bknudson has joined #openstack-oslo | 01:10 | |
| *** ViswaV has quit IRC | 01:10 | |
| *** ccrouch has quit IRC | 01:14 | |
| *** vilobhmm1 has quit IRC | 01:51 | |
| *** arnaud___ has joined #openstack-oslo | 01:56 | |
| openstackgerrit | lifeless proposed openstack-dev/pbr: Strip markers from test requirements. https://review.openstack.org/193372 | 01:59 |
| *** arnaud___ has quit IRC | 02:01 | |
| *** tobe has joined #openstack-oslo | 02:24 | |
| *** amotoki_ has joined #openstack-oslo | 02:24 | |
| *** amotoki_ has quit IRC | 02:30 | |
| *** sdake_ has quit IRC | 02:34 | |
| *** prad has quit IRC | 02:35 | |
| *** cburgess has quit IRC | 02:39 | |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/oslo-incubator: Get rid of oslo-incubator copy of middleware https://review.openstack.org/187354 | 02:40 |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/oslo-incubator: Get rid of oslo-incubator copy of middleware https://review.openstack.org/187354 | 02:40 |
| *** cburgess has joined #openstack-oslo | 02:44 | |
| *** vilobhmm has joined #openstack-oslo | 02:45 | |
| *** jamielennox is now known as jamielennox|away | 02:46 | |
| openstackgerrit | Merged openstack/oslo-incubator: Get rid of oslo-incubator copy of middleware https://review.openstack.org/187354 | 02:51 |
| openstackgerrit | Merged openstack/oslotest: Allow ``OS_DEBUG`` environment variable to specify log level. https://review.openstack.org/192055 | 02:51 |
| *** jamielennox|away is now known as jamielennox | 02:53 | |
| *** stevemar has joined #openstack-oslo | 03:01 | |
| *** cburgess has quit IRC | 03:04 | |
| *** cburgess has joined #openstack-oslo | 03:05 | |
| *** dims has quit IRC | 03:06 | |
| *** cburgess has quit IRC | 03:07 | |
| *** cburgess has joined #openstack-oslo | 03:10 | |
| openstackgerrit | Peng Wu proposed openstack/oslo-specs: Add spec for more-gettext-support https://review.openstack.org/121459 | 03:19 |
| *** stevemar has quit IRC | 03:22 | |
| openstackgerrit | Komei Shimamura proposed openstack/oslo-specs: Add Spec for Adding Kafka Support https://review.openstack.org/189003 | 03:25 |
| *** amotoki_ has joined #openstack-oslo | 03:25 | |
| *** stevemar has joined #openstack-oslo | 03:26 | |
| openstackgerrit | Komei Shimamura proposed openstack/oslo-specs: Add Spec for Adding Kafka Support https://review.openstack.org/189003 | 03:28 |
| *** amotoki_ has quit IRC | 03:30 | |
| *** crc32 has quit IRC | 03:31 | |
| *** sdake has joined #openstack-oslo | 03:33 | |
| openstackgerrit | Merged openstack/oslo.db: Drop use of 'oslo' namespace package https://review.openstack.org/193339 | 03:37 |
| openstackgerrit | Merged openstack/oslo.middleware: Drop use of 'oslo' namespace package https://review.openstack.org/193341 | 03:38 |
| *** browne has joined #openstack-oslo | 03:40 | |
| *** ashleighfarnham has joined #openstack-oslo | 03:42 | |
| *** arnaud___ has joined #openstack-oslo | 03:44 | |
| *** arnaud___ has quit IRC | 03:49 | |
| *** mriedem has quit IRC | 03:55 | |
| *** zz_jgrimm has quit IRC | 04:05 | |
| *** zz_jgrimm has joined #openstack-oslo | 04:06 | |
| *** sdake_ has joined #openstack-oslo | 04:07 | |
| *** sdake has quit IRC | 04:10 | |
| *** harlowja_at_home has joined #openstack-oslo | 04:21 | |
| *** ashleighfarnham has quit IRC | 04:26 | |
| *** amotoki_ has joined #openstack-oslo | 04:26 | |
| *** amotoki_ has quit IRC | 04:31 | |
| *** achanda has joined #openstack-oslo | 04:33 | |
| openstackgerrit | Joshua Harlow proposed openstack/futurist: Allow providing customized scheduling strategies https://review.openstack.org/191163 | 04:38 |
| *** arnaud___ has joined #openstack-oslo | 04:45 | |
| *** arnaud___ has quit IRC | 04:50 | |
| *** rpodolyaka1 has joined #openstack-oslo | 04:52 | |
| harlowja_at_home | lifeless, u should try https://review.openstack.org/#/c/189526/5/taskflow/examples/99_bottles.py | 04:53 |
| harlowja_at_home | u'd probably like it :-P | 04:53 |
| harlowja_at_home | it really will resume on another worker process when one dies | 04:54 |
| harlowja_at_home | automagically | 04:54 |
| harlowja_at_home | follow instructions at top of example for a good time, lol | 04:57 |
| *** achanda has quit IRC | 04:59 | |
| openstackgerrit | Joshua Harlow proposed openstack/taskflow: Handle conductor ctrl-c more appropriately https://review.openstack.org/189536 | 05:02 |
| openstackgerrit | Merged openstack/tooz: Remove file-driver special no-async abilities https://review.openstack.org/192758 | 05:05 |
| harlowja_at_home | maybe i should make a screen-cast of that | 05:06 |
| openstackgerrit | Joshua Harlow proposed openstack/taskflow: Notify on the individual engine steps https://review.openstack.org/189554 | 05:06 |
| *** boris-42 has quit IRC | 05:12 | |
| harlowja_at_home | ok, i made one https://vid.me/a3ox if anyone is interested in seeing what that example can do (the power!!!) | 05:20 |
| *** nkrinner has joined #openstack-oslo | 05:26 | |
| *** amotoki_ has joined #openstack-oslo | 05:27 | |
| *** vilobhmm has quit IRC | 05:28 | |
| *** harlowja_at_home has quit IRC | 05:29 | |
| *** amotoki_ has quit IRC | 05:31 | |
| *** sdake_ has quit IRC | 05:33 | |
| *** rpodolyaka1 has quit IRC | 05:37 | |
| *** Kennan2 has joined #openstack-oslo | 06:07 | |
| *** Kennan has quit IRC | 06:07 | |
| *** stevemar has quit IRC | 06:10 | |
| *** vilobhmm has joined #openstack-oslo | 06:15 | |
| *** e0ne has joined #openstack-oslo | 06:25 | |
| *** e0ne has quit IRC | 06:31 | |
| lifeless | harlowja: heh, k | 06:33 |
| lifeless | harlowja: btw I'm still not sure what that patch is really doing :0 | 06:34 |
| *** arnaud___ has joined #openstack-oslo | 06:34 | |
| *** rpodolyaka1 has joined #openstack-oslo | 06:36 | |
| *** arnaud___ has quit IRC | 06:38 | |
| *** vilobhmm has quit IRC | 06:38 | |
| *** vilobhmm has joined #openstack-oslo | 06:39 | |
| *** vilobhmm has quit IRC | 06:46 | |
| *** vilobhmm has joined #openstack-oslo | 06:49 | |
| openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/oslo.messaging: Don't reply when we known that client is gone https://review.openstack.org/193037 | 06:53 |
| openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/oslo.messaging: Ensures that some assumptions are true. https://review.openstack.org/192437 | 06:57 |
| sileht | flaper87, hi, that should be good now: https://review.openstack.org/#/c/193037/ | 06:58 |
| *** vilobhmm has quit IRC | 07:01 | |
| openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/oslo.messaging: Don't reply when we known that client is gone https://review.openstack.org/193037 | 07:03 |
| openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/oslo.messaging: Don't reply when we known that client is gone https://review.openstack.org/193037 | 07:10 |
| *** rpodolyaka1 has quit IRC | 07:11 | |
| *** rpodolyaka1 has joined #openstack-oslo | 07:12 | |
| *** ihrachyshka has joined #openstack-oslo | 07:17 | |
| *** ashleighfarnham has joined #openstack-oslo | 07:26 | |
| *** ashleighfarnham has quit IRC | 07:30 | |
| *** rpodolyaka1 has quit IRC | 07:32 | |
| *** otherwiseguy has quit IRC | 07:36 | |
| *** browne has quit IRC | 07:49 | |
| *** jamielennox is now known as jamielennox|away | 08:00 | |
| *** otherwiseguy has joined #openstack-oslo | 08:02 | |
| *** cdent has joined #openstack-oslo | 08:16 | |
| *** boris-42 has joined #openstack-oslo | 08:17 | |
| *** elarson has quit IRC | 08:17 | |
| *** e0ne has joined #openstack-oslo | 08:18 | |
| *** rushiagr_away is now known as rushiagr | 08:18 | |
| *** yassine has joined #openstack-oslo | 08:21 | |
| *** rpodolyaka1 has joined #openstack-oslo | 08:21 | |
| *** arnaud___ has joined #openstack-oslo | 08:23 | |
| openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/oslo.messaging: Don't reply when we known that client is gone https://review.openstack.org/193037 | 08:24 |
| *** e0ne has quit IRC | 08:25 | |
| *** arnaud___ has quit IRC | 08:27 | |
| *** amotoki_ has joined #openstack-oslo | 08:29 | |
| *** alexpilotti has joined #openstack-oslo | 08:33 | |
| *** amotoki_ has quit IRC | 08:34 | |
| *** haypo has joined #openstack-oslo | 08:38 | |
| *** ozamiatin has joined #openstack-oslo | 08:45 | |
| haypo | dhellmann, hi. i saw that the PyMySQL was merged into oslo.db cool! can you schedule a release next week? https://review.openstack.org/184392 | 08:59 |
| haypo | dhellmann, i don't understand if it's you or dims who release oslo.* libraries | 08:59 |
| *** arnaud___ has joined #openstack-oslo | 09:00 | |
| *** rpodolyaka1 has quit IRC | 09:01 | |
| *** viktors has quit IRC | 09:03 | |
| *** rpodolyaka1 has joined #openstack-oslo | 09:05 | |
| *** arnaud___ has quit IRC | 09:05 | |
| *** ndipanov has joined #openstack-oslo | 09:09 | |
| openstackgerrit | Thomas Bechtold proposed openstack-dev/pbr: Remove sphinx_config.init_values() manual call https://review.openstack.org/193462 | 09:10 |
| *** exploreshaifali has joined #openstack-oslo | 09:24 | |
| *** e0ne has joined #openstack-oslo | 09:28 | |
| *** tobe has quit IRC | 09:28 | |
| *** amotoki_ has joined #openstack-oslo | 09:30 | |
| *** amotoki_ has quit IRC | 09:34 | |
| *** Kennan2 is now known as Kennan | 09:49 | |
| *** e0ne is now known as e0ne_ | 09:52 | |
| *** e0ne_ is now known as e0ne | 09:54 | |
| *** rpodolyaka1 has quit IRC | 10:05 | |
| *** amotoki_ has joined #openstack-oslo | 10:07 | |
| *** dims has joined #openstack-oslo | 10:09 | |
| *** arnaud___ has joined #openstack-oslo | 10:50 | |
| *** marzif_ has joined #openstack-oslo | 10:54 | |
| *** arnaud___ has quit IRC | 10:55 | |
| *** dims has quit IRC | 10:57 | |
| *** dims has joined #openstack-oslo | 11:06 | |
| *** rpodolyaka1 has joined #openstack-oslo | 11:19 | |
| *** e0ne is now known as e0ne_ | 11:19 | |
| *** e0ne_ has quit IRC | 11:29 | |
| openstackgerrit | Vipin Balachandran proposed openstack/oslo.vmware: Exception hierarchy refactoring https://review.openstack.org/114503 | 11:38 |
| openstackgerrit | Vipin Balachandran proposed openstack/oslo.vmware: Raise VimFaultException for unknown faults https://review.openstack.org/192139 | 11:49 |
| openstackgerrit | Vipin Balachandran proposed openstack/oslo.vmware: Deprecate unused exceptions https://review.openstack.org/192149 | 11:50 |
| *** boris-42 has quit IRC | 11:52 | |
| *** ozamiatin has quit IRC | 11:58 | |
| *** bknudson has quit IRC | 12:01 | |
| *** e0ne has joined #openstack-oslo | 12:08 | |
| eezhova | ihrachyshka, dims, hi! I think that I ran into a problem while porting neutron to oslo.service. There is a circular dependency between neutron (https://review.openstack.org/#/c/192999/7) and neutron-lbaas (https://review.openstack.org/#/c/193454/1). The problem is that neutron_lbaas.agent.agent.LbaasAgentService derives from neutron.rpc.common.Service which derives from neutron.openstack.common.service.Service/oslo_service.service.Service. And in re | 12:15 |
| eezhova | sult tempest tests for neutron-lbaas fail with http://logs.openstack.org/54/193454/1/check/check-tempest-dsvm-neutron-full/5b20d38/logs/screen-q-lbaas.txt.gz . If I make neutron-lbaas patch depend on neutron patch then tempest tests for neutron will fail. | 12:15 |
| dims | eezhova: so you can whip up a devstack patch that depends on the neutron and neutron-lbaas patch and test it there? | 12:16 |
| ihrachyshka | eezhova, that's an interesting one! :) | 12:17 |
| ihrachyshka | eezhova, we can (temporarily?) unbind them by copying rpc.common.Service from neutron into lbaas | 12:18 |
| ihrachyshka | eezhova, or we can try to make rpc.common.Service a subclass of ServiceBase (?) | 12:19 |
| ihrachyshka | it's an abstract class, so should be doable | 12:19 |
| eezhova | ihrachyshka, the problem is ServiceBase isn't present in common.service it was introduced only in oslo.service | 12:20 |
| ihrachyshka | eezhova, yeah, so you make neutron depend on oslo.service, what's the deal? | 12:21 |
| eezhova | ihrachyshka, if I just port neutron to oslo.service many *aaS tempest tests will fail. Or I just didn't get your idea | 12:23 |
| toabctl | dims: what's the reason to not include oslo.service in global-requirements now? https://review.openstack.org/#/c/192886/ | 12:24 |
| dims | toabctl: wanted to wait to see if the API looked right | 12:25 |
| dims | guess we can unblock it now | 12:25 |
| *** bknudson has joined #openstack-oslo | 12:25 | |
| eezhova | ihrachyshka, but I like your idea about copying rpc.common.Service into lbaas | 12:25 |
| eezhova | ihrachyshka, I guess it should work | 12:26 |
| *** kgiusti has joined #openstack-oslo | 12:26 | |
| ihrachyshka | eezhova, so would anything break if you first make rpc.common.Service a subclass of the abs; then you port *aas; then you port neutron? | 12:26 |
| dims | toabctl: unblocked - https://review.openstack.org/#/c/192886/ | 12:27 |
| ihrachyshka | all steps are separate, you don't switch neutron right away, you only make it mimic oslo.service enough to pass *aas | 12:27 |
| ihrachyshka | eezhova, note that rpc.common.Service is itself a copy-pasted code from old rpc incubator library | 12:27 |
| openstackgerrit | Vipin Balachandran proposed openstack/oslo.vmware: Exception hierarchy refactoring https://review.openstack.org/114503 | 12:28 |
| openstackgerrit | Vipin Balachandran proposed openstack/oslo.vmware: Raise VimFaultException for unknown faults https://review.openstack.org/192139 | 12:28 |
| openstackgerrit | Vipin Balachandran proposed openstack/oslo.vmware: Deprecate unused exceptions https://review.openstack.org/192149 | 12:29 |
| eezhova | ihrachyshka, no need to make rpc.common.Service subclass of the abs class, because, if we use oslo.service, it is already its subclass (We have oslo_service.service.ServiceBase(abs) -> oslo_service.service.Service -> rpc.common.Service) | 12:30 |
| *** haypo has left #openstack-oslo | 12:30 | |
| eezhova | ihrachyshka, so we get something like that: 1. Copy rpc.common.Service to lbaas 2. port lbaas to oslo.service 3. port neutron 4. remove rpc.common.Service from lbaas | 12:32 |
| eezhova | ihrachyshka, and luckily other *aas projects don't have such problems | 12:33 |
| ihrachyshka | it works for me :) thanks, looking forward to see patches | 12:34 |
| *** arnaud___ has joined #openstack-oslo | 12:39 | |
| eezhova | ihrachyshka, I'll add you as a reviewer as soon as they are ready. Thanks for helping with a solution :) | 12:39 |
| *** amrith has quit IRC | 12:42 | |
| *** arnaud___ has quit IRC | 12:43 | |
| *** amrith has joined #openstack-oslo | 12:46 | |
| *** dims has quit IRC | 12:46 | |
| *** dims has joined #openstack-oslo | 12:46 | |
| *** jaypipes has quit IRC | 12:51 | |
| *** rpodolyaka1 has quit IRC | 12:51 | |
| *** rpodolyaka1 has joined #openstack-oslo | 12:55 | |
| *** pblaho has joined #openstack-oslo | 12:56 | |
| dims | dhellmann: good morning | 12:59 |
| *** ganso_ has joined #openstack-oslo | 13:00 | |
| *** sdake has joined #openstack-oslo | 13:05 | |
| *** HenryG has quit IRC | 13:05 | |
| *** HenryG has joined #openstack-oslo | 13:05 | |
| *** HenryG has quit IRC | 13:06 | |
| *** sdake_ has joined #openstack-oslo | 13:07 | |
| *** HenryG has joined #openstack-oslo | 13:07 | |
| dhellmann | dims: hi! | 13:08 |
| dims | dhellmann: feature/keystone-initial-import feature branch is ready to merge with some tests skipped/commented | 13:09 |
| dims | dhellmann: when you get a chance please merge it to trunk and we can continue there as usual | 13:09 |
| dhellmann | dims: don't we do that with a patch? | 13:10 |
| * dhellmann doesn't have gerrit admin | 13:10 | |
| *** katyafervent has joined #openstack-oslo | 13:10 | |
| dims | dhellmann: ha, i was not sure how to do that process. first time for me. | 13:10 |
| *** sdake has quit IRC | 13:10 | |
| katyafervent | Hi | 13:10 |
| dhellmann | dims: I think you do the git merge locally, then git review the results | 13:11 |
| dims | dhellmann: i see, let me try again | 13:11 |
| dims | katyafervent: hi | 13:11 |
| katyafervent | do you know how to make policy options from config available? | 13:11 |
| katyafervent | I have [oslo_policy] section | 13:11 |
| dhellmann | dims: if that doesn't work, we'll need more advice from fungi | 13:12 |
| katyafervent | but cfg.CONF.oslo_policy doesn't exist | 13:12 |
| katyafervent | what I've missed? | 13:12 |
| dims | dhellmann: ack | 13:12 |
| dhellmann | katyafervent: how are you trying to use the options? | 13:12 |
| katyafervent | I want to get value of policy_default_rule | 13:12 |
| fungi | dims: dhellmann: yep see http://docs.openstack.org/infra/manual/drivers.html#feature-branches | 13:13 |
| dhellmann | katyafervent: in oslo.policy code? | 13:13 |
| dhellmann | fungi: I should have known that would be documented, thanks | 13:13 |
| katyafervent | no, in project that imports oslo-policy | 13:13 |
| fungi | dims: as long as you're a member of the group that was granted pushmerge in your recent acl update, it should "just work" | 13:13 |
| katyafervent | should I register opts by myslef? | 13:13 |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/oslo.reports: Guru Meditation Reports broken without version_string https://review.openstack.org/193529 | 13:13 |
| dhellmann | katyafervent: you should not access configuration options from application code like that. What are you actually trying to do? | 13:13 |
| dhellmann | katyafervent: config options are not part of the API because they get renamed | 13:14 |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/oslo.reports: Guru Meditation Reports broken without version_string https://review.openstack.org/193529 | 13:14 |
| katyafervent | since I want to get name of default policy :) so it should be used only by policy module and not by applications, correct? | 13:15 |
| openstackgerrit | Merged openstack-dev/pbr: Strip markers from test requirements. https://review.openstack.org/193372 | 13:15 |
| dhellmann | katyafervent: yes. The fact that the option exists is an implementation detail of the library, and applications should not rely on it. | 13:16 |
| dhellmann | what do you want to do with the default policy name? | 13:16 |
| dhellmann | if you're building a UI or something we could talk about adding an API to support you | 13:16 |
| dhellmann | but if you're doing policy checking, you should use the regular API | 13:17 |
| *** nkrinner has quit IRC | 13:17 | |
| dims | fungi: need advice on how to merge a feature branch into master (git review after a local git merge fails - http://paste.openstack.org/show/304622/) | 13:18 |
| katyafervent | Now I got it, I'll figure out. Thanks a lot! | 13:18 |
| dims | dhellmann: ^^ will wait for fungi | 13:18 |
| dims | thanks | 13:18 |
| fungi | dims: the problem is here https://review.openstack.org/#/admin/groups/687,members | 13:19 |
| fungi | dhellmann: dims: who should be in that group? | 13:19 |
| *** sdake has joined #openstack-oslo | 13:20 | |
| dims | fungi: oslo-release should be added to oslo-cache-release. please | 13:20 |
| fungi | dims: done, but i should flush gerrit's caches too before you try again. just a sec | 13:21 |
| dims | thanks fungi! | 13:22 |
| fungi | dims: oh, also, see the note in that document about omitting a file. it looks like your merge commit may be dragging in a change to the .gitreview in master | 13:23 |
| fungi | you probably want to use the `git checkout origin/master -- .gitreview` suggestion there | 13:23 |
| *** achanda has joined #openstack-oslo | 13:23 | |
| *** sdake_ has quit IRC | 13:24 | |
| *** mriedem has joined #openstack-oslo | 13:24 | |
| dims | fungi: ah. right | 13:24 |
| fungi | perhaps do similar for any other files changed in your feature branch which should be left untouched in master | 13:24 |
| fungi | dims: the cache flush is done, so it shouldn't reject you now | 13:25 |
| dims | thanks, i'll run the tests one last time before trying | 13:25 |
| fungi | awesome. let me know if you bump into any other issues | 13:25 |
| dims | thanks a bunch fungi, now i know how to work with feature branches :) | 13:26 |
| *** jaypipes has joined #openstack-oslo | 13:26 | |
| fungi | "what i learned today: 1. working with feature branches" | 13:28 |
| dims | :) | 13:30 |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/oslo.cache: Merge branch 'feature/keystone-initial-import' https://review.openstack.org/193542 | 13:30 |
| * dims looking up who was willing to help with oslo.cache :) | 13:31 | |
| dims | pleia2: is Xiaoyuan Lu still around? http://markmail.org/message/prb4zojxgbshzkwt | 13:35 |
| *** edleafe is now known as figleaf | 13:37 | |
| dhellmann | dims: morganfainberg, stevemar, bknudson, and some of the other keystone folks might be interested | 13:40 |
| dstanek | dhellmann: interested in what? reading up.... | 13:41 |
| dhellmann | dstanek: oslo.cache | 13:41 |
| bknudson | dstanek: .gitreview is finally being updated! | 13:41 |
| dstanek | ah, sure. i have no problem helping out with that | 13:41 |
| dstanek | dhellmann: dims: is there specific things that are needed right now? | 13:42 |
| dhellmann | bknudson: that patch merges all of the code exported from keystone into the master branch of the lib. The diff only shows .gitreview because that change is only in the merge commit | 13:42 |
| dstanek | or just more help | 13:42 |
| bknudson | http://git.openstack.org/cgit/openstack/oslo.cache/tree/ | 13:42 |
| dhellmann | dstanek: I think dims wants to populate the review team | 13:42 |
| bknudson | http://git.openstack.org/cgit/openstack/oslo.cache/tree/oslo_cache/tests/test_cache.py -- weak! | 13:42 |
| dims | bknudson: dstanek: yes, i have a handful of tests commented out or skipped | 13:42 |
| dims | we have to fix them to start with | 13:43 |
| bknudson | the first thing I look at is the tests of course | 13:43 |
| dhellmann | bknudson: http://git.openstack.org/cgit/openstack/oslo.cache/tree/?h=feature%2Fkeystone-initial-import | 13:43 |
| dims | yes, want to populate the review team :) | 13:43 |
| dims | https://review.openstack.org/#/c/193542/ is merging now | 13:43 |
| dstanek | dims: i'll go ahead a star the project so that it lands in my todo reviews | 13:44 |
| dims | bknudson: ported a bunch from keystone | 13:44 |
| bknudson | http://git.openstack.org/cgit/openstack/oslo.cache/tree/oslo_cache/tests/test_cache.py?h=feature/keystone-initial-import looks much better | 13:44 |
| *** prad has joined #openstack-oslo | 13:44 | |
| openstackgerrit | Merged openstack/oslo.cache: Merge branch 'feature/keystone-initial-import' https://review.openstack.org/193542 | 13:44 |
| dims | bknudson: y, see the FIXME and the SkipIf's | 13:44 |
| dims | so master is ready now | 13:45 |
| *** dansmith is now known as superdan | 13:45 | |
| dims | i'll let you all poke at it :) | 13:45 |
| bknudson | who's core on the project? | 13:46 |
| dims | bknudson: it's empty at the moment | 13:46 |
| * dhellmann nominates bknudson for his clear interest in the test quality of the project | 13:46 | |
| dims | yay :) and dstanek too | 13:46 |
| dstanek | dims: sure | 13:47 |
| dims | dstanek: bknudson: cool, it's your baby now :) https://review.openstack.org/#/admin/groups/686,members | 13:48 |
| bknudson | I assume morgan is on there since he wrote most of this? | 13:48 |
| bknudson | morganfainberg | 13:48 |
| dims | bknudson: ack, can add him if he is interested | 13:48 |
| dims | morganfainberg: ^^ | 13:49 |
| *** ozamiatin has joined #openstack-oslo | 13:49 | |
| *** jaypipes is now known as leakypipes | 13:50 | |
| morganfainberg | Huh? | 13:51 |
| bknudson | I guess it's early for morganfainberg | 13:51 |
| morganfainberg | me is still asleep | 13:51 |
| bknudson | no coffee yet | 13:51 |
| * dims hands morganfainberg some coffee :) | 13:51 | |
| morganfainberg | It's 0650 here | 13:51 |
| bknudson | and he's on vacation or something | 13:52 |
| bknudson | oslo.service is tagged with 0.1.0 ... so can we use it in, e.g., keystone at this point? | 13:52 |
| morganfainberg | I am supposed to be off this week. I might turn off IRC next week | 13:52 |
| dims | morganfainberg: happy vacationing, talk to you when you get back | 13:53 |
| morganfainberg | What is oslo. Service ? | 13:53 |
| morganfainberg | dims: too late :p | 13:53 |
| bknudson | morganfainberg: http://git.openstack.org/cgit/openstack/keystone/tree/keystone/openstack/common/service.py | 13:53 |
| bknudson | moved out to a library | 13:54 |
| dhellmann | morganfainberg: you should definitely have turned off irc this week if you're on vacation :-/ | 13:54 |
| dstanek | bknudson: i've delete much of that in my flask poc branch | 13:54 |
| *** arnaud___ has joined #openstack-oslo | 13:54 | |
| bknudson | in some ways we shouldn't use it in keystone, since we've deprecated eventlet | 13:54 |
| bknudson | dstanek: flask handles all this service stuff? I thought it had to do with eventlet and not wsgi really | 13:55 |
| morganfainberg | bknudson: lets not use it unless it is relevant for apache wsgi | 13:55 |
| *** subscope has quit IRC | 13:56 | |
| morganfainberg | Or if it helps dstanek with using flask | 13:56 |
| bknudson | we haven't removed the code yet and I think oslo is going to want to delete the file from oslo-incubator. Should be an easy switch. | 13:56 |
| dstanek | bknudson: you can run flask as a wsgi service, but it does handle that for a dev server | 13:56 |
| bknudson | dstanek: what about the backdoor port? | 13:56 |
| *** pblaho has quit IRC | 13:56 | |
| dstanek | i basically replaced keystone-all with keystone-devserver | 13:56 |
| dstanek | bknudson: by default it does not do the backdoor port | 13:57 |
| * dims switches on his friday nick | 13:57 | |
| bknudson | interesting | 13:57 |
| *** dims is now known as dimsum__ | 13:57 | |
| morganfainberg | dhellmann: i was only partially I vacation | 13:57 |
| *** rpodolyaka1 has quit IRC | 13:58 | |
| morganfainberg | Or I would have turned IRC off | 13:58 |
| *** stpierre has joined #openstack-oslo | 13:58 | |
| *** arnaud___ has quit IRC | 13:59 | |
| morganfainberg | Also was oslo.cache the reason I was pinged, not oslo.service? | 13:59 |
| dimsum__ | morganfainberg: yes, oslo.cache | 14:00 |
| dhellmann | morganfainberg: yep | 14:00 |
| bknudson | I think if we switch to oslo.service that would get rid of pretty much everything in keystone.openstack.common | 14:00 |
| bknudson | which would be nice by itself | 14:00 |
| morganfainberg | Sure. I assume it all mostly goes away in m release as well. | 14:01 |
| morganfainberg | Anyway. | 14:01 |
| dhellmann | we'll be removing that code from the incubator this cycle | 14:01 |
| morganfainberg | Wish I could delete eventlet from keystone today :P | 14:02 |
| dstanek | morganfainberg: ++ | 14:02 |
| bknudson | dstanek: how does flask handle multiple connections? | 14:03 |
| bknudson | single-threaded? | 14:03 |
| bknudson | not that it really matters since we're not going to support -devserver either! | 14:03 |
| *** rpodolyaka1 has joined #openstack-oslo | 14:04 | |
| *** f13o has joined #openstack-oslo | 14:05 | |
| dstanek | bknudson: it doesn't directly. i am using a little shim that runs it twice on two different ports similar to the process launcher, but not as robust because it's only for development purposes | 14:05 |
| *** morganfainberg is now known as morgan-devserver | 14:06 | |
| * morgan-devserver can handle one request at a time now. | 14:07 | |
| *** jecarey has joined #openstack-oslo | 14:07 | |
| *** morgan-devserver is now known as CaptainMorgan | 14:08 | |
| CaptainMorgan | dimsum__: im happy to help review oslo.cache code | 14:08 |
| CaptainMorgan | Btw | 14:08 |
| dstanek | CaptainMorgan: you should handle_one_request() and sleep(90) | 14:08 |
| dimsum__ | aye aye captain | 14:09 |
| CaptainMorgan | dstanek: .sleep(86400) | 14:09 |
| *** ihrachyshka has quit IRC | 14:09 | |
| openstackgerrit | Jakub Libosvar proposed openstack/oslo.rootwrap: Log that rootwrap was spawned after check https://review.openstack.org/190977 | 14:11 |
| *** sigmavirus24_awa is now known as sigmavirus24 | 14:12 | |
| *** exploreshaifali has quit IRC | 14:13 | |
| *** zz_jgrimm is now known as jgrimm | 14:20 | |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/oslo.reports: [WIP] Dummy review just to run tests https://review.openstack.org/193570 | 14:22 |
| *** stevemar has joined #openstack-oslo | 14:24 | |
| *** stevemar is now known as stevedoor | 14:29 | |
| *** alaski is now known as lascii | 14:29 | |
| *** yamahata has joined #openstack-oslo | 14:35 | |
| *** achanda has quit IRC | 14:36 | |
| *** yamahata has quit IRC | 14:38 | |
| *** yamahata has joined #openstack-oslo | 14:38 | |
| *** exploreshaifali has joined #openstack-oslo | 14:39 | |
| *** rpodolyaka1 has quit IRC | 14:42 | |
| *** yamahata has quit IRC | 14:44 | |
| *** yamahata has joined #openstack-oslo | 14:45 | |
| *** dvorak is now known as clayton | 14:49 | |
| *** pblaho has joined #openstack-oslo | 14:50 | |
| *** rpodolyaka1 has joined #openstack-oslo | 14:51 | |
| *** f13o has quit IRC | 14:53 | |
| *** f13o has joined #openstack-oslo | 14:54 | |
| *** rushiagr is now known as rushiagr_away | 15:02 | |
| *** zzzeek has joined #openstack-oslo | 15:03 | |
| *** browne has joined #openstack-oslo | 15:04 | |
| *** elarson has joined #openstack-oslo | 15:04 | |
| *** elarson has quit IRC | 15:05 | |
| *** elarson has joined #openstack-oslo | 15:06 | |
| openstackgerrit | Rohit Jaiswal proposed openstack/oslo-incubator: Return timer after adding it to internal list https://review.openstack.org/193322 | 15:07 |
| dimsum__ | stevedoor: ping | 15:15 |
| *** mriedem has left #openstack-oslo | 15:15 | |
| *** mriedem has joined #openstack-oslo | 15:16 | |
| dimsum__ | stevedoor: you may be interested in oslo.cache as well, if you pull master you will see a handful of tests either commented or skipped for python3. please take a look | 15:16 |
| *** mtanino has joined #openstack-oslo | 15:18 | |
| stevedoor | dimsum__, rgr that, adding it to my list to look @ | 15:19 |
| *** exploreshaifali has quit IRC | 15:23 | |
| *** arnaud___ has joined #openstack-oslo | 15:25 | |
| *** cdent_ has joined #openstack-oslo | 15:25 | |
| *** ozamiatin has quit IRC | 15:26 | |
| *** cdent has quit IRC | 15:29 | |
| *** cdent_ is now known as cdent | 15:29 | |
| ekarlso | zzzeek: hey, is it possible to make a server_default that can be a substring of the id ? | 15:29 |
| *** arnaud___ has quit IRC | 15:30 | |
| ekarlso | like setting server_default for a col to "CONV(SUBSTR(id, 1, 3), 16, 10)" | 15:30 |
| *** vilobhmm has joined #openstack-oslo | 15:31 | |
| *** rpodolyaka1 has quit IRC | 15:31 | |
| *** browne has quit IRC | 15:33 | |
| *** rpodolyaka1 has joined #openstack-oslo | 15:34 | |
| *** vilobhmm has quit IRC | 15:40 | |
| openstackgerrit | Ilya Shakhat proposed openstack/oslo.service: Allow LoopingCall to continue on exception in callee https://review.openstack.org/193605 | 15:43 |
| *** yassine has quit IRC | 15:44 | |
| openstackgerrit | Alexander Tivelkov proposed openstack/oslo.versionedobjects: Compound fields now coerce their elements' values https://review.openstack.org/193077 | 15:49 |
| *** tsekiyama has joined #openstack-oslo | 15:54 | |
| *** ozamiatin has joined #openstack-oslo | 15:58 | |
| *** lascii is now known as alaski | 15:59 | |
| *** ozamiatin has quit IRC | 15:59 | |
| *** vilobhmm has joined #openstack-oslo | 16:02 | |
| *** yamahata has quit IRC | 16:03 | |
| *** ozamiatin has joined #openstack-oslo | 16:09 | |
| *** e0ne has quit IRC | 16:10 | |
| pleia2 | dimsum__: re: Xiaoyuan, I know she wanted to do more work on the project, but she got wrapped up in graduation, I'll drop her an email | 16:14 |
| *** ashleighfarnham has joined #openstack-oslo | 16:16 | |
| *** mriedem is now known as mriedem_burgers | 16:22 | |
| *** achanda has joined #openstack-oslo | 16:23 | |
| *** rpodolyaka1 has quit IRC | 16:23 | |
| *** rpodolyaka1 has joined #openstack-oslo | 16:25 | |
| *** ozamiatin has quit IRC | 16:26 | |
| *** sdake_ has joined #openstack-oslo | 16:27 | |
| *** sputnik13 has joined #openstack-oslo | 16:30 | |
| *** sdake has quit IRC | 16:31 | |
| *** rpodolyaka1 has quit IRC | 16:35 | |
| *** rpodolyaka1 has joined #openstack-oslo | 16:37 | |
| *** browne has joined #openstack-oslo | 16:42 | |
| *** f13o has quit IRC | 16:49 | |
| *** rpodolyaka1 has quit IRC | 16:50 | |
| *** rpodolyaka1 has joined #openstack-oslo | 16:51 | |
| *** sdake has joined #openstack-oslo | 16:52 | |
| *** yamahata has joined #openstack-oslo | 16:53 | |
| *** sdake_ has quit IRC | 16:55 | |
| *** ViswaV has joined #openstack-oslo | 16:58 | |
| *** yamahata has quit IRC | 16:59 | |
| *** ViswaV_ has joined #openstack-oslo | 17:00 | |
| *** ViswaV has quit IRC | 17:03 | |
| *** ashleighfarnham has quit IRC | 17:03 | |
| *** sdake_ has joined #openstack-oslo | 17:05 | |
| *** sputnik13 has quit IRC | 17:06 | |
| *** sdake has quit IRC | 17:09 | |
| *** ashleighfarnham has joined #openstack-oslo | 17:12 | |
| *** sputnik13 has joined #openstack-oslo | 17:12 | |
| *** arnaud___ has joined #openstack-oslo | 17:14 | |
| *** arnaud___ has quit IRC | 17:19 | |
| *** mriedem_burgers is now known as mriedem | 17:22 | |
| *** exploreshaifali has joined #openstack-oslo | 17:33 | |
| *** exploreshaifali has quit IRC | 17:33 | |
| *** amrith is now known as _amrith_ | 17:34 | |
| *** sputnik13 has quit IRC | 17:34 | |
| superdan | ativelkov: on this: https://review.openstack.org/#/c/193077/5/oslo_versionedobjects/fields.py,cm | 17:35 |
| *** exploreshaifali has joined #openstack-oslo | 17:35 | |
| superdan | ativelkov: I still am not sure why you have obj and field as optional.. I get that they will just end up with None in a log message, but why not just require them? Is there somewhere we need to not pass them? | 17:36 |
| *** _amrith_ is now known as amrith | 17:36 | |
| *** exploreshaifali has quit IRC | 17:36 | |
| dimsum__ | stevedoor: thanks | 17:38 |
| dimsum__ | pleia2: thanks | 17:38 |
| *** e0ne has joined #openstack-oslo | 17:38 | |
| *** exploreshaifali has joined #openstack-oslo | 17:40 | |
| *** exploreshaifali has quit IRC | 17:42 | |
| zzzeek | ekarlso: probably not. you’d want to use a trigger for that | 17:43 |
| *** exploreshaifali has joined #openstack-oslo | 17:43 | |
| *** cdent has quit IRC | 17:45 | |
| *** exploreshaifali has quit IRC | 17:47 | |
| *** exploreshaifali has joined #openstack-oslo | 17:49 | |
| *** exploreshaifali has quit IRC | 17:52 | |
| harlowja | lifeless i fixed that ctrl-c patch, ha, i must of been seeing something else when i added that :-P | 17:53 |
| dhellmann | dimsum__: so oslo.service is ready for test adoption? https://review.openstack.org/#/c/192886/ | 17:53 |
| harlowja | or on drugs, lol | 17:53 |
| *** exploreshaifali has joined #openstack-oslo | 17:54 | |
| dimsum__ | dhellmann: y, i had a -W on it and the neutron folks wanted to get it in as they have some trouble (cyclic) between neutron and neutron-lbaas (that they can't work around with Depends-On) | 17:55 |
| dhellmann | dimsum__: ok | 17:55 |
| dhellmann | dimsum__: +2 | 17:55 |
| dimsum__ | thanks | 17:55 |
| *** exploreshaifali has quit IRC | 17:55 | |
| *** rpodolyaka1 has quit IRC | 17:58 | |
| *** rpodolyaka1 has joined #openstack-oslo | 17:58 | |
| *** rpodolyaka1 has quit IRC | 17:58 | |
| *** prad_ has joined #openstack-oslo | 18:01 | |
| vilobhmm | dimsum__ : thanks for the review here https://review.openstack.org/#/c/190322/....answered your question…please check.. | 18:02 |
| openstackgerrit | Joshua Harlow proposed openstack/futurist: Allow providing customized scheduling strategies https://review.openstack.org/191163 | 18:04 |
| *** Rockyg has joined #openstack-oslo | 18:05 | |
| openstackgerrit | Doug Hellmann proposed openstack/oslo.messaging: Drop use of 'oslo' namespace package https://review.openstack.org/193340 | 18:07 |
| *** arnaud___ has joined #openstack-oslo | 18:15 | |
| *** alexpilotti has quit IRC | 18:18 | |
| *** arnaud___ has quit IRC | 18:19 | |
| dimsum__ | zzzeek: are you unclogged on the enginefacade work since the changes got in? | 18:22 |
| zzzeek | dimsum__: ahhmmmmm | 18:22 |
| * zzzeek looks | 18:22 | |
| zzzeek | dimsum__: yeah we merged, and mdbooth has a spec to put it in nova which i think also got approved | 18:22 |
| dimsum__ | cool | 18:22 |
| zzzeek | dimsum__: so i think at this point either mdbooth is going to work on it, or someone will :) | 18:22 |
| dimsum__ | k thanks. rpodolyaka was willing to help if i remember right | 18:23 |
| *** stevedoor has quit IRC | 18:40 | |
| openstackgerrit | Joshua Harlow proposed openstack/taskflow: Remove 2.6 classifier + 2.6 ordereddict requirement https://review.openstack.org/193354 | 19:00 |
| openstackgerrit | Joshua Harlow proposed openstack/tooz: Share most of the `run_watchers` code via a common mixin https://review.openstack.org/191252 | 19:01 |
| *** rpodolyaka1 has joined #openstack-oslo | 19:02 | |
| Rockyg | dhellmann: you around? | 19:06 |
| openstackgerrit | Joshua Harlow proposed openstack/tooz: Expose 'run_elect_coordinator' and call it from 'run_watchers' https://review.openstack.org/193693 | 19:15 |
| *** ViswaV_ has quit IRC | 19:16 | |
| dimsum__ | Rockyg: something i can help with? | 19:16 |
| *** ViswaV has joined #openstack-oslo | 19:16 | |
| Rockyg | dimsum__: don't think so, but maybe. Product_wg had been talking with dhellmann about either a new repo for Usecases, or a new directory under openstack-specs for use cases. Wanted to get info on way to proceed and direction | 19:18 |
| dimsum__ | ack. i dunno :) thought it was probably something related to oslo :) | 19:19 |
| Rockyg | nope. just know Doug's likeliest hangout :-) | 19:19 |
| *** sdake_ has quit IRC | 19:19 | |
| dimsum__ | haha ++ | 19:20 |
| *** achanda has quit IRC | 19:21 | |
| lifeless | dhellmann: answers on 192019 | 19:24 |
| dhellmann | Rockyg: hi! | 19:25 |
| Rockyg | Hey! | 19:25 |
| Rockyg | My assignment from Product_wg is to make progress on getting a use case repo instantiated. | 19:26 |
| dhellmann | lifeless: +2a | 19:26 |
| *** sputnik13 has joined #openstack-oslo | 19:26 | |
| *** achanda has joined #openstack-oslo | 19:27 | |
| dhellmann | Rockyg: ok, I should be able to help with that. How about if we go to #openstack-relmgr-office to keep this channel clear for the oslo folks? | 19:27 |
| lifeless | dhellmann: oh wow, thats going to be a big chunk landing | 19:27 |
| dhellmann | lifeless: land all the patches! | 19:27 |
| Rockyg | Will do. | 19:27 |
| lifeless | dhellmann: if you have time to look at https://review.openstack.org/#/c/189249/ that would be awesome | 19:28 |
| *** mriedem has quit IRC | 19:30 | |
| dhellmann | lifeless: yep, when I'm done helping Rockyg I'm going to go back to reviewing | 19:30 |
| zzzeek | dhellmann and/or lifeless , are you aware of these pbr issues people are hitting | 19:34 |
| zzzeek | such as | 19:34 |
| zzzeek | ValueError: git history requires a target version of pbr.version.SemanticVersion(2014.1.5), but target version is pbr.version.SemanticVersion(2014.1.4) | 19:34 |
| zzzeek | error in setup command: Error parsing /home/cloud-user/testbed/nova/setup.cfg: ValueError: git history requires a target version of pbr.version.SemanticVersion(2014.1.5), but target version is pbr.version.SemanticVersion(2014.1.4) | 19:34 |
| zzzeek | or | 19:34 |
| zzzeek | error in setup command: Error parsing /var/jenkins/workspace/openstack_sqla_master/keystone/setup.cfg: ValueError: git history requires a target version of pbr.version.SemanticVersion(2015.1.1), but target version is pbr.version.SemanticVersion(8.0.0) | 19:34 |
| *** ihrachyshka has joined #openstack-oslo | 19:36 | |
| lifeless | zzzeek: yes. https://www.mail-archive.com/openstack-dev@lists.openstack.org/msg51936.html | 19:38 |
| zzzeek | lifeless: woop | 19:38 |
| lifeless | zzzeek: tl;dr the setup.cfg is incorrect in that context, whatever it is, and rather than ignoring it, or generating backwards versions, pbr is complaining | 19:38 |
| zzzeek | lifeless: OK, just odd that me and some othe rppl seeing this error for the 1st time ever like a day ago | 19:39 |
| lifeless | so you may have just upgraded to a newer pbr | 19:39 |
| lifeless | if you hadn't done anything to force >=0.11 until now | 19:39 |
| lifeless | or you may have done a tag locally that was == the version you had specified | 19:39 |
| zzzeek | lifeless: erm. this is like a jenkins CI that does a full build, and its just like, keystone and nova | 19:40 |
| zzzeek | i dont know that pbr found its way into the system python. hope not | 19:40 |
| *** exploreshaifali has joined #openstack-oslo | 19:40 | |
| lifeless | it's a runtime dep, its certainly likely to be in your repos | 19:41 |
| *** exploreshaifali has quit IRC | 19:41 | |
| zzzeek | lifeless: my “repos”? like my pip cache, sort of thing? | 19:41 |
| *** amotoki_ has quit IRC | 19:41 | |
| fungi | we (well the stable release managers) just started tagging the last icehouse release (2014.1.5) and also the release managers are doing the tag rotation to downgrade to semver-type versions for liberty/master, so that would explain the timing | 19:42 |
| fungi | these are likely the first tags on server projects since the semver-aware pbr | 19:43 |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/oslo.reports: Guru Meditation Reports broken without version_string https://review.openstack.org/193529 | 19:43 |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/oslo.reports: Fix intermittent test failures because of sort order https://review.openstack.org/193705 | 19:43 |
| *** rpodolyaka1 has quit IRC | 19:43 | |
| lifeless | zzzeek: distro repositories | 19:44 |
| lifeless | but fungi's answer also explains it | 19:44 |
| zzzeek | lifeless: im using pip here | 19:44 |
| lifeless | zzzeek: I was answering your 'pbr in system python comment | 19:44 |
| openstackgerrit | Joshua Harlow proposed openstack/tooz: Ensure lock(s) acquire/release returns boolean values https://review.openstack.org/193373 | 19:44 |
| openstackgerrit | Merged openstack/oslo.versionedobjects: fields: report what the wrong type was when coerce fails https://review.openstack.org/193185 | 19:47 |
| *** yamahata has joined #openstack-oslo | 19:48 | |
| lifeless | zzzeek: so in detail | 19:48 |
| *** yamahata has quit IRC | 19:48 | |
| lifeless | zzzeek: the first one - there is a missing commit to icehouse to open up the version window, until thats landed local commits will cause that error. | 19:48 |
| *** yamahata has joined #openstack-oslo | 19:49 | |
| zzzeek | lifeless: Ok | 19:49 |
| lifeless | zzzeek: the second one is the same. | 19:49 |
| zzzeek | lifeless: so it is upstream | 19:49 |
| lifeless | zzzeek: the third one says there is a missing tag in the local repo | 19:49 |
| lifeless | zzzeek: the commit resetting the veresion namespace to 8 is present, but the tag to do it isn't. | 19:49 |
| zzzeek | lifeless: howver, something else important happened here, which is that I noticed in my build failure, it was dumping a full env, and a f’ing aws secret key is in there. so just deleted that and now have to figure out wtf to do about however that key needed to be there | 19:49 |
| lifeless | zzzeek: score | 19:50 |
| zzzeek | lifeless: yes | 19:50 |
| zzzeek | lifeless: so this PBR problem saved my bacon! | 19:50 |
| zzzeek | good job | 19:50 |
| lifeless | fungi: are version-bump commits wending their way ? | 19:50 |
| lifeless | fungi: (to stable-icehouse) | 19:51 |
| fungi | lifeless: no, since those are the last tags, no new changes will be proposed to that branch and in another week i'll tag them and delete them | 19:51 |
| fungi | that is to say, the last commit on e.g. nova stable/icehouse will have a tag of 2014.1.5 and also a tag of icehouse-eol, and there will no longer be a stable/icehouse branch | 19:52 |
| fungi | consistent with how i've been doing them back at least as far as diablo | 19:53 |
| fungi | er, looks like ttx actually put the eol tags on diablo and essex, i didn't take over that until folsom i guess | 19:54 |
| dhellmann | zzzeek, lifeless: fungi is probably right about the timing and cause | 19:54 |
| fungi | as for master, yes i hope there are already setup.cfg patches up to make those target 8.0.0 or whatever? | 19:55 |
| fungi | dhellmann would likely know | 19:55 |
| lifeless | fungi: ok cool | 19:56 |
| lifeless | fungi: master - the 8.0.0 patches seem to be there before the tag :) | 19:56 |
| lifeless | fungi: which should have barfed, so my theory is that the tag is there but didn't propogate to zzzeek's CI repo for some reason | 19:56 |
| fungi | oh, i see. pbr is complaining that the last tag it sees is 2015.1.1 but setup.cfg is targeting 8.0.0 so i guess that was the reason for the alpha tags | 19:57 |
| lifeless | right | 19:57 |
| dhellmann | yeah, so maybe we need to have new alpha tags? | 19:57 |
| lifeless | dhellmann: shouldn't | 19:57 |
| lifeless | let me poke, what repo was it | 19:57 |
| lifeless | keystone | 19:57 |
| fungi | yeah, agreed | 19:58 |
| fungi | it has a 8.0.0a0 already | 19:58 |
| lifeless | I get python setup.py --version | 19:58 |
| lifeless | 8.0.0.0a1.dev4 | 19:58 |
| lifeless | in keystone | 19:58 |
| * dhellmann has only skimmed the scrollback, so might be spouting nonsense | 19:58 | |
| *** ViswaV has quit IRC | 19:58 | |
| lifeless | so the issue is local to zzzeek's (or whoever encountered it) setup | 19:58 |
| dhellmann | oh, I bet he needs to rebase his branch | 19:58 |
| lifeless | if he's merged in the version change in setup.cfg, the tag is also in his history | 19:59 |
| lifeless | I don't think we have side-sensitive code around tags, though I guess its possible | 19:59 |
| dhellmann | if it's an old change up for review, it might not be | 19:59 |
| lifeless | dhellmann: but an old change wouldn't see the new version line | 19:59 |
| dhellmann | true, I didn't realize he was seeing that, too | 19:59 |
| lifeless | dhellmann: (and if it does, the zuul ref created to get that brings in the tag) | 20:00 |
| dhellmann | zzzeek: is this error showing up in a CI job, or locally? | 20:00 |
| fungi | same. if i build an sdist in a current master branch tip i get keystone-8.0.0.0a1.dev4 | 20:01 |
| fungi | no error | 20:01 |
| openstackgerrit | Joshua Harlow proposed openstack/taskflow: Remove kazoo hack/fix for issue no longer needed https://review.openstack.org/182416 | 20:01 |
| *** achanda has quit IRC | 20:03 | |
| *** arnaud___ has joined #openstack-oslo | 20:03 | |
| zzzeek | dhellmann: two different CI jobs | 20:04 |
| zzzeek | dhellmann: one in RH one on my own server | 20:04 |
| *** boris-42 has joined #openstack-oslo | 20:04 | |
| bknudson | we have internal repos too with tags so hit the complaints with pbr, too. | 20:05 |
| dhellmann | zzzeek: ok, the alpha tag I added to that repo was the fix for the problem, so make sure you have that synced locally | 20:05 |
| zzzeek | dhellmann: yeah im on the server now so am jsut going to run the whole thing and see if it still happens | 20:05 |
| dhellmann | zzzeek: ok, let me know and I'll try to help debug | 20:05 |
| dhellmann | bknudson: hrm | 20:05 |
| *** stevemar has joined #openstack-oslo | 20:07 | |
| *** arnaud___ has quit IRC | 20:08 | |
| bknudson | we worked around it... we don't really care about the version of the server packages. | 20:09 |
| dhellmann | bknudson: ok, good | 20:09 |
| *** e0ne has quit IRC | 20:11 | |
| lifeless | bknudson: that was today? or a while back? | 20:21 |
| bknudson | lifeless: it was right after the pbr with the change was released. We run tests often. | 20:21 |
| bknudson | so it was a few weeks ago now | 20:21 |
| *** prad_ has quit IRC | 20:26 | |
| *** ViswaV has joined #openstack-oslo | 20:30 | |
| *** ViswaV_ has joined #openstack-oslo | 20:32 | |
| *** ViswaV has quit IRC | 20:35 | |
| lifeless | bknudson: yeah, I remember - I wasn't sure if you'd hit a new glitch | 20:38 |
| lifeless | bknudson: If you had, I would have been sad. | 20:38 |
| bknudson | maybe zzzeek is running into something new | 20:38 |
| zzzeek | bknudson: ive just wiped out all the .tox dirs and the pip caches and am trying again | 20:39 |
| zzzeek | tehre’s no dist-level pbr on this box bc im working with built pythons anyway | 20:39 |
| lifeless | zzzeek: so there were two distinct failures you had; the keystone one was a missing tag | 20:39 |
| zzzeek | lifeless: ok | 20:39 |
| lifeless | zzzeek: (or perhaps side-sensitive history handling). So .tox and pip caches won't help. We need to look at the tags in the repo. | 20:40 |
| lifeless | zzzeek: for nova, it sounds like you might be adding commits to icehouse locally? | 20:40 |
| zzzeek | lifeless: also wiped out all the git roots which are re-cloning | 20:40 |
| zzzeek | lifeless: nope :) | 20:40 |
| zzzeek | lifeless: this is git pull + tox :) | 20:40 |
| lifeless | zzzeek: so the failure indicates there's at least one commit after the tag | 20:41 |
| *** CaptainMorgan is now known as morgan | 20:41 | |
| bknudson | we commit stuff to keystone all the time | 20:41 |
| lifeless | zzzeek: (and the tag is the same as the version in setup.cfg) | 20:41 |
| zzzeek | lifeless: i dont touch any setup.cfg | 20:41 |
| *** amotoki_ has joined #openstack-oslo | 20:42 | |
| lifeless | zzzeek: I didn't claim you did :) | 20:42 |
| bknudson | http://git.openstack.org/cgit/openstack/keystone/log/ | 20:42 |
| lifeless | bknudson: yes, but zzzeek's pbr failure indicates that the repo it was running out of was missing the 8.0.0.0a0 tag | 20:42 |
| bknudson | is there a way to pull without fetching tags? | 20:43 |
| lifeless | yes | 20:43 |
| lifeless | --no-tags | 20:43 |
| lifeless | or remote.name.tagopt in the config | 20:44 |
| lifeless | but updating refs isn't atoic | 20:44 |
| lifeless | its possible that e.g. one of the infra mirrors failed at the wrong spot | 20:44 |
| lifeless | and needs tickling | 20:44 |
| lifeless | unlikely though | 20:44 |
| lifeless | its a very narrow window | 20:44 |
| *** amotoki_ has quit IRC | 20:47 | |
| *** ihrachyshka has quit IRC | 20:47 | |
| harlowja | sdague do u know if http://logs.openstack.org/63/191163/16/check/gate-tempest-dsvm-neutron-src-futurist/73aedc2/logs/devstacklog.txt.gz ('Cloning not allowed in this configuration') will go away when https://review.openstack.org/#/c/187373/ merges? | 20:56 |
| * harlowja is pretty sure it will, but just wanted to double check | 21:01 | |
| openstackgerrit | Joshua Harlow proposed openstack/taskflow: Perform a few optimizations to decrease persistence interactions https://review.openstack.org/190833 | 21:04 |
| dimsum__ | harlowja: remember the weird mox3 AttributeError with self | 21:05 |
| dimsum__ | harlowja: where did you see it? | 21:05 |
| harlowja | dimsum__ ya | 21:05 |
| harlowja | oslo.service | 21:05 |
| harlowja | but then i got asked to rebase ontop of them switching to mock | 21:06 |
| *** ndipanov has quit IRC | 21:06 | |
| dimsum__ | ah ok | 21:06 |
| dimsum__ | thanks | 21:06 |
| harlowja | dimsum__ http://logs.openstack.org/72/190372/5/check/gate-oslo.service-python34/12f5612/console.html.gz | 21:06 |
| harlowja | 'AttributeError: No values given for arguments: self' | 21:06 |
| harlowja | ^ == weird | 21:07 |
| *** rushiagr_away is now known as rushiagr | 21:08 | |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/mox3: [WIP] Fix broken test and remove Bad test https://review.openstack.org/193726 | 21:10 |
| dimsum__ | harlowja: cool, this one ^^^ fixes that problem | 21:10 |
| harlowja | ah, cool | 21:10 |
| dimsum__ | i checked out 190372/5 and repro'ed the problem and applied my fix and it ran fine | 21:11 |
| harlowja | niceee | 21:11 |
| dimsum__ | am wondering if i should dare pick up another one :) https://bugs.launchpad.net/python-mox3/+bug/1396144 | 21:11 |
| openstack | Launchpad bug 1396144 in python-mox3 "mox3 fails for context manager based mock testing" [Undecided,New] | 21:11 |
| dimsum__ | maybe later... | 21:12 |
| dimsum__ | have a good weekend everyone | 21:12 |
| harlowja | just use mock :-P | 21:14 |
| *** figleaf is now known as edleafe | 21:16 | |
| bknudson | we switched keystoneclient to use mock | 21:19 |
| bknudson | https://review.openstack.org/#/c/191502/ | 21:19 |
| bknudson | never looking back | 21:20 |
| harlowja | :-P | 21:22 |
| *** rjaiswal has joined #openstack-oslo | 21:28 | |
| *** Rockyg has quit IRC | 21:30 | |
| *** ganso_ has quit IRC | 21:38 | |
| *** kgiusti has left #openstack-oslo | 21:39 | |
| openstackgerrit | Flavio Percoco proposed openstack/oslo.messaging: Don't use dynamic sources/targets https://review.openstack.org/193144 | 21:50 |
| bknudson | https://review.openstack.org/#/c/193732/3 -- switch keystone to oslo.service | 21:50 |
| bknudson | +7, -1112 | 21:51 |
| *** stevemar has quit IRC | 21:52 | |
| *** arnaud___ has joined #openstack-oslo | 21:52 | |
| bknudson | zzzeek: looks like sqlalchemy upgrade breaks keystone unit tests -- http://logs.openstack.org/05/190405/7/check/gate-keystone-python27/46ab1b6/console.html#_2015-06-16_19_32_44_080 | 21:55 |
| bknudson | this is from https://review.openstack.org/#/c/190405/7 | 21:56 |
| zzzeek | bknudson: so, why would sqlalchemy-migrate be here attempting to run an invalid operation on SQlite ? | 21:57 |
| *** arnaud___ has quit IRC | 21:57 | |
| *** rushiagr is now known as rushiagr_away | 22:01 | |
| zzzeek | bknudson: also, wonder if anyone has run sqlalchemy-migrate tests with 1.0 :) | 22:01 |
| openstackgerrit | Joshua Harlow proposed openstack/taskflow: Retain atom `revert` result (or failure) https://review.openstack.org/192942 | 22:01 |
| bknudson | zzzeek: it's calling conn.execute(schema.DropConstraint(fkc)) | 22:01 |
| bknudson | and if I add an if-check for sqlite the tests now pass. | 22:02 |
| *** sputnik13 has quit IRC | 22:03 | |
| zzzeek | bknudson: so how did that work before | 22:04 |
| *** sputnik13 has joined #openstack-oslo | 22:04 | |
| *** isq has joined #openstack-oslo | 22:11 | |
| bknudson | zzzeek: https://review.openstack.org/#/c/193741/ | 22:13 |
| bknudson | I'll see if I can figure out what older sqlalchemy did | 22:14 |
| *** yamahata has quit IRC | 22:14 | |
| zzzeek | bknudson: whole suite passes after that? | 22:14 |
| bknudson | yep | 22:14 |
| zzzeek | sqlalchemy-migrate must have something it deos | 22:14 |
| bknudson | zzzeek: with old sqlalchemy, inspector.get_foreign_keys(table_name) returned 0 foreign keys | 22:18 |
| zzzeek | bknudson: oh oh right | 22:18 |
| zzzeek | bknudson: :) | 22:18 |
| zzzeek | yeah thats new | 22:18 |
| zzzeek | e.g. it was broke before | 22:18 |
| zzzeek | so yes | 22:18 |
| *** dimsum__ has quit IRC | 22:19 | |
| bknudson | y, definitely quesionable. | 22:19 |
| zzzeek | i only run a subset of keystone tests and i wasnt doing migrations | 22:19 |
| zzzeek | bknudson: the patch you have it the correct approach | 22:19 |
| bknudson | and I'll admit I don't care that much since it's sqlite. | 22:19 |
| *** dimsum__ has joined #openstack-oslo | 22:20 | |
| zzzeek | bknudson: there’s no need to drop the FKs on SQLite because they aren’t enforcing anyway | 22:21 |
| zzzeek | bknudson: even if you make SQLite fks enforce htey still let you drop the tables | 22:21 |
| zzzeek | because SQlite is kind of …crap :) | 22:21 |
| harlowja | http://www.sqlite.org/opcode.html is neat though :-P | 22:22 |
| harlowja | and/or http://www.sqlite.org/vdbe.html | 22:22 |
| bknudson | we have had bugs where constraints weren't working on mysql because tests only cover sqlite. | 22:22 |
| * harlowja didn't know that sqlite had a VM layer in it, till i read all that | 22:26 | |
| zzzeek | bknudson: yep, the whole “lets let devs run openstack against sqlite” is a silly use case IMO | 22:26 |
| harlowja | *or VM-like-layer | 22:26 |
| bknudson | every database manager must do something like that. | 22:27 |
| bknudson | db2 has a neat visual explain tool that I've used to optimize | 22:28 |
| bknudson | although optimizing was mostly just -- hey, it's a table scan, don't do that. | 22:28 |
| *** jecarey has quit IRC | 22:29 | |
| *** bknudson has quit IRC | 22:30 | |
| *** dimsum__ has quit IRC | 22:32 | |
| *** stpierre has quit IRC | 22:35 | |
| *** browne has quit IRC | 22:42 | |
| *** amotoki_ has joined #openstack-oslo | 22:43 | |
| harlowja | :) | 22:46 |
| *** amotoki_ has quit IRC | 22:49 | |
| *** rpodolyaka1 has joined #openstack-oslo | 22:52 | |
| *** ViswaV_ has quit IRC | 22:57 | |
| *** zzzeek has quit IRC | 22:58 | |
| *** zzzeek has joined #openstack-oslo | 23:01 | |
| *** zigo has quit IRC | 23:05 | |
| *** zigo has joined #openstack-oslo | 23:07 | |
| *** jgrimm is now known as zz_jgrimm | 23:10 | |
| *** zzzeek has quit IRC | 23:15 | |
| *** jlvillal has quit IRC | 23:21 | |
| *** vilobhmm has quit IRC | 23:23 | |
| *** jlvillal has joined #openstack-oslo | 23:33 | |
| *** arnaud___ has joined #openstack-oslo | 23:41 | |
| *** arnaud___ has quit IRC | 23:45 | |
| *** hogepodge has quit IRC | 23:45 | |
| *** browne has joined #openstack-oslo | 23:52 | |
| openstackgerrit | Joshua Harlow proposed openstack/oslo.concurrency: Use monotonic.monotonic() instead of time.time() https://review.openstack.org/193759 | 23:58 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!