*** slaweq has joined #openstack-nova | 00:02 | |
*** namnh has quit IRC | 00:04 | |
*** slaweq has quit IRC | 00:06 | |
*** gyee has quit IRC | 00:07 | |
*** Apoorva_ has quit IRC | 00:08 | |
*** Apoorva has joined #openstack-nova | 00:08 | |
*** cdent has quit IRC | 00:10 | |
*** ociuhandu_ has joined #openstack-nova | 00:12 | |
*** ociuhandu has quit IRC | 00:12 | |
*** Sukhdev_ has quit IRC | 00:13 | |
*** thorst has joined #openstack-nova | 00:16 | |
*** itlinux has quit IRC | 00:16 | |
*** takedakn has joined #openstack-nova | 00:18 | |
*** takedakn has quit IRC | 00:19 | |
*** takedakn has joined #openstack-nova | 00:21 | |
*** ijw has quit IRC | 00:23 | |
*** ijw has joined #openstack-nova | 00:23 | |
*** sbezverk has quit IRC | 00:25 | |
*** sbezverk has joined #openstack-nova | 00:26 | |
*** jmlowe has joined #openstack-nova | 00:27 | |
*** bigdogstl has joined #openstack-nova | 00:29 | |
*** Apoorva_ has joined #openstack-nova | 00:30 | |
*** zhurong has joined #openstack-nova | 00:31 | |
*** slaweq has joined #openstack-nova | 00:32 | |
*** Apoorva has quit IRC | 00:34 | |
*** Apoorva_ has quit IRC | 00:34 | |
*** bigdogstl has quit IRC | 00:36 | |
*** hshiina has joined #openstack-nova | 00:36 | |
*** baoli has joined #openstack-nova | 00:36 | |
*** baoli has quit IRC | 00:36 | |
*** slaweq has quit IRC | 00:36 | |
*** itlinux has joined #openstack-nova | 00:38 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements https://review.openstack.org/505839 | 00:39 |
---|---|---|
*** openstackgerrit has quit IRC | 00:41 | |
*** openstackgerrit has joined #openstack-nova | 00:41 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/os-traits master: Updated from global requirements https://review.openstack.org/503646 | 00:41 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/os-vif master: Updated from global requirements https://review.openstack.org/502708 | 00:41 |
*** slaweq has joined #openstack-nova | 00:42 | |
*** Swami has quit IRC | 00:45 | |
*** slaweq has quit IRC | 00:46 | |
*** Sree has joined #openstack-nova | 00:48 | |
*** TuanLA has joined #openstack-nova | 00:50 | |
*** slaweq has joined #openstack-nova | 00:51 | |
*** Sree has quit IRC | 00:52 | |
*** bigdogstl has joined #openstack-nova | 00:53 | |
*** thorst has quit IRC | 00:54 | |
*** litao__ has joined #openstack-nova | 00:54 | |
*** jichen has joined #openstack-nova | 00:54 | |
*** kenperkins has quit IRC | 00:56 | |
*** slaweq has quit IRC | 00:56 | |
*** hoonetorg has joined #openstack-nova | 00:56 | |
*** takedakn has quit IRC | 00:57 | |
*** slaweq has joined #openstack-nova | 01:01 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/os-vif stable/pike: Updated from global requirements https://review.openstack.org/493146 | 01:04 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/python-novaclient stable/pike: Updated from global requirements https://review.openstack.org/493187 | 01:06 |
*** gbarros has joined #openstack-nova | 01:07 | |
*** phuongnh has joined #openstack-nova | 01:07 | |
*** slaweq has quit IRC | 01:08 | |
*** bigdogstl has quit IRC | 01:15 | |
*** trungnv has joined #openstack-nova | 01:16 | |
*** gbarros has quit IRC | 01:18 | |
*** dave-mccowan has joined #openstack-nova | 01:22 | |
mriedem | owalsh: 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/1717915 | 01:23 |
openstack | Launchpad bug 1717915 in oslo.messaging "nova services and transport_url, cannot connect to vhost if specified" [Undecided,New] | 01:23 |
*** READ10 has joined #openstack-nova | 01:25 | |
*** suresh12 has quit IRC | 01:28 | |
*** itlinux has quit IRC | 01:28 | |
openstackgerrit | jichenjc proposed openstack/nova master: Add quiesce and unquiesce in support matrix https://review.openstack.org/482390 | 01:30 |
*** kevi9132 has joined #openstack-nova | 01:32 | |
*** kevi9132_ has quit IRC | 01:32 | |
openstackgerrit | jichenjc proposed openstack/nova master: check query param for service's index function https://review.openstack.org/489492 | 01:33 |
*** dave-mcc_ has joined #openstack-nova | 01:34 | |
*** dave-mccowan has quit IRC | 01:35 | |
openstackgerrit | Matt Riedemann proposed openstack/nova stable/pike: Split out the core of the ironic flavor migration https://review.openstack.org/505901 | 01:36 |
openstackgerrit | Matt Riedemann proposed openstack/nova stable/pike: Add ComputeNodeList.get_by_hypervisor_type() https://review.openstack.org/505902 | 01:36 |
openstackgerrit | Matt Riedemann proposed openstack/nova stable/pike: Add nova-manage db command for ironic flavor migrations https://review.openstack.org/505903 | 01:36 |
*** Sukhdev has quit IRC | 01:39 | |
*** jmlowe has quit IRC | 01:42 | |
*** jmlowe has joined #openstack-nova | 01:43 | |
openstackgerrit | jichenjc proposed openstack/nova master: Remove 400 as expected error https://review.openstack.org/505485 | 01:45 |
*** yamahata has quit IRC | 01:46 | |
*** itlinux has joined #openstack-nova | 01:49 | |
*** mriedem has quit IRC | 01:51 | |
*** itlinux has quit IRC | 01:51 | |
*** kevi9132_ has joined #openstack-nova | 01:52 | |
*** mingyu has quit IRC | 01:53 | |
*** thorst has joined #openstack-nova | 01:53 | |
*** thorst has quit IRC | 01:53 | |
*** crushil has joined #openstack-nova | 01:54 | |
*** kevi9132 has quit IRC | 01:55 | |
*** amotoki has quit IRC | 01:55 | |
*** amotoki has joined #openstack-nova | 01:56 | |
*** hamzy has quit IRC | 01:57 | |
*** hamzy has joined #openstack-nova | 01:57 | |
*** ijw has quit IRC | 01:58 | |
*** mingyu has joined #openstack-nova | 01:59 | |
*** kaisers_ has quit IRC | 02:00 | |
*** ijw has joined #openstack-nova | 02:01 | |
openstackgerrit | Merged openstack/nova master: Optimize MiniDNS for fewer syscalls https://review.openstack.org/486829 | 02:01 |
openstackgerrit | jichenjc proposed openstack/nova master: Fix warning on {'cell_id': 1} is an invalid UUID https://review.openstack.org/505471 | 02:01 |
*** slaweq has joined #openstack-nova | 02:03 | |
*** hongbin has joined #openstack-nova | 02:04 | |
*** ijw has quit IRC | 02:05 | |
*** slaweq has quit IRC | 02:08 | |
*** Shunli has joined #openstack-nova | 02:10 | |
*** READ10 has quit IRC | 02:12 | |
*** trungnv has quit IRC | 02:16 | |
*** gouthamr has joined #openstack-nova | 02:19 | |
*** crushil has quit IRC | 02:20 | |
*** dave-mcc_ has quit IRC | 02:23 | |
*** lihg has joined #openstack-nova | 02:24 | |
*** lihg_ has joined #openstack-nova | 02:25 | |
*** lihg has quit IRC | 02:25 | |
*** lihg_ has quit IRC | 02:25 | |
*** lihg has joined #openstack-nova | 02:25 | |
*** lihg_ has joined #openstack-nova | 02:25 | |
*** bigdogstl has joined #openstack-nova | 02:25 | |
*** lihg_ has left #openstack-nova | 02:26 | |
*** trungnv has joined #openstack-nova | 02:28 | |
*** efried has quit IRC | 02:31 | |
*** efried has joined #openstack-nova | 02:32 | |
*** bigdogstl has quit IRC | 02:33 | |
masber | good 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/Docker | 02:36 |
*** vladikr has quit IRC | 02:41 | |
*** vladikr has joined #openstack-nova | 02:41 | |
*** awaugama has joined #openstack-nova | 02:44 | |
*** mingyu has quit IRC | 02:45 | |
*** vladikr has quit IRC | 02:47 | |
*** mingyu has joined #openstack-nova | 02:47 | |
*** vladikr has joined #openstack-nova | 02:47 | |
*** mingyu has quit IRC | 02:48 | |
*** coreywright has quit IRC | 02:49 | |
*** kevi9132_ has quit IRC | 02:51 | |
*** kevi9132 has joined #openstack-nova | 02:52 | |
*** coreywright has joined #openstack-nova | 02:52 | |
*** vladikr has quit IRC | 02:52 | |
*** vladikr has joined #openstack-nova | 02:53 | |
*** thorst has joined #openstack-nova | 02:53 | |
*** awaugama has quit IRC | 02:56 | |
*** kevi9132 has quit IRC | 02:56 | |
*** thorst has quit IRC | 02:59 | |
*** mfisch` is now known as mfisch | 02:59 | |
*** mfisch is now known as Guest44234 | 03:00 | |
*** lbragstad has joined #openstack-nova | 03:00 | |
*** lihg__ has joined #openstack-nova | 03:01 | |
*** lihg has quit IRC | 03:01 | |
*** baoli has joined #openstack-nova | 03:01 | |
*** lihg has joined #openstack-nova | 03:02 | |
*** slaweq has joined #openstack-nova | 03:02 | |
gmann | jichen: sorry about api-ref comment which is not valid - https://review.openstack.org/#/c/505485/ | 03:07 |
gmann | jichen: your PS3 was good to go | 03:07 |
*** slaweq has quit IRC | 03:07 | |
jichen | gmann: ok, got it, you are right, I need keep 400 as validation stuff, thanks :) | 03:08 |
gmann | jichen: thanks | 03:09 |
*** vladikr has quit IRC | 03:10 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements https://review.openstack.org/505839 | 03:10 |
*** vladikr has joined #openstack-nova | 03:11 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/os-traits master: Updated from global requirements https://review.openstack.org/503646 | 03:13 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/os-vif master: Updated from global requirements https://review.openstack.org/502708 | 03:13 |
openstackgerrit | jichenjc proposed openstack/nova master: Remove 400 as expected error https://review.openstack.org/505485 | 03:15 |
*** lbragstad has quit IRC | 03:21 | |
*** armax has quit IRC | 03:22 | |
*** baoli has quit IRC | 03:23 | |
*** Sukhdev has joined #openstack-nova | 03:24 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements https://review.openstack.org/505839 | 03:27 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/os-traits master: Updated from global requirements https://review.openstack.org/503646 | 03:29 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/os-vif master: Updated from global requirements https://review.openstack.org/502708 | 03:29 |
*** bigdogstl has joined #openstack-nova | 03:31 | |
*** namnh has joined #openstack-nova | 03:34 | |
*** bigdogstl has quit IRC | 03:35 | |
*** trinaths has joined #openstack-nova | 03:39 | |
*** udesale has joined #openstack-nova | 03:41 | |
*** trinaths1 has joined #openstack-nova | 03:41 | |
*** hshiina has quit IRC | 03:42 | |
*** slaweq has joined #openstack-nova | 03:42 | |
*** cfriesen has quit IRC | 03:43 | |
*** trinaths has quit IRC | 03:44 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements https://review.openstack.org/505839 | 03:45 |
*** hshiina has joined #openstack-nova | 03:46 | |
*** Sree has joined #openstack-nova | 03:46 | |
*** slaweq has quit IRC | 03:47 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/os-traits master: Updated from global requirements https://review.openstack.org/503646 | 03:48 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/os-vif master: Updated from global requirements https://review.openstack.org/502708 | 03:48 |
*** yamahata has joined #openstack-nova | 03:49 | |
*** slaweq has joined #openstack-nova | 03:52 | |
*** Guest44234 is now known as mfisch | 03:52 | |
*** mfisch is now known as Guest13446 | 03:53 | |
openstackgerrit | Takashi NATSUME proposed openstack/nova master: Enable cold migration with target host(1/2) https://review.openstack.org/408955 | 03:54 |
*** thorst has joined #openstack-nova | 03:55 | |
*** suresh12 has joined #openstack-nova | 03:55 | |
*** suresh12 has quit IRC | 03:56 | |
*** slaweq has quit IRC | 03:56 | |
*** takedakn has joined #openstack-nova | 03:58 | |
*** thorst has quit IRC | 03:59 | |
*** bigdogstl has joined #openstack-nova | 04:00 | |
*** takashin has joined #openstack-nova | 04:03 | |
*** erlon has quit IRC | 04:03 | |
openstackgerrit | Takashi NATSUME proposed openstack/nova master: Enable cold migration with target host(2/2) https://review.openstack.org/408964 | 04:05 |
*** hongbin has quit IRC | 04:06 | |
*** kbaegis has quit IRC | 04:08 | |
*** kenperkins has joined #openstack-nova | 04:08 | |
*** bigdogstl has quit IRC | 04:11 | |
*** gouthamr has quit IRC | 04:12 | |
*** takedakn has quit IRC | 04:16 | |
*** chyka has quit IRC | 04:17 | |
*** chyka has joined #openstack-nova | 04:18 | |
*** chyka has quit IRC | 04:22 | |
*** bigdogstl has joined #openstack-nova | 04:38 | |
*** claudiub|2 has joined #openstack-nova | 04:40 | |
*** psachin has joined #openstack-nova | 04:41 | |
*** bigdogstl has quit IRC | 04:43 | |
*** links has joined #openstack-nova | 04:43 | |
*** thorst has joined #openstack-nova | 04:55 | |
*** mdnadeem has joined #openstack-nova | 04:56 | |
*** itlinux has joined #openstack-nova | 05:00 | |
*** thorst has quit IRC | 05:00 | |
*** shan has joined #openstack-nova | 05:06 | |
*** sridharg has joined #openstack-nova | 05:09 | |
*** pcaruana has joined #openstack-nova | 05:09 | |
*** markvoelker has quit IRC | 05:16 | |
*** markvoelker has joined #openstack-nova | 05:17 | |
*** markvoelker has quit IRC | 05:22 | |
*** Sukhdev has quit IRC | 05:23 | |
openstackgerrit | Takashi NATSUME proposed openstack/nova-specs master: List/show all server migration types https://review.openstack.org/489029 | 05:27 |
*** udesale__ has joined #openstack-nova | 05:35 | |
*** ratailor has joined #openstack-nova | 05:36 | |
openstackgerrit | Hironori Shiina proposed openstack/nova-specs master: Ironic: Resize and cold migration support https://review.openstack.org/449155 | 05:36 |
*** udesale has quit IRC | 05:38 | |
*** bigdogstl has joined #openstack-nova | 05:40 | |
*** bigdogstl has quit IRC | 05:44 | |
*** udesale has joined #openstack-nova | 05:50 | |
jichen | gmann: could you please help check https://review.openstack.org/#/c/505485/ ? I think gerrit has some issue | 05:50 |
*** slaweq has joined #openstack-nova | 05:51 | |
jichen | so PS5's update didn't take effect and your PS4 -1 is still there.. | 05:51 |
*** udesale__ has quit IRC | 05:52 | |
gmann | jichen: done. | 05:52 |
jichen | gmann: thanks~ | 05:52 |
gmann | jichen: 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 not | 05:53 |
*** lihg_ has joined #openstack-nova | 05:53 | |
gmann | jichen: if you see my reply actually that is handled in schema | 05:54 |
*** lihg has quit IRC | 05:54 | |
*** lihg__ has quit IRC | 05:54 | |
*** lihg has joined #openstack-nova | 05:54 | |
*** slaweq has quit IRC | 05:55 | |
gmann | the schema and delete flag is actually very kind of hidden in that and does all magic | 05:55 |
jichen | yes, that's also my question , I just saw the mail, yes, exactly, it's hidden in the code logic and I missed that | 05:56 |
jichen | gmann: thanks for point that out and I think that's a little bit hack, we have update_all which can handle one key situation | 05:56 |
*** thorst has joined #openstack-nova | 05:56 | |
jichen | I mean we have update_all already and update one key like the API implemented looks to me seems useless and confusing | 05:57 |
*** rmcall has quit IRC | 05:57 | |
gmann | jichen: but both have separate purpose, update_all remove the existing keys and update by key does not | 05:57 |
jichen | gmann: ah...ok , you are right | 05:58 |
jichen | thanks for the info, I will check the code again , thanks~ | 05:59 |
*** thorst has quit IRC | 06:01 | |
*** rmcall has joined #openstack-nova | 06:02 | |
*** avolkov has joined #openstack-nova | 06:02 | |
*** vito_ has joined #openstack-nova | 06:04 | |
*** vito_ is now known as Guest92796 | 06:04 | |
*** bigdogstl has joined #openstack-nova | 06:05 | |
*** rmcall has quit IRC | 06:06 | |
*** coreywright has quit IRC | 06:08 | |
*** bigdogstl has quit IRC | 06:09 | |
*** claudiub has joined #openstack-nova | 06:10 | |
*** claudiub|2 has quit IRC | 06:12 | |
*** Oku_OS-away is now known as Oku_OS | 06:13 | |
*** kenperkins has quit IRC | 06:15 | |
openstackgerrit | Alex Xu proposed openstack/nova-specs master: Add trait support in the allocation candidates API https://review.openstack.org/497713 | 06:16 |
*** lihg__ has joined #openstack-nova | 06:16 | |
*** lihg_ has quit IRC | 06:17 | |
*** lihg has quit IRC | 06:17 | |
*** lihg has joined #openstack-nova | 06:18 | |
*** andreas_s has joined #openstack-nova | 06:21 | |
*** Guest92796 has quit IRC | 06:25 | |
*** josecastroleon has joined #openstack-nova | 06:27 | |
*** rcernin has joined #openstack-nova | 06:28 | |
*** udesale has quit IRC | 06:29 | |
*** udesale has joined #openstack-nova | 06:30 | |
*** coreywright has joined #openstack-nova | 06:30 | |
*** lajoskatona has joined #openstack-nova | 06:32 | |
*** itlinux has quit IRC | 06:37 | |
*** lihg has quit IRC | 06:38 | |
*** lihg_ has joined #openstack-nova | 06:38 | |
*** lihg__ has quit IRC | 06:39 | |
*** lihg has joined #openstack-nova | 06:39 | |
*** itlinux has joined #openstack-nova | 06:40 | |
*** itlinux has quit IRC | 06:40 | |
*** edand has joined #openstack-nova | 06:40 | |
*** ralonsoh has joined #openstack-nova | 06:45 | |
*** kevi9132 has joined #openstack-nova | 06:53 | |
*** thorst has joined #openstack-nova | 06:57 | |
*** xinliang has quit IRC | 06:57 | |
*** kevi9132 has quit IRC | 06:58 | |
*** thorst has quit IRC | 07:02 | |
*** pcaruana has quit IRC | 07:05 | |
*** sahid has joined #openstack-nova | 07:08 | |
*** ioggstream has joined #openstack-nova | 07:09 | |
*** xinliang has joined #openstack-nova | 07:10 | |
*** mingyu has joined #openstack-nova | 07:13 | |
*** mingyu has quit IRC | 07:14 | |
*** ChanServ sets mode: +o openstack | 07:15 | |
*** bigdogstl has joined #openstack-nova | 07:17 | |
*** markvoelker has joined #openstack-nova | 07:18 | |
*** sapcc-bot has quit IRC | 07:18 | |
*** sapcc-bot1 has joined #openstack-nova | 07:18 | |
*** NehaAlhat has joined #openstack-nova | 07:19 | |
*** tesseract has joined #openstack-nova | 07:19 | |
*** bigdogstl has quit IRC | 07:22 | |
*** neha_alhat has quit IRC | 07:22 | |
*** trungnv has quit IRC | 07:22 | |
*** pcaruana has joined #openstack-nova | 07:23 | |
*** alexchadin has joined #openstack-nova | 07:29 | |
*** slaweq has joined #openstack-nova | 07:30 | |
*** sridharg has quit IRC | 07:31 | |
*** sridharg has joined #openstack-nova | 07:32 | |
*** slaweq has quit IRC | 07:34 | |
*** zhurong has quit IRC | 07:35 | |
*** Tom_ has joined #openstack-nova | 07:37 | |
*** chyka has joined #openstack-nova | 07:39 | |
*** slaweq has joined #openstack-nova | 07:40 | |
*** jpena|off is now known as jpena | 07:41 | |
*** chyka has quit IRC | 07:43 | |
*** slaweq has quit IRC | 07:44 | |
openstackgerrit | Yikun Jiang proposed openstack/nova-specs master: Add pagination and timestamp filtering support for os-migrations API https://review.openstack.org/506030 | 07:48 |
*** slaweq has joined #openstack-nova | 07:50 | |
*** trinaths1 has quit IRC | 07:50 | |
*** markvoelker has quit IRC | 07:52 | |
*** slaweq has quit IRC | 07:54 | |
*** zhurong has joined #openstack-nova | 07:55 | |
gibi | good morning nova | 07:56 |
*** thorst has joined #openstack-nova | 07:58 | |
*** Sree has quit IRC | 08:02 | |
openstackgerrit | Yikun Jiang proposed openstack/nova-specs master: Add pagination and timestamp filtering support for os-migrations API https://review.openstack.org/506030 | 08:02 |
*** Sree has joined #openstack-nova | 08:02 | |
*** thorst has quit IRC | 08:02 | |
*** lajoskatona has quit IRC | 08:11 | |
openstackgerrit | Claudiu Belu proposed openstack/nova-specs master: Adds spec for instance live resize https://review.openstack.org/141219 | 08:11 |
jichen | claudiub: hi | 08:17 |
claudiub | jichen: hello :) | 08:18 |
jichen | appreciate 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 IRC | 08:18 | |
*** lihg_ has quit IRC | 08:18 | |
claudiub | jichen: sure, what's up? | 08:18 |
*** slaweq has joined #openstack-nova | 08:19 | |
openstackgerrit | James Page proposed openstack/nova master: Support qemu >= 2.10 https://review.openstack.org/505748 | 08:19 |
mdbooth | If 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 |
jichen | claudiub: should we update the spec to indicate Matt's comments or you already did the update? | 08:20 |
claudiub | jichen: i didn't include that bit yet. | 08:21 |
claudiub | i'll update it | 08:21 |
mdbooth | Highlight is that it now does log format auto-detection, as well as parsing qemu logs. | 08:21 |
jichen | claudiub: ok, great, thanks ~ | 08:21 |
* mdbooth gets back to actually debugging this customer live migration issue, rather than faffing with tooling | 08:21 | |
claudiub | jichen: do you have any particular requirements regarding live-resize, that haven't been included in the spec? | 08:22 |
jichen | claudiub: 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 cases | 08:23 |
*** slaweq has quit IRC | 08:24 | |
*** nmathew has joined #openstack-nova | 08:24 | |
*** bigdogstl has joined #openstack-nova | 08:28 | |
openstackgerrit | Claudiu Belu proposed openstack/nova-specs master: Adds spec for instance live resize https://review.openstack.org/141219 | 08:29 |
claudiub | jichen: which driver are you most interested in having the live-resize feature? | 08:29 |
jichen | claudiub: 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 feature | 08:31 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: Restore '[vnc] vnc_*' option support https://review.openstack.org/505831 | 08:31 |
stephenfin | efried, alex_xu, mriedem: ^ Fixed that 'vnc_' opt issue | 08:31 |
claudiub | jichen: 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 IRC | 08:33 | |
*** bigdogstl has quit IRC | 08:34 | |
jichen | claudiub: ok, this might take some time :( | 08:34 |
claudiub | jichen: 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 |
claudiub | they'll mostly care about the libvirt implementation, btw. | 08:36 |
jichen | claudiub: 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 well | 08:40 |
*** derekh has joined #openstack-nova | 08:43 | |
*** lajoskatona has joined #openstack-nova | 08:43 | |
claudiub | jichen: sounds good. we could restore, rebase and reuse whatever code was submitted for the feature | 08:43 |
jichen | claudiub: ok, I knew a patch set there , and we may first try to get the spec accepted :) | 08:44 |
claudiub | jichen: yeah. :) maybe the 4th time is the charm. :) | 08:45 |
jichen | claudiub: ok, let's push it again and see what we can do | 08:45 |
*** markvoelker has joined #openstack-nova | 08:48 | |
*** kevi9132 has joined #openstack-nova | 08:55 | |
*** thorst has joined #openstack-nova | 08:59 | |
*** kevi9132 has quit IRC | 08:59 | |
openstackgerrit | Yikun Jiang proposed openstack/nova-specs master: Add pagination and timestamp filtering support for os-migrations API https://review.openstack.org/506030 | 09:01 |
*** lajoskatona has quit IRC | 09:03 | |
*** thorst has quit IRC | 09:03 | |
*** trinaths1 has joined #openstack-nova | 09:06 | |
*** hshiina has quit IRC | 09:06 | |
*** lihg has joined #openstack-nova | 09:07 | |
*** lihg_ has joined #openstack-nova | 09:08 | |
ioggstream | hi@all | 09:09 |
*** chyka has joined #openstack-nova | 09:10 | |
*** bigdogstl has joined #openstack-nova | 09:14 | |
*** chyka has quit IRC | 09:14 | |
*** edmondsw has joined #openstack-nova | 09:15 | |
*** bigdogstl has quit IRC | 09:19 | |
*** edmondsw has quit IRC | 09:20 | |
*** liverpooler has quit IRC | 09:21 | |
*** markvoelker has quit IRC | 09:21 | |
*** yamamoto has quit IRC | 09:21 | |
*** yamamoto has joined #openstack-nova | 09:22 | |
*** tejaswi has joined #openstack-nova | 09:22 | |
tejaswi | hi 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 |
ioggstream | tejaswi: which newton distribution? | 09:24 |
tejaswi | ioggstream: standard openstack distribution - https://docs.openstack.org/newton/install-guide-ubuntu/ | 09:26 |
ioggstream | tejaswi: 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 way | 09:28 |
tejaswi | ioggstream: I tried those instructions suitable for Ubuntu . However, the server got stuck when I changed grub and rebooted it | 09:31 |
*** Shunli has quit IRC | 09:31 | |
ioggstream | tejaswi: I think you should use the content of that doc to search similar references to the ubuntu counterpart | 09:32 |
tejaswi | ioggstream: Yes, I already tried something similar. | 09:32 |
*** alexchadin has quit IRC | 09:34 | |
*** alexchadin has joined #openstack-nova | 09:35 | |
*** lihg has quit IRC | 09:35 | |
*** lihg_ has quit IRC | 09:35 | |
*** sambetts|afk is now known as sambetts | 09:36 | |
*** slaweq has joined #openstack-nova | 09:37 | |
openstackgerrit | Radoslav Gerganov proposed openstack/nova master: Add connection pool size to vSphere settings https://review.openstack.org/414085 | 09:37 |
*** alexchadin has quit IRC | 09:38 | |
tejaswi | ioggstream: I did not find anything similar for Ubuntu. So, I am just checking if anyone did CPU pinning in Ubuntu OpenStack | 09:39 |
*** tesseract has quit IRC | 09:41 | |
*** tesseract has joined #openstack-nova | 09:41 | |
openstackgerrit | Sylvain Bauza proposed openstack/nova master: Add a regression test for bug 1718455 https://review.openstack.org/506092 | 09:42 |
openstack | bug 1718455 in OpenStack Compute (nova) pike "[pike] Nova host disable and Live Migrate all instances fail." [Medium,Confirmed] https://launchpad.net/bugs/1718455 | 09:42 |
openstackgerrit | Sylvain Bauza proposed openstack/nova master: Fix definitely move single instance when created concurrently https://review.openstack.org/506093 | 09:42 |
*** jichen has quit IRC | 09:46 | |
*** slaweq_ has joined #openstack-nova | 09:48 | |
*** mvk has quit IRC | 09:49 | |
*** syqian has joined #openstack-nova | 09:49 | |
*** slaweq_ has quit IRC | 09:53 | |
*** Sree has quit IRC | 09:54 | |
*** zhurong has quit IRC | 09:55 | |
*** thorst has joined #openstack-nova | 10:00 | |
*** thorst has quit IRC | 10:04 | |
*** gszasz has joined #openstack-nova | 10:06 | |
*** alexchadin has joined #openstack-nova | 10:09 | |
*** alexchadin has quit IRC | 10:10 | |
*** zhurong has joined #openstack-nova | 10:11 | |
*** udesale has quit IRC | 10:16 | |
*** markvoelker has joined #openstack-nova | 10:19 | |
*** mvk has joined #openstack-nova | 10:19 | |
*** Tom_ has quit IRC | 10:27 | |
openstackgerrit | James Page proposed openstack/nova master: Support qemu >= 2.10 https://review.openstack.org/505748 | 10:28 |
openstackgerrit | Takashi NATSUME proposed openstack/nova master: Fix hyperlinks in document https://review.openstack.org/506101 | 10:29 |
stephenfin | takashin: One comment left in ^ | 10:33 |
*** bigdogstl has joined #openstack-nova | 10:38 | |
openstackgerrit | Balazs Gibizer proposed openstack/nova master: Add live.migration.force.complete to the legacy notification whitelist https://review.openstack.org/506104 | 10:40 |
*** smatzek has joined #openstack-nova | 10:40 | |
*** smatzek has quit IRC | 10:41 | |
*** smatzek has joined #openstack-nova | 10:41 | |
gibi | stephenfin: it wasn't that soon I promised but here is the bugfix for the force complete legacy notitification whitelisting ^^ | 10:42 |
gibi | stephenfin: now going to rebase the patch we had to stopped yestarday due to this | 10:42 |
openstackgerrit | zhangyanxian proposed openstack/nova-specs master: Sriov-bond https://review.openstack.org/463526 | 10:43 |
*** bigdogstl has quit IRC | 10:45 | |
openstackgerrit | Balazs Gibizer proposed openstack/nova master: Add functional migrate force_complete test https://review.openstack.org/496202 | 10:45 |
openstackgerrit | Balazs Gibizer proposed openstack/nova master: Add functional for live migrate delete https://review.openstack.org/499583 | 10:46 |
*** tejaswi has quit IRC | 10:49 | |
*** ijw has joined #openstack-nova | 10:50 | |
*** markvoelker has quit IRC | 10:52 | |
*** gbarros has joined #openstack-nova | 10:52 | |
*** avolkov has quit IRC | 10:53 | |
*** avolkov has joined #openstack-nova | 10:53 | |
avolkov | c | 10:53 |
*** ijw has quit IRC | 10:54 | |
*** sapd has quit IRC | 10:59 | |
*** Sree has joined #openstack-nova | 10:59 | |
*** jpena is now known as jpena|lunch | 11:00 | |
*** thorst has joined #openstack-nova | 11:00 | |
*** namnh has quit IRC | 11:01 | |
*** dtantsur|afk is now known as dtantsur | 11:02 | |
*** Sree has quit IRC | 11:03 | |
*** tbachman has quit IRC | 11:04 | |
*** claudiub|2 has joined #openstack-nova | 11:05 | |
*** thorst has quit IRC | 11:05 | |
*** claudiub has quit IRC | 11:06 | |
*** Tom has joined #openstack-nova | 11:07 | |
*** trinaths1 has left #openstack-nova | 11:10 | |
*** Sree has joined #openstack-nova | 11:12 | |
*** ociuhandu_ has quit IRC | 11:15 | |
*** tbachman has joined #openstack-nova | 11:21 | |
*** ioggstream has quit IRC | 11:28 | |
*** chyka has joined #openstack-nova | 11:30 | |
*** sapd has joined #openstack-nova | 11:32 | |
*** phuongnh has quit IRC | 11:32 | |
*** nmathew has quit IRC | 11:32 | |
*** ociuhandu has joined #openstack-nova | 11:33 | |
*** chyka has quit IRC | 11:34 | |
*** kbaegis has joined #openstack-nova | 11:36 | |
*** sdague has joined #openstack-nova | 11:38 | |
*** sapd has quit IRC | 11:39 | |
*** syqian has quit IRC | 11:39 | |
*** sapd has joined #openstack-nova | 11:43 | |
*** Tom has quit IRC | 11:46 | |
*** Sree has quit IRC | 11:46 | |
*** Tom has joined #openstack-nova | 11:47 | |
*** lajoskatona has joined #openstack-nova | 11:49 | |
*** markvoelker has joined #openstack-nova | 11:49 | |
*** zhurong has quit IRC | 11:51 | |
*** markvoelker has quit IRC | 11:54 | |
*** sapd has quit IRC | 11:54 | |
*** sapd has joined #openstack-nova | 11:55 | |
*** sapd has quit IRC | 11:56 | |
*** sapd has joined #openstack-nova | 11:56 | |
*** markvoelker has joined #openstack-nova | 11:56 | |
*** litao__ has quit IRC | 11:59 | |
*** thorst has joined #openstack-nova | 11:59 | |
*** TuanLA has quit IRC | 12:00 | |
*** udesale has joined #openstack-nova | 12:02 | |
*** udesale has quit IRC | 12:04 | |
*** udesale has joined #openstack-nova | 12:04 | |
openstackgerrit | Elod Illes proposed openstack/nova master: Functional test: evacuate with no compute https://review.openstack.org/498482 | 12:06 |
*** psachin has quit IRC | 12:06 | |
*** kevi9132 has joined #openstack-nova | 12:07 | |
*** jpena|lunch is now known as jpena | 12:08 | |
*** dave-mccowan has joined #openstack-nova | 12:09 | |
*** bigdogstl has joined #openstack-nova | 12:10 | |
*** yamahata has quit IRC | 12:12 | |
*** edmondsw has joined #openstack-nova | 12:13 | |
openstackgerrit | Elod Illes proposed openstack/nova master: Functional test: evacuate with no compute https://review.openstack.org/498482 | 12:19 |
*** shan has quit IRC | 12:25 | |
*** ociuhandu has quit IRC | 12:25 | |
openstackgerrit | Sean Dague proposed openstack/nova master: Support qemu >= 2.10 https://review.openstack.org/505673 | 12:29 |
*** kylek3h has quit IRC | 12:31 | |
*** jaypipes has joined #openstack-nova | 12:33 | |
*** artom has joined #openstack-nova | 12:33 | |
*** artom has quit IRC | 12:34 | |
*** lyan has joined #openstack-nova | 12:34 | |
*** artom has joined #openstack-nova | 12:34 | |
*** MVenesio has joined #openstack-nova | 12:34 | |
*** erlon has joined #openstack-nova | 12:34 | |
* jaypipes has Internet. Ahoy! | 12:37 | |
edmondsw | jaypipes congrats! | 12:37 |
*** udesale has quit IRC | 12:39 | |
*** abalutoiu has joined #openstack-nova | 12:41 | |
*** gbarros has quit IRC | 12:41 | |
*** catintheroof has joined #openstack-nova | 12:41 | |
*** claudiub has joined #openstack-nova | 12:42 | |
*** kbaegis has quit IRC | 12:42 | |
jaypipes | edmondsw: :) thanks! | 12:43 |
jaypipes | edmondsw: $12K later... https://twitter.com/jaypipes/status/910846347963748354 | 12:43 |
edmondsw | ouch!!! | 12:43 |
*** claudiub|2 has quit IRC | 12:43 | |
jaypipes | edmondsw: this is what I was going in and out of the Nova room on Friday to deal with ;) | 12:43 |
jaypipes | edmondsw: blessed be the A/C. | 12:44 |
edmondsw | jaypipes may it run well and long | 12:44 |
*** mriedem has joined #openstack-nova | 12:46 | |
*** bigdogstl has quit IRC | 12:46 | |
*** Sree has joined #openstack-nova | 12:46 | |
jaypipes | edmondsw: indeed. 12-year, full replacement warranty :) | 12:46 |
*** ociuhandu has joined #openstack-nova | 12:48 | |
tonyb | mriedem: Can you look at: https://review.openstack.org/#/c/493187/ You had an objection in the past that I think has been dealt with | 12:49 |
*** Tom has quit IRC | 12:50 | |
*** xyang1 has joined #openstack-nova | 12:51 | |
jaypipes | tonyb: back home safe and sound? :) | 12:51 |
sdague | jaypipes: congrats on internets and AC | 12:51 |
tonyb | jaypipes: I'm in MA heading home tormorrow | 12:51 |
tonyb | jaypipes: how's the clean-up going? | 12:52 |
jaypipes | sdague: :) thx | 12:52 |
jaypipes | tonyb: 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/shrubs | 12:52 |
jaypipes | tonyb: 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 |
mriedem | tonyb: so we're going to release that in stable/pike as 9.2.0 right? | 12:53 |
tonyb | mriedem: correct. Sadly no other choice :( | 12:53 |
tonyb | jaypipes: :) | 12:54 |
mriedem | we could just not merge that change with the dependency bumps, but does that cause install issues? | 12:54 |
*** ociuhandu has quit IRC | 12:54 | |
tonyb | mriedem: 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 queens | 12:55 |
mriedem | oh it's *my* fault now is it | 12:56 |
tonyb | mriedem: It's all your fault | 12:57 |
bhagyashris | mriedem, johnthetubaguy: Hi, Could you please take some insights on patch https://review.openstack.org/#/c/409644/ . Thank you :) | 12:58 |
*** esberglu has joined #openstack-nova | 13:01 | |
*** Tom has joined #openstack-nova | 13:01 | |
bauzas | jaypipes: hah, if you were not having an A/C replacement, you could just have felt like an European :p | 13:02 |
* bauzas is glad summer is over here | 13:03 | |
stephenfin | bauzas: You take that back. You take that back right now | 13:03 |
* stephenfin misses sunshine :'( | 13:03 | |
bauzas | stephenfin: well, I have sunshine here, but with decent temperatures, that's it :p | 13:04 |
*** abalutoiu has quit IRC | 13:04 | |
* stephenfin is not jealous. Not even slightly... | 13:04 | |
*** Tom has quit IRC | 13:06 | |
mdbooth | mriedem: Did I see a patch from you the other day which added --force-shared to qemu-img everywhere? | 13:06 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements https://review.openstack.org/505839 | 13:07 |
*** pchavva has joined #openstack-nova | 13:08 | |
kashyap | mdbooth: From a quick `grep` of #openstack-nova IRC log, I can't find it here | 13:08 |
jaypipes | bauzas: no thanks :) me and heat don't jive all that well. ;) | 13:08 |
*** gbarros has joined #openstack-nova | 13:08 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/os-traits master: Updated from global requirements https://review.openstack.org/503646 | 13:09 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/os-vif master: Updated from global requirements https://review.openstack.org/502708 | 13:09 |
mdbooth | Found it. Yesterday, 17:34 | 13:09 |
mdbooth | https://review.openstack.org/#/c/505673/ | 13:09 |
*** ioggstream has joined #openstack-nova | 13:11 | |
efried | stephenfin 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 |
bauzas | jaypipes: you don't imagine how much I love sweating... | 13:11 |
edmondsw | efried stephenfin haven't the vnc options in DEFAULT been around long enough that we could just remove deprecated_group for those? | 13:11 |
stephenfin | efried: Yup. I don't see why we shouldn't | 13:11 |
stephenfin | efried, edmondsw: I mean, we _could_ remove them, but it costs us basically nothing to retain them | 13:12 |
kashyap | mdbooth: Ah, thanks | 13:12 |
edmondsw | stephenfin wasn't that always the plan when we first deprecated them? | 13:12 |
mriedem | mdbooth: yes there are 2 patches | 13:13 |
edmondsw | either way... the powervm CI is going to be busted until we get something merged for this | 13:13 |
stephenfin | edmondsw: Yes and no. We do emit warnings that option have been renamed, but we've never gone and removed the old aliases | 13:13 |
mriedem | mdbooth: https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:master+topic:bug/1718295 | 13:13 |
mdbooth | mriedem: I found 1. Interested to know how this ended up on your radar, btw. | 13:13 |
edmondsw | stephenfin k | 13:13 |
*** ociuhandu has joined #openstack-nova | 13:14 | |
stephenfin | efried: 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 it | 13:14 |
mriedem | mdbooth: because the live migration job shit the bed the other day when devstack was changed to use pike uca packages | 13:14 |
mriedem | it's in the ML | 13:14 |
mdbooth | mriedem: Ah, ok. | 13:14 |
*** claudiub has quit IRC | 13:14 | |
efried | stephenfin wfm. But how long are we planning to keep the original opts around? | 13:15 |
mriedem | jamespage pointed out the change in 2.10 | 13:15 |
*** kylek3h has joined #openstack-nova | 13:15 | |
*** andreas_s has quit IRC | 13:15 | |
*** claudiub has joined #openstack-nova | 13:15 | |
stephenfin | That's a question for the PTL. mriedem - do you think we should ever remove old deprecated opt aliases? | 13:15 |
stephenfin | mriedem: i.e. where we moved an opt from '[DEFAULT]' to, say, '[libvirt]' | 13:15 |
efried | Last night his only complaint was the proliferation of renos :) | 13:15 |
kashyap | mdbooth: It's this bug - https://bugs.launchpad.net/nova/+bug/1718295 | 13:15 |
openstack | Launchpad 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 | :D | 13:16 |
mdbooth | Interesting that UCA has newer libvirt/qemu than RHEL. | 13:16 |
stephenfin | efried: 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, heh | 13:16 |
*** lucasxu has joined #openstack-nova | 13:16 | |
mdbooth | Or Fedora, even | 13:17 |
mdbooth | That's pretty darned fresh | 13:17 |
sdague | mriedem: I reved the patch to pass unit tests | 13:17 |
kashyap | Yeah, UCA is supposed to be stable/long-term | 13:17 |
kashyap | Isn't it? | 13:17 |
efried | stephenfin 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 |
sdague | it turns out the broken hypervisor error pops a little earlier with this code path | 13:18 |
stephenfin | efried: gotcha | 13:18 |
sdague | kashyap: UCA is how they get latest bits to users. For the ubuntu package they unconditionally patched it | 13:18 |
stephenfin | efried: 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 fix | 13:18 |
sdague | because they know what version of qemu is required | 13:18 |
stephenfin | Though I do prefer this approach, given that we've never removed deprecated aliases before #justsaying | 13:19 |
efried | stephenfin 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 |
kashyap | sdague: Ah, latest bits; shipping QEMU 2.10 is a bit pre-mature with OpenStack, no? (But good for finding bugs, though.) | 13:19 |
stephenfin | efried: +1 | 13:19 |
efried | "never removed deprecated aliases before" ? | 13:19 |
efried | For what value of "never"? | 13:20 |
stephenfin | i.e. dropped 'deprecated_group' or 'deprecated_name' arguments | 13:20 |
*** kenperkins has joined #openstack-nova | 13:20 | |
sdague | kashyap: well someone has to ship things | 13:20 |
stephenfin | Like, ever, from what I can tell | 13:20 |
kashyap | Sure; question is not about shipping :-) There can be different repos for bleeding edge users. | 13:20 |
efried | I'm *sure* I've seen it done. But I'm not one to stand on principle and go find an example :) | 13:21 |
sdague | kashyap: 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 |
kashyap | Yep, agreed on that bit. | 13:21 |
kashyap | We still have to figure out that CI job we've been discussing on the list forever | 13:22 |
kashyap | Where latest libvirt / QEMU (from Git, newest upstream tarballs) are tested w/ Nova Git. | 13:22 |
mriedem | efried: stephenfin: deprecated options can go after a release | 13:23 |
jamespage | sdague, 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 |
mriedem | jamespage: ok | 13:23 |
jamespage | mriedem: sdague's approach seems a bit more rugged... | 13:24 |
mriedem | kashyap: you're the one that's always saying we should run against latest libvirt/qemu :) | 13:24 |
sdague | well, mriedem actually wrote the patch | 13:24 |
kashyap | mriedem: Yes, I say it to be done in a _separate_ CI job :-) | 13:24 |
mriedem | oooooooooooooooooooooooooooooooooohhhhhhhhhhhhhhhhhhhhhhhhhh | 13:24 |
jamespage | must have missed that.. | 13:24 |
stephenfin | mriedem: This is more about the deprecated aliases, where we rename an opt or change its group, but still keep it around | 13:24 |
jamespage | apols | 13:24 |
* mriedem is away for a bit | 13:25 | |
stephenfin | efried: Proof or it didn't happen ;) | 13:25 |
sdague | this only showed up in live migration though, honestly, CI isn't going to catch every edge casee | 13:25 |
*** gouthamr has joined #openstack-nova | 13:25 | |
mriedem | stephenfin: deprecated is deprecated what's the difference | 13:25 |
mriedem | this config option deprecation stuff is grating on my brain | 13:25 |
stephenfin | mriedem: So I _shouldn't_ push a twenty-patch "remove deprecated opts" series? Aww :'( | 13:26 |
* stephenfin kids...kind of | 13:26 | |
efried | The 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-nova | 13:29 | |
*** weshay is now known as weshay_bbiab | 13:30 | |
efried | stephenfin Oh, snap. Here's one. Guess who authored it? I41a6be48380999c49d6db3ee5221174a41014002 | 13:31 |
efried | reviewed, rather. | 13:32 |
efried | It was in the same blueprint as the one that moved the vnc opts originally (which was your change set IIRC) | 13:33 |
openstackgerrit | Merged openstack/os-vif master: Updated from global requirements https://review.openstack.org/502708 | 13:34 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements https://review.openstack.org/505839 | 13:34 |
efried | stephenfin And here's one you did co-author: https://review.openstack.org/#/c/463220/ | 13:35 |
efried | I'll stop now. | 13:35 |
*** jmlowe has quit IRC | 13:35 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/os-traits master: Updated from global requirements https://review.openstack.org/503646 | 13:36 |
mriedem | efried: stephenfin: i've been dealing with getting my kid to school, so haven't been following along, what is the actual issue? | 13:38 |
mriedem | i know about the vnc rename thing | 13:38 |
dansmith | gibi: responded to some of your comments, thanks for looking at this stuff | 13:39 |
openstackgerrit | Radoslav Gerganov proposed openstack/nova master: VMware: implement get_inventory() driver method https://review.openstack.org/506175 | 13:40 |
efried | mriedem 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-nova | 13:43 | |
efried | powervm 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-nova | 13:43 | |
mriedem | efried: +2 | 13:46 |
mriedem | alex_xu: sdague: can you take a look at https://review.openstack.org/#/c/505831/ ? unblocks some 3rd party CIs | 13:46 |
efried | Thanks! | 13:47 |
mriedem | efried: are you going to follow that up and drop support for the options in the DEFAULT group/ | 13:47 |
mriedem | which were deprecated | 13:47 |
*** bigdogstl has quit IRC | 13:47 | |
efried | Can | 13:47 |
*** ratailor has quit IRC | 13:48 | |
sdague | what's the no valid host error all about in CI? | 13:49 |
bauzas | oomichi: you're around ? see my comments in https://review.openstack.org/#/c/506092/1 ? | 13:49 |
openstackgerrit | Thomas Bechtold proposed openstack/nova stable/pike: Handle deleted instances when refreshing the info_cache https://review.openstack.org/506183 | 13:51 |
mriedem | fancy pants nova meeting in 8 minutes | 13:52 |
openstackgerrit | Sylvain Bauza proposed openstack/nova master: Fix definitely move single instance when created concurrently https://review.openstack.org/506093 | 13:52 |
bauzas | mriedem: you could have interest in ^ | 13:52 |
*** Sree has quit IRC | 13:52 | |
*** Sree has joined #openstack-nova | 13:53 | |
takashin | bauzas: Could you review https://review.openstack.org/#/c/408955/ ? | 13:53 |
bauzas | takashin: yeah, that's in my pipe for a long way :) | 13:53 |
takashin | bauzas: Thank you. | 13:54 |
bauzas | takashin: now we merged the spec, let's go back on the implementation side :) | 13:54 |
mriedem | bauzas: yeah, after the meeting | 13:54 |
*** lucasxu has quit IRC | 13:54 | |
bauzas | mriedem: np, just wanted to make sure you were aware | 13:54 |
*** slaweq has quit IRC | 13:55 | |
mriedem | dansmith: oh yeah, remind me to talk to you about filtering on IPs | 13:55 |
mriedem | with your series | 13:55 |
mriedem | given the discussion we had in denver about how we don't do that in the DB | 13:55 |
dansmith | mriedem: okay, does my series change anything about that? | 13:55 |
mriedem | idk :) | 13:56 |
dansmith | okay | 13:56 |
mriedem | haven't gone through all of it, i just know it's a special snowflake in the existing code | 13:56 |
openstackgerrit | Merged openstack/nova master: Remove 400 as expected error https://review.openstack.org/505485 | 13:56 |
mriedem | i think, however, we have a functional regression test that i added for filtering on ip | 13:56 |
mriedem | because i remember once last year accusing steve kaufer of regressing that | 13:57 |
dansmith | oh, filtering not sorting? | 13:57 |
mriedem | right | 13:57 |
mriedem | filter by ip regex | 13:57 |
dansmith | yeah, that shouldn't be any different | 13:57 |
*** Sree has quit IRC | 13:57 | |
dansmith | you can only sort on access_ip_vX, which are strings and fine | 13:57 |
openstackgerrit | sahid proposed openstack/nova master: libvirt: bandwidth param should be set in guest migrate https://review.openstack.org/497455 | 13:57 |
openstackgerrit | sahid proposed openstack/nova master: libvirt: add method to configure migration speed https://review.openstack.org/497456 | 13:57 |
openstackgerrit | sahid proposed openstack/nova master: libvirt: slow live-migration to ensure network is ready https://review.openstack.org/497457 | 13:57 |
dansmith | filters basically go straight through like they did before | 13:57 |
openstackgerrit | Merged openstack/nova master: pci: Validate behavior of empty devname https://review.openstack.org/396130 | 13:58 |
mriedem | straight through to the db api? | 13:58 |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/nova master: Add default configuration files to data_files https://review.openstack.org/506188 | 13:58 |
mriedem | if yes, that's my concern, since we don't filter ips in the db | 13:58 |
mriedem | but i'll look for this test before declaring that the sky has fallen | 13:59 |
dansmith | if they were passed to the db api before then they still are, and if they weren't then they're not | 13:59 |
*** yangyapeng has joined #openstack-nova | 13:59 | |
dansmith | mriedem: nova/tests/functional/test_list_servers_ip_filter.py | 14:00 |
mriedem | dansmith: that's the one | 14:00 |
mriedem | cool | 14:00 |
*** josecastroleon has quit IRC | 14:02 | |
sahid | mriedem: 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 |
sahid | you were asking for a spec | 14:03 |
*** lucasxu has joined #openstack-nova | 14:03 | |
mriedem | sahid: sure, i'll star it for later today | 14:04 |
sahid | mriedem: ok thanks | 14:05 |
*** gomarivera has joined #openstack-nova | 14:05 | |
openstackgerrit | Merged openstack/nova master: Adopt new pypowervm power_off APIs https://review.openstack.org/476945 | 14:07 |
*** liliars has quit IRC | 14:08 | |
*** gbarros has quit IRC | 14:11 | |
*** bigdogstl has joined #openstack-nova | 14:12 | |
*** crushil_ has joined #openstack-nova | 14:14 | |
*** crushil has joined #openstack-nova | 14:14 | |
*** gomarivera has quit IRC | 14:14 | |
*** armax has joined #openstack-nova | 14:14 | |
*** coreywright has quit IRC | 14:14 | |
*** gomarivera has joined #openstack-nova | 14:15 | |
*** crushil has quit IRC | 14:15 | |
*** crushil_ is now known as crushil | 14:15 | |
*** bigdogstl has quit IRC | 14:16 | |
*** Yikun has quit IRC | 14:16 | |
*** cfriesen has joined #openstack-nova | 14:17 | |
*** Swami has joined #openstack-nova | 14:19 | |
*** gomarivera has quit IRC | 14:19 | |
*** dtantsur is now known as dtantsur|brb | 14:20 | |
*** tssurya has joined #openstack-nova | 14:21 | |
dansmith | tssurya: this is the place :) | 14:22 |
*** itlinux has joined #openstack-nova | 14:23 | |
tssurya | dansmith: yep I will be here then :) thank you | 14:23 |
*** itlinux has quit IRC | 14:23 | |
*** jmlowe has joined #openstack-nova | 14:28 | |
*** coreywright has joined #openstack-nova | 14:28 | |
*** lajoskatona has quit IRC | 14:30 | |
*** gbarros has joined #openstack-nova | 14:30 | |
*** openstackgerrit has quit IRC | 14:33 | |
*** cleong has joined #openstack-nova | 14:36 | |
gibi | dansmith: 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 |
gibi | dansmith: I even replaced the list() call with a generator expression in your first patch and that seems to still pass the tests | 14:36 |
*** felipemonteiro has joined #openstack-nova | 14:38 | |
*** felipemonteiro_ has joined #openstack-nova | 14:39 | |
dansmith | gibi: hmm, I initially was returning a generator from it and was hitting something in there that was trying to count the result | 14:39 |
dansmith | gibi: but I guess maybe I was generating an error in one of the threads and that threw me off | 14:40 |
dansmith | let me try to change it again locally now that things are all working and see | 14:40 |
gibi | the 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 result | 14:41 |
*** itlinux has joined #openstack-nova | 14:41 | |
dansmith | yeah | 14:42 |
dansmith | hmm, yeah, that seems to work | 14:42 |
dansmith | I 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 it | 14:43 |
*** felipemonteiro has quit IRC | 14:43 | |
dansmith | so... I dunno, but that was super early, so maybe something else was going on | 14:43 |
dansmith | I shall change it | 14:43 |
*** itlinux has quit IRC | 14:43 | |
*** hongbin has joined #openstack-nova | 14:43 | |
stephenfin | sdague: 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 work | 14:45 |
stephenfin | This is in relation to Gerrit and your 'dashboard query changes since upgrade' mail (which I've yet to receive :() | 14:46 |
johnthetubaguy | I think sdague refreshed the openstack dashboard url for the new gerrit version | 14:46 |
mriedem | dansmith: you won me over on the sort keys/dirs defaults thing in https://review.openstack.org/#/c/504983/ | 14:46 |
mriedem | i'd be cool with dropping that handling | 14:46 |
stephenfin | johnthetubaguy: Aye, but I don't think that change is in there. | 14:46 |
gibi | dansmith: cool | 14:46 |
johnthetubaguy | stephenfin: ah, fair enough | 14:46 |
*** jobewan has joined #openstack-nova | 14:46 | |
dansmith | mriedem: why not just leave what I have? | 14:47 |
dansmith | mriedem: you can't pass dirs without keys, and can't pass dirs that don't match keys in terms of length | 14:47 |
mriedem | you can, it's just kind of dead code as you said yesterday | 14:47 |
dansmith | yeah, but now I have tests for it | 14:47 |
mriedem | :/ | 14:47 |
mriedem | up to you, i'm +2 either way | 14:47 |
mriedem | are you going to redo this series to make do_query return a generator? | 14:48 |
dansmith | yeah, I'm doing it now | 14:48 |
dansmith | I wish I hadjust done it at the end because of the conflict it makes, | 14:48 |
dansmith | but I'm in the middle now anyway | 14:48 |
*** openstackgerrit has joined #openstack-nova | 14:49 | |
openstackgerrit | Dan Smith proposed openstack/nova master: Add base implementation for efficient cross-cell instance listing https://review.openstack.org/504983 | 14:49 |
openstackgerrit | Dan Smith proposed openstack/nova master: Make instance_list honor global query limit https://review.openstack.org/504984 | 14:49 |
openstackgerrit | Dan Smith proposed openstack/nova master: Add db.instance_get_by_sort_filters() https://review.openstack.org/504985 | 14:49 |
openstackgerrit | Dan Smith proposed openstack/nova master: Support pagination in instance_list https://review.openstack.org/504986 | 14:49 |
openstackgerrit | Dan Smith proposed openstack/nova master: Add fault-filling into instance_get_all_by_filters_sort() https://review.openstack.org/505391 | 14:49 |
openstackgerrit | Dan Smith proposed openstack/nova master: Add tests to validate instance_list handles faults correctly https://review.openstack.org/505392 | 14:49 |
openstackgerrit | Dan Smith proposed openstack/nova master: Add get_instance_objects_sorted() https://review.openstack.org/505417 | 14:49 |
openstackgerrit | Dan Smith proposed openstack/nova master: Copy some tests to a cellsv1 mixin https://review.openstack.org/505442 | 14:49 |
openstackgerrit | Dan Smith proposed openstack/nova master: Fix a pagination logic bug in test_bug_1689692 https://review.openstack.org/505661 | 14:49 |
openstackgerrit | Dan Smith proposed openstack/nova master: Use improved instance_list module in compute API https://review.openstack.org/505418 | 14:49 |
openstackgerrit | Dan Smith proposed openstack/nova master: Remove legacy fault-loading routines https://review.openstack.org/505456 | 14:49 |
dansmith | mriedem: gibi: ^ | 14:49 |
gibi | dansmith: looking... | 14:49 |
dansmith | gibi: I'm still trying to figure out why that weird raise is .. weird.. but I can stack whatever the fix is on top | 14:50 |
*** andreas_s has joined #openstack-nova | 14:52 | |
gibi | dansmith: OK. thanks for the explanation. As you guessed I haven't tried to move the raise in the except block myself | 14:53 |
gibi | dansmith: It seems you forget to update the code comments when you change the list to the generator expression in https://review.openstack.org/#/c/504983 | 14:55 |
mriedem | andreykurilin: hey i assume you like to do some scale and performance testing since you work on rally, right? | 14:55 |
dansmith | gibi: I updated the docstring... did I miss a reference? | 14:56 |
gibi | dansmith: https://review.openstack.org/#/c/504983/6/nova/compute/instance_list.py | 14:57 |
dansmith | what in the | 14:57 |
gibi | dansmith: L69 and L93 | 14:57 |
dansmith | craaap | 14:57 |
dansmith | I must have dumped it during a rebase | 14:58 |
dansmith | urgh | 14:58 |
dansmith | I _did_ update ;) | 14:58 |
gibi | I believe you :) | 14:58 |
openstackgerrit | Elod Illes proposed openstack/nova master: Add instance.interface_attach notification https://review.openstack.org/503089 | 14:58 |
dansmith | gibi: from my history: https://pastebin.com/u7nUYi38 | 14:58 |
dansmith | :P | 14:58 |
openstackgerrit | Eric Fried proposed openstack/nova-specs master: Spec: Use keystoneauth1 Adapter for endpoints https://review.openstack.org/500190 | 14:59 |
*** felipemonteiro_ has quit IRC | 14:59 | |
*** felipemonteiro_ has joined #openstack-nova | 14:59 | |
efried | mriedem edmondsw ^ | 14:59 |
openstackgerrit | Takashi NATSUME proposed openstack/nova master: Fix hyperlinks in document https://review.openstack.org/506101 | 14:59 |
edmondsw | efried ack | 14:59 |
openstackgerrit | Takashi NATSUME proposed openstack/nova master: Fix hyperlinks in document https://review.openstack.org/506101 | 15:01 |
openstackgerrit | Dan Smith proposed openstack/nova master: Add base implementation for efficient cross-cell instance listing https://review.openstack.org/504983 | 15:01 |
openstackgerrit | Dan Smith proposed openstack/nova master: Make instance_list honor global query limit https://review.openstack.org/504984 | 15:01 |
openstackgerrit | Dan Smith proposed openstack/nova master: Add db.instance_get_by_sort_filters() https://review.openstack.org/504985 | 15:01 |
openstackgerrit | Dan Smith proposed openstack/nova master: Support pagination in instance_list https://review.openstack.org/504986 | 15:01 |
openstackgerrit | Dan Smith proposed openstack/nova master: Add fault-filling into instance_get_all_by_filters_sort() https://review.openstack.org/505391 | 15:01 |
openstackgerrit | Dan Smith proposed openstack/nova master: Add tests to validate instance_list handles faults correctly https://review.openstack.org/505392 | 15:01 |
openstackgerrit | Dan Smith proposed openstack/nova master: Add get_instance_objects_sorted() https://review.openstack.org/505417 | 15:01 |
openstackgerrit | Dan Smith proposed openstack/nova master: Copy some tests to a cellsv1 mixin https://review.openstack.org/505442 | 15:01 |
openstackgerrit | Dan Smith proposed openstack/nova master: Fix a pagination logic bug in test_bug_1689692 https://review.openstack.org/505661 | 15:01 |
openstackgerrit | Dan Smith proposed openstack/nova master: Use improved instance_list module in compute API https://review.openstack.org/505418 | 15:01 |
openstackgerrit | Dan Smith proposed openstack/nova master: Remove legacy fault-loading routines https://review.openstack.org/505456 | 15:01 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: api-ref: fix default sort key when listing servers https://review.openstack.org/506227 | 15:02 |
andreykurilin | mriedem: 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 IRC | 15:02 | |
gibi | dansmith: I don't need proofs I have faith :) | 15:03 |
dansmith | gibi: I need proof for my own sanity :) | 15:03 |
gibi | :) | 15:03 |
*** takashin has left #openstack-nova | 15:04 | |
gibi | yeah, sanity is important | 15:04 |
*** yasufum has joined #openstack-nova | 15:05 | |
mriedem | andreykurilin: 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 |
mriedem | especially since godaddy is using cells v1 and presumably has to investigate migrating to cells v2 at some point | 15:06 |
mriedem | andreykurilin: more specifically, dansmith is working on a series of changes to make listing instances across cells more efficient, | 15:06 |
mriedem | it would be nice if we could get benchmarks before/after | 15:06 |
*** links has quit IRC | 15:07 | |
mriedem | i'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 computes | 15:07 |
mriedem | i could do this a bit sloppy with devstack and using the fake virt driver, but that doesn't give me multiple cells | 15:08 |
andreykurilin | mriedem: 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 research | 15:10 |
*** felipemonteiro has joined #openstack-nova | 15:11 | |
*** ociuhandu has quit IRC | 15:12 | |
*** bigdogstl has joined #openstack-nova | 15:12 | |
*** felipemonteiro_ has quit IRC | 15:13 | |
*** yasufum has quit IRC | 15:14 | |
mriedem | bauzas: a few comments inline https://review.openstack.org/#/c/506092/ | 15:15 |
mriedem | andreykurilin: great, thanks | 15:16 |
mriedem | we've been trying to get feedback from large cells v1 users for awhile now and so far we don't get much feedback | 15:16 |
*** coreycb has joined #openstack-nova | 15:17 | |
*** bigdogstl has quit IRC | 15:17 | |
*** Tom_ has joined #openstack-nova | 15:18 | |
efried | sdague 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-nova | 15:18 | |
*** tssurya has joined #openstack-nova | 15:18 | |
*** gbarros has quit IRC | 15:19 | |
bauzas | mriedem: k, cool, reading | 15:22 |
*** gomarivera has joined #openstack-nova | 15:23 | |
*** felipemonteiro_ has joined #openstack-nova | 15:24 | |
bauzas | mriedem: just a question for just one comment, I'm not sure why we need to disable the source host | 15:24 |
bauzas | mriedem: because you'd like to make sure you end up in some specific host? | 15:25 |
*** gbarros has joined #openstack-nova | 15:25 | |
*** bigdogstl has joined #openstack-nova | 15:25 | |
*** gbarros has quit IRC | 15:25 | |
*** andreas_s has quit IRC | 15:25 | |
*** gbarros has joined #openstack-nova | 15:25 | |
*** gbarros has quit IRC | 15:25 | |
*** Oku_OS is now known as Oku_OS-away | 15:26 | |
*** tssurya has quit IRC | 15:27 | |
*** felipemonteiro has quit IRC | 15:27 | |
*** burt has joined #openstack-nova | 15:30 | |
dansmith | edleafe: I dunno why, but gerrit is being confusing about the stack of actual code patches for your selection/alternates bit | 15:30 |
*** bigdogstl has quit IRC | 15:30 | |
dansmith | edleafe: is this the bottom? https://review.openstack.org/#/c/486215/7 | 15:30 |
*** felipemonteiro_ has quit IRC | 15:32 | |
*** felipemonteiro_ has joined #openstack-nova | 15:33 | |
mriedem | bauzas: that's what was reported in the bug for the recreate, | 15:33 |
mriedem | but i guess that's not really necessary to reproduce the issue | 15:33 |
mriedem | because the scheduler will compare 1 host to 2 instances and fail | 15:34 |
mriedem | bauzas: yeah so nevermind that part | 15:34 |
bauzas | okay | 15:34 |
*** felipemonteiro has joined #openstack-nova | 15:34 | |
*** chyka has joined #openstack-nova | 15:34 | |
*** bigdogstl has joined #openstack-nova | 15:34 | |
edleafe | dansmith: yep | 15:35 |
edleafe | dansmith: it seems to be adding specs and code together | 15:35 |
dansmith | edleafe: well, it is also jumping around with the related-changes view of the code | 15:37 |
*** gyee has joined #openstack-nova | 15:37 | |
*** felipemonteiro_ has quit IRC | 15:37 | |
dansmith | I guess it's because the add-allocations patch has been orphaned for a while | 15:37 |
*** bigdogstl has quit IRC | 15:39 | |
mriedem | sdague: 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 |
mriedem | edleafe: 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 |
mriedem | or should those be merged? | 15:40 |
edleafe | mriedem: alternate hosts was a concept we agreed to in Pike but never wrote up in a spec | 15:41 |
edleafe | mriedem: the Selection object is independent | 15:41 |
*** slaweq has joined #openstack-nova | 15:41 | |
dansmith | edleafe: 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 |
sdague | mriedem: will look | 15:44 |
sdague | mriedem: I'm trying to sort out a final set of unit tests fails on the QEMU_IMG version thing | 15:44 |
edleafe | dansmith: I think so. That host state stuff is still black magic to me | 15:45 |
dansmith | edleafe: oh actually it's host.updated=None.. how. strange. | 15:45 |
dansmith | edleafe: okay I will comment based on that assumption and hedge a bit | 15:45 |
*** drwahl has joined #openstack-nova | 15:46 | |
drwahl | does placement-api (in newton) require keystone v3? | 15:47 |
bauzas | dansmith: that's because updated is the flag for saying "please refresh my state" | 15:47 |
*** cdent has joined #openstack-nova | 15:47 | |
dansmith | bauzas: okay I'm confused about why we need to do that in this context then | 15:48 |
bauzas | dansmith: what's the exact concern you have in mind ? | 15:48 |
dansmith | oohh, | 15:48 |
bauzas | I just jumped on your above question | 15:48 |
dansmith | maybe this is "I'm going to consume this host, so make sure to refresh it soon" ? | 15:49 |
dansmith | https://review.openstack.org/#/c/486215/7/nova/scheduler/filter_scheduler.py | 15:49 |
dansmith | L92 | 15:49 |
*** felipemonteiro has quit IRC | 15:49 | |
*** rods has joined #openstack-nova | 15:49 | |
*** tidwellr has joined #openstack-nova | 15:49 | |
*** felipemonteiro_ has joined #openstack-nova | 15:49 | |
bauzas | dansmith: no | 15:50 |
bauzas | dansmith: that's because we consume_from_request() before | 15:50 |
bauzas | dansmith: so the in-memory HostState is modified | 15:50 |
mriedem | stvnoyes: 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_125937 | 15:50 |
bauzas | dansmith: in case the calculations were wrong, we need to unset those modifications and force a fresh new DB call | 15:51 |
bauzas | we generally keep in memory the HostStates once per request | 15:51 |
dansmith | bauzas: meaning if we fail to do the consume, we want to trigger a refresh? | 15:51 |
bauzas | but for a multi-schedule call, then we could iterate wrongly | 15:51 |
openstackgerrit | Merged openstack/os-vif master: Rehome OVO unit tests to tests.unit.test_object.py https://review.openstack.org/489922 | 15:51 |
bauzas | dansmith: yup that sort of | 15:51 |
dansmith | hrm | 15:51 |
dansmith | not sure I fully get it, but it's not the thing I thought, so that's fine | 15:52 |
*** Tom_ has quit IRC | 15:52 | |
mriedem | stvnoyes: looks like it fails here https://review.openstack.org/#/c/487884/7/tempest/api/compute/admin/test_live_migration.py@185 | 15:52 |
dansmith | edleafe: so, what follows is some rambling for improvements we can make later, but listen and nod (or not): | 15:52 |
*** bigdogstl has joined #openstack-nova | 15:52 | |
bauzas | dansmith: see HostManager.consume_from_request() and you'll understand | 15:52 |
*** felipemonteiro has joined #openstack-nova | 15:53 | |
dansmith | edleafe: 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 |
dansmith | edleafe: 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_retries | 15:54 |
dansmith | right? | 15:54 |
dansmith | so, hmm, yeah if that's right, that seems like a problem | 15:54 |
edleafe | dansmith: yes. Selecting an alternate doesn't consume anything | 15:54 |
*** sambetts is now known as sambetts|afk | 15:54 | |
dansmith | what we probably need to do is go through and pick primaries, and then pick alternates for each after that | 15:54 |
dansmith | the alternates can overlap, but they shouldn't overlap with any primaries | 15:55 |
edleafe | dansmith: ok, that could be done | 15:55 |
dansmith | edleafe: 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 |
dansmith | edleafe: I shall add comments about this | 15:56 |
edleafe | I think in an earlier version of this we didn't think this would be a significant issue | 15:56 |
*** felipemonteiro_ has quit IRC | 15:56 | |
dansmith | yeah, I think we discussed it before, indeed | 15:56 |
dansmith | I think i was focusing on the overlapping of the alternates, | 15:56 |
edleafe | since a) alternates should rarely be used and b) hosts might still fit an additional instance | 15:56 |
dansmith | and not considering the overlapping of primaries | 15:56 |
dansmith | yeah | 15:57 |
stvnoyes | mriedem: ok, Ill take a look | 15:57 |
*** bigdogstl has quit IRC | 15:58 | |
gibi | dansmith: 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@85 | 15:58 |
dansmith | gibi: ahh, that makes more sense.. I forgot we were using the figure's target_cell here | 15:59 |
dansmith | gibi: I wrote that fixture, so I'll fix that up and make this change at the end of this series | 15:59 |
*** sbezverk has quit IRC | 15:59 | |
dansmith | gibi: now, please, go find bugs in someone else's code :) | 16:00 |
efried | sdague 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 IRC | 16:00 | |
efried | cdent ^ you may also have an opinion | 16:00 |
openstackgerrit | Merged openstack/nova-specs master: Spec: Use keystoneauth1 Adapter for endpoints https://review.openstack.org/500190 | 16:01 |
*** sahid has quit IRC | 16:01 | |
sdague | efried: it depends on what the subcall is doing | 16:01 |
*** sbezverk has joined #openstack-nova | 16:01 | |
sdague | if it is acting on behalf of the user, it should use the context auth | 16:01 |
gibi | dansmith: OK, I will review that follow up too. But instead of looking at others code I will just stop looking at any code for today | 16:02 |
sdague | optionally wrapped in a service token so that it doesn't expire | 16:02 |
dansmith | gibi: as long as it's not finding bugs in _my_ code I'm fine with whatever :P | 16:02 |
*** ralonsoh has quit IRC | 16:02 | |
*** ralonsoh has joined #openstack-nova | 16:02 | |
*** ralonsoh has quit IRC | 16:02 | |
gibi | dansmith: :) | 16:03 |
efried | sdague I guess there's also c) Don't use the context auth; require the conf auth. | 16:03 |
sdague | efried: yeh | 16:03 |
efried | sdague I hear you saying it's gonna be case by case. | 16:03 |
dansmith | gibi: I'll add you to the review of the fixture cleanup when I have it | 16:03 |
sdague | it's going to be case by case | 16:03 |
efried | sdague I'll likely need some help identifying which is which. | 16:03 |
sdague | efried: so, I think the answer ends up being roughly | 16:04 |
sdague | glance, cinder, barbican, keystone always act as user | 16:04 |
sdague | ironic always from conf, because nova is the ironic multi tenancy solution | 16:04 |
sdague | and neutron, it depends on the operation | 16:04 |
*** slaweq has quit IRC | 16:05 | |
efried | sdague 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 |
sdague | efried: for glance, I don't think so | 16:05 |
sdague | I can't think of a case where we should "sudo" on image things | 16:06 |
sdague | or that image things have to happen outside of a user context | 16:06 |
*** slaweq has joined #openstack-nova | 16:06 | |
*** yamahata has joined #openstack-nova | 16:07 | |
sdague | we 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 escalation | 16:07 |
sdague | ironic is a special case, ironic doesn't have regular users | 16:07 |
dansmith | edleafe: 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 weshay | 16:07 | |
mordred | efried, sdague: "optionally wrapped in a service token" doesn't need nova-specific creds does it? | 16:08 |
edleafe | dansmith: on a call and an IRC meeting, so it'll be later | 16:08 |
dansmith | edleafe: sure | 16:08 |
efried | mordred It's just service_auth.get_auth_plugin(context) | 16:08 |
sdague | mordred: i foreget exactly what it needs on disk | 16:09 |
*** suresh12 has joined #openstack-nova | 16:09 | |
sdague | mordred: it does need some creds | 16:09 |
efried | oh | 16:09 |
efried | mordred sdague https://github.com/openstack/nova/blob/master/nova/service_auth.py | 16:10 |
efried | So it loads up the auth from the [service_user] group. | 16:10 |
efried | and we get two auths in the thing. | 16:10 |
efried | yeesh. | 16:10 |
*** suresh12 has quit IRC | 16:11 | |
mordred | so - you need at least some of the ksa settings for each service no matter what | 16:11 |
efried | Only if CONF.service_user.send_service_user_token is set | 16:11 |
mordred | like 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 IRC | 16:12 | |
efried | mordred 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-nova | 16:14 | |
*** edand has joined #openstack-nova | 16:15 | |
mordred | nod. 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 used | 16:16 |
mordred | but I defer to smarter nova humans | 16:17 |
sdague | mordred: ++ lets keep things trimmed down | 16:17 |
sdague | efried: it's probably going to be worth writing a doc section on configuring nova with other services as well | 16:18 |
sdague | because I agree there is confusion here | 16:18 |
*** Apoorva has joined #openstack-nova | 16:18 | |
efried | sdague 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 IRC | 16:20 | |
*** edand has quit IRC | 16:20 | |
sdague | efried: I'm ok if it's just admin docs as part of the work | 16:20 |
efried | rgr | 16:20 |
sdague | mordred: on https://review.openstack.org/#/c/488137/17/nova/utils.py@1309 | 16:23 |
sdague | is the service types library not encoding this in python? | 16:23 |
sdague | like, I'd kind of expect that to be happening all behind the scenes | 16:23 |
efried | sdague Without a session param, it's pretty lightweight: https://github.com/openstack/os-service-types/blob/master/os_service_types/service_types.py#L51 | 16:25 |
mordred | sdague: 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 open | 16:25 |
efried | Almost all of the work is done at import time, actually. | 16:25 |
efried | https://github.com/openstack/os-service-types/blob/master/os_service_types/service_types.py#L22 | 16:25 |
mordred | efried: 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 about | 16:26 |
efried | The race would be harmless. I can take the lock out. | 16:26 |
mordred | so we can get live-update landed as a follow on | 16:26 |
sdague | um... https://github.com/openstack/os-service-types/blob/master/os_service_types/service_types.py#L59 ? closet network call? | 16:26 |
edmondsw | efried sdague mordred to be clear, long-term I think we'll need auth conf options for everything. But we don't today | 16:26 |
sdague | that seems like something people should have to opt into | 16:26 |
mordred | sdague: yah - it's there - but the keystoneauth consumption of the library at least at first will not pass a session | 16:26 |
mordred | sdague: yup | 16:27 |
efried | sdague The opt-in is by passing a `session` param. | 16:27 |
mordred | sdague: 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 dtrt | 16:27 |
mordred | we'll make sure subsequent turning on of remote access/ network calls is appropriately opt-in when we add it | 16:28 |
sdague | I'm actually not super clear why the remote part is there | 16:28 |
efried | to get a fresh copy of the service-types-authority data | 16:28 |
efried | os-service-types ships with a cached copy | 16:29 |
openstackgerrit | Merged openstack/os-vif master: Update reno for stable/pike https://review.openstack.org/488671 | 16:29 |
sdague | I thought the point of this was a no requirements version which we rev every time there is a service types update | 16:29 |
sdague | so people just replace it with the new one | 16:29 |
efried | Yup. Unless they don't. | 16:29 |
sdague | if they don't, then they don't | 16:29 |
openstackgerrit | Merged openstack/os-traits master: Update reno for stable/pike https://review.openstack.org/488669 | 16:30 |
sdague | closet network calls that could end up with very interesting results or random network hangs | 16:30 |
sdague | don't seem useful | 16:30 |
*** tbachman has quit IRC | 16:30 | |
sdague | like https://github.com/openstack/os-service-types/blob/master/os_service_types/service_types.py#L59 under some circumstances can hang forever | 16:30 |
mordred | sdague: yah - I don't actually think it's useful for nova to opt-in to the fetch behavior | 16:31 |
efried | Hence the comment in the nova change | 16:31 |
sdague | mordred: sure, but I'm not super clear where it's a good idea to put a potentially arbitrary delay into anyone's code path | 16:32 |
sdague | I get that it's clever, but the potential failure domains get huge | 16:32 |
openstackgerrit | Eric Fried proposed openstack/nova master: nova.utils.get_ksa_adapter() https://review.openstack.org/488137 | 16:32 |
efried | sdague mordred ^ updated to remove lock | 16:33 |
mordred | sdague: totally. this is why it will always be opt-in | 16:33 |
openstackgerrit | Merged openstack/nova master: Restore '[vnc] vnc_*' option support https://review.openstack.org/505831 | 16:33 |
*** bigdogstl has joined #openstack-nova | 16:33 | |
sdague | mordred: ok. I'm still not sure why it's useful, but I've said my piece :) | 16:33 |
mordred | efried: yah- I think honestly you could just initialize the _SERVICE_TYPES at the top and have it done at import | 16:34 |
sdague | having 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 it | 16:34 |
efried | mordred Considering that ost loads the local file at import time (which I didn't realize) I think you're right. | 16:34 |
openstackgerrit | Eric Fried proposed openstack/nova master: nova.utils.get_ksa_adapter() https://review.openstack.org/488137 | 16:36 |
efried | mordred Did that ^ | 16:36 |
*** zhenq has joined #openstack-nova | 16:36 | |
*** zhenq has quit IRC | 16:36 | |
*** drwahl has left #openstack-nova | 16:37 | |
sdague | efried: +2 | 16:37 |
efried | thanks! | 16:38 |
openstackgerrit | Sean Dague proposed openstack/nova master: Support qemu >= 2.10 https://review.openstack.org/505673 | 16:38 |
mriedem | efried: 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/Q8JdpjM0PZhAzsv32 | 16:39 |
*** slaweq has joined #openstack-nova | 16:39 | |
sdague | cburgess: https://review.openstack.org/#/c/505673 - live snapshot by default | 16:39 |
*** cdent has quit IRC | 16:41 | |
dansmith | mriedem: lol | 16:41 |
dansmith | how ... analog of you | 16:41 |
*** shaohe_feng has quit IRC | 16:41 | |
mriedem | dansmith: i think i'm just going to slap that into our official docs | 16:42 |
mriedem | "Technical reference deep dive > live migration > THIS" | 16:42 |
dansmith | heh | 16:42 |
*** bigdogstl has quit IRC | 16:42 | |
openstackgerrit | Elod Illes proposed openstack/nova master: Add instance.interface_detach notification https://review.openstack.org/506284 | 16:43 |
*** shaohe_feng has joined #openstack-nova | 16:43 | |
*** suresh12 has joined #openstack-nova | 16:43 | |
mriedem | if only i were on the twitters | 16:43 |
sdague | mriedem: that is a solvable problem | 16:44 |
efried | mriedem What does "cast" mean? | 16:45 |
*** itlinux has joined #openstack-nova | 16:45 | |
efried | async invocation? | 16:45 |
*** dtantsur|brb is now known as dtantsur | 16:46 | |
mriedem | rpc cast | 16:46 |
mriedem | vs call | 16:46 |
mriedem | which is important to understand the hot potato between the computes during live migration | 16:46 |
mriedem | especially if you love rpc timeouts | 16:46 |
mriedem | because your instance has 20 ports and 20 volumes attached to it | 16:46 |
mriedem | and token timeouts | 16:47 |
dansmith | mmmm, rpc timeout | 16:47 |
*** artom has quit IRC | 16:48 | |
*** slaweq has quit IRC | 16:48 | |
sdague | mriedem: service tokens fix that | 16:49 |
*** slaweq has joined #openstack-nova | 16:49 | |
sdague | well, some of it | 16:49 |
*** tesseract has quit IRC | 16:50 | |
mriedem | yeah i know | 16:50 |
mriedem | wonder if anyone is using those yet | 16:50 |
*** gomarivera has quit IRC | 16:51 | |
*** yamahata has quit IRC | 16:52 | |
*** rcernin has quit IRC | 16:52 | |
*** Swami has quit IRC | 16:53 | |
*** dave-mcc_ has joined #openstack-nova | 16:55 | |
*** dave-mccowan has quit IRC | 16:58 | |
*** tbachman has joined #openstack-nova | 16:59 | |
*** baoli has quit IRC | 17:00 | |
*** slaweq has quit IRC | 17:00 | |
*** slaweq has joined #openstack-nova | 17:01 | |
*** mvk has quit IRC | 17:01 | |
*** baoli has joined #openstack-nova | 17:01 | |
*** tbachman_ has joined #openstack-nova | 17:02 | |
*** derekh has quit IRC | 17:02 | |
*** tbachman has quit IRC | 17:03 | |
*** tbachman_ is now known as tbachman | 17:03 | |
bauzas | dansmith: 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 |
bauzas | dansmith: 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 node | 17:05 |
*** jpena is now known as jpena|off | 17:05 | |
bauzas | dansmith: 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#L5735 | 17:06 |
bauzas | dansmith: mriedem: so since the destination RT doesn't know the source node, it fails with a KeyError | 17:07 |
bauzas | amirite? | 17:07 |
bauzas | unless post_live_mig runs on the source node | 17:07 |
bauzas | that's confusing | 17:07 |
mriedem | post live migrate runs on the source onde | 17:07 |
mriedem | *node | 17:07 |
mriedem | see my awesome call flow diagram above | 17:08 |
mriedem | gd this is already paying for itself | 17:08 |
mriedem | https://photos.app.goo.gl/Q8JdpjM0PZhAzsv32 | 17:08 |
bauzas | grmblbl | 17:08 |
openstackgerrit | Dan Smith proposed openstack/nova master: Fix CellDatabases fixture swallowing exceptions https://review.openstack.org/506312 | 17:08 |
bauzas | mriedem: any reason you would see why I'm getting that KeyError ? I guess it's because I don't run the periodics ? | 17:09 |
openstackgerrit | Merged openstack/nova master: Updated from global requirements https://review.openstack.org/505839 | 17:09 |
*** Swami has joined #openstack-nova | 17:09 | |
openstackgerrit | Merged openstack/nova master: Fix hyperlinks in document https://review.openstack.org/506101 | 17:10 |
*** smatzek has quit IRC | 17:11 | |
*** bigdogstl has joined #openstack-nova | 17:12 | |
mriedem | bauzas: that should get set in memory when the compute service starts up | 17:12 |
mriedem | when it calls pre_start_hook | 17:12 |
bauzas | mmm | 17:12 |
mriedem | that will call update_available_resource_for_node | 17:12 |
bauzas | looking at the func tests, jay needed to explicitly start them | 17:13 |
mriedem | start what? | 17:13 |
mriedem | the computes? | 17:13 |
mriedem | the fixture starts those | 17:13 |
jaypipes | hmm? | 17:13 |
bauzas | the RT update things | 17:13 |
bauzas | here, I'm getting a trace because self.compute_nodes[my_node] isn't set yet | 17:13 |
mriedem | when the compute service starts, it will get the available nodes from the driver, and use those to call rt.update_available_resource | 17:13 |
mriedem | passing in the node name which gets populated in the rt.compute_nodes dict | 17:13 |
bauzas | which is populated AFAIK by running the RT update call | 17:13 |
bauzas | mmm, I'm introspecting | 17:15 |
mriedem | what you have in the test setup looks fine to me | 17:15 |
mriedem | and it's what we're doing in other tests | 17:15 |
bauzas | I'll upload my latest rev and see what the job insults me | 17:16 |
*** bigdogstl has quit IRC | 17:17 | |
*** smatzek has joined #openstack-nova | 17:17 | |
edleafe | mriedem: IMO, we need lots more flow diagrams like that | 17:17 |
openstackgerrit | Sylvain Bauza proposed openstack/nova master: Add a regression test for bug 1718455 https://review.openstack.org/506092 | 17:18 |
openstack | bug 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 |
openstackgerrit | Sylvain Bauza proposed openstack/nova master: Fix definitely move single instance when created concurrently https://review.openstack.org/506093 | 17:18 |
*** gszasz has quit IRC | 17:18 | |
openstackgerrit | Merged openstack/nova master: Remove compatibility code for flavors https://review.openstack.org/460377 | 17:18 |
*** itlinux has quit IRC | 17:18 | |
openstackgerrit | Merged openstack/nova master: Test InstanceNotFound handling in 'nova usage' https://review.openstack.org/468514 | 17:19 |
*** felipemonteiro_ has joined #openstack-nova | 17:22 | |
*** felipemonteiro has quit IRC | 17:22 | |
*** artom has joined #openstack-nova | 17:25 | |
*** felipemonteiro has joined #openstack-nova | 17:25 | |
openstackgerrit | Merged openstack/nova master: VMware: Factor out relocate_vm() https://review.openstack.org/270115 | 17:26 |
mriedem | edleafe: agree, just needs to be put into docs | 17:26 |
openstackgerrit | Merged openstack/nova master: neutron: handle binding:profile=None during migration https://review.openstack.org/504260 | 17:26 |
*** felipemonteiro_ has quit IRC | 17:28 | |
*** kenperkins has joined #openstack-nova | 17:30 | |
*** vks1 has joined #openstack-nova | 17:31 | |
openstackgerrit | Matt Riedemann proposed openstack/nova stable/pike: neutron: handle binding:profile=None during migration https://review.openstack.org/506319 | 17:33 |
*** vks1 has quit IRC | 17:37 | |
*** erlon has quit IRC | 17:37 | |
openstackgerrit | Matt Riedemann proposed openstack/nova stable/ocata: neutron: handle binding:profile=None during migration https://review.openstack.org/506320 | 17:37 |
*** rods has left #openstack-nova | 17:38 | |
*** crushil has quit IRC | 17:38 | |
*** crushil has joined #openstack-nova | 17:38 | |
*** mvk has joined #openstack-nova | 17:41 | |
*** vvargaszte has joined #openstack-nova | 17:44 | |
*** vvargaszte has quit IRC | 17:46 | |
*** sridharg has quit IRC | 17:47 | |
openstackgerrit | Matt Riedemann proposed openstack/nova stable/newton: neutron: handle binding:profile=None during migration https://review.openstack.org/506323 | 17:49 |
*** elod has quit IRC | 17:50 | |
*** elod has joined #openstack-nova | 17:52 | |
*** ioggstream has quit IRC | 17:52 | |
efried | mriedem 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 |
mriedem | me | 17:55 |
mriedem | i guess | 17:55 |
mriedem | i'm doing about 6 things at once right now though, so it's going to have to wait | 17:55 |
*** ijw has joined #openstack-nova | 17:56 | |
dansmith | efried: and three of those six things are my patches, which are very important | 17:56 |
efried | mriedem 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 |
efried | dansmith No doubt. | 17:57 |
openstackgerrit | Merged openstack/nova master: Update docs to include standardization of VM diagnostics https://review.openstack.org/500408 | 17:57 |
*** Sukhdev has joined #openstack-nova | 17:57 | |
*** gomarivera has joined #openstack-nova | 17:58 | |
*** Sukhdev has quit IRC | 18:03 | |
sdague | mtreinish: 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 |
sdague | worker 7 just hung, and eventually that was a fail | 18:05 |
*** crushil has quit IRC | 18:06 | |
*** bigdogstl has joined #openstack-nova | 18:06 | |
*** esberglu has quit IRC | 18:07 | |
*** esberglu has joined #openstack-nova | 18:08 | |
*** dtantsur is now known as dtantsur|afk | 18:08 | |
mriedem | stvnoyes: the bdm attachment ids stuff in the migrate_data object is going to get weird when we have multiattach | 18:11 |
mriedem | since in your change it's a 1:1 between volume id and attachment id | 18:12 |
mriedem | although, | 18:12 |
*** esberglu has quit IRC | 18:12 | |
mriedem | i guess you wouldn't have a volume attached multiple times to the same instance, unless we're migrating it | 18:12 |
mriedem | so maybe i'm overthinking things | 18:12 |
*** bigdogstl has quit IRC | 18:14 | |
*** tonygunk has joined #openstack-nova | 18:15 | |
openstackgerrit | Ildiko Vancsa proposed openstack/nova-specs master: Add multiattach support to Nova https://review.openstack.org/499777 | 18:15 |
-openstackstatus- NOTICE: Gerrit OpenIDs have been accidentally overwritten and are in the process of being restored | 18:17 | |
*** esberglu has joined #openstack-nova | 18: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 shortly | 18: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 IRC | 18:25 | |
*** ijw has quit IRC | 18:25 | |
*** crushil has joined #openstack-nova | 18:28 | |
mriedem | and just when i was about to -1 this change | 18:29 |
*** ijw has joined #openstack-nova | 18:29 | |
*** yamahata has joined #openstack-nova | 18:36 | |
*** smatzek has quit IRC | 18:37 | |
mriedem | dansmith: so for your 2nd change in the series to honor the global limit, | 18:38 |
mriedem | i was posting this comment, but can't | 18: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-nova | 18:42 | |
dansmith | mriedem: we can't do that partitioning of the limit until/unless we support going back to the cells for more instances | 18:44 |
dansmith | mriedem: 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 sync | 18:44 |
*** armax has quit IRC | 18:44 | |
dansmith | mriedem: we basically have to have at least $limit results from each cell to prevent that | 18:44 |
*** armax has joined #openstack-nova | 18:44 | |
dansmith | or emulate it by refilling our batch when we run out before we continue the merge | 18:44 |
dansmith | that 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 problem | 18:45 |
dansmith | also, yes it may be more traffic at times, but something else I was discussing with people was: | 18:45 |
dansmith | if 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 pieces | 18:46 |
dansmith | the db driver doesn't even fetch partial results today or with this, it fetches everything in the result | 18:46 |
dansmith | does that make sense at all? | 18:47 |
mriedem | yeah, 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 800 | 18:50 |
*** ioggstream has joined #openstack-nova | 18:50 | |
*** suresh12 has quit IRC | 18:51 | |
dansmith | well, 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 cut | 18:51 |
dansmith | and if those were ever after max_limit/N cells, you'd never be able to limit them ever | 18:51 |
dansmith | er, never be able to list them | 18:51 |
*** ijw has quit IRC | 18:52 | |
*** gbarros has joined #openstack-nova | 18:52 | |
*** suresh12 has joined #openstack-nova | 18:54 | |
*** avolkov` has joined #openstack-nova | 18:55 | |
*** mriedem1 has joined #openstack-nova | 18:56 | |
*** avolkov has quit IRC | 18:56 | |
*** mriedem has quit IRC | 18:56 | |
*** gbarros has quit IRC | 18:57 | |
*** Apoorva_ has joined #openstack-nova | 19:01 | |
*** Apoorva has quit IRC | 19:04 | |
*** penick_ has joined #openstack-nova | 19:07 | |
*** mriedem1 is now known as mriedem | 19:07 | |
*** hieulq has quit IRC | 19:09 | |
*** bigdogstl has joined #openstack-nova | 19:11 | |
*** avolkov` has quit IRC | 19:12 | |
openstackgerrit | Eric Fried proposed openstack/nova master: Get auth from context for glance endpoint https://review.openstack.org/490057 | 19:15 |
*** awaugama has quit IRC | 19:16 | |
efried | sdague mordred ^ removed conf auth for glance | 19:16 |
*** bigdogstl has quit IRC | 19:16 | |
*** artom has quit IRC | 19:17 | |
efried | dangit, 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 IRC | 19:17 | |
openstackgerrit | Eric Fried proposed openstack/nova master: Get auth from context for glance endpoint https://review.openstack.org/490057 | 19:17 |
*** ijw has joined #openstack-nova | 19:17 | |
mordred | efried: lgtm | 19:19 |
efried | cool | 19:20 |
*** cdent has joined #openstack-nova | 19:20 | |
*** suresh12 has joined #openstack-nova | 19:20 | |
mriedem | stvnoyes: 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 today | 19:21 | |
stvnoyes | mriedem: thanks i'll take a look | 19:21 |
*** ioggstream has quit IRC | 19:22 | |
*** ijw has quit IRC | 19:22 | |
*** hieulq has joined #openstack-nova | 19:25 | |
*** suresh12 has quit IRC | 19:25 | |
openstackgerrit | Sean Dague proposed openstack/nova master: Support qemu >= 2.10 https://review.openstack.org/505673 | 19:25 |
*** xyang1 has quit IRC | 19:26 | |
*** suresh12 has joined #openstack-nova | 19:34 | |
*** jaypipes has quit IRC | 19:35 | |
*** xyang1 has joined #openstack-nova | 19:37 | |
openstackgerrit | Merged openstack/nova master: Transform servergroup.create notification https://review.openstack.org/454023 | 19:37 |
openstackgerrit | Merged openstack/nova master: Transform servergroup.delete notification https://review.openstack.org/483902 | 19:41 |
*** pcaruana has quit IRC | 19:41 | |
openstackgerrit | Merged openstack/nova master: Move libvirts dmcrypt support to privsep. https://review.openstack.org/490737 | 19:41 |
openstackgerrit | Merged openstack/nova master: api-ref: fix default sort key when listing servers https://review.openstack.org/506227 | 19:42 |
*** crushil has quit IRC | 19:43 | |
*** crushil has joined #openstack-nova | 19:44 | |
*** dave-mcc_ is now known as dave-mccowan | 19:45 | |
*** weshay is now known as weshay_bbiab | 19:50 | |
*** pchavva has quit IRC | 19:56 | |
*** thorst has quit IRC | 19:58 | |
*** jmlowe has quit IRC | 20:02 | |
*** thorst has joined #openstack-nova | 20:03 | |
*** mvk has quit IRC | 20:05 | |
*** ijw has joined #openstack-nova | 20:06 | |
mriedem | dansmith: 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 needed | 20:06 |
*** thorst has quit IRC | 20:07 | |
dansmith | heh | 20:07 |
dansmith | mriedem: that should be slicing self.instances not uuids | 20:07 |
dansmith | I'm in the middle of fixing migration uuid stuff in my tree so I can't I update it right now | 20:08 |
dansmith | but I will when I'm done | 20:08 |
mriedem | ok yeah that makes more sense :) | 20:08 |
dansmith | optionally you can locally test and then I'll slap something on the end, or you can push a fix | 20:08 |
mriedem | i had to rebase the entire thing, so we can slap it on the end | 20:09 |
*** crushil has quit IRC | 20:09 | |
dansmith | ack | 20:09 |
*** crushil has joined #openstack-nova | 20:09 | |
dansmith | I would be happy if we merge everything at least up to the pagination one this week | 20:10 |
mriedem | i'll try to push it to the limit | 20:10 |
dansmith | +1 | 20:10 |
*** ijw has quit IRC | 20:11 | |
mriedem | does that mean all these other tests doing "uuids = [inst['uuid'] for inst in insts]" should be using self.instances? | 20:13 |
*** jobewan has quit IRC | 20:13 | |
dansmith | no, | 20:15 |
dansmith | just that one was intending to compare the list to the snipped list of known instances and sliced the wrong list | 20:15 |
dansmith | I mean | 20:15 |
mriedem | alright left a diff in the comments | 20:15 |
dansmith | point me to something, I could have made the same mistake I guess, but.. | 20:15 |
dansmith | cool, in the limit one? | 20:15 |
mriedem | yeah | 20:16 |
*** artom has joined #openstack-nova | 20:16 | |
dansmith | cool thanks | 20:16 |
dansmith | also | 20:16 |
dansmith | our migration and rt stuff is hell on earth | 20:17 |
dansmith | that is all | 20:17 |
*** mvk has joined #openstack-nova | 20:17 | |
mriedem | i thought branson, mo was hell on earth? | 20:17 |
*** artom has quit IRC | 20:17 | |
dansmith | we have a new contender | 20:17 |
*** armax has quit IRC | 20:18 | |
*** artom has joined #openstack-nova | 20:18 | |
*** artom has quit IRC | 20:19 | |
*** artom has joined #openstack-nova | 20:19 | |
dansmith | mriedem: that one was checking that the list of uuids was the same as returned and as sorted | 20:20 |
*** artom has quit IRC | 20:20 | |
*** artom has joined #openstack-nova | 20:20 | |
*** artom has quit IRC | 20:21 | |
*** artom has joined #openstack-nova | 20:21 | |
*** artom has quit IRC | 20:22 | |
*** annegentle has joined #openstack-nova | 20:22 | |
*** artom has joined #openstack-nova | 20:22 | |
*** artom has quit IRC | 20:23 | |
*** artom has joined #openstack-nova | 20:23 | |
*** crushil has quit IRC | 20:26 | |
*** ijw has joined #openstack-nova | 20:27 | |
*** gouthamr has quit IRC | 20:28 | |
mriedem | in test_get_sorted ? | 20:30 |
mriedem | ah ok, nvm then | 20:30 |
*** tbachman has quit IRC | 20:30 | |
mriedem | oh yeah, for the migration uuid stuff, we still have a couple of outstanding bugs related to cleaning up allocations during a failed move | 20:31 |
dansmith | that you'll want to get in before this so you can backport them I guess? | 20:32 |
mriedem | yeah | 20:32 |
mriedem | been meaning to at least write the recreate tests | 20:32 |
mriedem | maybe i can do that at 10pm tonight after all others are asleep | 20:32 |
openstackgerrit | Chris Dent proposed openstack/nova master: Move project_id and user_id to Allocation object https://review.openstack.org/500410 | 20:32 |
openstackgerrit | Chris Dent proposed openstack/nova master: [placement] Allow _set_allocations to delete allocations https://review.openstack.org/501051 | 20:32 |
openstackgerrit | Chris Dent proposed openstack/nova master: WIP: [placement] POST /allocations to set allocations for >1 consumers https://review.openstack.org/500073 | 20:32 |
*** lyan has quit IRC | 20:33 | |
*** cleong has quit IRC | 20:33 | |
*** lyan has joined #openstack-nova | 20:38 | |
*** ijw has quit IRC | 20:38 | |
*** thorst has joined #openstack-nova | 20:40 | |
mriedem | shit 1 week for forum topics? | 20:42 |
*** gszasz has quit IRC | 20:42 | |
cdent | barely digested the ptg, too soon! | 20:42 |
mriedem | i threw up half of it | 20:43 |
mriedem | couldn't even digest it all | 20:43 |
openstackgerrit | Eric Berglund proposed openstack/nova master: Add PowerVM hypervisor configuration doc https://review.openstack.org/505665 | 20:43 |
*** annegentle has quit IRC | 20:43 | |
*** artom has quit IRC | 20:49 | |
*** edmondsw has quit IRC | 20:51 | |
*** artom has joined #openstack-nova | 20:51 | |
*** artom has quit IRC | 20:52 | |
*** artom has joined #openstack-nova | 20:52 | |
*** artom has quit IRC | 20:53 | |
*** artom has joined #openstack-nova | 20:53 | |
*** Apoorva_ has quit IRC | 20:55 | |
*** Apoorva has joined #openstack-nova | 20:55 | |
*** edmondsw has joined #openstack-nova | 20:57 | |
mriedem | ha http://forumtopics.openstack.org/ | 20:58 |
mriedem | so apparently this has been getting communicated all month, i've just missed it | 20:59 |
mriedem | not sure why i'd be distracted... | 20:59 |
mriedem | but so has everyone else | 20:59 |
cdent | huh | 20:59 |
cdent | first I started hearing about it was last week | 21:00 |
*** Apoorva_ has joined #openstack-nova | 21:01 | |
*** edmondsw has quit IRC | 21:01 | |
mriedem | god 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 queens | 21:02 |
*** hamzy has quit IRC | 21:03 | |
*** Apoorva has quit IRC | 21:04 | |
melwitt | I 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 ready | 21:04 |
*** lucasxu has quit IRC | 21:04 | |
mriedem | and then how to use placement to make your toast | 21:05 |
melwitt | I think that could have been accomplished with fewer hours than were spent, but yeah | 21:05 |
mriedem | with the "grape jelly" trait | 21:05 |
melwitt | heh | 21:05 |
dansmith | we had already been over a lot of that stuff before | 21:06 |
cdent | I 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 year | 21:06 |
dansmith | I was fairly annoyed, but I'm sure nobody noticed | 21:06 |
mriedem | the 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 groups | 21:06 |
*** catintheroof has quit IRC | 21:07 | |
mriedem | i have accepted that everytime we get together we're going to talk about stuff that we aren't going to do in that release | 21:07 |
mriedem | just 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 overwhelming | 21:08 |
cdent | it is indeed, but it is that overwhelmedness that makes people want to talk | 21:09 |
mriedem | i come from swedish heritage, | 21:09 |
mriedem | we don't want to talk about anything | 21:09 |
mriedem | except jellied fish carcass | 21:10 |
cdent | mmmmm jellied fish | 21:10 |
melwitt | and 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 us | 21:10 |
melwitt | so ... I dunno. I see both sides | 21:11 |
cdent | I wanted creamed herring, not pickled! | 21:11 |
mriedem | i 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 hitting | 21:11 |
mriedem | assuming those talks aren't just pushing some product message | 21:12 |
melwitt | I went to one ops midcycle when it was near where I live, and it was like that | 21:12 |
cdent | are those ops concerned with all this nfv, enhanced platform hoopa joop? | 21:12 |
mriedem | that's what sdague said the ops meetup in philly was like too | 21:12 |
mriedem | cdent: i don't think so | 21:12 |
melwitt | no, they were mostly focused on running clusters, upgrading, deploying, that kind of stuff | 21:13 |
mriedem | dansmith: L16 is for you https://etherpad.openstack.org/p/SYD-nova-brainstorming | 21:13 |
mriedem | mikal: ^ and you | 21:13 |
*** bigdogstl has joined #openstack-nova | 21:13 | |
dansmith | mriedem: oh good. | 21:13 |
cdent | yeah, that’s what I would assume too, so the force of all this epa stuff is depressing | 21:14 |
melwitt | yeah | 21:15 |
*** ijw has joined #openstack-nova | 21:15 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/os-traits master: Updated from global requirements https://review.openstack.org/503646 | 21:15 |
*** felipemonteiro has quit IRC | 21:16 | |
*** slaweq has quit IRC | 21:17 | |
*** bigdogstl has quit IRC | 21:18 | |
*** felipemonteiro_ has joined #openstack-nova | 21:18 | |
mriedem | btw, i thought the ptg was very productive overall | 21:18 |
mriedem | even though maybe i don't express that | 21:18 |
cdent | we see it in your eyes | 21:19 |
*** felipemonteiro__ has joined #openstack-nova | 21:19 | |
*** ijw has quit IRC | 21:19 | |
cdent | I 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 providers | 21:20 |
mriedem | i do take solace in the fact that the other etherpads on the wiki are pretty much empty | 21:20 |
cdent | I’d much rather see us focus on listening, as you said, to users/ops | 21:20 |
melwitt | I agree all sessions should be ops-focused and more about listening | 21:21 |
mriedem | like, i want to go here https://etherpad.openstack.org/p/SYD-ops-session-ideas | 21:21 |
*** felipemonteiro_ has quit IRC | 21:23 | |
*** baoli has quit IRC | 21:23 | |
dansmith | forum 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 |
dansmith | because they shifted the schedule | 21:25 |
dansmith | so I certainly don't want to avoid talking about bits of placement (or anything else) material to the act of getting important things landed | 21:26 |
dansmith | and of course, the listening to ops stuff will likely necessarily end up talking about future things anyway, if it's feature gaps, etc | 21:26 |
cdent | dansmith: that’s not how I remember it? I thought it was “shortly after the latest release to get feedback from people”. | 21:26 |
dansmith | cdent: very few people install the release shortly after the release | 21:27 |
cdent | I know, but that was the pitch | 21:27 |
mriedem | http://lists.openstack.org/pipermail/openstack-dev/2017-September/121783.html | 21: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 |
dansmith | there was definitely a "people who need a midcycle could do it during the forum" argument | 21:27 |
cdent | “on the priorities and 'blue sky' ideas for the next release.” is both terrifying and exciting | 21:28 |
*** armax has joined #openstack-nova | 21:28 | |
mriedem | we 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 |
mriedem | i don't know what we have like that for queens | 21:28 |
*** Sukhdev has joined #openstack-nova | 21:28 | |
cdent | huh, I thought ptg was the replacement for the midcycle, not forum for the midcycle | 21:28 |
cdent | mriedem: is that because the dance card is full? | 21:28 |
dansmith | the format of the midcycle was moved to ptg | 21:28 |
dansmith | instead of design summit style | 21:29 |
mriedem | and the ptg replaced the dev summit at the start of the release | 21:29 |
dansmith | but they shifted the schedle | 21:29 |
dansmith | right | 21:29 |
mriedem | so now it's semi scheduled blocks of things in a midcycle setting with no ops or users | 21:29 |
mriedem | well, some ops | 21:29 |
mriedem | cburgess and med_ <3 | 21:30 |
edleafe | cdent: PTG is supposed to be planning for the upcoming cycle. Forum is supposed to be more like a midcycle | 21:30 |
mriedem | except midcycles of yore weren't time boxed | 21:30 |
mriedem | and didn't cost $3000 to go to | 21:30 |
dansmith | more like a midcycle, meaning scheduling | 21:30 |
cdent | edleafe: that’s not at all my understanding, and nor does it correspond to the email that mriedem just linked to | 21:31 |
dansmith | cdent: certainly you've noticed that the early messaging, the late messaging, and the actual intent for changing things are totally disjoint :) | 21:31 |
mriedem | there 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 here | 21:31 |
cdent | dansmith: yes, very true, but I _never_ thought the forum was a midcycle | 21:31 |
dansmith | right, 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 late | 21:32 |
mriedem | dansmith: we can, if you can figure out what those things are going to be in the next 8 days :) | 21:32 |
edleafe | cdent: 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 PTG | 21:32 |
edleafe | dansmith: sorta jinx | 21:32 |
cdent | if that’s an expectation people have, I think they will be disappointed | 21:32 |
dansmith | edleafe: exactly | 21:32 |
mriedem | dansmith: and minus a lot of the developers involved | 21:33 |
cdent | a) there won’t be time, b) there won’t be the people | 21:33 |
dansmith | mriedem: certainly for this one | 21:33 |
cdent | b jinx with mriedem | 21:33 |
cdent | we should treat it as feedback time | 21:33 |
mriedem | ^ is why i'd rather just sit in on ops sessions | 21:33 |
edleafe | many developers couldn't get companies to pay for the midcycles, since it wasn't an "official" event | 21:33 |
cdent | mriedem: maybe just make that so then? state: we’re not gonna do nova-oriented forum sessions, we’re going to go to ops ones | 21:34 |
mriedem | cdent: that will be the case if we don't have anything to propose | 21:34 |
mriedem | which might just happen naturally | 21:34 |
*** Apoorva_ has quit IRC | 21:34 | |
cdent | I suspect most people whould approve of that, especially if you spread the idea around | 21:34 |
mriedem | ultimately i just don't want to have to propose topics to fill time | 21:35 |
*** Apoorva has joined #openstack-nova | 21:35 | |
mriedem | s/i/us/we/everyone/ | 21:35 |
dansmith | the topics we proposed in boston were pretty well-attended by ops no? | 21:35 |
mriedem | yeah | 21:36 |
mriedem | some of the stuff we talked about in boston is just carrying over into queens | 21:37 |
mriedem | so that's where i'm thinking, this isn't news, this is stuff we're still working on | 21:37 |
openstackgerrit | Eric Berglund proposed openstack/nova master: Add PowerVM hypervisor configuration doc https://review.openstack.org/505665 | 21:37 |
dansmith | I 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 things | 21:37 |
mriedem | i don't know what specifically we're trying to get feedback on | 21:38 |
mriedem | and we've done the general ops feedback sessions in the past which weren't well attended | 21:38 |
*** tbachman has joined #openstack-nova | 21:39 | |
mriedem | getting feedback from pike would be great, hell even ocata | 21:39 |
dansmith | isn'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 |
mriedem | so are you thinking like a pike feedback session? | 21:40 |
*** artom has quit IRC | 21:40 | |
*** artom has joined #openstack-nova | 21:41 | |
dansmith | I think that by the time that rolls around several of the people we know of as early movers will be on pike already | 21:41 |
dansmith | cern, vexxhost already being there, etc | 21:41 |
mriedem | L10 in https://etherpad.openstack.org/p/SYD-ops-session-ideas | 21:42 |
mriedem | we do have the project update talk for pike | 21:42 |
dansmith | surely someone has an opinion on the quotas changes by that point | 21:42 |
mriedem | i'm totally fine with a pike feedback / update session if people are armed with issues/questions | 21:42 |
melwitt | I 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 easier | 21:42 |
mriedem | ok, so, | 21:43 |
mriedem | you all know you can throw stuff in this etherpad right? | 21:43 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/os-traits master: Updated from global requirements https://review.openstack.org/503646 | 21:43 |
melwitt | I 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 gotchas | 21:43 |
melwitt | yes, sorry | 21:43 |
* melwitt goes to etherpad | 21:43 | |
dansmith | mriedem: sure, but ya'll were arguing about not having anything here | 21:43 |
*** tbachman has quit IRC | 21:44 | |
mriedem | i'm basically just trolling myself at this point | 21:44 |
melwitt | in my case, my brain is totally fried from the PTG and the ideas are emerging slowly | 21:44 |
mriedem | right | 21:44 |
mriedem | same here | 21:44 |
dansmith | mriedem: do you want my alternate color up on L20 or to delete your dupes on L37? | 21:45 |
mriedem | i would love to have ops people come armed with issues/questions for a pike feedback session | 21:45 |
dansmith | er, 38-39 | 21:45 |
*** zhenguo has quit IRC | 21:45 | |
*** bauzas has quit IRC | 21:45 | |
*** redondo-mk has quit IRC | 21:45 | |
mriedem | a 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 someone | 21:45 |
*** adisky has quit IRC | 21:46 | |
dansmith | same goes for us just waiting for other people to propose topics | 21:46 |
*** thorst has quit IRC | 21:46 | |
edleafe | mriedem: are you bringing a cattle prod to Sydney? | 21:46 |
*** vladikr has quit IRC | 21:46 | |
mriedem | edleafe: kangerprod | 21:46 |
mriedem | i think you can get them at the corner store | 21:46 |
*** coreycb has quit IRC | 21:46 | |
*** simondodsley has quit IRC | 21:46 | |
*** jbryce has quit IRC | 21:46 | |
edleafe | damn you, I was typing that! | 21:46 |
*** tbachman has joined #openstack-nova | 21:46 | |
*** gmann has quit IRC | 21:47 | |
*** efried has quit IRC | 21:47 | |
*** evrardjp has quit IRC | 21:47 | |
*** rm_work has quit IRC | 21:47 | |
*** dr_gogeta86 has quit IRC | 21:48 | |
*** sergek_ has quit IRC | 21:48 | |
*** coreycb has joined #openstack-nova | 21:48 | |
*** simondodsley has joined #openstack-nova | 21:48 | |
*** adisky has joined #openstack-nova | 21:48 | |
*** amotoki has quit IRC | 21:48 | |
*** tanee has quit IRC | 21:49 | |
*** pkoniszewski has quit IRC | 21:49 | |
*** MVenesio has quit IRC | 21:49 | |
*** tanee has joined #openstack-nova | 21:50 | |
*** sergek_ has joined #openstack-nova | 21:50 | |
*** pkoniszewski has joined #openstack-nova | 21:51 | |
*** amotoki has joined #openstack-nova | 21:51 | |
*** evrardjp has joined #openstack-nova | 21:52 | |
*** bauzas has joined #openstack-nova | 21:52 | |
*** dr_gogeta86 has joined #openstack-nova | 21:53 | |
*** dr_gogeta86 has quit IRC | 21:53 | |
*** dr_gogeta86 has joined #openstack-nova | 21:53 | |
*** felipemonteiro__ has quit IRC | 21:59 | |
*** felipemonteiro__ has joined #openstack-nova | 22:00 | |
*** zhenguo has joined #openstack-nova | 22:01 | |
*** redondo-mk has joined #openstack-nova | 22:01 | |
*** rm_work has joined #openstack-nova | 22:02 | |
*** rm_work has quit IRC | 22:02 | |
*** rm_work has joined #openstack-nova | 22:02 | |
*** efried has joined #openstack-nova | 22:03 | |
*** owalsh_ has joined #openstack-nova | 22:04 | |
dansmith | mriedem: did you see this? https://review.openstack.org/#/c/506370/ | 22:05 |
dansmith | I don't think the bot barfed it into here else I missed it | 22:06 |
*** felipemonteiro__ has quit IRC | 22:06 | |
mriedem | ha, no | 22:06 |
*** owalsh- has joined #openstack-nova | 22:06 | |
*** kylek3h has quit IRC | 22:06 | |
*** owalsh has quit IRC | 22:07 | |
*** owalsh- is now known as owalsh | 22:07 | |
mriedem | http://docs-draft.openstack.org/70/506370/1/check/gate-nova-docs-ubuntu-xenial/08cdb00//doc/build/html/reference/live-migration.html | 22:07 |
dansmith | I almost have migration_uuid claims working for cold migration | 22:07 |
dansmith | gibi laid down the gauntlet with his tests | 22:08 |
*** owalsh_ has quit IRC | 22:09 | |
mriedem | efried: you're a docs machine | 22:09 |
efried | mriedem I was waiting for that to build so I could send you the rendered version. But you what, did it locally? | 22:10 |
mriedem | no looked at the job results | 22:10 |
efried | oh, you found it before it was done, cool. | 22:10 |
mriedem | status.openstack.org/zuul | 22:10 |
melwitt | he probably spied it on the zuul status page | 22:10 |
efried | So it ain't perfect, but it's not bad. | 22:10 |
melwitt | dammit | 22:10 |
mriedem | efried: 1000% better than what we have today, which is nothing | 22:10 |
mriedem | you'll have to add that new dependency to global-requirements | 22:11 |
efried | It's already there | 22:11 |
*** ijw has joined #openstack-nova | 22:13 | |
efried | Oh - I misread the dot version: https://github.com/openstack/requirements/blob/master/global-requirements.txt#L424 | 22:13 |
efried | mriedem Do you want to add some words around that doc in this patch? | 22:13 |
openstackgerrit | Eric Fried proposed openstack/nova master: Live Migration sequence diagram https://review.openstack.org/506370 | 22:15 |
*** bigdogstl has joined #openstack-nova | 22:16 | |
efried | mriedem Fixed that version and added co-author line ^ | 22:16 |
mriedem | we can describe the operations in a follow up | 22:17 |
*** hferenc has quit IRC | 22:17 | |
mriedem | not as pretty is right! | 22:17 |
*** slaweq has joined #openstack-nova | 22:17 | |
mriedem | not sure why the reqs job was failing then | 22:18 |
*** hferenc has joined #openstack-nova | 22:18 | |
mriedem | oh it's the space | 22:18 |
mriedem | it literally has to match "sphinxcontrib-seqdiag>=0.8.4 # BSD" | 22:18 |
*** artom has quit IRC | 22:19 | |
mriedem | maybe not, this might work | 22:19 |
*** artom has joined #openstack-nova | 22:20 | |
efried | I had it as .5 before | 22:20 |
mriedem | yup see that now | 22:20 |
mriedem | cool, this should pass | 22:20 |
mriedem | glorious | 22:20 |
efried | yeah, it's not complaining about the actdiag one, which has two spaces before the comment as well. | 22:20 |
*** artom has quit IRC | 22:21 | |
*** artom has joined #openstack-nova | 22:22 | |
*** artom has quit IRC | 22:22 | |
*** slaweq has quit IRC | 22:22 | |
*** bigdogstl has quit IRC | 22:23 | |
*** artom has joined #openstack-nova | 22:23 | |
*** artom has quit IRC | 22:23 | |
*** ioggstream has joined #openstack-nova | 22:23 | |
*** artom has joined #openstack-nova | 22:24 | |
*** cdent has quit IRC | 22:27 | |
*** penick_ has quit IRC | 22:31 | |
*** artom has quit IRC | 22:32 | |
*** sdague has quit IRC | 22:34 | |
*** mriedem has quit IRC | 22:37 | |
*** xyang1 has quit IRC | 22:38 | |
*** slaweq has joined #openstack-nova | 22:39 | |
*** lyan has quit IRC | 22:40 | |
openstackgerrit | Merged openstack/nova master: Drop support for the Cinder v2 API https://review.openstack.org/501874 | 22:42 |
*** tidwellr has quit IRC | 22:43 | |
openstackgerrit | Jay Pipes proposed openstack/nova master: placement: set/check if inventory change in tree https://review.openstack.org/470575 | 22:45 |
openstackgerrit | Jay Pipes proposed openstack/nova master: placement: integrate ProviderTree to report client https://review.openstack.org/415921 | 22:45 |
openstackgerrit | Jay Pipes proposed openstack/nova master: placement: add nested resource providers https://review.openstack.org/377138 | 22:45 |
openstackgerrit | Jay Pipes proposed openstack/nova master: placement: allow filter providers in tree https://review.openstack.org/377215 | 22:45 |
openstackgerrit | Jay Pipes proposed openstack/nova master: placement: adds REST API for nested providers https://review.openstack.org/384807 | 22:45 |
*** yassine_ has joined #openstack-nova | 22:45 | |
openstackgerrit | Jay Pipes proposed openstack/nova master: placement: update client to set parent provider https://review.openstack.org/385693 | 22:45 |
*** slaweq has quit IRC | 22:45 | |
*** tbachman has quit IRC | 22:53 | |
*** tidwellr has joined #openstack-nova | 22:53 | |
*** bigdogstl has joined #openstack-nova | 22:53 | |
*** Sree has joined #openstack-nova | 22:54 | |
*** Sukhdev has quit IRC | 22:57 | |
*** edmondsw has joined #openstack-nova | 22:58 | |
*** Sree has quit IRC | 22:59 | |
*** bigdogstl has quit IRC | 23:00 | |
*** edmondsw has quit IRC | 23:02 | |
*** penick has joined #openstack-nova | 23:02 | |
*** slaweq has joined #openstack-nova | 23:03 | |
*** penick has quit IRC | 23:06 | |
*** bigdogstl has joined #openstack-nova | 23:06 | |
*** slaweq has quit IRC | 23:08 | |
*** bigdogstl has quit IRC | 23:11 | |
*** ioggstream has quit IRC | 23:12 | |
*** tasker has quit IRC | 23:13 | |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Add recreate test for unshelve offloaded instance spawn fail https://review.openstack.org/506414 | 23:15 |
*** hamzy has joined #openstack-nova | 23:15 | |
*** claudiub has quit IRC | 23:17 | |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Add a regression test for bug 1718455 https://review.openstack.org/506092 | 23:18 |
openstack | bug 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 IRC | 23:18 | |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Fix definitely move single instance when created concurrently https://review.openstack.org/506093 | 23:19 |
*** hongbin has quit IRC | 23:23 | |
*** slaweq has joined #openstack-nova | 23:24 | |
*** gmann has joined #openstack-nova | 23:29 | |
openstackgerrit | Hongbin Li proposed openstack/nova master: Remove method "_get_host_ref_from_name" https://review.openstack.org/504796 | 23:31 |
*** slaweq has quit IRC | 23:31 | |
*** chyka has quit IRC | 23:34 | |
*** chyka has joined #openstack-nova | 23:34 | |
*** jmlowe has joined #openstack-nova | 23:35 | |
openstackgerrit | Dan Smith proposed openstack/nova master: Fix a pagination logic bug in test_bug_1689692 https://review.openstack.org/505661 | 23:35 |
openstackgerrit | Dan Smith proposed openstack/nova master: Use improved instance_list module in compute API https://review.openstack.org/505418 | 23:35 |
openstackgerrit | Dan Smith proposed openstack/nova master: Remove legacy fault-loading routines https://review.openstack.org/505456 | 23:35 |
openstackgerrit | Dan Smith proposed openstack/nova master: Fix CellDatabases fixture swallowing exceptions https://review.openstack.org/506312 | 23:35 |
openstackgerrit | Dan Smith proposed openstack/nova master: Fix minor input items from previous patches https://review.openstack.org/506416 | 23:35 |
*** gouthamr has joined #openstack-nova | 23:40 | |
*** markvoelker has quit IRC | 23:40 | |
*** bigdogstl has joined #openstack-nova | 23:41 | |
*** takashin has joined #openstack-nova | 23:42 | |
*** tbachman has joined #openstack-nova | 23:43 | |
*** jbryce has joined #openstack-nova | 23:45 | |
*** thorst has joined #openstack-nova | 23:46 | |
*** Swami has quit IRC | 23:46 | |
*** slaweq has joined #openstack-nova | 23:48 | |
*** bigdogstl has quit IRC | 23:50 | |
*** thorst has quit IRC | 23:50 | |
*** thorst has joined #openstack-nova | 23:50 | |
*** thorst has quit IRC | 23:53 | |
*** slaweq has quit IRC | 23:54 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!