Tuesday, 2018-09-25

*** mriedem_away has quit IRC00:10
*** alex_xu has joined #openstack-placement02:27
*** tetsuro has quit IRC04:05
*** e0ne has joined #openstack-placement04:28
*** e0ne has quit IRC04:51
*** e0ne has joined #openstack-placement05:09
*** e0ne has quit IRC05:10
*** belmoreira has joined #openstack-placement05:58
openstackgerritMerged openstack/osc-placement stable/rocky: import zuul job settings from project-config  https://review.openstack.org/60140206:00
*** s10 has joined #openstack-placement06:37
*** s10 has quit IRC06:51
*** helenafm has joined #openstack-placement07:35
*** e0ne has joined #openstack-placement07:52
*** e0ne has quit IRC07:53
*** s10 has joined #openstack-placement08:01
*** ttsiouts has joined #openstack-placement08:27
*** ttsiouts has quit IRC08:46
*** ttsiouts has joined #openstack-placement08:47
*** ttsiouts has quit IRC08:51
*** alex_xu has quit IRC08:54
*** alex_xu has joined #openstack-placement08:56
*** e0ne has joined #openstack-placement08:59
*** ttsiouts has joined #openstack-placement09:07
*** ttsiouts has quit IRC09:10
*** ttsiouts has joined #openstack-placement09:10
*** tetsuro has joined #openstack-placement09:11
*** tssurya has joined #openstack-placement09:17
*** s10 has quit IRC09:51
*** takashin has left #openstack-placement10:01
*** e0ne has quit IRC10:12
*** ttsiouts has quit IRC10:35
*** s10 has joined #openstack-placement10:42
*** s10 has quit IRC10:47
*** s10 has joined #openstack-placement10:47
*** stephenfin_ is now known as stephenfin10:48
*** s10 has quit IRC10:54
*** e0ne has joined #openstack-placement10:59
*** ttsiouts has joined #openstack-placement11:04
*** helenafm has quit IRC11:10
*** e0ne_ has joined #openstack-placement11:22
*** e0ne has quit IRC11:25
*** helenafm has joined #openstack-placement11:57
openstackgerritTetsuro Nakamura proposed openstack/placement master: Fix aggregate members in nested alloc candidates  https://review.openstack.org/60263912:08
openstackgerritTetsuro Nakamura proposed openstack/placement master: Add a document for allcation candidates constraint  https://review.openstack.org/60424312:08
*** ttsiouts has quit IRC12:09
*** ttsiouts has joined #openstack-placement12:21
*** tetsuro has quit IRC12:28
lyarwoodI don't see cdent around today but if anyone else is around I'd like to land https://review.openstack.org/#/c/604167/ sooner rather than later as it's blocking some downstream CI at present.12:32
lyarwoodFWIW the openstack/nova ports are passing12:32
edleafelyarwood: FYI cdent is off M-W this week12:57
efriedlyarwood: I was deliberately leaving that one for cdent since he said he wanted to play with it some more.12:58
efriedcan it wait for Thursday?12:58
edleafeefried: it seems pretty straightforward. If it borks the CI, we can always revert12:59
efriedswhat I was thinking12:59
efriedbut wanted to understand whether that was warranted12:59
edleafeIt would seem pretty fragile if any tests depended on configuration being left over from a prior test :)13:00
*** tssurya has quit IRC13:01
*** tssurya has joined #openstack-placement13:01
lyarwoodedleafe: ack thanks13:01
efriedI'm curious how anything in openstack/placement is blocking downstream CI13:01
efriedbut yeah, I'll approve it.13:01
lyarwoodefried: sorry keep getting dragged away13:02
edleafeefried: lyarwood: I'd be curious about how this was discovered13:02
lyarwoodefried: openstack/placement isn't the issue here, it's openstack/nova on stable/rocky13:02
lyarwoodedleafe: there's a deployment race in TripleO at the moment that borks the wsgi app, the reload then fails thanks to this.13:02
lyarwoodedleafe: it's not an easy race to fix at present13:03
efriedlyarwood: so you need this merged in openstack/placement master in order to backport it to openstack/nova stable/rocky ?13:03
lyarwoodefried: yeah correct13:03
efriedo... kay.13:03
efried+A13:03
edleafeCode splitting side effects FTW!!13:04
lyarwoodFWIW we've seen this outside of the deployment race in the past as well, we just never got around to fixing it.13:04
efriedI mean, we don't need it merged to backport, but I guess the point either way is that cdent wanted to play with it.13:04
lyarwoodyeah understood, I ported / backported the change to openstack/nova and everything appears to be happy13:05
efriedlyarwood: So why are you blocking the nova changes?13:05
lyarwoodefried: I just assumed we would want to get this into openstack/placement first13:05
*** belmoreira has quit IRC13:05
lyarwoodefried: that's how I've handled ports between split out projects like this in the past13:05
efriedokay13:05
lyarwoodefried: between os-brick and nova for example13:05
efriedIMO we should merge the master ones in parallel. Can you unblock https://review.openstack.org/#/c/604693/ ?13:06
lyarwoodsure13:06
efriedbauzas: quick +A on https://review.openstack.org/#/c/604693/ ?13:07
lyarwoodefried / edleafe ; thanks again :)13:08
edleafenp13:08
efriedyahyoubetcha13:08
bauzasefried: yup13:09
efriedthx13:09
bauzasefried: tbc, should we have Depends-On for Nova changes ?13:09
bauzasI think we said yes in the PTG13:09
efriedbauzas: You mean a depends-on *from* the nova change *to* the placement change?13:09
bauzasnope, the other way13:10
efriedthat doesn't make sense13:10
bauzasokay, fine by me13:10
efriedI mean, it doesn't make sense to me. What would that accomplish?13:10
lyarwoodtbh I've only ever used depends-on to pull in changes from other projects into the same underlying devstack build for CI13:11
efriedright13:16
openstackgerritMerged openstack/placement master: wsgi: Always reset conf.CONF when starting the application  https://review.openstack.org/60416713:19
bauzasanyway, nevermind13:22
*** ttsiouts has quit IRC13:27
*** belmoreira has joined #openstack-placement13:29
*** ttsiouts has joined #openstack-placement14:05
*** e0ne_ has quit IRC14:10
*** e0ne has joined #openstack-placement14:10
*** mriedem has joined #openstack-placement14:26
*** yikun has quit IRC14:47
*** ttsiouts has quit IRC15:08
*** ttsiouts has joined #openstack-placement15:10
*** e0ne has quit IRC15:37
*** ttsiouts has quit IRC15:40
*** ttsiouts has joined #openstack-placement15:43
*** helenafm has quit IRC15:44
*** belmoreira has quit IRC15:52
openstackgerritsean mooney proposed openstack/nova-specs master: Add spec for sriov live migration  https://review.openstack.org/60511615:54
*** ttsiouts has quit IRC16:20
*** ttsiouts has joined #openstack-placement16:20
*** ttsiouts has quit IRC16:25
*** s10 has joined #openstack-placement16:37
*** s10 has quit IRC16:46
openstackgerritEric Fried proposed openstack/placement master: Add a document for allcation candidates constraint  https://review.openstack.org/60424316:47
*** tssurya has quit IRC17:51
*** e0ne has joined #openstack-placement19:41
*** e0ne has quit IRC20:05
*** e0ne has joined #openstack-placement20:19
*** e0ne has quit IRC20:43
*** ttsiouts has joined #openstack-placement20:53
openstackgerritMerged openstack/placement master: oslo_config fixture in policy tests and 'placement' in policy  https://review.openstack.org/60112921:41
openstackgerritMerged openstack/placement master: Use both unit and functional for coverage testing  https://review.openstack.org/60050121:47
openstackgerritMerged openstack/placement master: Add a zuul check job for coverage  https://review.openstack.org/60050221:52
openstackgerritMerged openstack/placement master: Put stestr group_regex in .stestr.conf  https://review.openstack.org/60107321:52
*** ttsiouts has quit IRC22:05
*** ttsiouts has joined #openstack-placement22:06
*** mriedem has quit IRC22:10
*** ttsiouts has quit IRC22:10
*** jaypipes has quit IRC23:21

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