Tuesday, 2018-09-04

openstackgerritBrin Zhang proposed openstack/nova master: Need further updates, no need to review  https://review.openstack.org/59927600:06
*** Nel1x has joined #openstack-placement00:57
openstackgerritZhenyu Zheng proposed openstack/nova-specs master: Make scheduling weight more granular  https://review.openstack.org/59930801:08
*** fried_rice is now known as efried01:27
*** tetsuro has joined #openstack-placement01:47
*** tetsuro has quit IRC01:53
openstackgerritNaichuan Sun proposed openstack/nova master: xenapi(N-R-P): support compute node resource provider update  https://review.openstack.org/52104102:47
openstackgerritfupingxie proposed openstack/nova master: Support list for alias in pci section in nova.conf  https://review.openstack.org/59224302:58
openstackgerritfupingxie proposed openstack/nova master: Add an example to add more pci devices in nova.conf  https://review.openstack.org/59224303:01
*** Nel1x has quit IRC03:14
openstackgerritjichenjc proposed openstack/nova master: WIP: add check for deleted flag  https://review.openstack.org/59949204:16
openstackgerritjichenjc proposed openstack/nova master: Move str to six.string_types  https://review.openstack.org/59949304:21
*** e0ne has joined #openstack-placement05:13
openstackgerritgaryk proposed openstack/nova master: Docs: update link for remote debugging  https://review.openstack.org/59131605:32
*** lei-zh has joined #openstack-placement05:39
openstackgerritgaryk proposed openstack/nova master: Docs: update link for remote debugging  https://review.openstack.org/59131606:03
*** e0ne has quit IRC06:44
openstackgerritsahid proposed openstack/nova stable/rocky: libvirt: skip setting rx/tx queue sizes for not virto interfaces  https://review.openstack.org/59950607:09
*** tssurya has joined #openstack-placement07:45
*** gibi has joined #openstack-placement07:45
gibio/07:47
openstackgerritfupingxie proposed openstack/nova master: Delete allocations for instances that have been moved to another node  https://review.openstack.org/58289908:08
openstackgerritfupingxie proposed openstack/nova master: Delete allocations for instances that have been moved to another node  https://review.openstack.org/58289908:14
openstackgerritfupingxie proposed openstack/nova master: Delete allocations for instances that have been moved to another node  https://review.openstack.org/58289908:18
openstackgerritBalazs Gibizer proposed openstack/placement master: Add api-ref job  https://review.openstack.org/59952408:32
*** cdent has joined #openstack-placement08:34
openstackgerritBalazs Gibizer proposed openstack/placement master: Add api-ref job  https://review.openstack.org/59952408:34
cdenthuzzah08:35
*** e0ne has joined #openstack-placement08:37
openstackgerritgaryk proposed openstack/nova master: Docs: update link for remote debugging  https://review.openstack.org/59131608:37
openstackgerritBalazs Gibizer proposed openstack/placement master: Add api-ref job  https://review.openstack.org/59952408:42
*** ttsiouts has joined #openstack-placement08:51
*** ttsiouts has quit IRC09:06
*** ttsiouts has joined #openstack-placement09:06
*** lei-zh has quit IRC09:33
openstackgerritNaichuan Sun proposed openstack/nova master: xenapi(N-R-P)(WIP): support compute node resource provider update  https://review.openstack.org/52104109:37
openstackgerritBalazs Gibizer proposed openstack/placement master: Make placement fixture usable from nova  https://review.openstack.org/59955110:05
cdentgibi: have you been experimenting with making that ^ work?10:12
openstackgerritChen proposed openstack/nova master: doc: update info for hypervisors  https://review.openstack.org/59955410:13
openstackgerritBalazs Gibizer proposed openstack/nova master: WIP: Use placement from separate repo in functional test  https://review.openstack.org/59955610:14
gibicdent: yes. It seems it works in ^^ at least for a subset of functional tests10:15
cdentyay!10:15
gibicdent: it does not seems too hard to make it work and it will gives us an early integration point between nova and the separated placement code10:15
cdentgreat10:16
openstackgerritfupingxie proposed openstack/nova master: Delete allocations for instances that have been moved to another node  https://review.openstack.org/58289910:16
cdentit looks like you're doing the smaller, tidier, easier to understand version of what I did at: https://review.openstack.org/#/c/596291/10:16
openstackgerritChris Dent proposed openstack/placement master: Add python 3.6 test jobs  https://review.openstack.org/59955810:17
gibicdent: yes, the idea is coming from your early patch making the test works10:20
* gibi goes finding some food10:21
*** ttsiouts has quit IRC10:26
*** rubasov has joined #openstack-placement10:30
*** rubasov has quit IRC10:53
*** cdent has quit IRC10:54
*** ttsiouts has joined #openstack-placement11:23
*** cdent has joined #openstack-placement11:33
*** jaypipes has joined #openstack-placement11:44
*** cdent has quit IRC11:49
*** dims has joined #openstack-placement11:56
*** ttsiouts has quit IRC12:04
*** ttsiouts has joined #openstack-placement12:05
efriedō/12:06
jaypipesmornin12:07
*** ttsiouts_ has joined #openstack-placement12:07
* gibi waves to efried and jaypipes 12:07
efriedAnything of urgence before I dive into the usual routine?12:08
jaypipesnothing from me12:08
* jaypipes waves at gibi12:08
*** ttsiouts_ has quit IRC12:09
gibiefried: nothing urgent as far as I see12:09
efriedroger wilco.12:09
*** ttsiouts has quit IRC12:09
*** ttsiouts has joined #openstack-placement12:10
openstackgerritSurya Seetharaman proposed openstack/nova master: Merge security groups extension response into server view builder  https://review.openstack.org/58547512:29
openstackgerritSurya Seetharaman proposed openstack/nova master: Merge extended_status extension response into server view builder  https://review.openstack.org/59209212:29
openstackgerritSurya Seetharaman proposed openstack/nova master: Add scatter-gather-single-cell utility  https://review.openstack.org/59494712:29
openstackgerritSurya Seetharaman proposed openstack/nova master: Merge extended_volumes extension response into server view builder  https://review.openstack.org/59628512:30
openstackgerritSurya Seetharaman proposed openstack/nova master: Making instance/migration listing skipping down cells configurable  https://review.openstack.org/59242812:30
openstackgerritSurya Seetharaman proposed openstack/nova master: Add get_by_cell_and_project() method to InstanceMappingList  https://review.openstack.org/59165612:30
openstackgerritSurya Seetharaman proposed openstack/nova master: Return a minimal construct for nova list when a cell is down  https://review.openstack.org/56778512:30
*** ttsiouts has quit IRC12:32
*** ttsiouts has joined #openstack-placement12:41
openstackgerritStephen Finucane proposed openstack/nova-specs master: Re-propose numa-aware-live-migration spec  https://review.openstack.org/59958712:47
openstackgerritBrin Zhang proposed openstack/nova master: Need further updates, no need to review  https://review.openstack.org/59927612:52
*** mriedem has joined #openstack-placement12:53
*** lyarwood has joined #openstack-placement13:15
openstackgerritJay Pipes proposed openstack/nova-specs master: allow transferring ownership of instance  https://review.openstack.org/59959813:26
openstackgerritJay Pipes proposed openstack/nova-specs master: allow transferring ownership of instance  https://review.openstack.org/59959813:44
openstackgerritJay Pipes proposed openstack/nova-specs master: allow transferring ownership of instance  https://review.openstack.org/59959813:58
*** ttsiouts has quit IRC13:59
*** cdent has joined #openstack-placement14:18
edleafecdent: Glad to see that you were busy while we Americans were goofing off this past weekend14:23
cdentall that stuff was yeterday. i managed to goof off most of the weekend too14:23
cdentI'm very good when test driven. Less good otherwise.14:24
*** ttsiouts has joined #openstack-placement14:27
cdentefried, edleafe : after reading through the copies of the two dead etherpads I decided there wasn't that much content on them that's relevant now, so instead I made the new one with a more "link to other things" focus and invite your input (and jaypipes, gibi, tetsuro, everyone) please14:41
efriedack14:41
edleafecdent: I had the same thought as I sent that to you14:41
cdenthttps://etherpad.openstack.org/p/placement-extract-stein-314:41
cdentjaypipes, mriedem, melwitt: if one of you've got a moment to look at and +W https://review.openstack.org/#/c/598988/ and the patch immediately following, that will unblock a cascade of already approved things following15:13
cdentstephenfin: or perhaps you ^15:14
mriedemthat's essentially a cherry pick right?15:15
cdentmriedem: yes15:16
mriedemis there any reason we can't use the same change IDs for stuff like this?15:16
efriedone of them is a "cherry-pick", not the other.15:16
efriedHm, same change-id, I discarded that idea assuming it wasn't a valid thing to do across repositories... but maybe it is...15:16
jaypipesI was just about to ask the exact same question as mriedem.15:17
cdentchange ids are gerrit-wide15:17
efriedright, but I guess I thought that would... idk... overwrite the other change. But that doesn't make sense, cause we do it for backports.15:17
openstackgerritEric Fried proposed openstack/placement master: Fix race condition in reshaper handler  https://review.openstack.org/59898815:20
efriedmriedem, jaypipes: done.15:20
jrollI think the "unique key" in gerrit is (change id + repo + branch)15:20
*** openstackgerrit has quit IRC15:20
efriedyeah, makes sense if I had thought it through that far, which I hadn't.15:20
jrollthough maybe "repo" isn't in there, given this goes straight to the nova change rather than listing both: https://review.openstack.org/#/q/Ie21bddc186364db2b10af45546c35b9ad5a0208b15:21
mriedemyeah i noticed that too15:21
mriedemi don't know if that's new gerrit behavior or what15:21
efriedor if it's some index that takes a bit to update15:22
mriedemyeah might be that15:22
mriedemi seem to remember clarkb saying something like that once15:22
efriedI've noticed that such a URL hits patches which mention the change-id even in review comments and stuff, so it must be indexed somehow15:23
efriedand it's not surprising that that's not immediate.15:23
efriedthough15:23
efriedgiven that it was in the commit message before (just not as the change-id) I guess I would have still expected it to show up.15:23
efriedanyway15:23
efriedyou want me to change it back or are we good?15:23
mriedemi'm asking clark in -infra15:23
efriedcdent: Okay, freeze the world, I'm going to need to rebase the whole series. Stand by.15:30
cdentaye aye15:30
*** openstackgerrit has joined #openstack-placement15:32
openstackgerritEric Fried proposed openstack/placement master: Apply placement.rst change from Idf8997d5efdfdfca6  https://review.openstack.org/59900115:32
openstackgerritEric Fried proposed openstack/placement master: Rename the 'nova' directories to 'placement'  https://review.openstack.org/59894115:32
openstackgerritEric Fried proposed openstack/placement master: Move the api-ref directories  https://review.openstack.org/59894215:32
openstackgerritEric Fried proposed openstack/placement master: Remove the Nova aggregate files.  https://review.openstack.org/59894315:32
openstackgerritEric Fried proposed openstack/placement master: Move the placement code to the base  https://review.openstack.org/59894415:32
openstackgerritEric Fried proposed openstack/placement master: Move the functional test directories  https://review.openstack.org/59894515:32
openstackgerritEric Fried proposed openstack/placement master: Move the unit tests  https://review.openstack.org/59894615:32
openstackgerritEric Fried proposed openstack/placement master: Update the functional test import paths  https://review.openstack.org/59894715:32
openstackgerritEric Fried proposed openstack/placement master: Remove the import pathing for the old structure  https://review.openstack.org/59894815:32
openstackgerritEric Fried proposed openstack/placement master: Replace the nova import paths with placement  https://review.openstack.org/59894915:32
openstackgerritEric Fried proposed openstack/placement master: Update nova.db import paths  https://review.openstack.org/59895015:32
openstackgerritEric Fried proposed openstack/placement master: Inspect and correct tox.ini, .stestr.conf and setup.cfg  https://review.openstack.org/59937715:32
openstackgerritEric Fried proposed openstack/placement master: Update i18n handling to be placement oriented.  https://review.openstack.org/59937815:32
openstackgerritEric Fried proposed openstack/placement master: Mechanically correct import of functional base class  https://review.openstack.org/59937915:32
openstackgerritEric Fried proposed openstack/placement master: Correct several nova.tests and nova.tests.functional imports  https://review.openstack.org/59938015:32
openstackgerritEric Fried proposed openstack/placement master: Empty __init__.py files that should be empty  https://review.openstack.org/59938115:32
openstackgerritEric Fried proposed openstack/placement master: Remove some imports from test/fixtures that will not be used  https://review.openstack.org/59938215:32
openstackgerritEric Fried proposed openstack/placement master: Remove unused db functionality and files  https://review.openstack.org/59938315:32
openstackgerritEric Fried proposed openstack/placement master: Make pep8 tests voting  https://review.openstack.org/59941615:32
openstackgerritEric Fried proposed openstack/placement master: Add python 3.6 test jobs  https://review.openstack.org/59955815:32
openstackgerritEric Fried proposed openstack/placement master: Fix race condition in reshaper handler  https://review.openstack.org/59963915:32
efriedmriedem, cdent, jaypipes: Done ^15:32
efriednote completely new review15:33
efriedwhich... makes sense.15:33
cdentefried: that was a great opportunity to use -T on git review. not that it really matters at this stage15:54
openstackgerritSurya Seetharaman proposed openstack/nova master: Return a minimal construct for nova list when a cell is down  https://review.openstack.org/56778515:58
efriedcdent: Ahcrap, yeah, I totally should have, forgot. I wish there was a way to set it as the default behavior.16:02
cdentefried: I think we will all survive. But if not, it was good to know you.16:08
*** ttsiouts has quit IRC16:13
*** ttsiouts has joined #openstack-placement16:14
openstackgerritMatt Riedemann proposed openstack/nova master: Document unset/reset wrinkle for *_allocation_ratio options  https://review.openstack.org/59967016:14
*** ttsiouts has quit IRC16:19
openstackgerritMatt Riedemann proposed openstack/nova stable/rocky: Don't persist zero allocation ratios in ResourceTracker  https://review.openstack.org/59967216:21
openstackgerritMatt Riedemann proposed openstack/nova stable/rocky: Document unset/reset wrinkle for *_allocation_ratio options  https://review.openstack.org/59967316:21
openstackgerritMerged openstack/placement master: Fix race condition in reshaper handler  https://review.openstack.org/59963916:50
cdenthuzzah16:50
openstackgerritMerged openstack/placement master: Apply placement.rst change from Idf8997d5efdfdfca6  https://review.openstack.org/59900116:51
openstackgerritMerged openstack/placement master: Rename the 'nova' directories to 'placement'  https://review.openstack.org/59894116:51
openstackgerritMerged openstack/placement master: Move the api-ref directories  https://review.openstack.org/59894216:51
openstackgerritMerged openstack/placement master: Remove the Nova aggregate files.  https://review.openstack.org/59894316:51
openstackgerritMerged openstack/placement master: Move the placement code to the base  https://review.openstack.org/59894416:51
openstackgerritMerged openstack/placement master: Move the functional test directories  https://review.openstack.org/59894516:51
cdenthuzzah16:51
efriedCool, https://review.openstack.org/#/c/598946/ is the bottom now. I'll be getting back to reviewing these this afternoon.16:51
edleafeProgress!16:52
*** tssurya has quit IRC16:54
openstackgerritMerged openstack/nova master: Combine error handling blocks in _do_build_and_run_instance  https://review.openstack.org/54596016:56
openstackgerritBalazs Gibizer proposed openstack/placement master: Make placement fixture usable from nova  https://review.openstack.org/59955117:06
openstackgerritBalazs Gibizer proposed openstack/nova master: WIP: Use placement from separate repo in functional test  https://review.openstack.org/59955617:08
openstackgerritBalazs Gibizer proposed openstack/placement master: Make placement fixture usable from nova  https://review.openstack.org/59955117:10
openstackgerritBalazs Gibizer proposed openstack/nova master: WIP: Use placement from separate repo in functional test  https://review.openstack.org/59955617:10
*** tetsuro has joined #openstack-placement17:18
*** openstackgerrit has quit IRC17:22
*** openstackgerrit has joined #openstack-placement17:22
openstackgerritMerged openstack/nova master: Removing pip-missing-reqs from default tox jobs  https://review.openstack.org/59944217:22
cdentedleafe: your recent comment about "adding a TODO" on the database fixture reminded me of a thing: there were plenty of TODO places that were more designish or general and often I couldn't decide how/where to record them. things like "refactor the database fixture to be more tight"17:33
cdentin some contexts i'd use a bug for that, but feels too vague17:33
edleafeI added that to the extraction etherpad. That's probably the best place to track these things17:33
cdentah cool, good17:33
edleafeIt's still the removal of a nova-ism, not a bug per se17:34
cdentmaybe someday it can be a story17:35
*** e0ne has quit IRC17:44
efriedcdent: You feel like proposing an oslo.utils release to get uuidsentinel?18:06
cdentI wasn't in any rush on that because I figured we had enough stuff queued up in placement itself18:07
cdentbut I'll put it in my reminder queue18:07
efriedcdent: or I can do it, up to you.18:09
cdentif you're just gonna do it, go for it18:09
* cdent tries to keep his place in his current queue18:14
cdentit's not working18:14
* cdent still hasn't written the tc report18:14
*** tetsuro has quit IRC18:16
*** tetsuro has joined #openstack-placement18:16
openstackgerritmelanie witt proposed openstack/nova stable/rocky: Add functional test for affinity with multiple cells  https://review.openstack.org/59973118:18
openstackgerritmelanie witt proposed openstack/nova stable/rocky: Make scheduler.utils.setup_instance_group query all cells  https://review.openstack.org/59973218:18
*** tetsuro has quit IRC18:20
*** tetsuro has joined #openstack-placement18:35
*** tetsuro has quit IRC18:53
openstackgerritMatt Riedemann proposed openstack/nova master: WIP: Swallow fewer exceptions in _run_migration  https://review.openstack.org/59974418:55
openstackgerritMerged openstack/placement master: Move the unit tests  https://review.openstack.org/59894619:06
efriededleafe: Did you know you can avoid lts by using something other than / (e.g. comma) in your sed command?19:07
efriededleafe: Never mind, forget I mentioned it, I misread a thing.19:08
openstackgerritJay Pipes proposed openstack/nova-specs master: Standardize CPU resource tracking  https://review.openstack.org/55508119:08
edleafeefried: FWIW I was actually running Perl scripts with those replacement expressions :)19:09
efriededleafe: Same ability there, of course.19:09
efriedperl regex is a (more awesomer) superset of sed regex19:09
openstackgerritJay Pipes proposed openstack/nova-specs master: allow transferring ownership of instance  https://review.openstack.org/59959819:19
*** e0ne has joined #openstack-placement19:28
openstackgerritJay Pipes proposed openstack/nova-specs master: allow transferring ownership of instance  https://review.openstack.org/59959819:33
*** e0ne has quit IRC19:33
openstackgerritmelanie witt proposed openstack/nova stable/queens: Fix the request context in ServiceFixture  https://review.openstack.org/59976219:41
openstackgerritmelanie witt proposed openstack/nova stable/queens: Honor availability_zone hint via placement  https://review.openstack.org/59976319:41
openstackgerritmelanie witt proposed openstack/nova stable/queens: Improve NeutronFixture and remove unncessary stubbing  https://review.openstack.org/59976419:41
openstackgerritmelanie witt proposed openstack/nova stable/queens: Add functional test for affinity with multiple cells  https://review.openstack.org/59976519:41
openstackgerritmelanie witt proposed openstack/nova stable/queens: Make scheduler.utils.setup_instance_group query all cells  https://review.openstack.org/59976619:41
*** e0ne has joined #openstack-placement19:52
openstackgerritMatt Riedemann proposed openstack/nova master: WIP: Configure placement DB context manager for online_data_migrations  https://review.openstack.org/59982220:00
openstackgerritMatt Riedemann proposed openstack/nova master: Configure placement DB context manager for online_data_migrations  https://review.openstack.org/59974420:05
openstackgerritMatt Riedemann proposed openstack/nova master: Configure placement DB context manager for online_data_migrations  https://review.openstack.org/59974420:13
openstackgerritmelanie witt proposed openstack/nova stable/queens: Add functional test for affinity with multiple cells  https://review.openstack.org/59976520:42
openstackgerritmelanie witt proposed openstack/nova stable/queens: Make scheduler.utils.setup_instance_group query all cells  https://review.openstack.org/59976620:42
openstackgerritmelanie witt proposed openstack/nova stable/pike: Fix the request context in ServiceFixture  https://review.openstack.org/59983920:45
openstackgerritmelanie witt proposed openstack/nova stable/pike: Add functional test for affinity with multiple cells  https://review.openstack.org/59984020:45
openstackgerritmelanie witt proposed openstack/nova stable/pike: Make scheduler.utils.setup_instance_group query all cells  https://review.openstack.org/59984120:45
openstackgerritmelanie witt proposed openstack/nova stable/pike: Fix the request context in ServiceFixture  https://review.openstack.org/59983920:46
openstackgerritmelanie witt proposed openstack/nova stable/pike: Add functional test for affinity with multiple cells  https://review.openstack.org/59984020:46
openstackgerritmelanie witt proposed openstack/nova stable/pike: Make scheduler.utils.setup_instance_group query all cells  https://review.openstack.org/59984120:46
* cdent has to call it a day20:47
* cdent waves20:47
*** cdent has quit IRC20:47
openstackgerritChris Dent proposed openstack/placement master: WIP: Update requirements and test-requirements  https://review.openstack.org/59944320:49
openstackgerritChris Dent proposed openstack/placement master: Removing non-existent job from tox envlist  https://review.openstack.org/59944420:49
openstackgerritEric Fried proposed openstack/placement master: Fix aesthetic issues from I4974a28de541aace043504f  https://review.openstack.org/59984620:54
*** e0ne has quit IRC21:13
openstackgerritMerged openstack/placement master: Update the functional test import paths  https://review.openstack.org/59894721:47
openstackgerritMatt Riedemann proposed openstack/nova master: Fix TypeError in nova-manage cell_v2 list_cells  https://review.openstack.org/59986122:30
openstackgerritEric Fried proposed openstack/placement master: De-nova-ify and reformat contributor/placement.rst  https://review.openstack.org/59986423:06

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