| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-05-10 | |||
| 20:27:22 | dansmith | I would like to circle back, | |
| 20:27:35 | dansmith | and point out that the only reason we're discussing this is because we emit a few warnings during unit tests :) | |
| 20:30:03 | mriedem | that's not why i'm talking about it, | |
| 20:30:06 | mriedem | i don't care about the map_instances thing | |
| 20:30:18 | efried | arvindn05: I was totally going to let the typos slide, but that last docstring is wrong enough to warrant a respin. | |
| 20:30:23 | efried | arvindn05: karma ^^ | |
| 20:31:10 | melwitt | it's true that the unit test warnings drew attention to the marker-record-storing pattern. but I think lots of people have been unaware of that pattern and it's pretty surprising, or at least it was to me the first time I learned about it | |
| 20:31:34 | arvindn05 | arvindn05: ...good catches...will fix ASAP | |
| 20:31:38 | dansmith | ...because it has just been working? :) | |
| 20:32:19 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Add contributor docs on deprecating and removing compute REST APIs https://review.openstack.org/567687 | |
| 20:32:21 | melwitt | yep, I guess so. no problems, no eyeballs drawn to it | |
| 20:35:00 | mriedem | dansmith: want to hit these backports https://review.openstack.org/#/q/Idea86d6bb648b1e6fef8813dbe569724ce81a750 - then we can safely remove the legacy live migration job from project-config and openstack-zuul-jobs | |
| 20:35:07 | mriedem | dansmith: shouldn't require 2 +2s i don't think | |
| 20:35:20 | dansmith | OMG YES | |
| 20:35:26 | mriedem | otherwise right now we're getting hit on master b/c we're running 2 live migration jbos | |
| 20:35:26 | dansmith | <-- loves hitting backports | |
| 20:35:46 | mriedem | it will take that as (1) serious and (2) literal | |
| 20:35:58 | dansmith | mriedem: hah | |
| 20:36:05 | melwitt | :D | |
| 20:36:10 | mriedem | i've been holding https://review.openstack.org/#/q/Ib10081150e125961cba19cfa821bddfac4614408 | |
| 20:36:34 | mriedem | then we can do a (sweet) release | |
| 20:36:52 | mriedem | this is ready too https://review.openstack.org/#/c/567271/ | |
| 20:39:45 | openstackgerrit | Merged openstack/nova master: Remove mox in test_compute_api.py (1) https://review.openstack.org/564415 | |
| 20:40:40 | mriedem | oomichi: ha https://review.openstack.org/#/c/564415/3/nova/tests/unit/compute/test_compute_api.py@1013 | |
| 20:40:52 | mriedem | oomichi: check out the vmware virt driver unit tests at some point | |
| 20:41:15 | oomichi | mriedem: hehe, more longer ? :) | |
| 20:43:38 | openstackgerrit | Merged openstack/nova master: libvirt: check image type before removing snapshots in _cleanup_resize https://review.openstack.org/566367 | |
| 20:44:18 | mriedem | ok now i can bug dan about those backports | |
| 20:45:53 | melwitt | nova meeting in 15 minutes | |
| 20:48:13 | openstackgerrit | Jay Pipes proposed openstack/nova master: placement: always create consumer records https://review.openstack.org/567678 | |
| 20:52:28 | oomichi | mriedem: btw where does 410 come from ? related to https://review.openstack.org/#/c/567687 | |
| 20:52:52 | oomichi | 410 means http410 status code | |
| 20:53:41 | openstackgerrit | Arvind Nadendla proposed openstack/nova master: update scheduler to use image-traits https://review.openstack.org/560596 | |
| 20:54:30 | mriedem | https://www.w3.org/Protocols/rfc2616/rfc2616-sec10.html | |
| 20:54:33 | arvindn05 | efried: not going to say anything...but new PS uploaded :) | |
| 20:54:38 | efried | looking now | |
| 20:54:53 | mriedem | oomichi: "The requested resource is no longer available at the server and no forwarding address is known. This condition is expected to be considered permanent." | |
| 20:55:16 | efried | arvindn05: +2 from me. Now you have to get by mriedem... | |
| 20:55:50 | arvindn05 | nice...thanks for the quick turnaround efried | |
| 20:57:22 | oomichi | mriedem: I didn't see http410 case and api-wg guideline doesn't mention about http410 case, so I was curious about that | |
| 20:58:06 | mriedem | oomichi: it's what we used when we removed os-cloudpipe and os-certificates, and it makes more sense then 404 | |
| 20:58:14 | mriedem | *than | |
| 20:58:33 | mriedem | 404 is what we used for capping / deprecating with microversions | |
| 20:58:48 | mriedem | technically we also use 404 when a route is not found | |
| 20:59:48 | oomichi | mriedem: yeah, 404 is my base thinking. Nice to have conversation widely | |
| 21:00:27 | mriedem | oomichi: feel free to bring it up in the ML thread | |
| 21:00:41 | mriedem | but at this point it would be weird to use 410 for those other removed apis but 404 for fping | |
| 21:00:45 | mriedem | IMO | |
| 21:01:23 | oomichi | OK, let me do that | |
| 21:07:19 | openstack | Launchpad bug 1770378 in OpenStack Compute (nova) "windows can‘t start when use vhostuser interface" [Undecided,New] | |
| 21:07:19 | mriedem | sean-k-mooney: this is for you https://bugs.launchpad.net/nova/+bug/1770378 | |
| 21:08:57 | mriedem | sean-k-mooney: i noticed that we have 0 docs in nova or neutron on using vhostuser for networking | |
| 21:08:59 | mriedem | is that somewhere else? | |
| 21:09:19 | mriedem | oh hmm here it is https://docs.openstack.org/neutron/latest/contributor/internals/ovs_vhostuser.html | |
| 21:09:48 | sean-k-mooney | mriedem: stephen added docs to the networking guide | |
| 21:10:06 | mriedem | right but https://docs.openstack.org/neutron/latest/search.html?q=vhostuser | |
| 21:10:48 | mriedem | oh b/c it's vhost-user | |
| 21:12:09 | sean-k-mooney | mriedem: this is where the docs used to be https://docs.openstack.org/neutron/pike/admin/config-ovs-dpdk.html | |
| 21:13:14 | sean-k-mooney | anyway regarding the bug you should be able to run windows guest on vhost-user ports however i do remember issues in the past due to virtio diriver versions | |
| 21:14:31 | sean-k-mooney | mriedem: this might be the fix for the issue http://dpdk.org/dev/patchwork/patch/15541/ | |
| 21:41:16 | mnaser | mriedem: https://review.openstack.org/#/c/565611/ ok for me to merge this or wait till https://review.openstack.org/#/q/Idea86d6bb648b1e6fef8813dbe569724ce81a750 is in? | |
| 21:41:38 | openstack | Launchpad bug 1770378 in OpenStack Compute (nova) "windows can‘t start when use vhostuser interface" [Undecided,Incomplete] | |
| 21:41:38 | sean-k-mooney | mriedem: i have responded to https://bugs.launchpad.net/nova/+bug/1770378 and set it to incomplete until they respond with the version of ovs and dpdk they are using | |
| 21:48:08 | openstackgerrit | Merged openstack/nova master: Move create_tap_dev to the new utility module. https://review.openstack.org/557835 | |
| 21:50:02 | mriedem | takashin: this is the event https://www.openstack.org/summit/vancouver-2018/summit-schedule/events/21749/missing-features-in-openstack-for-public-clouds | |
| 21:50:35 | takashin | mriedem: Thanks. I will attend it. | |
| 21:51:15 | mriedem | https://etherpad.openstack.org/p/YVR-publiccloud-wg-brainstorming | |
| 21:51:20 | mriedem | that's the only etherpad i'm aware of right now | |
| 21:53:06 | mriedem | sean-k-mooney: thanks for commenting on that vhostuser bug | |
| 21:53:16 | mriedem | mnaser: that's good to go now | |
| 21:53:36 | mnaser | mriedem: +w :) | |
| 22:03:12 | openstack | Launchpad bug 1770348 in OpenStack Compute (nova) "Unable to access console via dashboard" [Undecided,Incomplete] | |
| 22:03:12 | mriedem | melwitt: i saw this in triage today https://bugs.launchpad.net/nova/+bug/1770348 | |
| 22:03:13 | mriedem | mnaser: thanks | |
| 22:06:34 | openstackgerrit | Hongbin Lu proposed openstack/nova-specs master: Choose default network on ambiguity https://review.openstack.org/520247 | |
| 22:16:29 | melwitt | mriedem: looking | |
| 22:16:47 | openstackgerrit | Merged openstack/nova stable/queens: Migrate tempest-dsvm-multinode-live-migration job in-tree https://review.openstack.org/567612 | |
| 22:16:52 | openstackgerrit | Merged openstack/nova stable/ocata: Migrate tempest-dsvm-multinode-live-migration job in-tree https://review.openstack.org/567619 | |
| 22:21:49 | openstackgerrit | Merged openstack/nova stable/pike: log stale allocations as WARNING instead of DEBUG https://review.openstack.org/567271 | |
| 22:25:46 | openstackgerrit | Hongbin Lu proposed openstack/nova-specs master: Choose default network on ambiguity https://review.openstack.org/520247 | |
| 22:28:05 | melwitt | mriedem: that would have been my first question too, whether they're using devstack or a real deployment. what's probably wrong is they need to pass the cell nova.conf to the console proxy service instead of the old global nova.conf | |
| 22:29:42 | melwitt | I'm thinking maybe I need to make the release note better to call that out | |
| 22:31:11 | mrf | Hi | |
| 22:31:27 | mrf | does images_rbd_pool at nova.conf support multiple pools? | |
| 22:31:31 | openstackgerrit | Merged openstack/nova master: Make get_instance_objects_sorted() be smart about cells https://review.openstack.org/509003 | |
| 22:33:04 | melwitt | mrf: I don't believe so. mnaser could you confirm? ^ | |
| 22:33:15 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Add nova-slow job https://review.openstack.org/567697 | |
| 22:33:30 | mriedem | there's your slow job | |
| 22:33:32 | mnaser | mriedem: it does not unfortunately, it can only use a single rbd image backend | |
| 22:33:36 | mnaser | err | |
| 22:33:39 | mnaser | mrf: ** ^ | |
| 22:33:52 | melwitt | mriedem: awesome thanks | |
| 22:33:59 | melwitt | wznoinsk: ^ | |
| 22:36:35 | mriedem | ok stable backports for this https://review.openstack.org/#/q/Ib10081150e125961cba19cfa821bddfac4614408 are ready | |
| 22:36:38 | mriedem | but probably no one around | |
| 22:39:59 | mriedem | oh tonyb mtreinish ^ :) | |
| 22:42:52 | mrf | mmm i got a trouble with 2 computes nodes , maybe some could help each one got diferent images_rbd_pool | |
| 22:43:04 | mrf | and now i cant migrate instances beetween them | |
| 22:45:13 | tonyb | mriedem: Thanks, I'll review them after this meeting | |
| 23:02:45 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Fix 500 error while passing 4-byte unicode data https://review.openstack.org/407514 | |