| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-04-24 | |||
| 15:58:08 | sfinucan | (y) | |
| 15:58:39 | ralonsoh | ralonsoh: in a moment | |
| 15:59:04 | ralonsoh | jaypipes: in a moment (pfff I'm talking to myself) | |
| 15:59:20 | sean-k-mooney | jaypipes: running to meeting but will do after | |
| 16:01:32 | sfinucan | jaypipes: nice easy ones for you here too https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:master+topic:bump-min-libvirt | |
| 16:04:01 | edleafe | jaypipes: ack | |
| 16:04:49 | jaypipes | sfinucan: done. | |
| 16:05:09 | sfinucan | ta | |
| 16:16:44 | openstackgerrit | Stephen Finucane proposed openstack/os-vif master: Add documentation for OVS plugin https://review.openstack.org/459258 | |
| 16:16:45 | openstackgerrit | Stephen Finucane proposed openstack/os-vif master: HW offload support for openvswitch https://review.openstack.org/398277 | |
| 16:16:47 | openstackgerrit | Stephen Finucane proposed openstack/os-vif master: Add documentation for Linux Bridge plugin https://review.openstack.org/459291 | |
| 16:27:29 | mriedem | sfinucan: nice easy cleanup win here https://review.openstack.org/#/c/391597/ | |
| 16:27:41 | mriedem | and closes a bp to boot | |
| 16:45:37 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Remove BuildRequest.block_device_mapping clone workaround https://review.openstack.org/459406 | |
| 16:47:06 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Remove BuildRequest.block_device_mapping clone workaround https://review.openstack.org/459406 | |
| 16:48:21 | mriedem | dansmith: melwitt: ^ dredged up a lot of confusion over why we needed to do that in the first place https://review.openstack.org/#/c/394321/36/nova/compute/api.py@1056 | |
| 16:48:50 | mriedem | i don't want to do the same thing for tags | |
| 16:49:07 | mriedem | i think we just pass the original dirty tag list to conductor like we do for bdms and we should be fine | |
| 17:02:51 | cuhler | Is it a known problem that nova.tests.unit.virt.libvirt.volume.test_iscsi.LibvirtISCSIVolumeDriverTestCase.test_iscsiadm_discover_parsing is failing? | |
| 17:02:57 | cuhler | It is holding up https://review.openstack.org/#/c/458688/ | |
| 17:03:09 | cuhler | Is someone looking at it already? | |
| 17:06:21 | cuhler | The problem is that LibvirtISCSIVolumeDriver connector's _get_target_portals_from_iscsiadm_output method now returns a list instead of an array as it used to. | |
| 17:06:35 | cuhler | Either that's a bug or the unit test didn't get updated. | |
| 17:18:33 | dansmith | mriedem: do you plan to write tests for that cleanup patch? | |
| 17:18:49 | dansmith | doesn't fail anything now and it'd be an odd assertion in isolation I guess | |
| 17:19:13 | openstackgerrit | Chris Dent proposed openstack/nova master: Add pbr-installed wsgi application for metadata api https://review.openstack.org/459413 | |
| 17:34:54 | rfolco | jaypipes, start using new delete inventories method implies in chaning put to delete in https://github.com/openstack/nova/blob/master/nova/scheduler/client/report.py#L625 - sounds correct ? I am just wondering where is the delete_inventory for a single inventory. | |
| 17:38:31 | sean-k-mooney | jaypipes: https://review.openstack.org/#/c/448283/5 looks good to me in it current form | |
| 17:38:51 | mriedem | dansmith: i didn't plan on it no | |
| 17:39:04 | mriedem | functional tests passing should be sufficient | |
| 17:40:07 | dansmith | okay | |
| 17:41:04 | mriedem | dansmith: unrelated, but looking at https://review.openstack.org/#/c/457338/ i didn't even think about how server groups could have members in different cells | |
| 17:41:18 | mriedem | at least for anti-affinity | |
| 17:41:30 | dansmith | yup | |
| 17:55:27 | mriedem | dansmith: ok comments in that patch | |
| 17:55:31 | mriedem | -1 for a test thing | |
| 17:55:57 | dansmith | thanks, will look in a sec | |
| 17:57:42 | edleafe | rfolco: there isn't a way to delete a single type of inventory for a RP. You should use set_inventory_for_provider() instead, and only include the inventory for the types that the RP has. | |
| 18:00:18 | rfolco | edleafe, that explains, thank you. Also, I am using a specific error message (previously returning db api error) when 409 conflict for inventory in use... but there is no way to emulate it in gabbits since they run serially. The only emulation possible is bad generation id. Any thoughts ? | |
| 18:00:30 | sdague | mriedem: you were too hopeful that the uuid thing wasn't a problem :) | |
| 18:01:45 | edleafe | rfolco: you would allocate against that inventory, checking for success. Then the next test would try to delete that same inventory, which should raise the 409 | |
| 18:02:12 | mriedem | sdague: the jsonschema thing? | |
| 18:02:18 | sdague | mriedem: yep | |
| 18:02:21 | mriedem | i was looking for an easy button | |
| 18:02:31 | sdague | I responded | |
| 18:03:25 | rfolco | edleafe, cool, will check on allocation then, thx++ | |
| 18:03:27 | sdague | the format isn't jsonschema native, it just means it calls a dedicated validating function | |
| 18:05:05 | mriedem | sdague: ha | |
| 18:05:06 | mriedem | sob | |
| 18:06:00 | mriedem | i see your reply and raise you | |
| 18:06:45 | sdague | mriedem: yeh, we all see different parts of the elephant | |
| 18:14:51 | openstackgerrit | Matt Riedemann proposed openstack/nova master: libvirt: remove redundant and broken iscsi volume test https://review.openstack.org/459420 | |
| 18:14:53 | mriedem | ^ fixes cuhler's issue pointed out earlier, and unblocks us from using brick 1.12.0 | |
| 18:15:32 | smcginnis | Following the docs.o.o install instructions for Ubuntu. Getting this error :The requested URL /placement/resource_providers was not found on this server" | |
| 18:15:35 | smcginnis | Any pointers? | |
| 18:16:02 | mriedem | smcginnis: hello new user, please see the channel topic and gtfo | |
| 18:16:09 | mriedem | :) | |
| 18:16:23 | mriedem | smcginnis: where are you getting the error? which service? | |
| 18:16:29 | smcginnis | mriedem: Just remember that when you want us to adopt placement usage too. :P | |
| 18:16:51 | smcginnis | mriedem: In the nova-compute log when trying to create a first test instance. | |
| 18:17:07 | smcginnis | Actually, regular intervals it looks like. | |
| 18:17:10 | mriedem | if compute couldn't talk to placement it shouldn't have started up in the first place | |
| 18:17:15 | mriedem | yeah, it should be spewing errors | |
| 18:17:28 | mriedem | are you able to make curl requests to placement from your compute node? | |
| 18:17:33 | smcginnis | From this: Failed to create resource provider record in placement API | |
| 18:17:59 | mriedem | smcginnis: source the same credentials that are in nova.conf on your compute host, and then create a token for those creds and try a curl request to the same placement endpoint url in the service catalog | |
| 18:18:12 | mriedem | to see if it's compute misconfig or something else | |
| 18:18:27 | mriedem | also, is the placement service logging anything? | |
| 18:18:30 | mriedem | it runs under apache | |
| 18:18:45 | smcginnis | Where does that go? | |
| 18:19:08 | mriedem | /var/log/httpd? i think it has to be configured | |
| 18:19:15 | mriedem | devstack puts it into it's own log file | |
| 18:19:23 | smcginnis | Ah, /var/log/apache/nova_placement* | |
| 18:19:49 | smcginnis | Hmm, "nova" placement. Yeah, good luck getting me to use this. | |
| 18:19:52 | smcginnis | jk | |
| 18:19:57 | mriedem | https://github.com/openstack-dev/devstack/blob/master/lib/placement#L168 | |
| 18:20:20 | mriedem | yeah, i think the ubuntu packagers were throwing nova in the name since that's where the code is sourced from | |
| 18:20:29 | mriedem | which is going to be a fun packaging rename later | |
| 18:20:41 | smcginnis | mriedem: Actual deployment looks like it puts them in nova_placement_error.log and nova_placement_access.log on Ubuntu. | |
| 18:21:00 | smcginnis | Nothing in either file though. | |
| 18:21:07 | mriedem | is it running? | |
| 18:22:10 | smcginnis | mriedem: Nope. Looks like the packages must not configure Apache correctly. | |
| 18:22:13 | smcginnis | mriedem: Loaded: not-found (Reason: No such file or directory) | |
| 18:22:25 | smcginnis | OK, go the clue I needed. I'll dig from here. | |
| 18:22:31 | smcginnis | Taking it this is not a known issue though. | |
| 18:22:32 | mriedem | i had heard of people complaining that the ubuntu packages were messed up | |
| 18:22:45 | mriedem | at least while they were working on the docs, | |
| 18:22:57 | mriedem | if there was a known issue, i'd think it would have been capture in the install guide though, but i've been out of the loop there | |
| 18:23:02 | mriedem | cdent might know but he's gone | |
| 18:23:21 | smcginnis | mriedem: If I find anything I'll try to update the docs. Thanks. | |
| 18:23:25 | smcginnis | :q | |
| 18:30:23 | mriedem | sdague: can we get this into stable/ocata to unblock the LM job there? https://review.openstack.org/#/c/458843/ | |
| 18:35:27 | openstackgerrit | Dan Smith proposed openstack/nova master: WIP: De-complicate some of the instance delete path https://review.openstack.org/458636 | |
| 18:35:27 | openstackgerrit | Dan Smith proposed openstack/nova master: Make server groups api aware of multiple cells for membership https://review.openstack.org/457338 | |
| 18:35:28 | openstackgerrit | Dan Smith proposed openstack/nova master: WIP: Move to proper target_cell calling convention https://review.openstack.org/458634 | |
| 18:35:28 | openstackgerrit | Dan Smith proposed openstack/nova master: Make target_cell() yield a new context https://review.openstack.org/458635 | |
| 18:35:29 | openstackgerrit | Dan Smith proposed openstack/nova master: Sort CellMappingList.get_all() for safety https://review.openstack.org/443174 | |
| 18:35:29 | openstackgerrit | Dan Smith proposed openstack/nova master: Clean up ClientRouter debt https://review.openstack.org/444487 | |
| 18:35:30 | openstackgerrit | Dan Smith proposed openstack/nova master: DNM: Canary test against multi-cell devstack https://review.openstack.org/458537 | |
| 18:35:36 | openstackgerrit | Dan Smith proposed openstack/nova master: Add workaround to disable group policy check upcall https://review.openstack.org/442736 | |