*** achanda has quit IRC | 00:00 | |
*** achanda has joined #openstack-oslo | 00:02 | |
*** yamahata has joined #openstack-oslo | 00:09 | |
*** mriedem1 has joined #openstack-oslo | 00:23 | |
*** mriedem has quit IRC | 00:24 | |
*** dimsum__ has joined #openstack-oslo | 00:35 | |
*** dims_ has joined #openstack-oslo | 00:36 | |
*** dimsum__ has quit IRC | 00:39 | |
*** oomichi has joined #openstack-oslo | 00:41 | |
*** achanda has quit IRC | 00:52 | |
*** daniel3_ has joined #openstack-oslo | 01:30 | |
*** daniel3_ has quit IRC | 01:32 | |
*** jraim has quit IRC | 01:38 | |
*** zhiyan has quit IRC | 01:40 | |
*** jecarey has joined #openstack-oslo | 01:45 | |
*** jraim has joined #openstack-oslo | 01:45 | |
*** zhiyan has joined #openstack-oslo | 01:46 | |
*** achanda has joined #openstack-oslo | 01:52 | |
*** achanda has quit IRC | 01:57 | |
*** Ritz has joined #openstack-oslo | 02:41 | |
Ritz | good evening !! Does Any one know why i am getting this error https://bugs.launchpad.net/oslo.config/+bug/1419585 | 02:41 |
---|---|---|
openstack | Launchpad bug 1419585 in oslo.config ""ImportError: cannot import name types" when doing import oslo_config.cfg" [Undecided,New] | 02:41 |
*** sigmavirus24_awa is now known as sigmavirus24 | 02:42 | |
sigmavirus24 | Ritz: you still around? | 02:58 |
sigmavirus24 | Ritz: can you put the output of `ls /opt/stack/venv/lib/python2.7/site-packages/oslo.config-1.6.0-py2.7.egg/oslo/config/` on paste.openstack.org and share the link? | 02:59 |
*** sigmavirus24 is now known as sigmavirus24_awa | 03:40 | |
*** mriedem1 has quit IRC | 03:40 | |
*** miqui has quit IRC | 03:41 | |
*** BrianShang has joined #openstack-oslo | 03:49 | |
*** rushiagr_away is now known as rushiagr | 03:53 | |
*** amotoki has joined #openstack-oslo | 04:13 | |
*** BrianShang has quit IRC | 04:22 | |
*** dims_ has quit IRC | 04:23 | |
*** BrianShang has joined #openstack-oslo | 04:24 | |
*** rushiagr is now known as rushiagr_away | 04:24 | |
*** rushiagr_away is now known as rushiagr | 04:31 | |
*** achanda has joined #openstack-oslo | 04:39 | |
*** dimsum__ has joined #openstack-oslo | 05:24 | |
*** dimsum__ has quit IRC | 05:29 | |
*** achanda has quit IRC | 05:38 | |
*** stevemar has quit IRC | 05:41 | |
*** boris-42 has quit IRC | 05:52 | |
*** achanda has joined #openstack-oslo | 05:57 | |
*** ViswaV has joined #openstack-oslo | 06:08 | |
*** ViswaV_ has joined #openstack-oslo | 06:12 | |
*** ViswaV has quit IRC | 06:12 | |
*** achanda has quit IRC | 06:22 | |
*** boris-42 has joined #openstack-oslo | 06:26 | |
*** pcaruana has quit IRC | 06:28 | |
Ritz | hello sigmavirus24 | 06:39 |
Ritz | i just logged in again | 06:39 |
Ritz | here is the output you asked | 06:42 |
Ritz | shettyri@managementnode:~$ ls -lrta /opt/stack/venv/lib/python2.7/site-packages/oslo.config-1.6.0-py2.7.egg/oslo/config/ total 64 -rw-r--r-- 1 root root 613 Feb 6 08:16 types.py -rw-r--r-- 1 root root 874 Feb 6 08:16 __init__.py -rw-r--r-- 1 root root 617 Feb 6 08:16 iniparser.py -rw-r--r-- 1 root root 617 Feb 6 08:16 generator.py -rw-r--r-- 1 root root 615 Feb 6 08:16 fixture.py -rw-r--r-- 1 root root 611 Feb 6 08:1 | 06:42 |
Ritz | it does have the types.py | 06:43 |
*** vigneshvar has joined #openstack-oslo | 07:02 | |
*** achanda has joined #openstack-oslo | 07:15 | |
*** jaosorior has joined #openstack-oslo | 07:35 | |
*** stevemar has joined #openstack-oslo | 07:35 | |
*** noelbk has quit IRC | 07:36 | |
*** ViswaV_ has quit IRC | 07:46 | |
*** achanda has quit IRC | 07:47 | |
*** achanda has joined #openstack-oslo | 07:51 | |
*** e0ne has joined #openstack-oslo | 07:55 | |
*** dulek_ has joined #openstack-oslo | 08:00 | |
*** e0ne has quit IRC | 08:03 | |
*** ajo has quit IRC | 08:10 | |
*** dtantsur|afk is now known as dtantsur | 08:10 | |
*** ajo has joined #openstack-oslo | 08:10 | |
*** shardy has joined #openstack-oslo | 08:17 | |
*** mtanino has quit IRC | 08:20 | |
*** stevemar has quit IRC | 08:20 | |
*** ihrachyshka has joined #openstack-oslo | 08:38 | |
*** andreykurilin_ has joined #openstack-oslo | 08:40 | |
*** oomichi has quit IRC | 08:42 | |
*** achanda has quit IRC | 08:52 | |
*** dulek_ has quit IRC | 08:55 | |
*** achanda has joined #openstack-oslo | 08:56 | |
*** dulek has joined #openstack-oslo | 08:56 | |
*** Ritz has quit IRC | 08:58 | |
*** dulek has quit IRC | 09:03 | |
*** dulek has joined #openstack-oslo | 09:04 | |
*** vigneshvar has quit IRC | 09:05 | |
*** achanda has quit IRC | 09:08 | |
*** vigneshvar has joined #openstack-oslo | 09:17 | |
*** achanda has joined #openstack-oslo | 09:18 | |
*** yamahata has quit IRC | 09:31 | |
*** andreykurilin_ has quit IRC | 09:41 | |
*** achanda has quit IRC | 09:46 | |
*** denis_makogon has joined #openstack-oslo | 09:50 | |
openstackgerrit | Denis M. proposed openstack/oslo-specs: Added spec for splitting common requirements into driver-specific https://review.openstack.org/153966 | 09:52 |
*** dtantsur is now known as dtantsur|brb | 10:00 | |
*** e0ne has joined #openstack-oslo | 10:00 | |
*** jd__ has quit IRC | 10:06 | |
*** jd__ has joined #openstack-oslo | 10:07 | |
*** e0ne has quit IRC | 10:09 | |
*** e0ne has joined #openstack-oslo | 10:09 | |
jd__ | is anybody working on pbr extra require? did we regain control of pbr to the point we can do a release? | 10:12 |
openstackgerrit | Radomir Dopieralski proposed openstack/oslo-incubator: Log deprecation message when catching deprecated exceptions https://review.openstack.org/123383 | 10:12 |
denis_makogon | jd__, hi there, i'm looking at it right now | 10:13 |
jd__ | denis_makogon: cool! | 10:13 |
jd__ | I've been frustrated about the lack of extra_require support for a large amount of time | 10:13 |
denis_makogon | jd__, there's small problem, i'm pretty new to pbr, so right now i'm trying to figure out how can i do that | 10:14 |
denis_makogon | jd__, oh, i see | 10:14 |
jd__ | denis_makogon: ok, let me know if I can help | 10:14 |
jd__ | though I've not hacked on pbr for a while | 10:14 |
denis_makogon | jd__, correct me if i'm wrong, extras_require is the separate section in setup.cfg but not the part of metadata section | 10:17 |
jd__ | denis_makogon: you mean, it should be or it is? AFAIK it's not anywhere since it's not supported | 10:19 |
*** e0ne is now known as e0ne_ | 10:19 | |
denis_makogon | jd__, never-mind, already figured out =) | 10:20 |
jd__ | ok | 10:20 |
jd__ | it does not seem to be even defined in distutils2 | 10:20 |
jd__ | so I'd build the support like we have requirements.txt | 10:20 |
*** e0ne_ has quit IRC | 10:24 | |
*** e0ne has joined #openstack-oslo | 10:24 | |
*** pblaho has joined #openstack-oslo | 10:25 | |
denis_makogon | jd__, i was trying to inject extras_require directly through setuptools.setup method, and it does work for me | 10:29 |
jd__ | could you detail? | 10:31 |
*** vigneshvar has quit IRC | 10:37 | |
denis_makogon | jd__, https://gist.github.com/denismakogon/d397d6d09415b7eb8760 | 10:39 |
*** vigneshvar has joined #openstack-oslo | 10:40 | |
jd__ | denis_makogon: ah, yeah I know that works for decades | 10:41 |
jd__ | denis_makogon: does that work with setup.cfg? | 10:41 |
denis_makogon | jd__, no, it doesn't, at least with pbr | 10:42 |
jd__ | ok so back to square one | 10:42 |
*** eezhova has joined #openstack-oslo | 10:47 | |
*** andrearosa has quit IRC | 10:55 | |
*** andrearosa has joined #openstack-oslo | 10:56 | |
*** dtantsur|brb is now known as dtantsur | 10:59 | |
*** dulek has quit IRC | 11:03 | |
*** dulek has joined #openstack-oslo | 11:04 | |
*** e0ne is now known as e0ne_ | 11:08 | |
*** dulek has quit IRC | 11:09 | |
*** dulek has joined #openstack-oslo | 11:09 | |
*** dulek has quit IRC | 11:11 | |
*** dulek has joined #openstack-oslo | 11:11 | |
ihrachyshka | flaper87, hey. do you by chance know what's the status of redis in oslo.msg? | 11:12 |
ihrachyshka | flaper87, I see a unit test failure that suggests that its implementation is broken, so I wonder whether it's worth reporting a bug etc. | 11:12 |
*** e0ne_ has quit IRC | 11:14 | |
*** dulek_ has joined #openstack-oslo | 11:14 | |
*** dulek has quit IRC | 11:17 | |
*** pblaho has quit IRC | 11:29 | |
*** pblaho has joined #openstack-oslo | 11:29 | |
*** pblaho has quit IRC | 11:30 | |
*** e0ne has joined #openstack-oslo | 11:52 | |
*** cdent has joined #openstack-oslo | 11:54 | |
*** e0ne is now known as e0ne_ | 11:58 | |
*** e0ne_ is now known as e0ne | 11:59 | |
denis_makogon | jd__, i guess i'm ready to push some code onto review, can you try it out if it works for you as well as for me? | 12:05 |
jd__ | denis_makogon: send me the review link | 12:19 |
denis_makogon | jd__, i will send you a link yo github repo, since i need to figure out if i'm not braking any tests. | 12:20 |
*** rushiagr is now known as rushiagr_away | 12:35 | |
*** inc0 has joined #openstack-oslo | 12:44 | |
*** dimsum__ has joined #openstack-oslo | 12:44 | |
*** takedakn has joined #openstack-oslo | 12:45 | |
*** dims_ has joined #openstack-oslo | 12:45 | |
*** dimsum__ has quit IRC | 12:49 | |
*** takedakn has quit IRC | 12:49 | |
*** takedakn has joined #openstack-oslo | 12:50 | |
*** e0ne is now known as e0ne_ | 12:59 | |
*** dulek_ is now known as dulek | 13:02 | |
*** e0ne_ is now known as e0ne | 13:06 | |
denis_makogon | jd__, https://github.com/denismakogon/pbr/tree/extras_require-feature | 13:10 |
denis_makogon | jd__, i could provide some info how to test it, if needed | 13:12 |
*** eezhova has quit IRC | 13:14 | |
jd__ | denis_makogon: I hope you duplicated a lot of code as the parsing of that is probably the same for requirements | 13:14 |
denis_makogon | jd__, i did it my own manner, a bit =) | 13:14 |
denis_makogon | * in my own | 13:14 |
jd__ | yeah but that's gonna make it :) | 13:15 |
*** takedakn has quit IRC | 13:15 | |
denis_makogon | jd__, so, anyway, it does work for both cases: explicit requirements (env1 = a b c d) and redirections (env1 = -r somewhere.txt) | 13:16 |
*** rushiagr_away is now known as rushiagr | 13:16 | |
*** e0ne is now known as e0ne_ | 13:30 | |
*** bknudson has quit IRC | 13:36 | |
*** gordc has joined #openstack-oslo | 13:37 | |
*** mtanino has joined #openstack-oslo | 13:37 | |
*** e0ne_ is now known as e0ne | 13:38 | |
*** jecarey has quit IRC | 13:38 | |
*** rushiagr is now known as rushiagr_away | 13:41 | |
*** _amrith_ is now known as amrith | 13:47 | |
openstackgerrit | Merged openstack-dev/pbr: Fixes Windows generated scripts headers https://review.openstack.org/151595 | 13:51 |
*** vigneshvar has quit IRC | 13:55 | |
*** bknudson has joined #openstack-oslo | 13:59 | |
*** david-lyle_afk is now known as david-lyle | 14:01 | |
*** e0ne has quit IRC | 14:05 | |
*** e0ne has joined #openstack-oslo | 14:05 | |
*** amotoki has quit IRC | 14:09 | |
*** inc0 has quit IRC | 14:09 | |
*** jaypipes has joined #openstack-oslo | 14:15 | |
*** jecarey has joined #openstack-oslo | 14:19 | |
*** e0ne is now known as e0ne_ | 14:23 | |
openstackgerrit | Denis M. proposed openstack-dev/pbr: Added extras_require feature https://review.openstack.org/154057 | 14:26 |
denis_makogon | jd__, done, now you can do the review https://review.openstack.org/154057 | 14:26 |
openstackgerrit | Denis M. proposed openstack-dev/pbr: Added extras_require feature https://review.openstack.org/154057 | 14:28 |
*** rushiagr_away is now known as rushiagr | 14:29 | |
openstackgerrit | Denis M. proposed openstack-dev/pbr: Added extras_require feature https://review.openstack.org/154057 | 14:41 |
*** tedross has joined #openstack-oslo | 14:48 | |
*** e0ne_ is now known as e0ne | 14:49 | |
openstackgerrit | Thomas Goirand proposed openstack/oslo.middleware: Adds missing files https://review.openstack.org/154063 | 14:49 |
*** kgiusti has joined #openstack-oslo | 14:55 | |
amrith | dhellmann, I took an action item at the trove mid-cycle to speak with you about the oslo stand on namespaces in the context of review https://review.openstack.org/#/c/150709/. The change proposes to use oslo_* instead of oslo.* for imports. Is this a change being mandated by oslo, if yes, is it required to be made by a certain time? Thanks! | 14:59 |
amrith | dhellmann, ^^ | 14:59 |
*** mriedem has joined #openstack-oslo | 15:01 | |
ihrachyshka | amrith, it's expected old namespace will go away during L cycle | 15:04 |
ihrachyshka | amrith, so you better be prepared | 15:04 |
amrith | ihrachyshka, thanks. So I guess this change isn't mandatory for K cycle. That was primarily what we were looking for. | 15:05 |
ihrachyshka | amrith, I don't see hacking rule there, I guess you would like to have it not to introduce new instances of old imports | 15:05 |
*** ChuckC has joined #openstack-oslo | 15:06 | |
*** ChuckC has quit IRC | 15:06 | |
*** ChuckC has joined #openstack-oslo | 15:07 | |
*** stevemar has joined #openstack-oslo | 15:08 | |
zigo | The oslo namespace warning is a very good idea. But the unit test for it isn't. | 15:09 |
amrith | was there a recommended rule that we should include? | 15:09 |
zigo | It's making my package fail to build ! :( | 15:09 |
zigo | Why? Well... because it uses the namespace... | 15:10 |
*** amotoki has joined #openstack-oslo | 15:10 | |
zigo | dhellmann: ^ | 15:10 |
amrith | ihrachyshka, thanks for the review. | 15:12 |
ihrachyshka | amrith, there was no common rule, everyone is reinventing wheel (or stealing from neighbour) | 15:17 |
ihrachyshka | amrith, there were suggestions to add rule into hacking, though iirc dhellmann was not really impressed by it | 15:17 |
amrith | ihrachyshka, I'm seeing that. I think I'll wait till someone comes up with a good and working wheel and then we'll adopt it ;) Right now a bunch of wheels appear to be pretty and square leading to a bumpy ride. | 15:17 |
denis_makogon | jd__, ping | 15:18 |
ihrachyshka | amrith, neutron rule is merged, I think it's solid | 15:21 |
ihrachyshka | (cause I wrote it lol) | 15:21 |
amrith | ihrachyshka, i see, the 'unbiased' view ;) | 15:21 |
amrith | will follow that, I notice you referenced it in your comment. | 15:21 |
amrith | at least for now, we'll wait. that's what we decided in the mid-cycle. we figured that if there was an urgency to get it in for Kilo we'd do it but if it was a mandate for L release, we'd wait till post Kilo. | 15:22 |
ihrachyshka | amrith, I wonder why you're afraid of merging it in. it's quite a safe step | 15:31 |
amrith | ihrachyshka, let me bring that back to the team and see what they think. | 15:32 |
ihrachyshka | sure, just wondering | 15:32 |
amrith | ihrachyshka, I don't have a good answer for you. I didn't understand the need for the change and so I couldn't specifically say. | 15:33 |
amrith | I brought the issue up and therefore took the action item. | 15:33 |
amrith | let me follow up with the team and I'll ping you with an answer. | 15:33 |
ihrachyshka | ok, thanks | 15:33 |
dhellmann | zigo: why is a test making your packaging fail? is the test failing? | 15:34 |
zigo | dhellmann: Correct. | 15:34 |
dhellmann | amrith: I would *like* to drop namespace packages next cycle | 15:35 |
dhellmann | amrith: that's not a definite plan, yet | 15:35 |
dhellmann | zigo: so why is the test failing? | 15:35 |
*** achanda has joined #openstack-oslo | 15:35 | |
zigo | dhellmann: Plus on *many* packages, the "python setup.py install" is failling to install so many stuff, and I have to patch MANIFEST.in | 15:35 |
amrith | dhellmann, that is what was mentioned at the mid-cycle. so we felt we could wait a cycle. but to ihrachyshka's point, this may not be something of such risk that we need to wait. | 15:36 |
zigo | dhellmann: Same stuff as before: because of the Python 3 namespace thing. If I have an oslo package installed which has namespace, then anything local will fail. | 15:36 |
amrith | I will bring it up to trove team and get back with an answer. | 15:36 |
dhellmann | ihrachyshka: it would really be better if you didn't wait | 15:36 |
dhellmann | zigo: this is the problem we're trying to solve by moving away from namespace packages | 15:37 |
zigo | dhellmann: Yes, and your namespace test fails for the same reason !!! | 15:37 |
zigo | dhellmann: So it's easy: you can't do that test (or I have to blacklist it when running testr). | 15:37 |
dhellmann | zigo: it doesn't fail on any of our test systems, so what is different about your setup? | 15:37 |
dhellmann | ok, that seems like a fair solution | 15:37 |
ihrachyshka | dhellmann, neutron is patched already, all four repos | 15:37 |
zigo | dhellmann: The difference is that you're using a venv, and I'm using packages. | 15:37 |
dhellmann | ihrachyshka: sorry, I meant that for amrith (misread the "from" on that line -- I blame lack of caffeine) | 15:38 |
zigo | dhellmann: On a venv, no problem, on my Debian system, where requirements are installed by build-dependencies, it always fails... | 15:38 |
zigo | dhellmann: Do you prefer that I just blacklist the test_warning tests? | 15:38 |
zigo | IIRC, I even have to *delete* the file, because listing it is a problem. | 15:39 |
dhellmann | zigo: yes, just blacklist it; as I was just saying the goal is to remove the obsolete namespace next cycle | 15:39 |
zigo | dhellmann: Right, and I salute the goal ! :) | 15:39 |
dhellmann | zigo: if you're running the tests yourself, you could just run the ones inside the package (there are 2 copies of some of the tests) | 15:39 |
dhellmann | that is, if you're not using tox to run the tests | 15:39 |
zigo | I'm not using tox. | 15:40 |
zigo | I'm just using testr run --subunit | subunit2pyunit | 15:40 |
dhellmann | ok, so run the tests in ./oslo_*/tests and ignore ./tests | 15:40 |
zigo | Oh... | 15:40 |
dhellmann | oh, you probably need a different .testr.conf for that though | 15:40 |
dhellmann | and you don't want to maintain that | 15:40 |
zigo | dhellmann: So, stuff in ./tests are a copy of oslo_*/tests ? | 15:40 |
dhellmann | ./tests has tests that use the namespace package to verify that we don't break that until we can delete it | 15:41 |
dhellmann | ./oslo_*/tests has the new import style | 15:41 |
dhellmann | we're not adding tests to ./tests | 15:41 |
zigo | dhellmann: Good to know. | 15:41 |
zigo | Thanks. | 15:41 |
zigo | dhellmann: The other big issue I'm having is that setup.py install doesn't install everything, it used to ... | 15:41 |
zigo | dhellmann: Could this be because I'm using an outdated pbr version ? | 15:42 |
*** achanda has quit IRC | 15:42 | |
dhellmann | zigo: possibly | 15:42 |
zigo | (0.10.0 instead of 0.10.7 or something...) | 15:42 |
zigo | dhellmann: Ok, I'll try to upgrade pbr, and I'll let you know then. | 15:42 |
dhellmann | we're building wheels successfully so something is figuring out the right manifest | 15:42 |
dhellmann | make sure you delete any .egg-info directories before building, too | 15:42 |
dhellmann | I've found that some of the metadata isn't rebuilt properly if it already exists | 15:43 |
dhellmann | including the filelist | 15:43 |
*** tsekiyama has joined #openstack-oslo | 15:46 | |
*** hemnafk is now known as hemna | 15:46 | |
zigo | dhellmann: I'm building using a git clone, so I don't get the egg-info folder. | 15:47 |
dhellmann | zigo: ok, good | 15:47 |
zigo | dhellmann: The thing is, if I do a "python setup.py install" from the git, it works, if I do it outside of it, it somehow fails to copy all folders. | 15:47 |
dhellmann | why would you do it anywhere other than from the root of the project? | 15:48 |
zigo | dhellmann: I'm using git-buildpackage and a cowbuilder, that's my normal process. | 15:48 |
zigo | That does a copy of all in a working directory. | 15:48 |
zigo | (after creating the orig.tar.xz and the debian.tar.xz | 15:48 |
zigo | ) | 15:49 |
dhellmann | does it copy the .git directory, too? | 15:49 |
zigo | Nop, that's the point ! | 15:49 |
zigo | Because the .git folder shouldn't be in the orig.tar.xz. | 15:50 |
dhellmann | ok, so you need to build a valid filelist before you do that copy | 15:51 |
*** zz_jgrimm is now known as jgrimm | 15:51 | |
dhellmann | normally pbr looks at the git history, so if you don't have that how are have you told it about files in the past? | 15:51 |
*** ChuckC has quit IRC | 15:51 | |
zigo | Well, in the past, it just worked... | 15:51 |
dhellmann | perhaps this is a bug in pbr, then | 15:52 |
zigo | It magically doesn't, somehow, for all the latest releases. | 15:52 |
*** ChuckC has joined #openstack-oslo | 15:52 | |
dhellmann | try running "python setup.py egg_info" in the dir with the git history and then copying the *.egg_info directory it builds over to where you're packaging | 15:52 |
zigo | dhellmann: Good news, it seems that upgrading to pbr 0.10.7 solved the issue ! :) | 15:52 |
dhellmann | cool | 15:53 |
zigo | dhellmann: I spoke too fast. | 15:53 |
zigo | dhellmann: Folders inside oslo_messaging were *not* installed. The oslo/messaging folder was though. | 15:53 |
zigo | Hum... | 15:54 |
zigo | No, not even. | 15:54 |
zigo | it stoped at oslo/messaging, but for example, oslo/messaging/rpc wasn't isntalled. | 15:55 |
dhellmann | zigo: I have a meeting coming up in a couple of minutes. Maybe you should go ahead and file a bug for this, so we can collect all of the info. Keep in mind as you write it up that I don't know anything about Debian packaging, so you may have to tell me commands to run that seem obvious to you. | 15:55 |
dhellmann | zigo: also, try the egg_info test I mentioned | 15:55 |
zigo | dhellmann: Thanks. | 15:56 |
zigo | Will do. | 15:56 |
dhellmann | it sounds like maybe pbr is not finding packages properly when git history isn't there | 15:56 |
*** dims_ has quit IRC | 15:56 | |
*** harlowja_at_home has joined #openstack-oslo | 15:56 | |
openstackgerrit | Oleksii Zamiatin proposed openstack/oslo-specs: Implement Call via Request-Reply pattern https://review.openstack.org/154094 | 15:59 |
beekneemech | zigo: dhellmann: I think Dan Prince hit the same issue packaging after the namespace changes. | 15:59 |
beekneemech | He fixed some of them by explicitly adding the namespace packages to setup.cfg. | 15:59 |
*** beekneemech is now known as bnemec | 15:59 | |
* bnemec notices that it's not Friday anymore | 15:59 | |
dhellmann | bnemec: yeah, that shouldn't be necessary -- that's one of the things pbr is supposed to do for us | 16:00 |
openstackgerrit | Oleksii Zamiatin proposed openstack/oslo-specs: Implement Call via Request-Reply pattern https://review.openstack.org/154094 | 16:00 |
*** dimsum__ has joined #openstack-oslo | 16:01 | |
*** dimsum__ has quit IRC | 16:08 | |
*** dimsum__ has joined #openstack-oslo | 16:10 | |
*** dimsum__ has quit IRC | 16:11 | |
*** dimsum__ has joined #openstack-oslo | 16:12 | |
*** dimsum__ has quit IRC | 16:13 | |
*** dimsum__ has joined #openstack-oslo | 16:13 | |
*** dimsum__ has quit IRC | 16:14 | |
*** dimsum__ has joined #openstack-oslo | 16:17 | |
*** zzzeek has joined #openstack-oslo | 16:17 | |
*** daniel3_ has joined #openstack-oslo | 16:27 | |
*** flashgordon is now known as jogo | 16:35 | |
*** daniel3_ has quit IRC | 16:48 | |
openstackgerrit | Matthew Treinish proposed openstack/debtcollector: Add a removal decorator https://review.openstack.org/153429 | 16:49 |
*** daniel3_ has joined #openstack-oslo | 16:50 | |
*** vigneshvar has joined #openstack-oslo | 16:55 | |
*** dulek has quit IRC | 16:58 | |
*** tedross has left #openstack-oslo | 17:02 | |
*** amotoki has quit IRC | 17:03 | |
bnemec | dhellmann: For reference, no subprocess patch: https://github.com/eventlet/eventlet/blob/master/eventlet/patcher.py#L243 | 17:04 |
bnemec | And thread only patches Queue, thread, and threading: https://github.com/eventlet/eventlet/blob/master/eventlet/patcher.py#L328 | 17:04 |
dhellmann | ugh | 17:04 |
dhellmann | sileht: have you tried the threading executor in a real setup? I wonder what would have to be changed in the apps to make that work | 17:05 |
sileht | dhellmann, no I have not tested it | 17:05 |
*** ihrachyshka has quit IRC | 17:06 | |
* harlowja_at_home wonders if that would work for the keystone folks better than the eventlet one (in a non-eventlet situation that they are aiming for) | 17:06 | |
* stevemar hears a disturbance in the keystone force... | 17:07 | |
sileht | harlowja_at_home, keystone use only the notifier client, it's just a blocking code, even with eventlet (except if you monkeypatch it) | 17:08 |
harlowja_at_home | ya, so it stays blocking if they use eventlet executor (without patching anything) | 17:09 |
harlowja_at_home | as long as they avoid patching anything (if they do it gets weird quick) | 17:09 |
sileht | harlowja_at_home, you don't define an executor for the Notifier and the RPCClient | 17:09 |
harlowja_at_home | k | 17:09 |
sileht | harlowja_at_home, just the server parts use executor to dispatch them into threads/greenthread/corotine/whatever | 17:10 |
harlowja_at_home | ok dokie | 17:10 |
openstackgerrit | Mehdi Abaakouk proposed openstack/oslo.messaging: TEST https://review.openstack.org/154129 | 17:11 |
harlowja_at_home | bb | 17:11 |
*** harlowja_at_home has quit IRC | 17:11 | |
*** isviridov_away is now known as isviridov_vacati | 17:13 | |
*** isviridov_vacati is now known as isviridov_vction | 17:14 | |
sileht | kgiusti, hi | 17:14 |
sileht | kgiusti, I have setuped the amqp1 jobs on gate, but it seems the installation of python-qpid-proton from pypi fail on fedora21: | 17:15 |
sileht | http://logs.openstack.org/47/146047/20/check/gate-oslo.messaging-dsvm-functional-amqp1-f21/d08c55c/console.html.gz | 17:15 |
*** tsekiyama has quit IRC | 17:16 | |
sileht | kgiusti, tell me if you have an idea, or where should I report the bug issue | 17:17 |
*** crc32 has joined #openstack-oslo | 17:27 | |
*** noelbk has joined #openstack-oslo | 17:27 | |
*** zigo has quit IRC | 17:29 | |
stevemar | dhellmann, if we get this review +A'ed then we can pretty much resolve all the patches for oslo.policy, figured you would want one last look at it: https://review.openstack.org/#/c/153310/ | 17:32 |
*** elmiko has joined #openstack-oslo | 17:32 | |
dhellmann | stevemar: ok | 17:33 |
elmiko | hey folks, i'm curious about ssl configs with oslo.messaging. is it the case that i need to set qpid_protocol=ssl _or_ rabbit_use_ssl=true, and then the kombu_* configs inform about where the files are for either impl? | 17:34 |
sileht | dhellmann, without eventlet monkeypatch and with threading executor, functional tests suite pass: https://jenkins05.openstack.org/job/gate-oslo.messaging-dsvm-functional-rabbit/6/console | 17:35 |
sileht | the assiociate review: https://review.openstack.org/#/c/154129/ | 17:35 |
dhellmann | stevemar: is oslo.policy in the list of projects in global requirements? we usually let the bot do the requirements syncs | 17:37 |
dhellmann | sileht: that's good news. I'd be interested to see if tempest would pass with nova configured that way. | 17:37 |
sileht | dhellmann, lets me send a patch :) | 17:38 |
sileht | dhellmann, if you want to follow: https://review.openstack.org/154140 | 17:39 |
*** kbyrne has quit IRC | 17:40 | |
sileht | it's quick and dirty, but it should at least start | 17:40 |
dhellmann | sileht: nice | 17:41 |
stevemar | dhellmann, rookie mistake, it's not. i'll add it | 17:41 |
stevemar | the hacking upgrade patch is still valid as GR doesn't update that i believe | 17:41 |
stevemar | i knew there was a reason i split those up :D | 17:42 |
dhellmann | yes, that's right | 17:42 |
*** kbyrne has joined #openstack-oslo | 17:44 | |
dhellmann | stevemar: see the checklist of things on https://blueprints.launchpad.net/oslo-incubator/+spec/graduate-policy | 17:44 |
dhellmann | stevemar: those are all described in https://wiki.openstack.org/wiki/Oslo/CreatingANewLibrary or the project creator's guide linked from there | 17:44 |
stevemar | dhellmann, yeah i normally look at that, but it was incorrectly marked as Done in the blueprint | 17:45 |
dhellmann | stevemar: ah! k, sorry, I wasn't sure you'd seen those docs before | 17:46 |
stevemar | dhellmann, it's all good, i tossed up a patch for adding it to projects.txt https://review.openstack.org/#/c/154145/ | 17:49 |
dhellmann | stevemar: +2 | 17:51 |
stevemar | \o/ | 17:51 |
kgiusti | sileht: pong - sorry, I was at lunch. I suspect the problem may be due to a version mismatch between the proton C libraries and the python extension pulled from pypi. | 17:54 |
stevemar | dhellmann, oh i wanted to ask, whats the criteria for adding oslo.* libraries to icehouse/juno ? (asking about oslo.context) | 17:54 |
kgiusti | sileht: I'll fire up a f21 vm and try it out. | 17:54 |
sileht | kgiusti, thanks, they are no more f20 for gate jobs, so we have to run them into a f21 | 17:57 |
sileht | bb | 17:57 |
sileht | dhellmann, arf, the threading executor entry point have not been released yet | 17:59 |
sileht | we have to wait the next oslo.msg release | 17:59 |
*** dtantsur is now known as dtantsur|afk | 18:01 | |
*** yamahata has joined #openstack-oslo | 18:03 | |
*** tsekiyama has joined #openstack-oslo | 18:04 | |
*** harlowja has joined #openstack-oslo | 18:04 | |
dhellmann | stevemar: we shouldn't be adding requirements to stable branches in general; what's the case for oslo.context? | 18:04 |
harlowja | sileht so u still wanting to share that futures stuff in taskflow; between oslo.messaging and taskflow (and others)? | 18:05 |
dhellmann | sileht: ok, and that's blocked on the current stable branch issues. | 18:05 |
* harlowja needs a nifty name if we do that, ha | 18:05 | |
harlowja | maybe call the library 'destiny' | 18:05 |
stevemar | dhellmann, keystonemiddleware (and pycadf) uses oslo.context, and they test on tempest on icehouse and juno | 18:05 |
harlowja | or 'fate', ha (all similar to future) | 18:06 |
dhellmann | stevemar: the middleware should probably be capped in the stable branches, like the other libs | 18:06 |
stevemar | dhellmann, sorry, they *could* use oslo.context, for now they are using incubator code | 18:06 |
* bnemec suggests density | 18:07 | |
bnemec | as in, "I am your density" ;-) | 18:08 |
*** e0ne has quit IRC | 18:09 | |
openstackgerrit | Merged openstack/oslo.policy: Make use of private modules https://review.openstack.org/153310 | 18:10 |
harlowja | bnemec destiny or density ? :-P | 18:11 |
harlowja | lol | 18:11 |
openstackgerrit | Merged openstack/oslo.policy: Do not use global enforcer for tests https://review.openstack.org/153321 | 18:11 |
bnemec | harlowja: Density. Definitely density: http://www.imdb.com/title/tt0088763/quotes?item=qt0416283 :-) | 18:12 |
harlowja | lol | 18:12 |
*** rushiagr is now known as rushiagr_away | 18:18 | |
*** ViswaV has joined #openstack-oslo | 18:18 | |
openstackgerrit | Merged openstack/oslo.policy: Stop shouting test attribute names https://review.openstack.org/153322 | 18:20 |
openstackgerrit | Merged openstack/oslo.policy: Remove oslo.concurrency from requirements https://review.openstack.org/153480 | 18:20 |
openstackgerrit | Merged openstack/oslo.policy: Remove globals that were introduced for compatibility https://review.openstack.org/153714 | 18:20 |
*** ViswaV has quit IRC | 18:23 | |
*** ViswaV has joined #openstack-oslo | 18:23 | |
openstackgerrit | Merged openstack/oslo.policy: Upgrade hacking to >=0.10.0 https://review.openstack.org/153482 | 18:23 |
*** yamahata has quit IRC | 18:29 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/oslo.db: Updated from global requirements https://review.openstack.org/150502 | 18:30 |
*** yamahata has joined #openstack-oslo | 18:30 | |
dimsum__ | harlowja: i've been waiting to name something firefly | 18:35 |
*** achanda has joined #openstack-oslo | 18:43 | |
kgiusti | sileht: ping - that failure is definitely a version mismatch: f21 has proton 0.8 libraries, but 'pip install python-qpid-proton' pulls down version 0.7 of the bindings. | 18:45 |
kgiusti | sileht: both 0.8 and 0.7 are available at pypi, btw | 18:46 |
*** ViswaV has quit IRC | 18:46 | |
*** vigneshvar has quit IRC | 18:47 | |
*** achanda has quit IRC | 18:47 | |
*** BrianShang_ has joined #openstack-oslo | 18:50 | |
harlowja | dimsum__ hmmmmmm, ha | 18:50 |
harlowja | dimsum__ i think someone got it already :( | 18:50 |
harlowja | https://pypi.python.org/pypi/firefly | 18:50 |
*** BrianShang has quit IRC | 18:52 | |
*** ChuckC has quit IRC | 18:53 | |
harlowja | anyone want to look over https://review.openstack.org/#/c/153752/ (doctested sphinx examples for debtcollector) | 18:53 |
*** achanda has joined #openstack-oslo | 18:53 | |
harlowja | jogo ^ | 18:53 |
jogo | harlowja: a little later, still fighting with stable/juno | 18:54 |
harlowja | kk | 18:54 |
harlowja | np | 18:54 |
*** ChuckC has joined #openstack-oslo | 18:54 | |
harlowja | beat those requirements silly jogo :-P | 18:54 |
dimsum__ | harlowja: darn! | 18:55 |
jogo | harlowja: I'm trying they are putting up a good fight | 18:55 |
harlowja | :) | 18:55 |
dimsum__ | jogo: hahaha | 18:57 |
*** ViswaV has joined #openstack-oslo | 18:59 | |
*** ViswaV_ has joined #openstack-oslo | 19:03 | |
*** ViswaV has quit IRC | 19:03 | |
*** zigo has joined #openstack-oslo | 19:03 | |
*** vigneshvar has joined #openstack-oslo | 19:07 | |
dimsum__ | dhellmann: time to unlock the objects review in project-config? https://review.openstack.org/#/c/151792/ | 19:10 |
dhellmann | dimsum__: yeah, thanks for reminding me | 19:15 |
*** achanda has quit IRC | 19:25 | |
*** achanda has joined #openstack-oslo | 19:27 | |
*** ChuckC_ has joined #openstack-oslo | 19:28 | |
*** ChuckC has quit IRC | 19:29 | |
*** e0ne has joined #openstack-oslo | 19:30 | |
*** sileht has quit IRC | 19:33 | |
*** stevemar has quit IRC | 19:37 | |
*** stevemar has joined #openstack-oslo | 19:38 | |
*** andreykurilin_ has joined #openstack-oslo | 19:38 | |
*** openstackgerrit has quit IRC | 19:41 | |
*** openstackgerrit has joined #openstack-oslo | 19:41 | |
*** sileht has joined #openstack-oslo | 19:41 | |
*** daniel3_ has quit IRC | 19:43 | |
openstackgerrit | Merged openstack/oslo.concurrency: Add eventlet test check to new tests __init__.py https://review.openstack.org/153635 | 19:45 |
*** daniel3_ has joined #openstack-oslo | 19:45 | |
sdague | dhellmann: just approved - https://review.openstack.org/#/c/154211/1 | 19:47 |
dhellmann | sdague: ty | 19:47 |
sdague | there's some propogation time on job definitions | 19:47 |
dhellmann | yeah | 19:47 |
sdague | so my guess is it will be an hour before it take effect | 19:47 |
dhellmann | k | 19:47 |
dhellmann | jecarey, sileht: ^^ that change should unblock messaging and i18n, after the job definitions are rebuilt | 19:48 |
*** andreykurilin_ has quit IRC | 20:00 | |
*** ViswaV_ has quit IRC | 20:00 | |
openstackgerrit | Joshua Harlow proposed openstack/taskflow: Subclass event and add a poll_wait() method onto it https://review.openstack.org/154220 | 20:08 |
*** ViswaV has joined #openstack-oslo | 20:11 | |
dimsum__ | dhellmann: want me to take over the oslo.log patch in nova? | 20:13 |
dhellmann | dimsum__: yeah, it looks like I'm on stable/juno firefighting duty for a while | 20:13 |
dimsum__ | dhellmann: i'll open a fresh review as it would be easier for me to track. is that ok? | 20:14 |
dimsum__ | ouch, the review is way too stale, because of the namespace removal patches... | 20:16 |
dimsum__ | am on it | 20:16 |
dhellmann | dimsum__: sure, whatever is easier | 20:32 |
dhellmann | dimsum__: I went ahead and abandoned https://review.openstack.org/#/c/147635/ | 20:33 |
openstackgerrit | Vilobh Meshram proposed openstack/taskflow: Docstrings should document parameters return values https://review.openstack.org/152375 | 20:39 |
*** salv-orlando has joined #openstack-oslo | 20:44 | |
openstackgerrit | Mehdi Abaakouk proposed openstack/oslo.messaging: amqp1: fix functional tests deps https://review.openstack.org/153754 | 21:07 |
openstackgerrit | Joshua Harlow proposed openstack/taskflow: Subclass event and add a poll_wait() method onto it https://review.openstack.org/154220 | 21:10 |
*** achanda has quit IRC | 21:20 | |
*** ChuckC_ has quit IRC | 21:20 | |
*** ChuckC_ has joined #openstack-oslo | 21:21 | |
*** ChuckC_ is now known as ChuckC | 21:21 | |
openstackgerrit | Joshua Harlow proposed openstack/taskflow: Subclass event and add a poll_wait() method onto it https://review.openstack.org/154220 | 21:26 |
elmiko | hey folks, is anyone around who knows the ssl oslo.messaging configs? | 21:30 |
*** dulek has joined #openstack-oslo | 21:31 | |
*** cdent has quit IRC | 21:33 | |
*** ChuckC has quit IRC | 21:38 | |
*** ChuckC has joined #openstack-oslo | 21:39 | |
*** ChuckC_ has joined #openstack-oslo | 21:42 | |
*** ChuckC has quit IRC | 21:46 | |
*** achanda has joined #openstack-oslo | 21:54 | |
*** oomichi has joined #openstack-oslo | 22:06 | |
*** vigneshvar has quit IRC | 22:25 | |
*** vipul has quit IRC | 22:42 | |
*** vipul has joined #openstack-oslo | 22:45 | |
*** crc32 has quit IRC | 22:46 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/oslo.policy: Updated from global requirements https://review.openstack.org/154275 | 22:46 |
*** crc32 has joined #openstack-oslo | 22:47 | |
*** miqui has joined #openstack-oslo | 22:48 | |
*** dulek has quit IRC | 22:48 | |
*** dulek has joined #openstack-oslo | 22:49 | |
*** mriedem has quit IRC | 22:51 | |
*** gordc has quit IRC | 22:51 | |
openstackgerrit | Clark Boylan proposed openstack/oslo.versionedobjects: DO NOT MERGE testing git replication https://review.openstack.org/154280 | 22:52 |
*** e0ne has quit IRC | 22:53 | |
openstackgerrit | Joshua Harlow proposed openstack/debtcollector: Add a removal decorator https://review.openstack.org/153429 | 22:54 |
*** dulek has quit IRC | 22:56 | |
openstackgerrit | Min Pae proposed openstack/taskflow: Modify stop and add wait on conductor to prevent lockups https://review.openstack.org/154281 | 22:57 |
*** jaosorior has quit IRC | 23:02 | |
*** gordc has joined #openstack-oslo | 23:02 | |
*** andreykurilin_ has joined #openstack-oslo | 23:03 | |
openstackgerrit | Joshua Harlow proposed openstack/taskflow: Subclass event and add a poll_wait() method onto it https://review.openstack.org/154220 | 23:06 |
*** bknudson has quit IRC | 23:31 | |
openstackgerrit | Min Pae proposed openstack/taskflow: Modify stop and add wait on conductor to prevent lockups https://review.openstack.org/154281 | 23:32 |
*** jgrimm is now known as zz_jgrimm | 23:35 | |
openstackgerrit | Joshua Harlow proposed openstack/taskflow: Allow loading conductors via entrypoints https://review.openstack.org/124617 | 23:39 |
*** gordc has quit IRC | 23:40 | |
*** mriedem has joined #openstack-oslo | 23:43 | |
*** mriedem has quit IRC | 23:44 | |
*** mriedem has joined #openstack-oslo | 23:46 | |
*** andreykurilin_ has quit IRC | 23:51 | |
*** dimsum__ has quit IRC | 23:54 | |
*** dimsum__ has joined #openstack-oslo | 23:55 | |
*** dimsum__ is now known as dims | 23:57 | |
openstackgerrit | Joshua Harlow proposed openstack/taskflow: Allow loading conductors via entrypoints https://review.openstack.org/124617 | 23:57 |
*** dims is now known as Guest70247 | 23:58 | |
*** Guest70247 has quit IRC | 23:58 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!