Monday, 2019-10-28

openstackgerritMerged openstack/os-traits master: Stop testing with py2 and switch to ussuri jobs  https://review.opendev.org/69099701:26
openstackgerritMerged openstack/os-resource-classes master: Stop testing with py2  https://review.opendev.org/69099601:28
*** ttsiouts has joined #openstack-placement04:51
*** ttsiouts has quit IRC05:00
*** tetsuro has quit IRC05:32
*** ttsiouts has joined #openstack-placement05:39
*** ttsiouts has quit IRC05:59
*** ttsiouts has joined #openstack-placement06:19
*** ttsiouts has quit IRC06:28
*** tetsuro has joined #openstack-placement06:36
*** ttsiouts has joined #openstack-placement07:52
*** ttsiouts has quit IRC08:42
*** ttsiouts has joined #openstack-placement08:43
*** cdent has joined #openstack-placement08:47
*** ttsiouts has quit IRC08:48
*** ttsiouts has joined #openstack-placement08:57
*** artom has joined #openstack-placement11:34
*** ttsiouts has quit IRC12:04
*** ttsiouts has joined #openstack-placement12:17
*** dklyle has quit IRC13:12
*** mriedem has joined #openstack-placement13:16
*** ttsiouts has quit IRC13:18
*** ttsiouts has joined #openstack-placement13:32
*** dklyle has joined #openstack-placement14:07
*** artom has quit IRC14:50
*** dklyle has quit IRC14:58
*** dklyle has joined #openstack-placement14:59
*** ttsiouts has quit IRC15:55
*** ttsiouts has joined #openstack-placement15:56
efriedcdent: I wouldn't mind an opinion from you on https://review.opendev.org/#/c/690996/ (post-merge comments about mentioning openstack releases in os-resource-classes release notes)15:58
* cdent looks15:58
efriedtrust tetsuro to ask the hard questions :P16:01
cdentquite16:01
efriedcdent: slightly related, I happened to notice that the sort order on https://releases.openstack.org/teams/placement.html is pretty strange from the reader's perspective.16:02
efriedI get why it is the way it is, and I'm not quite sure what would be better.16:02
cdenti think on both that and the release notes the issue is fairly moot16:03
cdentanyone who cares about release cycles is caring about the placement service: that shows up where it should on placement.html16:03
efriedin the sense that "nobody is gonna care"?16:03
efriedokay.16:03
cdentand for os-* for placement and nove they are installed as part of the requirements resolution process16:04
efriedyes16:04
efriedand those guys are going to be trumpeting py3-only-ness anyway.16:04
efriedokay, I'm happy with that answer. Thanks.16:04
cdentI reckon it would be nice is the os-* release notes could have some signal of "oh by the way we don't use those names anymore" but if it is hard to do, I don't think it is a huge loss16:05
efried"those names" -- stein/train/ussuri etc?16:07
* cdent nod16:07
cdents16:07
efriedThat toc pretty much implies as much.16:07
efriedthough you kind of have to think it through to figure that out.16:08
efriedfact that os-* doesn't show up after stein16:08
efriedand os-traits never did16:08
cdentA large part of me thinks: let's wait to see if anyone cares/notices and then fix it according to what that person wants16:08
efriedwfm16:08
cdentsolid16:09
cdentI linked back to this chat from the review, for sake of ref16:09
efriedthanks.16:09
efriedIn other news, I'm a little bit concerned that we done merged our drop-support patches before all of our consumers (well, nova) did same.16:10
efriedof course, the chances of merging into ot/orc a py2-incompatible change that breaks nova is negligible.16:10
efriedbut we probably should have held off for the sake of form.16:10
cdentthey can always pin16:14
efriedmm16:14
cdentor rather to be more clear16:14
efriedand actually, I suppose as long as we don't *release* it yet, we're fine.16:14
cdentlet's be the incentive that makes time catch up to the present16:14
efriedheh16:14
cdentI would have chosen to drop py2 in placement last year if "allowed"16:15
efriedI guess at some point soon I ought to start learning py316:15
efriedI know absolutely nothing about it that doesn't have to do with six.16:15
cdenthere's all you need to know:16:15
cdentprint "foo" must become print("foo")16:16
cdentdone16:16
efriedwell, yeah, I knew *that*.16:16
efriedbut people are going to start using py3-isms in code16:16
efried...that I will have to review.16:16
efriedLike, can't you type method params now?16:16
cdentyou can but don't have to16:19
gibiwhat I leard recently that you can have dataclasses but I cannot really use them in inheritance if the base class has any field that has default value16:19
cdentthat's unfortunate16:20
gibiso I want back and typed in my classes the old way that works :)16:20
*** ttsiouts has quit IRC16:39
*** artom has joined #openstack-placement16:55
*** dklyle has quit IRC18:08
*** david-lyle has joined #openstack-placement18:08
*** cdent has quit IRC18:20
*** openstackgerrit has quit IRC18:50
*** artom has quit IRC19:06
*** david-lyle is now known as dklyle19:29
*** artom has joined #openstack-placement20:40
*** openstackgerrit has joined #openstack-placement20:50
openstackgerritMatt Riedemann proposed openstack/osc-placement master: Add --no-provider option to resource provider allocation set  https://review.opendev.org/69142220:50
openstackgerritMatt Riedemann proposed openstack/osc-placement master: Add --no-provider option to resource provider allocation set  https://review.opendev.org/69142220:55
*** gibi has quit IRC21:43
*** gibi has joined #openstack-placement21:53
*** gibi has quit IRC22:06
*** gibi has joined #openstack-placement22:51
*** mriedem has quit IRC23:13

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