Thursday, 2017-09-21

*** slaweq has joined #openstack-nova00:02
*** namnh has quit IRC00:04
*** slaweq has quit IRC00:06
*** gyee has quit IRC00:07
*** Apoorva_ has quit IRC00:08
*** Apoorva has joined #openstack-nova00:08
*** cdent has quit IRC00:10
*** ociuhandu_ has joined #openstack-nova00:12
*** ociuhandu has quit IRC00:12
*** Sukhdev_ has quit IRC00:13
*** thorst has joined #openstack-nova00:16
*** itlinux has quit IRC00:16
*** takedakn has joined #openstack-nova00:18
*** takedakn has quit IRC00:19
*** takedakn has joined #openstack-nova00:21
*** ijw has quit IRC00:23
*** ijw has joined #openstack-nova00:23
*** sbezverk has quit IRC00:25
*** sbezverk has joined #openstack-nova00:26
*** jmlowe has joined #openstack-nova00:27
*** bigdogstl has joined #openstack-nova00:29
*** Apoorva_ has joined #openstack-nova00:30
*** zhurong has joined #openstack-nova00:31
*** slaweq has joined #openstack-nova00:32
*** Apoorva has quit IRC00:34
*** Apoorva_ has quit IRC00:34
*** bigdogstl has quit IRC00:36
*** hshiina has joined #openstack-nova00:36
*** baoli has joined #openstack-nova00:36
*** baoli has quit IRC00:36
*** slaweq has quit IRC00:36
*** itlinux has joined #openstack-nova00:38
openstackgerritOpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements  https://review.openstack.org/50583900:39
*** openstackgerrit has quit IRC00:41
*** openstackgerrit has joined #openstack-nova00:41
openstackgerritOpenStack Proposal Bot proposed openstack/os-traits master: Updated from global requirements  https://review.openstack.org/50364600:41
openstackgerritOpenStack Proposal Bot proposed openstack/os-vif master: Updated from global requirements  https://review.openstack.org/50270800:41
*** slaweq has joined #openstack-nova00:42
*** Swami has quit IRC00:45
*** slaweq has quit IRC00:46
*** Sree has joined #openstack-nova00:48
*** TuanLA has joined #openstack-nova00:50
*** slaweq has joined #openstack-nova00:51
*** Sree has quit IRC00:52
*** bigdogstl has joined #openstack-nova00:53
*** thorst has quit IRC00:54
*** litao__ has joined #openstack-nova00:54
*** jichen has joined #openstack-nova00:54
*** kenperkins has quit IRC00:56
*** slaweq has quit IRC00:56
*** hoonetorg has joined #openstack-nova00:56
*** takedakn has quit IRC00:57
*** slaweq has joined #openstack-nova01:01
openstackgerritOpenStack Proposal Bot proposed openstack/os-vif stable/pike: Updated from global requirements  https://review.openstack.org/49314601:04
openstackgerritOpenStack Proposal Bot proposed openstack/python-novaclient stable/pike: Updated from global requirements  https://review.openstack.org/49318701:06
*** gbarros has joined #openstack-nova01:07
*** phuongnh has joined #openstack-nova01:07
*** slaweq has quit IRC01:08
*** bigdogstl has quit IRC01:15
*** trungnv has joined #openstack-nova01:16
*** gbarros has quit IRC01:18
*** dave-mccowan has joined #openstack-nova01:22
mriedemowalsh: whenever you get a sec, if you know how to configure nova and cells v2 for clustered rabbit with the cell transport_url, it would be cool if you could help sort this out https://bugs.launchpad.net/nova/+bug/171791501:23
openstackLaunchpad bug 1717915 in oslo.messaging "nova services and transport_url, cannot connect to vhost if specified" [Undecided,New]01:23
*** READ10 has joined #openstack-nova01:25
*** suresh12 has quit IRC01:28
*** itlinux has quit IRC01:28
openstackgerritjichenjc proposed openstack/nova master: Add quiesce and unquiesce in support matrix  https://review.openstack.org/48239001:30
*** kevi9132 has joined #openstack-nova01:32
*** kevi9132_ has quit IRC01:32
openstackgerritjichenjc proposed openstack/nova master: check query param for service's index function  https://review.openstack.org/48949201:33
*** dave-mcc_ has joined #openstack-nova01:34
*** dave-mccowan has quit IRC01:35
openstackgerritMatt Riedemann proposed openstack/nova stable/pike: Split out the core of the ironic flavor migration  https://review.openstack.org/50590101:36
openstackgerritMatt Riedemann proposed openstack/nova stable/pike: Add ComputeNodeList.get_by_hypervisor_type()  https://review.openstack.org/50590201:36
openstackgerritMatt Riedemann proposed openstack/nova stable/pike: Add nova-manage db command for ironic flavor migrations  https://review.openstack.org/50590301:36
*** Sukhdev has quit IRC01:39
*** jmlowe has quit IRC01:42
*** jmlowe has joined #openstack-nova01:43
openstackgerritjichenjc proposed openstack/nova master: Remove 400 as expected error  https://review.openstack.org/50548501:45
*** yamahata has quit IRC01:46
*** itlinux has joined #openstack-nova01:49
*** mriedem has quit IRC01:51
*** itlinux has quit IRC01:51
*** kevi9132_ has joined #openstack-nova01:52
*** mingyu has quit IRC01:53
*** thorst has joined #openstack-nova01:53
*** thorst has quit IRC01:53
*** crushil has joined #openstack-nova01:54
*** kevi9132 has quit IRC01:55
*** amotoki has quit IRC01:55
*** amotoki has joined #openstack-nova01:56
*** hamzy has quit IRC01:57
*** hamzy has joined #openstack-nova01:57
*** ijw has quit IRC01:58
*** mingyu has joined #openstack-nova01:59
*** kaisers_ has quit IRC02:00
*** ijw has joined #openstack-nova02:01
openstackgerritMerged openstack/nova master: Optimize MiniDNS for fewer syscalls  https://review.openstack.org/48682902:01
openstackgerritjichenjc proposed openstack/nova master: Fix warning on {'cell_id': 1} is an invalid UUID  https://review.openstack.org/50547102:01
*** slaweq has joined #openstack-nova02:03
*** hongbin has joined #openstack-nova02:04
*** ijw has quit IRC02:05
*** slaweq has quit IRC02:08
*** Shunli has joined #openstack-nova02:10
*** READ10 has quit IRC02:12
*** trungnv has quit IRC02:16
*** gouthamr has joined #openstack-nova02:19
*** crushil has quit IRC02:20
*** dave-mcc_ has quit IRC02:23
*** lihg has joined #openstack-nova02:24
*** lihg_ has joined #openstack-nova02:25
*** lihg has quit IRC02:25
*** lihg_ has quit IRC02:25
*** lihg has joined #openstack-nova02:25
*** lihg_ has joined #openstack-nova02:25
*** bigdogstl has joined #openstack-nova02:25
*** lihg_ has left #openstack-nova02:26
*** trungnv has joined #openstack-nova02:28
*** efried has quit IRC02:31
*** efried has joined #openstack-nova02:32
*** bigdogstl has quit IRC02:33
masbergood morning all, I would like to run docker containers through nova, my environment is based on Newton. Is this the guide I need to follow? https://wiki.openstack.org/wiki/Docker02:36
*** vladikr has quit IRC02:41
*** vladikr has joined #openstack-nova02:41
*** awaugama has joined #openstack-nova02:44
*** mingyu has quit IRC02:45
*** vladikr has quit IRC02:47
*** mingyu has joined #openstack-nova02:47
*** vladikr has joined #openstack-nova02:47
*** mingyu has quit IRC02:48
*** coreywright has quit IRC02:49
*** kevi9132_ has quit IRC02:51
*** kevi9132 has joined #openstack-nova02:52
*** coreywright has joined #openstack-nova02:52
*** vladikr has quit IRC02:52
*** vladikr has joined #openstack-nova02:53
*** thorst has joined #openstack-nova02:53
*** awaugama has quit IRC02:56
*** kevi9132 has quit IRC02:56
*** thorst has quit IRC02:59
*** mfisch` is now known as mfisch02:59
*** mfisch is now known as Guest4423403:00
*** lbragstad has joined #openstack-nova03:00
*** lihg__ has joined #openstack-nova03:01
*** lihg has quit IRC03:01
*** baoli has joined #openstack-nova03:01
*** lihg has joined #openstack-nova03:02
*** slaweq has joined #openstack-nova03:02
gmannjichen:  sorry about api-ref comment which is not valid - https://review.openstack.org/#/c/505485/03:07
gmannjichen: your PS3 was good to go03:07
*** slaweq has quit IRC03:07
jichengmann: ok, got it, you are right, I need keep 400 as validation stuff, thanks :)03:08
gmannjichen: thanks03:09
*** vladikr has quit IRC03:10
openstackgerritOpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements  https://review.openstack.org/50583903:10
*** vladikr has joined #openstack-nova03:11
openstackgerritOpenStack Proposal Bot proposed openstack/os-traits master: Updated from global requirements  https://review.openstack.org/50364603:13
openstackgerritOpenStack Proposal Bot proposed openstack/os-vif master: Updated from global requirements  https://review.openstack.org/50270803:13
openstackgerritjichenjc proposed openstack/nova master: Remove 400 as expected error  https://review.openstack.org/50548503:15
*** lbragstad has quit IRC03:21
*** armax has quit IRC03:22
*** baoli has quit IRC03:23
*** Sukhdev has joined #openstack-nova03:24
openstackgerritOpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements  https://review.openstack.org/50583903:27
openstackgerritOpenStack Proposal Bot proposed openstack/os-traits master: Updated from global requirements  https://review.openstack.org/50364603:29
openstackgerritOpenStack Proposal Bot proposed openstack/os-vif master: Updated from global requirements  https://review.openstack.org/50270803:29
*** bigdogstl has joined #openstack-nova03:31
*** namnh has joined #openstack-nova03:34
*** bigdogstl has quit IRC03:35
*** trinaths has joined #openstack-nova03:39
*** udesale has joined #openstack-nova03:41
*** trinaths1 has joined #openstack-nova03:41
*** hshiina has quit IRC03:42
*** slaweq has joined #openstack-nova03:42
*** cfriesen has quit IRC03:43
*** trinaths has quit IRC03:44
openstackgerritOpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements  https://review.openstack.org/50583903:45
*** hshiina has joined #openstack-nova03:46
*** Sree has joined #openstack-nova03:46
*** slaweq has quit IRC03:47
openstackgerritOpenStack Proposal Bot proposed openstack/os-traits master: Updated from global requirements  https://review.openstack.org/50364603:48
openstackgerritOpenStack Proposal Bot proposed openstack/os-vif master: Updated from global requirements  https://review.openstack.org/50270803:48
*** yamahata has joined #openstack-nova03:49
*** slaweq has joined #openstack-nova03:52
*** Guest44234 is now known as mfisch03:52
*** mfisch is now known as Guest1344603:53
openstackgerritTakashi NATSUME proposed openstack/nova master: Enable cold migration with target host(1/2)  https://review.openstack.org/40895503:54
*** thorst has joined #openstack-nova03:55
*** suresh12 has joined #openstack-nova03:55
*** suresh12 has quit IRC03:56
*** slaweq has quit IRC03:56
*** takedakn has joined #openstack-nova03:58
*** thorst has quit IRC03:59
*** bigdogstl has joined #openstack-nova04:00
*** takashin has joined #openstack-nova04:03
*** erlon has quit IRC04:03
openstackgerritTakashi NATSUME proposed openstack/nova master: Enable cold migration with target host(2/2)  https://review.openstack.org/40896404:05
*** hongbin has quit IRC04:06
*** kbaegis has quit IRC04:08
*** kenperkins has joined #openstack-nova04:08
*** bigdogstl has quit IRC04:11
*** gouthamr has quit IRC04:12
*** takedakn has quit IRC04:16
*** chyka has quit IRC04:17
*** chyka has joined #openstack-nova04:18
*** chyka has quit IRC04:22
*** bigdogstl has joined #openstack-nova04:38
*** claudiub|2 has joined #openstack-nova04:40
*** psachin has joined #openstack-nova04:41
*** bigdogstl has quit IRC04:43
*** links has joined #openstack-nova04:43
*** thorst has joined #openstack-nova04:55
*** mdnadeem has joined #openstack-nova04:56
*** itlinux has joined #openstack-nova05:00
*** thorst has quit IRC05:00
*** shan has joined #openstack-nova05:06
*** sridharg has joined #openstack-nova05:09
*** pcaruana has joined #openstack-nova05:09
*** markvoelker has quit IRC05:16
*** markvoelker has joined #openstack-nova05:17
*** markvoelker has quit IRC05:22
*** Sukhdev has quit IRC05:23
openstackgerritTakashi NATSUME proposed openstack/nova-specs master: List/show all server migration types  https://review.openstack.org/48902905:27
*** udesale__ has joined #openstack-nova05:35
*** ratailor has joined #openstack-nova05:36
openstackgerritHironori Shiina proposed openstack/nova-specs master: Ironic: Resize and cold migration support  https://review.openstack.org/44915505:36
*** udesale has quit IRC05:38
*** bigdogstl has joined #openstack-nova05:40
*** bigdogstl has quit IRC05:44
*** udesale has joined #openstack-nova05:50
jichengmann: could you please help check https://review.openstack.org/#/c/505485/ ? I think gerrit has some issue05:50
*** slaweq has joined #openstack-nova05:51
jichenso PS5's update didn't take effect and your PS4 -1 is still there..05:51
*** udesale__ has quit IRC05:52
gmannjichen: done.05:52
jichengmann: thanks~05:52
gmannjichen: about your mail for PUT metadata/key . i thought you are asking about PUT metadata by key and will it allow to update more than 1 key or not05:53
*** lihg_ has joined #openstack-nova05:53
gmannjichen: if you see my reply actually that is handled in schema05:54
*** lihg has quit IRC05:54
*** lihg__ has quit IRC05:54
*** lihg has joined #openstack-nova05:54
*** slaweq has quit IRC05:55
gmannthe schema and delete flag is actually very kind of hidden in that and does all magic05:55
jichenyes, that's also my question , I just saw the mail, yes, exactly, it's hidden in the code logic and I missed that05:56
jichengmann: thanks for point that out and I think that's a little bit hack, we have update_all which can handle one key situation05:56
*** thorst has joined #openstack-nova05:56
jichenI mean we have update_all already and update one key like the API implemented looks to me seems useless and confusing05:57
*** rmcall has quit IRC05:57
gmannjichen: but both have separate purpose, update_all remove the existing keys and update by key does not05:57
jichengmann: ah...ok , you are right05:58
jichenthanks for the info, I will check the code again , thanks~05:59
*** thorst has quit IRC06:01
*** rmcall has joined #openstack-nova06:02
*** avolkov has joined #openstack-nova06:02
*** vito_ has joined #openstack-nova06:04
*** vito_ is now known as Guest9279606:04
*** bigdogstl has joined #openstack-nova06:05
*** rmcall has quit IRC06:06
*** coreywright has quit IRC06:08
*** bigdogstl has quit IRC06:09
*** claudiub has joined #openstack-nova06:10
*** claudiub|2 has quit IRC06:12
*** Oku_OS-away is now known as Oku_OS06:13
*** kenperkins has quit IRC06:15
openstackgerritAlex Xu proposed openstack/nova-specs master: Add trait support in the allocation candidates API  https://review.openstack.org/49771306:16
*** lihg__ has joined #openstack-nova06:16
*** lihg_ has quit IRC06:17
*** lihg has quit IRC06:17
*** lihg has joined #openstack-nova06:18
*** andreas_s has joined #openstack-nova06:21
*** Guest92796 has quit IRC06:25
*** josecastroleon has joined #openstack-nova06:27
*** rcernin has joined #openstack-nova06:28
*** udesale has quit IRC06:29
*** udesale has joined #openstack-nova06:30
*** coreywright has joined #openstack-nova06:30
*** lajoskatona has joined #openstack-nova06:32
*** itlinux has quit IRC06:37
*** lihg has quit IRC06:38
*** lihg_ has joined #openstack-nova06:38
*** lihg__ has quit IRC06:39
*** lihg has joined #openstack-nova06:39
*** itlinux has joined #openstack-nova06:40
*** itlinux has quit IRC06:40
*** edand has joined #openstack-nova06:40
*** ralonsoh has joined #openstack-nova06:45
*** kevi9132 has joined #openstack-nova06:53
*** thorst has joined #openstack-nova06:57
*** xinliang has quit IRC06:57
*** kevi9132 has quit IRC06:58
*** thorst has quit IRC07:02
*** pcaruana has quit IRC07:05
*** sahid has joined #openstack-nova07:08
*** ioggstream has joined #openstack-nova07:09
*** xinliang has joined #openstack-nova07:10
*** mingyu has joined #openstack-nova07:13
*** mingyu has quit IRC07:14
*** ChanServ sets mode: +o openstack07:15
*** bigdogstl has joined #openstack-nova07:17
*** markvoelker has joined #openstack-nova07:18
*** sapcc-bot has quit IRC07:18
*** sapcc-bot1 has joined #openstack-nova07:18
*** NehaAlhat has joined #openstack-nova07:19
*** tesseract has joined #openstack-nova07:19
*** bigdogstl has quit IRC07:22
*** neha_alhat has quit IRC07:22
*** trungnv has quit IRC07:22
*** pcaruana has joined #openstack-nova07:23
*** alexchadin has joined #openstack-nova07:29
*** slaweq has joined #openstack-nova07:30
*** sridharg has quit IRC07:31
*** sridharg has joined #openstack-nova07:32
*** slaweq has quit IRC07:34
*** zhurong has quit IRC07:35
*** Tom_ has joined #openstack-nova07:37
*** chyka has joined #openstack-nova07:39
*** slaweq has joined #openstack-nova07:40
*** jpena|off is now known as jpena07:41
*** chyka has quit IRC07:43
*** slaweq has quit IRC07:44
openstackgerritYikun Jiang proposed openstack/nova-specs master: Add pagination and timestamp filtering support for os-migrations API  https://review.openstack.org/50603007:48
*** slaweq has joined #openstack-nova07:50
*** trinaths1 has quit IRC07:50
*** markvoelker has quit IRC07:52
*** slaweq has quit IRC07:54
*** zhurong has joined #openstack-nova07:55
gibigood morning nova07:56
*** thorst has joined #openstack-nova07:58
*** Sree has quit IRC08:02
openstackgerritYikun Jiang proposed openstack/nova-specs master: Add pagination and timestamp filtering support for os-migrations API  https://review.openstack.org/50603008:02
*** Sree has joined #openstack-nova08:02
*** thorst has quit IRC08:02
*** lajoskatona has quit IRC08:11
openstackgerritClaudiu Belu proposed openstack/nova-specs master: Adds spec for instance live resize  https://review.openstack.org/14121908:11
jichenclaudiub: hi08:17
claudiubjichen: hello :)08:18
jichenappreciate you restore the spec, we have a set of customer requirements so hope we can contribute to the spec as well :)08:18
*** lihg has quit IRC08:18
*** lihg_ has quit IRC08:18
claudiubjichen: sure, what's up?08:18
*** slaweq has joined #openstack-nova08:19
openstackgerritJames Page proposed openstack/nova master: Support qemu >= 2.10  https://review.openstack.org/50574808:19
mdboothIf anybody uses os-log-merger, I posted a patch series for it yesterday which fixes a couple of bugbears. Top of the series is here: https://review.openstack.org/#/c/505757/08:20
jichenclaudiub: should we update the spec to indicate Matt's comments or you already did the update?08:20
claudiubjichen: i didn't include that bit yet.08:21
claudiubi'll update it08:21
mdboothHighlight is that it now does log format auto-detection, as well as parsing qemu logs.08:21
jichenclaudiub: ok, great, thanks ~08:21
* mdbooth gets back to actually debugging this customer live migration issue, rather than faffing with tooling08:21
claudiubjichen: do you have any particular requirements regarding live-resize, that haven't been included in the spec?08:22
jichenclaudiub: no, actually our first target is to resize the cpu/memory first , If I remember clearly that's already included , we can add follow up if we have other use cases08:23
*** slaweq has quit IRC08:24
*** nmathew has joined #openstack-nova08:24
*** bigdogstl has joined #openstack-nova08:28
openstackgerritClaudiu Belu proposed openstack/nova-specs master: Adds spec for instance live resize  https://review.openstack.org/14121908:29
claudiubjichen: which driver are you most interested in having the live-resize feature?08:29
jichenclaudiub: we have a out of tree driver (z/VM) and we are trying to get in tree soon, also I have some colleagues (libvirt on s390) etc also interested in this feature08:31
openstackgerritStephen Finucane proposed openstack/nova master: Restore '[vnc] vnc_*' option support  https://review.openstack.org/50583108:31
stephenfinefried, alex_xu, mriedem: ^ Fixed that 'vnc_' opt issue08:31
claudiubjichen: i see. I'm asking because one of the requirements for the feature to get approved was to be implemented in all / most drivers. Since so much time passed since it was last proposed, we'd have to find candidates for each driver once again. :)08:33
*** edand has quit IRC08:33
*** bigdogstl has quit IRC08:34
jichenclaudiub: ok, this might take some time :(08:34
claudiubjichen: well, for hyper-v, i'm still up for it. the nova cores won't ask for it to be implemented for out-of-tree drivers, but I assume you'll volunteer for z/VM, but what about libvirt?08:36
claudiubthey'll mostly care about the libvirt implementation, btw.08:36
jichenclaudiub: let me find whether we have internal guys (IBM) want to do it , or can we post ML to ask for volunteer ? as except libvirt driver, there's a lot of things to be done such as API which I can contribute as well08:40
*** derekh has joined #openstack-nova08:43
*** lajoskatona has joined #openstack-nova08:43
claudiubjichen: sounds good. we could restore, rebase and reuse whatever code was submitted for the feature08:43
jichenclaudiub: ok, I knew a patch set there , and we may first try to get the spec accepted :)08:44
claudiubjichen: yeah. :) maybe the 4th time is the charm. :)08:45
jichenclaudiub: ok, let's push it again and see what we can do08:45
*** markvoelker has joined #openstack-nova08:48
*** kevi9132 has joined #openstack-nova08:55
*** thorst has joined #openstack-nova08:59
*** kevi9132 has quit IRC08:59
openstackgerritYikun Jiang proposed openstack/nova-specs master: Add pagination and timestamp filtering support for os-migrations API  https://review.openstack.org/50603009:01
*** lajoskatona has quit IRC09:03
*** thorst has quit IRC09:03
*** trinaths1 has joined #openstack-nova09:06
*** hshiina has quit IRC09:06
*** lihg has joined #openstack-nova09:07
*** lihg_ has joined #openstack-nova09:08
ioggstreamhi@all09:09
*** chyka has joined #openstack-nova09:10
*** bigdogstl has joined #openstack-nova09:14
*** chyka has quit IRC09:14
*** edmondsw has joined #openstack-nova09:15
*** bigdogstl has quit IRC09:19
*** edmondsw has quit IRC09:20
*** liverpooler has quit IRC09:21
*** markvoelker has quit IRC09:21
*** yamamoto has quit IRC09:21
*** yamamoto has joined #openstack-nova09:22
*** tejaswi has joined #openstack-nova09:22
tejaswihi all. I have a OpenStack Newton setup on Ubuntu 16.04. I am looking for instructions on how to do CPU pinning. However, I did not find any. Please share any links.09:23
ioggstreamtejaswi: which newton distribution?09:24
tejaswiioggstream: standard openstack distribution - https://docs.openstack.org/newton/install-guide-ubuntu/09:26
ioggstreamtejaswi: I just cover rhosp, but check if this can help https://access.redhat.com/documentation/en-us/red_hat_openstack_platform/10/html/instances_and_images_guide/ch-cpu_pinning in some way09:28
tejaswiioggstream: I tried those instructions suitable for Ubuntu . However, the server got stuck when I changed grub and rebooted it09:31
*** Shunli has quit IRC09:31
ioggstreamtejaswi: I think you should use the content of that doc to search similar references to the ubuntu counterpart09:32
tejaswiioggstream: Yes, I already tried something similar.09:32
*** alexchadin has quit IRC09:34
*** alexchadin has joined #openstack-nova09:35
*** lihg has quit IRC09:35
*** lihg_ has quit IRC09:35
*** sambetts|afk is now known as sambetts09:36
*** slaweq has joined #openstack-nova09:37
openstackgerritRadoslav Gerganov proposed openstack/nova master: Add connection pool size to vSphere settings  https://review.openstack.org/41408509:37
*** alexchadin has quit IRC09:38
tejaswiioggstream: I did not find anything similar for Ubuntu. So, I am just checking if anyone did CPU pinning in Ubuntu OpenStack09:39
*** tesseract has quit IRC09:41
*** tesseract has joined #openstack-nova09:41
openstackgerritSylvain Bauza proposed openstack/nova master: Add a regression test for bug 1718455  https://review.openstack.org/50609209:42
openstackbug 1718455 in OpenStack Compute (nova) pike "[pike] Nova host disable and Live Migrate all instances fail." [Medium,Confirmed] https://launchpad.net/bugs/171845509:42
openstackgerritSylvain Bauza proposed openstack/nova master: Fix definitely move single instance when created concurrently  https://review.openstack.org/50609309:42
*** jichen has quit IRC09:46
*** slaweq_ has joined #openstack-nova09:48
*** mvk has quit IRC09:49
*** syqian has joined #openstack-nova09:49
*** slaweq_ has quit IRC09:53
*** Sree has quit IRC09:54
*** zhurong has quit IRC09:55
*** thorst has joined #openstack-nova10:00
*** thorst has quit IRC10:04
*** gszasz has joined #openstack-nova10:06
*** alexchadin has joined #openstack-nova10:09
*** alexchadin has quit IRC10:10
*** zhurong has joined #openstack-nova10:11
*** udesale has quit IRC10:16
*** markvoelker has joined #openstack-nova10:19
*** mvk has joined #openstack-nova10:19
*** Tom_ has quit IRC10:27
openstackgerritJames Page proposed openstack/nova master: Support qemu >= 2.10  https://review.openstack.org/50574810:28
openstackgerritTakashi NATSUME proposed openstack/nova master: Fix hyperlinks in document  https://review.openstack.org/50610110:29
stephenfintakashin: One comment left in ^10:33
*** bigdogstl has joined #openstack-nova10:38
openstackgerritBalazs Gibizer proposed openstack/nova master: Add live.migration.force.complete to the legacy notification whitelist  https://review.openstack.org/50610410:40
*** smatzek has joined #openstack-nova10:40
*** smatzek has quit IRC10:41
*** smatzek has joined #openstack-nova10:41
gibistephenfin: it wasn't that soon I promised but here is the bugfix for the force complete legacy notitification whitelisting ^^10:42
gibistephenfin: now going to rebase the patch we had to stopped yestarday due to this10:42
openstackgerritzhangyanxian proposed openstack/nova-specs master: Sriov-bond  https://review.openstack.org/46352610:43
*** bigdogstl has quit IRC10:45
openstackgerritBalazs Gibizer proposed openstack/nova master: Add functional migrate force_complete test  https://review.openstack.org/49620210:45
openstackgerritBalazs Gibizer proposed openstack/nova master: Add functional for live migrate delete  https://review.openstack.org/49958310:46
*** tejaswi has quit IRC10:49
*** ijw has joined #openstack-nova10:50
*** markvoelker has quit IRC10:52
*** gbarros has joined #openstack-nova10:52
*** avolkov has quit IRC10:53
*** avolkov has joined #openstack-nova10:53
avolkovc10:53
*** ijw has quit IRC10:54
*** sapd has quit IRC10:59
*** Sree has joined #openstack-nova10:59
*** jpena is now known as jpena|lunch11:00
*** thorst has joined #openstack-nova11:00
*** namnh has quit IRC11:01
*** dtantsur|afk is now known as dtantsur11:02
*** Sree has quit IRC11:03
*** tbachman has quit IRC11:04
*** claudiub|2 has joined #openstack-nova11:05
*** thorst has quit IRC11:05
*** claudiub has quit IRC11:06
*** Tom has joined #openstack-nova11:07
*** trinaths1 has left #openstack-nova11:10
*** Sree has joined #openstack-nova11:12
*** ociuhandu_ has quit IRC11:15
*** tbachman has joined #openstack-nova11:21
*** ioggstream has quit IRC11:28
*** chyka has joined #openstack-nova11:30
*** sapd has joined #openstack-nova11:32
*** phuongnh has quit IRC11:32
*** nmathew has quit IRC11:32
*** ociuhandu has joined #openstack-nova11:33
*** chyka has quit IRC11:34
*** kbaegis has joined #openstack-nova11:36
*** sdague has joined #openstack-nova11:38
*** sapd has quit IRC11:39
*** syqian has quit IRC11:39
*** sapd has joined #openstack-nova11:43
*** Tom has quit IRC11:46
*** Sree has quit IRC11:46
*** Tom has joined #openstack-nova11:47
*** lajoskatona has joined #openstack-nova11:49
*** markvoelker has joined #openstack-nova11:49
*** zhurong has quit IRC11:51
*** markvoelker has quit IRC11:54
*** sapd has quit IRC11:54
*** sapd has joined #openstack-nova11:55
*** sapd has quit IRC11:56
*** sapd has joined #openstack-nova11:56
*** markvoelker has joined #openstack-nova11:56
*** litao__ has quit IRC11:59
*** thorst has joined #openstack-nova11:59
*** TuanLA has quit IRC12:00
*** udesale has joined #openstack-nova12:02
*** udesale has quit IRC12:04
*** udesale has joined #openstack-nova12:04
openstackgerritElod Illes proposed openstack/nova master: Functional test: evacuate with no compute  https://review.openstack.org/49848212:06
*** psachin has quit IRC12:06
*** kevi9132 has joined #openstack-nova12:07
*** jpena|lunch is now known as jpena12:08
*** dave-mccowan has joined #openstack-nova12:09
*** bigdogstl has joined #openstack-nova12:10
*** yamahata has quit IRC12:12
*** edmondsw has joined #openstack-nova12:13
openstackgerritElod Illes proposed openstack/nova master: Functional test: evacuate with no compute  https://review.openstack.org/49848212:19
*** shan has quit IRC12:25
*** ociuhandu has quit IRC12:25
openstackgerritSean Dague proposed openstack/nova master: Support qemu >= 2.10  https://review.openstack.org/50567312:29
*** kylek3h has quit IRC12:31
*** jaypipes has joined #openstack-nova12:33
*** artom has joined #openstack-nova12:33
*** artom has quit IRC12:34
*** lyan has joined #openstack-nova12:34
*** artom has joined #openstack-nova12:34
*** MVenesio has joined #openstack-nova12:34
*** erlon has joined #openstack-nova12:34
* jaypipes has Internet. Ahoy!12:37
edmondswjaypipes congrats!12:37
*** udesale has quit IRC12:39
*** abalutoiu has joined #openstack-nova12:41
*** gbarros has quit IRC12:41
*** catintheroof has joined #openstack-nova12:41
*** claudiub has joined #openstack-nova12:42
*** kbaegis has quit IRC12:42
jaypipesedmondsw: :) thanks!12:43
jaypipesedmondsw: $12K later... https://twitter.com/jaypipes/status/91084634796374835412:43
edmondswouch!!!12:43
*** claudiub|2 has quit IRC12:43
jaypipesedmondsw: this is what I was going in and out of the Nova room on Friday to deal with ;)12:43
jaypipesedmondsw: blessed be the A/C.12:44
edmondswjaypipes may it run well and long12:44
*** mriedem has joined #openstack-nova12:46
*** bigdogstl has quit IRC12:46
*** Sree has joined #openstack-nova12:46
jaypipesedmondsw: indeed. 12-year, full replacement warranty :)12:46
*** ociuhandu has joined #openstack-nova12:48
tonybmriedem: Can you look at: https://review.openstack.org/#/c/493187/  You had an objection in the past that I think has been dealt with12:49
*** Tom has quit IRC12:50
*** xyang1 has joined #openstack-nova12:51
jaypipestonyb: back home safe and sound? :)12:51
sdaguejaypipes: congrats on internets and AC12:51
tonybjaypipes: I'm in MA heading home tormorrow12:51
tonybjaypipes: how's the clean-up going?12:52
jaypipessdague: :) thx12:52
jaypipestonyb: well, cleanup is 90% done at this point. Just some stragglers in the back yard. In all, 34 lawn bags and about 500 cubic feet of branches and downed bushes/shrubs12:52
jaypipestonyb: oddly, the yard and the house looks better than it did when we left :) amazing how a good trimming brightens up the place ;)12:53
mriedemtonyb: so we're going to release that in stable/pike as 9.2.0 right?12:53
tonybmriedem: correct.  Sadly no other choice :(12:53
tonybjaypipes: :)12:54
mriedemwe could just not merge that change with the dependency bumps, but does that cause install issues?12:54
*** ociuhandu has quit IRC12:54
tonybmriedem: It shoudl be fine if installed with constratints, but you'd be setting us up for manual munging of all the bot generated changes on queens12:55
mriedemoh it's *my* fault now is it12:56
tonybmriedem: It's all your fault12:57
bhagyashrismriedem, johnthetubaguy: Hi, Could you please take some insights on patch https://review.openstack.org/#/c/409644/ . Thank you :)12:58
*** esberglu has joined #openstack-nova13:01
*** Tom has joined #openstack-nova13:01
bauzasjaypipes: hah, if you were not having an A/C replacement, you could just have felt like an European :p13:02
* bauzas is glad summer is over here13:03
stephenfinbauzas: You take that back. You take that back right now13:03
* stephenfin misses sunshine :'(13:03
bauzasstephenfin: well, I have sunshine here, but with decent temperatures, that's it :p13:04
*** abalutoiu has quit IRC13:04
* stephenfin is not jealous. Not even slightly...13:04
*** Tom has quit IRC13:06
mdboothmriedem: Did I see a patch from you the other day which added --force-shared to qemu-img everywhere?13:06
openstackgerritOpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements  https://review.openstack.org/50583913:07
*** pchavva has joined #openstack-nova13:08
kashyapmdbooth: From a quick `grep` of #openstack-nova IRC log, I can't find it here13:08
jaypipesbauzas: no thanks :) me and heat don't jive all that well. ;)13:08
*** gbarros has joined #openstack-nova13:08
openstackgerritOpenStack Proposal Bot proposed openstack/os-traits master: Updated from global requirements  https://review.openstack.org/50364613:09
openstackgerritOpenStack Proposal Bot proposed openstack/os-vif master: Updated from global requirements  https://review.openstack.org/50270813:09
mdboothFound it. Yesterday, 17:3413:09
mdboothhttps://review.openstack.org/#/c/505673/13:09
*** ioggstream has joined #openstack-nova13:11
efriedstephenfin Catching up.  So you're leaving the release notes alone; and you're leaving support for the deprecated [DEFAULT] versions of the opts?13:11
bauzasjaypipes: you don't imagine how much I love sweating...13:11
edmondswefried stephenfin haven't the vnc options in DEFAULT been around long enough that we could just remove deprecated_group for those?13:11
stephenfinefried: Yup. I don't see why we shouldn't13:11
stephenfinefried, edmondsw: I mean, we _could_ remove them, but it costs us basically nothing to retain them13:12
kashyapmdbooth: Ah, thanks13:12
edmondswstephenfin wasn't that always the plan when we first deprecated them?13:12
mriedemmdbooth: yes there are 2 patches13:13
edmondsweither way... the powervm CI is going to be busted until we get something merged for this13:13
stephenfinedmondsw: Yes and no. We do emit warnings that option have been renamed, but we've never gone and removed the old aliases13:13
mriedemmdbooth: https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:master+topic:bug/171829513:13
mdboothmriedem: I found 1. Interested to know how this ended up on your radar, btw.13:13
edmondswstephenfin k13:13
*** ociuhandu has joined #openstack-nova13:14
stephenfinefried: The fix I've suggested seemed far simpler and involved less other changes (fixing the the powervm CI patch, for one) so that's why I went with it13:14
mriedemmdbooth: because the live migration job shit the bed the other day when devstack was changed to use pike uca packages13:14
mriedemit's in the ML13:14
mdboothmriedem: Ah, ok.13:14
*** claudiub has quit IRC13:14
efriedstephenfin wfm.  But how long are we planning to keep the original opts around?13:15
mriedemjamespage pointed out the change in 2.1013:15
*** kylek3h has joined #openstack-nova13:15
*** andreas_s has quit IRC13:15
*** claudiub has joined #openstack-nova13:15
stephenfinThat's a question for the PTL. mriedem - do you think we should ever remove old deprecated opt aliases?13:15
stephenfinmriedem: i.e. where we moved an opt from '[DEFAULT]' to, say, '[libvirt]'13:15
efriedLast night his only complaint was the proliferation of renos :)13:15
kashyapmdbooth: It's this bug - https://bugs.launchpad.net/nova/+bug/171829513:15
openstackLaunchpad bug 1718295 in OpenStack Compute (nova) "Live migration fails with qemu-img >= 2.10: "Failed to get shared "write" lock\nIs another process using the image?"" [High,In progress] - Assigned to Sean Dague (sdague)13:16
stephenfin:D13:16
mdboothInteresting that UCA has newer libvirt/qemu than RHEL.13:16
stephenfinefried: I was going to note that you can edit a reno in the same cycle, then noticed you actually did that. I can see where he's coming from, heh13:16
*** lucasxu has joined #openstack-nova13:16
mdboothOr Fedora, even13:17
mdboothThat's pretty darned fresh13:17
sdaguemriedem: I reved the patch to pass unit tests13:17
kashyapYeah, UCA is supposed to be stable/long-term13:17
kashyapIsn't it?13:17
efriedstephenfin Yeah... I was following the instructions in the reno template which said you need to duplicate information.  x This change seemed like it was applicable to both the 'upgrade' and the 'deprecation' sections.  x I was doing both things (removing the old deprecated opt and renaming the new one).13:18
sdagueit turns out the broken hypervisor error pops a little earlier with this code path13:18
stephenfinefried: gotcha13:18
sdaguekashyap: UCA is how they get latest bits to users. For the ubuntu package they unconditionally patched it13:18
stephenfinefried: In any case, if you'd prefer to just remove the old alias then go for it. I would've proposed a separate patch altogether but I didn't want to de-emphasise your work reporting the bug/working on an initial fix13:18
sdaguebecause they know what version of qemu is required13:18
stephenfinThough I do prefer this approach, given that we've never removed deprecated aliases before #justsaying13:19
efriedstephenfin No, I think it's appropriate to split those up.  Removing the original opts would seem to be a separate effort following on from the original bp that did the split-out-of-DEFAULT in the first place.13:19
kashyapsdague: Ah, latest bits; shipping QEMU 2.10 is a bit pre-mature with OpenStack, no? (But good for finding bugs, though.)13:19
stephenfinefried: +113:19
efried"never removed deprecated aliases before" ?13:19
efriedFor what value of "never"?13:20
stephenfini.e. dropped 'deprecated_group' or 'deprecated_name' arguments13:20
*** kenperkins has joined #openstack-nova13:20
sdaguekashyap: well someone has to ship things13:20
stephenfinLike, ever, from what I can tell13:20
kashyapSure; question is not about shipping :-)  There can be different repos for bleeding edge users.13:20
efriedI'm *sure* I've seen it done.  But I'm not one to stand on principle and go find an example :)13:21
sdaguekashyap: yeh, well, the work around was easy for them. Honestly, I'm never going to ding people for deciding to run with what latest upstream things is fine. :)13:21
kashyapYep, agreed on that bit.13:21
kashyapWe still have to figure out that CI job we've been discussing on the list forever13:22
kashyapWhere latest libvirt / QEMU (from Git, newest upstream tarballs) are tested w/ Nova Git.13:22
mriedemefried: stephenfin: deprecated options can go after a release13:23
jamespagesdague, mriedem: I'm going to abandon https://review.openstack.org/#/c/505748/ - its way to brittle relying on calling code to know whether to lock or not.13:23
mriedemjamespage: ok13:23
jamespagemriedem: sdague's approach seems a bit more rugged...13:24
mriedemkashyap: you're the one that's always saying we should run against latest libvirt/qemu :)13:24
sdaguewell, mriedem actually wrote the patch13:24
kashyapmriedem: Yes, I say it to be done in a _separate_ CI job :-)13:24
mriedemoooooooooooooooooooooooooooooooooohhhhhhhhhhhhhhhhhhhhhhhhhh13:24
jamespagemust have missed that..13:24
stephenfinmriedem: This is more about the deprecated aliases, where we rename an opt or change its group, but still keep it around13:24
jamespageapols13:24
* mriedem is away for a bit13:25
stephenfinefried: Proof or it didn't happen ;)13:25
sdaguethis only showed up in live migration though, honestly, CI isn't going to catch every edge casee13:25
*** gouthamr has joined #openstack-nova13:25
mriedemstephenfin: deprecated is deprecated what's the difference13:25
mriedemthis config option deprecation stuff is grating on my brain13:25
stephenfinmriedem: So I _shouldn't_ push a twenty-patch "remove deprecated opts" series? Aww :'(13:26
* stephenfin kids...kind of13:26
efriedThe difference is just code cleanup / tech debt.  We could keep every deprecated option around forever and it wouldn't hurt anything.  But it sure would look messy.13:28
*** awaugama has joined #openstack-nova13:29
*** weshay is now known as weshay_bbiab13:30
efriedstephenfin Oh, snap.  Here's one.  Guess who authored it?  I41a6be48380999c49d6db3ee5221174a4101400213:31
efriedreviewed, rather.13:32
efriedIt was in the same blueprint as the one that moved the vnc opts originally (which was your change set IIRC)13:33
openstackgerritMerged openstack/os-vif master: Updated from global requirements  https://review.openstack.org/50270813:34
openstackgerritOpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements  https://review.openstack.org/50583913:34
efriedstephenfin And here's one you did co-author: https://review.openstack.org/#/c/463220/13:35
efriedI'll stop now.13:35
*** jmlowe has quit IRC13:35
openstackgerritOpenStack Proposal Bot proposed openstack/os-traits master: Updated from global requirements  https://review.openstack.org/50364613:36
mriedemefried: stephenfin: i've been dealing with getting my kid to school, so haven't been following along, what is the actual issue?13:38
mriedemi know about the vnc rename thing13:38
dansmithgibi: responded to some of your comments, thanks for looking at this stuff13:39
openstackgerritRadoslav Gerganov proposed openstack/nova master: VMware: implement get_inventory() driver method  https://review.openstack.org/50617513:40
efriedmriedem That's all.  Just needing https://review.openstack.org/#/c/505831/ to merge pdq for the sake of a couple of 3rd party CIs that were testing with those opts.13:41
*** bigdogstl has joined #openstack-nova13:43
efriedpowervm is kinda bound up between that and https://review.openstack.org/#/c/504194/ -- we need ^ to merge first so our rename of < will pass.13:43
*** baoli has joined #openstack-nova13:43
mriedemefried: +213:46
mriedemalex_xu: sdague: can you take a look at https://review.openstack.org/#/c/505831/ ? unblocks some 3rd party CIs13:46
efriedThanks!13:47
mriedemefried: are you going to follow that up and drop support for the options in the DEFAULT group/13:47
mriedemwhich were deprecated13:47
*** bigdogstl has quit IRC13:47
efriedCan13:47
*** ratailor has quit IRC13:48
sdague what's the no valid host error all about in CI?13:49
bauzasoomichi: you're around ? see my comments in https://review.openstack.org/#/c/506092/1 ?13:49
openstackgerritThomas Bechtold proposed openstack/nova stable/pike: Handle deleted instances when refreshing the info_cache  https://review.openstack.org/50618313:51
mriedemfancy pants nova meeting in 8 minutes13:52
openstackgerritSylvain Bauza proposed openstack/nova master: Fix definitely move single instance when created concurrently  https://review.openstack.org/50609313:52
bauzasmriedem: you could have interest in ^13:52
*** Sree has quit IRC13:52
*** Sree has joined #openstack-nova13:53
takashinbauzas: Could you review https://review.openstack.org/#/c/408955/ ?13:53
bauzastakashin: yeah, that's in my pipe for a long way :)13:53
takashinbauzas: Thank you.13:54
bauzastakashin: now we merged the spec, let's go back on the implementation side :)13:54
mriedembauzas: yeah, after the meeting13:54
*** lucasxu has quit IRC13:54
bauzasmriedem: np, just wanted to make sure you were aware13:54
*** slaweq has quit IRC13:55
mriedemdansmith: oh yeah, remind me to talk to you about filtering on IPs13:55
mriedemwith your series13:55
mriedemgiven the discussion we had in denver about how we don't do that in the DB13:55
dansmithmriedem: okay, does my series change anything about that?13:55
mriedemidk :)13:56
dansmithokay13:56
mriedemhaven't gone through all of it, i just know it's a special snowflake in the existing code13:56
openstackgerritMerged openstack/nova master: Remove 400 as expected error  https://review.openstack.org/50548513:56
mriedemi think, however, we have a functional regression test that i added for filtering on ip13:56
mriedembecause i remember once last year accusing steve kaufer of regressing that13:57
dansmithoh, filtering not sorting?13:57
mriedemright13:57
mriedemfilter by ip regex13:57
dansmithyeah, that shouldn't be any different13:57
*** Sree has quit IRC13:57
dansmithyou can only sort on access_ip_vX, which are strings and fine13:57
openstackgerritsahid proposed openstack/nova master: libvirt: bandwidth param should be set in guest migrate  https://review.openstack.org/49745513:57
openstackgerritsahid proposed openstack/nova master: libvirt: add method to configure migration speed  https://review.openstack.org/49745613:57
openstackgerritsahid proposed openstack/nova master: libvirt: slow live-migration to ensure network is ready  https://review.openstack.org/49745713:57
dansmithfilters basically go straight through like they did before13:57
openstackgerritMerged openstack/nova master: pci: Validate behavior of empty devname  https://review.openstack.org/39613013:58
mriedemstraight through to the db api?13:58
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/nova master: Add default configuration files to data_files  https://review.openstack.org/50618813:58
mriedemif yes, that's my concern, since we don't filter ips in the db13:58
mriedembut i'll look for this test before declaring that the sky has fallen13:59
dansmithif they were passed to the db api before then they still are, and if they weren't then they're not13:59
*** yangyapeng has joined #openstack-nova13:59
dansmithmriedem: nova/tests/functional/test_list_servers_ip_filter.py14:00
mriedemdansmith: that's the one14:00
mriedemcool14:00
*** josecastroleon has quit IRC14:02
sahidmriedem: can i have at some point your eyes on https://review.openstack.org/#/c/485522/, the code https://review.openstack.org/#/c/458513/14:03
sahidyou were asking for a spec14:03
*** lucasxu has joined #openstack-nova14:03
mriedemsahid: sure, i'll star it for later today14:04
sahidmriedem: ok thanks14:05
*** gomarivera has joined #openstack-nova14:05
openstackgerritMerged openstack/nova master: Adopt new pypowervm power_off APIs  https://review.openstack.org/47694514:07
*** liliars has quit IRC14:08
*** gbarros has quit IRC14:11
*** bigdogstl has joined #openstack-nova14:12
*** crushil_ has joined #openstack-nova14:14
*** crushil has joined #openstack-nova14:14
*** gomarivera has quit IRC14:14
*** armax has joined #openstack-nova14:14
*** coreywright has quit IRC14:14
*** gomarivera has joined #openstack-nova14:15
*** crushil has quit IRC14:15
*** crushil_ is now known as crushil14:15
*** bigdogstl has quit IRC14:16
*** Yikun has quit IRC14:16
*** cfriesen has joined #openstack-nova14:17
*** Swami has joined #openstack-nova14:19
*** gomarivera has quit IRC14:19
*** dtantsur is now known as dtantsur|brb14:20
*** tssurya has joined #openstack-nova14:21
dansmithtssurya: this is the place :)14:22
*** itlinux has joined #openstack-nova14:23
tssuryadansmith: yep I will be here then :) thank you14:23
*** itlinux has quit IRC14:23
*** jmlowe has joined #openstack-nova14:28
*** coreywright has joined #openstack-nova14:28
*** lajoskatona has quit IRC14:30
*** gbarros has joined #openstack-nova14:30
*** openstackgerrit has quit IRC14:33
*** cleong has joined #openstack-nova14:36
gibidansmith: I'm looking at the scatter_gather code and I don't think I spotted the part that prevents us to return a generator as a result.14:36
gibidansmith: I even replaced the list() call with a generator expression in your first patch and that seems to still pass the tests14:36
*** felipemonteiro has joined #openstack-nova14:38
*** felipemonteiro_ has joined #openstack-nova14:39
dansmithgibi: hmm, I initially was returning a generator from it and was hitting something in there that was trying to count the result14:39
dansmithgibi: but I guess maybe I was generating an error in one of the threads and that threw me off14:40
dansmithlet me try to change it again locally now that things are all working and see14:40
gibithe only length calulation is here https://github.com/openstack/nova/blob/193729b93a11ff54da99386e881270209797f020/nova/context.py#L439 but that is only checking the number of results not the lenght of each result14:41
*** itlinux has joined #openstack-nova14:41
dansmithyeah14:42
dansmithhmm, yeah, that seems to work14:42
dansmithI dunno what I was hitting before, but I initially had this so it was generators all the way down, convinced myself that wouldn't work across that thread boundary and then wrote that docstring about it14:43
*** felipemonteiro has quit IRC14:43
dansmithso... I dunno, but that was super early, so maybe something else was going on14:43
dansmithI shall change it14:43
*** itlinux has quit IRC14:43
*** hongbin has joined #openstack-nova14:43
stephenfinsdague: Any tips on how to find changes that have not been reviewed by anyone? 'NOT label:Code-Review<=2 age:5d' no longer seems to work14:45
stephenfinThis is in relation to Gerrit and your 'dashboard query changes since upgrade' mail (which I've yet to receive :()14:46
johnthetubaguyI think sdague refreshed the openstack dashboard url for the new gerrit version14:46
mriedemdansmith: you won me over on the sort keys/dirs defaults thing in https://review.openstack.org/#/c/504983/14:46
mriedemi'd be cool with dropping that handling14:46
stephenfinjohnthetubaguy: Aye, but I don't think that change is in there.14:46
gibidansmith: cool14:46
johnthetubaguystephenfin: ah, fair enough14:46
*** jobewan has joined #openstack-nova14:46
dansmithmriedem: why not just leave what I have?14:47
dansmithmriedem: you can't pass dirs without keys, and can't pass dirs that don't match keys in terms of length14:47
mriedemyou can, it's just kind of dead code as you said yesterday14:47
dansmithyeah, but now I have tests for it14:47
mriedem:/14:47
mriedemup to you, i'm +2 either way14:47
mriedemare you going to redo this series to make do_query return a generator?14:48
dansmithyeah, I'm doing it now14:48
dansmithI wish I hadjust done it at the end because of the conflict it makes,14:48
dansmithbut I'm in the middle now anyway14:48
*** openstackgerrit has joined #openstack-nova14:49
openstackgerritDan Smith proposed openstack/nova master: Add base implementation for efficient cross-cell instance listing  https://review.openstack.org/50498314:49
openstackgerritDan Smith proposed openstack/nova master: Make instance_list honor global query limit  https://review.openstack.org/50498414:49
openstackgerritDan Smith proposed openstack/nova master: Add db.instance_get_by_sort_filters()  https://review.openstack.org/50498514:49
openstackgerritDan Smith proposed openstack/nova master: Support pagination in instance_list  https://review.openstack.org/50498614:49
openstackgerritDan Smith proposed openstack/nova master: Add fault-filling into instance_get_all_by_filters_sort()  https://review.openstack.org/50539114:49
openstackgerritDan Smith proposed openstack/nova master: Add tests to validate instance_list handles faults correctly  https://review.openstack.org/50539214:49
openstackgerritDan Smith proposed openstack/nova master: Add get_instance_objects_sorted()  https://review.openstack.org/50541714:49
openstackgerritDan Smith proposed openstack/nova master: Copy some tests to a cellsv1 mixin  https://review.openstack.org/50544214:49
openstackgerritDan Smith proposed openstack/nova master: Fix a pagination logic bug in test_bug_1689692  https://review.openstack.org/50566114:49
openstackgerritDan Smith proposed openstack/nova master: Use improved instance_list module in compute API  https://review.openstack.org/50541814:49
openstackgerritDan Smith proposed openstack/nova master: Remove legacy fault-loading routines  https://review.openstack.org/50545614:49
dansmithmriedem: gibi: ^14:49
gibidansmith: looking...14:49
dansmithgibi: I'm still trying to figure out why that weird raise is .. weird.. but I can stack whatever the fix is on top14:50
*** andreas_s has joined #openstack-nova14:52
gibidansmith: OK. thanks for the explanation. As you guessed I haven't tried to move the raise in the except block myself14:53
gibidansmith: It seems you forget to update the code comments when you change the list to the generator expression in https://review.openstack.org/#/c/50498314:55
mriedemandreykurilin: hey i assume you like to do some scale and performance testing since you work on rally, right?14:55
dansmithgibi: I updated the docstring... did I miss a reference?14:56
gibidansmith: https://review.openstack.org/#/c/504983/6/nova/compute/instance_list.py14:57
dansmithwhat in the14:57
gibidansmith: L69 and L9314:57
dansmithcraaap14:57
dansmithI must have dumped it during a rebase14:58
dansmithurgh14:58
dansmithI _did_ update ;)14:58
gibiI believe you :)14:58
openstackgerritElod Illes proposed openstack/nova master: Add instance.interface_attach notification  https://review.openstack.org/50308914:58
dansmithgibi: from my history: https://pastebin.com/u7nUYi3814:58
dansmith:P14:58
openstackgerritEric Fried proposed openstack/nova-specs master: Spec: Use keystoneauth1 Adapter for endpoints  https://review.openstack.org/50019014:59
*** felipemonteiro_ has quit IRC14:59
*** felipemonteiro_ has joined #openstack-nova14:59
efriedmriedem edmondsw ^14:59
openstackgerritTakashi NATSUME proposed openstack/nova master: Fix hyperlinks in document  https://review.openstack.org/50610114:59
edmondswefried ack14:59
openstackgerritTakashi NATSUME proposed openstack/nova master: Fix hyperlinks in document  https://review.openstack.org/50610115:01
openstackgerritDan Smith proposed openstack/nova master: Add base implementation for efficient cross-cell instance listing  https://review.openstack.org/50498315:01
openstackgerritDan Smith proposed openstack/nova master: Make instance_list honor global query limit  https://review.openstack.org/50498415:01
openstackgerritDan Smith proposed openstack/nova master: Add db.instance_get_by_sort_filters()  https://review.openstack.org/50498515:01
openstackgerritDan Smith proposed openstack/nova master: Support pagination in instance_list  https://review.openstack.org/50498615:01
openstackgerritDan Smith proposed openstack/nova master: Add fault-filling into instance_get_all_by_filters_sort()  https://review.openstack.org/50539115:01
openstackgerritDan Smith proposed openstack/nova master: Add tests to validate instance_list handles faults correctly  https://review.openstack.org/50539215:01
openstackgerritDan Smith proposed openstack/nova master: Add get_instance_objects_sorted()  https://review.openstack.org/50541715:01
openstackgerritDan Smith proposed openstack/nova master: Copy some tests to a cellsv1 mixin  https://review.openstack.org/50544215:01
openstackgerritDan Smith proposed openstack/nova master: Fix a pagination logic bug in test_bug_1689692  https://review.openstack.org/50566115:01
openstackgerritDan Smith proposed openstack/nova master: Use improved instance_list module in compute API  https://review.openstack.org/50541815:01
openstackgerritDan Smith proposed openstack/nova master: Remove legacy fault-loading routines  https://review.openstack.org/50545615:01
openstackgerritMatt Riedemann proposed openstack/nova master: api-ref: fix default sort key when listing servers  https://review.openstack.org/50622715:02
andreykurilinmriedem: hi! basically, I like to do great things, so rally is my choice :) but yeah, scale and performance testing are somehow related :)15:02
*** kenperkins has quit IRC15:02
gibidansmith: I don't need proofs I have faith :)15:03
dansmithgibi: I need proof for my own sanity :)15:03
gibi:)15:03
*** takashin has left #openstack-nova15:04
gibiyeah, sanity is important15:04
*** yasufum has joined #openstack-nova15:05
mriedemandreykurilin: ok some of us got talking at the PTG about how we need some scale testing done on nova, especially with multiple cells, and thought you might be interested in doing that,15:05
mriedemespecially since godaddy is using cells v1 and presumably has to investigate migrating to cells v2 at some point15:06
mriedemandreykurilin: more specifically, dansmith is working on a series of changes to make listing instances across cells more efficient,15:06
mriedemit would be nice if we could get benchmarks before/after15:06
*** links has quit IRC15:07
mriedemi'd also be interested in general in how long it takes to create instances in ocata vs pike, since pike is now doing quotas differently and doing resource claims in the scheduler rather than the computes15:07
mriedemi could do this a bit sloppy with devstack and using the fake virt driver, but that doesn't give me multiple cells15:08
andreykurilinmriedem: so basically, it is quite easy to add scenarios for cells v1/v2 (and generate the load), but the problem is in where to launch it. I'll talk to some folks to try find the place for such research15:10
*** felipemonteiro has joined #openstack-nova15:11
*** ociuhandu has quit IRC15:12
*** bigdogstl has joined #openstack-nova15:12
*** felipemonteiro_ has quit IRC15:13
*** yasufum has quit IRC15:14
mriedembauzas: a few comments inline https://review.openstack.org/#/c/506092/15:15
mriedemandreykurilin: great, thanks15:16
mriedemwe've been trying to get feedback from large cells v1 users for awhile now and so far we don't get much feedback15:16
*** coreycb has joined #openstack-nova15:17
*** bigdogstl has quit IRC15:17
*** Tom_ has joined #openstack-nova15:18
efriedsdague You want me to do anything about https://review.openstack.org/#/c/488137/17/nova/utils.py@1309 ?  (There or in a fup?)15:18
*** tssurya has left #openstack-nova15:18
*** tssurya has joined #openstack-nova15:18
*** gbarros has quit IRC15:19
bauzasmriedem: k, cool, reading15:22
*** gomarivera has joined #openstack-nova15:23
*** felipemonteiro_ has joined #openstack-nova15:24
bauzasmriedem: just a question for just one comment, I'm not sure why we need to disable the source host15:24
bauzasmriedem: because you'd like to make sure you end up in some specific host?15:25
*** gbarros has joined #openstack-nova15:25
*** bigdogstl has joined #openstack-nova15:25
*** gbarros has quit IRC15:25
*** andreas_s has quit IRC15:25
*** gbarros has joined #openstack-nova15:25
*** gbarros has quit IRC15:25
*** Oku_OS is now known as Oku_OS-away15:26
*** tssurya has quit IRC15:27
*** felipemonteiro has quit IRC15:27
*** burt has joined #openstack-nova15:30
dansmithedleafe: I dunno why, but gerrit is being confusing about the stack of actual code patches for your selection/alternates bit15:30
*** bigdogstl has quit IRC15:30
dansmithedleafe: is this the bottom? https://review.openstack.org/#/c/486215/715:30
*** felipemonteiro_ has quit IRC15:32
*** felipemonteiro_ has joined #openstack-nova15:33
mriedembauzas: that's what was reported in the bug for the recreate,15:33
mriedembut i guess that's not really necessary to reproduce the issue15:33
mriedembecause the scheduler will compare 1 host to 2 instances and fail15:34
mriedembauzas: yeah so nevermind that part15:34
bauzasokay15:34
*** felipemonteiro has joined #openstack-nova15:34
*** chyka has joined #openstack-nova15:34
*** bigdogstl has joined #openstack-nova15:34
edleafedansmith: yep15:35
edleafedansmith: it seems to be adding specs and code together15:35
dansmithedleafe: well, it is also jumping around with the related-changes view of the code15:37
*** gyee has joined #openstack-nova15:37
*** felipemonteiro_ has quit IRC15:37
dansmithI guess it's because the add-allocations patch has been orphaned for a while15:37
*** bigdogstl has quit IRC15:39
mriedemsdague: https://review.openstack.org/#/c/500190/ is the "re-proposal" of the ksa adapters spec if you want to go over that. it's not so much a re-proposal as nearly a re-write though.15:39
mriedemedleafe: do i need to review those alternative hosts specs in any order? there is the one about the object model and then the other one.15:40
mriedemor should those be merged?15:40
edleafemriedem: alternate hosts was a concept we agreed to in Pike but never wrote up in a spec15:41
edleafemriedem: the Selection object is independent15:41
*** slaweq has joined #openstack-nova15:41
dansmithedleafe: is the host.updated=True thing the flag we use to avoid choosing that same host for another instance of a multi-create?15:44
sdaguemriedem: will look15:44
sdaguemriedem: I'm trying to sort out a final set of unit tests fails on the QEMU_IMG version thing15:44
edleafedansmith: I think so. That host state stuff is still black magic to me15:45
dansmithedleafe: oh actually it's host.updated=None.. how. strange.15:45
dansmithedleafe: okay I will comment based on that assumption and hedge a bit15:45
*** drwahl has joined #openstack-nova15:46
drwahldoes placement-api (in newton) require keystone v3?15:47
bauzasdansmith: that's because updated is the flag for saying "please refresh my state"15:47
*** cdent has joined #openstack-nova15:47
dansmithbauzas: okay I'm confused about why we need to do that in this context then15:48
bauzasdansmith: what's the exact concern you have in mind ?15:48
dansmithoohh,15:48
bauzasI just jumped on your above question15:48
dansmithmaybe this is "I'm going to consume this host, so make sure to refresh it soon" ?15:49
dansmithhttps://review.openstack.org/#/c/486215/7/nova/scheduler/filter_scheduler.py15:49
dansmithL9215:49
*** felipemonteiro has quit IRC15:49
*** rods has joined #openstack-nova15:49
*** tidwellr has joined #openstack-nova15:49
*** felipemonteiro_ has joined #openstack-nova15:49
bauzasdansmith: no15:50
bauzasdansmith: that's because we consume_from_request() before15:50
bauzasdansmith: so the in-memory HostState is modified15:50
mriedemstvnoyes: the scsi volume live migration tempest test failed for the new style attach flows in a grenade job, meaning one of the computes in the live migration is pike and one is queens http://logs.openstack.org/90/481290/6/check/gate-grenade-dsvm-neutron-multinode-live-migration-nv/d8962cc/console.html#_2017-09-17_04_21_05_12593715:50
bauzasdansmith: in case the calculations were wrong, we need to unset those modifications and force a fresh new DB call15:51
bauzaswe generally keep in memory the HostStates once per request15:51
dansmithbauzas: meaning if we fail to do the consume, we want to trigger a refresh?15:51
bauzasbut for a multi-schedule call, then we could iterate wrongly15:51
openstackgerritMerged openstack/os-vif master: Rehome OVO unit tests to tests.unit.test_object.py  https://review.openstack.org/48992215:51
bauzasdansmith: yup that sort of15:51
dansmithhrm15:51
dansmithnot sure I fully get it, but it's not the thing I thought, so that's fine15:52
*** Tom_ has quit IRC15:52
mriedemstvnoyes: looks like it fails here https://review.openstack.org/#/c/487884/7/tempest/api/compute/admin/test_live_migration.py@18515:52
dansmithedleafe: so, what follows is some rambling for improvements we can make later, but listen and nod (or not):15:52
*** bigdogstl has joined #openstack-nova15:52
bauzasdansmith: see HostManager.consume_from_request() and you'll understand15:52
*** felipemonteiro has joined #openstack-nova15:53
dansmithedleafe: I think what you're doing here is going through the list of hosts like we do today, and picking the target and alternates for each one,15:53
dansmithedleafe: which may mean your first alternate for instance 2 may be the the target for instance 3 and so on, such that all the alternates for early instances are almost definitely going to be burned by the later instances for any num_instances>max_retries15:54
dansmithright?15:54
dansmithso, hmm, yeah if that's right, that seems like a problem15:54
edleafedansmith: yes. Selecting an alternate doesn't consume anything15:54
*** sambetts is now known as sambetts|afk15:54
dansmithwhat we probably need to do is go through and pick primaries, and then pick alternates for each after that15:54
dansmiththe alternates can overlap, but they shouldn't overlap with any primaries15:55
edleafedansmith: ok, that could be done15:55
dansmithedleafe: I'm just thinking that we're basically never going to be able to reschedule those early instances if we do it in the order you have, which is probably bad :)15:55
dansmithedleafe: I shall add comments about this15:56
edleafeI think in an earlier version of this we didn't think this would be a significant issue15:56
*** felipemonteiro_ has quit IRC15:56
dansmithyeah, I think we discussed it before, indeed15:56
dansmithI think i was focusing on the overlapping of the alternates,15:56
edleafesince a) alternates should rarely be used and b) hosts might still fit an additional instance15:56
dansmithand not considering the overlapping of primaries15:56
dansmithyeah15:57
stvnoyesmriedem: ok, Ill take a look15:57
*** bigdogstl has quit IRC15:58
gibidansmith: I think I found why the target_cell context manager eats our MarkerNotFoundException, see my comment in https://review.openstack.org/#/c/504986/6/nova/compute/instance_list.py@8515:58
dansmithgibi: ahh, that makes more sense.. I forgot we were using the figure's target_cell here15:59
dansmithgibi: I wrote that fixture, so I'll fix that up and make this change at the end of this series15:59
*** sbezverk has quit IRC15:59
dansmithgibi: now, please, go find bugs in someone else's code :)16:00
efriedsdague mordred Design point about bp/use-ksa-adapter-for-endpoints: When setting up for a service where we have the ability to get auth from context, what should we do about auth in conf?  Options: a) Always use the context auth, don't even register auth options in the conf; b) Register auth conf options and allow them to override the context auth.16:00
*** mdnadeem has quit IRC16:00
efriedcdent ^ you may also have an opinion16:00
openstackgerritMerged openstack/nova-specs master: Spec: Use keystoneauth1 Adapter for endpoints  https://review.openstack.org/50019016:01
*** sahid has quit IRC16:01
sdagueefried: it depends on what the subcall is doing16:01
*** sbezverk has joined #openstack-nova16:01
sdagueif it is acting on behalf of the user, it should use the context auth16:01
gibidansmith: OK, I will review that follow up too. But instead of looking at others code I will just stop looking at any code for today16:02
sdagueoptionally wrapped in a service token so that it doesn't expire16:02
dansmithgibi: as long as it's not finding bugs in _my_ code I'm fine with whatever :P16:02
*** ralonsoh has quit IRC16:02
*** ralonsoh has joined #openstack-nova16:02
*** ralonsoh has quit IRC16:02
gibidansmith: :)16:03
efriedsdague  I guess there's also c) Don't use the context auth; require the conf auth.16:03
sdagueefried: yeh16:03
efriedsdague I hear you saying it's gonna be case by case.16:03
dansmithgibi: I'll add you to the review of the fixture cleanup when I have it16:03
sdagueit's going to be case by case16:03
efriedsdague I'll likely need some help identifying which is which.16:03
sdagueefried: so, I think the answer ends up being roughly16:04
sdagueglance, cinder, barbican, keystone always act as user16:04
sdagueironic always from conf, because nova is the ironic multi tenancy solution16:04
sdagueand neutron, it depends on the operation16:04
*** slaweq has quit IRC16:05
efriedsdague Okay.  For that first set: You already pushed for glance to be able to get the auth from context, so not c).  Should we b) allow conf override or just a) always use context?16:05
sdagueefried: for glance, I don't think so16:05
sdagueI can't think of a case where we should "sudo" on image things16:06
sdagueor that image things have to happen outside of a user context16:06
*** slaweq has joined #openstack-nova16:06
*** yamahata has joined #openstack-nova16:07
sdaguewe should, whenever possible, really operate in the user context of the user token we got, because it ensures we don't have an unintended priv escalation16:07
sdagueironic is a special case, ironic doesn't have regular users16:07
dansmithedleafe: sanity check my tome of words on that review?16:07
efried(It's worth noting that part of the mission statement of this bp was consistency.  Sigh.)16:07
*** weshay_bbiab is now known as weshay16:07
mordredefried, sdague: "optionally wrapped in a service token" doesn't need nova-specific creds does it?16:08
edleafedansmith: on a call and an IRC meeting, so it'll be later16:08
dansmithedleafe: sure16:08
efriedmordred It's just service_auth.get_auth_plugin(context)16:08
sdaguemordred: i foreget exactly what it needs on disk16:09
*** suresh12 has joined #openstack-nova16:09
sdaguemordred: it does need some creds16:09
efriedoh16:09
efriedmordred sdague https://github.com/openstack/nova/blob/master/nova/service_auth.py16:10
efriedSo it loads up the auth from the [service_user] group.16:10
efriedand we get two auths in the thing.16:10
efriedyeesh.16:10
*** suresh12 has quit IRC16:11
mordredso - you need at least some of the ksa settings for each service no matter what16:11
efriedOnly if CONF.service_user.send_service_user_token is set16:11
mordredlike you need the adapter options and probably the session options - so it's really just a question of whether the authoptions are included right?16:11
*** slaweq has quit IRC16:12
efriedmordred Yeah, that's what we're discussing - whether we should even register the auth options in groups where that service can be contacted using the user context auth.16:13
*** bigdogstl has joined #openstack-nova16:14
*** edand has joined #openstack-nova16:15
mordrednod. well - consistency notwithstanding, I'd vote for not registering conf options if we're not ever going to use them - otherwise someone is going to configure them and then be confused why they're not used16:16
mordredbut I defer to smarter nova humans16:17
sdaguemordred: ++ lets keep things trimmed down16:17
sdagueefried: it's probably going to be worth writing a doc section on configuring nova with other services as well16:18
sdaguebecause I agree there is confusion here16:18
*** Apoorva has joined #openstack-nova16:18
efriedsdague Cool.  edmondsw brought this up in the spec review, and I addressed it with a brief parenthetical, but it has become a bigger thing quite suddenly.  Worth a delta to the spec, you think?  (It just merged.)16:20
*** bigdogstl has quit IRC16:20
*** edand has quit IRC16:20
sdagueefried: I'm ok if it's just admin docs as part of the work16:20
efriedrgr16:20
sdaguemordred: on https://review.openstack.org/#/c/488137/17/nova/utils.py@130916:23
sdagueis the service types library not encoding this in python?16:23
sdaguelike, I'd kind of expect that to be happening all behind the scenes16:23
efriedsdague Without a session param, it's pretty lightweight: https://github.com/openstack/os-service-types/blob/master/os_service_types/service_types.py#L5116:25
mordredsdague: yes- os-service-types is - os-service-types didn't make it in to the keystoneauth release for pike, so we need to add that now that queens is open16:25
efriedAlmost all of the work is done at import time, actually.16:25
efriedhttps://github.com/openstack/os-service-types/blob/master/os_service_types/service_types.py#L2216:25
mordredefried: yah, I think it's actually fine for this cycle - the contents really do not change frequently - and CERTAINLY not for the services that nova cares about16:26
efriedThe race would be harmless.  I can take the lock out.16:26
mordredso we can get live-update landed as a follow on16:26
sdagueum... https://github.com/openstack/os-service-types/blob/master/os_service_types/service_types.py#L59 ? closet network call?16:26
edmondswefried sdague mordred to be clear, long-term I think we'll need auth conf options for everything. But we don't today16:26
sdaguethat seems like something people should have to opt into16:26
mordredsdague: yah - it's there - but the keystoneauth consumption of the library at least at first will not pass a session16:26
mordredsdague: yup16:27
efriedsdague The opt-in is by passing a `session` param.16:27
mordredsdague: so when we land the next patch to ksa to consume that, nova can just switch to always passing the correct/official type to keystoneauth and keystoneauth will dtrt16:27
mordredwe'll make sure subsequent turning on of remote access/ network calls is appropriately opt-in when we add it16:28
sdagueI'm actually not super clear why the remote part is there16:28
efriedto get a fresh copy of the service-types-authority data16:28
efriedos-service-types ships with a cached copy16:29
openstackgerritMerged openstack/os-vif master: Update reno for stable/pike  https://review.openstack.org/48867116:29
sdagueI thought the point of this was a no requirements version which we rev every time there is a service types update16:29
sdagueso people just replace it with the new one16:29
efriedYup.  Unless they don't.16:29
sdagueif they don't, then they don't16:29
openstackgerritMerged openstack/os-traits master: Update reno for stable/pike  https://review.openstack.org/48866916:30
sdaguecloset network calls that could end up with very interesting results or random network hangs16:30
sdaguedon't seem useful16:30
*** tbachman has quit IRC16:30
sdaguelike https://github.com/openstack/os-service-types/blob/master/os_service_types/service_types.py#L59 under some circumstances can hang forever16:30
mordredsdague: yah - I don't actually think it's useful for nova to opt-in to the fetch behavior16:31
efriedHence the comment in the nova change16:31
sdaguemordred: sure, but I'm not super clear where it's a good idea to put a potentially arbitrary delay into anyone's code path16:32
sdagueI get that it's clever, but the potential failure domains get huge16:32
openstackgerritEric Fried proposed openstack/nova master: nova.utils.get_ksa_adapter()  https://review.openstack.org/48813716:32
efriedsdague mordred ^ updated to remove lock16:33
mordredsdague: totally. this is why it will always be opt-in16:33
openstackgerritMerged openstack/nova master: Restore '[vnc] vnc_*' option support  https://review.openstack.org/50583116:33
*** bigdogstl has joined #openstack-nova16:33
sdaguemordred: ok. I'm still not sure why it's useful, but I've said my piece :)16:33
mordredefried: yah- I think honestly you could just initialize the _SERVICE_TYPES at the top and have it done at import16:34
sdaguehaving spent a couple of months discovering requests can hang forever on simple get calls depending on what's happening on the network, which was locking up 1/3 of my smart home devices, I'm super twitchy on it16:34
efriedmordred Considering that ost loads the local file at import time (which I didn't realize) I think you're right.16:34
openstackgerritEric Fried proposed openstack/nova master: nova.utils.get_ksa_adapter()  https://review.openstack.org/48813716:36
efriedmordred Did that ^16:36
*** zhenq has joined #openstack-nova16:36
*** zhenq has quit IRC16:36
*** drwahl has left #openstack-nova16:37
sdagueefried: +216:37
efriedthanks!16:38
openstackgerritSean Dague proposed openstack/nova master: Support qemu >= 2.10  https://review.openstack.org/50567316:38
mriedemefried: stephenfin: you guys did that fancy scheduler call flow diagram. i will pay a shiny nickel to whoever can make this live migration call flow into a docs diagram https://photos.app.goo.gl/Q8JdpjM0PZhAzsv3216:39
*** slaweq has joined #openstack-nova16:39
sdaguecburgess: https://review.openstack.org/#/c/505673 - live snapshot by default16:39
*** cdent has quit IRC16:41
dansmithmriedem: lol16:41
dansmithhow ... analog of you16:41
*** shaohe_feng has quit IRC16:41
mriedemdansmith: i think i'm just going to slap that into our official docs16:42
mriedem"Technical reference deep dive > live migration > THIS"16:42
dansmithheh16:42
*** bigdogstl has quit IRC16:42
openstackgerritElod Illes proposed openstack/nova master: Add instance.interface_detach notification  https://review.openstack.org/50628416:43
*** shaohe_feng has joined #openstack-nova16:43
*** suresh12 has joined #openstack-nova16:43
mriedemif only i were on the twitters16:43
sdaguemriedem: that is a solvable problem16:44
efriedmriedem What does "cast" mean?16:45
*** itlinux has joined #openstack-nova16:45
efriedasync invocation?16:45
*** dtantsur|brb is now known as dtantsur16:46
mriedemrpc cast16:46
mriedemvs call16:46
mriedemwhich is important to understand the hot potato between the computes during live migration16:46
mriedemespecially if you love rpc timeouts16:46
mriedembecause your instance has 20 ports and 20 volumes attached to it16:46
mriedemand token timeouts16:47
dansmithmmmm, rpc timeout16:47
*** artom has quit IRC16:48
*** slaweq has quit IRC16:48
sdaguemriedem: service tokens fix that16:49
*** slaweq has joined #openstack-nova16:49
sdaguewell, some of it16:49
*** tesseract has quit IRC16:50
mriedemyeah i know16:50
mriedemwonder if anyone is using those yet16:50
*** gomarivera has quit IRC16:51
*** yamahata has quit IRC16:52
*** rcernin has quit IRC16:52
*** Swami has quit IRC16:53
*** dave-mcc_ has joined #openstack-nova16:55
*** dave-mccowan has quit IRC16:58
*** tbachman has joined #openstack-nova16:59
*** baoli has quit IRC17:00
*** slaweq has quit IRC17:00
*** slaweq has joined #openstack-nova17:01
*** mvk has quit IRC17:01
*** baoli has joined #openstack-nova17:01
*** tbachman_ has joined #openstack-nova17:02
*** derekh has quit IRC17:02
*** tbachman has quit IRC17:03
*** tbachman_ is now known as tbachman17:03
bauzasdansmith: mriedem: question, when I was testing https://review.openstack.org/#/c/506093/2 I discovered a weird stack http://paste.openstack.org/show/621641/17:04
bauzasdansmith: mriedem: when looking at https://github.com/openstack/nova/blob/master/nova/compute/resource_tracker.py#L1227 it lookups the in-memory dict of all compute nodes for finding the right node17:05
*** jpena is now known as jpena|off17:05
bauzasdansmith: mriedem: but AFAICS, we're passing the source node as attribute to the destination RT https://github.com/openstack/nova/blob/master/nova/compute/manager.py#L573517:06
bauzasdansmith: mriedem: so since the destination RT doesn't know the source node, it fails with a KeyError17:07
bauzasamirite?17:07
bauzasunless post_live_mig runs on the source node17:07
bauzasthat's confusing17:07
mriedempost live migrate runs on the source onde17:07
mriedem*node17:07
mriedemsee my awesome call flow diagram above17:08
mriedemgd this is already paying for itself17:08
mriedemhttps://photos.app.goo.gl/Q8JdpjM0PZhAzsv3217:08
bauzasgrmblbl17:08
openstackgerritDan Smith proposed openstack/nova master: Fix CellDatabases fixture swallowing exceptions  https://review.openstack.org/50631217:08
bauzasmriedem: any reason you would see why I'm getting that KeyError ? I guess it's because I don't run the periodics ?17:09
openstackgerritMerged openstack/nova master: Updated from global requirements  https://review.openstack.org/50583917:09
*** Swami has joined #openstack-nova17:09
openstackgerritMerged openstack/nova master: Fix hyperlinks in document  https://review.openstack.org/50610117:10
*** smatzek has quit IRC17:11
*** bigdogstl has joined #openstack-nova17:12
mriedembauzas: that should get set in memory when the compute service starts up17:12
mriedemwhen it calls pre_start_hook17:12
bauzasmmm17:12
mriedemthat will call update_available_resource_for_node17:12
bauzaslooking at the func tests, jay needed to explicitly start them17:13
mriedemstart what?17:13
mriedemthe computes?17:13
mriedemthe fixture starts those17:13
jaypipeshmm?17:13
bauzasthe RT update things17:13
bauzashere, I'm getting a trace because self.compute_nodes[my_node] isn't set yet17:13
mriedemwhen the compute service starts, it will get the available nodes from the driver, and use those to call rt.update_available_resource17:13
mriedempassing in the node name which gets populated in the rt.compute_nodes dict17:13
bauzaswhich is populated AFAIK by running the RT update call17:13
bauzasmmm, I'm introspecting17:15
mriedemwhat you have in the test setup looks fine to me17:15
mriedemand it's what we're doing in other tests17:15
bauzasI'll upload my latest rev and see what the job insults me17:16
*** bigdogstl has quit IRC17:17
*** smatzek has joined #openstack-nova17:17
edleafemriedem: IMO, we need lots more flow diagrams like that17:17
openstackgerritSylvain Bauza proposed openstack/nova master: Add a regression test for bug 1718455  https://review.openstack.org/50609217:18
openstackbug 1718455 in OpenStack Compute (nova) "[pike] Nova host disable and Live Migrate all instances fail." [Medium,In progress] https://launchpad.net/bugs/1718455 - Assigned to Sylvain Bauza (sylvain-bauza)17:18
openstackgerritSylvain Bauza proposed openstack/nova master: Fix definitely move single instance when created concurrently  https://review.openstack.org/50609317:18
*** gszasz has quit IRC17:18
openstackgerritMerged openstack/nova master: Remove compatibility code for flavors  https://review.openstack.org/46037717:18
*** itlinux has quit IRC17:18
openstackgerritMerged openstack/nova master: Test InstanceNotFound handling in 'nova usage'  https://review.openstack.org/46851417:19
*** felipemonteiro_ has joined #openstack-nova17:22
*** felipemonteiro has quit IRC17:22
*** artom has joined #openstack-nova17:25
*** felipemonteiro has joined #openstack-nova17:25
openstackgerritMerged openstack/nova master: VMware: Factor out relocate_vm()  https://review.openstack.org/27011517:26
mriedemedleafe: agree, just needs to be put into docs17:26
openstackgerritMerged openstack/nova master: neutron: handle binding:profile=None during migration  https://review.openstack.org/50426017:26
*** felipemonteiro_ has quit IRC17:28
*** kenperkins has joined #openstack-nova17:30
*** vks1 has joined #openstack-nova17:31
openstackgerritMatt Riedemann proposed openstack/nova stable/pike: neutron: handle binding:profile=None during migration  https://review.openstack.org/50631917:33
*** vks1 has quit IRC17:37
*** erlon has quit IRC17:37
openstackgerritMatt Riedemann proposed openstack/nova stable/ocata: neutron: handle binding:profile=None during migration  https://review.openstack.org/50632017:37
*** rods has left #openstack-nova17:38
*** crushil has quit IRC17:38
*** crushil has joined #openstack-nova17:38
*** mvk has joined #openstack-nova17:41
*** vvargaszte has joined #openstack-nova17:44
*** vvargaszte has quit IRC17:46
*** sridharg has quit IRC17:47
openstackgerritMatt Riedemann proposed openstack/nova stable/newton: neutron: handle binding:profile=None during migration  https://review.openstack.org/50632317:49
*** elod has quit IRC17:50
*** elod has joined #openstack-nova17:52
*** ioggstream has quit IRC17:52
efriedmriedem Who are we looking to for the second +2 & +W for bp/use-ksa-adapter-for-endpoints work?  (E.g. https://review.openstack.org/#/c/488137/)17:54
mriedemme17:55
mriedemi guess17:55
mriedemi'm doing about 6 things at once right now though, so it's going to have to wait17:55
*** ijw has joined #openstack-nova17:56
dansmithefried: and three of those six things are my patches, which are very important17:56
efriedmriedem Sure, no worries.  From this morning's meeting, you indicated it should all be done in the next month, and there's actually a nontrivial amount of code left to write (cinder, barbican, keystone)17:57
efrieddansmith No doubt.17:57
openstackgerritMerged openstack/nova master: Update docs to include standardization of VM diagnostics  https://review.openstack.org/50040817:57
*** Sukhdev has joined #openstack-nova17:57
*** gomarivera has joined #openstack-nova17:58
*** Sukhdev has quit IRC18:03
sdaguemtreinish: http://logs.openstack.org/74/501874/2/gate/gate-nova-python35/0dcede7/console.html#_2017-09-21_18_02_47_155101 - is that an stestr issue?18:05
sdagueworker 7 just hung, and eventually that was a fail18:05
*** crushil has quit IRC18:06
*** bigdogstl has joined #openstack-nova18:06
*** esberglu has quit IRC18:07
*** esberglu has joined #openstack-nova18:08
*** dtantsur is now known as dtantsur|afk18:08
mriedemstvnoyes: the bdm attachment ids stuff in the migrate_data object is going to get weird when we have multiattach18:11
mriedemsince in your change it's a 1:1 between volume id and attachment id18:12
mriedemalthough,18:12
*** esberglu has quit IRC18:12
mriedemi guess you wouldn't have a volume attached multiple times to the same instance, unless we're migrating it18:12
mriedemso maybe i'm overthinking things18:12
*** bigdogstl has quit IRC18:14
*** tonygunk has joined #openstack-nova18:15
openstackgerritIldiko Vancsa proposed openstack/nova-specs master: Add multiattach support to Nova  https://review.openstack.org/49977718:15
-openstackstatus- NOTICE: Gerrit OpenIDs have been accidentally overwritten and are in the process of being restored18:17
*** esberglu has joined #openstack-nova18:18
-openstackstatus- NOTICE: The Gerrit service on review.openstack.org is being taken offline briefly to perform database repair work but should be back up shortly18:23
*** ChanServ changes topic to "The Gerrit service on review.openstack.org is being taken offline briefly to perform database repair work but should be back up shortly"18:23
*** tonygunk has quit IRC18:25
*** ijw has quit IRC18:25
*** crushil has joined #openstack-nova18:28
mriedemand just when i was about to -1 this change18:29
*** ijw has joined #openstack-nova18:29
*** yamahata has joined #openstack-nova18:36
*** smatzek has quit IRC18:37
mriedemdansmith: so for your 2nd change in the series to honor the global limit,18:38
mriedemi was posting this comment, but can't18:38
mriedem"So today we iterate the cells in order and just pull $limit instances and if we keep going to another cell we subtract how many we pulled out of the previous cell for a new limit on the next cell. Now we're going to be pulling $limit instances out of each cell concurrently, merge sorting them all and then enforcing the limit below. That is more traffic, right? Maybe it's negligible.",18:38
mriedem"I was wondering if we wanted to divide the limit by the number of cells and then pull that many instances from each. So if we had 2 cells and the limit was 1000 (which is the default in the API), we'd just pull 500 instances from each cell and merge sort them."18:38
*** gszasz has joined #openstack-nova18:42
dansmithmriedem: we can't do that partitioning of the limit until/unless we support going back to the cells for more instances18:44
dansmithmriedem: otherwise we'd hit the end of one limit batch, assume there are no more in that cell that sort after anything in other cells and then get out of sync18:44
*** armax has quit IRC18:44
dansmithmriedem: we basically have to have at least $limit results from each cell to prevent that18:44
*** armax has joined #openstack-nova18:44
dansmithor emulate it by refilling our batch when we run out before we continue the merge18:44
dansmiththat distinction is the thing that made us think we couldn't do this easily initially and why we went down the path of punting on the problem18:45
dansmithalso, yes it may be more traffic at times, but something else I was discussing with people was:18:45
dansmithif each tenant has N instances now, and then later those N instances are spread across a bunch of cells, this list operation is still pulling the same number of rows from the DB in total, but smaller pieces18:46
dansmiththe db driver doesn't even fetch partial results today or with this, it fetches everything in the result18:46
dansmithdoes that make sense at all?18:47
mriedemyeah, hadn't considered the case that there are like 200 instances in cell1 and 600 insteances in cell2, if we split the limit to 500 each, we'd end up with 700 instead of 80018:50
*** ioggstream has joined #openstack-nova18:50
*** suresh12 has quit IRC18:51
dansmithwell, there's that, but we'd also be lossy in that we'd ignore things from cell1 that sorted before some in cell2, but just didn't make the limit cut18:51
dansmithand if those were ever after max_limit/N cells, you'd never be able to limit them ever18:51
dansmither, never be able to list them18:51
*** ijw has quit IRC18:52
*** gbarros has joined #openstack-nova18:52
*** suresh12 has joined #openstack-nova18:54
*** avolkov` has joined #openstack-nova18:55
*** mriedem1 has joined #openstack-nova18:56
*** avolkov has quit IRC18:56
*** mriedem has quit IRC18:56
*** gbarros has quit IRC18:57
*** Apoorva_ has joined #openstack-nova19:01
*** Apoorva has quit IRC19:04
*** penick_ has joined #openstack-nova19:07
*** mriedem1 is now known as mriedem19:07
*** hieulq has quit IRC19:09
*** bigdogstl has joined #openstack-nova19:11
*** avolkov` has quit IRC19:12
openstackgerritEric Fried proposed openstack/nova master: Get auth from context for glance endpoint  https://review.openstack.org/49005719:15
*** awaugama has quit IRC19:16
efriedsdague mordred ^ removed conf auth for glance19:16
*** bigdogstl has quit IRC19:16
*** artom has quit IRC19:17
efrieddangit, tried to re-author, but failed.  Now the co-authored line in the commit message doesn't make sense.  Fixing.19:17
*** suresh12 has quit IRC19:17
openstackgerritEric Fried proposed openstack/nova master: Get auth from context for glance endpoint  https://review.openstack.org/49005719:17
*** ijw has joined #openstack-nova19:17
mordredefried: lgtm19:19
efriedcool19:20
*** cdent has joined #openstack-nova19:20
*** suresh12 has joined #openstack-nova19:20
mriedemstvnoyes: ok comments inline https://review.openstack.org/#/c/463987/19:20
*** ChanServ changes topic to "This channel is for Nova development. For support of Nova deployments, please use #openstack. Please see: https://wiki.openstack.org/wiki/Nova/Pike_Release_Schedule"19:20
-openstackstatus- NOTICE: OpenIDs for the Gerrit service have been restored from a recent backup and the service is running again; before/after table states are being analyzed now to identify any remaining cleanup needed for changes made to accounts today19:21
stvnoyesmriedem: thanks i'll take a look19:21
*** ioggstream has quit IRC19:22
*** ijw has quit IRC19:22
*** hieulq has joined #openstack-nova19:25
*** suresh12 has quit IRC19:25
openstackgerritSean Dague proposed openstack/nova master: Support qemu >= 2.10  https://review.openstack.org/50567319:25
*** xyang1 has quit IRC19:26
*** suresh12 has joined #openstack-nova19:34
*** jaypipes has quit IRC19:35
*** xyang1 has joined #openstack-nova19:37
openstackgerritMerged openstack/nova master: Transform servergroup.create notification  https://review.openstack.org/45402319:37
openstackgerritMerged openstack/nova master: Transform servergroup.delete notification  https://review.openstack.org/48390219:41
*** pcaruana has quit IRC19:41
openstackgerritMerged openstack/nova master: Move libvirts dmcrypt support to privsep.  https://review.openstack.org/49073719:41
openstackgerritMerged openstack/nova master: api-ref: fix default sort key when listing servers  https://review.openstack.org/50622719:42
*** crushil has quit IRC19:43
*** crushil has joined #openstack-nova19:44
*** dave-mcc_ is now known as dave-mccowan19:45
*** weshay is now known as weshay_bbiab19:50
*** pchavva has quit IRC19:56
*** thorst has quit IRC19:58
*** jmlowe has quit IRC20:02
*** thorst has joined #openstack-nova20:03
*** mvk has quit IRC20:05
*** ijw has joined #openstack-nova20:06
mriedemdansmith: replied in https://review.openstack.org/#/c/504984/8/nova/tests/functional/compute/test_instance_list.py - not sure what i'm missing there why the slice is needed20:06
*** thorst has quit IRC20:07
dansmithheh20:07
dansmithmriedem: that should be slicing self.instances not uuids20:07
dansmithI'm in the middle of fixing migration uuid stuff in my tree so I can't I update it right now20:08
dansmithbut I will when I'm done20:08
mriedemok yeah that makes more sense :)20:08
dansmithoptionally you can locally test and then I'll slap something on the end, or you can push a fix20:08
mriedemi had to rebase the entire thing, so we can slap it on the end20:09
*** crushil has quit IRC20:09
dansmithack20:09
*** crushil has joined #openstack-nova20:09
dansmithI would be happy if we merge everything at least up to the pagination one this week20:10
mriedemi'll try to push it to the limit20:10
dansmith+120:10
*** ijw has quit IRC20:11
mriedemdoes that mean all these other tests doing "uuids = [inst['uuid'] for inst in insts]" should be using self.instances?20:13
*** jobewan has quit IRC20:13
dansmithno,20:15
dansmithjust that one was intending to compare the list to the snipped list of known instances and sliced the wrong list20:15
dansmithI mean20:15
mriedemalright left a diff in the comments20:15
dansmithpoint me to something, I could have made the same mistake I guess, but..20:15
dansmithcool, in the limit one?20:15
mriedemyeah20:16
*** artom has joined #openstack-nova20:16
dansmithcool thanks20:16
dansmithalso20:16
dansmithour migration and rt stuff is hell on earth20:17
dansmiththat is all20:17
*** mvk has joined #openstack-nova20:17
mriedemi thought branson, mo was hell on earth?20:17
*** artom has quit IRC20:17
dansmithwe have a new contender20:17
*** armax has quit IRC20:18
*** artom has joined #openstack-nova20:18
*** artom has quit IRC20:19
*** artom has joined #openstack-nova20:19
dansmithmriedem: that one was checking that the list of uuids was the same as returned and as sorted20:20
*** artom has quit IRC20:20
*** artom has joined #openstack-nova20:20
*** artom has quit IRC20:21
*** artom has joined #openstack-nova20:21
*** artom has quit IRC20:22
*** annegentle has joined #openstack-nova20:22
*** artom has joined #openstack-nova20:22
*** artom has quit IRC20:23
*** artom has joined #openstack-nova20:23
*** crushil has quit IRC20:26
*** ijw has joined #openstack-nova20:27
*** gouthamr has quit IRC20:28
mriedemin test_get_sorted ?20:30
mriedemah ok, nvm then20:30
*** tbachman has quit IRC20:30
mriedemoh yeah, for the migration uuid stuff, we still have a couple of outstanding bugs related to cleaning up allocations during a failed move20:31
dansmiththat you'll want to get in before this so you can backport them I guess?20:32
mriedemyeah20:32
mriedembeen meaning to at least write the recreate tests20:32
mriedemmaybe i can do that at 10pm tonight after all others are asleep20:32
openstackgerritChris Dent proposed openstack/nova master: Move project_id and user_id to Allocation object  https://review.openstack.org/50041020:32
openstackgerritChris Dent proposed openstack/nova master: [placement] Allow _set_allocations to delete allocations  https://review.openstack.org/50105120:32
openstackgerritChris Dent proposed openstack/nova master: WIP: [placement] POST /allocations to set allocations for >1 consumers  https://review.openstack.org/50007320:32
*** lyan has quit IRC20:33
*** cleong has quit IRC20:33
*** lyan has joined #openstack-nova20:38
*** ijw has quit IRC20:38
*** thorst has joined #openstack-nova20:40
mriedemshit 1 week for forum topics?20:42
*** gszasz has quit IRC20:42
cdentbarely digested the ptg, too soon!20:42
mriedemi threw up half of it20:43
mriedemcouldn't even digest it all20:43
openstackgerritEric Berglund proposed openstack/nova master: Add PowerVM hypervisor configuration doc  https://review.openstack.org/50566520:43
*** annegentle has quit IRC20:43
*** artom has quit IRC20:49
*** edmondsw has quit IRC20:51
*** artom has joined #openstack-nova20:51
*** artom has quit IRC20:52
*** artom has joined #openstack-nova20:52
*** artom has quit IRC20:53
*** artom has joined #openstack-nova20:53
*** Apoorva_ has quit IRC20:55
*** Apoorva has joined #openstack-nova20:55
*** edmondsw has joined #openstack-nova20:57
mriedemha http://forumtopics.openstack.org/20:58
mriedemso apparently this has been getting communicated all month, i've just missed it20:59
mriedemnot sure why i'd be distracted...20:59
mriedembut so has everyone else20:59
cdenthuh20:59
cdentfirst I started hearing about it was last week21:00
*** Apoorva_ has joined #openstack-nova21:01
*** edmondsw has quit IRC21:01
mriedemgod man idk, we just had the ptg and i was already annoyed with how much stuff we talked about which has no chance of making queens21:02
*** hamzy has quit IRC21:03
*** Apoorva has quit IRC21:04
melwittI feel that way too but try to remind myself that ppl just wanted to make sure placement would be able to model the stuff they want to do whenever nested RPs is ready21:04
*** lucasxu has quit IRC21:04
mriedemand then how to use placement to make your toast21:05
melwittI think that could have been accomplished with fewer hours than were spent, but yeah21:05
mriedemwith the "grape jelly" trait21:05
melwittheh21:05
dansmithwe had already been over a lot of that stuff before21:06
cdentI think it’s a bit unfair to expect people not to want to talk when they only have a chance to really talk maybe 2-4 times a year21:06
dansmithI was fairly annoyed, but I'm sure nobody noticed21:06
mriedemthe forum is now supposed to be the place to get the ops and users in the same room, so i kind of just want to get topic submissions from those groups21:06
*** catintheroof has quit IRC21:07
mriedemi have accepted that everytime we get together we're going to talk about stuff that we aren't going to do in that release21:07
mriedemjust hard to juggle that plus knowing we have bugs to fix in pike yet, and things to get done in queens, without thinking about all the future things, it's overwhelming21:08
cdentit is indeed, but it is that overwhelmedness that makes people want to talk21:09
mriedemi come from swedish heritage,21:09
mriedemwe don't want to talk about anything21:09
mriedemexcept jellied fish carcass21:10
cdentmmmmm jellied fish21:10
melwittand it's kind of lose lose in that if ppl didn't go over things repeatedly and make sure placement would work for them, if they waited quietly and when it was done, it couldn't model the things they needed, they'd be told, "why did you talk to us?"21:10
melwitt*why didn't you talk to us21:10
melwittso ... I dunno. I see both sides21:11
cdentI wanted creamed herring, not pickled!21:11
mriedemi think i said this after boston, but i'd kind of like to go to a summit to actually sit in on user/ops talks where they talk about how they are actually using openstack and problems they are hitting21:11
mriedemassuming those talks aren't just pushing some product message21:12
melwittI went to one ops midcycle when it was near where I live, and it was like that21:12
cdentare those ops concerned with all this nfv, enhanced platform hoopa joop?21:12
mriedemthat's what sdague said the ops meetup in philly was like too21:12
mriedemcdent: i don't think so21:12
melwittno, they were mostly focused on running clusters, upgrading, deploying, that kind of stuff21:13
mriedemdansmith: L16 is for you https://etherpad.openstack.org/p/SYD-nova-brainstorming21:13
mriedemmikal: ^ and you21:13
*** bigdogstl has joined #openstack-nova21:13
dansmithmriedem: oh good.21:13
cdentyeah, that’s what I would assume too, so the force of all this epa stuff is depressing21:14
melwittyeah21:15
*** ijw has joined #openstack-nova21:15
openstackgerritOpenStack Proposal Bot proposed openstack/os-traits master: Updated from global requirements  https://review.openstack.org/50364621:15
*** felipemonteiro has quit IRC21:16
*** slaweq has quit IRC21:17
*** bigdogstl has quit IRC21:18
*** felipemonteiro_ has joined #openstack-nova21:18
mriedembtw, i thought the ptg was very productive overall21:18
mriedemeven though maybe i don't express that21:18
cdentwe see it in your eyes21:19
*** felipemonteiro__ has joined #openstack-nova21:19
*** ijw has quit IRC21:19
cdentI think for the summit maybe we should just put a moratorium on placement? we have a backlog of at least two cycles, and we’ve had the discussions at the ptg. maybe generic device management stuff is a thing, but it’s only placement in the sense that those things are resource providers21:20
mriedemi do take solace in the fact that the other etherpads on the wiki are pretty much empty21:20
cdentI’d much rather see us focus on listening, as you said, to users/ops21:20
melwittI agree all sessions should be ops-focused and more about listening21:21
mriedemlike, i want to go here https://etherpad.openstack.org/p/SYD-ops-session-ideas21:21
*** felipemonteiro_ has quit IRC21:23
*** baoli has quit IRC21:23
dansmithforum was also originally pitched as being the middle of the cycle where we'd sync up on things in progress like we used to,21:25
dansmithbecause they shifted the schedule21:25
dansmithso I certainly don't want to avoid talking about bits of placement (or anything else) material to the act of getting important things landed21:26
dansmithand of course, the listening to ops stuff will likely necessarily end up talking about future things anyway, if it's feature gaps, etc21:26
cdentdansmith: that’s not how I remember it? I thought it was “shortly after the latest release to get feedback from people”.21:26
dansmithcdent: very few people install the release shortly after the release21:27
cdentI know, but that was the pitch21:27
mriedemhttp://lists.openstack.org/pipermail/openstack-dev/2017-September/121783.html21:27
mriedem"Sydney  marks the start of Rocky's release cycle, where ideas and requirements  will be gathered. Users should aim to come armed with feedback from  Augusts's Pike release if at all possible."21:27
dansmiththere was definitely a "people who need a midcycle could do it during the forum" argument21:27
cdent“on the priorities and 'blue sky' ideas for the next release.” is both terrifying and exciting21:28
*** armax has joined #openstack-nova21:28
mriedemwe had some sessions in boston which were "here is a problem, this is what we plan on doing about it, does that work for you opsy people?"21:28
mriedemi don't know what we have like that for queens21:28
*** Sukhdev has joined #openstack-nova21:28
cdenthuh, I thought ptg was the replacement for the midcycle, not forum for the midcycle21:28
cdentmriedem: is that because the dance card is full?21:28
dansmiththe format of the midcycle was moved to ptg21:28
dansmithinstead of design summit style21:29
mriedemand the ptg replaced the dev summit at the start of the release21:29
dansmithbut they shifted the schedle21:29
dansmithright21:29
mriedemso now it's semi scheduled blocks of things in a midcycle setting with no ops or users21:29
mriedemwell, some ops21:29
mriedemcburgess and med_ <321:30
edleafecdent: PTG is supposed to be planning for the upcoming cycle. Forum is supposed to be more like a midcycle21:30
mriedemexcept midcycles of yore weren't time boxed21:30
mriedemand didn't cost $3000 to go to21:30
dansmithmore like a midcycle, meaning scheduling21:30
cdentedleafe: that’s not at all my understanding, and nor does it correspond to the email that mriedem just linked to21:31
dansmithcdent: certainly you've noticed that the early messaging, the late messaging, and the actual intent for changing things are totally disjoint :)21:31
mriedemthere are numerous flame wars on what the ptg is and isn't, or supposed to be or not, in the ML, so probably don't need to enumerate those here21:31
cdentdansmith: yes, very true, but I _never_ thought the forum was a midcycle21:31
dansmithright, all I'm saying is I hope that if we need to have discussions for in-progress things, we can do that as that was part of why the midcycle was important.. for rescuing things that were flailing before it was too late21:32
mriedemdansmith: we can, if you can figure out what those things are going to be in the next 8 days :)21:32
edleafecdent: no, not a midcycle, but that would be the time to make sure you are on course for completing what you set out to do at the PTG21:32
edleafedansmith: sorta jinx21:32
cdentif that’s an expectation people have, I think they will be disappointed21:32
dansmithedleafe: exactly21:32
mriedemdansmith: and minus a lot of the developers involved21:33
cdenta) there won’t be time, b) there won’t be the people21:33
dansmithmriedem: certainly for this one21:33
cdentb jinx with mriedem21:33
cdentwe should treat it as feedback time21:33
mriedem^ is why i'd rather just sit in on ops sessions21:33
edleafemany developers couldn't get companies to pay for the midcycles, since it wasn't an "official" event21:33
cdentmriedem: maybe just make that so then? state: we’re not gonna do nova-oriented forum sessions, we’re going to go to ops ones21:34
mriedemcdent: that will be the case if we don't have anything to propose21:34
mriedemwhich might just happen naturally21:34
*** Apoorva_ has quit IRC21:34
cdentI suspect most people whould approve of that, especially if you spread the idea around21:34
mriedemultimately i just don't want to have to propose topics to fill time21:35
*** Apoorva has joined #openstack-nova21:35
mriedems/i/us/we/everyone/21:35
dansmiththe topics we proposed in boston were pretty well-attended by ops no?21:35
mriedemyeah21:36
mriedemsome of the stuff we talked about in boston is just carrying over into queens21:37
mriedemso that's where i'm thinking, this isn't news, this is stuff we're still working on21:37
openstackgerritEric Berglund proposed openstack/nova master: Add PowerVM hypervisor configuration doc  https://review.openstack.org/50566521:37
dansmithI think if we don't propose sessions to get specific feedback from ops, we'll be perceived as not caring to gather input, even if the actual goal is to attend other things21:37
mriedemi don't know what specifically we're trying to get feedback on21:38
mriedemand we've done the general ops feedback sessions in the past which weren't well attended21:38
*** tbachman has joined #openstack-nova21:39
mriedemgetting feedback from pike would be great, hell even ocata21:39
dansmithisn't that likely to be the outcome if we just say we're not proposing things and expect everyone else to do it?21:39
mriedemso are you thinking like a pike feedback session?21:40
*** artom has quit IRC21:40
*** artom has joined #openstack-nova21:41
dansmithI think that by the time that rolls around several of the people we know of as early movers will be on pike already21:41
dansmithcern, vexxhost already being there, etc21:41
mriedemL10 in https://etherpad.openstack.org/p/SYD-ops-session-ideas21:42
mriedemwe do have the project update talk for pike21:42
dansmithsurely someone has an opinion on the quotas changes by that point21:42
mriedemi'm totally fine with a pike feedback / update session if people are armed with issues/questions21:42
melwittI think a good topic would be something like "upgrades", how did it go when you upgraded nova and what issues did you run into, what would make it easier21:42
mriedemok, so,21:43
mriedemyou all know you can throw stuff in this etherpad right?21:43
openstackgerritOpenStack Proposal Bot proposed openstack/os-traits master: Updated from global requirements  https://review.openstack.org/50364621:43
melwittI know we've covered it before in the past but it's kind of a moving target I think. each release has its own upgrade gotchas21:43
melwittyes, sorry21:43
* melwitt goes to etherpad21:43
dansmithmriedem: sure, but ya'll were arguing about not having anything here21:43
*** tbachman has quit IRC21:44
mriedemi'm basically just trolling myself at this point21:44
melwittin my case, my brain is totally fried from the PTG and the ideas are emerging slowly21:44
mriedemright21:44
mriedemsame here21:44
dansmithmriedem: do you want my alternate color up on L20 or to delete your dupes on L37?21:45
mriedemi would love to have ops people come armed with issues/questions for a pike feedback session21:45
dansmither, 38-3921:45
*** zhenguo has quit IRC21:45
*** bauzas has quit IRC21:45
*** redondo-mk has quit IRC21:45
mriedema lot of times i feel like we get a room full of people and then are pushing the topics on them, without much response unless you cattle prod someone21:45
*** adisky has quit IRC21:46
dansmithsame goes for us just waiting for other people to propose topics21:46
*** thorst has quit IRC21:46
edleafemriedem: are you bringing a cattle prod to Sydney?21:46
*** vladikr has quit IRC21:46
mriedemedleafe: kangerprod21:46
mriedemi think you can get them at the corner store21:46
*** coreycb has quit IRC21:46
*** simondodsley has quit IRC21:46
*** jbryce has quit IRC21:46
edleafedamn you, I was typing that!21:46
*** tbachman has joined #openstack-nova21:46
*** gmann has quit IRC21:47
*** efried has quit IRC21:47
*** evrardjp has quit IRC21:47
*** rm_work has quit IRC21:47
*** dr_gogeta86 has quit IRC21:48
*** sergek_ has quit IRC21:48
*** coreycb has joined #openstack-nova21:48
*** simondodsley has joined #openstack-nova21:48
*** adisky has joined #openstack-nova21:48
*** amotoki has quit IRC21:48
*** tanee has quit IRC21:49
*** pkoniszewski has quit IRC21:49
*** MVenesio has quit IRC21:49
*** tanee has joined #openstack-nova21:50
*** sergek_ has joined #openstack-nova21:50
*** pkoniszewski has joined #openstack-nova21:51
*** amotoki has joined #openstack-nova21:51
*** evrardjp has joined #openstack-nova21:52
*** bauzas has joined #openstack-nova21:52
*** dr_gogeta86 has joined #openstack-nova21:53
*** dr_gogeta86 has quit IRC21:53
*** dr_gogeta86 has joined #openstack-nova21:53
*** felipemonteiro__ has quit IRC21:59
*** felipemonteiro__ has joined #openstack-nova22:00
*** zhenguo has joined #openstack-nova22:01
*** redondo-mk has joined #openstack-nova22:01
*** rm_work has joined #openstack-nova22:02
*** rm_work has quit IRC22:02
*** rm_work has joined #openstack-nova22:02
*** efried has joined #openstack-nova22:03
*** owalsh_ has joined #openstack-nova22:04
dansmithmriedem: did you see this? https://review.openstack.org/#/c/506370/22:05
dansmithI don't think the bot barfed it into here else I missed it22:06
*** felipemonteiro__ has quit IRC22:06
mriedemha, no22:06
*** owalsh- has joined #openstack-nova22:06
*** kylek3h has quit IRC22:06
*** owalsh has quit IRC22:07
*** owalsh- is now known as owalsh22:07
mriedemhttp://docs-draft.openstack.org/70/506370/1/check/gate-nova-docs-ubuntu-xenial/08cdb00//doc/build/html/reference/live-migration.html22:07
dansmithI almost have migration_uuid claims working for cold migration22:07
dansmithgibi laid down the gauntlet with his tests22:08
*** owalsh_ has quit IRC22:09
mriedemefried: you're a docs machine22:09
efriedmriedem I was waiting for that to build so I could send you the rendered version.  But you what, did it locally?22:10
mriedemno looked at the job results22:10
efriedoh, you found it before it was done, cool.22:10
mriedemstatus.openstack.org/zuul22:10
melwitthe probably spied it on the zuul status page22:10
efriedSo it ain't perfect, but it's not bad.22:10
melwittdammit22:10
mriedemefried: 1000% better than what we have today, which is nothing22:10
mriedemyou'll have to add that new dependency to global-requirements22:11
efriedIt's already there22:11
*** ijw has joined #openstack-nova22:13
efriedOh - I misread the dot version: https://github.com/openstack/requirements/blob/master/global-requirements.txt#L42422:13
efriedmriedem Do you want to add some words around that doc in this patch?22:13
openstackgerritEric Fried proposed openstack/nova master: Live Migration sequence diagram  https://review.openstack.org/50637022:15
*** bigdogstl has joined #openstack-nova22:16
efriedmriedem Fixed that version and added co-author line ^22:16
mriedemwe can describe the operations in a follow up22:17
*** hferenc has quit IRC22:17
mriedemnot as pretty is right!22:17
*** slaweq has joined #openstack-nova22:17
mriedemnot sure why the reqs job was failing then22:18
*** hferenc has joined #openstack-nova22:18
mriedemoh it's the space22:18
mriedemit literally has to match "sphinxcontrib-seqdiag>=0.8.4  # BSD"22:18
*** artom has quit IRC22:19
mriedemmaybe not, this might work22:19
*** artom has joined #openstack-nova22:20
efriedI had it as .5 before22:20
mriedemyup see that now22:20
mriedemcool, this should pass22:20
mriedemglorious22:20
efriedyeah, it's not complaining about the actdiag one, which has two spaces before the comment as well.22:20
*** artom has quit IRC22:21
*** artom has joined #openstack-nova22:22
*** artom has quit IRC22:22
*** slaweq has quit IRC22:22
*** bigdogstl has quit IRC22:23
*** artom has joined #openstack-nova22:23
*** artom has quit IRC22:23
*** ioggstream has joined #openstack-nova22:23
*** artom has joined #openstack-nova22:24
*** cdent has quit IRC22:27
*** penick_ has quit IRC22:31
*** artom has quit IRC22:32
*** sdague has quit IRC22:34
*** mriedem has quit IRC22:37
*** xyang1 has quit IRC22:38
*** slaweq has joined #openstack-nova22:39
*** lyan has quit IRC22:40
openstackgerritMerged openstack/nova master: Drop support for the Cinder v2 API  https://review.openstack.org/50187422:42
*** tidwellr has quit IRC22:43
openstackgerritJay Pipes proposed openstack/nova master: placement: set/check if inventory change in tree  https://review.openstack.org/47057522:45
openstackgerritJay Pipes proposed openstack/nova master: placement: integrate ProviderTree to report client  https://review.openstack.org/41592122:45
openstackgerritJay Pipes proposed openstack/nova master: placement: add nested resource providers  https://review.openstack.org/37713822:45
openstackgerritJay Pipes proposed openstack/nova master: placement: allow filter providers in tree  https://review.openstack.org/37721522:45
openstackgerritJay Pipes proposed openstack/nova master: placement: adds REST API for nested providers  https://review.openstack.org/38480722:45
*** yassine_ has joined #openstack-nova22:45
openstackgerritJay Pipes proposed openstack/nova master: placement: update client to set parent provider  https://review.openstack.org/38569322:45
*** slaweq has quit IRC22:45
*** tbachman has quit IRC22:53
*** tidwellr has joined #openstack-nova22:53
*** bigdogstl has joined #openstack-nova22:53
*** Sree has joined #openstack-nova22:54
*** Sukhdev has quit IRC22:57
*** edmondsw has joined #openstack-nova22:58
*** Sree has quit IRC22:59
*** bigdogstl has quit IRC23:00
*** edmondsw has quit IRC23:02
*** penick has joined #openstack-nova23:02
*** slaweq has joined #openstack-nova23:03
*** penick has quit IRC23:06
*** bigdogstl has joined #openstack-nova23:06
*** slaweq has quit IRC23:08
*** bigdogstl has quit IRC23:11
*** ioggstream has quit IRC23:12
*** tasker has quit IRC23:13
openstackgerritMatt Riedemann proposed openstack/nova master: Add recreate test for unshelve offloaded instance spawn fail  https://review.openstack.org/50641423:15
*** hamzy has joined #openstack-nova23:15
*** claudiub has quit IRC23:17
openstackgerritMatt Riedemann proposed openstack/nova master: Add a regression test for bug 1718455  https://review.openstack.org/50609223:18
openstackbug 1718455 in OpenStack Compute (nova) "[pike] Nova host disable and Live Migrate all instances fail." [Medium,In progress] https://launchpad.net/bugs/1718455 - Assigned to Sylvain Bauza (sylvain-bauza)23:18
*** ijw has quit IRC23:18
openstackgerritMatt Riedemann proposed openstack/nova master: Fix definitely move single instance when created concurrently  https://review.openstack.org/50609323:19
*** hongbin has quit IRC23:23
*** slaweq has joined #openstack-nova23:24
*** gmann has joined #openstack-nova23:29
openstackgerritHongbin Li proposed openstack/nova master: Remove method "_get_host_ref_from_name"  https://review.openstack.org/50479623:31
*** slaweq has quit IRC23:31
*** chyka has quit IRC23:34
*** chyka has joined #openstack-nova23:34
*** jmlowe has joined #openstack-nova23:35
openstackgerritDan Smith proposed openstack/nova master: Fix a pagination logic bug in test_bug_1689692  https://review.openstack.org/50566123:35
openstackgerritDan Smith proposed openstack/nova master: Use improved instance_list module in compute API  https://review.openstack.org/50541823:35
openstackgerritDan Smith proposed openstack/nova master: Remove legacy fault-loading routines  https://review.openstack.org/50545623:35
openstackgerritDan Smith proposed openstack/nova master: Fix CellDatabases fixture swallowing exceptions  https://review.openstack.org/50631223:35
openstackgerritDan Smith proposed openstack/nova master: Fix minor input items from previous patches  https://review.openstack.org/50641623:35
*** gouthamr has joined #openstack-nova23:40
*** markvoelker has quit IRC23:40
*** bigdogstl has joined #openstack-nova23:41
*** takashin has joined #openstack-nova23:42
*** tbachman has joined #openstack-nova23:43
*** jbryce has joined #openstack-nova23:45
*** thorst has joined #openstack-nova23:46
*** Swami has quit IRC23:46
*** slaweq has joined #openstack-nova23:48
*** bigdogstl has quit IRC23:50
*** thorst has quit IRC23:50
*** thorst has joined #openstack-nova23:50
*** thorst has quit IRC23:53
*** slaweq has quit IRC23:54

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