Friday, 2017-08-04

*** Nel1x has joined #openstack-nova00:03
openstackgerritSean Dague proposed openstack/nova master: Add Contributor Guide section page  https://review.openstack.org/49064400:09
*** thorst_afk has joined #openstack-nova00:11
*** catintheroof has quit IRC00:14
*** itlinux has joined #openstack-nova00:16
*** gmann has quit IRC00:17
*** gmann has joined #openstack-nova00:17
*** efoley has quit IRC00:18
*** yangyapeng has quit IRC00:20
*** yangyapeng has joined #openstack-nova00:20
*** Sukhdev_ has quit IRC00:20
*** gbarros has quit IRC00:23
*** acormier has joined #openstack-nova00:23
*** slaweq has joined #openstack-nova00:24
*** yangyapeng has quit IRC00:24
*** Swami has quit IRC00:28
*** slaweq has quit IRC00:28
*** kiennt has joined #openstack-nova00:39
*** Shunli has joined #openstack-nova00:45
*** dixiaoli has joined #openstack-nova00:46
*** gbarros has joined #openstack-nova00:48
*** dixiaoli has quit IRC00:49
*** dixiaoli has joined #openstack-nova00:49
*** jichen has joined #openstack-nova00:51
*** thorst_afk has quit IRC00:52
*** zhurong has joined #openstack-nova00:52
*** slagle has quit IRC00:53
*** slagle has joined #openstack-nova00:54
*** acormier has quit IRC00:57
*** itlinux has quit IRC00:57
*** iceyao has joined #openstack-nova00:57
*** baoli has joined #openstack-nova01:01
*** itlinux has joined #openstack-nova01:02
*** iceyao has quit IRC01:02
*** phuongnh has joined #openstack-nova01:05
*** acormier has joined #openstack-nova01:14
*** acormier has quit IRC01:16
*** acormier has joined #openstack-nova01:17
*** iceyao has joined #openstack-nova01:17
*** yangyapeng has joined #openstack-nova01:19
*** Apoorva_ has quit IRC01:23
*** Apoorva has joined #openstack-nova01:24
*** dave-mccowan has joined #openstack-nova01:27
*** Apoorva has quit IRC01:28
*** ijw has quit IRC01:28
*** hongbin has joined #openstack-nova01:34
openstackgerritGhanshyam Mann proposed openstack/nova master: Improve stable-api doc with current API state  https://review.openstack.org/48992601:35
*** phuongnh has quit IRC01:40
*** yamahata has quit IRC01:41
*** acormier has quit IRC01:41
*** phuongnh has joined #openstack-nova01:41
*** edmondsw has joined #openstack-nova01:42
*** acormier has joined #openstack-nova01:42
*** thorst_afk has joined #openstack-nova01:42
*** edmondsw has quit IRC01:46
*** acormier has quit IRC01:46
*** acormier has joined #openstack-nova01:48
*** diablo_rojo has joined #openstack-nova01:58
gmannmriedem1, alex_xu ^^02:00
mriedem1thanks02:01
*** slaweq has joined #openstack-nova02:01
alex_xugmann: thanks02:02
*** phuongnh has quit IRC02:02
*** phuongnh has joined #openstack-nova02:03
*** yushb has joined #openstack-nova02:03
*** mriedem1 is now known as mriedem02:05
*** slaweq has quit IRC02:07
*** thorst_a_ has joined #openstack-nova02:18
*** thorst_a_ has quit IRC02:18
*** Sukhdev has quit IRC02:18
*** thorst_afk has quit IRC02:20
openstackgerrithuangtianhua proposed openstack/python-novaclient master: Remove variable definition for command error msg  https://review.openstack.org/49070502:20
*** huangtianhua has joined #openstack-nova02:21
*** Shunli has quit IRC02:28
*** Shunli has joined #openstack-nova02:28
openstackgerritJonghan Park proposed openstack/nova master: Network should be distinguished by UUID instead of Name  https://review.openstack.org/49033202:29
*** dave-mccowan has quit IRC02:29
*** gbarros has quit IRC02:32
*** diablo_rojo has quit IRC02:36
*** jamesden_ has quit IRC02:48
*** jmlowe has quit IRC02:48
*** mingyu has joined #openstack-nova02:51
*** udesale has joined #openstack-nova02:51
*** jamesdenton has joined #openstack-nova02:52
mriedemjaypipes: dansmith: gibi_chaosmonkey: i've got a start on the shared storage resize functional tests, going to have to give up in a bit but will post it first02:56
dansmithI had to give up on the two failures in jaypipes' patch02:58
dansmithfigure I'll talk to him about it tomorrow02:58
*** litao__ has joined #openstack-nova02:58
*** acormier has quit IRC02:59
*** mingyu has quit IRC03:01
*** mingyu has joined #openstack-nova03:01
*** mingyu has quit IRC03:04
openstackgerritTetsuro Nakamura proposed openstack/nova master: fix nova accepting invalid availability zone name with ':'  https://review.openstack.org/49072203:07
*** Nel1x has quit IRC03:17
*** baoli has quit IRC03:23
mriedemseeing something weird where it appears the disk for the shared storage provider is only the value of the old flavor after the resize, but it should be the sum of old + new per the fix we merged today03:24
mriedembut it's late and i'm tired03:24
openstackgerritMatt Riedemann proposed openstack/nova master: WIP: Add functional resize tests using shared storage  https://review.openstack.org/49073303:28
*** bzhao_ has quit IRC03:29
*** edmondsw has joined #openstack-nova03:30
*** edmondsw has quit IRC03:34
*** slaweq has joined #openstack-nova03:39
*** shiyer has quit IRC03:42
*** ijw has joined #openstack-nova03:44
*** slaweq has quit IRC03:44
openstackgerritMichael Still proposed openstack/nova master: Avoid chowning console logs in libvirt  https://review.openstack.org/47222903:44
openstackgerritMichael Still proposed openstack/nova master: First attempt at adding a privsep user to nova itself.  https://review.openstack.org/45916603:44
openstackgerritMichael Still proposed openstack/nova master: Move execs of touch to privsep.  https://review.openstack.org/48919003:44
openstackgerritMichael Still proposed openstack/nova master: Move execs of tee to privsep.  https://review.openstack.org/48943803:44
openstackgerritMichael Still proposed openstack/nova master: Move libvirt usages of chown to privsep.  https://review.openstack.org/47197203:44
openstackgerritMichael Still proposed openstack/nova master: Read from console ptys using privsep.  https://review.openstack.org/48948603:44
openstackgerritMichael Still proposed openstack/nova master: Refactor libvirt.utils.execute() away.  https://review.openstack.org/48981603:45
openstackgerritMichael Still proposed openstack/nova master: Move libvirts dmcrypt support to privsep.  https://review.openstack.org/49073703:45
*** shiyer has joined #openstack-nova03:45
*** ijw has quit IRC03:49
*** yangyapeng has quit IRC03:51
*** mriedem has quit IRC03:53
*** tbachman has joined #openstack-nova03:53
*** crushil has joined #openstack-nova03:55
*** tbachman has quit IRC03:57
openstackgerritNaichuan Sun proposed openstack/nova master: xenapi: cached images should be cleaned up by time  https://review.openstack.org/46595403:58
*** yangyapeng has joined #openstack-nova03:58
*** ijw has joined #openstack-nova04:00
*** iceyao has quit IRC04:01
*** Sukhdev has joined #openstack-nova04:01
*** tbachman has joined #openstack-nova04:02
*** yangyapeng has quit IRC04:03
*** links has joined #openstack-nova04:03
*** yangyapeng has joined #openstack-nova04:07
*** ijw has quit IRC04:08
*** hshiina has joined #openstack-nova04:10
*** yangyapeng has quit IRC04:12
*** crushil has quit IRC04:12
*** crushil has joined #openstack-nova04:13
*** sree has joined #openstack-nova04:14
*** sree has quit IRC04:15
*** vks1 has joined #openstack-nova04:16
*** yangyapeng has joined #openstack-nova04:16
*** sree has joined #openstack-nova04:19
*** prateek has joined #openstack-nova04:19
*** thorst_afk has joined #openstack-nova04:19
*** hongbin has quit IRC04:20
*** yushb has quit IRC04:20
*** yangyapeng has quit IRC04:21
*** crushil has quit IRC04:22
*** yangyapeng has joined #openstack-nova04:22
*** trungnv has quit IRC04:25
*** trungnv has joined #openstack-nova04:25
*** yangyapeng has quit IRC04:27
*** iceyao has joined #openstack-nova04:29
*** Sukhdev has quit IRC04:30
*** edmondsw has joined #openstack-nova04:31
*** yangyapeng has joined #openstack-nova04:31
*** thorst_afk has quit IRC04:33
*** yushb has joined #openstack-nova04:33
*** iceyao has quit IRC04:34
*** hshiina has quit IRC04:35
*** yangyapeng has quit IRC04:35
*** edmondsw has quit IRC04:36
*** yangyapeng has joined #openstack-nova04:40
*** tuanluong has joined #openstack-nova04:41
*** yangyapeng has quit IRC04:41
*** yangyapeng has joined #openstack-nova04:41
*** hieulq has quit IRC04:42
*** kiennt has quit IRC04:42
*** kiennt has joined #openstack-nova04:42
*** hieulq has joined #openstack-nova04:42
*** trinaths has joined #openstack-nova04:46
*** iceyao has joined #openstack-nova04:47
*** sridharg has joined #openstack-nova04:59
*** mamandle has joined #openstack-nova05:03
*** ratailor has joined #openstack-nova05:07
openstackgerritSpencer Yu proposed openstack/python-novaclient master: Nova client should retry with Retry-After value  https://review.openstack.org/44776605:14
openstackgerritNaichuan Sun proposed openstack/nova master: xenapi: cached images should be cleaned up by time  https://review.openstack.org/46595405:18
*** phuongnh has quit IRC05:18
*** tbachman has quit IRC05:18
*** slaweq has joined #openstack-nova05:19
*** phuongnh has joined #openstack-nova05:19
*** yamahata has joined #openstack-nova05:21
*** slaweq has quit IRC05:24
openstackgerritTakashi NATSUME proposed openstack/nova-specs master: Enable cold migration with target host  https://review.openstack.org/48903105:26
*** sree has quit IRC05:27
*** psachin has joined #openstack-nova05:29
*** gouthamr has quit IRC05:31
*** sree has joined #openstack-nova05:31
*** larainema has quit IRC05:32
*** coreywright has quit IRC05:34
*** tbachman has joined #openstack-nova05:35
*** kiennt has quit IRC05:41
*** takashin has left #openstack-nova05:47
*** tbachman has quit IRC05:49
*** gaoyan has joined #openstack-nova05:51
mikaljaypipes: tonyb and I are trying to work out in https://review.openstack.org/#/c/489438 if _plug_bridge_with_port is dead code or not. I can't find a call tree which results in it being called. Is it perhaps a leftover from our pre os-vif times?05:52
*** markus_z has joined #openstack-nova05:53
openstackgerritNaichuan Sun proposed openstack/nova master: xenapi: cached images should be cleaned up by time  https://review.openstack.org/46595405:53
*** yushb has quit IRC05:54
*** lpetrut has joined #openstack-nova05:57
*** vishwana_ has quit IRC06:00
*** vishwanathj has joined #openstack-nova06:00
openstackgerritNaichuan Sun proposed openstack/nova master: xenapi: cached images should be cleaned up by time  https://review.openstack.org/46595406:01
*** thorst_afk has joined #openstack-nova06:01
*** coreywright has joined #openstack-nova06:03
*** thorst_afk has quit IRC06:05
*** markvoelker has quit IRC06:08
*** yushb has joined #openstack-nova06:09
*** oomichi has quit IRC06:09
*** mlakat has quit IRC06:10
*** oomichi has joined #openstack-nova06:10
*** mlakat has joined #openstack-nova06:11
*** ijw has joined #openstack-nova06:11
*** Oku_OS-away is now known as Oku_OS06:14
*** ijw has quit IRC06:16
*** kiennt has joined #openstack-nova06:17
*** edmondsw has joined #openstack-nova06:19
*** haobing has joined #openstack-nova06:21
*** edmondsw has quit IRC06:24
openstackgerritNaichuan Sun proposed openstack/nova master: xenapi: cached images should be cleaned up by time  https://review.openstack.org/46595406:25
*** slaweq has joined #openstack-nova06:29
*** haobing has quit IRC06:29
*** rcernin has joined #openstack-nova06:30
*** itlinux has quit IRC06:30
*** ijw has joined #openstack-nova06:31
*** gaoyan has quit IRC06:40
openstackgerritNaichuan Sun proposed openstack/nova master: xenapi: Live migration failed in xapi pool  https://review.openstack.org/48945106:51
*** jpena|off is now known as jpena06:52
*** lpetrut has quit IRC06:54
*** ijw has quit IRC06:57
*** slaweq_ has joined #openstack-nova06:58
openstackgerritMerged openstack/nova master: policies: Fix Sphinx issues  https://review.openstack.org/48051606:59
openstackgerritMerged openstack/nova master: doc: Start using oslo_policy.sphinxext  https://review.openstack.org/47935807:00
*** slaweq_ has quit IRC07:04
*** mingyu has joined #openstack-nova07:05
*** alexchadin has joined #openstack-nova07:10
*** tesseract has joined #openstack-nova07:16
*** phuongnh has quit IRC07:29
*** phuongnh has joined #openstack-nova07:29
*** jpena is now known as jpena|brb07:30
openstackgerritViktor Varga proposed openstack/nova master: [WIP] Availability zone name cannot contain ':'  https://review.openstack.org/49077607:30
*** http_GK1wmSU has joined #openstack-nova07:37
*** pcaruana has joined #openstack-nova07:38
*** faizy has joined #openstack-nova07:39
*** http_GK1wmSU has left #openstack-nova07:40
openstackgerritNaichuan Sun proposed openstack/nova master: xenapi: Live migration failed in xapi pool  https://review.openstack.org/48945107:44
*** ralonsoh has joined #openstack-nova07:49
*** rtjure has quit IRC07:50
*** rtjure has joined #openstack-nova07:52
openstackgerrityanghuichan proposed openstack/python-novaclient master: Replace six.itervalues() with dict.values() in python-novaclient  https://review.openstack.org/49078407:54
*** jpena|brb is now known as jpena07:55
*** ijw has joined #openstack-nova07:57
*** thorst_afk has joined #openstack-nova08:02
*** ralonsoh_ has joined #openstack-nova08:03
*** ralonsoh has quit IRC08:03
openstackgerritSpencer Yu proposed openstack/python-novaclient master: Nova client should retry with Retry-After value  https://review.openstack.org/44776608:04
*** ijw has quit IRC08:04
*** thorst_afk has quit IRC08:06
*** mamandle has quit IRC08:07
*** markvoelker has joined #openstack-nova08:09
openstackgerritNaichuan Sun proposed openstack/nova master: xenapi: Live migration failed in xapi pool  https://review.openstack.org/48945108:09
*** aarefiev_afk is now known as aarefiev08:10
*** belmoreira has joined #openstack-nova08:11
*** phuongnh has quit IRC08:11
*** oomichi has quit IRC08:15
*** mdavidson has quit IRC08:15
*** oomichi has joined #openstack-nova08:15
*** slaweq_ has joined #openstack-nova08:16
*** danpawlik has quit IRC08:16
*** slaweq has quit IRC08:16
*** danpawlik has joined #openstack-nova08:18
*** sree has quit IRC08:18
*** sree has joined #openstack-nova08:19
*** sree has quit IRC08:23
*** inara has quit IRC08:24
*** takedakn has joined #openstack-nova08:25
*** derekh has joined #openstack-nova08:25
*** yushb has quit IRC08:25
*** nicolasbock has joined #openstack-nova08:25
*** inara has joined #openstack-nova08:26
*** mamandle has joined #openstack-nova08:29
*** efoley has joined #openstack-nova08:30
*** zhurong has quit IRC08:36
*** abalutoiu has joined #openstack-nova08:37
*** yamamoto has quit IRC08:37
*** yamahata has quit IRC08:39
*** slaweq has joined #openstack-nova08:42
*** markvoelker has quit IRC08:43
*** zhurong has joined #openstack-nova08:44
openstackgerritNaichuan Sun proposed openstack/nova master: xenapi: cached images should be cleaned up by time  https://review.openstack.org/46595408:45
*** slaweq has quit IRC08:47
*** yamamoto has joined #openstack-nova08:50
*** vks1 has quit IRC08:51
*** lpetrut has joined #openstack-nova08:53
*** sree has joined #openstack-nova08:53
*** vvargaszte has quit IRC08:53
*** vks1 has joined #openstack-nova08:55
*** lucas-afk is now known as lucasagomes08:55
*** yushb has joined #openstack-nova08:58
*** vvargaszte has joined #openstack-nova09:00
*** ijw has joined #openstack-nova09:01
openstackgerritStephen Finucane proposed openstack/nova master: doc: Import installation guide  https://review.openstack.org/47748809:02
openstackgerritStephen Finucane proposed openstack/nova master: doc: Import administration guide  https://review.openstack.org/47749709:02
*** thorst_afk has joined #openstack-nova09:03
*** ijw has quit IRC09:06
*** thorst_afk has quit IRC09:07
*** takedakn has quit IRC09:10
*** mdavidson has joined #openstack-nova09:12
*** vvargaszte has quit IRC09:14
openstackgerritSpencer Yu proposed openstack/python-novaclient master: Nova client should retry with Retry-After value  https://review.openstack.org/44776609:16
openstackgerritMerged openstack/nova master: [placement] Require at least one allocation when PUT  https://review.openstack.org/49019509:18
*** Shunli has quit IRC09:32
openstackgerrityanghuichan proposed openstack/python-novaclient master: Replace six.itervalues() with dict.values() in python-novaclient  https://review.openstack.org/49078409:32
openstackgerrityuanyue proposed openstack/nova master: Complete dostring of live_migration related methods  https://review.openstack.org/49080709:33
*** dixiaoli has quit IRC09:34
*** mikal has quit IRC09:39
*** mikal has joined #openstack-nova09:41
*** markvoelker has joined #openstack-nova09:41
*** sdague has joined #openstack-nova09:42
*** sambetts|afk is now known as sambetts09:46
openstackgerritSpencer Yu proposed openstack/python-novaclient master: Nova client should retry with Retry-After value  https://review.openstack.org/44776609:46
*** jichen has quit IRC09:47
*** takedakn has joined #openstack-nova09:49
sdaguesfinucan: are you fixing the docs build for your big imports? or should I?09:50
stephenfinsdague: Which big imports, now?09:52
stephenfinI think it's only the index change that's actually broken now?09:53
stephenfinthough I am fixing a load of style issues in the admin guide import09:53
stephenfinETA about an hour09:53
openstackgerritBalazs Gibizer proposed openstack/nova master: Resize no valid host  https://review.openstack.org/49081309:53
openstackgerritBalazs Gibizer proposed openstack/nova master: Test resize with too big flavor  https://review.openstack.org/49081409:53
openstackgerritBalazs Gibizer proposed openstack/nova master: Test resize with too big flavor  https://review.openstack.org/49081409:54
*** dtantsur|afk is now known as dtantsur09:55
*** edmondsw has joined #openstack-nova09:55
sdaguestephenfin: the install guide failed doc build09:58
sdagueif you are working on the admin guide, I'll just figure out and fix the install guide09:58
stephenfinsdague: Ah, go for it so10:00
*** edmondsw has quit IRC10:00
openstackgerritJeremy Liu proposed openstack/python-novaclient master: Nova client should retry with Retry-After value  https://review.openstack.org/44776610:03
stephenfinsdague: We don't have any EC2 stuff in the nova tree any more, do we?10:04
*** slaweq has joined #openstack-nova10:04
*** trungnv has quit IRC10:07
openstackgerritSean Dague proposed openstack/nova master: doc: Import installation guide  https://review.openstack.org/47748810:08
sdaguewe don't10:08
sdagueyeh, all that euca stuff should get deleted from the admin manual10:08
*** tuanluong has quit IRC10:08
openstackgerritSpencer Yu proposed openstack/python-novaclient master: Nova client should retry with Retry-After value  https://review.openstack.org/44776610:09
*** zenoway has joined #openstack-nova10:09
sdaguestephenfin: turns out it was just an extra space in the install guide, should build now10:09
*** slaweq has quit IRC10:10
*** gabor_antal has quit IRC10:10
*** iceyao has quit IRC10:14
*** markvoelker has quit IRC10:14
openstackgerritRodolfo Alonso Hernandez proposed openstack/os-vif master: Add Port Profile info to VIF objects OVS plugin  https://review.openstack.org/49081910:15
gibi_chaosmonkeydansmith, jaypipes, mriedem, cdent: a Friday bugreport https://bugs.launchpad.net/nova/+bug/170863710:16
openstackLaunchpad bug 1708637 in OpenStack Compute (nova) "nova does not properly claim resources when server resized to a too big flavor" [Undecided,New]10:16
openstackgerritRodolfo Alonso Hernandez proposed openstack/os-vif master: Add Port Profile info to VIF objects OVS plugin  https://review.openstack.org/49081910:16
gibi_chaosmonkeydansmith, jaypipes, mriedem, cdent: this is the problem I tried to report yesterday but now with regression tests on master https://review.openstack.org/#/c/49081410:17
*** zhurong has quit IRC10:17
*** openstackgerrit has quit IRC10:18
gibi_chaosmonkeydansmith, jaypipes, mriedem, cdent: there is a continuation of it when instead of asking for too much vcpu I ask for too much custom resource but I don't have the test for that case yet10:18
*** trinaths has left #openstack-nova10:19
*** takedakn has quit IRC10:19
*** sean-k-mooney has joined #openstack-nova10:19
*** yushb has quit IRC10:23
sean-k-mooneystephenfin: cfriesen:  sdague:  https://bugs.launchpad.net/nova/+bug/1708433 i have updated this with a little more detail. im pretty sure this bug is caused because we are not calling bind port and updating the port binding profile before we are trying to retrive the pci_slot but i have not traced through the code to check yet10:26
openstackLaunchpad bug 1708433 in OpenStack Compute (nova) "Attaching sriov nic VM fail with keyError pci_slot" [Undecided,Incomplete]10:26
*** openstackgerrit has joined #openstack-nova10:26
openstackgerritStephen Finucane proposed openstack/nova master: doc: Import administration guide  https://review.openstack.org/47749710:26
stephenfinasettle: ^10:26
stephenfinsdague too ^10:27
asettleThanks :)10:27
*** zenoway has quit IRC10:28
*** zenoway has joined #openstack-nova10:29
openstackgerritStephen Finucane proposed openstack/nova master: doc: Import administration guide  https://review.openstack.org/47749710:29
*** mingyu has quit IRC10:30
openstackgerritBalazs Gibizer proposed openstack/nova master: Test resize with too big flavor  https://review.openstack.org/49081410:31
openstackgerritLee Yarwood proposed openstack/nova-specs master: Libvirt: Native LUKS file and host device decryption by QEMU  https://review.openstack.org/49082410:32
*** yamamoto has quit IRC10:32
*** zenoway has quit IRC10:33
*** yamamoto has joined #openstack-nova10:33
*** edmondsw has joined #openstack-nova10:36
sdaguestephenfin: so, I'm inclined that we should just merge these because even with flaws, getting the rest of the docs and TOC sorted is going to be hard with so many patches outstanding10:36
stephenfinI agree. Just push them10:37
stephenfinWe can clean up the mess gradually10:37
sdaguestephenfin: ok, I just hit +A on the install guide10:38
*** yamamoto has quit IRC10:38
sdagueyou feeling good on the admin guide?10:38
*** iceyao has joined #openstack-nova10:39
stephenfinsdague: I am, yeah. As above, we can clean anything up afterwards. Just want to get something in there for now10:39
*** edmondsw has quit IRC10:40
janguttersean-k-mooney: problems attaching sr-iov ports seems to be aaancient, or is this something different? https://review.openstack.org/#/c/139910/10:44
sdaguestephenfin: looks like there is a doc build issue with it10:44
sdagueif you've still got it up, probably a quick tox -e docs to sort it out10:44
openstackgerritLee Yarwood proposed openstack/nova-specs master: Libvirt: Native LUKS file and host device decryption by QEMU  https://review.openstack.org/49082410:44
*** udesale has quit IRC10:48
*** yamamoto has joined #openstack-nova10:48
*** zenoway has joined #openstack-nova10:49
sdaguestephenfin / asettle here is the first pass at one of those sub landing pages - https://review.openstack.org/#/c/490644/10:50
*** faizy has quit IRC10:51
sean-k-mooneyjangutter: that is related yes. this did used to work before i thaugt with some edge cases such as the host not haveing an available device still present10:51
asettlesdague: great, looking now :)10:52
asettlesdague: do you want to do these in individual patches, or slowly add to the one?10:52
asettleJust thinking of merge conflicts, actually10:52
sdaguehonestly, individual is probably easier to review10:52
*** zenoway has quit IRC10:53
janguttersean-k-mooney: this would need a fix in nova/network/neutronv2/api.py ?10:53
*** zenoway has joined #openstack-nova10:53
mamandlestephenfin: bauzas: mriedem: https://review.openstack.org/#/c/489848/1 is waiting for +2, can you please take a look? Thanks!10:54
asettlesdague: no problem, I'll just keep an eye out :) please keep adding me as a reviewer :)10:55
sean-k-mooneyjangutter: i think so we would need to do a pci claim and then update the neutron port binding profile10:55
sdaguesfinucan: ok, I'm going through on admin guide fixes looks mostly like bullet issues10:55
sean-k-mooneyjangutter: it looks like https://review.openstack.org/#/c/139910/20/specs/newton/approved/sriov-interface-attach-detach.rst was trying to adress the issue in https://bugs.launchpad.net/nova/+bug/170843310:56
openstackLaunchpad bug 1708433 in OpenStack Compute (nova) "Attaching sriov nic VM fail with keyError pci_slot" [Undecided,Incomplete]10:56
*** rmart04 has joined #openstack-nova10:57
*** abalutoiu has quit IRC10:57
sdagueasettle: do you know if the toc warning is ignorable from sphinx so we don't keep having to use hidden tocs?10:57
asettlesdague: ooo good question. I imagine it is.10:58
asettleI personally have not tried.10:58
sdagueI was looking at the class of warnings and it didn't jump out at me, and I've yet to go through the source code. So I thought i'd ask :)10:58
asettleYeah good question, hmm... Doug might know.10:59
asettleHe's been all up in that sphinx world lately10:59
asettleOr even maybe Andreas10:59
asettleAlso - just left minor nits/comments on your Contrib patch. If it's easier, I'm happy to pull down and do the updates.10:59
sdaguehttp://www.sphinx-doc.org/en/stable/config.html#confval-suppress_warnings10:59
asettleLooking11:00
sdagueasettle: sure, feel free to just jump in and make those changes11:00
asettleCoolio - gotta run for luncheon soon. Can do upon return :)11:00
openstackgerritSean Dague proposed openstack/nova master: doc: Import administration guide  https://review.openstack.org/47749711:01
sdagueok, I think that will build now11:01
asettleOkay cool :)11:02
asettlebrb11:02
openstackgerritRodolfo Alonso Hernandez proposed openstack/os-vif master: Add Port Profile info to VIF objects Linux Bridge plugin  https://review.openstack.org/49082911:02
*** thorst_afk has joined #openstack-nova11:04
*** kiennt has quit IRC11:06
*** thorst_afk has quit IRC11:08
*** sree has quit IRC11:09
*** markvoelker has joined #openstack-nova11:11
*** smatzek has joined #openstack-nova11:14
*** sree has joined #openstack-nova11:18
*** zenoway has quit IRC11:19
*** zenoway has joined #openstack-nova11:20
*** jamesden_ has joined #openstack-nova11:21
*** jamesdenton has quit IRC11:21
*** yangyapeng has quit IRC11:30
*** JoseMello has joined #openstack-nova11:30
*** lucasagomes is now known as lucas-hungry11:31
*** mingyu has joined #openstack-nova11:31
*** zenoway has quit IRC11:32
*** zenoway has joined #openstack-nova11:33
*** http_GK1wmSU has joined #openstack-nova11:33
*** http_GK1wmSU has left #openstack-nova11:35
*** mingyu has quit IRC11:35
*** pchavva has joined #openstack-nova11:36
*** yamamoto has quit IRC11:39
*** mdnadeem has joined #openstack-nova11:44
*** slaweq has joined #openstack-nova11:44
*** markvoelker has quit IRC11:45
*** edmondsw has joined #openstack-nova11:47
*** slaweq has quit IRC11:49
*** ratailor has quit IRC11:51
*** thorst_afk has joined #openstack-nova11:51
*** thorst_afk has quit IRC11:52
*** edmondsw has quit IRC11:52
*** jpena is now known as jpena|lunch11:54
*** dave-mccowan has joined #openstack-nova11:55
*** kristian__ has joined #openstack-nova11:56
*** litao__ has quit IRC11:58
openstackgerritMerged openstack/nova master: doc: Import installation guide  https://review.openstack.org/47748811:59
*** erlon has joined #openstack-nova12:03
openstackgerritMerged openstack/nova master: doc: Import administration guide  https://review.openstack.org/47749712:03
*** mamandle has quit IRC12:05
*** thorst has joined #openstack-nova12:06
openstackgerritOpenStack Proposal Bot proposed openstack/nova master: Imported Translations from Zanata  https://review.openstack.org/47709112:08
*** gszasz has joined #openstack-nova12:10
*** mdnadeem has quit IRC12:17
*** larainema has joined #openstack-nova12:20
*** iceyao has quit IRC12:20
*** edmondsw has joined #openstack-nova12:23
*** edleafe is now known as figleaf12:26
*** weshay_pto is now known as weshay12:26
*** liverpooler has joined #openstack-nova12:26
*** mingyu has joined #openstack-nova12:27
*** tikitavi_ has joined #openstack-nova12:28
*** vks1 has quit IRC12:29
*** markvoelker has joined #openstack-nova12:31
*** ftersin has joined #openstack-nova12:32
*** sree has quit IRC12:33
*** sree has joined #openstack-nova12:33
*** prateek has quit IRC12:35
*** gbarros has joined #openstack-nova12:38
*** yamamoto has joined #openstack-nova12:40
openstackgerritAlexandra Settle proposed openstack/nova master: Update install guide to clearly define between package installs  https://review.openstack.org/49084412:41
*** baoli has joined #openstack-nova12:43
*** sree_ has joined #openstack-nova12:44
*** sree_ is now known as Guest5570612:44
*** sree has quit IRC12:45
*** yamamoto has quit IRC12:45
openstackgerritAlexandra Settle proposed openstack/nova master: Update install guide to clearly define between package installs  https://review.openstack.org/49084412:45
stephenfinsdague: We'll leave stripping out the euca stuff til later. I put a TODO in there12:46
stephenfinsdague: might be good to assign willing SMEs to each doc for a thorough review at the PTG. There's a lot of outdated stuff in there that needs to be updated/removed/"synergized"12:46
stephenfinpossible PTG session, actually?12:47
stephenfinsdague: RE: resolving the ToC issues. This is what Django do12:48
stephenfinhttps://docs.djangoproject.com/en/1.8/contents/12:48
* stephenfin continues to profess his love for all things Django12:48
*** yangyapeng has joined #openstack-nova12:49
*** lucas-hungry is now known as lucasagomes12:50
*** yangyapeng has quit IRC12:53
*** kylek3h has joined #openstack-nova12:54
*** jaypipes is now known as leakypipes12:54
openstackgerritOpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements  https://review.openstack.org/49085912:54
*** jpena|lunch is now known as jpena12:55
leakypipesgibi_chaosmonkey: cool, will check it out shortly.12:56
*** gcb has quit IRC12:56
*** lyan has joined #openstack-nova12:59
bauzasgibi_chaosmonkey: do you have more explanations for https://bugs.launchpad.net/nova/+bug/1708637 ?13:01
openstackLaunchpad bug 1708637 in OpenStack Compute (nova) "nova does not properly claim resources when server resized to a too big flavor" [High,New]13:01
bauzasgibi_chaosmonkey: tbc, CoreFilter shall not be used in Pike unless you uses CachingScheduler13:02
tikitavi_leakypipes: Hi. We are ec2-api team and now we have problem in gating because of bug in nova #1708210. It is fixed in https://review.openstack.org/#/c/490340/ so we are looking forward when it will be merged.13:02
*** jmlowe has joined #openstack-nova13:04
*** aarefiev is now known as aarefiev_afk13:07
*** mingyu has quit IRC13:11
*** gbarros has quit IRC13:11
*** mingyu has joined #openstack-nova13:14
*** vks1 has joined #openstack-nova13:16
*** yangyapeng has joined #openstack-nova13:16
*** catintheroof has joined #openstack-nova13:17
*** gbarros has joined #openstack-nova13:17
*** esberglu has joined #openstack-nova13:25
*** xinliang has quit IRC13:27
dansmithleakypipes: did you see where the patch is? one unit and one functional failure that I didn't resolve13:34
dansmiththe unit test is so mocked up the butt I can't really even tell if it's useful13:34
dansmithand the functional one was a gabbit thing and I dunno why it would have changed or what to do about it13:34
*** lucasxu has joined #openstack-nova13:34
dansmithleakypipes: and did you see my comments about the fixes for the ocata-to-pike one?13:35
dansmithhmm, I guess the gabbit one didn't fail in jenkins, that's weird13:35
stephenfinleakypipes: Two easy reviews for you for later https://review.openstack.org/#/c/486536/ https://review.openstack.org/#/c/480092/13:36
stephenfin(gets os-vif off of https://doughellmann.com/doc-migration/)13:36
*** xyang1 has joined #openstack-nova13:39
*** liverpooler has quit IRC13:39
*** liverpooler has joined #openstack-nova13:39
*** xinliang has joined #openstack-nova13:40
leakypipesdansmith: no, sorry superdan, just getting back to things this morning.13:45
openstackgerritMerged openstack/os-vif master: Update the documentation link for doc migration  https://review.openstack.org/48653613:45
*** jistr is now known as jistr|mtg13:46
*** jmlowe has quit IRC13:47
dansmithheh, always with the friday guilt13:47
*** dansmith is now known as superdan13:47
*** Dinesh_Bhor has quit IRC13:47
leakypipes:)13:47
superdanleakypipes: so the remaining unit test fails because the fake resource tracker has no compute nodes, but tries to pull self.compute_nodes[cn].uuid out and fails with keyerror13:48
leakypipessuperdan: k.13:48
superdanbut the test seriously has like 30 mocks and so it's really hard to tell if it's even useful13:48
leakypipessuperdan: I know, it's rough13:48
*** baoli has quit IRC13:49
leakypipessuperdan: am I good to take the baton and pull code?13:49
superdanwhen I tried adding a compute node to it, it failed much earlier, which I found amusing13:49
superdanleakypipes: yes13:49
superdanleakypipes: look at my two latest comments on the review though and make sure my fixes for your ocata to pike test are legit13:49
*** efried_zzz is now known as fried_rice13:49
*** mriedem has joined #openstack-nova13:50
mriedemo/13:51
*** cleong has joined #openstack-nova13:51
mriedemstephenfin: the admin guide import was missing several things i pointed out https://review.openstack.org/#/c/477497/13:53
mriedemlike https://github.com/openstack/openstack-manuals/blob/stable/ocata/doc/admin-guide/source/compute-flavors.rst13:53
*** crushil has joined #openstack-nova13:53
stephenfinmriedem: s/compute-//13:53
stephenfinHe just dropped all the prefixes13:53
stephenfinThey're there alright :)13:53
mriedemall of the cli-nova-* stuff from https://github.com/openstack/openstack-manuals/tree/stable/ocata/doc/admin-guide/source isn't in there13:54
*** bhagyashris has quit IRC13:54
mriedemlike evacuate and specify-host13:54
ftersinmriedem: hi. who could merge https://review.openstack.org/#/c/490340/ ? it fixes a regression bug which blocks CI for ec2api13:54
stephenfinmriedem: Yeah, I'm not sure where those are going yet, tbh13:55
leakypipessuperdan: yup, will do. just wrapping up meetings.13:55
stephenfinThey're going to need some work as they're pretty out of date and don't tie into the 'oslo_config.sphinxext'. I'll take a shot at it shortly13:55
mriedemstephenfin: there is also https://github.com/openstack/openstack-manuals/blob/stable/ocata/doc/admin-guide/source/cli-set-compute-quotas.rst13:56
mriedemso the naming pattern isn't consistent13:56
mriedemhttps://github.com/openstack/openstack-manuals/blob/stable/ocata/doc/admin-guide/source/cli-os-migrate-cfg-ssh.rst13:56
stephenfinmriedem: Good point. Looks like I'll need to go through them all individually :(13:56
mriedemstephenfin: i'll just open a bug to track it13:57
stephenfinmriedem: In any case, sdague and I figured it was better to just get _something_ in and build upon it13:57
stephenfinDo - you can assign it to me13:57
*** rmart04 has quit IRC13:58
*** marst has joined #openstack-nova13:58
*** slaweq has joined #openstack-nova13:58
openstackgerritOpenStack Proposal Bot proposed openstack/os-vif master: Updated from global requirements  https://review.openstack.org/48808613:59
stephenfinmriedem: Can we still merge those req updates if they make sense ^14:00
stephenfin*?14:00
*** hongbin has joined #openstack-nova14:00
mriedemwe can't release them for pike at this point14:00
mriedemor in stable/pike14:00
mriedemi'd just hold off on those until queens14:00
mriedemfor the libraries14:00
*** baoli has joined #openstack-nova14:01
mriedemstephenfin: https://bugs.launchpad.net/nova/+bug/170866614:01
openstackLaunchpad bug 1708666 in OpenStack Compute (nova) "docs: admin guide import is missing several compute-specific docs" [High,Confirmed] - Assigned to Stephen Finucane (stephenfinucane)14:01
*** ratailor has joined #openstack-nova14:01
*** felipemonteiro has joined #openstack-nova14:03
*** slaweq has quit IRC14:04
stephenfinmriedem: (y) (for both)14:04
*** alexchadin has quit IRC14:04
*** felipemonteiro_ has joined #openstack-nova14:04
*** bnemec is now known as beekneemech14:07
*** felipemonteiro has quit IRC14:08
*** jmlowe has joined #openstack-nova14:09
*** tikitavi_ has quit IRC14:09
*** zenoway has quit IRC14:10
*** zenoway has joined #openstack-nova14:10
*** links has quit IRC14:12
*** smatzek has quit IRC14:13
*** smatzek has joined #openstack-nova14:13
*** zenoway has quit IRC14:15
mriedemftersin: the test needs work if you want to touch it up14:15
mriedemtakashin is probably no longer working14:15
ftersinmriedem: thx. we'll ping takashi tomorrow14:20
*** xyang1 has quit IRC14:20
*** tbachman has joined #openstack-nova14:20
*** xyang1 has joined #openstack-nova14:20
ftersinuhhm... tomorrow, in monday :/14:20
mriedemheh, ok, or i'll just update it14:21
mriedemit's a 1 line assertion in the test14:21
ftersinour developer's gone home, i'm busy with other things and do not have OS sources with git, etc right now14:23
*** burt has joined #openstack-nova14:23
mriedemthat's fine14:23
*** acormier has joined #openstack-nova14:24
openstackgerritMerged openstack/nova master: Update install guide to clearly define between package installs  https://review.openstack.org/49084414:26
*** slaweq_ has quit IRC14:27
*** josecastroleon has quit IRC14:30
openstackgerritJackie Truong proposed openstack/nova master: Add trusted_certs to instance_extra  https://review.openstack.org/45771114:30
*** trinaths has joined #openstack-nova14:31
*** ratailor has quit IRC14:33
*** ratailor has joined #openstack-nova14:33
*** nicolasbock has quit IRC14:39
*** ratailor has quit IRC14:44
openstackgerritMatt Riedemann proposed openstack/nova master: Fix getting instance bdms in multiple cells  https://review.openstack.org/49034014:45
*** gouthamr has joined #openstack-nova14:45
mriedemsuperdan: you should probably check this out - multi-cell fun https://review.openstack.org/#/c/490340/14:47
superdancool14:48
*** dfisher has joined #openstack-nova14:48
*** Guest55706 has quit IRC14:50
*** sree has joined #openstack-nova14:51
openstackgerritSean Dague proposed openstack/nova master: Add Contributor Guide section page  https://review.openstack.org/49064414:51
sdaguesfinucan: ^^^14:51
stephenfinsdague: looking14:51
sdaguesfinucan: also, just for you, I changed my word wrap for rst mode to 79 columns :)14:52
stephenfinYay! :D14:52
*** trinaths has left #openstack-nova14:53
*** trinaths has quit IRC14:53
*** baoli has quit IRC14:55
*** eharney has quit IRC14:55
*** sridharg has quit IRC14:56
stephenfinsdague: Yeah, I personally prefer that bullet point style. It's used in both the OVS and Django docs, and gives way more context14:58
*** sree has quit IRC14:59
superdanmriedem: not sure how that bdms.update() is doing the right thing15:00
*** jistr|mtg is now known as jistr15:01
*** Oku_OS is now known as Oku_OS-away15:02
mriedemsuperdan: if no bdms are found in a cell for the list of instance uuids it should just return {}15:03
leakypipessuperdan: k, just finished up with meetings. heading to code now.15:03
leakypipessorry for delay15:03
*** mdnadeem has joined #openstack-nova15:04
mriedemleakypipes: before heading back into the RT fun, mind looking at this regression fix? https://review.openstack.org/#/c/489763/15:04
leakypipesmriedem: yup, will do.15:04
superdanmriedem: the object method returns a List object though, not a dict15:04
mriedemsuperdan: which one?15:04
mriedemdid you see my reply in the patch?15:05
superdanBlockDeviceMappingList.get_by_instance_uuids15:05
superdanmriedem: mine is the latest15:05
mriedemsuperdan: that's not the method used15:05
mriedembdms_by_instance_uuid15:05
mriedemwhich returns a dict of bdm lists keyed by instance uuid15:05
superdanoh, indeed15:05
mriedemright, it's confusing as hell15:05
mriedemi had to look at it a few times this morning15:06
*** isq_ has quit IRC15:07
superdanand it's not even plural15:07
*** isq_ has joined #openstack-nova15:07
leakypipeswhat!? not plural? /me changes clothes into SuperGrammarCorrector15:08
*** pchavva has quit IRC15:08
superdansomeone on a ML I'm on yesterday was making fun of someone who couldn't spell, saying "watch your grammer"15:08
superdanI was tickled15:09
leakypipeslol15:09
*** rizwan has joined #openstack-nova15:10
leakypipessuperdan: k, I'm +2 on mriedem15:12
leakypipeshttps://review.openstack.org/#/c/489763/115:12
openstackgerritMerged openstack/python-novaclient master: Replace six.itervalues() with dict.values() in python-novaclient  https://review.openstack.org/49078415:13
*** gbarros has quit IRC15:13
*** rcernin has quit IRC15:14
openstackgerritRodolfo Alonso Hernandez proposed openstack/os-vif master: Add Port Profile info to VIF objects OVS plugin  https://review.openstack.org/49081915:14
superdanmriedem: leakypipes is +2 on you15:15
*** lucasxu has quit IRC15:15
*** sridharg has joined #openstack-nova15:15
leakypipeskinky.15:16
openstackgerritRodolfo Alonso Hernandez proposed openstack/os-vif master: Add Port Profile info to VIF objects Linux Bridge plugin  https://review.openstack.org/49082915:17
superdanI said +2 not #215:17
leakypipessuperdan: rofl15:19
leakypipessuperdan: FYI, I'm fixing up mriedem'15:19
leakypipess review comments on https://review.openstack.org/#/c/490514/15:19
superdanokay thanks15:19
*** slaweq has joined #openstack-nova15:20
mriedemi'll start digging back into my shared storage resize functional test patch to see what weirdness is going on there15:21
*** gbarros has joined #openstack-nova15:21
*** acormier has quit IRC15:22
*** jaosorior has quit IRC15:23
*** slaweq has quit IRC15:31
*** crushil has quit IRC15:32
*** mamandle has joined #openstack-nova15:34
ftersinmriedem: thanks for the review15:35
*** slaweq has joined #openstack-nova15:36
openstackgerritEric Fried proposed openstack/nova master: nova.utils.get_ksa_adapter()  https://review.openstack.org/48813715:37
*** slaweq has quit IRC15:37
*** iceyao has joined #openstack-nova15:38
*** slaweq has joined #openstack-nova15:38
*** iceyao has quit IRC15:42
*** saneax has left #openstack-nova15:44
*** belmoreira has quit IRC15:44
*** dtantsur is now known as dtantsur|afk15:46
*** ftersin has quit IRC15:46
openstackgerritStephen Finucane proposed openstack/nova master: doc: Add additional content to admin guide  https://review.openstack.org/49095215:46
*** rtjure has quit IRC15:50
*** yamahata has joined #openstack-nova15:53
*** slaweq has quit IRC15:53
*** eharney has joined #openstack-nova15:56
*** Apoorva has joined #openstack-nova16:00
*** lucasxu has joined #openstack-nova16:00
*** krtaylor has joined #openstack-nova16:00
*** mamandle has quit IRC16:03
*** psachin has quit IRC16:07
leakypipessuperdan, mriedem: relatively painless reviews on placement-api-ref series starting here: https://review.openstack.org/#/c/450550/16:08
*** hferenc has quit IRC16:09
*** pcaruana has quit IRC16:10
mriedemyeah i've got the series starred, was coming in handy last night writing this functional test for shared storage + resize16:10
mriedemi've also figured out part of what is going weird in this test, which is also the bug we need to fix16:10
leakypipesya16:10
mriedemfor the report client stomping the disk allocations for the shared storage provider16:11
openstackgerritJay Pipes proposed openstack/nova master: remove provider allocs in confirm/revert resize  https://review.openstack.org/48851016:11
openstackgerritJay Pipes proposed openstack/nova master: Add resource utilities to scheduler utils  https://review.openstack.org/49051416:11
leakypipessuperdan, mriedem: k, good to go. all tests passing. ^^16:11
*** markus_z has quit IRC16:11
*** gszasz has quit IRC16:11
mriedemleakypipes: ok one thing in there16:14
mriedemmisunderstanding16:14
*** slaweq has joined #openstack-nova16:14
mriedemthe merge of the resources should pick up the new key in the new flavor16:14
*** marst has quit IRC16:16
*** marst_ has joined #openstack-nova16:16
leakypipesmriedem: ah, ok, yes I misunderstood you. will have a fix up pronto.16:16
mriedemwe should probably drop this from INFO to DEBUG: "Synced traits from os_traits into API DB"16:16
mriedemit dumps out a bunch of traits16:16
*** armax_ has joined #openstack-nova16:16
leakypipesmriedem: you want me to do that in this patch?16:17
mriedemleakypipes: no,16:17
*** itlinux has joined #openstack-nova16:17
leakypipesk16:17
mriedemjust seeing it in my functional test run that uses traits16:17
superdanleakypipes: I still really want that last patch split between the confirm and service version pieces, but I'm happy to work on that if you want16:18
* mriedem goes to lunch16:18
*** mriedem is now known as mriedem_lunch16:18
*** armax has quit IRC16:19
*** armax_ is now known as armax16:19
*** jmlowe has quit IRC16:19
leakypipessuperdan: yeah, that would be cool. just gimme a minute to push a mriedem_lunch request16:19
superdanif we had to revert the confirm bit, for example, we'd not want to revert the service version piece which is harder to undo16:19
superdanokay16:19
*** slaweq has quit IRC16:19
leakypipessuperdan: yeah, agreed16:19
*** sridharg has quit IRC16:19
openstackgerritJay Pipes proposed openstack/nova master: remove provider allocs in confirm/revert resize  https://review.openstack.org/48851016:20
openstackgerritJay Pipes proposed openstack/nova master: Add resource utilities to scheduler utils  https://review.openstack.org/49051416:20
leakypipessuperdan: tag, you're it. :)16:20
* superdan takes the baton16:21
*** esberglu is now known as esberglu_lunch16:25
*** derekh has quit IRC16:26
*** lucasagomes is now known as lucas-afk16:28
superdanugh, pretty sure I'm going to be talking myself out of this one in a couple hours16:31
*** eharney has quit IRC16:34
*** slaweq has joined #openstack-nova16:35
*** http_GK1wmSU has joined #openstack-nova16:38
*** http_GK1wmSU has quit IRC16:38
*** slaweq has quit IRC16:39
openstackgerritEric Fried proposed openstack/nova master: nova.utils.get_ksa_adapter()  https://review.openstack.org/48813716:40
*** lyan has quit IRC16:41
*** baoli has joined #openstack-nova16:42
openstackgerritOpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements  https://review.openstack.org/49085916:46
*** mingyu has quit IRC16:46
*** slaweq has joined #openstack-nova16:48
superdanleakypipes: so I'm going to split this up and push for you to look at and make sure I didn't fubar anything and if we decide it's too much work to separate, we can just pull and resubmit this current PS where things are unified okay?16:48
superdanI've got unit tests passing on the confirm part without the service version part, and am running functional tests now16:48
fried_ricemriedem_lunch Pushing use-service-catalog-for-endpoints to Q at this point?16:49
leakypipessuperdan: ack16:50
openstackgerritMerged openstack/nova master: Fix 409 handling in report client when deleting inventory  https://review.openstack.org/48976316:52
*** crushil has joined #openstack-nova16:54
*** efoley has quit IRC16:54
*** sree has joined #openstack-nova16:56
*** ralonsoh_ has quit IRC17:00
*** sree has quit IRC17:00
openstackgerritMerged openstack/nova master: [placement] Add api-ref for allocations  https://review.openstack.org/47093317:00
*** rajathagasthya has joined #openstack-nova17:02
*** rizwan has quit IRC17:03
*** mdnadeem has quit IRC17:04
*** slaweq has quit IRC17:04
*** yamahata has quit IRC17:04
*** slaweq has joined #openstack-nova17:04
*** lyan has joined #openstack-nova17:08
*** slaweq has quit IRC17:09
*** kristian__ has quit IRC17:11
*** Swami has joined #openstack-nova17:11
*** kristian__ has joined #openstack-nova17:11
*** sambetts is now known as sambetts|afk17:11
*** lpetrut has quit IRC17:12
*** lyan has quit IRC17:13
*** jpena is now known as jpena|off17:14
*** lyan has joined #openstack-nova17:15
*** kristian__ has quit IRC17:15
*** pchavva has joined #openstack-nova17:18
openstackgerritSean Dague proposed openstack/nova master: Create reference subpage  https://review.openstack.org/49099417:20
*** edmondsw_ has joined #openstack-nova17:23
*** slaweq has joined #openstack-nova17:25
*** edmondsw has quit IRC17:26
*** openstackstatus has quit IRC17:27
*** openstack has joined #openstack-nova17:29
*** armax has quit IRC17:29
*** slaweq has quit IRC17:29
*** jeremyfreudberg has joined #openstack-nova17:32
*** jeremyfreudberg has left #openstack-nova17:32
*** priteau has joined #openstack-nova17:34
*** yamahata has joined #openstack-nova17:39
*** armax has joined #openstack-nova17:41
openstackgerritSylvain Bauza proposed openstack/nova master: Pass requested_destination in filter_properties  https://review.openstack.org/48111617:42
openstackgerritSylvain Bauza proposed openstack/nova master: Functional regression test for evacuate with a target  https://review.openstack.org/49099717:42
*** eharney has joined #openstack-nova17:43
*** mingyu has joined #openstack-nova17:47
*** dtp has joined #openstack-nova17:47
*** pchavva has quit IRC17:49
*** hferenc has joined #openstack-nova17:49
bauzasmriedem_lunch: I'll be off in the next minute but see the above changes for https://bugs.launchpad.net/nova/+bug/170245417:49
openstackLaunchpad bug 1702454 in OpenStack Compute (nova) "Transforming the RequestSpec object into legacy dicts doesn't support the requested_destination field" [High,In progress] - Assigned to Sylvain Bauza (sylvain-bauza)17:49
bauzasmriedem_lunch: as we discussed the backports are slightly different17:49
*** esberglu_lunch is now known as esberglu17:49
* bauzas relocating to the Mediteranean coast for the next 2 working days, see ya17:50
*** mingyu has quit IRC17:51
*** rajathagasthya has quit IRC17:55
*** itlinux has quit IRC18:03
*** itlinux has joined #openstack-nova18:05
*** rajathagasthya has joined #openstack-nova18:10
*** kristian__ has joined #openstack-nova18:12
edmondsw_fried_rice you might know how to improve https://review.openstack.org/#/c/48512118:13
superdanleakypipes: okay get ready to rumble18:15
leakypipesok dokey18:15
superdanhang on, forgot to run pep818:15
superdanstand by rumble18:15
leakypipeshehe18:16
*** gbarros has quit IRC18:16
*** kristian__ has quit IRC18:16
*** mriedem has joined #openstack-nova18:17
mriedemyes18:18
mriedemERROR [nova.scheduler.client.report] Failed to allocate resources on provider 9f098d35-83b2-46e1-a2c2-b51ee1a33f66 for instance aeb45d5b-4787-4afa-9b19-430da6f33699. Error:18:18
mriedemUnable to allocate inventory: Inventory for 'DISK_GB' on resource provider '9f098d35-83b2-46e1-a2c2-b51ee1a33f66' invalid.18:18
*** mriedem_lunch has quit IRC18:18
mriedemwhat do you mean you can't allocate against a provider with 0 DISK_GB inventory?!18:18
superdanokay here comes18:18
openstackgerritDan Smith proposed openstack/nova master: remove provider allocs in confirm/revert resize  https://review.openstack.org/48851018:18
openstackgerritDan Smith proposed openstack/nova master: Add resource utilities to scheduler utils  https://review.openstack.org/49051418:18
openstackgerritDan Smith proposed openstack/nova master: Resource tracker compatibility with Ocata and Pike  https://review.openstack.org/49101218:18
*** sambetts|afk has quit IRC18:19
*** slagle has quit IRC18:19
mriedem+2 on https://review.openstack.org/#/c/490514/18:19
leakypipesmriedem: I'd +2 that but I added a lot to it...18:20
leakypipesperhaps melwitt or bauzas could look at it?18:20
mriedembauzas is going to the mediterraneannenaanna coast18:21
superdanbauzas hasn't been on PTO in over 48 hours, so he can't be asked to do things by french law18:21
mriedemzing18:21
*** sambetts_ has joined #openstack-nova18:21
mriedemfried_rice: there was no FFE for the service catalog endpoint thingy18:21
mriedemfried_rice: and as far as i know there hasn't been much core reviewer attention on it18:22
mriedemfried_rice: and you keep changing things :)18:22
mriedemsuperdan: leakypipes: yeah so fun fact, with the shared storage test, we hit https://github.com/openstack/nova/blob/85cd4574b8347d032be2285a277a0abe4d4a6869/nova/scheduler/client/report.py#L1084 all the time b/c i've got the compute node's reporting 0 DISK_GB inventory, so i've at least sorted out and recreated the problem18:23
mriedemit's just not as obvious in the tests until much later18:23
leakypipesk18:24
openstackgerritMerged openstack/os-vif master: doc: Remove cruft from releasenotes conf.py  https://review.openstack.org/48009218:24
*** markmcclain has quit IRC18:24
*** markmcclain has joined #openstack-nova18:24
*** slaweq has joined #openstack-nova18:25
*** openstackstatus has quit IRC18:26
*** openstack has joined #openstack-nova18:28
*** slaweq has quit IRC18:29
*** vks1 has quit IRC18:31
mriedemfor someone doing shared storage for their computes, would we expect them to post like DISK_GB with total=100 and reserved=100, or just not post any DISK_GB for the compute node provider?18:34
openstackgerritJackie Truong proposed openstack/nova master: Add trusted_certs to instance_extra  https://review.openstack.org/45771118:34
mriedemdifference between yes i have some but i'm using it all, vs i want you to think i have no local disk18:34
mriedemeven if i do18:34
*** rizwan has joined #openstack-nova18:35
*** randomhack has joined #openstack-nova18:40
*** esberglu has quit IRC18:43
*** kristian__ has joined #openstack-nova18:43
*** esberglu has joined #openstack-nova18:43
*** slaweq has joined #openstack-nova18:43
*** esberglu has quit IRC18:48
*** slagle has joined #openstack-nova18:51
*** randomhack has quit IRC18:54
*** rizwan has quit IRC18:54
*** sree has joined #openstack-nova18:56
*** s-dean has quit IRC18:59
*** esberglu has joined #openstack-nova19:01
*** sree has quit IRC19:01
*** cleong has quit IRC19:01
*** s-dean has joined #openstack-nova19:03
*** thorst has quit IRC19:03
*** mingyu has joined #openstack-nova19:05
*** thorst has joined #openstack-nova19:06
*** sambetts_ has quit IRC19:06
*** gbarros has joined #openstack-nova19:07
*** sambetts_ has joined #openstack-nova19:08
*** mingyu has quit IRC19:10
*** thorst has quit IRC19:10
mriedemleakypipes: just thought of something - is there anything making VCPU/MEMORY_MB indivisable between providers? for example, if i'm creating a server with x cpu and y ram, and CN1 can provide the vcpu but not the ram, and CN2 can provide the ram but not the vcpu, would placement return both compute node providers to the scheduler and the scheduler would pick one and fail?19:12
mriedemi don't think that's possible unless vcpu/memory_mb is marked shared or something19:12
mriedemwhich is how it can work to get disk from another provider and not the compute node19:12
superdanthat would be really bad19:14
superdanbut pretty sure that we're looking for single providers that provide all the things we want,19:14
superdanor are related to one via aggregate that does19:15
fried_ricemriedem The changes have been in response to reviews, fwiw.19:15
superdanI forget if/how a shared provider is considered and not just another compute node in the same aggregate19:15
superdanbut I think there's a provision for that19:15
superdanmisc_shares_via_aggregate or something maybe?19:15
mriedemright the compute node providers have to be in an aggregate relationship with an rp that has the MISC_SHARES_VIA_AGGREGATE trait19:16
fried_ricemriedem At this pace, it won't be fully completed (because neutron & cinder integration still needed) for at least another week, even assuming my latest rev gets +2s19:16
fried_riceI don't have a problem pushing it; your call whether it's important enough to squeeze in late.19:16
mriedemfried_rice: is there no way that can't be staggered per service?19:16
mriedemfried_rice: i don't think there is justification for getting that in after FF and before RC119:16
fried_ricemriedem Certainly.  That's basically what's happening.  The outstanding change set just does glance & ironic.  The other services can be added one at a time, or whatever.19:17
fried_ricemriedem Anyway, I'll just keep plugging away at the changes, and let y'all decide whether/when to push 'em.19:18
*** thorst has joined #openstack-nova19:18
mriedemsuperdan: another thing i thought of which complicates the shared storage trampling in the heal task, but if we're doing a move and there are allocations for the instance against 3 providers, like CN1 provides ram/cpu and gets DISK_GB from a shared storage provider (which is provider #2), and then the dest compute provider provides ram/cpu/disk (local disk)19:18
mriedemthings get wonky19:18
mriedembecause during a move, there is nothing preventing the scheduler from picking a compute node that is sharing storage with the source node19:18
mriedemso it could pike a node with locak disk19:18
mriedem*pick19:19
superdanwell,19:19
superdanif there is not request in the reqspec that prevents that, then I would say that's fine19:19
superdanbut you'd have to have block migration for that, which I think is automatic now,right?19:19
mriedemright i think it's fine from a scheduler pov19:19
superdancould be a thing with a hypervisor needing shared storage all te time for a migration though, if that's what you mean19:20
mriedemwe don't consult block vs shared migration in the scheduler to determine which compute nodes to use based on if they have a shared storage provider aggregate19:20
*** thorst has quit IRC19:20
mriedemi was just thinking resize/cold migrate19:20
*** thorst has joined #openstack-nova19:20
mriedemcould move the instance from a source node using shared storage to a dest node using locak disk19:20
mriedem*local19:21
mriedemwhich i think is ok19:21
superdanyeah'19:21
mriedemit's just that things in the report client trying to sort this mess out is going to be...messy19:21
fried_riceedmondsw_ Looking at https://review.openstack.org/#/c/485121 - I think the answer is yes.  Stay tuned.19:21
edmondsw_fried_rice tx19:22
superdanwell, as leakypipes has said, we probably need to be providing more info the scheduler in those situations to account for some of this19:22
superdanbut yteah19:22
*** krtaylor has quit IRC19:29
*** slaweq has quit IRC19:32
*** slaweq has joined #openstack-nova19:33
fried_riceedmondsw_ Checkacheckacheck it out.19:33
cfriesensuperdan: mriedem: I think that if the end-user hasn't explicitly requested shared storage than doing a block migration to/from a compute with local storage is fine19:34
cfriesenso I'm not sure the scheduler needs to care about the storage the instance is currently on19:35
cfriesenunless the hypervisor physically can't handle that case19:35
*** rajathagasthya has quit IRC19:37
*** slaweq has quit IRC19:37
edmondsw_fried_rice ty sir19:38
*** rajathagasthya has joined #openstack-nova19:39
*** randomhack has joined #openstack-nova19:39
*** krtaylor has joined #openstack-nova19:43
*** randomhack has quit IRC19:44
cfriesendo we even have a way to say "I want to be on a hypervisor with shared storage for instance "localdisk"?19:46
cfriesenalmost seems like that would be a host-aggregate thing19:46
*** itlinux has quit IRC19:49
*** lpetrut has joined #openstack-nova19:55
*** rajathagasthya has quit IRC19:56
sean-k-mooneycfriesen: are you assuming that if the tenant wants there instance to be backed by shared storage then they would request the instance to be backed by a cinder volume19:57
*** rajathagasthya has joined #openstack-nova19:58
superdanleakypipes: you're going to look at my split of your patch right?19:59
*** dfisher has left #openstack-nova19:59
cfriesensean-k-mooney: either that or they would specify a flavor that the admin has set up to map to a host aggregate that has shared storage.20:11
cfriesenI can't think of any other way to request shared-storage in a mixed shared/local cloud20:12
sean-k-mooneycfriesen: the issue with that is its not programatically  discoverable unless we used traits to model that20:13
cfriesensean-k-mooney: right.  so we don't currently support the scenario where we happen to schedule on a hypervisor with shared storage that doesn't support block-migrating to non-shared storage.  (are there such hypervisors?)20:14
*** kristian__ has quit IRC20:15
sean-k-mooneycfriesen: does that work for ceph to non ceph today?20:15
cfriesensean-k-mooney: I think it'd be a question of whether qemu supports it20:15
sean-k-mooneycfriesen: you can configure nova to always use ceph on one node without the tenant requesting it but not sure if you can do a block migration to another host20:16
openstackgerritDan Smith proposed openstack/nova master: Resource tracker compatibility with Ocata and Pike  https://review.openstack.org/49101220:16
cfriesensean-k-mooney: yeah, I don't know if it works or not20:17
sean-k-mooneycfriesen: well even if qemu could i dont think anything would endup deleteing the ceph voluome in that case unless maybe libvirt would?20:17
openstackgerritMerged openstack/nova master: Fix getting instance bdms in multiple cells  https://review.openstack.org/49034020:17
cfriesensean-k-mooney: no clue.  I'd hope it'd be covered by the normal post-live-migration cleanup on the source side.20:18
sean-k-mooneyit proably is not a good idea to mix nova with default ceph stoage and nova with default local in the same availablity zone in anycase20:18
* sean-k-mooney not sure they want to go down that rabbit hole and test it20:19
*** slaweq has joined #openstack-nova20:19
cfriesencertainly that'd be the simplest option....."don't do that"20:19
sean-k-mooneywell if you want to keep your openstack sysadmins sane mixing default storage configs in the same availablity zone is certenly not the way to go20:21
*** kristian__ has joined #openstack-nova20:22
*** kristian__ has quit IRC20:26
*** sree has joined #openstack-nova20:30
*** sree has quit IRC20:35
-openstackstatus- NOTICE: Gerrit is being restarted to pick up CSS changes and should be back momentarily20:36
openstackgerritEric Fried proposed openstack/nova master: nova.utils.get_ksa_adapter()  https://review.openstack.org/48813720:43
leakypipessuperdan: just back from doctor... have you pushed the split yet?20:47
superdanleakypipes: yeah dude.. but I just realized there's still a functional failure in the middle, so fixing that up,20:47
leakypipessuperdan: gotcha. just ping me when ready.20:48
superdanleakypipes: you could still look, it's just a minor thing20:48
leakypipesah, k20:48
leakypipeslooking now20:48
superdanleakypipes: just to see if you're okay with it or I missed something major20:48
leakypipesk20:48
*** randomhack has joined #openstack-nova20:50
*** smatzek has quit IRC20:50
*** lucasxu has quit IRC20:51
superdanah just a thing we can't assert reliably in your patch, so I'm commenting that and then the following one will assert it always20:53
superdanfix coming in a sec20:53
*** mingyu has joined #openstack-nova20:54
*** krtaylor has quit IRC20:54
openstackgerritDan Smith proposed openstack/nova master: remove provider allocs in confirm/revert resize  https://review.openstack.org/48851020:55
openstackgerritDan Smith proposed openstack/nova master: Resource tracker compatibility with Ocata and Pike  https://review.openstack.org/49101220:55
*** ijw has joined #openstack-nova20:56
leakypipessuperdan: see my comment on previous patch about the min service version mock that will want to move to the ocata computes patch...20:56
superdanokay20:56
superdanI did this super mechanically so not surprised to miss something like that20:56
leakypipessuperdan: ya, no worries. I annotated what needs to move to the next patch.20:57
leakypipessuperdan: other than that, looks like a good split.20:57
*** ijw has quit IRC20:57
*** ijw has joined #openstack-nova20:57
leakypipessuperdan: maybe add me as co-author on that ocata patch, too? :)20:58
superdanoh sure20:58
superdansorry20:58
leakypipesno worries :)20:58
*** mingyu has quit IRC20:59
leakypipesthese patches have been through the proverbial ringer.20:59
*** sambetts_ has quit IRC21:02
openstackgerritDan Smith proposed openstack/nova master: remove provider allocs in confirm/revert resize  https://review.openstack.org/48851021:02
openstackgerritDan Smith proposed openstack/nova master: Resource tracker compatibility with Ocata and Pike  https://review.openstack.org/49101221:02
* superdan leakypipes ^21:02
*** Apoorva_ has joined #openstack-nova21:03
*** sambetts_ has joined #openstack-nova21:06
*** Apoorva has quit IRC21:07
*** slaweq has quit IRC21:07
*** sdague has quit IRC21:08
*** slaweq has joined #openstack-nova21:08
*** imacdonn has quit IRC21:08
*** imacdonn has joined #openstack-nova21:09
*** krtaylor has joined #openstack-nova21:09
mriedemi'm about to dump a pile on you two also21:10
mriedemno pair programming over here for this shared storage mess21:11
mriedemi'm about to give up and go see how pied piper is doing21:11
*** slaweq has quit IRC21:12
*** krtaylor has quit IRC21:14
*** randomhack has quit IRC21:14
* smcginnis needs to catch up on SV21:15
*** itlinux has joined #openstack-nova21:15
*** Apoorva_ has quit IRC21:16
*** Apoorva has joined #openstack-nova21:16
openstackgerritMatt Riedemann proposed openstack/nova master: WIP: Add functional resize tests using shared storage  https://review.openstack.org/49073321:17
openstackgerritMatt Riedemann proposed openstack/nova master: WIP: Account for shared storage in the report client  https://review.openstack.org/49109821:17
mriedemsuperdan: leakypipes: ^ gets all sorts of gross and several unhandled cases21:17
mriedemduring a move21:17
*** crushil has quit IRC21:20
*** thorst has quit IRC21:21
*** thorst has joined #openstack-nova21:22
leakypipessuperdan: ++ from me on those split patches.21:25
leakypipesmriedem: ok, will look at those later this evening. thanks for the heads up.21:26
*** thorst has quit IRC21:26
*** krtaylor has joined #openstack-nova21:27
*** cfriesen has quit IRC21:28
*** priteau has quit IRC21:34
mriedemoh i see where i was confused by something in the tests - when we 'double stuff' in the scheduler during the move claim, we keep the source and dest compute node allocations along with the shared storage provider allocation, but we don't sum the shared storage allocation, like we would if we resized to the same host21:36
mriedemnoticed that from this test21:36
mriedemhttps://review.openstack.org/#/c/490085/7/nova/tests/unit/scheduler/client/test_report.py@36121:36
*** lpetrut has quit IRC21:36
mriedemsuperdan: when we were talking about handling resize to same host the other day for https://review.openstack.org/#/c/490085/ didn't we say we thought that we should sum the disk even if it's shared?21:37
mriedemyou can see that sum for disk happen on the same host in test_claim_resources_success_resize_to_same_host_with_shared21:38
*** dave-mccowan has quit IRC21:38
*** baoli has quit IRC21:38
mriedembut we don't sum disk if it's on different hosts, from test_claim_resources_success_move_operation_with_shared21:38
*** kylek3h has quit IRC21:38
*** edmondsw_ has quit IRC21:47
*** gouthamr has quit IRC21:47
*** felipemonteiro_ has quit IRC21:48
*** leakypipes has quit IRC21:48
openstackgerritEric Fried proposed openstack/nova master: WIP: Get auth from context for glance endpoint  https://review.openstack.org/49005721:56
*** zz9pzza has joined #openstack-nova22:00
*** yamamoto has joined #openstack-nova22:00
*** zz9pzza has quit IRC22:06
*** burt has quit IRC22:06
*** JoseMello has quit IRC22:06
*** yassine has quit IRC22:09
*** yassine has joined #openstack-nova22:11
openstackgerritEric Fried proposed openstack/nova master: Get auth from context for glance endpoint  https://review.openstack.org/49005722:11
fried_ricemordred ^^ I *think* will pass tox now, and I *think* will pass sdague's https://review.openstack.org/49003122:14
*** fried_rice is now known as efried_WEEKEND22:16
*** ijw has quit IRC22:21
*** thorst has joined #openstack-nova22:22
*** marst_ has quit IRC22:24
*** krtaylor has quit IRC22:26
*** thorst has quit IRC22:28
*** esberglu has quit IRC22:28
*** esberglu has joined #openstack-nova22:29
superdanmriedem: not sure what you mean.. we don't sum anything for different hosts22:31
*** sree has joined #openstack-nova22:31
superdanmriedem: you saying we don't claim against the shared storage again?22:31
*** esberglu has quit IRC22:33
*** sree has quit IRC22:36
*** rajathagasthya has quit IRC22:37
*** yamamoto has quit IRC22:41
*** figleaf is now known as edleafe22:41
*** mingyu has joined #openstack-nova22:43
*** mingyu has quit IRC22:47
*** efried_WEEKEND has quit IRC22:48
*** baoli has joined #openstack-nova22:52
*** superdan is now known as dansmith22:52
*** xyang1 has quit IRC22:53
*** catintheroof has quit IRC22:56
*** rajathagasthya has joined #openstack-nova22:57
*** efried_WEEKEND has joined #openstack-nova22:58
*** rajathagasthya has quit IRC22:59
*** rajathagasthya has joined #openstack-nova22:59
*** yamamoto has joined #openstack-nova23:09
*** randomhack has joined #openstack-nova23:10
*** ijw has joined #openstack-nova23:12
*** Apoorva has quit IRC23:13
*** lyan has quit IRC23:13
*** Apoorva has joined #openstack-nova23:14
*** randomhack has quit IRC23:15
*** ijw has quit IRC23:17
*** jamesden_ has quit IRC23:17
*** hongbin has quit IRC23:19
*** jamesdenton has joined #openstack-nova23:21
*** lbragstad has quit IRC23:22
*** gouthamr has joined #openstack-nova23:32
*** ijw has joined #openstack-nova23:33
*** edmondsw has joined #openstack-nova23:36
*** edmondsw has quit IRC23:40
*** rajathagasthya has quit IRC23:50

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