Monday, 2015-02-16

*** kragniz_ is now known as kragniz00:00
*** dims_ has quit IRC00:12
*** dimsum__ has joined #openstack-oslo00:49
*** dimsum__ has quit IRC00:54
openstackgerritMin Pae proposed openstack/taskflow: adding check for str/unicode type in requires  https://review.openstack.org/15609400:58
*** liusheng has quit IRC01:02
*** bknudson has quit IRC01:07
*** zz_jgrimm has quit IRC01:27
*** achanda has joined #openstack-oslo01:28
*** zz_jgrimm has joined #openstack-oslo01:31
*** achanda has quit IRC01:33
*** ndonegan has quit IRC02:05
*** amotoki has quit IRC02:38
openstackgerritIan Wienand proposed openstack/oslo.policy: Do not log on missing or empty policy_dirs  https://review.openstack.org/15474202:43
openstackgerritIan Wienand proposed openstack/oslo.policy: Do not log on missing or empty policy_dirs  https://review.openstack.org/15474202:47
*** achanda has joined #openstack-oslo02:52
*** jaypipes has joined #openstack-oslo03:10
*** harlowja_at_home has joined #openstack-oslo03:34
*** amotoki has joined #openstack-oslo03:37
*** amotoki has quit IRC03:43
*** amotoki has joined #openstack-oslo03:49
*** achanda has quit IRC04:00
*** ndonegan has joined #openstack-oslo04:01
openstackgerritJoshua Harlow proposed openstack/taskflow: Make the dispatcher handler be an actual type  https://review.openstack.org/15611404:05
openstackgerritJoshua Harlow proposed openstack/taskflow: Add retries to fetching the zookeeper server version  https://review.openstack.org/15595004:07
*** harlowja_at_home has quit IRC04:07
*** jaosorior has quit IRC04:11
openstackgerritJoshua Harlow proposed openstack/taskflow: Make the dispatcher handler be an actual type  https://review.openstack.org/15611404:13
openstackgerritJoshua Harlow proposed openstack/taskflow: Make the dispatcher handler be an actual type  https://review.openstack.org/15611404:16
*** stevemar has joined #openstack-oslo04:17
openstackgerritJoshua Harlow proposed openstack/taskflow: Make the dispatcher handler be an actual type  https://review.openstack.org/15611404:21
*** BAKfr has quit IRC04:32
*** ndonegan has quit IRC04:32
*** BAKfr has joined #openstack-oslo04:32
*** BAKfr has quit IRC04:37
*** stevemar has quit IRC04:47
*** BAKfr has joined #openstack-oslo04:48
*** achanda has joined #openstack-oslo05:00
*** achanda has quit IRC05:06
*** stevemar has joined #openstack-oslo05:25
*** ndonegan has joined #openstack-oslo06:04
*** ankit_ag has joined #openstack-oslo06:08
*** achanda has joined #openstack-oslo06:13
*** stevemar has quit IRC06:20
*** achanda has quit IRC06:45
openstackgerritSridhar Gaddam proposed openstack/oslo.utils: Utility API to generate EUI-64 IPv6 address  https://review.openstack.org/13777406:51
openstackgerritAndreas Jaeger proposed openstack/oslo.versionedobjects: Set up translations  https://review.openstack.org/15613906:53
*** SridharGaddam has joined #openstack-oslo06:57
*** achanda has joined #openstack-oslo07:04
*** andreykurilin_ has joined #openstack-oslo07:13
openstackgerritMin Pae proposed openstack/taskflow: adding check for str/unicode type in requires  https://review.openstack.org/15609407:21
*** inc0 has joined #openstack-oslo07:33
*** andreykurilin_ has quit IRC07:35
*** ihrachyshka has joined #openstack-oslo07:35
openstackgerritSridhar Gaddam proposed openstack/oslo.utils: Utility API to generate EUI-64 IPv6 address  https://review.openstack.org/13777407:36
*** ihrachyshka has quit IRC07:45
*** snikitin has joined #openstack-oslo08:00
*** shardy has joined #openstack-oslo08:02
*** takedakn has joined #openstack-oslo08:13
*** achanda has quit IRC08:18
*** takedakn has quit IRC08:19
*** achanda has joined #openstack-oslo08:28
*** achanda has quit IRC08:35
*** achanda has joined #openstack-oslo08:40
*** dulek has joined #openstack-oslo08:40
*** takedakn has joined #openstack-oslo08:42
*** ihrachyshka has joined #openstack-oslo08:48
ihrachyshkadhellmann, https://review.openstack.org/#/c/137913/ please remove W-1.08:48
*** i159 has joined #openstack-oslo08:49
*** rpodolyaka has left #openstack-oslo08:53
*** rpodolyaka has joined #openstack-oslo08:54
*** takedakn has quit IRC08:54
*** amotoki_ has joined #openstack-oslo08:56
*** openstackgerrit has quit IRC08:57
*** openstackgerrit has joined #openstack-oslo08:57
*** viktors has joined #openstack-oslo08:58
*** amotoki_ has quit IRC09:02
*** achanda has quit IRC09:02
openstackgerritGrzegorz Grasza proposed openstack/oslo.versionedobjects: Rename utils -> _utils  https://review.openstack.org/15440909:08
*** ajo has joined #openstack-oslo09:10
*** achanda has joined #openstack-oslo09:12
*** salv-orlando has quit IRC09:22
*** achanda has quit IRC09:23
openstackgerritVipin Balachandran proposed openstack/oslo.vmware: Handle SPBM SecurityError  https://review.openstack.org/15616409:36
*** sreshetnyak has joined #openstack-oslo09:37
*** amotoki_ has joined #openstack-oslo09:45
*** pblaho has joined #openstack-oslo09:52
*** takedakn has joined #openstack-oslo10:01
openstackgerritGrzegorz Grasza proposed openstack/oslo.versionedobjects: Rename utils -> _utils  https://review.openstack.org/15440910:13
*** takedakn has quit IRC10:14
*** e0ne has joined #openstack-oslo10:20
openstackgerritGrzegorz Grasza proposed openstack/oslo.versionedobjects: Rename utils -> _utils  https://review.openstack.org/15440910:20
*** rushiagr_away is now known as rushiagr10:23
*** xek_ is now known as xek10:36
*** rushiagr is now known as rushiagr_away10:40
*** amotoki_ has quit IRC10:52
*** e0ne is now known as e0ne_10:57
*** amotoki_ has joined #openstack-oslo10:58
*** e0ne_ is now known as e0ne11:00
*** amotoki_ has quit IRC11:03
*** salv-orlando has joined #openstack-oslo11:12
*** amotoki_ has joined #openstack-oslo11:13
*** salv-orlando has quit IRC11:14
*** salv-orlando has joined #openstack-oslo11:14
*** miqui_ has quit IRC11:18
*** miqui has joined #openstack-oslo11:18
*** amotoki_ has quit IRC11:33
*** amotoki_ has joined #openstack-oslo11:38
*** amotoki_ has quit IRC11:40
openstackgerritMerged openstack/oslo.versionedobjects: Remove more Nova-isms  https://review.openstack.org/15539311:41
*** e0ne is now known as e0ne_11:45
*** e0ne_ has quit IRC11:55
*** e0ne has joined #openstack-oslo11:57
*** andreykurilin_ has joined #openstack-oslo11:59
inc0https://review.openstack.org/#/c/154835/ guys can I have this reviewed please? Its for versionedobjects12:00
*** amotoki_ has joined #openstack-oslo12:04
*** dimsum__ has joined #openstack-oslo12:11
*** amotoki_ has quit IRC12:21
*** amotoki_ has joined #openstack-oslo12:23
*** andreykurilin_ has quit IRC12:25
*** andreykurilin_ has joined #openstack-oslo12:26
*** amotoki_ has quit IRC12:31
*** amotoki_ has joined #openstack-oslo12:31
*** e0ne is now known as e0ne_12:35
*** kbyrne has joined #openstack-oslo12:37
*** amotoki_ has quit IRC12:39
*** e0ne_ is now known as e0ne12:39
openstackgerritMerged openstack-dev/cookiecutter: Bump cookiecutter's hacking to 0.10  https://review.openstack.org/15584312:49
openstackgerritMerged openstack-dev/oslo-cookiecutter: Bump cookiecutter's hacking to 0.10  https://review.openstack.org/15584612:52
*** takedakn has joined #openstack-oslo12:56
*** takedakn has quit IRC12:56
*** palendae has quit IRC12:57
*** cdent has joined #openstack-oslo12:57
*** palendae has joined #openstack-oslo12:58
*** jaosorior has joined #openstack-oslo12:59
*** hogepodge has quit IRC13:03
*** hogepodge has joined #openstack-oslo13:04
*** e0ne is now known as e0ne_13:04
*** takedakn has joined #openstack-oslo13:05
*** dimsum__ is now known as dims13:05
*** dims is now known as Guest7026613:05
*** takedakn has quit IRC13:07
*** e0ne_ has quit IRC13:14
*** e0ne has joined #openstack-oslo13:16
*** ankit_ag has quit IRC13:19
openstackgerritMerged openstack/oslo.versionedobjects: Set up translations  https://review.openstack.org/15613913:22
openstackgerritMerged openstack/oslo.versionedobjects: Use six.wraps for py2/py3 compat  https://review.openstack.org/15588213:22
openstackgerritMerged openstack/oslo.serialization: add dependency warning to requirements.txt  https://review.openstack.org/15601513:27
*** gordc has joined #openstack-oslo13:32
*** rushiagr_away is now known as rushiagr13:34
*** e0ne is now known as e0ne_13:40
*** e0ne_ has quit IRC13:45
*** kgiusti has joined #openstack-oslo13:46
*** zzzeek has joined #openstack-oslo13:47
*** Guest70266 is now known as dims__13:48
*** e0ne has joined #openstack-oslo13:50
*** e0ne is now known as e0ne_13:51
*** e0ne_ is now known as e0ne13:51
*** e0ne is now known as e0ne_13:51
*** e0ne_ is now known as e0ne13:52
*** e0ne is now known as e0ne_13:52
*** e0ne_ is now known as e0ne13:52
*** e0ne is now known as e0ne_13:53
*** e0ne_ is now known as e0ne13:53
*** inc0 has quit IRC13:54
*** inc0_ has joined #openstack-oslo13:55
*** andreykurilin_ has quit IRC13:56
*** andreykurilin_ has joined #openstack-oslo13:56
*** Nakato has quit IRC13:59
*** Nakato has joined #openstack-oslo13:59
*** Guest59076 is now known as dansmith14:05
*** dansmith is now known as Guest2845614:06
*** salv-orlando has quit IRC14:08
*** Guest28456 is now known as dansmith_14:09
*** salv-orlando has joined #openstack-oslo14:09
*** salv-orlando has quit IRC14:12
*** salv-orlando has joined #openstack-oslo14:13
*** salv-orlando has quit IRC14:15
inc0_dansmith_, hi14:28
dansmith_inc0_: warning, only one half cup of coffee ingested so far14:28
dansmith_inc0_: hi :)14:28
inc0_I'll tread softly14:29
inc0_regarding https://review.openstack.org/#/c/154835/6 2 questions14:29
dansmith_sure14:29
inc0_1. Can I safely remove this tzinfo thingy? You've mentioned in comment thats its for transition14:29
inc0_2. About comparing objects - you've proposed that it is to be done as separate mixin, I've made just that14:30
inc0_why -1 then?:(14:31
dansmith_ah, I really thought I commented on the datetime thing the first time but I guess I didn't14:31
*** mriedem has joined #openstack-oslo14:31
dansmith_when we pull things out of the DB, they have no tz-aware-ness, so we want to coerce them to tz-aware, but UTC14:32
dansmith_why do you want to remove that?14:32
*** amotoki_ has joined #openstack-oslo14:32
*** jecarey has joined #openstack-oslo14:32
dansmith_and on #2: because I was looking at an older patch (and also, see: coffee)14:32
inc0_because we don't always want to use tzaware datetime14:32
inc0_in my opinion its directly messing with data14:33
dansmith_well, we do, having made a lot of changes to get there14:33
dansmith_how about we have two fields then? DateTime and DateTimeNoTZ (or some better name)14:33
inc0_or a flag defaulting to False?14:34
inc0_True == no-tz14:34
dansmith_sure that's cool14:35
dansmith_so, if I may expand on a side tangent:14:35
dansmith_we have a test in nova that calculates the hash of all the objects, across every aspect of a thing that we consider "versioned", which includes the formats of the fields, and thus would need to cover this flag14:35
dansmith_to make sure nobody changes it and thinks it's a minor thing14:36
dansmith_I think we probably want to have some sort of infrastructure in here that makes it easy for projects to do that in their tree14:36
dansmith_there are remnants of it in test_objects.py, but it doesn't really make sense to be here since it's a library with no objects inside14:37
dhellmanngood morning, all14:37
dansmith_I haven't done much thinking about what that will look like, but just wanted to throw it out there if you're working on the heat side of this as well14:37
inc0_well in heat I had mostly problems with tests, and these failed on this tzinfo thing14:41
inc0_it seems they explicitly didn't suspect to have tzinfo in datetime14:42
inc0_hello dhellmann14:42
dhellmannhi, inc0_14:42
inc0_by the way, when do we want to have first release?14:42
inc0_anything remain to be done before that?14:43
dhellmannwe have to be pretty careful with the first release, because whatever we include there is going to be supported for a long time14:43
dhellmannI don't want to rush into it14:44
inc0_how about having a rc or something? so we could try out cinder and heat implementation14:44
inc0_?14:44
dhellmanninc0_: as soon as we release something, no matter how we tag it, someone somewhere is going to start using it as though it is fully baked14:45
inc0_(myself included)14:46
inc0_dansmith_, can you think of anything needing to be done before release?14:47
dansmith_inc0_: yeah, we have a list of things to accomplish first, right? the work items?14:47
dansmith_the registry change was the big thing I wanted to do14:47
*** amotoki_ has quit IRC14:48
inc0_do we have this list somewhere summarized? So we can split work?14:48
dansmith_inc0_: https://github.com/openstack/oslo-specs/blob/master/specs/kilo/adopt-oslo-versionedobjects.rst#work-items14:49
dansmith_I think we have a couple more things to make private in base.py14:49
dansmith_I can do that14:49
openstackgerritDoug Hellmann proposed openstack/oslo.vmware: Add missing links to README  https://review.openstack.org/15623714:50
inc0_when we merge this comparason+datetime thing I should be able to make prototype in heat14:50
dansmith_cool14:50
*** tsekiyama has joined #openstack-oslo14:51
openstackgerritDoug Hellmann proposed openstack/oslo-incubator: Update release tool for governance change  https://review.openstack.org/15623814:53
*** jecarey has quit IRC14:53
*** cdent_ has joined #openstack-oslo14:57
*** amotoki_ has joined #openstack-oslo14:57
*** cdent has quit IRC14:57
*** cdent_ is now known as cdent14:57
*** amotoki_ has quit IRC14:59
*** tsekiyam_ has joined #openstack-oslo15:00
*** amotoki_ has joined #openstack-oslo15:00
*** sigmavirus24_awa is now known as sigmavirus2415:01
*** ihrachyshka has quit IRC15:02
*** amotoki_ has quit IRC15:03
*** zz_jgrimm is now known as jgrimm15:04
*** tsekiyama has quit IRC15:04
*** dansmith_ is now known as dansmith15:05
dhellmannsileht: are we ready for an oslo.messaging release?15:10
dhellmannviktors, rpodolyaka, zzzeek: are we ready for an oslo.db release?15:11
rpodolyakaviktors: could you please trigger your magic 'run tests on all projects' script?15:12
rpodolyakadhellmann: ^ other than that, we should be good to go15:12
dhellmannbeekneemech: it looks like we could do with an oslo.concurrency release to update the requirements and imports15:13
dhellmannrpodolyaka, viktors: we have a script like that in the incubator, hang on15:13
*** beekneemech is now known as bnemec15:13
dhellmannrpodolyaka: tools/pre_release_tests.py15:13
dhellmannyou have to have everything checked out already for that to work15:14
bnemecdhellmann: Okay, I'll look into that today.15:14
bnemecdhellmann: I may not be around much during the meeting.  Somebody decided to schedule another one off meeting for Monday morning. :-/15:15
dhellmannbnemec: :-/15:15
rpodolyakadhellmann: ah, didn't know that, thanks! but viktors has hardware for that ;)15:15
dhellmannbnemec: ok, I can do the release if you have a few minutes to sanity check the need15:15
dhellmannrpodolyaka: yeah, hardware is the tricky part :-)15:15
inc0_cya guys, gtg15:15
haypohi. i don't understand why people send new versions of a patch just after i review it?15:15
haypoex: i voted +2 to https://review.openstack.org/#/c/154409/ and then xek just posted a new version15:16
haypothere are 7 versions of the patch set. and no changelog for each changelog. it's not convinient :-/15:16
haypois there a way to see the change between two versions of a change set? except of manually comparing them?15:16
xekhaypo, hi, I just added a line in the commit message, like you suggested15:17
*** ihrachyshka has joined #openstack-oslo15:17
haypoxek: my questions are not specific to your issue. i have the problem every day :)15:18
xeka way of reviewing the changes between patches would be nice15:18
haypoit's just annoying to have to vote 3 times for exactly the same patch :-p15:18
viktorsrpodolyaka, dhellmann: ok, I'll do15:19
bnemechaypo: xek: There should be some links at the top of the diff for each file that say "Patch Set Base 1 2 3.....17"  If you select the two versions you want to compare it will only show the differences between those two.15:19
viktorsdhellmann: do we have meeting today as usual?15:19
*** inc0_ has quit IRC15:20
haypobnemec: oh wait, i never noticed that they are "Patch Set .. " links at left too15:20
dhellmannhaypo: if you set the "reference version" in gerrit, it will show the diff of the current patch against that version. So you can set it to N-1 and see the changes between N-1 and N, or you can set it to N-2, etc. It defaults to the first patch.15:20
dhellmannviktors: yes15:20
haypoi see https://review.openstack.org/#/c/154409/6..7//COMMIT_MSG,unified15:20
haypocool15:21
bnemechaypo: Yep.  It's still a good idea for people to leave comments saying what they changed if they push new patch sets without prompting though.15:21
bnemecAnything that makes the reviewer's life easier gets a +1 from me. :-)15:21
xekbnemec, thanks :)15:22
bnemecxek: np15:22
hayposometimes, i think that it would be faster to just accept the patch instead of having the dance of new change set => vote => conflict => rebase => vote => conflict => ... :)15:22
*** jecarey has joined #openstack-oslo15:22
haypo(olso_versionedobject is moving fast ;-))15:22
*** jecarey_ has joined #openstack-oslo15:23
viktorszzzeek: hi! This patch may be interesting for you - https://review.openstack.org/#/c/154521/ (Implement online schema migrations)15:26
zigodhellmann: http://paste.debian.net/149738/15:26
zigodhellmann: As I wrote previously, we still have an issue with Python3 and oslo.utils: unit tests can't be run with it.15:27
*** jecarey has quit IRC15:28
dhellmannzigo: I *just* released 1.3.0 with the packaging fix. Can you try that version?15:28
*** e0ne is now known as e0ne_15:28
zigodhellmann: Will it work for Juno? Because the global-reqs for Juno are saying <= 1.2.115:28
dhellmannzigo: no, it is not meant to be used for juno15:28
zigodhellmann: My issue currently is that I can't upload Kilo stuff in Experimental, as this would overwrite Juno... :(15:29
zigodhellmann: And I need oslo.utils as a dependency for other stuff, like debtcollector.15:29
dhellmannzigo: nothing in juno should be using debtcollector15:29
zigodhellmann: I know, that's my own kitchen and I shouldn't bother anyone with it.15:29
zigodhellmann: Yeah, I just want to upload it so that it goes in the NEW queue early enough, so it gets a chance to be approved by FTP masters before Kilo is released.15:30
zigodhellmann: Anyway, could you point to me the packaging fix commit, or is it burried with so many other stuff?15:30
dhellmannlet me look15:31
zigoThanks.15:31
dhellmannzigo: http://git.openstack.org/cgit/openstack/oslo.utils/commit/?id=942cf060a1ece4999234191da19aea48e005f74015:31
zigo.oO(if only we had Debian PPAMAIN, my troubles with nowhere to upload would be over...)15:31
zigodhellmann: This will not fix unit testing, will it?15:32
dhellmannzigo: if we need to create a stable/juno branch of oslo.utils and release a 1.2.2 or something we can do that15:32
*** ihrachyshka has quit IRC15:32
zigoThis fixeds the setup.py install though...15:32
dhellmannzigo: it should ensure that the files missing from the sdist are -- right15:32
zigoYeah, got that point ! :)15:32
zigodhellmann: Though what I'm trying to fix is unit test listing...15:34
zigo(in python3)15:34
*** pradk has joined #openstack-oslo15:35
* zigo is trying to patch .testr.conf15:35
*** e0ne_ has quit IRC15:38
dhellmannzigo: oslo.utils is tested under python 3, so maybe this is a different issue than I thought. Do you have a traceback or something?15:42
dhellmannbnemec: how about an oslo.serialization release, too?15:42
zigodhellmann: That's the same issue which we had before moving to no namespace: if there's *any* oslo lib installed in the system (in this case, oslo.config and oslo.i18n, which are build-depends), then the namespace fails in Python3.15:43
dhellmannzigo: ok, well, that doesn't have anything to do with python 3.15:44
bnemecdhellmann: Yep, looks like they could both use releases.15:44
*** ihrachyshka has joined #openstack-oslo15:44
dhellmannthat's just an issue with the namespace package, and AFAICT the only fix for that right now is to not install any of the libraries on the build machine until we can get all of them out of the namespace package15:44
dhellmannbnemec: ok, do you need a chance to look them over or should I go ahead and release them?15:45
zigodhellmann: Which I can do with Build-Conflicts, though *not* when there's build-depends (here, oslo.config and oslo.i18n, when building oslo.utils)15:45
dhellmannzigo: I'm not sure of the difference. Why does building oslo.utils depend on oslo.config? just because of the tests?15:46
zigodhellmann: http://paste.debian.net/149758/15:46
zigodhellmann: Yes, because of tests.15:46
*** tsekiyam_ has quit IRC15:46
zigoHum...15:46
dhellmannzigo: ok. I'm not sure what to say. We're working on getting rid of the ns packages, but I'm not sure what to tell you to do in the mean time.15:47
zigodhellmann: I'm just telling you that you *cannot* do tests.test_warning.DeprecationWarningTest15:48
zigoLet me try something else again.15:48
dhellmannzigo: is there an environment variable we could check for to skip that test while you're packaging? something that is only set for debian packaging builds?15:48
bnemecdhellmann: I don't see any issues with releasing them.15:49
dhellmannbnemec: ok, I'll do those after the meeting15:50
*** tsekiyama has joined #openstack-oslo15:50
* zigo is checking15:51
dhellmannzigo: if that's easy to do, and easier than you ignoring that one test file, then we can put the needed changes in all the libs15:51
zigodhellmann: I'm just trying to find a *smart* way to do stuff so that it works in all cases, and I'm failing at doing that.15:52
zigo:(15:52
zigodhellmann: Patching your upstream code and/or .testr.conf is what I'm trying to avoid.15:52
*** mtanino has joined #openstack-oslo15:53
dhellmannzigo: ok, I'm trying to be a bit flexible :-)15:53
zigodhellmann: I know you are, and I want to thank you so much, I'm just trying to figure out what to do... :)15:54
dhellmannzigo: ok, let me know what you find15:55
zigodhellmann: Currently, here's the issue:15:55
zigohttp://paste.debian.net/149765/15:56
zigo(I've disabled python 2.7 tests, which are passing ok)15:56
zigoIt's looking like subunit will list all tests whatever I put as argument for testr.15:56
dhellmannzigo: there are arguments to tell it where to start scanning for tests15:57
zigodhellmann: Is this in the .testr.conf, or can it be overridden?15:58
dhellmannzigo: it would help if you could make it show what the import error really is by just running or importing the test script directly15:58
dhellmannzigo: it's in that file, but I don't know how you would override it15:58
dhellmannmeeting time coming up15:59
zigoWell, tests.tests_encodeutils tries to import oslo.utils, and fails...15:59
dhellmannis oslo.utils present and installed?15:59
dhellmannthis should all work the same way under python 2 and 315:59
*** david-lyle_afk is now known as david-lyle16:01
*** jecarey_ is now known as jecarey16:01
*** palendae has quit IRC16:02
*** palendae has joined #openstack-oslo16:03
zigodhellmann: Of course, it's *NOT* present and installed, it can't build-depends on itself. Anyway, seems patching .testr.conf does the trick, though I would have prefer avoiding it.16:10
zigoThat, plus explicitly writting that I want to do testr run --subunit 'oslo_utils.*'16:10
zigoWhich is really fine.16:10
*** jgrimm is now known as zz_jgrimm16:11
*** zz_jgrimm is now known as jgrimm16:12
openstackgerritMerged openstack/oslo.vmware: Add missing links to README  https://review.openstack.org/15623716:18
*** hogepodge has quit IRC16:18
*** sreshetnyak has quit IRC16:21
*** sreshetnyak has joined #openstack-oslo16:22
*** hogepodge has joined #openstack-oslo16:23
openstackgerritMerged openstack/taskflow: Add retries to fetching the zookeeper server version  https://review.openstack.org/15595016:25
openstackgerritJulien Danjou proposed openstack/oslo.utils: Deprecate strtime  https://review.openstack.org/14850016:26
*** zigo has quit IRC16:26
*** zigo has joined #openstack-oslo16:31
*** e0ne has joined #openstack-oslo16:38
*** ramishra has joined #openstack-oslo16:38
*** ramishra has left #openstack-oslo16:38
openstackgerritMerged openstack/oslo.policy: Do not log on missing or empty policy_dirs  https://review.openstack.org/15474216:47
*** i159 has quit IRC16:50
*** dulek has quit IRC16:52
*** vigneshvar has joined #openstack-oslo16:53
*** e0ne is now known as e0ne_16:56
*** e0ne_ is now known as e0ne17:00
amrithdhellmann, is this the right place to ask a python question (that is specific to openstack and may have an OSLO answer?)17:02
dhellmannamrith: It's hard to say without knowing what the question is. :-)17:03
amrithok, 1 paste bomb coming up. ;)17:03
dhellmannamrith: if it's that big, maybe the mailing list is a better place17:05
amrith<paste-bomb>17:05
amriththere are tons of places in trove where we do imports.17:05
amriththese are of the form17:05
amrithfrom x import y17:05
amrithfrom x import y as z17:05
amrithimport x17:06
amrithI'd like to restructure the code and move modules (x) around within the code base.17:06
amrithAnd I expect that over time these same modules will move again and again.17:06
amrithBasically we're trying to create a notion of "stable", "experimental" and "preview" guest agents.17:06
amrithTo that end, I'd like to make the 'x' a string and dynamically load 'x' from that string.17:06
amrithThat makes it easier for me to fix all the code in the future when I move a module from point A to point B; fix the string and stuff magically works.17:06
dansmithdhellmann: the unit tests don't end up in the release such that a project like nova could import/use stuff in tests/ right?17:06
amrithWith importlib I can deal with the third and first kinds of imports. Having trouble with the second. Was wondering whether (a) there's a simple reference you could recommend and (b) is there any OSLO support for this kind of thing.17:06
amrith</paste-bomb>17:06
amrithoops17:06
amrithjust overlapped with your question.17:06
amrithyes, maybe ML17:06
dhellmanndansmith: they do, but we do not expect anyone to use anything in oslo_foo/tests/ and so we don't provide a stable API for them17:06
dansmithdhellmann: okay, so there is some object-sanity and object-version-hash test stuff that we really should export for other projects to use so they don't recreate the wheel.. should that stuff live outside of tests/ for that reason?17:07
dhellmannamrith: can you use stevedore with separate namespaces?17:07
* amrith ponders17:08
dhellmanndansmith: good question, I think we do something like that in oslo.db where we provide a base class for tests17:08
amrithdhellmann, let me look. not sure but I'll look.17:08
dhellmanndansmith: in the other libs we've been able to just use fixtures, I think, but it sounds like this wouldn't fit into that sort of structure?17:08
dansmithdhellmann: yeah, it's not really a fixture sort of thing17:09
dhellmanndansmith: see oslo_db/sqlalchemy/test_*.py17:09
dhellmannalthough IIRC, using test_* as the filename caused issues, so maybe we can come up with a name that doesn't match the regex testr uses to load test cases17:09
dansmithdhellmann: hmm, sqlalchemy/test/test_base.py seems to just import * from itself.. am I missing something?17:10
dhellmanndansmith: oslo_db not oslo/db17:11
dansmithdhellmann: okay17:11
dhellmanndansmith: http://git.openstack.org/cgit/openstack/oslo.db/tree/oslo_db/sqlalchemy/test_base.py17:11
dansmithyeah17:11
dansmithokay, I'll do something like this, without the word test in the path17:12
*** ihrachyshka has quit IRC17:12
dhellmannamrith: stevedore may not help you if you're needing to update the imports inside the experimental modules themselves17:12
dhellmannamrith: it would be better to organize the code in a way that doesn't have to change when the experimental status of the driver changes17:12
dhellmanndansmith: sounds good17:13
*** noelbk has joined #openstack-oslo17:18
openstackgerritMerged openstack/oslo-incubator: Switch from oslo.config to oslo_config  https://review.openstack.org/15586917:23
*** dtantsur has quit IRC17:25
*** miqui_away has quit IRC17:25
*** mtreinish has quit IRC17:25
*** tsufiev has quit IRC17:25
*** JoshNang has quit IRC17:25
*** SergeyLukjanov has quit IRC17:25
*** rharwood has quit IRC17:25
*** dtantsur has joined #openstack-oslo17:31
*** miqui_away has joined #openstack-oslo17:31
*** mtreinish has joined #openstack-oslo17:31
*** tsufiev has joined #openstack-oslo17:31
*** JoshNang has joined #openstack-oslo17:31
*** SergeyLukjanov has joined #openstack-oslo17:31
*** rharwood has joined #openstack-oslo17:31
dhellmannjd__, harlowja_away, dims__: is https://review.openstack.org/#/c/154547/6 a backwards-incompatible change to the msgpack format?17:35
jd__dhellmann: yes, though we can make it read old format if needed – i'm not sure we released it though?17:36
dhellmannjd__: ah, good point, we haven't released the msgpack serializer yet17:37
*** harlowja_at_home has joined #openstack-oslo17:37
dhellmannjd__: I'm doing a release for oslo.serialization right now17:37
jd__maybe we could mark it experimental as I expect us to change a few things17:37
jd__ok :)17:37
jd__dhellmann: don't you want to review my patches first?17:37
jd__dhellmann: https://review.openstack.org/#/q/status:open+project:openstack/oslo.serialization+branch:master+topic:jd/serialize-set,n,z17:38
dhellmannjd__: ok, I see 2 are already approved so let me look at the other17:38
*** andrearosa has quit IRC17:39
dhellmanndims__, bnemec, flaper87 : if you have a minute to look at https://review.openstack.org/#/c/147194/ we can include it in the serialization release17:39
*** andreykurilin_ has quit IRC17:40
dims__147194 - Done17:42
openstackgerritJoshua Harlow proposed openstack/taskflow: Allow turning off the version check  https://review.openstack.org/15630317:50
*** terryw is now known as otherwiseguy17:50
*** hogepodge has quit IRC17:53
*** stevemar has joined #openstack-oslo17:53
*** hogepodge has joined #openstack-oslo17:54
*** dtantsur is now known as dtantsur|afk17:56
harlowja_at_homejd__, if u get some time let me know what u think of https://review.openstack.org/#/c/156013/17:59
harlowja_at_homeits a nice little change i think17:59
*** amotoki_ has joined #openstack-oslo18:04
*** e0ne has quit IRC18:05
*** amotoki_ has quit IRC18:08
*** rushiagr is now known as rushiagr_away18:11
openstackgerritJoshua Harlow proposed openstack/taskflow: Revert "Add retries to fetching the zookeeper server version"  https://review.openstack.org/15631718:13
openstackgerritMerged openstack/oslo.concurrency: Ability to set working directory  https://review.openstack.org/15584718:17
openstackgerritMerged openstack/oslo.serialization: Use default in dumps()  https://review.openstack.org/14719418:28
openstackgerritMerged openstack/oslo.serialization: jsonutils: support UUID encoding  https://review.openstack.org/14719818:29
openstackgerritMerged openstack/oslo.serialization: jsonutils: add set() tests and simplify recursive code  https://review.openstack.org/14719518:29
*** MasterPiece has joined #openstack-oslo18:38
*** andreykurilin_ has joined #openstack-oslo18:46
*** e0ne has joined #openstack-oslo18:56
*** crc32 has joined #openstack-oslo19:01
*** crc32 has quit IRC19:14
openstackgerritMerged openstack/taskflow: adding check for str/unicode type in requires  https://review.openstack.org/15609419:15
*** dims__ has quit IRC19:25
openstackgerritMerged openstack/taskflow: Allow turning off the version check  https://review.openstack.org/15630319:25
*** noelbk has quit IRC19:31
openstackgerritMarian Horban proposed openstack/oslo-incubator: Using of waitpid system call was optimized  https://review.openstack.org/15634519:39
openstackgerritMerged openstack/taskflow: Revert "Add retries to fetching the zookeeper server version"  https://review.openstack.org/15631719:44
*** noelbk has joined #openstack-oslo19:47
*** MasterPiece has quit IRC19:49
*** amrith is now known as _amrith_19:55
jogozzzeek: nice blog post post, I really enjoyed it19:55
*** achanda has joined #openstack-oslo19:58
openstackgerritMarian Horban proposed openstack/oslo-incubator: Using of waitpid system call was optimized  https://review.openstack.org/15634520:08
*** achanda has quit IRC20:09
*** andreykurilin_ has quit IRC20:21
*** andreykurilin_ has joined #openstack-oslo20:21
*** dims__ has joined #openstack-oslo20:25
*** miqui_away has quit IRC20:30
*** miqui_ has joined #openstack-oslo20:30
*** andreykurilin_ has quit IRC20:30
*** dims__ has quit IRC20:30
*** andreykurilin_ has joined #openstack-oslo20:31
*** harlowja_at_home has quit IRC20:31
zzzeekjogo: thank!20:34
zzzeeks!20:34
*** achanda has joined #openstack-oslo20:45
*** dulek has joined #openstack-oslo20:45
*** samueldmq has joined #openstack-oslo20:54
*** samueldmq has left #openstack-oslo20:54
*** ajo has quit IRC20:55
*** dulek has quit IRC20:56
*** devlaps has joined #openstack-oslo21:01
*** achanda has quit IRC21:06
*** jgrimm is now known as zz_jgrimm21:12
*** dims__ has joined #openstack-oslo21:33
*** dims_ has joined #openstack-oslo21:34
*** dims__ has quit IRC21:38
*** himangi has joined #openstack-oslo21:49
*** andreykurilin_ has quit IRC21:56
openstackgerritDan Smith proposed openstack/oslo.versionedobjects: Remove internal items from public namespace  https://review.openstack.org/15639322:18
openstackgerritDan Smith proposed openstack/oslo.versionedobjects: Generalize the indirection_api interface  https://review.openstack.org/15639422:18
openstackgerritDan Smith proposed openstack/oslo.versionedobjects: Remove Nova objects module registration code  https://review.openstack.org/15639522:18
openstackgerritDan Smith proposed openstack/oslo.versionedobjects: Add conditional object registration  https://review.openstack.org/15639622:18
openstackgerritDan Smith proposed openstack/oslo.versionedobjects: Generalize remote testing infrastructure  https://review.openstack.org/15639722:18
openstackgerritDan Smith proposed openstack/oslo.versionedobjects: Generalize object hash-based change detection  https://review.openstack.org/15639822:18
openstackgerritDan Smith proposed openstack/oslo.versionedobjects: Generalize object dependency change detection  https://review.openstack.org/15639922:18
dansmithdims_: jaypipes: ^22:21
dansmithI'm making my way through the work items22:21
*** himangi has quit IRC22:22
dims_dansmith: ack. heading out to get some pizza and will review after that22:23
dansmithdims_: I'll allow it22:23
dims_thanks :)22:23
openstackgerritSteve Martinelli proposed openstack/oslo.policy: Use single quotes consistently  https://review.openstack.org/15640422:27
openstackgerritSteve Martinelli proposed openstack/oslo.policy: Fix minor spelling issues in oslo.policy  https://review.openstack.org/15640522:27
jaypipesdansmith: will try to get to those in a bit.22:28
dansmiththey're not critical22:28
dansmithand I'm probably failing some py3 stuff still22:28
*** achanda has joined #openstack-oslo22:34
*** mriedem has quit IRC22:46
*** e0ne has quit IRC22:50
*** dims_ has quit IRC22:51
*** dims__ has joined #openstack-oslo22:51
*** kgiusti has left #openstack-oslo22:52
*** trown is now known as trown|outttypeww22:52
*** tsekiyama has quit IRC22:58
*** salv-orlando has joined #openstack-oslo23:02
*** tsekiyama has joined #openstack-oslo23:03
openstackgerritDan Smith proposed openstack/oslo.versionedobjects: Generalize object dependency change detection  https://review.openstack.org/15639923:07
openstackgerritDan Smith proposed openstack/oslo.versionedobjects: Generalize object hash-based change detection  https://review.openstack.org/15639823:07
openstackgerritDan Smith proposed openstack/oslo.versionedobjects: Generalize remote testing infrastructure  https://review.openstack.org/15639723:07
openstackgerritDan Smith proposed openstack/oslo.versionedobjects: Add conditional object registration  https://review.openstack.org/15639623:07
*** cdent has quit IRC23:09
*** achanda has quit IRC23:21
*** dims_ has joined #openstack-oslo23:30
*** dims__ has quit IRC23:32
*** gordc has quit IRC23:32
openstackgerritDoug Hellmann proposed openstack/oslo-specs: Open specs repository for Liberty proposals  https://review.openstack.org/15641823:44
openstackgerritDan Smith proposed openstack/oslo.versionedobjects: Generalize object dependency change detection  https://review.openstack.org/15639923:47
openstackgerritDan Smith proposed openstack/oslo.versionedobjects: Generalize object hash-based change detection  https://review.openstack.org/15639823:47
openstackgerritDan Smith proposed openstack/oslo.versionedobjects: Generalize remote testing infrastructure  https://review.openstack.org/15639723:47
*** dims__ has joined #openstack-oslo23:47
*** dims_ has quit IRC23:47
openstackgerritMerged openstack/oslo-specs: Open specs repository for Liberty proposals  https://review.openstack.org/15641823:49
*** himangi has joined #openstack-oslo23:50
*** dims__ has quit IRC23:56
*** dims_ has joined #openstack-oslo23:56

Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!