| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-06-01 | |||
| 14:52:17 | sfinucan | egonzalez: as a quick hack, what happens if you change [1] to 'return "%s.%s" % (self.instance.hostname, "bananas")' [1] https://github.com/openstack/nova/blob/a1eca94d89ba49eaa521303dfa8d991237cb5eb7/nova/api/metadata/base.py#L535-L538 | |
| 14:52:21 | egonzalez | mriedem, last manual test worked in ocata yesterday. gates are happy at too | |
| 14:52:27 | sfinucan | easiest way to check if that's where the problem lies ;) | |
| 14:52:35 | sean-k-mooney | mriedem: but regardign that patch was there something specific you wanted my input on? | |
| 14:52:54 | bauzas | mhenkel: sure, I understand that, I just wanted to explain that we only need to name the BP as what it's provided by the plugin | |
| 14:53:04 | mriedem | egonzalez: is that what you mean by "Only source images from master fail, binary(rdo or ubuntu packages) works for now" - are the binary packages from nova 15.0.0 (ocata)? | |
| 14:53:24 | mriedem | sean-k-mooney: in general are you ok with it? besides needing to probably change the vif type name to something less generic, and needing a release note | |
| 14:53:49 | egonzalez | mriedem, nope, probably packages are not build or propagated to main delorean repo | |
| 14:53:50 | sean-k-mooney | mriedem: yes i have previously revied this and i just reviewed it again | |
| 14:54:19 | sean-k-mooney | mriedem: it looks fine to me bar the name as you said | |
| 14:54:28 | mhenkel | bauzas: something like: nova-vif-plugin-for-opencontrail-vrouter? | |
| 14:55:53 | mriedem | sfinucan: egonzalez: so basically it sounds like if you're using neutron, we should set dhcp_domain to '' as the default so you'll just use the instance.hostname https://github.com/openstack/nova/blob/a1eca94d89ba49eaa521303dfa8d991237cb5eb7/nova/api/metadata/base.py#L537 | |
| 14:56:33 | mriedem | maybe people getting dns from neutron/designate just knew to change the dhcp_domain config option? i think miguel lavalle added that support in mitaka | |
| 14:56:35 | mriedem | for neutron dns i mean | |
| 14:57:35 | mlavalle | mriedem: hey | |
| 14:57:44 | sfinucan | mriedem: Yeah, we should probably ignore the nova-network-only option | |
| 14:58:18 | openstack | Launchpad bug 1694417 in OpenStack Compute (nova) "dhcp_domain used when use_neutron is not set" [Undecided,Incomplete] | |
| 14:58:18 | mriedem | mlavalle: trying to sort out this bug https://bugs.launchpad.net/nova/+bug/1694417 | |
| 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 a list of instance UUIDs to scheduler https://review.openstack.org/465171 | |
| 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: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 | openstack | Launchpad bug 1694897 in Zun "neutron tag xxx.xxx doesn't work correctly" [Undecided,New] | |
| 15:19:25 | mriedem | shouldn't https://bugs.launchpad.net/zun/+bug/1694897 be targeted at python-neutronclient rather than zun? | |
| 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 | openstack | Launchpad bug 1694666 in OpenStack Compute (nova) "metadata service PicklingError: Can't pickle <class 'sqlalchemy.orm.session.Session'>" [Undecided,New] | |
| 15:58:43 | mriedem | artom: have you seen something like this? https://bugs.launchpad.net/nova/+bug/1694666 | |
| 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. | |