| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-04-13 | |||
| 07:59:53 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Enable cold migration with target host(2/2) https://review.openstack.org/408964 | |
| 08:00:53 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Add functional tests for cold migration to same host https://review.openstack.org/414926 | |
| 08:01:05 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: api-ref: Add parameters in cold migrate action https://review.openstack.org/410042 | |
| 08:01:36 | openstackgerrit | Takashi NATSUME proposed openstack/python-novaclient master: Microversion 2.42 - Fix tag attribute disappearing https://review.openstack.org/429512 | |
| 08:01:57 | openstackgerrit | Takashi NATSUME proposed openstack/python-novaclient master: Microversion 2.43 - Enable cold migration with target host https://review.openstack.org/406707 | |
| 08:02:02 | openstackgerrit | Takashi NATSUME proposed openstack/python-novaclient master: Microversion 2.43: List/Show all server migration types https://review.openstack.org/430839 | |
| 08:02:49 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Avoid forcing translation on logging calls https://review.openstack.org/413876 | |
| 08:15:06 | gmann | alex_xu: ping | |
| 08:19:52 | openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/nova master: Read Neutron port 'binding_profile' during boot https://review.openstack.org/449257 | |
| 08:22:15 | openstackgerrit | Jan Gutter proposed openstack/nova-specs master: Spec to enable Netronome SmartNIC support https://review.openstack.org/456386 | |
| 08:22:38 | alex_xu | gmann: pong | |
| 08:23:54 | gmann | alex_xu: on extension removal works | |
| 08:23:59 | alex_xu | gmann: yup | |
| 08:24:01 | gmann | alex_xu: i am doing on falvor - https://github.com/openstack/nova/blob/master/setup.cfg#L99-L103 | |
| 08:24:21 | gmann | alex_xu: to avoid any conflict in case we end up doing on same APIs | |
| 08:24:32 | alex_xu | gmann: thanks :) | |
| 08:24:59 | alex_xu | gmann: do you think we are worth to sync at somewhere? | |
| 08:25:23 | alex_xu | like etherpad | |
| 08:25:35 | alex_xu | or it is just fine, we can ping eachother anytime | |
| 08:25:53 | gmann | alex_xu: may be or like i start from end and you from starting in setup.cfg | |
| 08:26:27 | gmann | alex_xu: or etherpad also fine for me we can keep entering name on APIs we start | |
| 08:27:03 | alex_xu | gmann: ok, how about etherpad, we already have one https://etherpad.openstack.org/p/api-no-more-extensions-pike | |
| 08:27:48 | gmann | alex_xu: yup, writing there :) | |
| 08:28:30 | gmann | alex_xu: added flavor i should be able to push patch today | |
| 08:28:57 | alex_xu | gmann: cool, thanks :) | |
| 08:29:18 | gmann | alex_xu: np!, thank you | |
| 08:50:28 | openstackgerrit | jichenjc proposed openstack/nova master: Deprecate os-hosts API https://review.openstack.org/456504 | |
| 08:50:29 | openstackgerrit | jichenjc proposed openstack/nova master: Update inc to deprecate os-hosts https://review.openstack.org/456505 | |
| 08:58:13 | openstackgerrit | jichenjc proposed openstack/nova master: Check instance existing before check in mapping https://review.openstack.org/433470 | |
| 09:03:23 | cdent | bauzas, johnthetubaguy, alex_xu: if you guys have some input on this spec about changing putting resource classes, that would be appreciated. it's a bit stalled, but the associated code is mostly done and useful: https://review.openstack.org/#/c/453732/ | |
| 09:03:36 | bauzas | okay | |
| 09:04:10 | johnthetubaguy | cdent: is that part of making things more efficient / less chatty? | |
| 09:04:18 | cdent | yes | |
| 09:04:29 | johnthetubaguy | I am sold then, I will take a peak at that today | |
| 09:04:38 | cdent | thanks | |
| 09:07:05 | alex_xu | cdent: got it | |
| 09:07:11 | cdent | thanks | |
| 09:07:15 | openstackgerrit | Gábor Antal proposed openstack/nova master: Transform instance.volume_attach.error notification https://review.openstack.org/455801 | |
| 09:07:51 | alex_xu | cdent: one more for you https://review.openstack.org/450125, it is last one for traits API, i'm afraid people forget that patch | |
| 09:08:17 | alex_xu | ok, I sent message late one second | |
| 09:08:38 | takashin | johnthetubaguy: Would you review https://review.openstack.org/#/c/407237/ again? | |
| 09:09:16 | johnthetubaguy | takashin: yes, I must get to that one again, thanks for the update | |
| 09:09:46 | takashin | johnthetubaguy: Thank you. | |
| 09:15:25 | jangutter | Hi, in the spirit of last-minute mad dashes, we finally obtained the clearance to submit the w-i-p code for the spec: https://review.openstack.org/#/c/456386/ | |
| 09:16:05 | jangutter | Would anyone here with SR-IOV and/or OVS experience be interested in reviewing? | |
| 09:16:56 | openstackgerrit | jichenjc proposed openstack/nova master: Deprecate os-hosts API https://review.openstack.org/456504 | |
| 09:16:57 | openstackgerrit | jichenjc proposed openstack/nova master: Update inc to deprecate os-hosts https://review.openstack.org/456505 | |
| 09:38:18 | openstackgerrit | jichenjc proposed openstack/nova master: [placement] Add doc for resource provider usages https://review.openstack.org/450105 | |
| 09:41:09 | openstackgerrit | Lee Yarwood proposed openstack/nova master: encryptors: Switch to os-brick encryptor classes https://review.openstack.org/391597 | |
| 09:42:07 | openstackgerrit | Lee Yarwood proposed openstack/nova master: encryptors: Do not overwrite file volumes on attach_volume https://review.openstack.org/452938 | |
| 09:43:29 | lyarwood | mdbooth: ^ if you have time this morning, I'm looking to land the bugfix for attaching encrypted files so I can backport it _before_ the change to rm -rf the code from our codebase. | |
| 09:43:52 | mdbooth | lyarwood: If my connection stays up long enough | |
| 09:44:22 | lyarwood | mdbooth: cheers | |
| 09:44:37 | openstackgerrit | Stephen Finucane proposed openstack/os-vif master: doc: Add glossary https://review.openstack.org/452666 | |
| 09:44:38 | openstackgerrit | Stephen Finucane proposed openstack/os-vif master: docs: Stop building anything but html output https://review.openstack.org/456535 | |
| 09:51:43 | openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Fix HTTP 500 raised for getConsoleLog for stopped instance https://review.openstack.org/455183 | |
| 09:52:43 | cdent | alex_xu: thanks for the remind on the sync command. I thought I had commented on that already,but I must have dismissed it without saving. Will look at it asap | |
| 10:04:11 | mdbooth | lyarwood: Just thinking about https://review.openstack.org/#/c/452938/ | |
| 10:04:35 | lyarwood | mdbooth: ah the hackaround | |
| 10:04:47 | mdbooth | I don't like that either, tbh, because I don't think Nova should be messing with volumes owned by cinder | |
| 10:04:59 | mdbooth | How about we just explicitly don't support it? | |
| 10:05:16 | mdbooth | It wouldn't be a regression, as it previously blew away your data anyway | |
| 10:05:23 | lyarwood | mdbooth: yeah it's hackaround or block it tbh | |
| 10:05:44 | lyarwood | mdbooth: there are downstream users jumping up and down asking for this | |
| 10:05:57 | mdbooth | I think the fix should be to pass a different path to qemu | |
| 10:06:38 | mdbooth | lyarwood: Hmm, we're still risking their data, though | |
| 10:06:55 | mdbooth | Surely cinder operations are going to be broken while Nova is doing its thing | |
| 10:06:59 | mdbooth | For eg | |
| 10:07:24 | lyarwood | mdbooth: yeah with this renaming they will be | |
| 10:07:46 | mdbooth | Ideally we'd leave it alone entirely, create a dev mapper device on top of it, then use the dev mapper device instead of the original device, which remains untouched | |
| 10:08:31 | mdbooth | IIRC we're only doing this hack because the existing code doesn't allow the encryptor to modify the path used by other Nova code | |
| 10:09:20 | lyarwood | mdbooth: yeah because we do all of this _after_ creating the libvirtdiskconfig objects iirc | |
| 10:09:25 | mdbooth | I'm really not comfortable with Nova moving cinder volumes around physically, especially when we don't tell cinder about it | |
| 10:09:48 | lyarwood | ack that's ah great point that I missed tbh | |
| 10:09:52 | lyarwood | a* | |
| 10:10:09 | mdbooth | What's the plan for a real fix, btw? | |
| 10:10:56 | lyarwood | well, my QEMU 2.6 native LUKS spec landed yesterday so that's one way of `fixing` this | |
| 10:12:03 | mdbooth | lyarwood: Of course. | |
| 10:12:04 | lyarwood | really not sure what we can do for the existing encryptors aside from block this for file based volumes for now | |
| 10:13:28 | mdbooth | lyarwood: Honestly my short term fix would be to raise an unsupported exception instead of deleting the user's volume | |
| 10:14:24 | mdbooth | lyarwood: Long term qemu native luks | |
| 10:14:40 | lyarwood | mdbooth: cool, I'll post a new change for that later today. | |
| 10:14:59 | mdbooth | If we're going to fix it as it is, I think a Nova refactor to allow us to switch out the path | |
| 10:15:42 | mdbooth | Disgusting hack potential: change the nfs volume driver to use a loopback device instead of the file directly | |
| 10:28:28 | johnthetubaguy | cdent: so the PUT request has no body in your spec, is that correct? | |
| 10:28:51 | cdent | johnthetubaguy: that's right, because there's nothing useful to put in it | |
| 10:29:14 | johnthetubaguy | cdent: saves the need to check it matches the URL or fail, makes sense I think | |
| 10:30:26 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Use plain routes list for flavor endpoint instead of stevedore[1] https://review.openstack.org/456549 | |
| 10:31:30 | cdent | johnthetubaguy, bauzas: I'll make a new version in few minutes to include the info that bauzas asked for | |
| 10:40:53 | sfinucan | cdent: In the absence of jaypipes (wherever he's gone to), could you look at https://review.openstack.org/#/c/361140/ again and see if there's any must-have changes needed today? | |
| 10:41:13 | cdent | sfinucan: sure, will do right after fixing this other spec | |
| 10:41:23 | sfinucan | cdent: spot on | |
| 10:51:26 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Use plain routes list for flavor endpoint instead of stevedore[1] https://review.openstack.org/456549 | |
| 10:53:25 | sfinucan | pkoniszewski: You around? Question on the 'live_migration_scheme' and 'live_migraton_inbound_addr' options | |
| 10:53:52 | pkoniszewski | sfinucan: yes, what's up? | |
| 10:54:08 | openstackgerrit | Chris Dent proposed openstack/nova-specs master: A spec for idempotent custom resource class creation https://review.openstack.org/453732 | |
| 10:54:18 | sfinucan | pkoniszewski: What was your rationale behind df334b4 ? | |
| 10:54:31 | sfinucan | I'm guessing it's a usability thing, but was there anything more subtle than that? | |
| 10:54:55 | sfinucan | pkoniszewski: https://github.com/openstack/nova/commit/df334b4f414b8c341709df37ca10065d3b50fcef for reference | |
| 10:56:36 | pkoniszewski | sfinucan: so my point was to deprecate and finally remove live_migration_uri which is used for hacking the interface used for live migrations, now there is live_migration_inbound_addr to choose an interface | |