| *** morganfainberg is now known as morganfainberg_Z | 00:33 | |
| *** yamahata has quit IRC | 00:34 | |
| *** mriedem has joined #openstack-oslo | 00:37 | |
| openstackgerrit | Joshua Harlow proposed a change to openstack-dev/hacking: Add a tool that can fix the import order https://review.openstack.org/68988 | 01:00 |
|---|---|---|
| openstackgerrit | Joshua Harlow proposed a change to openstack/taskflow: Persistence cleanup part uno https://review.openstack.org/81405 | 01:25 |
| *** yamahata has joined #openstack-oslo | 01:58 | |
| *** harlowja is now known as harlowja_away | 02:27 | |
| *** zhiyan_ is now known as zhiyan | 02:32 | |
| *** mriedem has left #openstack-oslo | 03:06 | |
| *** mriedem has quit IRC | 03:06 | |
| *** arnaud has quit IRC | 03:26 | |
| openstackgerrit | Zhongyue Luo proposed a change to openstack/oslo-incubator: Adds test condition in test_periodic https://review.openstack.org/72071 | 05:38 |
| *** markmcclain has joined #openstack-oslo | 07:37 | |
| *** flaper87|afk is now known as flaper87 | 07:57 | |
| *** markmcclain has quit IRC | 07:57 | |
| openstackgerrit | Anastasia Karpinska proposed a change to openstack/taskflow: Move 'inputs and outputs' to developers docs https://review.openstack.org/83010 | 08:56 |
| openstackgerrit | Ivan Melnikov proposed a change to openstack/taskflow: Use same code to reset flow and parts of it https://review.openstack.org/82825 | 09:28 |
| openstackgerrit | Ivan Melnikov proposed a change to openstack/taskflow: Optimize dependency links in flattening https://review.openstack.org/82469 | 09:28 |
| openstackgerrit | Anastasia Karpinska proposed a change to openstack/taskflow: Move 'inputs and outputs' to developers docs https://review.openstack.org/83010 | 09:43 |
| openstackgerrit | Ivan Melnikov proposed a change to openstack/taskflow: Add timeout to WaitForOneFromTask https://review.openstack.org/83018 | 09:49 |
| *** bauzas has joined #openstack-oslo | 09:51 | |
| *** yamahata has quit IRC | 10:15 | |
| openstackgerrit | Ivan Melnikov proposed a change to openstack/taskflow: SQLAlchemy requirements put in order https://review.openstack.org/83025 | 10:39 |
| openstackgerrit | Anastasia Karpinska proposed a change to openstack/taskflow: Move 'inputs and outputs' to developers docs https://review.openstack.org/83010 | 10:41 |
| *** yamahata has joined #openstack-oslo | 11:40 | |
| *** beav has quit IRC | 11:43 | |
| *** bogdando has quit IRC | 11:58 | |
| *** yamahata has quit IRC | 12:06 | |
| *** bogdando has joined #openstack-oslo | 12:10 | |
| *** dims_ has quit IRC | 12:18 | |
| *** dims_ has joined #openstack-oslo | 12:21 | |
| *** beav has joined #openstack-oslo | 12:49 | |
| *** bknudson has joined #openstack-oslo | 13:03 | |
| *** yamahata has joined #openstack-oslo | 13:18 | |
| *** zhiyan is now known as zhiyan_ | 13:19 | |
| *** zhiyan_ is now known as zhiyan | 13:19 | |
| *** bknudson has quit IRC | 13:23 | |
| *** lbragstad has quit IRC | 13:38 | |
| openstackgerrit | Ivan Melnikov proposed a change to openstack/taskflow: Run worker-based engine tests faster https://review.openstack.org/83069 | 13:39 |
| openstackgerrit | Ihar Hrachyshka proposed a change to openstack/oslo-incubator: Qpid: advance thru the list of brokers on reconnect https://review.openstack.org/67449 | 13:50 |
| openstackgerrit | Ihar Hrachyshka proposed a change to openstack/oslo-incubator: Qpid: advance thru the list of brokers on reconnect https://review.openstack.org/67449 | 13:53 |
| ihrachys | dhellmann: please consider the following patches for incubator: https://review.openstack.org/#/c/67450/ , https://review.openstack.org/#/c/67449/ | 13:54 |
| dhellmann | ihrachys: regarding cycle(), how many brokers do you expect to have in the parameters list? | 13:55 |
| * dhellmann wonders why having a copy is a bad thing | 13:55 | |
| ihrachys | dhellmann: well, not that I consider this as a really bad thing... | 13:56 |
| ihrachys | dhellmann: but those incubator patches are backports from oslo.messaging | 13:56 |
| ihrachys | dhellmann: so I think possible enhancements belong there | 13:56 |
| dhellmann | ihrachys: I keep forgetting that, yeah, nevermind | 13:56 |
| ihrachys | you have excuse: this already took looots of time :) | 13:57 |
| ihrachys | I myself keep forgetting it :D | 13:57 |
| ihrachys | dhellmann: there's actually a patch for oslo.messaging for this: https://review.openstack.org/#/c/81183/4/oslo/messaging/_drivers/impl_qpid.py | 13:58 |
| *** lbragstad has joined #openstack-oslo | 14:21 | |
| *** beav has quit IRC | 14:27 | |
| dims_ | bnemec, here's a blast from the past (https://bugs.launchpad.net/heat/+bug/1297880) saw your name on the first review of multi_cloud blueprint :) | 14:46 |
| bnemec | dims_: Nice. :-) | 14:50 |
| bnemec | That was basically an interview question. | 14:50 |
| bnemec | I said if I didn't get hired I was going to send them a bill. ;-) | 14:51 |
| dims_ | :) | 14:52 |
| *** yamahata has quit IRC | 15:03 | |
| *** yamahata has joined #openstack-oslo | 15:05 | |
| *** yamahata has quit IRC | 15:09 | |
| *** yamahata has joined #openstack-oslo | 15:10 | |
| openstackgerrit | Anastasia Karpinska proposed a change to openstack/taskflow: Move 'inputs and outputs' to developers docs https://review.openstack.org/83010 | 15:26 |
| *** bknudson has joined #openstack-oslo | 15:30 | |
| openstackgerrit | Anastasia Karpinska proposed a change to openstack/taskflow: Move 'inputs and outputs' to developers docs https://review.openstack.org/83010 | 16:12 |
| *** boris-42 has quit IRC | 16:24 | |
| openstackgerrit | Jenkins proposed a change to openstack/cliff: Updated from global requirements https://review.openstack.org/83120 | 16:25 |
| *** bauzas has quit IRC | 16:29 | |
| dhellmann | have any of you been looking at pbr changes? I'd like to merge https://review.openstack.org/#/c/82356/ this week so we can release a new version and stop breaking installation of anything other than OpenStack libs using it. | 16:35 |
| *** boris-42 has joined #openstack-oslo | 16:36 | |
| dims_ | dhellmann, don't we have to update global-requirements.txt in requirements repo first? | 16:40 |
| dhellmann | dims_: not in this case -- this is unsyncing the requirement so we don't enforce the version of pip in pbr, but we will still enforce it when we configure our CI systems | 16:41 |
| dims_ | ah ok | 16:41 |
| dhellmann | the issue is anyone using cliff or another library packaged with pbr can't install the lib without a recent pip, even if they wouldn't need a newer pip for their environment for any other reason | 16:42 |
| dhellmann | and pbr itself is not actually using the new version of pip | 16:42 |
| dims_ | +A. thanks for the explanation | 16:43 |
| dhellmann | dims_: thank you! | 16:44 |
| *** lbragstad has quit IRC | 16:49 | |
| *** harlowja_away is now known as harlowja | 16:58 | |
| *** arnaud has joined #openstack-oslo | 17:30 | |
| *** lbragstad has joined #openstack-oslo | 17:50 | |
| *** morganfainberg_Z is now known as morganfainberg | 17:50 | |
| harlowja | dhellmann sweet, let me know when that fix goes in a and i can try to see if anvil can happily build all those packages on rhel6 | 17:54 |
| harlowja | *due to the pip1.4 issue | 17:54 |
| harlowja | will run my anvil cloud-init script to see how that goes (fails automatically if it can't build) | 17:55 |
| openstackgerrit | Ben Nemec proposed a change to openstack-dev/pbr: Add support for nested requirements files https://review.openstack.org/83149 | 18:00 |
| openstackgerrit | Ben Nemec proposed a change to openstack/oslo.messaging: WIP: Split requirements https://review.openstack.org/83150 | 18:01 |
| *** ruhe has joined #openstack-oslo | 18:10 | |
| *** mriedem has joined #openstack-oslo | 18:26 | |
| openstackgerrit | Jenkins proposed a change to openstack/oslo.version: Updated from global requirements https://review.openstack.org/83166 | 18:33 |
| openstackgerrit | Andreas Jaeger proposed a change to openstack/oslo-incubator: Improve help strings https://review.openstack.org/71992 | 18:45 |
| *** zhiyan is now known as zhiyan_ | 18:52 | |
| *** markmcclain has joined #openstack-oslo | 19:08 | |
| *** _cjones_ has joined #openstack-oslo | 19:11 | |
| _cjones_ | Hello. I have a quick question about using/developing oslo. I'm creating a new neutron driver, and it requires its own cfg options. How do I get these to be properly registered in my unit tests? | 19:12 |
| _cjones_ | https://gist.github.com/cjones-/1f534d6fa09421a46309 | 19:13 |
| _cjones_ | Oops. Just forgot to import my cfg. Sorry to bother you guys. | 19:18 |
| harlowja | np | 19:19 |
| dhellmann | _cjones_: the ConfFixture for oslo.messaging has a good example of how to do that cleanly: http://git.openstack.org/cgit/openstack/oslo.messaging/tree/oslo/messaging/conffixture.py#n28 | 19:25 |
| dhellmann | _cjones_: and see http://git.openstack.org/cgit/openstack/oslo.messaging/tree/tests/test_qpid.py for an example of it being used | 19:25 |
| *** bauzas has joined #openstack-oslo | 20:10 | |
| openstackgerrit | Joshua Harlow proposed a change to openstack/taskflow: Persistence cleanup part one https://review.openstack.org/81405 | 20:30 |
| *** asalkeld has joined #openstack-oslo | 20:43 | |
| asalkeld | https://review.openstack.org/#/c/83197/ | 20:43 |
| asalkeld | dhellmann, is this ^ do-able | 20:44 |
| dhellmann | asalkeld: we should just add an option to the theme to turn off the logo, no? | 20:45 |
| asalkeld | if we can | 20:45 |
| *** pvaneck has joined #openstack-oslo | 20:45 | |
| asalkeld | if you know a way, I can do the work | 20:45 |
| asalkeld | I am not sure how options are passed in | 20:45 |
| dhellmann | asalkeld: what's the timeline for this? can it wait a bit? | 20:46 |
| asalkeld | well I can do a tempory workaround | 20:46 |
| asalkeld | and do the oslosphinx change later | 20:46 |
| dhellmann | let me find an example of using an option | 20:47 |
| asalkeld | cool | 20:47 |
| dhellmann | https://bitbucket.org/dhellmann/sphinxcontrib-spelling/src/600ad91be8c00d5f0681be2424c011d24ed1e79f/sphinxcontrib/spelling/__init__.py?at=master#cl-17 | 20:47 |
| dhellmann | that defines an option | 20:47 |
| dhellmann | I'm not sure if that automatically makes it available to the theme | 20:47 |
| dhellmann | I think probably? | 20:47 |
| asalkeld | ooo, nice | 20:47 |
| asalkeld | I'll play | 20:47 |
| _cjones_ | dhellmann. Thanks. I'll take a look. | 20:53 |
| *** boris-42 has quit IRC | 20:59 | |
| *** boris-42 has joined #openstack-oslo | 21:15 | |
| *** zaneb has joined #openstack-oslo | 21:31 | |
| zaneb | o hai | 21:31 |
| zaneb | dhellmann: did you want to discuss bug 1288586? | 21:32 |
| dhellmann | hi, zaneb | 21:32 |
| dhellmann | yeah, what's going on there? | 21:32 |
| * dhellmann re-reads bug report | 21:32 | |
| zaneb | so basically, we register identical options in a bunch of groups | 21:33 |
| zaneb | and because we only ever look for one group for an option... | 21:33 |
| zaneb | and now they all compare equal... | 21:33 |
| dhellmann | ah, yeah, that's almost certainly not a case the generator expected to see | 21:33 |
| zaneb | they all show up in the first group :/ | 21:34 |
| zaneb | evidently not ;) | 21:34 |
| asalkeld | partly an issue of __eq__ in oslo.config i think | 21:34 |
| zaneb | so the other issue is that keystone client registers an option in two groups too | 21:35 |
| zaneb | once in the group where it's required | 21:35 |
| dhellmann | I'm not sure an individual Opt knows the group it is registered in | 21:35 |
| zaneb | and once in the default group where it's not | 21:35 |
| zaneb | dhellmann: correct, it doesn't | 21:35 |
| dhellmann | we should be able to work around this in heat, though | 21:35 |
| dhellmann | the latest version of the generator has a feature we added for getting options out of libraries using entry points | 21:36 |
| dhellmann | that mechanism requires you to explicitly provide the group, since we're not scanning code | 21:36 |
| * dhellmann looks for example | 21:36 | |
| dhellmann | this module has the function we defined for oslo.messaging: http://git.openstack.org/cgit/openstack/oslo.messaging/tree/oslo/messaging/opts.py | 21:37 |
| dhellmann | basically, return an iterable of (group_name, option_list) pairs | 21:37 |
| dhellmann | use it by calling generate_sample.sh with the -l option | 21:37 |
| dhellmann | so you might run "generate_sample.sh -l heat.opts" instead of "generate_sample.sh heat" | 21:38 |
| zaneb | hacky, yet interesting ;) | 21:38 |
| dhellmann | yes, well, deadlines :-) | 21:39 |
| dhellmann | there is a non-zero, but not > 50%, chance that code scanning is going to be deprecated at some point, since that is equally or more challenging to do properly | 21:39 |
| zaneb | wait, I can just stick this in the original heat.common.config file that contains the options already? | 21:40 |
| zaneb | that doesn't actually seem so bad | 21:40 |
| dhellmann | and, of course, if you have drivers or something where you don't want all the options loaded at one point, you can define more than one entry point | 21:40 |
| dhellmann | yeah, I don't know where heat defines its options | 21:40 |
| dhellmann | if you have them in one place already, you're 90% of the way to done | 21:41 |
| zaneb | https://github.com/openstack/heat/blob/master/heat/common/config.py | 21:41 |
| zaneb | sweet | 21:41 |
| zaneb | next question is if we have a version of generator.py synced that works with this ;) | 21:43 |
| dhellmann | zaneb: you also need to register the entry point: http://git.openstack.org/cgit/openstack/oslo.messaging/tree/setup.cfg#n53 | 21:43 |
| dhellmann | and then generating the sample will require having heat installed, to make that entry point work | 21:43 |
| dhellmann | nova is doing that already, to get the oslo.messaging options, so you can look at how they do it | 21:44 |
| *** asalkeld is now known as asalkeld_bbq | 21:44 | |
| *** mriedem has quit IRC | 21:48 | |
| harlowja | wooohoo, https://review.openstack.org/#/c/82356/ merged | 21:52 |
| openstackgerrit | Joshua Harlow proposed a change to openstack/taskflow: Set logbook/flowdetail/atomdetail meta to empty dict https://review.openstack.org/82944 | 22:02 |
| zaneb | harlowja: for a brief moment there I thought the tox version was dropping back to 1.4, and I rejoiced | 22:03 |
| *** dhellmann is now known as dhellmann_ | 22:03 | |
| harlowja | zaneb just pip (less rejoice) | 22:03 |
| zaneb | sigh | 22:03 |
| harlowja | building packages on rhel seems to be broke | 22:03 |
| harlowja | rhel6 | 22:03 |
| harlowja | due to pbr trying to pull in pip 1.4 | 22:04 |
| * zaneb is looking forward to rhel 7 ;) | 22:04 | |
| harlowja | def | 22:04 |
| harlowja | anyways zaneb your duck discussion was the highlight of my morning, ha | 22:06 |
| harlowja | (maybe thats sad, lol) | 22:06 |
| zaneb | harlowja: I aim to entertain ;) | 22:07 |
| openstackgerrit | Joshua Harlow proposed a change to openstack/taskflow: Add last_modified & created_on attributes to jobs https://review.openstack.org/64092 | 22:07 |
| harlowja | zaneb although i'm waiting for the calculator/matlab response, lol | 22:08 |
| *** bknudson has quit IRC | 22:09 | |
| harlowja | *i know people that worked at mathworks and on matlab code, its not something u want to see, lol | 22:09 |
| harlowja | *it will make u sick, lol | 22:09 |
| zaneb | everyone uses calculators, approximately 0% of people use Matlab | 22:09 |
| zaneb | approximately | 22:10 |
| harlowja | ya, rounding errors taken into account (0.01 scientists) | 22:10 |
| zaneb | therefore give everyone a calculator and self-service access to a VM to run Matlab in | 22:10 |
| harlowja | matlab is apparently some montrosity of java code + fortran + c++ from what i remember | 22:10 |
| harlowja | *what i remember those folks telling me | 22:11 |
| zaneb | orly, Java too? | 22:11 |
| zaneb | ouch | 22:11 |
| harlowja | gui is in java i think | 22:11 |
| harlowja | ya, def ouch (part of the reason all those people that i knew that worked there left, lol) | 22:14 |
| *** bauzas has quit IRC | 22:27 | |
| *** dhellmann_ is now known as dhellmann | 22:31 | |
| harlowja | dhellmann so if anvil built dep packages, would anyone u know use them | 22:35 |
| harlowja | a guy here wants to try to see about adding that support | 22:35 |
| dhellmann | dep? | 22:35 |
| dhellmann | or deb? | 22:35 |
| harlowja | *deb | 22:35 |
| dhellmann | we build our own based on virtualenvs, so we probably wouldn't use them -- that's not to say *no one* would | 22:36 |
| harlowja | its complicated but might be interesting (the hard part is the building of all python dependencies that can't be found in known repositories) | 22:36 |
| harlowja | ah, u guys want the pkg + virtualenv route to | 22:36 |
| harlowja | *went with the pkg | 22:36 |
| *** dhellmann is now known as dhellmann_ | 22:41 | |
| openstackgerrit | Joshua Harlow proposed a change to openstack/taskflow: Should be greater or equal to zero and not greater than https://review.openstack.org/83236 | 22:56 |
| *** harlowja has quit IRC | 22:58 | |
| *** harlowja has joined #openstack-oslo | 23:03 | |
| *** bknudson has joined #openstack-oslo | 23:05 | |
| *** dims_ has quit IRC | 23:12 | |
| *** flaper87 is now known as flaper87|afk | 23:18 | |
| openstackgerrit | Joshua Harlow proposed a change to openstack/taskflow: Add conditional execution support (WIP) https://review.openstack.org/83244 | 23:18 |
| openstackgerrit | Joshua Harlow proposed a change to openstack/taskflow: Fix spelling mistake https://review.openstack.org/83246 | 23:23 |
| *** asalkeld_bbq has quit IRC | 23:26 | |
| *** dims_ has joined #openstack-oslo | 23:30 | |
| *** mriedem has joined #openstack-oslo | 23:32 | |
| *** yamahata has quit IRC | 23:37 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!