| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-05-09 | |||
| 09:15:33 | stephenfin | kashyap: No, you can remove it but call that out and fix the test accordingly | |
| 09:15:46 | stephenfin | No point keeping dead code around | |
| 09:16:04 | kashyap | stephenfin: Right. So the plan is: Deprecate it in Rocky; delete support for it in Stein. | |
| 09:16:22 | kashyap | I have it locally as you indicated | |
| 09:24:50 | openstackgerrit | lei zhang proposed openstack/python-novaclient master: Fix the incorrect cirros default password https://review.openstack.org/567170 | |
| 09:28:25 | openstackgerrit | jichenjc proposed openstack/nova master: [Doc]Link policies file into api https://review.openstack.org/561503 | |
| 09:30:27 | kashyap | stephenfin: With the 'assert_has_calls', you meant: | |
| 09:30:28 | kashyap | ]) | |
| 09:30:28 | kashyap | mock.call(...), | |
| 09:30:28 | kashyap | mock_warn.assert_has_calls([ | |
| 09:30:33 | kashyap | ? | |
| 09:30:36 | stephenfin | yup | |
| 09:31:12 | kashyap | Maybe I should consider that. | |
| 09:31:38 | openstackgerrit | jichenjc proposed openstack/nova master: Enhance api-guide general info some updates https://review.openstack.org/561773 | |
| 09:37:10 | openstackgerrit | Chris Dent proposed openstack/nova master: Optional separate database for placement API https://review.openstack.org/362766 | |
| 09:37:11 | openstackgerrit | Chris Dent proposed openstack/nova master: Isolate placement database config https://review.openstack.org/541435 | |
| 09:37:12 | openstackgerrit | Chris Dent proposed openstack/nova master: WIP: Ensure that os-traits sync is attempted only at start of process https://review.openstack.org/553857 | |
| 09:37:29 | openstackgerrit | Chris Dent proposed openstack/nova master: WIP: Add PLACEMENT_DB_ENABLED=True to the nova-next job https://review.openstack.org/564067 | |
| 09:44:40 | openstackgerrit | jichenjc proposed openstack/nova master: [doc] Add soft_deleted flag https://review.openstack.org/562897 | |
| 09:53:52 | openstackgerrit | Jan Gutter proposed openstack/nova stable/queens: Add plumbing for Contrail/Tungsten Fabric offloads https://review.openstack.org/567177 | |
| 09:59:06 | liuzz_ | hello everyone, Who can help me? i want to create a new role and only allow it to manage keystone resource, can't manager other reources(instances, volumes, ...),What should I do？ | |
| 10:01:24 | kashyap | stephenfin: How about I just assert that the warning has happened N number of times for each event? | |
| 10:01:44 | kashyap | Like: self.assertEquals(N, mock_warn.call_count) | |
| 10:03:27 | stephenfin | I'm not sure. I'd just do what you've done there | |
| 10:03:57 | kashyap | No problem; I'll tinker a bit | |
| 10:04:32 | kashyap | (Calling the exact warning message isn't nice.) | |
| 10:04:47 | kashyap | Anyhow, disregard me for now | |
| 10:06:06 | openstackgerrit | Kashyap Chamarthy proposed openstack/nova master: libvirt: Deprecate support for monitoring Intel CMT `perf` events https://review.openstack.org/565242 | |
| 10:26:42 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Simplify instance name generation https://review.openstack.org/516573 | |
| 10:29:17 | openstackgerrit | Stephen Finucane proposed openstack/nova master: tox: Add mypy target https://review.openstack.org/538221 | |
| 10:29:18 | openstackgerrit | Stephen Finucane proposed openstack/nova master: tox: Store list of converted files https://review.openstack.org/538222 | |
| 10:29:19 | openstackgerrit | Stephen Finucane proposed openstack/nova master: mypy: Add type annotations to 'nova.pci' https://review.openstack.org/538224 | |
| 10:29:20 | openstackgerrit | Stephen Finucane proposed openstack/nova master: zuul: Add 'mypy' job https://review.openstack.org/539168 | |
| 10:31:46 | openstackgerrit | Chris Dent proposed openstack/nova master: VMware: Live migration of instances https://review.openstack.org/270116 | |
| 10:32:05 | cdent | johnthetubaguy: there's a spec and real ci behind that ^ now | |
| 10:32:12 | cdent | (it's got your -2 on it) | |
| 10:37:45 | andreas_s | melwitt: Hi, I'm facing an issue with the following devstack patch you commited a few days ago: https://github.com/openstack-dev/devstack/commit/12579c3db7b28381c8ec97945aa23ee02d54d22b | |
| 10:38:23 | stephenfin | bauzas: If you're about today, could you take a look at https://review.openstack.org/#/c/541290/ ? | |
| 10:38:40 | andreas_s | melwitt: It seems like this patch is moving the console configurtion (vnc, spice, serial) from nova.conf into the nova-cell config | |
| 10:39:12 | andreas_s | melwitt: but the n-cpu systemd service is not consuming the nova_cell config - with the result, that the console setting is being ignored | |
| 10:40:44 | stephenfin | andreas_s: I was thinking the exact same thing but chose to ignore my intuition https://review.openstack.org/#/c/550141/1/ | |
| 10:41:07 | stephenfin | (vs. https://review.openstack.org/#/c/550141/2/) | |
| 10:41:36 | andreas_s | stephenfin: ah good to know | |
| 10:43:22 | andreas_s | stephenfin: but the challenge is still that devstack seems to specify the cell_config file as input parameter to configure_console_proxies, doesn't it? | |
| 10:43:55 | andreas_s | at least in our thirdparty CI the config landed in the cell config file: http://sng01.objectstorage.softlayer.net/v1/AUTH_1940ea10-6e82-4501-b2f9-eb236510e575/ibmzkvmci/production/510897/13/check-tempest-dsvm-neutron-full-ubuntu-xenial-s390x/3e67649/logs/etc/nova/ | |
| 10:44:57 | stephenfin | andreas_s: Yup. I'm still trying to figure out which specific setting the conductors (which I think are the only services to use those files) need configured | |
| 10:45:24 | stephenfin | because I thought most/all of them are only used by nova-compute | |
| 10:45:59 | andreas_s | yeah, nova-compute needs this config in definitively! | |
| 10:47:52 | stephenfin | andreas_s: I'd file a bug, personally. I'll take a look at it after lunch and see if we can come up with something | |
| 10:49:01 | jmccarthy | mriedem: Thanks for the update on https://review.openstack.org/#/c/566367 ! I tested this out, and patchest2 worked great for me | |
| 10:49:38 | andreas_s | stephenfin: that would be great. thanks. Can you subscribe me to this bug once opened (andreas scheuring)? thx. If you need any more input, let know... | |
| 10:50:24 | stephenfin | andreas_s: "I would file a bug, personally", e.g. I suggest _you_ file a bug :) | |
| 10:50:33 | stephenfin | I could do it, but you probably have the most context | |
| 10:50:36 | stephenfin | and logs | |
| 10:50:41 | andreas_s | ok, will do | |
| 10:50:57 | andreas_s | stephenfin: just was not sure how to interpret "I would" ;) | |
| 10:51:13 | stephenfin | That's an Irish'ism. Sorry :) | |
| 10:51:55 | andreas_s | hehe, enjoy your lunch! | |
| 10:53:58 | openstackgerrit | Merged openstack/nova master: Remove [scheduler]/host_manager config option https://review.openstack.org/565806 | |
| 11:02:53 | openstackgerrit | Kashyap Chamarthy proposed openstack/nova master: libvirt: Deprecate support for monitoring Intel CMT `perf` events https://review.openstack.org/565242 | |
| 11:04:38 | openstackgerrit | Rajat Sharma proposed openstack/nova master: Imposing restrictions on resource providers create uuid https://review.openstack.org/567191 | |
| 11:06:18 | openstack | Launchpad bug 1770143 in devstack "Nova Console setup broken" [Undecided,New] | |
| 11:06:18 | andreas_s | stephenfin, melwitt: opened the bug against devstack: https://bugs.launchpad.net/devstack/+bug/1770143 | |
| 11:11:05 | ShilpaSD | dansmith: hi | |
| 11:29:25 | openstackgerrit | Naichuan Sun proposed openstack/nova master: XenAPI: deprecate the config for image handler class path https://review.openstack.org/497201 | |
| 12:09:29 | openstackgerrit | Chris Dent proposed openstack/nova master: Optional separate database for placement API https://review.openstack.org/362766 | |
| 12:09:30 | openstackgerrit | Chris Dent proposed openstack/nova master: Isolate placement database config https://review.openstack.org/541435 | |
| 12:09:31 | openstackgerrit | Chris Dent proposed openstack/nova master: WIP: Ensure that os-traits sync is attempted only at start of process https://review.openstack.org/553857 | |
| 12:10:03 | openstackgerrit | Chris Dent proposed openstack/nova master: WIP: Add PLACEMENT_DB_ENABLED=True to the nova-next job https://review.openstack.org/564067 | |
| 12:19:31 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Followup for multiple member_of qparams support https://review.openstack.org/567209 | |
| 12:30:39 | openstackgerrit | Radoslav Gerganov proposed openstack/nova stable/queens: Handle PortNotFoundClient exception when getting ports https://review.openstack.org/567211 | |
| 12:41:34 | openstackgerrit | Jan Gutter proposed openstack/nova-specs master: Spec to implement vRouter HW offloads https://review.openstack.org/567148 | |
| 13:01:15 | mdbooth | efried: Morning! https://review.openstack.org/#/c/564017/ | |
| 13:01:51 | mdbooth | That's causing me a problem, and I considering proposing a revert. However, I think it only exists due to bad naming. | |
| 13:02:15 | efried | mdbooth: stand by. | |
| 13:02:30 | efried | mdbooth: Okay, do tell. | |
| 13:02:47 | mdbooth | efried: I see you proposed it because of the loop in the ironic driver | |
| 13:03:13 | efried | mdbooth: Well, actually because of [2] (in the commit message), but [1] was a good excuse. | |
| 13:03:25 | mdbooth | efried: Sec, lemme check 2 | |
| 13:03:26 | openstackgerrit | Aditya Vaja proposed openstack/nova master: remove IVS plug/unplug as they're moved to separate plugin https://review.openstack.org/534371 | |
| 13:03:50 | efried | mdbooth: TL;DR, we were checking for volume-ness in a different (and apparently wrong | |
| 13:03:55 | efried | ) way in the powervm driver. | |
| 13:04:15 | mdbooth | efried: Thing is, the ironic driver doesn't need that check | |
| 13:04:16 | efried | mdbooth: mriedem pointed out that we should be using is_volume... which led us to find out that it didn't exist. | |
| 13:04:38 | efried | mdbooth: Well, is the new version of the check doing something the old one wasn't? | |
| 13:04:47 | mdbooth | Because, confusingly, block_device_info_get_mapping() should actually be called block_device_info_get_volumes() | |
| 13:04:55 | efried | mdbooth: I.e. did we break something afresh, or just leave something that (we didn't know) was broken before? | |
| 13:05:11 | mdbooth | If it was called *_get_volumes(), it would be obvious that an is_volume call was redundant subsequently | |
| 13:05:39 | jaypipes | stephenfin: yes, sorry, I understand the differences between gre, vlan, vxlan, local, etc. what I'm wondering about is why you used the term "mechanism". Is that because of the use of "mechanism driver" to describe the implementation module that Neutron uses under the hood for constructing a particular network? | |
| 13:06:10 | mdbooth | Where's _vol_drv_iter() called in your patch? | |
| 13:06:21 | stephenfin | jaypipes: I think I saw it used in documentation somewhere and couldn't think of a better term :) | |
| 13:07:41 | mdbooth | efried: Where I'm going with that is... are you sure they aren't all volumes anyway? And if they're not, why not just get the volumes in the first place? | |
| 13:08:35 | jaypipes | stephenfin: the reason I asked is because neither of those pages mentions the term "mechanism", and it seems to me (I may very well be wrong) that you are conflating two things (segmentation protocol and whether overlapping IP address ranges can occur on the network) | |
| 13:09:12 | mdbooth | efried: Ah, yes. You're doing the same thing. | |
| 13:09:18 | mdbooth | efried: So, they're all volumes. | |
| 13:09:28 | efried | mdbooth: So let's back up just for a sec. | |
| 13:09:45 | mdbooth | Have a look in driver.get_block_device_info(), which is what generates that data | |
| 13:09:49 | mdbooth | See: | |
| 13:09:55 | mdbooth | 'block_device_mapping': | |
| 13:09:56 | mdbooth | virt_block_device.convert_all_volumes(*block_device_mapping) | |