| *** efried1 has joined #openstack-nova | 00:16 | |
| *** efried has quit IRC | 00:20 | |
| *** efried1 is now known as efried | 00:20 | |
| *** frankwang has joined #openstack-nova | 00:27 | |
| *** gbarros has quit IRC | 00:30 | |
| *** gbarros has joined #openstack-nova | 00:51 | |
| *** fanzhang has joined #openstack-nova | 01:04 | |
| *** mrsoul has quit IRC | 01:05 | |
| *** chason has joined #openstack-nova | 01:21 | |
| *** tetsuro_ has quit IRC | 01:23 | |
| *** chenyb4 has joined #openstack-nova | 01:35 | |
| *** hongbin has joined #openstack-nova | 01:39 | |
| *** Nel1x has joined #openstack-nova | 01:43 | |
| *** liuyulong has joined #openstack-nova | 01:54 | |
| openstackgerrit | zhufl proposed openstack/nova master: Fix none-ascii char in doc https://review.openstack.org/588422 | 01:55 |
|---|---|---|
| openstackgerrit | Pierre Blanc proposed openstack/nova master: Docs: Add guide to migrate instance with snapshot https://review.openstack.org/584442 | 02:01 |
| *** bkopilov has quit IRC | 02:14 | |
| *** Dinesh_Bhor has joined #openstack-nova | 02:17 | |
| openstackgerrit | lei zhang proposed openstack/nova master: [Docs] Update the confusing console output https://review.openstack.org/589004 | 02:30 |
| *** psachin has joined #openstack-nova | 02:31 | |
| openstackgerrit | Yikun Jiang (Kero) proposed openstack/nova master: Add microversion info in the os-server-groups API samples https://review.openstack.org/589006 | 02:34 |
| *** kiseok7 has quit IRC | 02:35 | |
| *** alex_xu has joined #openstack-nova | 02:42 | |
| *** gbarros has quit IRC | 02:46 | |
| *** gbarros has joined #openstack-nova | 02:50 | |
| openstackgerrit | lei zhang proposed openstack/nova master: [Docs] Update the confusing console output https://review.openstack.org/589004 | 02:52 |
| *** gbarros has quit IRC | 02:52 | |
| *** vivsoni has joined #openstack-nova | 02:56 | |
| openstackgerrit | lei zhang proposed openstack/nova master: [Docs] Update the compute node installation steps https://review.openstack.org/589008 | 03:02 |
| *** Nel1x has quit IRC | 03:34 | |
| *** udesale has joined #openstack-nova | 03:52 | |
| *** kevinbenton has quit IRC | 04:00 | |
| *** d34dh0r53 has quit IRC | 04:00 | |
| *** kevinbenton has joined #openstack-nova | 04:00 | |
| *** d34dh0r53 has joined #openstack-nova | 04:04 | |
| *** d34dh0r53 has quit IRC | 04:05 | |
| *** d34dh0r53 has joined #openstack-nova | 04:06 | |
| *** Dinesh_Bhor has quit IRC | 04:08 | |
| *** hongbin has quit IRC | 04:09 | |
| *** frankwang has quit IRC | 04:11 | |
| *** frankwang has joined #openstack-nova | 04:12 | |
| openstackgerrit | Tetsuro Nakamura proposed openstack/nova stable/queens: Don't filter out sibling sets with one core https://review.openstack.org/588570 | 04:15 |
| openstackgerrit | Tetsuro Nakamura proposed openstack/nova stable/queens: Ensure emulator threads are always calculated https://review.openstack.org/588571 | 04:15 |
| openstackgerrit | Tetsuro Nakamura proposed openstack/nova stable/queens: Always pass 'NUMACell.siblings' to _pack_instance_onto_cores' https://review.openstack.org/588572 | 04:15 |
| openstackgerrit | Tetsuro Nakamura proposed openstack/nova stable/queens: trivialfix: cleanup _pack_instance_onto_cores() https://review.openstack.org/588573 | 04:15 |
| openstackgerrit | Tetsuro Nakamura proposed openstack/nova stable/queens: Add unit tests for EmulatorThreadsTestCase https://review.openstack.org/588574 | 04:15 |
| openstackgerrit | Tetsuro Nakamura proposed openstack/nova stable/queens: Not use thread alloc policy for emulator thread https://review.openstack.org/588575 | 04:15 |
| *** frankwang has quit IRC | 04:51 | |
| *** icey has quit IRC | 05:13 | |
| *** icey has joined #openstack-nova | 05:14 | |
| *** Dinesh_Bhor has joined #openstack-nova | 05:24 | |
| *** hoonetorg has quit IRC | 05:28 | |
| *** udesale has quit IRC | 05:34 | |
| *** janki has joined #openstack-nova | 05:38 | |
| *** frankwang has joined #openstack-nova | 05:40 | |
| *** hoonetorg has joined #openstack-nova | 05:42 | |
| *** ircuser-1 has quit IRC | 05:43 | |
| *** Luzi has joined #openstack-nova | 05:53 | |
| *** nicolasbock has joined #openstack-nova | 06:05 | |
| *** moshele has joined #openstack-nova | 06:06 | |
| *** pcaruana has joined #openstack-nova | 06:07 | |
| *** udesale has joined #openstack-nova | 06:11 | |
| *** ratailor has joined #openstack-nova | 06:17 | |
| *** tetsuro_ has joined #openstack-nova | 06:34 | |
| *** adrianc_ has joined #openstack-nova | 06:36 | |
| *** tetsuro_ has quit IRC | 06:41 | |
| *** luksky has joined #openstack-nova | 06:42 | |
| *** ivve has joined #openstack-nova | 06:46 | |
| *** rcernin has quit IRC | 06:52 | |
| *** slaweq has quit IRC | 06:55 | |
| *** maciejjozefczyk has joined #openstack-nova | 06:56 | |
| *** slaweq has joined #openstack-nova | 06:58 | |
| openstackgerrit | Chen proposed openstack/nova master: Trivial fix on migration doc https://review.openstack.org/589028 | 07:08 |
| *** ajo has joined #openstack-nova | 07:09 | |
| *** deepak_mourya_ has joined #openstack-nova | 07:10 | |
| *** jhesketh_ has joined #openstack-nova | 07:10 | |
| *** jhesketh has quit IRC | 07:11 | |
| *** rmart04 has joined #openstack-nova | 07:14 | |
| *** gvrangan has joined #openstack-nova | 07:14 | |
| *** quiquell has joined #openstack-nova | 07:15 | |
| quiquell | Good morning | 07:15 |
| quiquell | We have found this http://logs.openstack.org/95/583195/18/check/tripleo-ci-centos-7-containers-multinode/769edac/logs/undercloud/var/log/containers/nova/nova-scheduler.log.txt.gz | 07:15 |
| quiquell | What it means ? | 07:15 |
| quiquell | 2018-08-03 22:57:32.138 26 ERROR nova.scheduler.client.report [req-e207e96f-2e1c-4648-afc7-be5c51d396ef 0dd904c9905d4ab7962c306a23e8e4ff 6825b026e19a450283f76bcc85f30a43 - default default] Failed to retrieve allocation candidates from placement API for filters: RequestGroup(use_same_provider=False, resources={CUSTOM_BAREMETAL:1}, traits=[], aggregates=[]) | 07:16 |
| quiquell | Got 400: {"errors": [{"status": 400, "request_id": "req-8b3d0e7f-0238-4290-99f9-4a3df8213231", "code": "placement.undefined_code", "detail": "The server could not comply with the request since it is either malformed or otherwise incorrect.\n\n Invalid resource class in resources parameter: No such resource class CUSTOM_BAREMETAL. ", "title": "Bad Request"}]}. | 07:16 |
| *** Dinesh_Bhor has quit IRC | 07:18 | |
| *** vishakha_ has joined #openstack-nova | 07:19 | |
| *** Bhujay has joined #openstack-nova | 07:21 | |
| *** ccamacho has joined #openstack-nova | 07:23 | |
| *** Dinesh_Bhor has joined #openstack-nova | 07:24 | |
| alex_xu | quiquell: 'CUSTOM_BAREMETAL' is custom trait, you must create it by the placement api before you refer it. In the ironic case, nova-manager will create that custom trait for ironic virt driver. | 07:30 |
| alex_xu | quiquell: check the nova-manager log, sounds like nova-manager doesn't startup correctly http://logs.openstack.org/95/583195/18/check/tripleo-ci-centos-7-containers-multinode/769edac/logs/undercloud/var/log/containers/nova/nova-compute.log.txt.gz#_2018-08-03_23_06_47_764 | 07:30 |
| *** Bhujay has quit IRC | 07:30 | |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Define irrelevant-files for tempest-full-py3 job https://review.openstack.org/589039 | 07:30 |
| alex_xu | quiquell: also zero node return by the ironic driver http://logs.openstack.org/95/583195/18/check/tripleo-ci-centos-7-containers-multinode/769edac/logs/undercloud/var/log/containers/nova/nova-compute.log.txt.gz#_2018-08-03_23_06_47_882 | 07:30 |
| alex_xu | quiquell: sorry, s/nova-maanger/nova-compute/ | 07:31 |
| alex_xu | quiquell: that means nova-compute doesn't reach to the code to create that CUSTOM_BARETAL for the ironic virt driver | 07:31 |
| alex_xu | quiquell: also sorry, CUSTOM_BARETAL is custom resource class, it is not custom trait....really typo too much wrong words... | 07:32 |
| quiquell | alex_xu: Do we have any log about virt driver to check why it returns 0 ? | 07:32 |
| alex_xu | quiquell: i guess there isn't too much, probably needs to check ironic log to see whether ironic works correctly | 07:34 |
| *** Bhujay has joined #openstack-nova | 07:39 | |
| *** jpenag is now known as jpena | 07:40 | |
| *** tssurya has joined #openstack-nova | 07:40 | |
| gmann | giblet: should we mark these bug fixed/invalid now (fixed by zuul) - https://bugs.launchpad.net/nova/+bug/1745405 https://bugs.launchpad.net/nova/+bug/1745431 | 07:43 |
| openstack | Launchpad bug 1745405 in OpenStack Compute (nova) "tempest-full job triggered for irrelevant changes" [Undecided,In progress] - Assigned to Balazs Gibizer (balazs-gibizer) | 07:43 |
| openstack | Launchpad bug 1745431 in OpenStack Compute (nova) "neutron-grenade job is triggered for irrelevant changes " [Undecided,In progress] - Assigned to Balazs Gibizer (balazs-gibizer) | 07:43 |
| *** Kvisle is now known as Kvisle_ | 07:44 | |
| *** avolkov has joined #openstack-nova | 07:44 | |
| *** dpawlik has joined #openstack-nova | 07:50 | |
| *** rpittau has joined #openstack-nova | 07:53 | |
| *** Bhujay has quit IRC | 07:54 | |
| *** dtantsur|afk is now known as dtantsur | 07:59 | |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Add tempest-slow job to run the tempest slow tests https://review.openstack.org/567697 | 07:59 |
| *** cdent has joined #openstack-nova | 08:00 | |
| *** tetsuro_ has joined #openstack-nova | 08:10 | |
| *** jaosorior has joined #openstack-nova | 08:21 | |
| *** Dinesh_Bhor has quit IRC | 08:22 | |
| *** Bhujay has joined #openstack-nova | 08:45 | |
| *** cdent has quit IRC | 08:45 | |
| *** Dinesh_Bhor has joined #openstack-nova | 08:58 | |
| *** maciejjozefczyk has quit IRC | 09:04 | |
| *** amarao has joined #openstack-nova | 09:06 | |
| *** mdbooth has joined #openstack-nova | 09:09 | |
| *** maciejjozefczyk has joined #openstack-nova | 09:15 | |
| openstackgerrit | Sylvain Bauza proposed openstack/nova master: Pass allocations to virt drivers when resizing https://review.openstack.org/589085 | 09:18 |
| *** panda is now known as panda|rover|off | 09:25 | |
| *** maciejjozefczyk has quit IRC | 09:29 | |
| *** adrianc_ has quit IRC | 09:36 | |
| *** maciejjozefczyk has joined #openstack-nova | 09:36 | |
| openstackgerrit | Surya Seetharaman proposed openstack/nova master: Cleanup comp_node, res_prov, services, aggregate_hosts during cell deletion https://review.openstack.org/546660 | 09:40 |
| *** vishakha_ is now known as vishakha | 09:40 | |
| *** tetsuro has joined #openstack-nova | 09:43 | |
| *** sahid has joined #openstack-nova | 09:47 | |
| *** kaliya has joined #openstack-nova | 09:54 | |
| *** vivsoni has quit IRC | 10:00 | |
| *** giblet is now known as gibi | 10:04 | |
| *** liuyulong has quit IRC | 10:05 | |
| *** cdent has joined #openstack-nova | 10:06 | |
| cdent | gibi: another good thing to get in: https://review.openstack.org/#/c/587772/ | 10:06 |
| *** do3meli has joined #openstack-nova | 10:10 | |
| gibi | gmann: marked both bug invalid in nova. Thanks for the notice | 10:10 |
| gibi | cdent: looking... | 10:10 |
| *** do3meli has left #openstack-nova | 10:11 | |
| *** frankwang has quit IRC | 10:15 | |
| openstackgerrit | Lee Yarwood proposed openstack/nova master: fixtures: Track volume attachments within CinderFixtureNewAttachFlow https://review.openstack.org/587013 | 10:18 |
| openstackgerrit | Lee Yarwood proposed openstack/nova master: Add regression test for bug#1784353 https://review.openstack.org/587014 | 10:19 |
| openstackgerrit | Lee Yarwood proposed openstack/nova master: compute: Recreate volume attachments during a reschedule https://review.openstack.org/587071 | 10:19 |
| lyarwood | dansmith: morning, nothing urgent but could you take a look at ^ specifically a question raised by mriedem on a previous patchset - https://review.openstack.org/#/c/587071/4/nova/compute/manager.py | 10:19 |
| gibi | cdent: +2 | 10:19 |
| openstackgerrit | Lee Yarwood proposed openstack/nova master: compute: Recreate volume attachments during a reschedule https://review.openstack.org/587071 | 10:20 |
| cdent | thanks gibi | 10:22 |
| *** sambetts_ is now known as sambetts|afk | 10:24 | |
| gmann | gibi: thanks | 10:25 |
| *** chenyb4 has quit IRC | 10:27 | |
| *** luksky has quit IRC | 10:28 | |
| *** vivsoni has joined #openstack-nova | 10:29 | |
| *** hughsaunders has quit IRC | 10:32 | |
| *** hughsaunders has joined #openstack-nova | 10:33 | |
| *** udesale has quit IRC | 10:34 | |
| *** adrianc has joined #openstack-nova | 10:36 | |
| *** Dinesh_Bhor has quit IRC | 10:46 | |
| *** dave-mccowan has joined #openstack-nova | 10:47 | |
| *** sahid has quit IRC | 10:55 | |
| *** tetsuro_ has quit IRC | 10:56 | |
| *** tetsuro_ has joined #openstack-nova | 10:56 | |
| *** luksky has joined #openstack-nova | 11:02 | |
| *** dave-mccowan has quit IRC | 11:03 | |
| *** cdent has quit IRC | 11:09 | |
| *** gvrangan has quit IRC | 11:09 | |
| *** jpena is now known as jpena|lunch | 11:14 | |
| *** tetsuro has quit IRC | 11:19 | |
| *** cdent has joined #openstack-nova | 11:23 | |
| *** leakypipes is now known as jaypipes | 11:28 | |
| *** Dinesh_Bhor has joined #openstack-nova | 11:32 | |
| *** Dinesh_Bhor has quit IRC | 11:32 | |
| *** slagle has joined #openstack-nova | 11:49 | |
| *** mardim has joined #openstack-nova | 11:59 | |
| mardim | hello guys | 11:59 |
| *** Kevin_Zheng has joined #openstack-nova | 11:59 | |
| mardim | I have one question | 11:59 |
| mardim | I have this libvirt xml cpu topology | 12:00 |
| mardim | <cpu mode='host-passthrough' check='none'> | 12:00 |
| mardim | <topology sockets='1' cores='2' threads='2'/> | 12:00 |
| mardim | </cpu> | 12:00 |
| mardim | Then I tried to spinup a cirros instance with 2 vcpus and hw:cpu_policy='dedicated', hw:cpu_thread_policy='isolate' | 12:01 |
| mardim | But the instance is in error state because I get no valid host was found | 12:01 |
| mardim | Apparently scheduler cannot pin and isolate the processes of the instance | 12:01 |
| mardim | but I do not know why | 12:01 |
| mardim | here are some scheduler logs which might help | 12:02 |
| mardim | cpu_usage": 4, "memory_usage": 64, "cpuset": [0, 1, 2, 3], "pinned_cpus": [0, 1], "siblings": [[0, 1], [2, 3]], | 12:02 |
| *** cdent has quit IRC | 12:02 | |
| mardim | do you have any clue what is the problem ? | 12:03 |
| mardim | Thanks !!! | 12:03 |
| *** ratailor has quit IRC | 12:04 | |
| *** gbarros has joined #openstack-nova | 12:15 | |
| *** jpena|lunch is now known as jpena | 12:18 | |
| *** vivsoni has quit IRC | 12:21 | |
| *** vivsoni has joined #openstack-nova | 12:22 | |
| *** vivsoni has quit IRC | 12:23 | |
| *** cdent has joined #openstack-nova | 12:23 | |
| *** vivsoni_ has joined #openstack-nova | 12:23 | |
| *** tbachman_ has joined #openstack-nova | 12:31 | |
| *** quiquell is now known as quiquell|lunch | 12:32 | |
| *** tbachman has quit IRC | 12:33 | |
| *** tbachman_ is now known as tbachman | 12:33 | |
| *** Luzi has quit IRC | 12:34 | |
| *** dtantsur is now known as dtantsur|brb | 12:36 | |
| openstackgerrit | Lee Yarwood proposed openstack/nova master: fixtures: Track volume attachments within CinderFixtureNewAttachFlow https://review.openstack.org/587013 | 12:38 |
| openstackgerrit | Lee Yarwood proposed openstack/nova master: Add regression test for bug#1784353 https://review.openstack.org/587014 | 12:38 |
| openstackgerrit | Lee Yarwood proposed openstack/nova master: compute: Recreate volume attachments during a reschedule https://review.openstack.org/587071 | 12:38 |
| *** cdent has quit IRC | 12:39 | |
| *** Sigyn has quit IRC | 12:41 | |
| *** Sigyn has joined #openstack-nova | 12:42 | |
| *** cdent has joined #openstack-nova | 12:43 | |
| *** Luzi has joined #openstack-nova | 12:49 | |
| *** dave-mccowan has joined #openstack-nova | 12:53 | |
| *** mchlumsky has joined #openstack-nova | 12:56 | |
| *** quiquell|lunch is now known as quiquell|off | 12:57 | |
| *** Luzi has quit IRC | 12:58 | |
| *** panda|rover|off is now known as panda|rover-ish | 12:58 | |
| *** eharney has joined #openstack-nova | 13:00 | |
| *** quiquell|off has quit IRC | 13:01 | |
| *** dave-mccowan has quit IRC | 13:03 | |
| *** mchlumsky has quit IRC | 13:04 | |
| *** mchlumsky has joined #openstack-nova | 13:06 | |
| mdbooth | I'm trying to make 'python setup.py test' in the osc-placement repo exclude functional tests. Does anybody have any ideas where to start looking? | 13:10 |
| mdbooth | It seems to be pbr voodoo, so... any advice short of rtfs appreciated :) | 13:11 |
| * mdbooth hasn't found any obvious documentation | 13:12 | |
| *** Luzi has joined #openstack-nova | 13:13 | |
| *** edmondsw has joined #openstack-nova | 13:13 | |
| cdent | mdbooth: sorry, I've got no clues. I've not been much involved in osc-placement's birth | 13:16 |
| efried | mdbooth: --test-path=... | 13:16 |
| mdbooth | efried: Is that passed to testr? | 13:17 |
| mdbooth | https://docs.openstack.org/pbr/latest/user/using.html#testing This suggests it might be tox.ini, although there's a deprecation notice | 13:17 |
| efried | mdbooth: have we not switched osc-placement to stestr yet? | 13:18 |
| mdbooth | efried: pbr is the darkest most arcane magik | 13:18 |
| mdbooth | I have no idea :) | 13:19 |
| cdent | are you building rpms mdbooth, and thus forced through setup.py? | 13:19 |
| *** jroll has quit IRC | 13:19 | |
| mdbooth | cdent: Yes | 13:19 |
| efried | mdbooth: I would run to mtreinish if it were me. | 13:19 |
| * cdent gives mdbooth a hug | 13:19 | |
| mdbooth | Well not *forced*. It's just a script, I can run whatever I want. | 13:19 |
| *** jroll has joined #openstack-nova | 13:19 | |
| mdbooth | But everything else runs setup.py test, so I'd prefer to make that work for consistency | 13:19 |
| cdent | you might also try efried's suggestion with a --<space> before the --test-path | 13:19 |
| mdbooth | What is the argument to --test-path? | 13:20 |
| efried | not having the repo in front of me, I would guess ./osc_placement/tests/unit | 13:20 |
| * cdent nods | 13:21 | |
| mdbooth | efried: Ah, ok. | 13:21 |
| *** janki has quit IRC | 13:24 | |
| mdbooth | Eurgh... It doesn't work and I'm way too hot and I just want to lie in an ice bucket and weep until it all goes away. | 13:24 |
| *** Bhujay has quit IRC | 13:31 | |
| *** Luzi has quit IRC | 13:34 | |
| *** takashin has joined #openstack-nova | 13:36 | |
| mdbooth | So I have a behaviour difference here between the CI environment and locally. CI runs 'python2 setup.py test' and it does All The Things(tm). Locally it does nothing: http://paste.openstack.org/show/727412/ | 13:40 |
| *** mriedem has joined #openstack-nova | 13:41 | |
| mdbooth | CI: (search for 'python2 setyp.py test') https://logs.rdoproject.org/81/15181/3/check/legacy-rdoinfo-DLRN-check/83a6ba9/buildset/centos-rpm-master/repos/95/77/9577cd899541b4c2a5b9fa74a59f4c346bd5addf_dev/rpmbuild.log | 13:41 |
| *** MasterofJOKers has quit IRC | 13:41 | |
| mdbooth | Any idea what environment factors affect setup.py test? | 13:42 |
| *** MasterofJOKers has joined #openstack-nova | 13:43 | |
| *** lbragstad has joined #openstack-nova | 13:43 | |
| efried | n-sch meeting in 10 minutes in #openstack-meeting-alt | 13:50 |
| dansmith | lyarwood: mriedem I left comments on that review.. definitely seems like obligatory pings to cinder for every build is a less-than-awesome change, and would rather see it be something we do on reschedule if we need | 13:51 |
| dansmith | also, queried about whether or not the GET is enough. I guess I would expect there are some attachment states that can't be reversed blindly by the boot process? | 13:51 |
| lyarwood | dansmith: ack thanks | 13:52 |
| openstackgerrit | sahid proposed openstack/nova master: hardware: fix memory check usage for small/large pages https://review.openstack.org/532168 | 13:55 |
| openstackgerrit | Lee Yarwood proposed openstack/nova master: DNM/WIP compute: Reduce likelihood of bdm creation race during attach https://review.openstack.org/589164 | 13:56 |
| mriedem | dansmith: lyarwood: that's where i'm leaning - re-create attachments in conductor build_instances if we're rescheduling | 13:59 |
| dansmith | yeah | 13:59 |
| dansmith | t'would make more sense to me to do that | 14:00 |
| lyarwood | mriedem / dansmith ; kk, tbh I've spent very little time looking around within conductor and just assumed it wouldn't be the right place, I'll try to move things over now and respin. | 14:00 |
| mdbooth | OS_TEST_PATH | 14:04 |
| efried | mdbooth: \o/ nice one | 14:07 |
| mdbooth | Although I can't see why rpmbuild isn't using the default | 14:08 |
| *** spotz has joined #openstack-nova | 14:08 | |
| * mriedem continues the starlingx diff dive | 14:14 | |
| mdbooth | mriedem: Wow, brave. | 14:14 |
| mriedem | already mostly done | 14:15 |
| mdbooth | Any good nuggets? | 14:15 |
| *** efried1 has joined #openstack-nova | 14:16 | |
| *** efried has quit IRC | 14:16 | |
| *** efried1 is now known as efried | 14:16 | |
| mriedem | mdbooth: i'll post a summary with a link to my spreadsheet in the ML once i'm done | 14:16 |
| mriedem | there are definitely some things we can upstream to nova | 14:16 |
| mriedem | given it was a snapshot based on pike, there are also several bug fixes they have backported so not really forks, just cherry picks we don't have in stable/pike | 14:17 |
| mriedem | and other bug fixes that weren't reported upstream | 14:17 |
| mriedem | e.g. https://review.openstack.org/#/c/588689/ | 14:17 |
| *** Bhujay has joined #openstack-nova | 14:18 | |
| mriedem | and https://review.openstack.org/#/c/588657/ | 14:18 |
| mdbooth | Nice | 14:18 |
| mdbooth | Ah, so you do at least have a git repo | 14:18 |
| mdbooth | You're not just staring at a massive unified diff | 14:18 |
| mriedem | i am | 14:18 |
| mriedem | https://github.com/starlingx-staging/stx-nova/commit/71acfeae0d1c59fdc77704527d763bd85a276f9a | 14:18 |
| mriedem | oh yeah i do have the actual git repo cloned | 14:19 |
| mriedem | so i can search things for context | 14:19 |
| mriedem | but i'm mostly starting with the diff | 14:19 |
| mriedem | i'm also starting to glaze over things toward the end here out of sheaer fatigue | 14:20 |
| mriedem | *sheer | 14:20 |
| *** kklimonda_ has joined #openstack-nova | 14:23 | |
| *** Tahvok_ has joined #openstack-nova | 14:23 | |
| *** tetsuro_ has quit IRC | 14:25 | |
| *** kklimonda has quit IRC | 14:26 | |
| *** Tahvok has quit IRC | 14:26 | |
| *** baffle has quit IRC | 14:26 | |
| *** colby_ has quit IRC | 14:26 | |
| *** szaher has quit IRC | 14:26 | |
| *** bjolo has quit IRC | 14:26 | |
| *** dtantsur|brb has quit IRC | 14:26 | |
| *** Tahvok_ is now known as Tahvok | 14:26 | |
| *** kklimonda_ is now known as kklimonda | 14:26 | |
| gibi | dansmith: hi! I'm looking at some migation revert functional test case and it seems we are still hitting the legacy allocation handling code path when reverting the allocation on the destination | 14:26 |
| gibi | dansmith: I think it is a bug in https://github.com/openstack/nova/blob/8688b25ca7379391cba28fab30b5a628957e673e/nova/compute/manager.py#L3952-L3955 | 14:27 |
| gibi | dansmith: cn_uuid points to the destinantion host based but the migration.uuid only supposed to hold allocation on the source host | 14:27 |
| gibi | dansmith: So the if condition never be true | 14:27 |
| *** eharney has quit IRC | 14:28 | |
| *** dtantsur has joined #openstack-nova | 14:28 | |
| gibi | dansmith: Did I missed something or it is really a bug? | 14:28 |
| *** Bhujay has quit IRC | 14:29 | |
| *** naichuans has quit IRC | 14:29 | |
| dansmith | gibi: there's something tricky about this, let me re-load context | 14:30 |
| *** dpawlik has quit IRC | 14:31 | |
| openstackgerrit | Lee Yarwood proposed openstack/nova master: conductor: Recreate volume attachments during a reschedule https://review.openstack.org/587071 | 14:37 |
| dansmith | gibi: this is the context I was thinking of, although reading that review comment now doesn't seem to make sense: https://review.openstack.org/#/c/498948/9/nova/compute/manager.py@3648 | 14:40 |
| dansmith | gibi: however, I rarely add code like this without some test fail making me do it, | 14:41 |
| dansmith | so I'm not sure | 14:41 |
| dansmith | maybe raise there and run all the test_servers tests to see if something fails? | 14:41 |
| dansmith | it's possible that it was there for some case which has since been removed, but.. worth a shot | 14:43 |
| dansmith | I was thinking that we actually call that on both the source and destination | 14:43 |
| dansmith | because of finish_resize_revert_on_destination(), | 14:43 |
| dansmith | but i don't see it in there | 14:44 |
| gibi | dansmith: I will check resize-same-host sceneario. a simple migrate revert func test now goes to the legacy path | 14:44 |
| dansmith | so I wonder if it was at some point | 14:44 |
| dansmith | gibi: we have those tests in tree though right? | 14:44 |
| dansmith | to verify the migration-holding allocations? | 14:44 |
| gibi | dansmith: we have functional tests | 14:44 |
| gibi | dansmith: for same host resize and migrate too | 14:44 |
| dansmith | gibi: right, which validate the non-legacy path works yeah? | 14:45 |
| gibi | dansmith: which validates that allocations handled properly in non-legacy case. But I think those tests are still hitting the legacy path | 14:45 |
| *** cdent has quit IRC | 14:45 | |
| gibi | let me reproduce both with migrate and same host resize | 14:46 |
| *** gbarros has quit IRC | 14:46 | |
| dansmith | hmm | 14:46 |
| dansmith | so the tests aren't noticing that we're leaking an allocation or something? | 14:46 |
| gibi | dansmith: I think we are eventually not leaking as the legacy codepath also handles the alloction properly | 14:47 |
| dansmith | okay I'm not sure how that could be, if it's not handling the migration uuid, but I'll wait for your analysis :) | 14:48 |
| gibi | dansmith: https://github.com/openstack/nova/blob/8688b25ca7379391cba28fab30b5a628957e673e/nova/compute/manager.py#L3982 this will remove the allocation on the destination | 14:48 |
| dansmith | on the dest, yeah, but the source? | 14:48 |
| openstackgerrit | Merged openstack/nova master: Reload oslo_context after calling monkey_patch() https://review.openstack.org/587772 | 14:48 |
| gibi | dansmith: yeah, I'm confused now. I will do the reproduction | 14:49 |
| mriedem | the allocation on the source should get moved by conductor right? or is this for the "old computes" scenario? | 14:50 |
| dansmith | mriedem: he thinks we're taking the old doubled patch during some cleanups | 14:51 |
| dansmith | but I'm not sure how we could, without failing those tests (or having big holes in them | 14:51 |
| dansmith | well, and his point is there is a clause in the delete_after_move function we can't explain | 14:51 |
| dansmith | which he thinks might be related | 14:52 |
| *** gyee has joined #openstack-nova | 14:52 | |
| dansmith | gibi: I added an exception to that clause and we _do_ hit it in the functional tests | 14:56 |
| dansmith | so that _is_ true somewhere :) | 14:56 |
| dansmith | during revert resize | 14:56 |
| dansmith | in three tests | 14:57 |
| *** dave-mccowan has joined #openstack-nova | 14:57 | |
| dansmith | test_migrate_revert, test_resize_revert, and test_resize_revert_reverse in ServerMovingTests | 14:57 |
| *** moshele has quit IRC | 15:03 | |
| *** ircuser-1 has joined #openstack-nova | 15:04 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Define irrelevant-files for tempest-full-py3 job https://review.openstack.org/589039 | 15:05 |
| *** luksky has quit IRC | 15:05 | |
| gibi | dansmith: did you put the raise in the legacy path? | 15:06 |
| dansmith | gibi: no, in the clause you think can't ever be true | 15:06 |
| dansmith | gibi: this: https://pastebin.com/GC17gjG4 | 15:07 |
| gibi | dansmith: I put it in front of the legacy code and hit the same tests you hit | 15:08 |
| gibi | dansmith: it doesn't make any sense | 15:09 |
| *** amarao has quit IRC | 15:09 | |
| dansmith | hrm. | 15:09 |
| dansmith | oh, well, | 15:09 |
| dansmith | that can happen in the legit case, | 15:09 |
| dansmith | because we're just checking for allocs there. if we find none, we will fall through to the legacy path as expected | 15:10 |
| *** eharney has joined #openstack-nova | 15:10 | |
| gibi | dansmith: ahh. When I question the existence of an if condition above it was the if allocs | 15:10 |
| dansmith | oh | 15:11 |
| gibi | as we are querying alloctions held by the migration.uuid on the _dest_ host | 15:11 |
| gibi | but that is always empty | 15:11 |
| gibi | as migration holds allocation on the source host only | 15:11 |
| dansmith | right right | 15:12 |
| dansmith | except for same-host, but that should never hit this because of the source check above | 15:12 |
| *** pcaruana has quit IRC | 15:12 | |
| dansmith | I'll move the exception and re-run to confirm | 15:12 |
| gibi | I did that and got green results | 15:12 |
| gibi | so my theory is that in case of revert on dest we always see empty allocs and don't return so we hit the legacy path below | 15:13 |
| dansmith | okay | 15:14 |
| dansmith | hmm, which I guess isn't a problem, | 15:14 |
| dansmith | if we're deleting dest allocs and restoring source allocs, | 15:15 |
| dansmith | except for the case of if we miss the migration allocs we need to delete | 15:15 |
| gibi | it seams it doesn't casue a leak, but I have to leak a bit more closer | 15:15 |
| dansmith | gibi: still not sure where this is coming from though -- are you chasing a bug or writing a test or what? | 15:15 |
| gibi | dansmith: trying implement placement 1.28 support in report client (consumer_generation) and writing consumer gen conflict tests and I hit this legacy path that I thought I should not hit any more | 15:16 |
| dansmith | okay | 15:16 |
| dansmith | we should be able to clean all this up at this point anyway I think | 15:17 |
| dansmith | perhaps we should make a point of ripping this all out for stein? | 15:17 |
| *** rmart04 has quit IRC | 15:17 | |
| *** cdent has joined #openstack-nova | 15:17 | |
| dansmith | anyone with an unconfirmed migration between pike and stein probably has other problems :) | 15:17 |
| *** janki has joined #openstack-nova | 15:17 | |
| *** eharney has quit IRC | 15:18 | |
| gibi | dansmith: to avoid hitting the legacy and later rip it out I need to change https://github.com/openstack/nova/blob/8688b25ca7379391cba28fab30b5a628957e673e/nova/compute/manager.py#L3952-L3955 to query the migration allocation from the source host. Does that make sense to you? | 15:19 |
| dansmith | well.. | 15:19 |
| dansmith | the point of that is to make the destination node not do anything at all (i.e. not run the legacy path) if the new-style allocations were used | 15:20 |
| dansmith | so, I guess, but you'll have to do a lookup of the source uuid I think in order to use that method | 15:21 |
| gibi | dansmith: yes that is the goal. if that code sees the allocation held by the migration on the source host it returns and let the finish_revert_resize do the work | 15:21 |
| dansmith | which is a little bit icky | 15:21 |
| *** eharney has joined #openstack-nova | 15:21 | |
| *** hoonetorg has quit IRC | 15:22 | |
| gibi | dansmith: ohh. Then we just assume there is no legacy migration any more and unconditioanlly not doing anython on the dest host | 15:22 |
| dansmith | gibi: ah, just change that call to get_allocations_by_consumer() and if the migration has any allocations, then we must be doing new-style | 15:22 |
| gibi | dansmith: good point, we can do that | 15:22 |
| gibi | then I will file a bug and this small change as a bugfix. then later when Stein is open I can remove the whole legacy path in a separate patch | 15:23 |
| dansmith | yar | 15:23 |
| *** ebbex has joined #openstack-nova | 15:23 | |
| gibi | dansmith: thanks for the brainpower | 15:23 |
| dansmith | np :) | 15:24 |
| *** jiapei has joined #openstack-nova | 15:26 | |
| mdbooth | Incidentally, my pbr test problem was that I wasn't explicitly installing python-testrepository. This resulted in <unfathomable pbr-related weirdness> ultimately resulting in the tests attempting to run functional rather than just unit. | 15:30 |
| * mdbooth would have preferred a failure with a message that python-testrepository wasn't installed. | 15:31 | |
| efried | jaypipes, mriedem, dansmith: What is it going to take to have sufficient confidence in any of the various attempts to reduce the number of redundant placement calls per periodic to actually merge code? | 15:35 |
| efried | e.g. these two are quite similar | 15:36 |
| efried | https://review.openstack.org/#/c/588091/ (removes the "first" _update) | 15:36 |
| efried | https://review.openstack.org/#/c/587050/ (removes the "last" _update) | 15:36 |
| efried | and are both showing green across the board test-wise. | 15:36 |
| jaypipes | efried: which one do you prefer? | 15:37 |
| efried | jaypipes: It makes little difference to me. The first one removes more lines of code. At the moment it's sitting on top of the reshaper series, but could be extracted easily. | 15:38 |
| mriedem | idk | 15:42 |
| mriedem | stuff like that likely won't show a side effect until we've merged it and run it awhile and found some weird failures, | 15:42 |
| mriedem | which will likely be non-trivial to debug | 15:42 |
| efried | just so. | 15:42 |
| mriedem | so definitely not rocky | 15:42 |
| *** jiapei_ has joined #openstack-nova | 15:42 | |
| *** jiapei has left #openstack-nova | 15:43 | |
| mriedem | and i'd expect what you'd see are weird scheduling failures | 15:43 |
| mriedem | due to some timing issue | 15:43 |
| dansmith | and maybe not at all at gate-level scale | 15:43 |
| mriedem | would be nice to have some company with a stress test lab kick around either of those to see what falls out | 15:44 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Delete instance_group_member records from API DB during archive https://review.openstack.org/588943 | 15:44 |
| mriedem | but i'm pretty sure at this point in my time working on openstack, no company has a stress test lab that shares results publicly :) | 15:45 |
| mriedem | if we don't see it in the gate, | 15:45 |
| mriedem | we'll see it 18-24 months from now when someone actually upgrades to use it and finds problems | 15:45 |
| efried | right | 15:45 |
| jaypipes | mriedem: 36-64 months from now. | 15:46 |
| mriedem | well, cern would be the first to hit it probably | 15:46 |
| efried | But even that is a big "maybe". | 15:46 |
| mriedem | i also haven't had the time to read cdent's write up on this either yet | 15:46 |
| efried | mriedem: TL;DR we're calling _update twice every periodic. | 15:47 |
| mriedem | yeah i knew that much :) | 15:47 |
| efried | which calls all the placement things to refresh the cache etc. | 15:47 |
| efried | that's really all there is to it. | 15:47 |
| cdent | mriedem: there's not much more than that what efried says, must more words to indicate the code path | 15:47 |
| efried | Do we need both calls for some reason? | 15:47 |
| mriedem | also https://review.openstack.org/#/c/520024/ | 15:48 |
| *** jaosorior has quit IRC | 15:48 | |
| mriedem | given ^ you should probably see if the ovh gang is interested in testing out either of those changes | 15:48 |
| mriedem | when did we start doing the 2 calls? | 15:48 |
| mriedem | b/c ovh might not have a region running that yet | 15:48 |
| cdent | >18 months ago | 15:49 |
| cdent | it was there when I did the first version of the post | 15:49 |
| *** jaosorior has joined #openstack-nova | 15:49 | |
| efried | oh look, https://review.openstack.org/#/c/520024 is exactly the same as https://review.openstack.org/#/c/588091/ | 15:50 |
| mriedem | looking at ^ from ovh they opted to leave in the _update call at the end | 15:50 |
| mriedem | the +1 from minho on that was also because they had done a duplicate of the same patch | 15:50 |
| openstackgerrit | Eric Fried proposed openstack/nova master: Update resources once in update_available_resource https://review.openstack.org/520024 | 15:51 |
| *** adrianc has quit IRC | 15:52 | |
| mriedem | ok i have voted | 15:52 |
| openstackgerrit | Chen proposed openstack/nova master: Trivial fix on migration doc https://review.openstack.org/589028 | 15:52 |
| mriedem | if it's a coin toss, go with what ovh is already using | 15:52 |
| mriedem | and drop the duplicate change from efried and coalesce on the ovh patch | 15:53 |
| efried | Note that there's also https://review.openstack.org/#/c/588094/ which is failing tox (expected because I haven't updated those tests yet) but also live migration consistently. | 15:53 |
| efried | mriedem: The coalesce is a no-op; the extras in mine are only there because of the reshaper series. I've abandoned it. | 15:53 |
| mriedem | maciejjozefczyk: are you guys running with this in production now? https://review.openstack.org/#/c/520024/ | 15:54 |
| mriedem | maciejjozefczyk: any side effects or issues with that patch? | 15:54 |
| *** dklyle has joined #openstack-nova | 15:57 | |
| *** efried has quit IRC | 15:58 | |
| mriedem | jaypipes: has cfriesen talked with you about how they account for hosting shared and pinned cpus on the same host by making VCPU inventory a fraction? | 16:00 |
| mriedem | i assume it would have come up during https://review.openstack.org/#/c/555081/ | 16:00 |
| jaypipes | mriedem: nope. | 16:01 |
| jaypipes | mriedem: and I'm 100% against making amount a non-integer value. | 16:01 |
| cdent | It was briefly glossed over in Dublin, but the response then was "placement can't do that" | 16:03 |
| cdent | or maybe s/can't/won't/ | 16:04 |
| dansmith | I'm also 100% against that :) | 16:04 |
| mriedem | i'm not suggesting we do that, | 16:04 |
| mriedem | but i thought there was an alternative way to model it via nested providers, or some other kind of inventory | 16:04 |
| mriedem | PCPUs? | 16:04 |
| mriedem | looks like that is the proposal in the spec | 16:05 |
| dansmith | we talked about making dedicated cpus a different inventory item in placement | 16:05 |
| dansmith | if that's what youmean | 16:05 |
| mriedem | yes | 16:05 |
| mriedem | there is just a shit load of code in starlingx to deal with shared and pinned cpus on the same host, | 16:06 |
| mriedem | and i'm mostly at the point of glossing over all of it and just saying, we have a spec for this | 16:06 |
| mriedem | at the end of my diff dive i want to be able to give a sort of tl;dr on the major changes | 16:07 |
| mriedem | which at this point is i think just shared/pinned on same host, live resize (cpu only), and l3 cache partitioning | 16:07 |
| mriedem | there are a lot of other things, but those are the big ones that affect the entire stack | 16:07 |
| mriedem | if you're all lucky, i might even make some m'fing charts! | 16:08 |
| *** tssurya has quit IRC | 16:09 | |
| *** efried has joined #openstack-nova | 16:10 | |
| *** jpena is now known as jpena|off | 16:12 | |
| *** janki has quit IRC | 16:15 | |
| mriedem | surprisingly i don't see the tpm stuff in her | 16:17 |
| mriedem | *here | 16:18 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: api-ref: Add descriptions for rebuild https://review.openstack.org/588931 | 16:19 |
| *** takashin has left #openstack-nova | 16:20 | |
| mdbooth | lyarwood: https://review.openstack.org/#/c/587071/ I think that's the wrong build | 16:21 |
| mdbooth | lyarwood: I won't be bowled over with surprise if I'm wrong, though. | 16:22 |
| *** kaliya has quit IRC | 16:27 | |
| mdbooth | lyarwood: Hmm, looks like compute calls build_instances directly on reschedule? So... I could be wrong. | 16:29 |
| *** imacdonn has quit IRC | 16:38 | |
| *** imacdonn has joined #openstack-nova | 16:38 | |
| *** betherly_ is now known as betherly-afk | 16:40 | |
| *** SamYaple has joined #openstack-nova | 16:44 | |
| mdbooth | mriedem: I assume it's not possible to 'demote' an attachment to a reservation? | 16:46 |
| mriedem | first, that sounds borderline genocidal | 16:50 |
| mriedem | second, i don't know what that means | 16:50 |
| mriedem | an empty volume attachment reserves the volume | 16:50 |
| mriedem | actually i think even a volume attachment that has a host connector is still not considering the volume as in-use, | 16:50 |
| mriedem | that's why we have to call the 'complete' action on the attachment | 16:51 |
| *** evrardjp has quit IRC | 16:51 | |
| mriedem | to make the volume to in-use | 16:51 |
| mdbooth | mriedem: Right. I'm just thinking of lyarwood's bug. The issue, IIUC, is that we create a 'reservation', which is an empty attachment, then the compute turns it into a real attachment, right? | 16:51 |
| mdbooth | And the only way to get rid of that is to delete it. | 16:51 |
| mriedem | depends on where we fail, | 16:51 |
| mriedem | if we didn't get to the point of calling the complete action on the attachment, the volume is not in-use | 16:52 |
| mdbooth | Sure, but if it's after we hydrated the attachment, this is how to rollback. | 16:52 |
| *** hongbin has joined #openstack-nova | 16:52 | |
| openstackgerrit | Eric Fried proposed openstack/nova master: Make get_allocations_for_resource_provider sane https://review.openstack.org/584598 | 16:52 |
| openstackgerrit | Eric Fried proposed openstack/nova master: Report client: Real get_allocs_for_consumer https://review.openstack.org/584599 | 16:52 |
| openstackgerrit | Eric Fried proposed openstack/nova master: Report client: get_allocations_for_provider_tree https://review.openstack.org/584648 | 16:52 |
| openstackgerrit | Eric Fried proposed openstack/nova master: Report client: _reshape helper, placement min bump https://review.openstack.org/585034 | 16:52 |
| openstackgerrit | Eric Fried proposed openstack/nova master: Report client: update_from_provider_tree w/reshape https://review.openstack.org/585049 | 16:52 |
| openstackgerrit | Eric Fried proposed openstack/nova master: Compute: Handle reshaped provider trees https://review.openstack.org/576236 | 16:52 |
| mriedem | i believe i talked to myself at length on an earlier PS on the review about how we could just simply not delete the attachment before rescheduling | 16:52 |
| mdbooth | I'm pretty sure the answer's no, but I'm just wondering if there's any direct opposite to the compute's action which would cause the attachment to go back to being just a 'reservation' | 16:53 |
| mdbooth | Ok, sounds like you already did that dance. | 16:53 |
| mriedem | https://review.openstack.org/#/c/587071/3/nova/compute/manager.py@1631 | 16:54 |
| mriedem | tl;dr is i think this is no worse than what happened during reschedule *before* the attachments stuff, | 16:55 |
| mriedem | we could go either way and there are pros/cons both ways | 16:55 |
| mriedem | e.g. if compute doesn't delete the attachment, conductor would have to when we exhausted retries | 16:56 |
| *** gbarros has joined #openstack-nova | 16:57 | |
| mdbooth | mriedem: Nice writeup, thanks. | 16:58 |
| *** luksky has joined #openstack-nova | 16:59 | |
| mriedem | if anyone knows about or cares about uefi instances, this might be a bug fix we need in nova https://github.com/starlingx-staging/stx-nova/commit/71acfeae0d1c59fdc77704527d763bd85a276f9a#diff-f4019782d93a196a0d026479e6aa61b1R8514 | 17:00 |
| mriedem | ah looky here https://bugs.launchpad.net/nova/+bug/1785123 | 17:00 |
| openstack | Launchpad bug 1785123 in OpenStack Compute (nova) "UEFI NVRAM lost on cold migration or resize" [Undecided,New] | 17:00 |
| *** hemna_ has joined #openstack-nova | 17:06 | |
| *** rmart04 has joined #openstack-nova | 17:11 | |
| melwitt | . | 17:23 |
| mdbooth | mriedem: I wonder if live migration handles that. | 17:24 |
| mdbooth | mriedem: Pretty sure we care, btw. | 17:24 |
| *** tssurya has joined #openstack-nova | 17:30 | |
| *** evrardjp has joined #openstack-nova | 17:41 | |
| *** vivsoni_ has quit IRC | 17:44 | |
| *** vivsoni_ has joined #openstack-nova | 17:44 | |
| *** psachin has quit IRC | 17:45 | |
| *** moshele has joined #openstack-nova | 17:45 | |
| *** jiapei_ has quit IRC | 17:51 | |
| mriedem | mdbooth: they also have lvm thin pools; i looked up the patch to add that to nova and you nacked it saying we already have sparse lvms, | 17:53 |
| mriedem | but sparse_logical_volumes was deprecated in rocky | 17:53 |
| * mriedem makes trombone sound | 17:53 | |
| *** dtantsur is now known as dtantsur|afk | 17:53 | |
| mriedem | they have quite a bit of support for lvm, not sure if that's just for their 1-2 node configs, and then use ceph for their 100 node deployment or what | 17:55 |
| mriedem | and i'm done with the diff \o/ | 17:57 |
| penick | I saw the diff and backed away | 18:04 |
| * penick moonwalks away from the diff | 18:04 | |
| *** tbachman has quit IRC | 18:08 | |
| *** tbachman has joined #openstack-nova | 18:09 | |
| openstackgerrit | Radoslav Gerganov proposed openstack/nova stable/queens: Reload oslo_context after calling monkey_patch() https://review.openstack.org/589249 | 18:16 |
| openstackgerrit | Radoslav Gerganov proposed openstack/nova stable/queens: Reload oslo_context after calling monkey_patch() https://review.openstack.org/589249 | 18:17 |
| openstackgerrit | Radoslav Gerganov proposed openstack/nova stable/pike: Reload oslo_context after calling monkey_patch() https://review.openstack.org/589251 | 18:18 |
| *** moshele has quit IRC | 18:19 | |
| *** nicolasbock has quit IRC | 18:23 | |
| *** nicolasbock has joined #openstack-nova | 18:24 | |
| mriedem | penick: here you go https://docs.google.com/spreadsheets/d/1ugp1FVWMsu4x3KgrmPf7HGX8Mh1n80v-KVzweSDZunU/edit?usp=sharing | 18:32 |
| penick | damn, that's handy. Thanks! | 18:36 |
| mriedem | now i'll figure out how to digest that a bit for some simple charts | 18:38 |
| *** moshele has joined #openstack-nova | 18:49 | |
| *** gbarros has quit IRC | 18:51 | |
| *** gbarros has joined #openstack-nova | 18:54 | |
| *** moshele has quit IRC | 18:58 | |
| *** rmart04 has quit IRC | 18:59 | |
| *** hoonetorg has joined #openstack-nova | 19:17 | |
| *** _ix has joined #openstack-nova | 19:45 | |
| openstackgerrit | melanie witt proposed openstack/nova stable/ocata: [stable only] Add functional regression test for bug 1783613 https://review.openstack.org/588416 | 19:59 |
| openstack | bug 1783613 in OpenStack Compute (nova) ocata "[ocata only] quota usage not decremented during boot/delete race" [Undecided,In progress] https://launchpad.net/bugs/1783613 - Assigned to melanie witt (melwitt) | 19:59 |
| openstackgerrit | melanie witt proposed openstack/nova stable/ocata: [stable only] Handle quota usage during create/delete races https://review.openstack.org/582413 | 19:59 |
| *** dtroyer has joined #openstack-nova | 20:02 | |
| *** moshele has joined #openstack-nova | 20:05 | |
| dansmith | mriedem: I dunno when this ^ regressed exactly, but it seems like a minor enough fix to be worth getting into the older releases that suffer from it | 20:05 |
| dansmith | given where most people are, and to make sure anyone holding onto a non-counting-quotas release has a snowball's chance | 20:05 |
| mriedem | ok | 20:07 |
| openstackgerrit | Merged openstack/nova master: Increase max_unit in placement test fixture https://review.openstack.org/588158 | 20:21 |
| *** moshele has quit IRC | 20:29 | |
| *** rtjure has joined #openstack-nova | 20:31 | |
| *** tbachman_ has joined #openstack-nova | 20:34 | |
| *** tbachman has quit IRC | 20:36 | |
| *** tbachman_ has quit IRC | 20:39 | |
| *** eharney has quit IRC | 20:39 | |
| *** tbachman has joined #openstack-nova | 20:46 | |
| openstackgerrit | Merged openstack/nova master: [placement] Debug log per granular request group https://review.openstack.org/588350 | 20:46 |
| *** moshele has joined #openstack-nova | 20:46 | |
| *** tbachman has quit IRC | 21:02 | |
| openstackgerrit | melanie witt proposed openstack/nova stable/ocata: [stable only] Add functional regression test for bug 1783613 https://review.openstack.org/588416 | 21:02 |
| openstack | bug 1783613 in OpenStack Compute (nova) ocata "[ocata only] quota usage not decremented during boot/delete race" [Undecided,In progress] https://launchpad.net/bugs/1783613 - Assigned to melanie witt (melwitt) | 21:02 |
| openstackgerrit | melanie witt proposed openstack/nova stable/ocata: [stable only] Handle quota usage during create/delete races https://review.openstack.org/582413 | 21:02 |
| *** tbachman has joined #openstack-nova | 21:04 | |
| *** edmondsw has quit IRC | 21:15 | |
| *** harlowja has joined #openstack-nova | 21:21 | |
| *** moshele has quit IRC | 21:27 | |
| *** tssurya has quit IRC | 21:31 | |
| *** cdent has quit IRC | 21:40 | |
| *** slagle has quit IRC | 21:53 | |
| *** rcernin has joined #openstack-nova | 22:15 | |
| jaypipes | mriedem: looks like you had to spend a lot of time going through that starling-x diff. :( | 22:18 |
| *** avolkov has quit IRC | 22:24 | |
| *** nicolasbock has quit IRC | 22:25 | |
| *** luksky has quit IRC | 22:25 | |
| *** Kevin_Zheng has quit IRC | 22:28 | |
| mriedem | definitely > 0 | 22:29 |
| mriedem | i put it off for a couple of weeks and then once i got going it was actually kind of interesting | 22:33 |
| mriedem | although i'll say i glossed over a ton of the l3 cache and shared/pinned floating cpus and scaling (live resize) stuff | 22:33 |
| mriedem | because it's just a ton of code | 22:33 |
| mriedem | plus it's all super low-level in the hardware.py module which i avoid at all costs | 22:33 |
| melwitt | nice job on the spreadsheet, lots of interesting info there | 22:42 |
| *** bitskrie1 has joined #openstack-nova | 22:43 | |
| *** mvkr has joined #openstack-nova | 22:45 | |
| mriedem | heh, maybe don't do this https://bugs.launchpad.net/nova/+bug/1785193 | 22:51 |
| openstack | Launchpad bug 1785193 in OpenStack Compute (nova) "changing a node's cell results in duplicate hypervisors" [Undecided,New] - Assigned to Chen (chenn2) | 22:51 |
| mriedem | "it hurts when i do x." "then don't do x" | 22:52 |
| melwitt | we need a safety fence | 22:52 |
| melwitt | oh, they did it by editing nova.conf? heh. I had assumed it was through a nova-manage command of some sort | 22:53 |
| openstackgerrit | melanie witt proposed openstack/nova master: Add a prelude release note for the 18.0.0 Rocky GA https://review.openstack.org/589303 | 22:56 |
| * melwitt will bbl | 22:59 | |
| mriedem | well, this isn't really a cells thing | 23:00 |
| mriedem | changing the db config at any point randomly would have caused weird issues | 23:01 |
| melwitt | yeah, I hadn't yet read the bug and assumed a thing had occurred from use of a nova-manage command | 23:02 |
| melwitt | when I said, "we need a safety fence" | 23:02 |
| mriedem | nova is not enterprise ready | 23:06 |
| melwitt | :) | 23:07 |
| *** hongbin has quit IRC | 23:19 | |
| *** tbachman has quit IRC | 23:21 | |
| *** hemna_ has quit IRC | 23:36 | |
| *** tetsuro_ has joined #openstack-nova | 23:48 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!