| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-06-01 | |||
| 12:55:25 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Avoid forcing translation on logging calls https://review.openstack.org/413876 | |
| 13:01:15 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Add PCIWeigher https://review.openstack.org/379524 | |
| 13:01:16 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Prefer non-PCI host nodes for non-PCI instances https://review.openstack.org/379625 | |
| 13:05:46 | mriedem | can i get someone to run the nova team meeting in about 55 minutes? i'll be taking my kid to school. | |
| 13:06:35 | mriedem | meeting agenda is already updated https://wiki.openstack.org/wiki/Meetings/Nova | |
| 13:12:28 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Fix call to driver_detach in remove_volume_connection https://review.openstack.org/469740 | |
| 13:15:25 | bhagyashris | johnthetubguy_ : Hi, | |
| 13:20:01 | bhagyashris | johnthetubguy_ : Addressed review comments given patch : https://review.openstack.org/#/c/409644/ .Could you please take some insights on patch? Thank you in advance :) | |
| 13:25:18 | bauzas | mriedem: I can | |
| 13:30:34 | mriedem | bauzas: thanks | |
| 13:30:43 | mriedem | bhagyashris: john is out until next week | |
| 13:38:53 | bauzas | mriedem: just in case, I have a couple of changes you can look at | |
| 13:39:24 | bauzas | mriedem: https://review.openstack.org/#/c/469037/ could be nice for starting a new BP :) | |
| 13:45:30 | mriedem | sfinucan: can you take a look at https://bugs.launchpad.net/nova/+bug/1694417 - i think the dhcp_domain option is only used for nova-network, but the config option says use_neutron is a related option, so it's kind of confusing | |
| 13:45:30 | openstack | Launchpad bug 1694417 in OpenStack Compute (nova) "dhcp_domain used when use_neutron is not set" [Undecided,Invalid] | |
| 13:45:46 | sfinucan | mriedem: Sure | |
| 13:50:39 | bauzas | mriedem: is it me or most of the 3rd-party CI are trampled ? | |
| 13:52:23 | bauzas | wow, lots of reds | |
| 13:52:30 | bauzas | something is wrong with CI atm | |
| 13:52:41 | bauzas | wtf | |
| 13:54:12 | mriedem | vmware is always down, see my email to the ML | |
| 13:54:20 | mriedem | xenserver is also non-voting for now | |
| 13:54:31 | mriedem | xenproject also is failing once ssh validation was turned on by default | |
| 13:55:51 | bauzas | mriedem: my email address is stupidely having problems for getting new emails, it's fuckily delayed | |
| 13:59:37 | bhagyashris | mriedem: ok. Thank you for update. Could you please take some insights on patch: https://review.openstack.org/#/c/409644/ if you will get free time. | |
| 14:00:04 | egonzalez | mriedem, ping | |
| 14:28:24 | openstackgerrit | Alex Szarka proposed openstack/nova master: Add method for verify multiple versioned notifications https://review.openstack.org/465526 | |
| 14:29:13 | openstackgerrit | Alex Szarka proposed openstack/nova master: Transform instance.exists notification https://review.openstack.org/403660 | |
| 14:39:04 | openstackgerrit | Alex Szarka proposed openstack/nova master: Transform instance.exists notification https://review.openstack.org/403660 | |
| 14:41:10 | mriedem | sean-k-m1: at some point i wanted to ask you about https://review.openstack.org/#/c/441183/ which came up in the nova meeting today | |
| 14:41:11 | mriedem | you and jay | |
| 14:41:40 | mriedem | i.e. is this fine as a plugin way of doing things? i think everything else nova has supported for os-vif types has been in the os-vif library, so this would be the first out of tree plugin | |
| 14:42:11 | mriedem | egonzalez: hi | |
| 14:42:27 | openstackgerrit | Alex Szarka proposed openstack/nova master: Add method for verify multiple versioned notifications https://review.openstack.org/465526 | |
| 14:43:07 | mriedem | "THE place for all your vrouter needs!" | |
| 14:43:14 | egonzalez | mriedem, just wanted to explain the use_neutron/dns bug | |
| 14:43:37 | takashin | dansmith: Would you review https://review.openstack.org/#/c/408955/ again? | |
| 14:43:42 | sfinucan | egonzalez: I left comments on said bug | |
| 14:44:56 | takashin | dansmith: Are you around? | |
| 14:45:24 | egonzalez | mriedem, sfinucan thanks will wait until the metadata issue is fixed to try re verify | |
| 14:45:43 | openstackgerrit | Alex Szarka proposed openstack/nova master: Refactor create_delete_server_with_instance_update https://review.openstack.org/466296 | |
| 14:45:51 | sfinucan | egonzalez: which metadata issue? | |
| 14:46:20 | egonzalez | sfinucan, opened another bug for that https://bugs.launchpad.net/nova/+bug/1694666 | |
| 14:46:20 | openstack | Launchpad bug 1694666 in OpenStack Compute (nova) "metadata service PicklingError: Can't pickle <class 'sqlalchemy.orm.session.Session'>" [Undecided,New] | |
| 14:47:11 | sean-k-m1 | mriedem: looking at it now | |
| 14:47:12 | sfinucan | egonzalez: Ah, well I guess that answers my question - you _are_ using the metadata service so? | |
| 14:47:20 | sfinucan | ...presumably with cloud-init? | |
| 14:47:26 | mriedem | mhenkel: i commented on this in the patch and bp, but i'd like to see documentation about how to use this plugin somewhere, which should probably at least be in the readme for the plugin itself https://github.com/Juniper/contrail-nova-vif-driver | |
| 14:47:49 | mhenkel | yes, got the point | |
| 14:47:55 | egonzalez | sfinucan, no cloud-init at all, both issues are different | |
| 14:48:18 | mriedem | i thought we've had this pickling issue before, or reported before at least | |
| 14:48:36 | egonzalez | sfinucan, this last one is, fresh deploy. create a cirros instance and metadata raising those errors | |
| 14:48:48 | sfinucan | egonzalez: and the other one? | |
| 14:49:24 | sean-k-mooney | mriedem: so i would like to make it so that https://review.openstack.org/#/c/441183 would not be required by having neutron pass the plugin to use to nova when we update neutron to pass os-vif objects | |
| 14:50:41 | mriedem | sean-k-mooney: that's the long-term goal but no one is working on that, correct? | |
| 14:50:55 | mriedem | i think ian wells said he'd work on it at both the austin and barcelona summits but it hasn't happened | |
| 14:51:13 | sean-k-mooney | well i am if i can get to it. | |
| 14:51:29 | mriedem | egonzalez: do you know if that metadata pickling issue also happens in ocata? | |
| 14:51:37 | mhenkel | bauzas: on the relevance to dpdk | |
| 14:51:51 | sean-k-mooney | mriedem: i want to do a poc of that before the end of pike | |
| 14:51:57 | mhenkel | this patch is only needed for dpdk | |
| 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 | 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 | |