| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-06-01 | |||
| 14:58:18 | mriedem | mlavalle: trying to sort out this bug https://bugs.launchpad.net/nova/+bug/1694417 | |
| 14:58:18 | openstack | Launchpad bug 1694417 in OpenStack Compute (nova) "dhcp_domain used when use_neutron is not set" [Undecided,Incomplete] | |
| 14:58:37 | mriedem | and wondering if neutron + dns means you need to configure nova's dhcp_domain option to '' when using neutron and the metadata service | |
| 14:58:53 | mriedem | mlavalle: feel free to reply on the bug later if you're busy | |
| 14:59:28 | mlavalle | mriedem: at first glance, it should't be related | |
| 14:59:51 | mlavalle | mriedem: at look at the bug later, though | |
| 14:59:58 | mriedem | mlavalle: your spec https://specs.openstack.org/openstack/nova-specs/specs/mitaka/implemented/internal-dns-resolution.html just talks about cloud-init, not about the metadata service, so i think that might be the issue | |
| 14:59:59 | mriedem | thanks | |
| 15:00:01 | egonzalez | going to add config file as reference in the bug | |
| 15:00:11 | mriedem | egonzalez: can you also show the port details? | |
| 15:00:19 | mriedem | would like to see if dns_name is set on the port | |
| 15:01:03 | egonzalez | mriedem, sure | |
| 15:12:14 | openstackgerrit | Ed Leafe proposed openstack/nova master: Have scheduler claim resources https://review.openstack.org/463993 | |
| 15:12:15 | openstackgerrit | Ed Leafe proposed openstack/nova master: Pass the number of hosts to return explicitly https://review.openstack.org/465767 | |
| 15:12:15 | openstackgerrit | Ed Leafe proposed openstack/nova master: Pass a list of instance UUIDs to scheduler https://review.openstack.org/465171 | |
| 15:12:16 | openstackgerrit | Ed Leafe proposed openstack/nova master: Return multiple hosts from select_destinations https://review.openstack.org/465768 | |
| 15:12:42 | edleafe | cdent: bauzas: ^^ rebased and updated | |
| 15:12:57 | cdent | yay! | |
| 15:14:12 | edleafe | cdent: You'll be happy to learn that your precious blank line has been returned unharmed | |
| 15:14:29 | cdent | doubleyay! | |
| 15:17:14 | hongbin | mriedem: hi matt, after discussing with neutron ptl, we would like to ban neutronclient 6.3.0 due to an issue: http://lists.openstack.org/pipermail/openstack-dev/2017-June/117763.html | |
| 15:17:40 | hongbin | mriedem: i saw nova team proposed to bump the version to 6.3.0, so want to check with you: https://github.com/openstack/requirements/commit/210b33e01658374ac248a188cfdcb35a64a35c80 | |
| 15:18:06 | mriedem | hongbin: that was for the global request id stuff sdague was working on | |
| 15:18:36 | hongbin | mriedem: yes, i couldn't find sdague in the irc right now | |
| 15:18:43 | mriedem | he's on vacation this week | |
| 15:19:00 | hongbin | ok | |
| 15:19:25 | mriedem | shouldn't https://bugs.launchpad.net/zun/+bug/1694897 be targeted at python-neutronclient rather than zun? | |
| 15:19:25 | openstack | Launchpad bug 1694897 in Zun "neutron tag xxx.xxx doesn't work correctly" [Undecided,New] | |
| 15:19:52 | hongbin | yes, i should change it | |
| 15:20:01 | alexchadin | hi Nova team | |
| 15:20:08 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Fix missing marker functions https://review.openstack.org/469920 | |
| 15:21:01 | alexchadin | how can I get compute.node metrics from nova-compute? I set compute_monitors=cpu.virt_driver but still don't get any compute-node related metrics | |
| 15:21:08 | alexchadin | Ocata version | |
| 15:29:10 | hongbin | mriedem: fyi: https://review.openstack.org/469923 , hope it doesn't break anything on nova side | |
| 15:30:36 | openstackgerrit | Michael Henkel proposed openstack/nova master: contrail: add vrouter VIF plugin type support https://review.openstack.org/441183 | |
| 15:34:28 | openstackgerrit | Alvaro Lopez Garcia proposed openstack/nova master: xen: pass Xen console in cmdline https://review.openstack.org/469826 | |
| 15:40:33 | mriedem | hongbin: we're good https://review.openstack.org/#/c/468356/ the nova change didn't land yet | |
| 15:40:55 | hongbin | mriedem: great. thanks for confirming | |
| 15:41:39 | openstackgerrit | Michael Henkel proposed openstack/nova master: contrail: add vrouter VIF plugin type support https://review.openstack.org/441183 | |
| 15:51:18 | openstackgerrit | Chris Dent proposed openstack/nova master: Sync os-traits to Traits database https://review.openstack.org/469578 | |
| 15:52:09 | cdent | edleafe: that ^ adds some locks and other checks but I haven't added a test that alex_xu asked for. if you feel like taking it on after the api-wg meeting go for it. i'll be gone after that | |
| 15:57:13 | edleafe | cdent: I'll try, but I have a pretty full plate today. If not today, then definitely tomorrow | |
| 15:57:31 | cdent | thanks | |
| 15:58:43 | mriedem | artom: have you seen something like this? https://bugs.launchpad.net/nova/+bug/1694666 | |
| 15:58:43 | openstack | Launchpad bug 1694666 in OpenStack Compute (nova) "metadata service PicklingError: Can't pickle <class 'sqlalchemy.orm.session.Session'>" [Undecided,New] | |
| 15:58:55 | mriedem | or know of someone working on reducing InstanceMetadata to a smaller primitive for pickling? | |
| 15:59:49 | artom | mriedem, whoa | |
| 16:00:15 | artom | mriedem, no, haven't seen anything like it | |
| 16:00:25 | artom | Why is it trying to pickle the session? o_O | |
| 16:00:42 | mriedem | yeah, it pickles everything in InstanceMetadata, which includes a bunch of versioned objects | |
| 16:00:44 | mriedem | like Instance | |
| 16:01:04 | mriedem | and since we do db api stuff in the objects now, i'm not surprised Session is pulled in | |
| 16:01:14 | artom | Hrmm | |
| 16:01:20 | artom | This is with memcache | |
| 16:01:25 | mriedem | i thought at some point someone from red hat was working on fixing this up because of performance/security issues with what we were serializing | |
| 16:01:31 | artom | Which I suspect isn't tested in the gate anywhere | |
| 16:01:47 | artom | Not to my knowledge | |
| 16:01:56 | artom | But I'm not omniscient ;) | |
| 16:03:05 | mriedem | we don't use memcached in the gate as far as i know, default dogpile cache is in-memory dict i think | |
| 16:03:20 | artom | Makes sense, which is why we never noticed this | |
| 16:03:28 | mriedem | there is a new etcd backend for oslo.cache, and etcd is installed in jobs on master now in devstack, on the primary node | |
| 16:03:43 | mriedem | so we could start testing with that.... | |
| 16:04:03 | clarkb | mriedem: I think we do install memcached at least | |
| 16:04:18 | clarkb | so you can strt testing with either or | |
| 16:04:19 | mriedem | ok we just don't configure to use it | |
| 16:04:20 | artom | Would it hit the issue though? I thought the issue is that python-memcache pickles the stuff we hand it | |
| 16:04:34 | mriedem | kolla is running with memcache configured for this, | |
| 16:04:37 | mriedem | that's how they hit the bug | |
| 16:04:45 | mriedem | http://logs.openstack.org/73/469373/1/check/gate-kolla-ansible-dsvm-deploy-centos-source-centos-7-nv/8cecb36/logs/kolla/nova/nova-api.log | |
| 16:05:00 | artom | So either we dramatically simplify InstanceMetadata so that it's picklable | |
| 16:05:27 | artom | Or... can python-memcache not pickle stuff? I guess it has to to put them in RAM | |
| 16:05:44 | mriedem | it uses pickle | |
| 16:05:48 | mriedem | i don't see any option not to | |
| 16:05:52 | mriedem | in python-memached i mean | |
| 16:05:57 | mriedem | i was looking at the code earlier | |
| 16:05:58 | artom | What if we just IntanceMetada.to_primitive()? | |
| 16:06:04 | artom | And then from_primitive() when we get it back? | |
| 16:06:07 | mriedem | i thought about something like that | |
| 16:06:26 | mriedem | since we have that for the versioned objects contained within | |
| 16:06:28 | artom | I sense a but coming | |
| 16:06:33 | mriedem | no but | |
| 16:06:38 | mriedem | would just need to hack on it and test it out | |
| 16:07:22 | artom | I *think* I could grab that? I kinda have other stuff going on downstream, but I could try to make it work | |
| 16:09:03 | efried | mriedem Re powervm blueprint: we need to keep it open at least long enough to get the docs changes in. We still need to have an internal powwow as to whether we want to drive more function this release; will get back to you soon on that. | |
| 16:12:51 | mriedem | artom: ack, not laying it on you, was just wondering if you had heard about this or knew someone working on it already | |
| 16:12:54 | mriedem | but don't feel obliged | |
| 16:13:10 | artom | mriedem, my parents raised me on guilt, I feel obliged ;) | |
| 16:13:37 | mriedem | that works for me | |
| 16:13:50 | artom | mriedem, in seriousness, it's a bug, I have the solution in my head, it's the responsible thing to do | |
| 16:14:14 | artom | Plus, how hard can it be? | |
| 16:58:31 | mriedem | sean-k-mooney: don't you have to configure nova.conf for os-vif plugins that are out of tree? | |
| 16:58:41 | mriedem | or does os-vif just automagically register those if it can import them via stevedore? | |
| 17:03:40 | mriedem | this would be their extension point https://github.com/Juniper/contrail-nova-vif-driver/blob/master/setup.py#L27 | |
| 17:03:46 | mriedem | which i guess needs to be renamed also | |
| 17:05:25 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Connecting Nova to DRBD storage nodes directly https://review.openstack.org/256292 | |
| 17:09:02 | mriedem | ^ is an easy bp win for something that's been sitting around nearly 2.5 years | |
| 17:09:10 | mriedem | i finally just cleaned it up myself | |
| 17:13:22 | openstackgerrit | Michael Henkel proposed openstack/nova master: contrail: add vrouter VIF plugin type support https://review.openstack.org/441183 | |
| 17:14:15 | sean-k-mooney | mriedem: reading back | |
| 17:14:36 | openstackgerrit | John Griffith proposed openstack/nova master: WIP/POC Implement new attach Cinder flow https://review.openstack.org/330285 | |