| *** ijw has quit IRC | 00:01 | |
| *** ijw has joined #openstack-nova | 00:02 | |
| *** Jeffrey4l has quit IRC | 00:03 | |
| *** Jeffrey4l has joined #openstack-nova | 00:03 | |
| *** Apoorva has quit IRC | 00:04 | |
| *** Apoorva has joined #openstack-nova | 00:04 | |
| *** ljxiash has joined #openstack-nova | 00:06 | |
| *** hongbin has quit IRC | 00:07 | |
| *** jamesdenton has joined #openstack-nova | 00:08 | |
| *** ducttape_ has quit IRC | 00:09 | |
| *** ducttape_ has joined #openstack-nova | 00:10 | |
| *** ljxiash has quit IRC | 00:10 | |
| *** wang_ has joined #openstack-nova | 00:10 | |
| wang_ | Hi all, i have some question regarding how to setup and configure the nova-lxd, is this the right channel ? | 00:11 |
|---|---|---|
| *** ducttape_ has quit IRC | 00:14 | |
| *** ijw has quit IRC | 00:22 | |
| *** ijw has joined #openstack-nova | 00:22 | |
| *** diga has joined #openstack-nova | 00:25 | |
| jaypipes | edleafe, mriedem, bauzas, dansmith: sorry guys had to step out for a few hours. reading back through the long discussions... | 00:27 |
| mriedem | wang_: probably not as it's not in the nova tree, | 00:28 |
| mriedem | wang_: try finding zigo in #openstack-dev | 00:28 |
| *** nic has quit IRC | 00:29 | |
| *** yuntongjin has joined #openstack-nova | 00:36 | |
| *** wang_ has quit IRC | 00:37 | |
| oomichi | gmann: re: https://review.openstack.org/#/c/402372 The corresponding Temepst patch is https://review.openstack.org/#/c/408254/ | 00:40 |
| oomichi | gmann: we can see the behavior from the log | 00:41 |
| *** jamielennox is now known as jamielennox|away | 00:45 | |
| *** ihrachys has joined #openstack-nova | 00:46 | |
| *** thorst_ has joined #openstack-nova | 00:47 | |
| *** thorst_ has quit IRC | 00:50 | |
| *** thorst_ has joined #openstack-nova | 00:51 | |
| *** ihrachys has quit IRC | 00:51 | |
| *** hoangcx has joined #openstack-nova | 00:54 | |
| *** yuntongjin has quit IRC | 00:55 | |
| oomichi | gmann: alex_xu: oops, maybe we did mistake related to block-device-mapping validation | 00:56 |
| oomichi | gmann: alex_xu: new bdm should contain no_device like https://github.com/openstack/nova/blob/master/nova/block_device.py#L47 | 00:57 |
| oomichi | but the corresponding json-schema doesn't contain it | 00:57 |
| oomichi | its additionalProperties: True now, so that is not so big problem | 00:58 |
| openstackgerrit | Jay Pipes proposed openstack/nova: placement: API filters providers by resources https://review.openstack.org/408285 | 00:58 |
| oomichi | but we missed it anyways | 00:58 |
| openstackgerrit | wanghao proposed openstack/nova: Set min_disk in the image meta based on the root volume's size https://review.openstack.org/407397 | 00:59 |
| *** thorst_ has quit IRC | 00:59 | |
| *** jistr|mtgs has quit IRC | 01:00 | |
| *** jistr has joined #openstack-nova | 01:00 | |
| openstackgerrit | Ken'ichi Ohmichi proposed openstack/nova: Remove unnecessary comment of BDM validation https://review.openstack.org/408370 | 01:00 |
| *** ijw has quit IRC | 01:02 | |
| *** diga has quit IRC | 01:03 | |
| *** jamesdenton has quit IRC | 01:04 | |
| gmann | oomichi: nice catch | 01:04 |
| *** Apoorva has quit IRC | 01:04 | |
| gmann | oomichi: even we can add another tests with no destination_type and see behavior of volume and server | 01:04 |
| *** Apoorva has joined #openstack-nova | 01:05 | |
| *** fragatin_ has joined #openstack-nova | 01:06 | |
| *** smatzek has joined #openstack-nova | 01:07 | |
| *** phuongnh has joined #openstack-nova | 01:07 | |
| *** fragatina has quit IRC | 01:08 | |
| *** tovin07 has quit IRC | 01:08 | |
| mriedem | dansmith: looking at the cell setup here https://review.openstack.org/#/c/406380/9/nova/test.py | 01:09 |
| *** hieulq has quit IRC | 01:09 | |
| mriedem | database_connection is a uuid? | 01:09 |
| mriedem | on the cell mapping | 01:09 |
| *** Apoorva has quit IRC | 01:09 | |
| dansmith | mriedem: it's anything, but I use the uuid | 01:09 |
| dansmith | technically it's a url for sqlalchemy, | 01:09 |
| dansmith | but it really needs to be invalid as a url here, because if we leak it too deep down, a non-conforming string will blow up and be easy to spot | 01:10 |
| dansmith | which happened multiple times while working on this | 01:10 |
| *** fragatin_ has quit IRC | 01:11 | |
| mriedem | heh, ServiceFixture | 01:11 |
| mriedem | def __init__(self, name, host=None, **kwargs): | 01:11 |
| mriedem | name = name | 01:11 |
| mriedem | that seems useful | 01:11 |
| mriedem | ok | 01:12 |
| *** tuanluong has joined #openstack-nova | 01:12 | |
| *** ijw has joined #openstack-nova | 01:12 | |
| *** smatzek has quit IRC | 01:14 | |
| mriedem | dansmith: alright matey, some comments in https://review.openstack.org/#/c/406380/9/nova/test.py | 01:16 |
| mriedem | otherwise looks ok | 01:16 |
| dansmith | mriedem: thanks will hit those tomorrow | 01:16 |
| *** jamesdenton has joined #openstack-nova | 01:17 | |
| *** Jeffrey4l has quit IRC | 01:18 | |
| *** Jeffrey4l has joined #openstack-nova | 01:19 | |
| *** portdirect is now known as portdirect_away_ | 01:20 | |
| *** thorst_ has joined #openstack-nova | 01:20 | |
| *** thorst_ has quit IRC | 01:20 | |
| *** thorst_ has joined #openstack-nova | 01:21 | |
| *** rfolco has joined #openstack-nova | 01:23 | |
| *** kevinz has joined #openstack-nova | 01:24 | |
| *** jschwarz has quit IRC | 01:25 | |
| *** portdirect_away_ has quit IRC | 01:25 | |
| *** dave-mccowan has joined #openstack-nova | 01:26 | |
| *** jschwarz has joined #openstack-nova | 01:27 | |
| *** zhangjl has joined #openstack-nova | 01:28 | |
| *** browne has quit IRC | 01:28 | |
| *** jaypipes has quit IRC | 01:29 | |
| *** thorst_ has quit IRC | 01:29 | |
| *** jamesdenton has quit IRC | 01:29 | |
| *** mingyu has joined #openstack-nova | 01:38 | |
| *** ducttape_ has joined #openstack-nova | 01:39 | |
| *** esberglu has joined #openstack-nova | 01:40 | |
| *** rfolco has quit IRC | 01:40 | |
| *** mtanino has quit IRC | 01:41 | |
| *** Sukhdev has quit IRC | 01:42 | |
| *** Kevin_Zheng has joined #openstack-nova | 01:46 | |
| *** diogogmt has quit IRC | 01:49 | |
| *** jamielennox|away is now known as jamielennox | 01:50 | |
| *** mriedem has quit IRC | 01:50 | |
| *** mriedem has joined #openstack-nova | 01:50 | |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova: Merge v20 view builder for limit API https://review.openstack.org/406787 | 01:54 |
| *** jwcroppe has joined #openstack-nova | 01:55 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: Support tag instances when boot https://review.openstack.org/394321 | 02:00 |
| *** yufei has joined #openstack-nova | 02:02 | |
| *** ayoung has joined #openstack-nova | 02:03 | |
| *** gmann has quit IRC | 02:03 | |
| *** gmann has joined #openstack-nova | 02:03 | |
| *** weshay has quit IRC | 02:07 | |
| *** weshay has joined #openstack-nova | 02:09 | |
| *** gcb has joined #openstack-nova | 02:12 | |
| *** ducttape_ has quit IRC | 02:12 | |
| *** ducttape_ has joined #openstack-nova | 02:12 | |
| *** sdague has quit IRC | 02:12 | |
| *** mdnadeem has joined #openstack-nova | 02:13 | |
| *** jamesdenton has joined #openstack-nova | 02:15 | |
| melwitt | mriedem: this patch reminded me of work I think you did to stash volume connectors https://review.openstack.org/#/c/387226 so I thought you might be able to weigh in | 02:17 |
| *** ijw has quit IRC | 02:17 | |
| openstackgerrit | ChangBo Guo(gcb) proposed openstack/nova: Port virt.disk.mount.test_nbd to Python 3 https://review.openstack.org/403508 | 02:23 |
| openstackgerrit | ChangBo Guo(gcb) proposed openstack/nova: Port libvirt.storage.test_rbd to Python 3 https://review.openstack.org/403425 | 02:23 |
| openstackgerrit | ChangBo Guo(gcb) proposed openstack/nova: Port libvirt.test_firewall to Python 3 https://review.openstack.org/403423 | 02:23 |
| *** gjayavelu has quit IRC | 02:24 | |
| *** Jeffrey4l has quit IRC | 02:26 | |
| *** _ducttape_ has joined #openstack-nova | 02:26 | |
| *** Jeffrey4l has joined #openstack-nova | 02:26 | |
| *** lamt has joined #openstack-nova | 02:29 | |
| *** ducttape_ has quit IRC | 02:30 | |
| *** Jeffrey4l has quit IRC | 02:31 | |
| *** Jeffrey4l has joined #openstack-nova | 02:31 | |
| openstackgerrit | int32bit proposed openstack/python-novaclient: Add minDisk and minRam filters to list flavors https://review.openstack.org/407855 | 02:33 |
| *** baoli has joined #openstack-nova | 02:36 | |
| *** thorst_ has joined #openstack-nova | 02:36 | |
| *** ijw has joined #openstack-nova | 02:37 | |
| *** mtanino has joined #openstack-nova | 02:37 | |
| *** Zer0Byte__ has quit IRC | 02:39 | |
| openstackgerrit | ChangBo Guo(gcb) proposed openstack/nova: Port xenapi test_vm_utils to Python 3 https://review.openstack.org/401827 | 02:40 |
| *** ijw has quit IRC | 02:42 | |
| *** bkopilov has quit IRC | 02:43 | |
| *** cdelatte has quit IRC | 02:43 | |
| gcb | gmann, I replied your comment in https://review.openstack.org/#/c/406103 | 02:44 |
| *** thorst_ has quit IRC | 02:45 | |
| *** Jeffrey4l has quit IRC | 02:46 | |
| *** Jeffrey4l has joined #openstack-nova | 02:47 | |
| gmann | gcb: thanks. checking.. | 02:48 |
| *** hfu has joined #openstack-nova | 02:48 | |
| *** jamesdenton has quit IRC | 02:50 | |
| *** diogogmt has joined #openstack-nova | 02:50 | |
| *** baoli has quit IRC | 02:50 | |
| mriedem | melwitt: oh boy, mark sturdevant stashed the connectors, i think i used the stashed connector in a local delete bug once though to terminate the vol connection with cinder rather than pass the fake connector | 02:51 |
| melwitt | mriedem: oh, that local delete thing must have been what I was thinking about | 02:53 |
| openstackgerrit | xianming.mao proposed openstack/nova: Wrap attachment operation with the try-except https://review.openstack.org/407828 | 02:54 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova: Enable cold migration with target host https://review.openstack.org/357105 | 02:54 |
| *** ijw has joined #openstack-nova | 02:55 | |
| *** kaisers_ has quit IRC | 02:56 | |
| *** artom has quit IRC | 02:57 | |
| gmann | gcb: m ok with list with index, actually dict_value would not allow the indexing | 03:01 |
| *** ijw has quit IRC | 03:01 | |
| gmann | gcb: please check my other comment, otherwise looks fine for me | 03:01 |
| *** mtanino has quit IRC | 03:02 | |
| gcb | gmann: thanks | 03:02 |
| mriedem | Kevin_Zheng: i left some more comments in https://review.openstack.org/#/c/394321/ | 03:02 |
| mriedem | Kevin_Zheng: we should probably talk to sdague about ideas on the extension for server create | 03:02 |
| mriedem | gmann might have ideas there too as I think he looked at some of the extension cleanup, and alex_xu | 03:02 |
| mriedem | Kevin_Zheng: we also need api samples functional tests and api-ref docs updates in there - and i left some comments about the actual tag creation in the API - I'm not sure if we'll do it there or later in conductor, i'd need to talk to dansmith about that | 03:03 |
| mriedem | as the instance creation is moving to conductor for cells v2 | 03:03 |
| gmann | i see, ll check today. as we are trying to cleanup those extensions things but let's see how much it adding | 03:04 |
| mriedem | gmann: it's pretty minimal, | 03:04 |
| mriedem | i think he's just trying to leverage some existing validation code, | 03:04 |
| mriedem | but i think we can just move that into a static utility method that both the server and tags extensions can use | 03:04 |
| Kevin_Zheng | mriedem: Thanks a lot, I've just done the validation rewrite and working on the tests and api-ref :) | 03:05 |
| gmann | mriedem: yea thats better. | 03:05 |
| *** esberglu has quit IRC | 03:06 | |
| mriedem | unrelated, but this was also confusing me https://github.com/openstack/nova/blob/1b785e5f0c8533b49590404d1463af80b8a1ba00/nova/api/openstack/compute/server_tags.py#L107 | 03:07 |
| mriedem | since it's an update | 03:07 |
| *** esberglu has joined #openstack-nova | 03:07 | |
| mriedem | but i guess that's a create of a new tag | 03:07 |
| mriedem | since tags don't have key=value | 03:07 |
| *** siva_krish has joined #openstack-nova | 03:07 | |
| mriedem | and we have to do that validation manually since we don't have the jsonschema validation on query parameters yet - once that's in it will clean that code up quite a bit | 03:08 |
| mriedem | maybe we should have done server-tags first instead of keypairs :) | 03:08 |
| gmann | mriedem: yes, we limit with total tags updated/created | 03:08 |
| gmann | mriedem: yea, i started to move those into json schema which should be movable, might push in this week | 03:09 |
| *** tbachman has joined #openstack-nova | 03:09 | |
| gmann | there are lot of validation code should go in schema side | 03:09 |
| mriedem | melwitt: i think that patch has a bug, it assumes the connection_info has a stashed connector | 03:10 |
| mriedem | melwitt: instances with bdms before liberty/mitaka won't have that | 03:10 |
| mriedem | so KeyError city | 03:10 |
| *** ijw has joined #openstack-nova | 03:10 | |
| mriedem | gmann: you tempest guys are all schema crazy | 03:10 |
| mriedem | :) | 03:10 |
| mriedem | melwitt: i'll comment | 03:11 |
| *** esberglu has quit IRC | 03:11 | |
| mriedem | gmann: btw i think i infected cinder and keystone with the request jsonschema validation bug, they want to do it now too, bknudson was looking for some help from oomichi i think on how the initial request schemas were created | 03:12 |
| mriedem | i told bknudson to talk to oomichi | 03:12 |
| *** _ducttape_ has quit IRC | 03:14 | |
| mriedem | melwitt: -1 on https://review.openstack.org/#/c/387226/ | 03:16 |
| mriedem | we should also sucker johnthetubaguy into looking at that when he's back | 03:16 |
| mriedem | and with that i'm done for the day | 03:19 |
| *** mriedem is now known as mriedem_afk | 03:19 | |
| stevemar | mriedem_afk: we've had json schemas for the request payloads for a while now | 03:25 |
| stevemar | mriedem_afk: we were just missing the one for token validation, which bknudson just did | 03:25 |
| *** weshay has quit IRC | 03:29 | |
| openstackgerrit | int32bit proposed openstack/python-novaclient: Add min-disk and min-ram filters to list flavors https://review.openstack.org/407855 | 03:30 |
| *** weshay has joined #openstack-nova | 03:33 | |
| openstackgerrit | ChangBo Guo(gcb) proposed openstack/nova: Port api.openstack.compute.test_disk_config to Python 3 https://review.openstack.org/403536 | 03:35 |
| openstackgerrit | ChangBo Guo(gcb) proposed openstack/nova: Port network.test_manager.LdapDNSTestCase to Python 3 https://review.openstack.org/406425 | 03:35 |
| openstackgerrit | ChangBo Guo(gcb) proposed openstack/nova: Port test_simple_tenant_usage to Python 3 https://review.openstack.org/406103 | 03:35 |
| openstackgerrit | ChangBo Guo(gcb) proposed openstack/nova: Port security group related tests to Python 3 https://review.openstack.org/401241 | 03:35 |
| openstackgerrit | ChangBo Guo(gcb) proposed openstack/nova: Port compute.test_user_data.ServersControllerCreateTest to Python 3 https://review.openstack.org/406435 | 03:35 |
| openstackgerrit | ChangBo Guo(gcb) proposed openstack/nova: Port test_matchers.TestDictMatches.test__str__ to Python 3 https://review.openstack.org/406437 | 03:35 |
| openstackgerrit | ChangBo Guo(gcb) proposed openstack/nova: Port compute unit tests to Python 3 https://review.openstack.org/402010 | 03:35 |
| *** tbachman has quit IRC | 03:36 | |
| *** Jeffrey4l has quit IRC | 03:38 | |
| *** Jeffrey4l has joined #openstack-nova | 03:38 | |
| *** Jeffrey4l has quit IRC | 03:42 | |
| *** Jeffrey4l has joined #openstack-nova | 03:43 | |
| *** mdnadeem has quit IRC | 03:43 | |
| *** thorst_ has joined #openstack-nova | 03:43 | |
| *** kaisers_ has joined #openstack-nova | 03:46 | |
| *** jdurgin1 has quit IRC | 03:49 | |
| *** thorst_ has quit IRC | 03:50 | |
| *** psachin has joined #openstack-nova | 03:54 | |
| gmann | mriedem_afk: cool. it will be nice and we can have their response schema in tempest | 03:55 |
| *** Jeffrey4l has quit IRC | 03:56 | |
| *** Jeffrey4l has joined #openstack-nova | 03:56 | |
| mriedem_afk | stevemar: all lies | 03:56 |
| *** liverpooler has quit IRC | 03:57 | |
| *** mingyu has quit IRC | 03:59 | |
| *** Jeffrey4l has quit IRC | 04:00 | |
| *** Jeffrey4l has joined #openstack-nova | 04:01 | |
| *** yamahata has quit IRC | 04:02 | |
| *** bkopilov has joined #openstack-nova | 04:03 | |
| *** krtaylor has quit IRC | 04:04 | |
| *** hieulq has joined #openstack-nova | 04:05 | |
| *** Jeffrey4l has quit IRC | 04:06 | |
| *** Jeffrey4l has joined #openstack-nova | 04:06 | |
| *** Jeffrey4l has quit IRC | 04:09 | |
| *** Jeffrey4l has joined #openstack-nova | 04:09 | |
| *** zhangjl has quit IRC | 04:12 | |
| *** zhangjl has joined #openstack-nova | 04:12 | |
| *** alexpilo_ has quit IRC | 04:14 | |
| *** alexpilotti has joined #openstack-nova | 04:15 | |
| *** ducttape_ has joined #openstack-nova | 04:15 | |
| *** zhangjl has quit IRC | 04:17 | |
| *** dave-mccowan has quit IRC | 04:19 | |
| *** ducttape_ has quit IRC | 04:20 | |
| *** dimtruck is now known as zz_dimtruck | 04:23 | |
| *** Jeffrey4l has quit IRC | 04:24 | |
| *** Jeffrey4l has joined #openstack-nova | 04:24 | |
| *** pbandark has joined #openstack-nova | 04:24 | |
| *** Jeffrey4l has quit IRC | 04:25 | |
| *** Cristina_ has joined #openstack-nova | 04:26 | |
| *** tuanluong has quit IRC | 04:27 | |
| *** Jeffrey4l has joined #openstack-nova | 04:27 | |
| *** Cristina__ has quit IRC | 04:30 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova: Add functional tests for cold migration with target https://review.openstack.org/368557 | 04:34 |
| *** ayoung has quit IRC | 04:34 | |
| *** ratailor has joined #openstack-nova | 04:37 | |
| *** harlowja has joined #openstack-nova | 04:37 | |
| *** hfu has quit IRC | 04:37 | |
| *** GB21 has joined #openstack-nova | 04:37 | |
| *** hfu has joined #openstack-nova | 04:38 | |
| *** krtaylor has joined #openstack-nova | 04:38 | |
| *** shangxdy has joined #openstack-nova | 04:43 | |
| *** mdnadeem has joined #openstack-nova | 04:47 | |
| *** nicolasbock has quit IRC | 04:48 | |
| *** gjayavelu has joined #openstack-nova | 04:52 | |
| stevemar | mriedem_afk: mostly lies | 04:56 |
| *** gjayavelu has quit IRC | 05:00 | |
| *** kaisers_ has quit IRC | 05:01 | |
| *** kaisers_ has joined #openstack-nova | 05:01 | |
| openstackgerrit | Takashi NATSUME proposed openstack/python-novaclient: Microversion 2.40 - Enable cold migration woth target host https://review.openstack.org/406707 | 05:02 |
| *** sudipto has joined #openstack-nova | 05:04 | |
| *** sudipto_ has joined #openstack-nova | 05:04 | |
| *** links has joined #openstack-nova | 05:05 | |
| melwitt | mriedem_afk: cool, thanks for taking a look | 05:07 |
| *** unicell1 has joined #openstack-nova | 05:08 | |
| *** unicell has quit IRC | 05:08 | |
| *** sridharg has joined #openstack-nova | 05:09 | |
| *** fragatina has joined #openstack-nova | 05:14 | |
| *** fragatina has quit IRC | 05:15 | |
| *** fragatina has joined #openstack-nova | 05:15 | |
| *** nandal has joined #openstack-nova | 05:15 | |
| *** shangxdy has quit IRC | 05:16 | |
| *** tlian has quit IRC | 05:17 | |
| *** narayrak has joined #openstack-nova | 05:17 | |
| *** prateek has joined #openstack-nova | 05:19 | |
| *** yamahata has joined #openstack-nova | 05:24 | |
| alex_xu | Kevin_Zheng: mriedem_afk gmann I reply the comment in https://review.openstack.org/#/c/394321/ | 05:24 |
| *** Sukhdev has joined #openstack-nova | 05:26 | |
| *** int32bit has joined #openstack-nova | 05:30 | |
| *** shangxdy has joined #openstack-nova | 05:41 | |
| *** jwcroppe has quit IRC | 05:43 | |
| openstackgerrit | ChangBo Guo(gcb) proposed openstack/nova: Enable test_bdm.BlockDeviceMappingEc2CloudTestCase on Python 3 https://review.openstack.org/408415 | 05:45 |
| *** ducttape_ has joined #openstack-nova | 05:46 | |
| *** harlowja has quit IRC | 05:46 | |
| *** int32bit has quit IRC | 05:46 | |
| *** GB21 has quit IRC | 05:49 | |
| *** ducttape_ has quit IRC | 05:50 | |
| *** thorst_ has joined #openstack-nova | 05:52 | |
| *** Guest78677 has quit IRC | 05:52 | |
| *** ijw has quit IRC | 05:53 | |
| *** liuyulong_ has quit IRC | 05:55 | |
| *** moshele has joined #openstack-nova | 05:55 | |
| *** tovin07 has joined #openstack-nova | 05:59 | |
| *** thorst_ has quit IRC | 05:59 | |
| *** jaosorior has joined #openstack-nova | 06:05 | |
| *** GB21 has joined #openstack-nova | 06:09 | |
| *** moshele has quit IRC | 06:10 | |
| *** fragatina has quit IRC | 06:13 | |
| *** irenab has joined #openstack-nova | 06:14 | |
| *** gouthamr has joined #openstack-nova | 06:16 | |
| *** sahid has joined #openstack-nova | 06:17 | |
| *** irenab has quit IRC | 06:18 | |
| *** irenab has joined #openstack-nova | 06:19 | |
| *** alex_xu has quit IRC | 06:20 | |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova: Move tags validation code to json scehme https://review.openstack.org/408424 | 06:21 |
| *** alex_xu has joined #openstack-nova | 06:24 | |
| gmann | alex_xu: thanks, | 06:25 |
| gmann | alex_xu: i just pushed the schema patch - https://review.openstack.org/#/c/408424/1 | 06:25 |
| gmann | alex_xu: with that we can just use schema in 2.40 without any extension | 06:26 |
| gmann | alaski: please check if i did not miss anything | 06:26 |
| *** fragatina has joined #openstack-nova | 06:26 | |
| gmann | Kevin_Zheng: mriedem_afk ^^ | 06:26 |
| *** mingyu has joined #openstack-nova | 06:28 | |
| *** markus_z has joined #openstack-nova | 06:31 | |
| *** lpetrut has joined #openstack-nova | 06:32 | |
| *** ijw has joined #openstack-nova | 06:32 | |
| *** voelzmo has joined #openstack-nova | 06:37 | |
| *** voelzmo has quit IRC | 06:38 | |
| *** voelzmo has joined #openstack-nova | 06:38 | |
| *** Cristina__ has joined #openstack-nova | 06:38 | |
| *** mjura has joined #openstack-nova | 06:39 | |
| *** oanson has joined #openstack-nova | 06:40 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: Support tag instances when boot https://review.openstack.org/394321 | 06:40 |
| openstackgerrit | Maho Koshiya proposed openstack/nova: Add create image functional negative tests https://review.openstack.org/408433 | 06:40 |
| *** Cristina_ has quit IRC | 06:41 | |
| *** esberglu has joined #openstack-nova | 06:42 | |
| *** fragatina has quit IRC | 06:44 | |
| *** paul-carlton2 has joined #openstack-nova | 06:45 | |
| *** mjura has quit IRC | 06:45 | |
| *** hferenc has quit IRC | 06:46 | |
| gmann | Kevin_Zheng: just reply, please check - https://review.openstack.org/#/c/408424/1 | 06:47 |
| *** esberglu has quit IRC | 06:47 | |
| openstackgerrit | Pavel Kholkin proposed openstack/python-novaclient: remove variable '_' from tests https://review.openstack.org/408295 | 06:47 |
| *** jaosorior has quit IRC | 06:48 | |
| openstackgerrit | Pavel Kholkin proposed openstack/python-novaclient: remove variable '_' from tests https://review.openstack.org/408295 | 06:50 |
| *** irenab has quit IRC | 06:50 | |
| *** sacharya has quit IRC | 06:51 | |
| *** sacharya has joined #openstack-nova | 06:52 | |
| openstackgerrit | Alex Xu proposed openstack/nova-specs: Add whitelist for filter and sort query parameters for server list API https://review.openstack.org/393205 | 06:56 |
| *** sacharya has quit IRC | 06:57 | |
| *** voelzmo has quit IRC | 06:57 | |
| *** thorst_ has joined #openstack-nova | 06:58 | |
| *** kristian__ has joined #openstack-nova | 06:58 | |
| openstackgerrit | Alex Xu proposed openstack/nova-specs: Add whitelist for filter and sort query parameters for server list API https://review.openstack.org/393205 | 06:58 |
| openstackgerrit | Pawel Koniszewski proposed openstack/nova: Mark live_migration_downtime_steps as deprecated for removal https://review.openstack.org/408002 | 06:58 |
| *** mjura has joined #openstack-nova | 06:59 | |
| *** nkrinner_afk is now known as nkrinner | 07:00 | |
| *** irenab has joined #openstack-nova | 07:01 | |
| *** thorst_ has quit IRC | 07:05 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova-specs: Add whitelist for filter and sort query parameters for server list API https://review.openstack.org/393205 | 07:06 |
| *** irenab has quit IRC | 07:10 | |
| *** moshele has joined #openstack-nova | 07:11 | |
| *** kristia__ has joined #openstack-nova | 07:12 | |
| *** shangxdy has quit IRC | 07:13 | |
| *** lpetrut has quit IRC | 07:13 | |
| *** mingyu has quit IRC | 07:14 | |
| *** kristian__ has quit IRC | 07:14 | |
| *** ducttape_ has joined #openstack-nova | 07:16 | |
| *** avolkov has quit IRC | 07:17 | |
| *** hfu has quit IRC | 07:17 | |
| *** edand has joined #openstack-nova | 07:17 | |
| *** irenab has joined #openstack-nova | 07:18 | |
| *** mingyu has joined #openstack-nova | 07:19 | |
| *** bhagyashris has quit IRC | 07:21 | |
| *** ducttape_ has quit IRC | 07:21 | |
| *** Sukhdev has quit IRC | 07:24 | |
| *** cfriesen has quit IRC | 07:25 | |
| *** rcernin has joined #openstack-nova | 07:26 | |
| *** gouthamr_ has joined #openstack-nova | 07:28 | |
| *** magic has joined #openstack-nova | 07:29 | |
| *** gouthamr has quit IRC | 07:29 | |
| *** lpetrut has joined #openstack-nova | 07:30 | |
| *** markus_z has quit IRC | 07:31 | |
| *** irenab has quit IRC | 07:31 | |
| *** xiaoyang has quit IRC | 07:32 | |
| *** xiaoyang has joined #openstack-nova | 07:32 | |
| *** andreas_s has joined #openstack-nova | 07:33 | |
| *** irenab has joined #openstack-nova | 07:33 | |
| *** magic has quit IRC | 07:33 | |
| *** zhangjl has joined #openstack-nova | 07:36 | |
| *** markus_z has joined #openstack-nova | 07:38 | |
| *** GB21 has quit IRC | 07:40 | |
| *** ijw has quit IRC | 07:40 | |
| *** esberglu has joined #openstack-nova | 07:42 | |
| *** WillUK2010 has quit IRC | 07:44 | |
| *** ratailor has quit IRC | 07:45 | |
| *** kristia__ has quit IRC | 07:46 | |
| *** esberglu has quit IRC | 07:46 | |
| *** kristian__ has joined #openstack-nova | 07:47 | |
| *** kristian__ has quit IRC | 07:48 | |
| openstackgerrit | Alex Xu proposed openstack/nova-specs: Add whitelist for filter and sort query parameters for server list API https://review.openstack.org/393205 | 07:48 |
| *** mingyu has quit IRC | 07:48 | |
| *** kristian__ has joined #openstack-nova | 07:48 | |
| *** nherciu has joined #openstack-nova | 07:48 | |
| *** irenab has quit IRC | 07:48 | |
| *** kristian__ has quit IRC | 07:49 | |
| alex_xu | Kevin_Zheng: ^ just update the working items, that is the way we can separate patch to avoid a huge patch | 07:50 |
| Kevin_Zheng | alex_xu: Hmm, looks like 5 patches | 07:51 |
| alex_xu | Kevin_Zheng: yea | 07:52 |
| *** GB21 has joined #openstack-nova | 07:52 | |
| *** sacharya has joined #openstack-nova | 07:53 | |
| *** irenab has joined #openstack-nova | 07:53 | |
| *** gszasz has joined #openstack-nova | 07:54 | |
| *** ratailor has joined #openstack-nova | 07:58 | |
| *** sacharya has quit IRC | 07:58 | |
| *** ihrachys has joined #openstack-nova | 07:58 | |
| *** kristian__ has joined #openstack-nova | 07:59 | |
| *** ihrachys has quit IRC | 08:00 | |
| *** xiaoyang has quit IRC | 08:00 | |
| *** sacharya has joined #openstack-nova | 08:00 | |
| *** ihrachys has joined #openstack-nova | 08:00 | |
| *** lpetrut has quit IRC | 08:02 | |
| *** thorst_ has joined #openstack-nova | 08:03 | |
| *** matrohon has joined #openstack-nova | 08:04 | |
| *** sacharya has quit IRC | 08:06 | |
| *** zenoway has joined #openstack-nova | 08:08 | |
| *** gouthamr_ has quit IRC | 08:08 | |
| *** andreas_s has quit IRC | 08:09 | |
| *** thorst_ has quit IRC | 08:10 | |
| *** tommy_ has joined #openstack-nova | 08:11 | |
| *** tpatzig_ has joined #openstack-nova | 08:11 | |
| *** tommy_ is now known as Guest61919 | 08:12 | |
| *** rmart04 has joined #openstack-nova | 08:13 | |
| *** Guest61919 has quit IRC | 08:13 | |
| *** tpatzig_ has quit IRC | 08:13 | |
| *** owalsh has joined #openstack-nova | 08:15 | |
| *** kristia__ has joined #openstack-nova | 08:15 | |
| *** TravT has quit IRC | 08:16 | |
| *** lpetrut has joined #openstack-nova | 08:17 | |
| *** ralonsoh has joined #openstack-nova | 08:18 | |
| *** kristian__ has quit IRC | 08:18 | |
| *** mjura has quit IRC | 08:19 | |
| *** stvnoyes has quit IRC | 08:19 | |
| *** stvnoyes has joined #openstack-nova | 08:20 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: Should not allow duplicate device name when provide bdm for create https://review.openstack.org/408475 | 08:20 |
| *** mingyu has joined #openstack-nova | 08:20 | |
| *** nkrinner has quit IRC | 08:20 | |
| *** nkrinner has joined #openstack-nova | 08:22 | |
| *** andreas_s has joined #openstack-nova | 08:23 | |
| *** kristia__ has quit IRC | 08:28 | |
| *** lpetrut has quit IRC | 08:28 | |
| *** nkrinner has quit IRC | 08:28 | |
| *** kristian__ has joined #openstack-nova | 08:29 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: Support tag instances when boot https://review.openstack.org/394321 | 08:30 |
| *** lpetrut has joined #openstack-nova | 08:31 | |
| *** avolkov has joined #openstack-nova | 08:31 | |
| *** jamielennox is now known as jamielennox|away | 08:33 | |
| *** gouthamr has joined #openstack-nova | 08:33 | |
| *** kristian__ has quit IRC | 08:39 | |
| *** kristian__ has joined #openstack-nova | 08:39 | |
| *** kristian__ has quit IRC | 08:40 | |
| *** mingyu has quit IRC | 08:40 | |
| *** kristian__ has joined #openstack-nova | 08:40 | |
| *** kristian__ has quit IRC | 08:41 | |
| *** lpetrut has quit IRC | 08:41 | |
| *** kristian__ has joined #openstack-nova | 08:41 | |
| *** irenab has quit IRC | 08:41 | |
| *** lpetrut has joined #openstack-nova | 08:41 | |
| *** nkrinner has joined #openstack-nova | 08:42 | |
| *** mingyu has joined #openstack-nova | 08:43 | |
| *** kristia__ has joined #openstack-nova | 08:43 | |
| *** kristian__ has quit IRC | 08:44 | |
| *** kristi___ has joined #openstack-nova | 08:44 | |
| *** mkoderer has joined #openstack-nova | 08:45 | |
| *** lpetrut has quit IRC | 08:46 | |
| *** ducttape_ has joined #openstack-nova | 08:47 | |
| *** kristia__ has quit IRC | 08:48 | |
| *** irenab has joined #openstack-nova | 08:49 | |
| openstackgerrit | YuYang proposed openstack/nova: VMware:fix get vm'root deivce conditional verify https://review.openstack.org/408490 | 08:50 |
| *** ducttape_ has quit IRC | 08:51 | |
| *** danpawlik has joined #openstack-nova | 08:54 | |
| danpawlik | Hello, pls review https://review.openstack.org/#/c/389216/ | 08:54 |
| *** pcaruana has joined #openstack-nova | 08:58 | |
| *** zzzeek has quit IRC | 09:00 | |
| *** zzzeek has joined #openstack-nova | 09:00 | |
| *** kristi___ has quit IRC | 09:03 | |
| openstackgerrit | ChangBo Guo(gcb) proposed openstack/nova: Set access_policy for messaging's dispatcher https://review.openstack.org/408494 | 09:07 |
| *** thorst_ has joined #openstack-nova | 09:08 | |
| *** zhangjl has quit IRC | 09:10 | |
| *** zhangjl has joined #openstack-nova | 09:11 | |
| *** irenab has quit IRC | 09:11 | |
| *** gouthamr has quit IRC | 09:11 | |
| *** yassine has joined #openstack-nova | 09:12 | |
| *** yassine is now known as Guest34488 | 09:13 | |
| *** irenab has joined #openstack-nova | 09:13 | |
| *** zhangjl has quit IRC | 09:13 | |
| *** roeyc has joined #openstack-nova | 09:14 | |
| *** roeyc has quit IRC | 09:14 | |
| *** roeyc has joined #openstack-nova | 09:15 | |
| *** thorst_ has quit IRC | 09:15 | |
| *** Cristina_ has joined #openstack-nova | 09:15 | |
| *** nandal_in has joined #openstack-nova | 09:15 | |
| *** kristian__ has joined #openstack-nova | 09:16 | |
| *** kristian__ has quit IRC | 09:16 | |
| *** kristian__ has joined #openstack-nova | 09:16 | |
| *** Cristina__ has quit IRC | 09:19 | |
| *** nandal has quit IRC | 09:19 | |
| *** kristian__ has quit IRC | 09:21 | |
| kashyap | mriedem_afk: (When you're back.) Just reading the scrollback | 09:23 |
| kashyap | Where do you see the -9 exit code? | 09:24 |
| lyarwood | kashyap: are you talking about https://bugs.launchpad.net/nova/+bug/1646181 by any chance? | 09:25 |
| openstack | Launchpad bug 1646181 in OpenStack Compute (nova) "NFS: Fail to boot VM out of large snapshots (30GB+)" [Undecided,Incomplete] | 09:25 |
| kashyap | lyarwood: Sounds so, /me clicks | 09:25 |
| kashyap | Thanks for the pointer | 09:25 |
| *** irenab has quit IRC | 09:26 | |
| *** derekh has joined #openstack-nova | 09:26 | |
| *** kristian__ has joined #openstack-nova | 09:27 | |
| openstackgerrit | Matthew Booth proposed openstack/nova: libvirt: Fix initialising of LVM ephemeral disks https://review.openstack.org/408152 | 09:29 |
| *** irenab has joined #openstack-nova | 09:29 | |
| *** efoley has joined #openstack-nova | 09:30 | |
| openstackgerrit | int32bit proposed openstack/python-novaclient: Fix can't show other tenant's server error https://review.openstack.org/397815 | 09:30 |
| openstackgerrit | Matthew Booth proposed openstack/nova: libvirt: Add create_from_image and create_from_func to Backend https://review.openstack.org/333244 | 09:31 |
| openstackgerrit | Li Yingjun proposed openstack/nova: Remove extra ^M for json file https://review.openstack.org/403989 | 09:31 |
| *** gszasz has quit IRC | 09:32 | |
| *** esberglu has joined #openstack-nova | 09:32 | |
| *** kristia__ has joined #openstack-nova | 09:33 | |
| openstackgerrit | Matthew Booth proposed openstack/nova: libvirt: Pass is_block_dev explicitly from create_from_func https://review.openstack.org/333263 | 09:33 |
| sfinucan | BobBall: Sounds good. I'll have a look for that mail, mriedem_afk | 09:34 |
| *** lucas-afk is now known as lucasagomes | 09:36 | |
| *** esberglu has quit IRC | 09:37 | |
| *** kristian__ has quit IRC | 09:37 | |
| *** mvk has quit IRC | 09:39 | |
| *** sapcc-bot has quit IRC | 09:40 | |
| *** carthaca_ has quit IRC | 09:40 | |
| *** carthaca_ has joined #openstack-nova | 09:40 | |
| *** sapcc-bot has joined #openstack-nova | 09:40 | |
| *** efoley_ has joined #openstack-nova | 09:40 | |
| *** mkoderer_ has joined #openstack-nova | 09:40 | |
| *** tpatzig_ has joined #openstack-nova | 09:40 | |
| *** dgonzalez_ has joined #openstack-nova | 09:40 | |
| *** tommy_ has joined #openstack-nova | 09:40 | |
| *** databus23_ has joined #openstack-nova | 09:40 | |
| *** tommy_ is now known as Guest55964 | 09:41 | |
| *** efoley has quit IRC | 09:41 | |
| *** kristia__ has quit IRC | 09:41 | |
| *** efoley has joined #openstack-nova | 09:41 | |
| *** jaosorior has joined #openstack-nova | 09:41 | |
| *** mkoderer_ has quit IRC | 09:42 | |
| *** tpatzig_ has quit IRC | 09:42 | |
| *** dgonzalez_ has quit IRC | 09:42 | |
| *** Guest55964 has quit IRC | 09:42 | |
| *** databus23_ has quit IRC | 09:42 | |
| *** kristian__ has joined #openstack-nova | 09:42 | |
| *** mingyu has quit IRC | 09:42 | |
| takashin | johnthetubaguy: Could you check my comment in https://review.openstack.org/#/c/349863/ ? | 09:42 |
| openstackgerrit | Matthew Booth proposed openstack/nova: libvirt: Refactor _create_image into storagepolicy https://review.openstack.org/356651 | 09:43 |
| *** efoley_ has quit IRC | 09:45 | |
| openstackgerrit | Matthew Booth proposed openstack/nova: libvirt: don't test cache() in test_driver https://review.openstack.org/406869 | 09:45 |
| sfinucan | takashin: He's on his honeymoon, so he won't see that :) | 09:45 |
| *** hogepodge has quit IRC | 09:45 | |
| *** gszasz has joined #openstack-nova | 09:46 | |
| takashin | sfinucan: Honeymoon? Thank you. | 09:46 |
| *** hogepodge has joined #openstack-nova | 09:46 | |
| *** kristian__ has quit IRC | 09:46 | |
| openstackgerrit | Matthew Booth proposed openstack/nova: libvirt: Introduce ImageCacheLocalDir https://review.openstack.org/279669 | 09:47 |
| *** roeyc has left #openstack-nova | 09:47 | |
| openstackgerrit | YuYang proposed openstack/nova: VMware:fix get vm'root deivce conditional verify https://review.openstack.org/402256 | 09:49 |
| *** ftersin has quit IRC | 09:49 | |
| openstackgerrit | Matthew Booth proposed openstack/nova: libvirt: Resize an existing disk https://review.openstack.org/406870 | 09:49 |
| *** irenab_ has joined #openstack-nova | 09:49 | |
| *** irenab has quit IRC | 09:49 | |
| *** irenab_ is now known as irenab | 09:49 | |
| *** roeyc has joined #openstack-nova | 09:50 | |
| openstackgerrit | Stephen Finucane proposed openstack/nova: Don't use 'updated_at' to check service's status https://review.openstack.org/297085 | 09:51 |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: Should not allow duplicate device name when provide bdm for create https://review.openstack.org/408475 | 09:51 |
| openstackgerrit | Matthew Booth proposed openstack/nova: libvirt: Add ensure_backing_from_* https://review.openstack.org/406871 | 09:51 |
| *** ftersin has joined #openstack-nova | 09:51 | |
| *** kevinz has quit IRC | 09:52 | |
| *** irenab has quit IRC | 09:53 | |
| openstackgerrit | Matthew Booth proposed openstack/nova: libvirt: Don't call _create_image from finish_migration https://review.openstack.org/337160 | 09:53 |
| openstackgerrit | Matthew Booth proposed openstack/nova: libvirt: Remove _create_image from driver https://review.openstack.org/406872 | 09:58 |
| sfinucan | F25 packages older versions of Python (3.3, 3.4) too now? No more installing dodgy coprs so I can run tests | 09:58 |
| *** tpatzig_ has joined #openstack-nova | 09:58 | |
| mdbooth | sfinucan: It does? | 09:58 |
| *** databus23_ has joined #openstack-nova | 09:58 | |
| *** david_1 has joined #openstack-nova | 09:58 | |
| *** dgonzalez_ has joined #openstack-nova | 09:58 | |
| *** mkoderer_ has joined #openstack-nova | 09:58 | |
| sfinucan | mdbooth: Aye - python3.3, 3.4 and 3.5 are all ready and waiting now | 09:59 |
| sfinucan | Should have read the release notes, I guess https://fedoramagazine.org/fedora-25-released/ | 10:00 |
| mdbooth | I'll bear that in mind should python3 ever become important | 10:00 |
| *** liusheng has quit IRC | 10:00 | |
| *** tpatzig_ has quit IRC | 10:00 | |
| *** databus23_ has quit IRC | 10:00 | |
| *** dgonzalez_ has quit IRC | 10:00 | |
| *** mkoderer_ has quit IRC | 10:00 | |
| *** david_1 has quit IRC | 10:00 | |
| slunkad | does anybody know if there is a plan to handle out of tree drivers in nova after the use for import_object_ns was removed? | 10:01 |
| *** liusheng has joined #openstack-nova | 10:01 | |
| openstackgerrit | Matthew Booth proposed openstack/nova: libvirt: Replace _create_images_and_backing in _hard_reboot https://review.openstack.org/337789 | 10:01 |
| *** carthaca_ has quit IRC | 10:02 | |
| *** carthaca_ has joined #openstack-nova | 10:02 | |
| * mdbooth is currently cursing https://review.openstack.org/#/c/400697/ | 10:02 | |
| mdbooth | What a lot of irritating noise! | 10:02 |
| *** owalsh has quit IRC | 10:02 | |
| mdbooth | slunkad: There is no plan to do anything with out of tree drivers. | 10:05 |
| *** efoley_ has joined #openstack-nova | 10:05 | |
| mdbooth | slunkad: In general, Nova doesn't consider out of tree drivers when planning. | 10:06 |
| *** Zer0Byte__ has joined #openstack-nova | 10:06 | |
| *** liusheng has quit IRC | 10:06 | |
| *** ihrachys has quit IRC | 10:06 | |
| openstackgerrit | Matthew Booth proposed openstack/nova: libvirt: Replace _create_images_and_backing in pre_live_migration https://review.openstack.org/342224 | 10:07 |
| *** irenab has joined #openstack-nova | 10:07 | |
| *** liusheng has joined #openstack-nova | 10:07 | |
| slunkad | mdbooth: ok seems like a choice they have made! thanks | 10:07 |
| *** yamahata has quit IRC | 10:08 | |
| *** efoley has quit IRC | 10:09 | |
| *** bks has joined #openstack-nova | 10:10 | |
| openstackgerrit | Matthew Booth proposed openstack/nova: libvirt: Add create_from_image & create_from_func for Qcow2 https://review.openstack.org/408543 | 10:10 |
| *** mvk has joined #openstack-nova | 10:11 | |
| *** GB21 has quit IRC | 10:11 | |
| *** bhagyashris has joined #openstack-nova | 10:12 | |
| *** irenab_ has joined #openstack-nova | 10:12 | |
| *** ihrachys has joined #openstack-nova | 10:13 | |
| *** thorst_ has joined #openstack-nova | 10:13 | |
| *** irenab has quit IRC | 10:14 | |
| *** irenab_ is now known as irenab | 10:14 | |
| *** gouthamr has joined #openstack-nova | 10:14 | |
| *** sandanar has joined #openstack-nova | 10:15 | |
| *** ducttape_ has joined #openstack-nova | 10:17 | |
| *** ducttape_ has quit IRC | 10:19 | |
| *** ducttape_ has joined #openstack-nova | 10:19 | |
| *** thorst_ has quit IRC | 10:20 | |
| sfinucan | gcb: Went through most of your py3 patches. Mostly +2d, though I had a couple of "couldn't we use six for this?" moments that I'm still looking into | 10:22 |
| *** gouthamr_ has joined #openstack-nova | 10:23 | |
| *** gouthamr has quit IRC | 10:25 | |
| *** karts has quit IRC | 10:27 | |
| openstackgerrit | jichenjc proposed openstack/nova: pi-ref: Body verification for os-hypervisors.inc https://review.openstack.org/383691 | 10:27 |
| *** gouthamr has joined #openstack-nova | 10:28 | |
| *** cdent has joined #openstack-nova | 10:30 | |
| *** gouthamr_ has quit IRC | 10:30 | |
| *** karthiks has joined #openstack-nova | 10:30 | |
| *** portdirect_away_ has joined #openstack-nova | 10:30 | |
| *** ducttape_ has quit IRC | 10:31 | |
| *** portdirect_away_ is now known as portdirect | 10:31 | |
| gcb | sfinucan: thanks, I planned to begin porting functional tests to Python 3 after we done with unit tests. | 10:31 |
| *** claudiub has quit IRC | 10:33 | |
| *** lpetrut has joined #openstack-nova | 10:33 | |
| *** GB21 has joined #openstack-nova | 10:34 | |
| *** hoangcx has quit IRC | 10:36 | |
| *** Cristina__ has joined #openstack-nova | 10:39 | |
| sfinucan | Are we fully off of rootwrap now in favour of oslo.policy? | 10:40 |
| openstackgerrit | Matthew Booth proposed openstack/nova: libvirt: Add create_from_image & create_from_func for Qcow2 https://review.openstack.org/408543 | 10:40 |
| openstackgerrit | ChangBo Guo(gcb) proposed openstack/nova: Port test_virt_drivers to Python 3 https://review.openstack.org/403417 | 10:41 |
| *** Cristina_ has quit IRC | 10:42 | |
| *** kristian__ has joined #openstack-nova | 10:45 | |
| gcb | sfinucan, do you mean oslo.privsep ? | 10:47 |
| *** efoley__ has joined #openstack-nova | 10:48 | |
| *** kristian__ has quit IRC | 10:50 | |
| sahid | johnthetubaguy: on his review dansmith suggested to me to use a "loader", i have no idea of what that should be, any ideas ? https://review.openstack.org/#/c/408240/1/nova/tests/unit/virt/test_hardware.py@1780 | 10:50 |
| *** yufei has quit IRC | 10:51 | |
| *** efoley_ has quit IRC | 10:51 | |
| *** owalsh has joined #openstack-nova | 10:54 | |
| *** carthaca_ has quit IRC | 10:55 | |
| *** sapcc-bot has quit IRC | 10:55 | |
| sfinucan | gcb: I do, yes :) | 10:57 |
| *** edand has quit IRC | 11:00 | |
| *** links has quit IRC | 11:00 | |
| *** tbachman has joined #openstack-nova | 11:01 | |
| *** hogepodge has quit IRC | 11:01 | |
| *** ducttape_ has joined #openstack-nova | 11:02 | |
| *** GB21 has quit IRC | 11:06 | |
| gcb | sfinucan: https://github.com/openstack/nova/commit/4a8f2b0d44ee10dfac2d3d828cd9dc574d5ddbb2 introduced it into nova, but have no idea about the progress | 11:08 |
| sfinucan | gcb: Ah, there's a load of rootwrap references in the code still so I guess not. Fair enough | 11:10 |
| gcb | sfinucan, yes | 11:11 |
| openstackgerrit | ChangBo Guo(gcb) proposed openstack/nova: [WIP] Add python 3 functional tests in tox.ini https://review.openstack.org/408561 | 11:12 |
| *** Drankis has joined #openstack-nova | 11:15 | |
| *** tojuvone has quit IRC | 11:15 | |
| *** esberglu has joined #openstack-nova | 11:16 | |
| sfinucan | gcb: You might take a look at this https://review.openstack.org/#/c/385686/ | 11:16 |
| *** links has joined #openstack-nova | 11:16 | |
| sfinucan | Looks related to ^ | 11:16 |
| *** zenoway has quit IRC | 11:17 | |
| *** zenoway has joined #openstack-nova | 11:17 | |
| *** ducttape_ has quit IRC | 11:17 | |
| *** asettle has joined #openstack-nova | 11:18 | |
| *** thorst_ has joined #openstack-nova | 11:18 | |
| asettle | Hey! I have a question for the nova team - have you guys completed converted to the openstack client? | 11:19 |
| *** bhagyashris has left #openstack-nova | 11:19 | |
| gcb | sfinucan, ah, guess I also fix that in WIP patch https://review.openstack.org/#/c/408561/, will look at https://review.openstack.org/#/c/385686/ and make that merged :-) | 11:19 |
| *** Zer0Byte__ has quit IRC | 11:20 | |
| *** esberglu has quit IRC | 11:20 | |
| sfinucan | asettle: Most things you can do in novaclient can be done in openstackclient, if that's what you mean? | 11:20 |
| *** Jeffrey4l has quit IRC | 11:20 | |
| sfinucan | but afaik, novaclient isn't going away | 11:20 |
| kashyap | mriedem_afk: That -9 is SIGKILL. Actually, for SIGKILL it's just '9', but the dash is mangled by Python's subprocess module. I'll add a comment in the bug. | 11:21 |
| asettle | Yes, that's what I thought too sfinucan - I'm reviewing a patch that reverts the use of 'openstack flavor list' back to nova flavor-list and I was fairly sure the openstack one was correct in context? | 11:21 |
| *** hogepodge has joined #openstack-nova | 11:21 | |
| openstackgerrit | sahid proposed openstack/nova: objects: remove related pining from __init__ of InstanceNUMATopology https://review.openstack.org/408242 | 11:21 |
| openstackgerrit | sahid proposed openstack/nova: objects: remove pagesize from __init__ of InstanceNUMATopology https://review.openstack.org/408241 | 11:21 |
| openstackgerrit | sahid proposed openstack/nova: objects: remove cpu_topology from __init__ of InstanceNUMATopology https://review.openstack.org/408240 | 11:21 |
| *** zenoway has quit IRC | 11:22 | |
| sfinucan | asettle: Yes, in my experience they're equivalent. Pretty sure I submitted fixes to openstack-manuals doing the opposite (novaclient -> openstackclient) :) | 11:23 |
| asettle | sfinucan: yes I thought you did too. But we just received this, could you take a look? https://review.openstack.org/#/c/408402/2/doc/cli-reference/source/nova.rst | 11:23 |
| asettle | I mean, it doesn't matter too much, because as Kato notes, it's an auto gen file so this change is wrong. | 11:24 |
| asettle | But I just wanted to check that I wasn't going crazy. | 11:24 |
| *** lamt has quit IRC | 11:24 | |
| *** kristian__ has joined #openstack-nova | 11:25 | |
| *** thorst_ has quit IRC | 11:25 | |
| openstackgerrit | Alex Xu proposed openstack/nova: Use JSON-Schema to validate query parameters for keypairs API https://review.openstack.org/398755 | 11:25 |
| openstackgerrit | Alex Xu proposed openstack/nova: Adds support for versioned schema validation for query parameters https://review.openstack.org/389003 | 11:25 |
| openstackgerrit | Alex Xu proposed openstack/nova: Add query parameter schema for server list/detail in microversion 2.1 https://review.openstack.org/408571 | 11:25 |
| mdbooth | Oh, FFS | 11:26 |
| mdbooth | Unit test failures in python3 | 11:26 |
| sfinucan | asettle: Nope, you're correct. That's his opinion, but 'openstack flavor list' is still valid | 11:26 |
| sfinucan | Here's the code on 'master' https://github.com/openstack/python-openstackclient/blob/master/openstackclient/compute/v2/flavor.py#L221-L302 | 11:26 |
| asettle | sfinucan: thanks :) good to know! | 11:26 |
| *** hogepodge has quit IRC | 11:26 | |
| mdbooth | sfinucan: I guess I'll have to install python3 after all :( | 11:28 |
| *** ducttape_ has joined #openstack-nova | 11:28 | |
| sfinucan | mdbooth: Don't listen to Zed. Embrace the __future__ | 11:28 |
| sfinucan | asettle: No problem at all :) | 11:28 |
| *** kristian__ has quit IRC | 11:29 | |
| mdbooth | It's a waste of effort. | 11:29 |
| *** kristian__ has joined #openstack-nova | 11:29 | |
| sfinucan | asettle: Though there is /one/ thing I'd point out, which is that the rest of that file uses 'nova xxx' commands so perhaps for consistency. I'll say as much in the review | 11:29 |
| *** kristian__ has quit IRC | 11:30 | |
| * mdbooth would rather port Nova to a better language. | 11:30 | |
| *** kristian__ has joined #openstack-nova | 11:30 | |
| *** kristian__ has quit IRC | 11:30 | |
| asettle | sfinucan: yeah I did notice that, which made sense. Just wanted to clarify though :) | 11:30 |
| asettle | Hopefully with the next auto-gen update it'll all change. | 11:30 |
| mdbooth | One with well defined interfaces and static types. | 11:30 |
| mdbooth | Porting from one bad language to a pointlessly incompatible bad language is just busy work imho,. | 11:31 |
| sfinucan | mdbooth: If only people would move to Python 3.x https://www.python.org/dev/peps/pep-0484/ | 11:31 |
| *** rtheis has joined #openstack-nova | 11:33 | |
| sfinucan | mdbooth: I see nova as glue between various other bits of software (hypervisors, storage backends, random command line utilities, ...) | 11:33 |
| mdbooth | Driving a spike into a horse's head doesn't turn it into a unicorn. | 11:33 |
| sfinucan | IMO, there's no better glue language than Python. Bash and Perl don't scale | 11:33 |
| sfinucan | But that's totally off-topic :) | 11:34 |
| mdbooth | Bash and Perl are the wrong direction. | 11:34 |
| asettle | ... While I'm here, this is relevant to your convo and I spotted yesterday: http://leftoversalad.com/c/015_programmingpeople/ | 11:34 |
| asettle | I found it amusing. | 11:34 |
| mdbooth | Nova is too large for a glue language. | 11:34 |
| *** kristian__ has joined #openstack-nova | 11:35 | |
| mdbooth | asettle: :) | 11:36 |
| sfinucan | asettle: Heh :) | 11:36 |
| asettle | Wow that was in sync. | 11:36 |
| masber | hi | 11:37 |
| cdent | mdbooth: s/for.*$// ? | 11:37 |
| masber | I have a very strange problem with nova | 11:37 |
| masber | somehow it does not resize my instance | 11:38 |
| masber | and I can't find the reason looking at the logs | 11:38 |
| masber | the scheduler says that the filter has not returned any host | 11:38 |
| masber | but doesn't say which filter is doing that | 11:38 |
| *** hogepodge has joined #openstack-nova | 11:38 | |
| masber | at the same time i can create an instance using same flavour | 11:39 |
| masber | so the problem seems to be resizing | 11:39 |
| *** voelzmo has joined #openstack-nova | 11:39 | |
| *** tbachman has quit IRC | 11:39 | |
| masber | this is the error I see --> There are 0 hosts available but 1 instances requested to build. select_destinations /usr/lib/python2.7/site-packages/nova/scheduler/filter_scheduler.py:71 | 11:39 |
| masber | any idea what can I do next? | 11:40 |
| *** kristia__ has joined #openstack-nova | 11:41 | |
| *** prateek has quit IRC | 11:42 | |
| *** kristian__ has quit IRC | 11:42 | |
| *** psachin has quit IRC | 11:42 | |
| *** kristian__ has joined #openstack-nova | 11:43 | |
| *** Jeffrey4l has joined #openstack-nova | 11:43 | |
| *** ducttape_ has quit IRC | 11:44 | |
| openstackgerrit | sahid proposed openstack/nova: objects: remove related pining from __init__ of InstanceNUMATopology https://review.openstack.org/408242 | 11:44 |
| *** GB21 has joined #openstack-nova | 11:44 | |
| masber | this is what I can see from the logs --> http://pastebin.com/raw/FLmwgnQp | 11:45 |
| *** kristia__ has quit IRC | 11:46 | |
| masber | I also put the filter to AllHostsFilter but still same result | 11:46 |
| *** hogepodge has quit IRC | 11:48 | |
| *** moshele has quit IRC | 11:48 | |
| sfinucan | masber: Afraid I can't help you that much, but it seems the host is being ignored so it never even make it the filtering stage ('Host filter ignoring hosts: openstack-dev') | 11:48 |
| sfinucan | I don't know what would cause it to be ignored, however. If no one chimes up here you might have better luck on #openstack | 11:49 |
| *** nicolasbock has joined #openstack-nova | 11:49 | |
| masber | sfinucan, yeah, I already asked in #openstack and no luck | 11:50 |
| masber | its weird | 11:50 |
| openstackgerrit | ChangBo Guo(gcb) proposed openstack/nova: conf: Remove config option compute_ manager https://review.openstack.org/405921 | 11:50 |
| masber | so the host is being ignored even before the filtering stage? | 11:50 |
| *** sudipto_ has quit IRC | 11:51 | |
| *** sudipto has quit IRC | 11:51 | |
| *** pbandark has quit IRC | 11:53 | |
| *** kristia__ has joined #openstack-nova | 11:53 | |
| *** gouthamr has quit IRC | 11:55 | |
| *** moshele has joined #openstack-nova | 11:55 | |
| *** GB21 has quit IRC | 11:55 | |
| *** kristian__ has quit IRC | 11:57 | |
| *** sapcc-bot has joined #openstack-nova | 11:59 | |
| *** tpatzig_ has joined #openstack-nova | 11:59 | |
| *** carthaca_ has joined #openstack-nova | 11:59 | |
| *** databus23_ has joined #openstack-nova | 11:59 | |
| *** dgonzalez_ has joined #openstack-nova | 11:59 | |
| *** hogepodge has joined #openstack-nova | 12:00 | |
| *** psachin has joined #openstack-nova | 12:01 | |
| *** carthaca_ has quit IRC | 12:01 | |
| *** databus23_ has quit IRC | 12:01 | |
| *** tpatzig_ has quit IRC | 12:01 | |
| *** dgonzalez_ has quit IRC | 12:01 | |
| *** dave-mccowan has joined #openstack-nova | 12:02 | |
| openstackgerrit | ChangBo Guo(gcb) proposed openstack/nova: Set access_policy for messaging's dispatcher https://review.openstack.org/408494 | 12:02 |
| *** sacharya has joined #openstack-nova | 12:02 | |
| *** rfolco has joined #openstack-nova | 12:03 | |
| *** carthaca_ has joined #openstack-nova | 12:05 | |
| *** databus23_ has joined #openstack-nova | 12:05 | |
| *** tommy_ has joined #openstack-nova | 12:05 | |
| *** mkoderer_ has joined #openstack-nova | 12:05 | |
| *** dgonzalez_ has joined #openstack-nova | 12:05 | |
| *** tpatzig_ has joined #openstack-nova | 12:05 | |
| *** tommy_ is now known as Guest42909 | 12:05 | |
| *** voelzmo has quit IRC | 12:06 | |
| openstackgerrit | ChangBo Guo(gcb) proposed openstack/nova: conf: Remove config option compute_ manager https://review.openstack.org/405921 | 12:06 |
| *** carthaca_ has quit IRC | 12:07 | |
| *** databus23_ has quit IRC | 12:07 | |
| *** dgonzalez_ has quit IRC | 12:07 | |
| *** Guest42909 has quit IRC | 12:07 | |
| *** mkoderer_ has quit IRC | 12:07 | |
| *** tpatzig_ has quit IRC | 12:07 | |
| *** sacharya has quit IRC | 12:07 | |
| *** psachin has quit IRC | 12:08 | |
| *** pbandark has joined #openstack-nova | 12:08 | |
| *** sahid has quit IRC | 12:09 | |
| *** psachin has joined #openstack-nova | 12:10 | |
| *** kristian__ has joined #openstack-nova | 12:10 | |
| *** jaosorior has quit IRC | 12:10 | |
| *** esberglu has joined #openstack-nova | 12:10 | |
| *** dinobot_ has joined #openstack-nova | 12:11 | |
| *** kristian__ has quit IRC | 12:13 | |
| *** kristia__ has quit IRC | 12:13 | |
| *** kristian__ has joined #openstack-nova | 12:14 | |
| *** kristian__ has quit IRC | 12:14 | |
| *** esberglu has quit IRC | 12:15 | |
| *** bkopilov has quit IRC | 12:15 | |
| gibi | syjulian: hi! ping | 12:17 |
| *** shangxdy has joined #openstack-nova | 12:18 | |
| *** erlon-airlong has joined #openstack-nova | 12:18 | |
| *** bks has quit IRC | 12:18 | |
| *** bvanhav_ has joined #openstack-nova | 12:23 | |
| *** thorst_ has joined #openstack-nova | 12:23 | |
| *** kristian__ has joined #openstack-nova | 12:25 | |
| *** sdague has joined #openstack-nova | 12:26 | |
| *** ratailor has quit IRC | 12:28 | |
| *** shangxdy has quit IRC | 12:28 | |
| *** sahid has joined #openstack-nova | 12:29 | |
| *** kristia__ has joined #openstack-nova | 12:29 | |
| *** thorst_ has quit IRC | 12:30 | |
| *** lucasagomes is now known as lucas-hungry | 12:32 | |
| *** takashin has left #openstack-nova | 12:32 | |
| *** kristian__ has quit IRC | 12:33 | |
| openstackgerrit | ChangBo Guo(gcb) proposed openstack/nova: test: drop unused config option fake_manager https://review.openstack.org/408589 | 12:34 |
| *** jaosorior has joined #openstack-nova | 12:37 | |
| *** pbandark has quit IRC | 12:39 | |
| *** RuiChen has left #openstack-nova | 12:39 | |
| *** sridharg has quit IRC | 12:41 | |
| *** tbachman has joined #openstack-nova | 12:44 | |
| *** tbachman has quit IRC | 12:44 | |
| *** tojuvone has joined #openstack-nova | 12:45 | |
| *** ducttape_ has joined #openstack-nova | 12:48 | |
| *** WillUK2010 has joined #openstack-nova | 12:49 | |
| *** psachin has quit IRC | 12:50 | |
| *** thorst_ has joined #openstack-nova | 12:51 | |
| *** psachin has joined #openstack-nova | 12:52 | |
| *** links has quit IRC | 12:53 | |
| *** ducttape_ has quit IRC | 12:53 | |
| *** kristia__ has quit IRC | 12:54 | |
| openstackgerrit | ChangBo Guo(gcb) proposed openstack/nova: Port libvirt.test_vif to Python 3 https://review.openstack.org/403419 | 12:55 |
| *** kristian__ has joined #openstack-nova | 12:56 | |
| openstackgerrit | Roman Dobosz proposed openstack/nova: placement: adds REST API for nested providers https://review.openstack.org/384807 | 12:58 |
| openstackgerrit | Roman Dobosz proposed openstack/nova: placement: allow filter providers in tree https://review.openstack.org/377215 | 12:58 |
| openstackgerrit | Roman Dobosz proposed openstack/nova: placement: update client to set parent provider https://review.openstack.org/385693 | 12:58 |
| openstackgerrit | Roman Dobosz proposed openstack/nova: placement: add nested resource providers https://review.openstack.org/377138 | 12:58 |
| *** psachin has quit IRC | 12:59 | |
| *** shangxdy has joined #openstack-nova | 12:59 | |
| *** rfolco has quit IRC | 13:01 | |
| *** psachin has joined #openstack-nova | 13:03 | |
| sdague | bauzas: on https://review.openstack.org/#/c/386242/12/nova/objects/resource_provider.py - is there some description in docs somewhere about what can_host means for RP? (still catching up on things) | 13:04 |
| bauzas | sdague: well, I think cdent wrote a bit of docs about that | 13:05 |
| bauzas | sdague: I'm not sure tho that we have a documentation yet for that | 13:05 |
| *** links has joined #openstack-nova | 13:05 | |
| bauzas | sdague: tbc, can_host means that it can be a destination for the scheduler | 13:05 |
| bauzas | sdague: because we could have some Resource Providers that are not compute nodes | 13:05 |
| cdent | sdague, bauzas: that was its original intent but is now sort of meaningless | 13:06 |
| sdague | ok | 13:06 |
| cdent | but since it is in the schema and the object, we still see it, kind of like an appendix | 13:06 |
| *** edmondsw__ has joined #openstack-nova | 13:06 | |
| *** edmondsw__ has quit IRC | 13:06 | |
| *** tblakes has joined #openstack-nova | 13:07 | |
| sdague | ok | 13:08 |
| kashyap | mriedem_afk: When you're about: https://bugs.launchpad.net/nova/+bug/1646181/comments/7 | 13:08 |
| openstack | Launchpad bug 1646181 in OpenStack Compute (nova) "NFS: Fail to boot VM out of large snapshots (30GB+)" [Undecided,Incomplete] | 13:08 |
| *** shangxdy has quit IRC | 13:08 | |
| sdague | well, +A on that patch hopefully gets things moved forward | 13:08 |
| kashyap | sdague: You, too, when get a moment ^ | 13:08 |
| sdague | bauzas: so you are respinning a patch for on top of that? Just want to make sure I understand what's next there | 13:09 |
| cdent | sdague: cool. I think once we have the scheduler using the data we'll probably have some reorientation in our assumptions (in a useful way) | 13:09 |
| sdague | kashyap: ah, interesting, so that supports the guess I made yesterday | 13:09 |
| *** boris-42 has quit IRC | 13:09 | |
| bauzas | sdague: yup, please, sec | 13:10 |
| *** boris-42 has joined #openstack-nova | 13:10 | |
| bauzas | sdague: I'm just squashing jay's change with mine | 13:10 |
| bauzas | I could be uploading it by the next 30 mins | 13:10 |
| *** kristian__ has quit IRC | 13:10 | |
| sdague | bauzas: ok, great | 13:11 |
| sdague | let me know when it hits | 13:11 |
| *** jheroux has joined #openstack-nova | 13:11 | |
| *** portdirect has quit IRC | 13:11 | |
| *** portdirect has joined #openstack-nova | 13:12 | |
| openstackgerrit | jichenjc proposed openstack/nova: check hw:mem_page_size when change extra_spec https://review.openstack.org/408610 | 13:14 |
| kashyap | sdague: Yep | 13:14 |
| sdague | kashyap: ok, so I guess it would be nice if oslo.concurrency gave a better message when we died on process limits | 13:15 |
| *** psachin has quit IRC | 13:15 | |
| cdent | sdague: I know what you mean on that SQL (in the thing you just approved). Unfortunately it is going to get more complex as aggregates are integrated, and nested resource providers. The benefit from that cost, though, is that it ought to be efficient and accurate. | 13:16 |
| sdague | cdent: would we be able to isolate it into a library that we could test the crap out of? | 13:16 |
| cdent | that's a good idea. | 13:17 |
| *** esberglu has joined #openstack-nova | 13:17 | |
| *** esberglu has quit IRC | 13:17 | |
| *** esberglu_ has joined #openstack-nova | 13:17 | |
| cdent | I think that would probably have to happen after we've hit some boundary that is akin to "done" because we are still changing things quite a lot | 13:18 |
| cdent | and in the process of the changing doing a lot of learning | 13:18 |
| sdague | I just get concerned we're going to have edge cases that normal unit tests are not going to setup, and that aren't obvious. We already had the one with a bad join that only showed up with a bunch of data late last cycle | 13:18 |
| * cdent nods | 13:18 | |
| *** bhagyashris has joined #openstack-nova | 13:19 | |
| cdent | A reasonoable start may be to move all the sql driven methods to their own module. We stuck them in the objects initially because that was the new-styleee for things that were being hosted by the api db. At that time I don't think anyone beyond jay realized how complex the SQL was going to get | 13:19 |
| sdague | cdent: yeh, understood. It's just that the normal trade off of more inspection to make up for incomplete test coverage ends up being much harder when you cross domain boundaries from python to SQL | 13:19 |
| * cdent nods again | 13:20 | |
| cdent | totally agree | 13:20 |
| robcresswell | Hey guys. We've settled on a plan for microversion support in Horizon that should suffice. I dont suppose there is a list of actions/states etc available by microversion? As in 2.3: Introduced 'foo' and 'bar'. I checked the release notes, but there doesnt seem to be any ordering to them. | 13:20 |
| *** WillUK2010 has quit IRC | 13:21 | |
| *** dinobot_ has quit IRC | 13:21 | |
| robcresswell | Ah, found it http://docs.openstack.org/developer/nova/api_microversion_history.html Sorry for the noise. | 13:21 |
| *** tbachman has joined #openstack-nova | 13:22 | |
| sdague | robcresswell: yeh, the source in tree is this - https://github.com/openstack/nova/blob/668f2f963170b40f68045976393ea4393d940949/nova/api/openstack/rest_api_version_history.rst#L1, I was trying to dig you up the link | 13:22 |
| robcresswell | sdague: Ah cheers | 13:23 |
| *** yamahata has joined #openstack-nova | 13:24 | |
| *** dkliban has joined #openstack-nova | 13:25 | |
| *** riccardo has joined #openstack-nova | 13:26 | |
| *** mriedem_afk is now known as mriedem | 13:26 | |
| *** dkliban has left #openstack-nova | 13:26 | |
| jroll | cdent: do you know if there's a spec out yet that can show someone what new ironic flavors would look like? (e.g. the requires: {'custom_baremetal_gold': 1} bit, I guess scheduling for custom resource classes at all) | 13:27 |
| cdent | jroll: not that I'm aware of. As I recall there was some discussion in portland but it didn't result in stuff being written down. I think it was sort of a matter of "we'll know better when we get there". | 13:29 |
| cdent | (note that me being not aware of it doesn't guarantee it doesn't exist...) | 13:29 |
| jroll | cdent: okay, cool, no worries | 13:29 |
| mriedem | sdague: cdent: bauzas: edleafe had opened a bug to rename that can_host field to 'shared' in the DB and object model, just fyi | 13:29 |
| jroll | (asking for a PM) | 13:29 |
| jroll | :P | 13:29 |
| *** prateek has joined #openstack-nova | 13:31 | |
| mriedem | jroll: closest i see lately is that thread in the ML from jaypipes | 13:33 |
| mriedem | on bm flavors | 13:33 |
| mriedem | http://lists.openstack.org/pipermail/openstack-dev/2016-December/108393.html | 13:33 |
| jroll | mriedem: right, ok | 13:34 |
| jroll | thanks | 13:34 |
| openstackgerrit | Sylvain Bauza proposed openstack/nova: Extend get_all_by_filters to support resource criteria https://review.openstack.org/386242 | 13:34 |
| openstackgerrit | Sylvain Bauza proposed openstack/nova: Expose a REST API for a specific list of RPs https://review.openstack.org/392569 | 13:34 |
| openstackgerrit | Sylvain Bauza proposed openstack/nova: Support multiple uuids and names for RP list https://review.openstack.org/407630 | 13:34 |
| *** ayoung has joined #openstack-nova | 13:35 | |
| *** kevinz has joined #openstack-nova | 13:35 | |
| bauzas | cdent: sdague: mriedem: squashed jay's commit in my branch | 13:35 |
| cdent | thanks bauzas will look soon | 13:35 |
| *** phuongnh has quit IRC | 13:36 | |
| bauzas | of course, I did put him as a co-author | 13:36 |
| bauzas | but he missed two points | 13:36 |
| mriedem | bauzas: are you addressing cdent's comments in https://review.openstack.org/#/c/407630/ ? | 13:36 |
| *** narayrak has quit IRC | 13:37 | |
| bauzas | mriedem: cdent: oops, not | 13:37 |
| bauzas | mriedem: cdent: give me yet another 20 mins then | 13:37 |
| *** liverpooler has joined #openstack-nova | 13:40 | |
| openstackgerrit | Diana Clarke proposed openstack/python-novaclient: Usage missing from generated docs https://review.openstack.org/408227 | 13:42 |
| mriedem | gibi: does it matter which order the rebuild.start/end vs rebuild.error notifications go in? | 13:43 |
| *** ljxiash has joined #openstack-nova | 13:44 | |
| *** riccardo has quit IRC | 13:44 | |
| *** lucas-hungry is now known as lucasagomes | 13:45 | |
| *** baoli has joined #openstack-nova | 13:45 | |
| gibi | mriedem: checking.. | 13:46 |
| gibi | mriedem: it seems there is no dependency between those patches so it can go in any order | 13:48 |
| mriedem | ok | 13:48 |
| *** baoli has quit IRC | 13:48 | |
| *** baoli has joined #openstack-nova | 13:49 | |
| mriedem | gibi: it's curious that the snapshot patch needed to make this change https://review.openstack.org/#/c/330128/37/nova/virt/fake.py | 13:50 |
| *** baoli has quit IRC | 13:50 | |
| edleafe | sdague: cdent: bauzas: Yesterday jaypipes explained that it was only used to distinguish disk from a shared provider and disk from a compute node. We agreed that that was too nova-specific, so it will be renamed 'shared'. | 13:50 |
| mriedem | but the rebuild start/stop patch, which tests right after resize revert, doesn't | 13:50 |
| edleafe | Of course, that flips all the logic, since can_host=1 means shared=0 | 13:50 |
| *** baoli has joined #openstack-nova | 13:50 | |
| *** tlian has joined #openstack-nova | 13:50 | |
| bauzas | edleafe: I missed that point but I disagree :) | 13:50 |
| bauzas | edleafe: the placement API is here for placing something to an host right? | 13:50 |
| edleafe | bauzas: <shock!> | 13:50 |
| cdent | edleafe: weird, I thought we had decided it was irrelevant as the presence of VCPU == can_host = 1 ? | 13:51 |
| bauzas | I mean, the end-goal of that is just to get a destination right? | 13:51 |
| edleafe | bauzas: now, yes. In the future, not necessarily | 13:51 |
| bauzas | edleafe: examples ? | 13:51 |
| cdent | bauzas: in the future it is hoped it will be used for placing disk, choosing networks | 13:51 |
| bauzas | edleafe: like, neutron wanting to place a router ? | 13:51 |
| edleafe | bauzas: yes. | 13:51 |
| mriedem | so the universal scheduler gantt never was | 13:51 |
| bauzas | mriedem: exactly my point | 13:51 |
| mriedem | can_host a volume :) | 13:52 |
| edleafe | mriedem: the idea is to design it in such a way as to make that possible if we ever get there | 13:52 |
| bauzas | edleafe: we discussed on that with a huge number of projects and none of them was interested in a generic scheduler | 13:52 |
| edleafe | mriedem: but not assume that we ever will | 13:52 |
| bauzas | but they wanted to have a cross-project scheduler, which is *very* different | 13:52 |
| mriedem | well, | 13:52 |
| mriedem | either way | 13:52 |
| mriedem | this is super low priority to change atm right? | 13:52 |
| mriedem | we can all agree there? | 13:52 |
| edleafe | mriedem: yep | 13:52 |
| cdent | mriedem: the can_host thing? yeah | 13:53 |
| edleafe | I made the bug to record the decision | 13:53 |
| mriedem | crisis averted | 13:53 |
| bauzas | edleafe: so, you said to me that we should not implement things that are not yet like a POST, but now we want to merge something related to some long-term discussion ? | 13:53 |
| mriedem | before my first cup of coffee is done | 13:53 |
| bauzas | mriedem: too late | 13:53 |
| mriedem | gdi | 13:53 |
| mriedem | bauzas: we're not doing anything about can_host now | 13:53 |
| mriedem | we can argue over the bug later, like in a year | 13:53 |
| bauzas | mriedem: well, we already propose to have RPs than can be shared | 13:53 |
| bauzas | mriedem: like a cinder volume | 13:54 |
| edleafe | bauzas: not the same. We will *always* need to be able to retrieve a filtered list of RPs. In the future, we will also need to have the placement engine select an RP, claim the resources, and return the selected RP | 13:54 |
| bauzas | mriedem: so, I think we still need to have a field for knowing if that can be a destination | 13:54 |
| edleafe | Two completely different actions | 13:54 |
| mriedem | bauzas: a volume can be shared, or a storage cluster? | 13:54 |
| mriedem | i thought it was the latter | 13:55 |
| bauzas | sorry the latter | 13:55 |
| bauzas | anywat | 13:55 |
| bauzas | anyway | 13:55 |
| bauzas | I just want to explain that discussing about whether the placement API could be called for something not returning a compute node seems long-term to me | 13:55 |
| bauzas | we need to know internally whether we can return a RP as a possible destination, fersur | 13:56 |
| * edleafe leaves to drop his kid off at school | 13:56 | |
| bauzas | but I don't think we would need to discuss now about whether it could be a router or a volume | 13:56 |
| *** artom has joined #openstack-nova | 13:56 | |
| bauzas | or a swift stuff | 13:56 |
| cdent | right now, the stuff that is implemented (and pending) in the placemet api doesn't care. it just says "give me a list of resource providers that match these filters" | 13:59 |
| openstackgerrit | Li Yingjun proposed openstack/nova: Notifications on flavor operations https://review.openstack.org/398171 | 13:59 |
| cdent | what a requestor chooses to do with that list is up to the requestor | 13:59 |
| *** READ10 has joined #openstack-nova | 13:59 | |
| cdent | that's a good thing, it means the api is generic | 13:59 |
| *** hamzy has quit IRC | 14:00 | |
| gibi | mriedem: your observation is correct about the fake change. It was a bit unfortunate that we noticed the problem after resize patch was merged | 14:02 |
| bauzas | sure, but I still think the question is good "can this resource provider can host my instance" | 14:02 |
| bauzas | while "is this RP shared" is meaningless to me | 14:02 |
| bauzas | but I'm French, so I leave you folks translate that to me | 14:03 |
| *** sacharya has joined #openstack-nova | 14:03 | |
| bauzas | I certainly don't want to enter a rabbithole now, just that edleafe wants to get something already merged that looks to me really unnecessary, with a very far away need and distracting us | 14:04 |
| andreaf | mriedem, sdague: I'm looking at https://bugs.launchpad.net/tempest/+bug/1646779 to see if I can find what went wrong there | 14:04 |
| openstack | Launchpad bug 1646779 in OpenStack Compute (nova) "Cannot connect to libvirt" [Undecided,Incomplete] | 14:04 |
| bauzas | and I honestly don't think it's a bug | 14:04 |
| *** kristian__ has joined #openstack-nova | 14:04 | |
| bauzas | it's just a BP | 14:04 |
| *** gszasz has quit IRC | 14:05 | |
| sdague | right, sorry, I was just asking for understanding on the can_host issue. And I agree touching it now seems cart before the horse. | 14:05 |
| sdague | anyway, is the next patch up for the resource provider list? | 14:05 |
| *** kristia__ has joined #openstack-nova | 14:05 | |
| bauzas | sdague: I just uploaded it | 14:06 |
| andreaf | mriedem. sdague: the syslog starts after the issue occurred, which is wierd - so I wanted to look at the dstat log - is there any tool you normally use to graph that log? | 14:06 |
| sdague | bauzas: ok, cool, was getting breakfast, I see the backscroll now | 14:06 |
| bauzas | sdague: but I'll put a new patchset for the top patch | 14:06 |
| bauzas | sdague: sorry, I missed to highlight you | 14:06 |
| *** mdrabe has joined #openstack-nova | 14:06 | |
| sdague | cdent: https://review.openstack.org/#/c/407630 you are still -1 on this? That's the next patch it seems | 14:07 |
| *** bvanhav__ has joined #openstack-nova | 14:07 | |
| *** psachin has joined #openstack-nova | 14:07 | |
| openstackgerrit | Sylvain Bauza proposed openstack/nova: Expose a REST API for a specific list of RPs https://review.openstack.org/392569 | 14:07 |
| bauzas | sdague: I'm just uploading a new PS | 14:07 |
| openstackgerrit | Sylvain Bauza proposed openstack/nova: Support multiple uuids and names for RP list https://review.openstack.org/407630 | 14:07 |
| bauzas | sdague: there it is $ | 14:07 |
| bauzas | sdague: taking cdent's points | 14:07 |
| *** sacharya has quit IRC | 14:07 | |
| *** bvanhav_ has quit IRC | 14:08 | |
| *** jed56 has joined #openstack-nova | 14:08 | |
| *** kristian__ has quit IRC | 14:08 | |
| *** bvanhav__ is now known as bvanhav | 14:09 | |
| *** eandersson has joined #openstack-nova | 14:11 | |
| *** burt has joined #openstack-nova | 14:12 | |
| *** jamesdenton has joined #openstack-nova | 14:13 | |
| *** Cristina__ is now known as CristinaPauna | 14:13 | |
| *** mriedem has quit IRC | 14:14 | |
| sdague | ah, ok, cool | 14:16 |
| *** bkopilov has joined #openstack-nova | 14:17 | |
| *** ducttape_ has joined #openstack-nova | 14:17 | |
| openstackgerrit | Li Yingjun proposed openstack/nova: Notifications on flavor operations https://review.openstack.org/398171 | 14:18 |
| *** gouthamr has joined #openstack-nova | 14:20 | |
| openstackgerrit | Diana Clarke proposed openstack/python-novaclient: Rename api_key to password https://review.openstack.org/406298 | 14:22 |
| *** kylek3h_away has joined #openstack-nova | 14:23 | |
| openstackgerrit | Sergey Nikitin proposed openstack/nova: Mark sibling CPUs as 'used' for cpu_thread_policy = 'isolated' https://review.openstack.org/391416 | 14:23 |
| sdague | bauzas: on https://review.openstack.org/#/c/392569/13/nova/api/openstack/placement/handlers/resource_provider.py ... that jsonschema is now preventing uuid=in:$UUID1,$UUID2 now right? | 14:24 |
| bauzas | sdague: /me facepalms | 14:25 |
| sdague | are there any tests for the uuid in params? | 14:25 |
| bauzas | yup, functional gabbi tests | 14:25 |
| bauzas | they should fail normally | 14:25 |
| bauzas | since I wrote one of them | 14:25 |
| sdague | I see it for name | 14:25 |
| sdague | oh, yeh, I see one now | 14:26 |
| *** oanson has quit IRC | 14:26 | |
| *** rmart04 has quit IRC | 14:26 | |
| bauzas | sdague: https://review.openstack.org/#/c/392569/13/nova/tests/functional/api/openstack/placement/gabbits/resource-provider.yaml@202 | 14:26 |
| sdague | right, so the name thing ends up passing because a string with in: is a string still, the uuid one will need to be tweaked some how | 14:26 |
| *** eharney has joined #openstack-nova | 14:26 | |
| bauzas | sdague: good point, I'll write a new patchset | 14:26 |
| sdague | bauzas: cool | 14:27 |
| bauzas | sdague: yeah, we need to accept a string but verify the UUID like I did in the bottom patch | 14:27 |
| bauzas | the reason why I wanted to squash jay's change into mine | 14:27 |
| bauzas | because we agreed to accept multiple uuids and use the "in:" tag | 14:27 |
| bauzas | since foo=bar&baz=moo means foo AND baz as per API WG rules | 14:28 |
| bauzas | oops | 14:28 |
| bauzas | I meant foo=bar&foo=baz meaning foo equals bar AND baz | 14:28 |
| *** kristia__ has quit IRC | 14:28 | |
| sdague | yep, makes sense | 14:29 |
| sdague | I think that validation piece should be all we need to handle | 14:29 |
| diana_clarke | Microversion 2.39 got scooped ;) So this is respun as 2.40 & ready for review again: https://review.openstack.org/#/c/386093 Thanks! | 14:29 |
| *** kevinz has quit IRC | 14:30 | |
| *** jwcroppe has joined #openstack-nova | 14:31 | |
| *** kevinz has joined #openstack-nova | 14:31 | |
| *** sudipto_ has joined #openstack-nova | 14:31 | |
| *** sudipto has joined #openstack-nova | 14:31 | |
| paul-carlton2 | I'm seeing live migration fail with libvirtError: Cannot check QEMU binary /usr/bin/kvm-spice: No such file or directory | 14:33 |
| paul-carlton2 | anyone seen this before? | 14:33 |
| *** eharney has quit IRC | 14:33 | |
| *** mlavalle has joined #openstack-nova | 14:34 | |
| paul-carlton2 | ah fixed it, somehow kvm got removed | 14:37 |
| * alex_xu just read the log and get to know the goal of placement service isn't a generic service | 14:38 | |
| *** tbachman has quit IRC | 14:38 | |
| cdent | bauzas: are you already working on a new version of https://review.openstack.org/#/c/392569/ (such that I should wait to review it)? | 14:39 |
| bauzas | cdent: yup, sdague made a good point | 14:40 |
| cdent | cool, Ill wait then, thanks | 14:40 |
| bauzas | cdent: np, thanks | 14:40 |
| *** jaypipes has joined #openstack-nova | 14:41 | |
| sdague | cdent: it was a pretty minor point on param validation, I think the rest of the code is probably fine to review in current form | 14:41 |
| cdent | sure, but I'll wait anyway, so my comments land (if any) land on the right version. I also want to look more mdbooth's magnum opus | 14:42 |
| mdbooth | It's been python3ed | 14:43 |
| * mdbooth is losing the will to live | 14:43 | |
| cdent | mdbooth: I like python3, if you can point me to an error message I might be able to help, or at least provide tea and sympathy | 14:44 |
| alex_xu | diana_clarke: 'doc/api_samples/keypairs/v2.35/keypairs-list-user2-resp.json' forget to rename :) but I didn't finish the review yet | 14:45 |
| mdbooth | Nah, I'm sure I can sort it technically. I just need to summon the emotional energy to blow even more time on busy work :/ | 14:45 |
| diana_clarke | alex_xu: I don't think that one needs renaming. Just the 2.39 stuff (that's 2.35). | 14:46 |
| *** ducttape_ has quit IRC | 14:46 | |
| *** kevinz has quit IRC | 14:46 | |
| *** tbachman has joined #openstack-nova | 14:47 | |
| *** eharney has joined #openstack-nova | 14:47 | |
| *** zhenq has joined #openstack-nova | 14:47 | |
| alex_xu | diana_clarke: oops, sorry, I missed that, that isn't part of api change | 14:47 |
| diana_clarke | alex_xu: No worries, thanks again for the reviews! | 14:48 |
| *** kevinz has joined #openstack-nova | 14:48 | |
| alex_xu | diana_clarke: np, try to finish the review before sleep | 14:48 |
| diana_clarke | alex_xu: No rush, get some sleep instead :) | 14:49 |
| *** tbachman has quit IRC | 14:49 | |
| *** daemontool has joined #openstack-nova | 14:49 | |
| alex_xu | just have passion to do it :) | 14:50 |
| bauzas | cdent: interesting. | 14:53 |
| bauzas | cdent: I had a gabbi test failing on some UUID verification | 14:54 |
| bauzas | cdent: but it blew up later gabbi tests | 14:54 |
| cdent | bauzas: yes, they are a sequence | 14:54 |
| bauzas | cdent: so, when fixing the first, it fixed the later | 14:54 |
| cdent | that's by design | 14:54 |
| *** takedakn has joined #openstack-nova | 14:54 | |
| bauzas | cdent: I know about that | 14:54 |
| bauzas | cdent: but the error wasn't related | 14:54 |
| cdent | if you had an unquoted ':' in your query string that may have been a factor | 14:55 |
| cdent | yaml will blow up on that | 14:55 |
| bauzas | not really | 14:55 |
| cdent | well tell me the error then and I can stop guessing :) | 14:55 |
| bauzas | anyway, I fixed the first test and now it's green | 14:55 |
| cdent | if you can tell me what was happening there might be a bug in gabbi I can fix, or something that isn't explained well enough in the docs | 14:56 |
| bauzas | cdent: here is the stack http://paste.openstack.org/show/591812/ | 14:56 |
| cdent | tnaks | 14:56 |
| openstackgerrit | Moshe Levi proposed openstack/nova: [WIP] OVS acceleration https://review.openstack.org/398265 | 14:56 |
| bauzas | cdent: it failed on the RP update | 14:56 |
| * cdent reads | 14:56 | |
| bauzas | cdent: which is right after my multiple UUID list test | 14:57 |
| bauzas | so I understand that it failed because of that | 14:57 |
| bauzas | but I thought it could be saying "meh." | 14:57 |
| bauzas | I mean | 14:57 |
| bauzas | we try to get the RP for updating it | 14:58 |
| bauzas | but we don't use the "in:" tag | 14:58 |
| bauzas | so it should work anyway | 14:58 |
| bauzas | and the test was ok | 14:58 |
| bauzas | (I mean the test for verifying that we can get a single RP) | 14:58 |
| bauzas | so, having an exception for updating the RP seems weird to me | 14:58 |
| bauzas | not sure you got me | 14:59 |
| cdent | the request for updating the rp is using data from the previous response | 14:59 |
| cdent | if the previous test failed that data will will be wrong | 14:59 |
| bauzas | which kind of data? | 14:59 |
| cdent | "No resource provider with uuid $.resource_providers[0].uuid found " | 14:59 |
| cdent | the request was: PUT /resource_providers/$.resource_providers[0].uuid | 15:00 |
| cdent | which means that the $RESPONSE did not work, | 15:00 |
| *** TravT has joined #openstack-nova | 15:00 | |
| bauzas | mmm okay I see | 15:00 |
| bauzas | cdent: thanks | 15:00 |
| *** cfriesen has joined #openstack-nova | 15:01 | |
| *** lamt has joined #openstack-nova | 15:01 | |
| cdent | bauzas: this is why it would probably make sense to break up the gabbi yaml files into smaller pieces: line 51 in https://etherpad.openstack.org/p/placement-newton-leftovers | 15:01 |
| *** esberglu_ has quit IRC | 15:01 | |
| cdent | they really ought to be distinct sequences | 15:02 |
| cdent | not just the dumping ground for all tests of a specific type of resources | 15:02 |
| *** esberglu has joined #openstack-nova | 15:02 | |
| bauzas | cdent: I totally agree | 15:02 |
| bauzas | (see? I can !) | 15:02 |
| bauzas | :p | 15:03 |
| *** mriedem has joined #openstack-nova | 15:03 | |
| *** kaisers_ has quit IRC | 15:03 | |
| *** nherciu has quit IRC | 15:03 | |
| *** psachin has quit IRC | 15:04 | |
| openstackgerrit | Sylvain Bauza proposed openstack/nova: Expose a REST API for a specific list of RPs https://review.openstack.org/392569 | 15:05 |
| *** takedakn has quit IRC | 15:05 | |
| *** esberglu has quit IRC | 15:06 | |
| *** kristian__ has joined #openstack-nova | 15:07 | |
| *** kristian__ has joined #openstack-nova | 15:08 | |
| *** ljxiash has quit IRC | 15:08 | |
| *** kristia__ has joined #openstack-nova | 15:09 | |
| *** kristian__ has quit IRC | 15:09 | |
| *** tbachman has joined #openstack-nova | 15:10 | |
| *** kevinz has quit IRC | 15:11 | |
| *** kevinz has joined #openstack-nova | 15:12 | |
| openstackgerrit | Sean Dague proposed openstack/nova: Bump prlimit cpu time for qemu from 2 to 8 https://review.openstack.org/408668 | 15:12 |
| alex_xu | mriedem: sdague, I can use the pagination marker parameter to test whether a instance uuid is valid or not, even that instance isn't belong to me. is there any security problem for this case? | 15:12 |
| bauzas | cdent: sdague: uploaded https://review.openstack.org/392569 | 15:12 |
| mriedem | alex_xu: don't you have to be admin? | 15:12 |
| sdague | alex_xu: interesting... | 15:12 |
| mriedem | oh, probably not, because the db api elevates the context to admin to lookup the marker | 15:13 |
| mriedem | i was just looking at that code | 15:13 |
| *** ljxiash has joined #openstack-nova | 15:13 | |
| sdague | mriedem: yeh | 15:13 |
| mriedem | we should probably filter on project/user | 15:13 |
| alex_xu | mriedem: sdague yea, the line 2511 of https://review.openstack.org/#/c/386093/9/nova/db/sqlalchemy/api.py | 15:13 |
| sdague | I'm not sure why we elevate there | 15:13 |
| alex_xu | also for the servers pagination | 15:13 |
| sdague | because we want deleted? | 15:13 |
| sdague | we definitely need more granularity | 15:14 |
| alex_xu | sdague: yes, i think so | 15:14 |
| sdague | alex_xu: I agree that is a problem we should address | 15:14 |
| *** mdnadeem has quit IRC | 15:14 | |
| alex_xu | sdague: got it, thanks | 15:14 |
| sdague | before we release this. Honestly, uuid probing is probably not a huge deal given the size of the namespace | 15:15 |
| sdague | but, it seems like we should try to avoid it | 15:15 |
| sdague | mriedem: https://review.openstack.org/408668 - for one of the NFS bugs | 15:15 |
| *** hongbin has joined #openstack-nova | 15:15 | |
| alex_xu | sdague: ok, got it, so it is ok for https://review.openstack.org/#/c/386093 now, but we should fix it later | 15:15 |
| mriedem | sdague: doesn't raising that limit sort of defeat the purpose of having the prlimit in place in the first place? | 15:16 |
| *** ducttape_ has joined #openstack-nova | 15:16 | |
| mriedem | sdague: i was thinking maybe change the limit based on the size of the image or something, but i guess that's a chicken/egg b/c aren't we using qemu-img info to determine the size? | 15:16 |
| sdague | mriedem: right, I thought the issue was a memory OOM killer problem | 15:17 |
| sdague | mdbooth: do you remember the original bug where we needed to add prlimit to qemu-img? | 15:17 |
| mdbooth | sdague: Only vaguely. | 15:18 |
| sdague | mriedem: I think bumping allowed cpu runtime from 2 seconds to 8 seconds isn't defeating our security | 15:18 |
| mdbooth | Is this memory exhaustion from a malicious image? | 15:18 |
| sdague | mdbooth: it might be | 15:18 |
| sdague | but there was also a cpu restrictor put on it | 15:18 |
| sdague | which turns out to be triggering for people in the wild with > 30GB images on slow NFS storage | 15:19 |
| mdbooth | Where's that done? oslo? | 15:19 |
| mdbooth | sdague: Interesting. Got a bug report? | 15:19 |
| mriedem | mdbooth: see above | 15:19 |
| mriedem | it's linked into the patch | 15:19 |
| mriedem | sdague: -1 there with comments inline | 15:19 |
| *** kristia__ has quit IRC | 15:19 | |
| *** links has quit IRC | 15:20 | |
| sdague | mriedem: ok, so the unit test is going to be custom exception presents a different error message? | 15:21 |
| sdague | I thought we weren't generally testing things like exact error messages | 15:21 |
| *** prateek has quit IRC | 15:22 | |
| mdbooth | Hmm, that has to be bad process accounting by the kernel | 15:23 |
| *** slaweq_ has joined #openstack-nova | 15:23 | |
| mriedem | sdague: no, that you're specifically handling a -9 | 15:24 |
| mriedem | i don't care about the message | 15:24 |
| mriedem | but we should test the conditional right? | 15:24 |
| sdague | https://bugs.launchpad.net/ossa/+bug/1449062 | 15:24 |
| openstack | Launchpad bug 1449062 in python-oslo.concurrency (Ubuntu Wily) "[OSSA 2016-012] qemu-img calls need to be restricted by ulimit (CVE-2015-5162)" [Medium,Fix committed] | 15:24 |
| mdbooth | By bad process accounting, I mean that I'm surprised qemu_img_info would take 2 seconds of cpu time, regardless of the speed of its storage | 15:24 |
| sdague | so the original bug never mentioned cpu time at all, that was I think just added for "2 seconds should be enough for anyone" | 15:25 |
| *** sudipto_ has quit IRC | 15:25 | |
| *** sudipto has quit IRC | 15:25 | |
| sdague | mdbooth: I don't fully know, but unfortunately with prlimit we're not getting information about what limit we exceeded. That was the recommendation by danpb and others that kashyap gathered. | 15:25 |
| *** _ducttape_ has joined #openstack-nova | 15:26 | |
| openstackgerrit | Gábor Antal proposed openstack/nova: Transform instance.volume_detach notification https://review.openstack.org/408676 | 15:26 |
| *** ijw has joined #openstack-nova | 15:26 | |
| *** slaweq_ has quit IRC | 15:27 | |
| *** slaweq_ has joined #openstack-nova | 15:28 | |
| mdbooth | sdague: Have you confirmed that it's the cpu limit users are hitting, btw? | 15:29 |
| mdbooth | Rather than the memory limit? | 15:29 |
| *** ducttape_ has quit IRC | 15:29 | |
| *** tbachman has quit IRC | 15:29 | |
| *** rfolco has joined #openstack-nova | 15:29 | |
| sdague | mdbooth: we have not fully, though when they run without prlimit it works fine | 15:30 |
| mdbooth | Does it work fine with cpu_time=8? | 15:30 |
| sdague | and when they patch in a naked qemu-img *before* the prlimit one it also works | 15:30 |
| sdague | so if it was blowing memory, that wouldn't make any sense | 15:30 |
| mdbooth | What's the image format? | 15:31 |
| sdague | https://bugs.launchpad.net/nova/+bug/1646181 | 15:31 |
| openstack | Launchpad bug 1646181 in OpenStack Compute (nova) "NFS: Fail to boot VM out of large snapshots (30GB+)" [Medium,In progress] - Assigned to Sean Dague (sdague) | 15:31 |
| sdague | everything known is in that bug | 15:32 |
| mdbooth | rwmjones: You around? | 15:33 |
| mdbooth | rwmjones: qemu-img info is exceeding its rlimit: https://bugs.launchpad.net/nova/+bug/1646181 | 15:33 |
| openstack | Launchpad bug 1646181 in OpenStack Compute (nova) "NFS: Fail to boot VM out of large snapshots (30GB+)" [Medium,In progress] - Assigned to Sean Dague (sdague) | 15:33 |
| sdague | mriedem: is there a sane pattern from testtools to inspect the exception from asertRaises | 15:33 |
| *** kaisers_ has joined #openstack-nova | 15:33 | |
| mriedem | sdague: exc = self.assertRaises(...) | 15:34 |
| sdague | mriedem: thanks | 15:34 |
| mriedem | self.assertIn('some words i expect', six.text_type(exc)) | 15:34 |
| mriedem | is what i use | 15:34 |
| *** slaweq_ has quit IRC | 15:34 | |
| *** slaweq__ has joined #openstack-nova | 15:34 | |
| edleafe | mriedem: hey thanks - I learned something today! | 15:34 |
| mdbooth | mriedem: Nice, I didn't know that | 15:35 |
| *** kevinz has quit IRC | 15:35 | |
| mdbooth | From kashyap: "And, it seems like the reporter downstream has confirmed by increasing | 15:35 |
| mdbooth | the CPU limit from 2 to 8, they're able to successfully boot instances." | 15:35 |
| mdbooth | So cpu seems to be pretty conclusively the culprit | 15:36 |
| rwmjones | mdbooth: hi | 15:38 |
| mdbooth | rwmjones: Hey. So it seems that on really slow NFS, qemu-img info can exceed the rlimit we gave it of 2 secs of cpu time. | 15:38 |
| *** esberglu has joined #openstack-nova | 15:39 | |
| mdbooth | Why would storage speed affect cpu time accounted to a process? Kernel bug, or qemu-img bug? | 15:39 |
| mdbooth | Or is that ok for some reason I don't understand. | 15:39 |
| mdbooth | rwmjones: Also, do I recall you posting something recently about memory exhaustion from malicious images? | 15:40 |
| rwmjones | mdbooth: that's a good question, it's not just total wallclock time? | 15:40 |
| *** mdrabe has quit IRC | 15:40 | |
| rwmjones | mdbooth: yes that's the reason why we added the limits in the first place | 15:40 |
| mdbooth | afaict it's RLIMIT_CPU | 15:40 |
| mdbooth | Didn't think that was wallclock | 15:40 |
| *** nandal_in has quit IRC | 15:41 | |
| rwmjones | getrlimit(2) isn't very specific, but CPU time would be the logical choice | 15:42 |
| *** cdelatte has joined #openstack-nova | 15:43 | |
| *** hamzy has joined #openstack-nova | 15:43 | |
| *** delattec has joined #openstack-nova | 15:43 | |
| openstackgerrit | Moshe Levi proposed openstack/nova: [WIP] OVS acceleration https://review.openstack.org/398265 | 15:43 |
| *** slaweq__ has quit IRC | 15:43 | |
| mdbooth | It's actually executing an external executable with the 'prlimit' tool, passing --cpu=2 | 15:43 |
| *** delattec has quit IRC | 15:44 | |
| *** cdelatte has quit IRC | 15:44 | |
| *** slaweq_ has joined #openstack-nova | 15:44 | |
| *** cdelatte has joined #openstack-nova | 15:44 | |
| openstackgerrit | Sean Dague proposed openstack/nova: Bump prlimit cpu time for qemu from 2 to 8 https://review.openstack.org/408668 | 15:45 |
| sdague | time prlimit --cpu=2 dd if=/dev/zero of=/dev/null | 15:46 |
| *** rmart04 has joined #openstack-nova | 15:47 | |
| sdague | it will get killed after 2 seconds, because the systime is still accounted against the process | 15:47 |
| mdbooth | sdague: Fun question, would your new test fail if the environment language wasn't english? | 15:47 |
| *** mdrabe has joined #openstack-nova | 15:47 | |
| sdague | mdbooth: probably, as would hundreds of other tests | 15:47 |
| openstackgerrit | Julian proposed openstack/nova: Transform keypair.create notification https://review.openstack.org/408689 | 15:47 |
| mdbooth | sdague: Ok :) | 15:47 |
| *** riccardo has joined #openstack-nova | 15:48 | |
| sdague | :) | 15:49 |
| *** mtanino has joined #openstack-nova | 15:49 | |
| sdague | mdbooth: actually, no | 15:49 |
| sdague | https://github.com/openstack/nova/blob/b78b1f8ce3aa407307a6adc5c60de1e960547897/tox.ini#L15-L16 | 15:49 |
| sdague | which we do for exactly that reason | 15:49 |
| openstackgerrit | Sarafraj Singh proposed openstack/nova: Objectify requests_and_created_ports https://review.openstack.org/405398 | 15:50 |
| sdague | mriedem: unit test now added, as well as some more context in the commit message | 15:51 |
| mriedem | ack | 15:51 |
| mriedem | fixing glance on stable/liberty atm | 15:51 |
| mriedem | it would be cool if the reporter of the bug, who seems to be pretty responsive, could test that patch out | 15:51 |
| sdague | mriedem: yeh, I asked that question in the bug | 15:52 |
| sdague | we'll see | 15:52 |
| *** jaosorior has quit IRC | 15:52 | |
| *** jaosorior has joined #openstack-nova | 15:53 | |
| mriedem | sdague: thanks for helping out with bug triage btw, i see a noticeable dip in open bugs | 15:57 |
| openstackgerrit | Gábor Antal proposed openstack/nova: Readibility enhancements in notification_sample_tests/ https://review.openstack.org/385884 | 15:57 |
| *** tbachman has joined #openstack-nova | 15:57 | |
| *** roeyc has quit IRC | 15:58 | |
| *** kristian__ has joined #openstack-nova | 16:00 | |
| openstackgerrit | Ludovic Beliveau proposed openstack/nova: Resource tracker doesn't free resources on confirm resize https://review.openstack.org/405575 | 16:00 |
| sdague | yeh, I was trying to hack away yesterday | 16:00 |
| lbeliveau | jaypipes: ^ RT bug, when you have some time | 16:01 |
| *** ihrachys has quit IRC | 16:02 | |
| *** ayogi has joined #openstack-nova | 16:02 | |
| *** rmart04 has quit IRC | 16:03 | |
| *** moshele has quit IRC | 16:03 | |
| *** sacharya has joined #openstack-nova | 16:04 | |
| *** nkrinner is now known as nkrinner_afk | 16:04 | |
| mriedem | https://bugs.launchpad.net/nova/+bug/1633734 is a fun upgrade bug from mitaka to newton | 16:05 |
| openstack | Launchpad bug 1633734 in OpenStack Compute (nova) "ValueError: Field `instance_uuid' cannot be None" [Undecided,New] - Assigned to Praveen N (praveenn) | 16:05 |
| *** daemontool has quit IRC | 16:05 | |
| *** owalsh has quit IRC | 16:05 | |
| mriedem | looks like we might need a migration that clears out build requests with a null instance | 16:06 |
| *** ayoung_ has joined #openstack-nova | 16:09 | |
| *** ayoung_ has quit IRC | 16:09 | |
| *** sacharya has quit IRC | 16:09 | |
| *** ayoung_ has joined #openstack-nova | 16:09 | |
| *** sandanar has quit IRC | 16:10 | |
| *** jaosorior has quit IRC | 16:11 | |
| *** ayoung has quit IRC | 16:13 | |
| *** ayoung_ is now known as ayoung | 16:14 | |
| *** ayoung_ has joined #openstack-nova | 16:14 | |
| *** whenry has joined #openstack-nova | 16:16 | |
| openstackgerrit | sahid proposed openstack/nova: objects: remove related pining from __init__ of InstanceNUMATopology https://review.openstack.org/408242 | 16:18 |
| openstackgerrit | sahid proposed openstack/nova: objects: remove pagesize from __init__ of InstanceNUMATopology https://review.openstack.org/408241 | 16:18 |
| *** pcaruana has quit IRC | 16:19 | |
| *** slaweq_ has quit IRC | 16:19 | |
| openstackgerrit | Julian proposed openstack/nova: Transform instance.add_fixed_ip notification https://review.openstack.org/332876 | 16:21 |
| *** markus_z has quit IRC | 16:22 | |
| *** _ducttape_ has quit IRC | 16:22 | |
| *** ducttape_ has joined #openstack-nova | 16:23 | |
| *** mingyu has joined #openstack-nova | 16:23 | |
| *** ducttape_ has quit IRC | 16:23 | |
| *** ducttape_ has joined #openstack-nova | 16:23 | |
| *** ijw has quit IRC | 16:25 | |
| sfinucan | Can anyone tell me what black magic is happening on this line in the tests https://github.com/openstack/nova/blob/master/nova/tests/unit/test_service.py#L369 | 16:25 |
| mriedem | melwitt: do you remember looking at this? https://bugs.launchpad.net/nova/+bug/1633734 | 16:25 |
| openstack | Launchpad bug 1633734 in OpenStack Compute (nova) "ValueError: Field `instance_uuid' cannot be None" [Undecided,Confirmed] - Assigned to Praveen N (praveenn) | 16:25 |
| mriedem | sfinucan: setting CONF.test_service_listen="::1" | 16:25 |
| mriedem | you've have to see where CONF.test_service_listen is used | 16:26 |
| sfinucan | mriedem: That's the point - it's not used anywhere | 16:26 |
| sfinucan | It's defined at the top of that file and overriden on that line, yet I can't find it anywhere when grepping | 16:26 |
| sfinucan | magic | 16:26 |
| mriedem | sfinucan: remove the line and see if the test fails | 16:27 |
| sfinucan | mriedem: I did, and it does :/ | 16:27 |
| openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient: Create keystone session instance if not present https://review.openstack.org/304035 | 16:27 |
| openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient: Restict usage *args for novaclient.client.Client https://review.openstack.org/406294 | 16:27 |
| openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient: Introduce helper for checking args deprecation https://review.openstack.org/406295 | 16:27 |
| openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient: Sort arguments for client's methods https://review.openstack.org/406293 | 16:27 |
| openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient: Make SessionClient interface similar to HTTPClient https://review.openstack.org/350140 | 16:27 |
| openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient: Rename api_key to password https://review.openstack.org/406298 | 16:27 |
| openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient: Rename interface to endpoint_type https://review.openstack.org/406299 | 16:27 |
| openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient: Remove redundant args of _construct_http_client https://review.openstack.org/406296 | 16:27 |
| openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient: Rename bypass_url to endpoint_override https://review.openstack.org/406297 | 16:27 |
| openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient: Clarify meaning of project_id var https://review.openstack.org/350106 | 16:27 |
| openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient: Deprecate proxy_token and proxy_tenant_id args https://review.openstack.org/408703 | 16:27 |
| openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient: Transmit all auth related vars from cli to inner methods https://review.openstack.org/408704 | 16:27 |
| openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient: Deprecate conection_pool variable https://review.openstack.org/408705 | 16:27 |
| mriedem | sfinucan: self.host = getattr(CONF, '%s_listen' % name, "0.0.0.0") | 16:27 |
| mriedem | it's magic | 16:27 |
| mriedem | %s = test_service | 16:27 |
| mriedem | the name of the service | 16:27 |
| mriedem | so then it reads from the config option test_service_listen | 16:27 |
| sfinucan | mriedem: getattr strikes again | 16:28 |
| sfinucan | So that should stay. Good stuff. Thanks :) | 16:28 |
| *** baoli has quit IRC | 16:29 | |
| openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient: Create keystone session instance if not present https://review.openstack.org/304035 | 16:29 |
| *** sacharya has joined #openstack-nova | 16:30 | |
| *** mvk has quit IRC | 16:30 | |
| *** rcernin has quit IRC | 16:31 | |
| *** mriedem1 has joined #openstack-nova | 16:32 | |
| cdent | thanks for the review jaypipes, will get that stuff in a mo | 16:33 |
| cdent | mriedem, jaypipes is it the case that any time we change a microversion_history doc we should also make a reno note, or is the history change sufficient? | 16:33 |
| *** paw has quit IRC | 16:34 | |
| *** krtaylor has quit IRC | 16:34 | |
| *** mriedem has quit IRC | 16:34 | |
| jaypipes | cdent: yeah, a reno note for each microversion is what I'd like to see. | 16:35 |
| cdent | roger that | 16:35 |
| edleafe | Is reno note like PIN number? | 16:35 |
| mriedem1 | cdent: if you're adding a new microversion then a reno with it yeah | 16:35 |
| mriedem1 | is what we usually do | 16:35 |
| *** mriedem1 is now known as mriedem | 16:35 | |
| *** mdnadeem has joined #openstack-nova | 16:36 | |
| mriedem | dansmith: so i'm looking at fixing bug 1633734 which is an upgrade issue from mitaka to newton because of how build requests worked and were modeled, if you have any zombie build requests from mitaka that don't have an instance_uuid set, they can blow up in newton, | 16:37 |
| openstack | bug 1633734 in OpenStack Compute (nova) "ValueError: Field `instance_uuid' cannot be None" [Undecided,Confirmed] https://launchpad.net/bugs/1633734 - Assigned to Praveen N (praveenn) | 16:37 |
| mriedem | dansmith: alaski suggested an online data migration in the bug report to fix that, | 16:37 |
| mriedem | so was going to look at that, but as it's m->n only do we need that fix in master too? | 16:37 |
| mriedem | or just newton? | 16:37 |
| *** slaweq_ has joined #openstack-nova | 16:38 | |
| *** romainh has joined #openstack-nova | 16:39 | |
| *** stewie925 has joined #openstack-nova | 16:41 | |
| stewie925 | hi guys, does the nova table "instance_types" correlate to flavors? | 16:42 |
| *** knikolla has quit IRC | 16:42 | |
| mriedem | stewie925: yes | 16:42 |
| *** tpatzig_ has joined #openstack-nova | 16:42 | |
| mriedem | although flavors are in the api db now | 16:42 |
| *** knikolla has joined #openstack-nova | 16:42 | |
| *** ducttape_ has quit IRC | 16:43 | |
| stewie925 | I see - so that explains why I tried to do a query on instance_types and it turned up empty :) | 16:43 |
| *** ducttape_ has joined #openstack-nova | 16:43 | |
| *** tpatzig_ has quit IRC | 16:44 | |
| stewie925 | thank you mriedem | 16:44 |
| * cdent wants vanilla | 16:44 | |
| cdent | or banana, but none of that real banana, artificial banana only | 16:44 |
| cdent | jaypipes: this rebase is going to be fun: the rewrite for the db filter query has merged | 16:46 |
| jaypipes | cdent: ya. | 16:47 |
| jaypipes | cdent: it's a nice ol' mutex. | 16:47 |
| edleafe | cdent: want me to rebase https://review.openstack.org/#/c/404279 | 16:47 |
| edleafe | since my patch sits on top of that? | 16:47 |
| cdent | edleafe: yes please | 16:48 |
| *** armax has joined #openstack-nova | 16:48 | |
| cdent | and thank you | 16:48 |
| edleafe | cdent: okie dokie | 16:48 |
| *** riccardo has quit IRC | 16:51 | |
| *** andreas_s has quit IRC | 16:54 | |
| openstackgerrit | Ed Leafe proposed openstack/nova: Cascade deletes of RP aggregate associations https://review.openstack.org/407707 | 16:56 |
| *** mingyu has quit IRC | 16:56 | |
| openstackgerrit | Ed Leafe proposed openstack/nova: Make resource provider objects not remotable https://review.openstack.org/404279 | 16:56 |
| edleafe | cdent: ^^ done | 16:57 |
| cdent | ✔ | 16:57 |
| BobBall | sfinucan / mriedem: OK - I've tracked down what's going on. It seems that Nova's websocketproxy has support for the XCP VNC handshake, which we are using (https://github.com/openstack/nova/blob/master/nova/console/websocketproxy.py#L146) As such I believe we can safely deprecate XVP proxy which was effectively doing thte same thing https://git.openstack.org/cgit/openstack/nova/tree/nova/vnc/xvp_proxy.py#n73 | 16:59 |
| *** browne has joined #openstack-nova | 16:59 | |
| *** alex_xu has quit IRC | 17:00 | |
| *** alex_xu has joined #openstack-nova | 17:02 | |
| *** rcernin has joined #openstack-nova | 17:03 | |
| *** sahid has quit IRC | 17:03 | |
| sfinucan | BobBall: Lovely stuff. If websocketproxy can do this for us it certainly doesn't make sense to keep the older option around. | 17:04 |
| BobBall | sfinucan: I also think we don't need to go to operators with this as it's just a configuration change and everything-will-work; agreed? If so I can propose deprecating the config options and we can remove the py file next cycle? | 17:04 |
| *** ralonsoh_ has joined #openstack-nova | 17:04 | |
| *** ralonsoh_ has quit IRC | 17:05 | |
| *** ralonsoh has quit IRC | 17:05 | |
| sfinucan | BobBall: Not 100% sure about that. From what I can see they appear to be separate services, rather than part of one larger service | 17:06 |
| BobBall | sfinucan: Indeed; it's a separate service, is which service is used a configuration option? | 17:07 |
| sfinucan | BobBall: That's the question. I don't think so because from what I can tell the 'XVPConsoleProxy' is the only valid, non-test value for 'console_driver' https://review.openstack.org/#/c/408193/1/nova/conf/console.py | 17:09 |
| *** matrohon has quit IRC | 17:09 | |
| sfinucan | (the link doc in that commit message suggests as much) | 17:09 |
| sfinucan | *linked | 17:09 |
| BobBall | Linked doc? | 17:10 |
| *** krtaylor has joined #openstack-nova | 17:10 | |
| sfinucan | That doesn't mean there isn't a config option to switch between xvp and websocketproxy, but that's not if fwict | 17:10 |
| sfinucan | I'd like to look at Devstack, see if it still does anything with the 'nova-console' service (xvp) and if we can switch it out for 'nova-consoleproxy' or whatever the websocketproxy service is called (if anything) | 17:11 |
| sfinucan | BobBall: Oh, 2 secs | 17:11 |
| sfinucan | BobBall: https://github.com/openstack/openstack-manuals/tree/4b1e8f1/doc/admin-guide/source/compute-remote-console-access.rst | 17:11 |
| sfinucan | from a dependency patch https://review.openstack.org/#/c/408191/1 - my bad | 17:11 |
| *** knikolla has quit IRC | 17:11 | |
| *** knikolla has joined #openstack-nova | 17:12 | |
| *** lpetrut has quit IRC | 17:12 | |
| *** ihrachys has joined #openstack-nova | 17:12 | |
| sfinucan | (i.e., "Do not confuse the nova-consoleauth shared service with nova-console, which is a XenAPI-specific service that most recent VNC proxy architectures do not use.") | 17:13 |
| BobBall | ohhhhhh - right. I definitely mis-understood then. | 17:13 |
| *** tpatzig_ has joined #openstack-nova | 17:13 | |
| sfinucan | BobBall: So assuming I've got that right (not a sure thing :)), I guess we'd still need to mail operators to make sure no one has anything they can't use websocketproxy for? | 17:14 |
| BobBall | "One or more nova-xvpvncproxy services. Supports the special Java client discussed here." There is no discussion of this "special Java client". I presume this is a java client similar to one used to connect directly to XenServer hosts, but I don't see why that would be needed | 17:14 |
| BobBall | No; I've totally mis-understood. | 17:14 |
| *** tpatzig_ has quit IRC | 17:15 | |
| sfinucan | I guess this? https://github.com/xvpsource/xvp | 17:15 |
| *** ijw has joined #openstack-nova | 17:15 | |
| BobBall | nova-xvpvncproxy may be used by Rackspace particularly; their VNC console is Java based rather than novnc based. | 17:16 |
| sdague | bauzas: you working through test failures on this - https://review.openstack.org/#/c/407630 ? | 17:16 |
| sfinucan | Drat :/ | 17:16 |
| openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient: Create keystone session instance if not present https://review.openstack.org/304035 | 17:16 |
| openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient: Remove all code related to HTTPClient https://review.openstack.org/408728 | 17:16 |
| *** smatzek has joined #openstack-nova | 17:17 | |
| andreykurilin | mriedem: I finished work on httpclient stuff ^ (hope patches will pass gates) | 17:17 |
| BobBall | sfinucan: Horizon by default will clearly be using novnc... but the Rackspace UI gives extra options | 17:17 |
| mriedem | melwitt: fyi https://review.openstack.org/#/c/408727/ | 17:17 |
| *** ijw has quit IRC | 17:18 | |
| BobBall | sfinucan: I'll have a chat with one of the folk @ rax to see what their thoughts are | 17:18 |
| openstackgerrit | Chris Dent proposed openstack/nova: rt: pass the nodename to public methods https://review.openstack.org/398470 | 17:19 |
| openstackgerrit | Chris Dent proposed openstack/nova: HTTP interface for resource providers by aggregates https://review.openstack.org/407742 | 17:19 |
| openstackgerrit | Chris Dent proposed openstack/nova: rt: remove fluff from test_resource_tracker https://review.openstack.org/398471 | 17:19 |
| openstackgerrit | Chris Dent proposed openstack/nova: rt: pass nodename to internal methods https://review.openstack.org/398469 | 17:19 |
| openstackgerrit | Chris Dent proposed openstack/nova: placement: create aggregate map in report client https://review.openstack.org/407309 | 17:19 |
| openstackgerrit | Chris Dent proposed openstack/nova: Support filtering resource providers by aggregate membership https://review.openstack.org/407629 | 17:19 |
| openstackgerrit | Chris Dent proposed openstack/nova: placement: correct improper test case inheritance https://review.openstack.org/406363 | 17:19 |
| openstackgerrit | Chris Dent proposed openstack/nova: rt: explicitly pass compute node to _update() https://review.openstack.org/398472 | 17:19 |
| openstackgerrit | Chris Dent proposed openstack/nova: rt: use a single ResourceTracker object instance https://review.openstack.org/398473 | 17:19 |
| sfinucan | BobBall: That would be helpful. If novnc is a drop in replacement then I guess we could deprecate like any other feature. If it's not, though, I guess it has to stick around a while longer | 17:19 |
| BobBall | sfinucan: It's not quite drop-in. novnc doesn't seem to have support for passing function keys through to the guest? | 17:20 |
| *** efoley__ has quit IRC | 17:21 | |
| *** roeyc has joined #openstack-nova | 17:21 | |
| *** tblakes has quit IRC | 17:21 | |
| *** ihrachys has quit IRC | 17:22 | |
| sfinucan | Ah, well then. That makes things a little trickier | 17:22 |
| BobBall | Indeed. But getting away from xvpproxy would be a welcome change I believe; I'll see whether they can look at it | 17:22 |
| sfinucan | "We need you to stop using this feature just so we don't have to maintain the code" doesn't have the best ring to it, heh | 17:23 |
| sfinucan | Ok, cool cool :) | 17:23 |
| *** antonym has joined #openstack-nova | 17:23 | |
| openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient: Create keystone session instance if not present https://review.openstack.org/304035 | 17:27 |
| openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient: Deprecate connection_pool variable https://review.openstack.org/408705 | 17:27 |
| openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient: Make SessionClient interface similar to HTTPClient https://review.openstack.org/350140 | 17:27 |
| openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient: Remove all code related to HTTPClient https://review.openstack.org/408728 | 17:27 |
| *** mdnadeem has quit IRC | 17:28 | |
| *** stewie925 has quit IRC | 17:29 | |
| cdent | bauzas, edleafe, jaypipes : I'm not sure how to address bauzas' concerns on https://review.openstack.org/#/c/404279/ If I understand you correctly bauzas what you're hoping for is some way to test that no one accidently adds a remotable method? | 17:29 |
| *** ljxiash_ has joined #openstack-nova | 17:29 | |
| openstackgerrit | Sarafraj Singh proposed openstack/nova: Objectify requests_and_created_ports https://review.openstack.org/405398 | 17:29 |
| *** rfolco has quit IRC | 17:30 | |
| *** baoli has joined #openstack-nova | 17:30 | |
| *** ljxiash has quit IRC | 17:32 | |
| edleafe | You mean like monkey patch base.remotable with something that would raise an exception? | 17:32 |
| edleafe | cdent: ^^ | 17:32 |
| edleafe | cdent: That wouldn't test it, but it would prevent it | 17:33 |
| cdent | edleafe: Dunno. I'm not sure of the goal, so I'm not willing to speculate on a solution | 17:33 |
| edleafe | cdent: I thought that was what human review was for | 17:33 |
| openstackgerrit | Julian proposed openstack/nova: Create schema generation for NetworkModel https://review.openstack.org/408737 | 17:34 |
| cdent | [t 21Ky] | 17:34 |
| purplerbot | <cdent> edleafe: Dunno. I'm not sure of the goal, so I'm not willing to speculate on a solution [2016-12-08 17:33:23] [n 21Ky] | 17:34 |
| cdent | :) | 17:34 |
| * cdent lets it ride for now | 17:34 | |
| *** ijw has joined #openstack-nova | 17:36 | |
| BobBall | sfinucan: OK - so this is potentially more complicated than I thought - I *THINK* that there are two things here; nova.vnc.xvp_proxy and nova.console.xvp. The latter (which is the subject of your change) includes some funky things on top of console access; e.g. the ability to change passwords. It does, however, require the use of the xvp utility too. I believe the xvp_proxy can probably be deprecated as novncproxy could be used in its place; that do | 17:37 |
| *** tbachman_ has joined #openstack-nova | 17:39 | |
| andreaf | tiny nova patch - it could use a 2nd review https://review.openstack.org/#/c/407037/ | 17:39 |
| *** tbachman has quit IRC | 17:40 | |
| *** tbachman_ is now known as tbachman | 17:40 | |
| sfinucan | BobBall: Ugh, what a rat nest | 17:43 |
| BobBall | sfinucan: Hey - you're the one who opened it :D | 17:43 |
| *** mdrabe has quit IRC | 17:43 | |
| sfinucan | BobBall: I know - silly me | 17:44 |
| sfinucan | So that would suggest 'nova-console' has some merit and should be retained | 17:44 |
| *** ducttape_ has quit IRC | 17:44 | |
| sfinucan | But if there's a config option that turns on nova.vnc.xvp_proxy then that might be deprecable (is that a word?) and the corresponding code eventually removed | 17:45 |
| *** nic has joined #openstack-nova | 17:45 | |
| *** TravT has quit IRC | 17:46 | |
| openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient: Create keystone session instance if not present https://review.openstack.org/304035 | 17:46 |
| openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient: Transmit all auth related vars from cli to inner methods https://review.openstack.org/408704 | 17:46 |
| openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient: Deprecate connection_pool variable https://review.openstack.org/408705 | 17:46 |
| openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient: Deprecate proxy_token and proxy_tenant_id args https://review.openstack.org/408703 | 17:46 |
| openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient: Make SessionClient interface similar to HTTPClient https://review.openstack.org/350140 | 17:46 |
| openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient: Clarify meaning of project_id var https://review.openstack.org/350106 | 17:46 |
| openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient: Remove all code related to HTTPClient https://review.openstack.org/408728 | 17:46 |
| openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient: Deprecate volume_service_name argument https://review.openstack.org/408745 | 17:46 |
| *** mingyu has joined #openstack-nova | 17:51 | |
| *** derekh has quit IRC | 17:52 | |
| *** ijw has quit IRC | 17:53 | |
| *** ihrachys has joined #openstack-nova | 17:55 | |
| *** browne has quit IRC | 17:56 | |
| *** lucasagomes is now known as lucas-pto | 17:56 | |
| *** kristian__ has quit IRC | 17:58 | |
| *** ijw has joined #openstack-nova | 17:59 | |
| *** moshele has joined #openstack-nova | 18:00 | |
| *** kristian__ has joined #openstack-nova | 18:00 | |
| *** romainh has quit IRC | 18:02 | |
| *** fragatina has joined #openstack-nova | 18:06 | |
| *** jed56 has quit IRC | 18:07 | |
| jaypipes | andreaf: done | 18:08 |
| *** ijw has quit IRC | 18:08 | |
| *** fragatina has quit IRC | 18:09 | |
| *** fragatina has joined #openstack-nova | 18:09 | |
| *** jgriffith has quit IRC | 18:10 | |
| openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient: Create keystone session instance if not present https://review.openstack.org/304035 | 18:14 |
| openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient: Make SessionClient interface similar to HTTPClient https://review.openstack.org/350140 | 18:14 |
| openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient: Remove all code related to HTTPClient https://review.openstack.org/408728 | 18:14 |
| openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient: Deprecate volume_service_name argument https://review.openstack.org/408745 | 18:14 |
| *** ducttape_ has joined #openstack-nova | 18:15 | |
| *** jgriffith has joined #openstack-nova | 18:15 | |
| *** ijw has joined #openstack-nova | 18:16 | |
| *** harlowja has joined #openstack-nova | 18:17 | |
| oomichi | mriedem: ok, I will try on requrest jsonschema validation thing with bknudson | 18:18 |
| jaypipes | dansmith: your signoff on https://review.openstack.org/#/c/404279/ would be appreciated. thanks in advance! | 18:19 |
| *** jgriffith has quit IRC | 18:19 | |
| moshele | lbeliveau: see my comment on https://review.openstack.org/#/c/405575/ | 18:20 |
| *** harlowja_ has joined #openstack-nova | 18:20 | |
| *** jgriffith has joined #openstack-nova | 18:20 | |
| lbeliveau | moshele: will have a look later, thanks | 18:21 |
| moshele | jaypipes: sorry for nagging but can you review the pci whitelist regex patch https://review.openstack.org/#/c/199488/ :) | 18:22 |
| moshele | lbeliveau: I think you fix will break resize on the same node | 18:22 |
| *** harlowja has quit IRC | 18:22 | |
| jaypipes | moshele: no reason to apologize. yep, will hop on that right now. | 18:22 |
| *** lpetrut has joined #openstack-nova | 18:22 | |
| *** dharinic is now known as dharinic|lunch | 18:22 | |
| openstackgerrit | Chris Dent proposed openstack/nova: HTTP interface for resource providers by aggregates https://review.openstack.org/407742 | 18:23 |
| cdent | jaypipes: ^ | 18:23 |
| *** sudipto_ has joined #openstack-nova | 18:24 | |
| *** ijw has quit IRC | 18:24 | |
| *** sudipto has joined #openstack-nova | 18:24 | |
| *** browne has joined #openstack-nova | 18:30 | |
| *** roeyc has quit IRC | 18:31 | |
| *** zul_ has joined #openstack-nova | 18:31 | |
| *** nherciu has joined #openstack-nova | 18:31 | |
| *** paw has joined #openstack-nova | 18:33 | |
| *** ayogi has quit IRC | 18:33 | |
| *** nic has quit IRC | 18:34 | |
| *** aweeks has joined #openstack-nova | 18:35 | |
| *** moshele has quit IRC | 18:35 | |
| *** ijw_ has joined #openstack-nova | 18:36 | |
| *** ijw_ has quit IRC | 18:36 | |
| *** ijw has joined #openstack-nova | 18:36 | |
| *** sudipto_ has quit IRC | 18:36 | |
| *** sudipto has quit IRC | 18:36 | |
| *** mkoderer has quit IRC | 18:36 | |
| *** paw has quit IRC | 18:39 | |
| *** ijw has quit IRC | 18:39 | |
| *** paw has joined #openstack-nova | 18:43 | |
| *** READ10 has quit IRC | 18:44 | |
| *** zul_ has quit IRC | 18:46 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova: Don't trace on ImageNotFound in delete_image_on_error https://review.openstack.org/408771 | 18:46 |
| mriedem | andreaf: commented in your e-r query | 18:47 |
| mriedem | i think i found a better query | 18:48 |
| *** zul_ has joined #openstack-nova | 18:48 | |
| *** dave-mccowan has quit IRC | 18:49 | |
| *** browne1 has joined #openstack-nova | 18:53 | |
| *** browne has quit IRC | 18:53 | |
| *** siva_krish has quit IRC | 18:53 | |
| kashyap | sdague: mriedem Just catching up on that CPU limits bug | 18:54 |
| kashyap | And the patch from Sean | 18:54 |
| kashyap | But isn't '8' a bit too high? | 18:54 |
| *** paul-carlton2 has quit IRC | 18:54 | |
| mriedem | kashyap: idk | 18:56 |
| kashyap | But at least two of them have confirmed | 18:56 |
| kashyap | That it works | 18:56 |
| kashyap | With real-world testing | 18:56 |
| *** ijw has joined #openstack-nova | 18:56 | |
| mriedem | did the bug reporter respond? | 18:56 |
| *** zz_dimtruck is now known as dimtruck | 18:57 | |
| mriedem | ah they said 8 worked | 18:57 |
| mriedem | so anyway i'm +2 on the patch | 18:57 |
| *** ijw has quit IRC | 18:58 | |
| *** dharinic|lunch is now known as dharinic | 18:58 | |
| sdague | ah, and the ceph recheck gets us working again. | 18:59 |
| sdague | there remains a ceph volume issue that's popping up? | 18:59 |
| mriedem | sdague: yeah | 18:59 |
| mriedem | have been some ceph issues over the last few months | 18:59 |
| mriedem | 2 were fixed | 18:59 |
| mriedem | there are 1 or 2 lingering | 18:59 |
| *** gjayavelu has joined #openstack-nova | 18:59 | |
| mriedem | http://status.openstack.org//elastic-recheck/index.html#1645530 should be fixed | 19:02 |
| mriedem | but it looks like something is still happening there | 19:02 |
| *** baoli has quit IRC | 19:05 | |
| *** baoli has joined #openstack-nova | 19:05 | |
| *** kristian__ has joined #openstack-nova | 19:06 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova: Don't trace on ImageNotFound in delete_image_on_error https://review.openstack.org/408771 | 19:06 |
| mriedem | easy serviceability win ^ | 19:06 |
| *** claudiub has joined #openstack-nova | 19:06 | |
| kashyap | mriedem: Yeah, they did. Also I noticed a downstream bug report | 19:06 |
| kashyap | (Mentioned it in the comment on the bug) | 19:07 |
| kashyap | Sorry for the lag here, didn't see your message | 19:07 |
| openstackgerrit | Stephen Finucane proposed openstack/os-vif: WIP: Always set MTU when plugging devices https://review.openstack.org/408779 | 19:09 |
| *** Sukhdev has joined #openstack-nova | 19:09 | |
| *** kristian__ has quit IRC | 19:11 | |
| openstackgerrit | Stephen Finucane proposed openstack/nova: WIP: Always set MTU when plugging devices https://review.openstack.org/408781 | 19:11 |
| *** ijw has joined #openstack-nova | 19:11 | |
| openstackgerrit | Julian proposed openstack/nova: Transform keypair.create notification https://review.openstack.org/408689 | 19:12 |
| *** ijw has quit IRC | 19:13 | |
| openstackgerrit | Stephen Finucane proposed openstack/nova: WIP: Always set MTU when plugging devices https://review.openstack.org/408781 | 19:15 |
| *** yamahata has quit IRC | 19:16 | |
| *** ijw has joined #openstack-nova | 19:17 | |
| *** whenry has quit IRC | 19:19 | |
| *** ijw has quit IRC | 19:19 | |
| *** cdent has quit IRC | 19:20 | |
| *** nic has joined #openstack-nova | 19:20 | |
| *** ijw has joined #openstack-nova | 19:21 | |
| *** ijw has quit IRC | 19:26 | |
| *** kristian__ has joined #openstack-nova | 19:27 | |
| *** slaweq_ has quit IRC | 19:29 | |
| *** slaweq_ has joined #openstack-nova | 19:30 | |
| *** paw has quit IRC | 19:31 | |
| cburgess | mriedem ping re new cinder attach stuffs | 19:32 |
| *** lpetrut has quit IRC | 19:35 | |
| *** ijw has joined #openstack-nova | 19:36 | |
| *** mingyu has quit IRC | 19:37 | |
| *** krtaylor has quit IRC | 19:37 | |
| *** tlian has quit IRC | 19:41 | |
| *** rcernin has quit IRC | 19:43 | |
| openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient: DO NOT MERGE: show current keystone v3 support https://review.openstack.org/357160 | 19:44 |
| *** ijw_ has joined #openstack-nova | 19:45 | |
| *** ijw has quit IRC | 19:48 | |
| *** kristia__ has joined #openstack-nova | 19:52 | |
| *** kristian__ has quit IRC | 19:52 | |
| *** kylek3h_away has quit IRC | 19:54 | |
| *** tbachman has quit IRC | 19:54 | |
| *** esberglu has quit IRC | 19:54 | |
| *** mingyu has joined #openstack-nova | 19:55 | |
| *** esberglu has joined #openstack-nova | 19:55 | |
| *** esberglu_ has joined #openstack-nova | 19:57 | |
| *** rcernin has joined #openstack-nova | 19:57 | |
| openstackgerrit | STEW TY proposed openstack/nova: Transform instance.resize_confirm notification https://review.openstack.org/390654 | 19:59 |
| *** esbergl__ has joined #openstack-nova | 19:59 | |
| *** vladikr has quit IRC | 19:59 | |
| *** esberglu has quit IRC | 20:00 | |
| *** mingyu has quit IRC | 20:00 | |
| *** esberglu_ has quit IRC | 20:02 | |
| *** esberglu has joined #openstack-nova | 20:03 | |
| *** fragatina has quit IRC | 20:03 | |
| *** esbergl__ has quit IRC | 20:03 | |
| *** smatzek_ has joined #openstack-nova | 20:03 | |
| *** tojuvone has quit IRC | 20:04 | |
| *** mingyu has joined #openstack-nova | 20:04 | |
| *** smatzek__ has joined #openstack-nova | 20:05 | |
| mriedem | cburgess: your ping is very important to me and will be handled in the order received, but i just got off a call and need a break, will pong shortly, thank you | 20:06 |
| *** smatzek has quit IRC | 20:06 | |
| *** smatzek_ has quit IRC | 20:08 | |
| *** fragatina has joined #openstack-nova | 20:08 | |
| *** ayoung has quit IRC | 20:12 | |
| *** ayoung_ has quit IRC | 20:12 | |
| *** ayoung has joined #openstack-nova | 20:12 | |
| *** ijw_ has quit IRC | 20:12 | |
| *** ijw has joined #openstack-nova | 20:13 | |
| *** jamielennox|away is now known as jamielennox | 20:13 | |
| *** paw has joined #openstack-nova | 20:14 | |
| *** ijw has quit IRC | 20:14 | |
| *** ihrachys has quit IRC | 20:14 | |
| *** yamahata has joined #openstack-nova | 20:17 | |
| kashyap | dansmith: When you're about, take a look at this, too? mriedem has already reviewed & +2ed it | 20:19 |
| kashyap | A clear fix: | 20:19 |
| kashyap | https://review.openstack.org/#/c/408668/ -- "Bump prlimit cpu time for qemu from 2 to 8" | 20:19 |
| kashyap | s/take/can you take/ (Must stay polite & professional) | 20:19 |
| *** mingyu has quit IRC | 20:21 | |
| openstackgerrit | Dan Smith proposed openstack/nova: WIP: Cleanup after any failed libvirt spawn https://review.openstack.org/408806 | 20:24 |
| sdague | jaypipes: placement API isn't doing front end validation of params? - https://bugs.launchpad.net/nova/+bug/1648314 | 20:25 |
| openstack | Launchpad bug 1648314 in OpenStack Compute (nova) "create/update respource_class APIs raises HTTP 500 Internal Server Error when name is greater than 255 characters" [Medium,Confirmed] - Assigned to Bhagyashri Shewale (bhagyashri-shewale) | 20:25 |
| *** ijw has joined #openstack-nova | 20:25 | |
| dansmith | kashyap: mdbooth maybe have a looksee at this and tell me if there's something I'm missing ^ | 20:25 |
| * kashyap clicks | 20:25 | |
| dansmith | because the explosion I've been looking at seems to potentially be because we don't do that cleanup if libvirt raises libvirtError from the create operation | 20:25 |
| openstackgerrit | Roman Dobosz proposed openstack/nova: placement: adds REST API for nested providers https://review.openstack.org/384807 | 20:26 |
| openstackgerrit | Roman Dobosz proposed openstack/nova: placement: allow filter providers in tree https://review.openstack.org/377215 | 20:26 |
| openstackgerrit | Roman Dobosz proposed openstack/nova: placement: update client to set parent provider https://review.openstack.org/385693 | 20:26 |
| openstackgerrit | Roman Dobosz proposed openstack/nova: placement: add nested resource providers https://review.openstack.org/377138 | 20:26 |
| dansmith | may also need destroy_disks=False, so that the delete does that and we don't delete data if we're just failing to start an existing instance | 20:26 |
| kashyap | dansmith: Nice commit message, always appreciate those | 20:29 |
| kashyap | Than "Fix bug" | 20:29 |
| kashyap | dansmith: For the above bug, can you let me know how you tested it on DevStack? Maybe I can reproduce it on my setup here, too? | 20:30 |
| kashyap | No rush, this can wait | 20:30 |
| melwitt | mriedem: yeah, I remember a little about that bug. nice repro in your WIP patch | 20:31 |
| *** slaweq_ has quit IRC | 20:31 | |
| dansmith | kashyap: super easy: http://pastebin.com/wDR7nGRH | 20:31 |
| *** slaweq_ has joined #openstack-nova | 20:32 | |
| kashyap | dansmith: Oh, not that way, I mean adding the interfaces in your DevStack env | 20:32 |
| kashyap | Did you do `brctl addbr [...]` in a script or some such? | 20:32 |
| dansmith | kashyap: I said I never did | 20:32 |
| kashyap | I was about to create one similar | 20:32 |
| *** claudiub has quit IRC | 20:32 | |
| dansmith | kashyap: I think probably because I wasn't naming them as they needed to be to be included in some libvirt list or some such | 20:32 |
| *** bvanhav has quit IRC | 20:32 | |
| kashyap | Ah-ha | 20:33 |
| *** zul_ has quit IRC | 20:33 | |
| *** kristia__ is now known as kristian__ | 20:33 | |
| kashyap | Ah I misread your downstream bug analysis, that you mentioned, you "couldn't reproduce even with 13K interfaces" | 20:33 |
| dansmith | kashyap: right, but it was just veth pairs and not really named or plumbed, so I think it was not being considered by libvirt | 20:34 |
| cburgess | mriedem LOL no worries. Not a super big rush | 20:34 |
| * kashyap nods | 20:35 | |
| kashyap | dansmith: The block_device_info is generated by virt.driver.get_block_device_info(), yes? | 20:38 |
| *** ijw has quit IRC | 20:38 | |
| *** ijw has joined #openstack-nova | 20:38 | |
| *** vladikr has joined #openstack-nova | 20:39 | |
| mriedem | cburgess: what's up? | 20:39 |
| dansmith | kashyap: L4976 right above | 20:39 |
| dansmith | kashyap: but regardless, this is called the same way for the two handlers above | 20:40 |
| cburgess | mriedem So... working with jgriffith now on using his proposed attach changes to accomplish my goals. | 20:40 |
| kashyap | dansmith: Yeah, saw your newer comment about it being semantically same change as the 'vif_plugging_is_fatal' case | 20:40 |
| cburgess | mriedem The question is.. assuming that his attach changes go into a cinder v3, what would be required to start using it on the nova side? Would we have to do a complete uplift of nova from v2 to v3? | 20:40 |
| jaypipes | sdague: just an oversight more than anything else I'm afraid. Happy to bugs reported and quickly fixed. | 20:41 |
| mriedem | cburgess: cinder v3 is backward compatible with v2, unless nova requests a specific microversion, | 20:42 |
| mriedem | cburgess: so nova is going to first be checking if cinder is at the version it can support these new apis | 20:42 |
| mriedem | and if the computes are at a new enough version to handle the new detach flow | 20:42 |
| sdague | mriedem: it's a different endpoint though... so there is going to need to be some default changes | 20:43 |
| *** ijw has quit IRC | 20:43 | |
| cburgess | sdague Is it though? I thought cinder was moving away from version specific endpoints in the catalog? | 20:43 |
| *** kaisers_ has quit IRC | 20:43 | |
| cburgess | sdague Oh but even thing we would have to tack the v3 onto the call to get the new micro-version goodness. | 20:44 |
| *** Drankis has quit IRC | 20:44 | |
| mriedem | i forgot that cinder has a v3 specific endpoint | 20:45 |
| mriedem | which i hate | 20:45 |
| cburgess | mriedem OK | 20:45 |
| sdague | mriedem: yep, I *really* tried to convince folks that was a bad idea :( | 20:45 |
| mriedem | well either way at some level we are setting a flag on startup or first use to tell us if we can do the new hotnes | 20:45 |
| mriedem | *hotness | 20:45 |
| *** tlian has joined #openstack-nova | 20:45 | |
| mriedem | so i guess that means (1) is cinder v3 available, (2) is it a the minimum microversion we need (3) is this compute host, or all compute hosts, at a new enough min version to handle the logic for talking to this new cinder hotness | 20:46 |
| *** fragatina has quit IRC | 20:46 | |
| mriedem | then and only then we can do said new hotness | 20:46 |
| cburgess | mriedem Right so flag to say "talk to v3" then we have to get the attach workflows in the API and compute side using v3 and the correct microversion, then... *profit* | 20:46 |
| sdague | cburgess: so the answer is, yes, have to teach nova about v3. Fortunately it's close enough to v2 that it's not a ton of logic | 20:46 |
| mriedem | yeah i imagine we can hide a lot of the v2 vs v3 stuff in the volume api code | 20:46 |
| mriedem | and abstract it from the rest of nova | 20:46 |
| sdague | cburgess: https://github.com/openstack/nova/blob/ca8d5ff533c1900eaf05c860d446ecccc916d20c/nova/conf/cinder.py#L26 | 20:46 |
| cburgess | mriedem sdague Hopefully yeah | 20:47 |
| sdague | that's the important bit | 20:47 |
| sdague | we use the catalog | 20:47 |
| cburgess | Oh so at least its already an option. So its not a new option at least. | 20:47 |
| sdague | right, but we don't have semantics I think to understand "oh, this is cinder v...." | 20:48 |
| *** ihrachys has joined #openstack-nova | 20:48 | |
| mriedem | don't we still handle cinder v1? | 20:48 |
| *** kaisers_ has joined #openstack-nova | 20:48 | |
| sdague | it's opaqueness send to the client | 20:48 |
| mriedem | https://github.com/openstack/nova/blob/ca8d5ff533c1900eaf05c860d446ecccc916d20c/nova/volume/cinder.py#L90 | 20:48 |
| sdague | mriedem: honestly, I have no idea | 20:48 |
| mriedem | we do | 20:48 |
| cburgess | mriedem So how would the API be able to determine that all compute nodes are "up-to-date enough" to use the new hotness? | 20:49 |
| mriedem | cburgess: we bump the nova-compute service version in the db | 20:49 |
| mriedem | and we have a call to see what the minimum nova-compute service version is in the env | 20:49 |
| *** Guest34488 has quit IRC | 20:49 | |
| mriedem | that's what we do for things like the 2.37 get me a network code | 20:49 |
| *** dave-mccowan has joined #openstack-nova | 20:49 | |
| cburgess | mriedem So when the API service gets this call to do this thing it has to query the min version in the db and do a compairion and uses either old workflow or new hotness? | 20:49 |
| mriedem | yar | 20:49 |
| sdague | mriedem: I do wonder if the v1 stuff actually works at all though, I don't think we've tested it in a long time | 20:50 |
| sdague | we honestly should probably delete it | 20:50 |
| sdague | it's been logging an error for a while | 20:50 |
| cburgess | OK I suppose if there is a prior art for get me a network then it shouldn't be an issue. | 20:50 |
| cburgess | sdague Since Juno even | 20:50 |
| mriedem | cburgess: like this https://github.com/openstack/nova/blob/ca8d5ff533c1900eaf05c860d446ecccc916d20c/nova/api/openstack/compute/servers.py#L574-L577 | 20:50 |
| *** esberglu has quit IRC | 20:51 | |
| *** rcernin has quit IRC | 20:51 | |
| sdague | cburgess: mostly is a bunch of moving parts to make sure everything converges in time, so start early :) | 20:51 |
| *** rcernin has joined #openstack-nova | 20:51 | |
| cburgess | sdague Oh this is going to take several cycles I suspect. | 20:51 |
| *** esberglu has joined #openstack-nova | 20:51 | |
| mriedem | something unrelated but dependent on this new work is just getting nova to handle cinder v3 | 20:51 |
| sdague | cburgess: ok, how close is the cinder API? | 20:52 |
| mriedem | so that could be tackled now on it's own | 20:52 |
| mriedem | sdague: it doesn't exist yet | 20:52 |
| mriedem | it's poc code | 20:52 |
| sdague | aj | 20:52 |
| sdague | ah | 20:52 |
| mriedem | cinder team would like to merge the new apis in ocata though | 20:52 |
| mriedem | and with microversions nova can start using in pike and iterate as needed | 20:52 |
| cburgess | sdague Tragically the far simpler approach was pooped by mriedem so I'm left with this... | 20:52 |
| mriedem | if we missed something | 20:52 |
| sdague | cburgess: yeh, so from planning point of view, I'd say get ahead of v3 infrastructure in nova | 20:52 |
| mriedem | we could switch the placement job to use cinder v3 | 20:53 |
| mriedem | in nova | 20:53 |
| *** kaisers_ has quit IRC | 20:53 | |
| cburgess | sdague mriedem Right so the goal is to merge the cinder side of this for Ocata. I have someone who is starting to work with jgriffith's POC patches now to start trying to figure out what the work in nova looks like but I'm under no delusion that its going to get done this cycle. | 20:53 |
| sdague | because if that's in place this cycle, we could probably even change the default next cycle | 20:53 |
| mriedem | so we'd have v2 and v3 coverage in the gate | 20:53 |
| mriedem | cburgess: have you read through johnthetubaguy's nova spec? | 20:53 |
| mriedem | for using the new cinder apis? | 20:53 |
| sdague | and then adding just that one feature on top of v3 would be small once it is out there (next cycle) | 20:53 |
| mriedem | or your minion i guess | 20:53 |
| *** takashin has joined #openstack-nova | 20:53 | |
| cburgess | mriedem Ahh no so that was going to be my next question which is do we have a spec or existing work to try and leverage v3. | 20:53 |
| *** esberglu_ has joined #openstack-nova | 20:54 | |
| cburgess | mriedem johnthetubaguy linky to cinder v3 API for nova spec? | 20:54 |
| mriedem | cburgess: https://review.openstack.org/#/c/373203/ | 20:55 |
| mriedem | john is on his honeymoon this week btw | 20:55 |
| *** esberglu has quit IRC | 20:56 | |
| *** xinliang has quit IRC | 20:57 | |
| auggy | what's the story with the paramiko issue in nova? i think i missed the memo | 20:57 |
| *** smatzek_ has joined #openstack-nova | 20:57 | |
| mriedem | nova meeting | 21:00 |
| mriedem | rfn | 21:00 |
| *** _ducttape_ has joined #openstack-nova | 21:00 | |
| diana_clarke | auggy: No news, that I know of. It's still a nova dependency. | 21:00 |
| *** smatzek__ has quit IRC | 21:01 | |
| *** lamt has quit IRC | 21:01 | |
| auggy | diana_clarke: ah cool, i noticed you had abandoned your change for managing the dependency | 21:02 |
| auggy | so i didn't know if something was decided or ?? | 21:02 |
| *** xyang1 has joined #openstack-nova | 21:02 | |
| *** mriedem is now known as kristia__ | 21:02 | |
| *** kristian__ is now known as kristia___ | 21:02 | |
| *** kristia__ is now known as mriedem | 21:02 | |
| diana_clarke | auggy: I removed paramiko downstream, but there didn't seem to be desire to remove it upstream, so I abandoned it. | 21:02 |
| *** kristia___ is now known as kristian____ | 21:03 | |
| auggy | diana_clarke: ah cool, i was just noticing there was a new nova bug about paramiko so i was just curious what was happening | 21:03 |
| *** ducttape_ has quit IRC | 21:03 | |
| *** ihrachys has quit IRC | 21:04 | |
| mriedem | diana_clarke: there wasn't desire to remove it from stable branches, or right before releasing newton | 21:04 |
| mriedem | diana_clarke: i thought browne1 was reviewing that change though | 21:04 |
| diana_clarke | auggy: I'm heading out to pick up the kids, so if I don't answer any follow-up questions, I'm not ignoring you ;) | 21:04 |
| *** kaisers_ has joined #openstack-nova | 21:04 | |
| cburgess | mriedem Thanks for the link. | 21:04 |
| diana_clarke | auggy: but I'll take a look at the new bug tonight/tomorrow. | 21:04 |
| auggy | diana_clarke: np thx! | 21:05 |
| browne1 | mriedem: yeah, i was reviewing that one until it was abandoned. i'd like to see paramiko gone, but use py-cryptography instead | 21:05 |
| *** mdrabe has joined #openstack-nova | 21:06 | |
| auggy | browne1: thx for the update | 21:06 |
| *** ihrachys has joined #openstack-nova | 21:08 | |
| *** rcernin has quit IRC | 21:09 | |
| openstackgerrit | Sarafraj Singh proposed openstack/nova: [WIP] Add service_token for nova-cinder interaction https://review.openstack.org/397399 | 21:09 |
| *** wadeholler has joined #openstack-nova | 21:09 | |
| *** kristian__ has joined #openstack-nova | 21:10 | |
| *** liusheng has quit IRC | 21:11 | |
| *** rcernin has joined #openstack-nova | 21:11 | |
| *** liusheng has joined #openstack-nova | 21:12 | |
| diana_clarke | auggy, browne1, mriedem: I just un-abandoned it. https://review.openstack.org/#/c/367395/ I assumed after a month of inactivity & mostly down votes that it was dead. | 21:12 |
| *** xinliang has joined #openstack-nova | 21:13 | |
| browne1 | diana_clarke: ha, i've had patches in review over a year | 21:13 |
| *** siva_krish has joined #openstack-nova | 21:14 | |
| mriedem | diana_clarke: it was a timing issue in newton | 21:14 |
| auggy | this is the bug i was skimming - https://bugs.launchpad.net/nova/+bug/1585515 | 21:14 |
| openstack | Launchpad bug 1585515 in OpenStack Compute (nova) "Paramiko doesn't work with Nova" [Undecided,New] | 21:14 |
| *** lamt has joined #openstack-nova | 21:19 | |
| *** READ10 has joined #openstack-nova | 21:20 | |
| *** timello has quit IRC | 21:22 | |
| *** siva_krish has quit IRC | 21:24 | |
| *** tbachman has joined #openstack-nova | 21:26 | |
| *** rtheis has quit IRC | 21:28 | |
| *** smatzek_ has quit IRC | 21:29 | |
| *** cdent has joined #openstack-nova | 21:32 | |
| *** mvk has joined #openstack-nova | 21:33 | |
| *** takashin has left #openstack-nova | 21:33 | |
| *** Jeffrey4l has quit IRC | 21:35 | |
| *** _ducttape_ has quit IRC | 21:35 | |
| *** ducttape_ has joined #openstack-nova | 21:36 | |
| *** Jeffrey4l has joined #openstack-nova | 21:36 | |
| *** takashin has joined #openstack-nova | 21:38 | |
| *** wadeholler has quit IRC | 21:38 | |
| kristian__ | mriedem: is it possible to overwrite flavor metadata using instance metadata? | 21:39 |
| mriedem | no | 21:40 |
| sdague | auggy: that bug got reopened because mitaka requirements are wrong I think | 21:40 |
| mriedem | flavor extra specs are set by the admin, server metadata is set by the user | 21:40 |
| mriedem | and server metadata is passed through to the config drive or metadata service so the guest can access it | 21:40 |
| auggy | sdague: ah ok, i just remembered the paramiko dependency stuff and assumed it was related | 21:41 |
| sdague | diana_clarke: I don't think folks didn't want to fix that issue, it just came in during tree freeze state | 21:41 |
| *** hamzy has quit IRC | 21:41 | |
| kristian__ | mriedem: oh so manipulating with the hardware like the display source will not work | 21:41 |
| mriedem | kristian__: i don't know the details of what needs to be passed in on server create and what needs to happen when the libvirt driver defines the guest domain xml | 21:42 |
| mriedem | so i can't really say | 21:42 |
| kristian__ | oh ok | 21:42 |
| *** vladikr has quit IRC | 21:42 | |
| *** vladikr has joined #openstack-nova | 21:42 | |
| mriedem | or what, if anything, needs to be configured on the compute host | 21:43 |
| *** dimtruck is now known as zz_dimtruck | 21:44 | |
| *** Sukhdev has quit IRC | 21:47 | |
| *** lamt has quit IRC | 21:48 | |
| *** lamt has joined #openstack-nova | 21:49 | |
| kristian__ | mriedem: its really hard to write that spec for this gpu as a primary display source because it doesnt have any impacts on the end user, just a little for nova and that is with generating the configurations | 21:50 |
| *** kristia__ has joined #openstack-nova | 21:52 | |
| mriedem | kristian__: you don't have to write it for the end user | 21:53 |
| *** zz_dimtruck is now known as dimtruck | 21:53 | |
| kristia__ | mriedem: also look at this. http://www.ibm.com/support/knowledgecenter/linuxonibm/liaat/liaatlibvirtaddpassdev.htm | 21:53 |
| *** eharney has quit IRC | 21:54 | |
| kristia__ | it should be the same like what Im proposing | 21:54 |
| *** jaypipes has quit IRC | 21:54 | |
| mriedem | kristia__: have you read through this? http://docs.openstack.org/admin-guide/compute-pci-passthrough.html | 21:55 |
| kristia__ | yes | 21:56 |
| *** kristian__ has quit IRC | 21:56 | |
| kristia__ | and I got the passthrough to work | 21:56 |
| kristia__ | but geforce experience had problems with installing nvidia drivers | 21:56 |
| kristia__ | and also launching games for testing | 21:57 |
| *** ayoung has quit IRC | 21:57 | |
| *** fragatina has joined #openstack-nova | 21:58 | |
| andreykurilin | mriedem: hi! do you have a minute? | 22:00 |
| *** ijw has joined #openstack-nova | 22:01 | |
| mriedem | andreykurilin: maybe | 22:01 |
| andreykurilin | mriedem: :) | 22:01 |
| andreykurilin | mriedem: I had bad news | 22:01 |
| andreykurilin | mriedem: we have a huge performance degradation | 22:02 |
| mriedem | oh i was going to say, i already knew trump won the election... | 22:02 |
| andreykurilin | lol | 22:03 |
| andreykurilin | mriedem: http://logs.openstack.org/55/407855/4/check/gate-novaclient-dsvm-functional-mitaka-ubuntu-trusty-nv/81209cc/console.html#_2016-12-08_03_56_46_787271 <- here you can find slowest tests at mitaka job and their durations | 22:03 |
| mriedem | i'm assuming something in server create? | 22:03 |
| andreykurilin | http://logs.openstack.org/55/407855/4/check/gate-novaclient-dsvm-functional-identity-v3-only-ubuntu-xenial-nv/0e39770/console.html#_2016-12-08_04_37_22_457376 <- for master | 22:03 |
| *** whenry has joined #openstack-nova | 22:04 | |
| mriedem | yikes | 22:04 |
| openstackgerrit | Andrea Frittoli proposed openstack/nova: Guestfs handle no passwd or group in image https://review.openstack.org/407037 | 22:04 |
| andreykurilin | yeah | 22:04 |
| mriedem | any idea what's causing the crash dump test to be so slow? | 22:05 |
| mriedem | is it specific to the cirros image used? | 22:05 |
| mriedem | b/c that changed recently | 22:05 |
| andreykurilin | I don't know. just noticed the difference in whole durations of jobs and went to see slowest ones... | 22:06 |
| *** slaweq_ has quit IRC | 22:06 | |
| mriedem | well it's obviously something in trigger crash dump, i'm assuming most of the time is spent on that | 22:06 |
| *** slaweq_ has joined #openstack-nova | 22:06 | |
| andreykurilin | I'll add add printing of timings tomorrow... | 22:07 |
| mriedem | andreykurilin: it's weird, because the test waits a max of 1 minute for the trigger_crash_dump instance action to show up | 22:08 |
| *** slaweq_ has quit IRC | 22:08 | |
| mriedem | i'm not even sure how this test is running for 8 minutes without timing out in testr | 22:08 |
| *** ducttape_ has quit IRC | 22:09 | |
| andreykurilin | I thing we do not have any configured timeouts in testr | 22:09 |
| mriedem | yeah i guess not https://github.com/openstack/python-novaclient/blob/master/.testr.conf#L2 | 22:10 |
| kristia__ | mriedem: please may you have a look at this spec? | 22:10 |
| kristia__ | https://gist.github.com/kristiandrucker/eaa863f5526c616f7452ca178fd6d241 | 22:10 |
| openstackgerrit | Alex Szarka proposed openstack/nova: Transform instance.rebuild.error notification https://review.openstack.org/396621 | 22:12 |
| openstackgerrit | Sylvain Bauza proposed openstack/nova: Expose a REST API for a specific list of RPs https://review.openstack.org/392569 | 22:13 |
| openstackgerrit | Sylvain Bauza proposed openstack/nova: Support multiple uuids and names for RP list https://review.openstack.org/407630 | 22:13 |
| *** ducttape_ has joined #openstack-nova | 22:13 | |
| *** slaweq_ has joined #openstack-nova | 22:13 | |
| bauzas | sdague: uploaded a new PS for fixing the functional db test issue ^ | 22:13 |
| bauzas | cdent: edleafe: I saw your comments but could you please clarify what you want to modify ? | 22:14 |
| *** lamt has quit IRC | 22:14 | |
| *** tbachman_ has joined #openstack-nova | 22:15 | |
| *** tbachman has quit IRC | 22:15 | |
| *** tbachman_ is now known as tbachman | 22:15 | |
| *** gouthamr has quit IRC | 22:15 | |
| *** esberglu_ has left #openstack-nova | 22:17 | |
| *** esberglu_ has joined #openstack-nova | 22:17 | |
| *** thorst_ has quit IRC | 22:17 | |
| *** jheroux has quit IRC | 22:17 | |
| *** fragatina has quit IRC | 22:18 | |
| openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient: DO NOT MERGE: add timings for test_trigger_crash_dump_in_rescued_state https://review.openstack.org/408832 | 22:20 |
| *** annegentle has joined #openstack-nova | 22:20 | |
| *** ducttape_ has quit IRC | 22:21 | |
| *** tlian has quit IRC | 22:23 | |
| *** lpetrut has joined #openstack-nova | 22:26 | |
| *** smatzek_ has joined #openstack-nova | 22:26 | |
| *** slaweq_ has quit IRC | 22:27 | |
| *** mriedem has quit IRC | 22:29 | |
| *** kristia__ has quit IRC | 22:29 | |
| *** slaweq_ has joined #openstack-nova | 22:29 | |
| *** jamielennox is now known as jamielennox|away | 22:29 | |
| *** nherciu has quit IRC | 22:34 | |
| *** jamielennox|away is now known as jamielennox | 22:34 | |
| *** baoli has quit IRC | 22:36 | |
| *** ijw has quit IRC | 22:38 | |
| *** ijw has joined #openstack-nova | 22:39 | |
| *** fragatina has joined #openstack-nova | 22:41 | |
| *** sacharya has quit IRC | 22:42 | |
| *** fragatin_ has joined #openstack-nova | 22:42 | |
| *** cdent has quit IRC | 22:43 | |
| *** slagle has quit IRC | 22:44 | |
| *** ijw has quit IRC | 22:44 | |
| *** rnoriega has quit IRC | 22:44 | |
| *** Prophidys has quit IRC | 22:45 | |
| *** jschlueter has quit IRC | 22:45 | |
| *** fragatina has quit IRC | 22:46 | |
| *** ijw has joined #openstack-nova | 22:48 | |
| *** jwcroppe has quit IRC | 22:49 | |
| *** READ10 has quit IRC | 22:50 | |
| *** ijw has quit IRC | 22:53 | |
| andreykurilin | bauzas: hi! | 22:54 |
| *** esberglu_ has quit IRC | 22:57 | |
| andreykurilin | bauzas: several months ago you put +2 to one of my patches at novaclient. it wasn't merged yet and it is rebased now. If you have a bit of free time, I will be glad if you re-review it - https://review.openstack.org/#/c/350140 | 22:58 |
| andreykurilin | bauzas: if you have more than a bit of free time, I'll be happy if you review the whole chain of patches(they quite simple) :) | 22:59 |
| *** pradk has quit IRC | 23:00 | |
| andreykurilin | there are just 15 patches. lol | 23:00 |
| *** david-lyle_ has joined #openstack-nova | 23:05 | |
| *** Sukhdev has joined #openstack-nova | 23:07 | |
| *** lpetrut has quit IRC | 23:07 | |
| *** david-lyle has quit IRC | 23:07 | |
| *** annegentle has quit IRC | 23:08 | |
| *** rcernin has quit IRC | 23:11 | |
| *** tbachman has quit IRC | 23:18 | |
| *** jgriffith is now known as jgriffith_AutoAw | 23:18 | |
| *** xyang1 has quit IRC | 23:19 | |
| *** smatzek__ has joined #openstack-nova | 23:21 | |
| *** ayoung has joined #openstack-nova | 23:24 | |
| *** smatzek_ has quit IRC | 23:24 | |
| *** ljxiash has joined #openstack-nova | 23:30 | |
| *** ntpttr has quit IRC | 23:31 | |
| *** ntpttr has joined #openstack-nova | 23:31 | |
| *** WillUK2010 has joined #openstack-nova | 23:32 | |
| *** ljxiash_ has quit IRC | 23:32 | |
| *** rcernin has joined #openstack-nova | 23:33 | |
| openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient: DO NOT MERGE: add timings for test_trigger_crash_dump_in_rescued_state https://review.openstack.org/408832 | 23:33 |
| openstackgerrit | Sarafraj Singh proposed openstack/nova: WIP: remove uses of network in _update_ports_for_instance https://review.openstack.org/405399 | 23:34 |
| *** david-lyle_ is now known as david-lyle | 23:35 | |
| *** erlon-airlong has quit IRC | 23:40 | |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova: Move tags validation code to json scehme https://review.openstack.org/408424 | 23:42 |
| *** liverpooler has quit IRC | 23:42 | |
| *** sacharya has joined #openstack-nova | 23:42 | |
| *** jgriffith_AutoAw is now known as jgriffith | 23:44 | |
| *** zhenq has quit IRC | 23:46 | |
| *** zhenq has joined #openstack-nova | 23:46 | |
| *** esberglu has joined #openstack-nova | 23:47 | |
| *** sacharya has quit IRC | 23:47 | |
| *** rcernin has quit IRC | 23:49 | |
| *** ijw has joined #openstack-nova | 23:49 | |
| *** jamielennox is now known as jamielennox|away | 23:49 | |
| *** hongbin has quit IRC | 23:50 | |
| *** yamahata has quit IRC | 23:51 | |
| *** esberglu has quit IRC | 23:51 | |
| *** ijw has quit IRC | 23:52 | |
| *** tbachman has joined #openstack-nova | 23:53 | |
| *** ijw has joined #openstack-nova | 23:53 | |
| *** ljxiash has quit IRC | 23:57 | |
| *** sdague has quit IRC | 23:59 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!