| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-01-04 | |||
| 23:37:21 | mnaser | (cant force delete an attach volume) | |
| 23:37:50 | mriedem | mnaser: https://developer.openstack.org/api-ref/block-storage/v2/#force-detach-volume | |
| 23:38:22 | mordred | wow. y'all are having all the fun | |
| 23:38:39 | mriedem | mordred: this channel is a barrel of laughs all day every day | |
| 23:38:52 | mordred | mriedem: that's what I tell people | |
| 23:39:15 | mordred | "looking for a barrel of laughs? go check out #openstack-nova!", I tell them | |
| 23:40:19 | mnaser | mriedem: interesting, i could probably even normal detach, but i wonder if the cinder cli client has it | |
| 23:41:01 | mnaser | hopefully scaling up our cinder api endpoints should mean that we don't see this again | |
| 23:41:32 | mriedem | i don't see a force detach command in cinderclient | |
| 23:46:22 | SamYaple | i dont think there is one... | |
| 23:46:50 | SamYaple | inconsitencies between nova and cinder ive always had to go to the db to solve | |
| 23:47:03 | mriedem | there is an api | |
| 23:47:05 | mriedem | so curl should work | |
| 23:47:20 | mriedem | should also be easy enough to add a cli for this in cinderclient | |
| 23:47:38 | SamYaple | indeed | |
| #openstack-nova - 2018-01-05 | |||
| 00:00:54 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: [placement] Add sending global request ID in put (2) https://review.openstack.org/531266 | |
| 00:17:24 | openstackgerrit | Matt Riedemann proposed openstack/python-novaclient master: Microversion 2.58 - Instance actions list pagination https://review.openstack.org/528601 | |
| 00:29:02 | mriedem | alex_xu: takashin: important question in https://review.openstack.org/#/c/330406/78/api-ref/source/parameters.yaml@4148 | |
| 00:29:19 | mriedem | this microversion adds a uuid field to the response for GET /os-migrations, | |
| 00:29:42 | mriedem | i suggested we could also take this opportunity to add uuid to the response for GET /servers/{uuid}/migrations as well | |
| 00:29:55 | mriedem | thoughts on that? | |
| 00:30:35 | mriedem | comment on the review if you like, i have to make dinner | |
| 01:29:56 | openstackgerrit | Merged openstack/nova master: Imported Translations from Zanata https://review.openstack.org/524795 | |
| 02:34:35 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: [placement] Add sending global request ID in put (3) https://review.openstack.org/531280 | |
| 03:51:14 | openstackgerrit | Hironori Shiina proposed openstack/nova master: virt/ironic: Implement rescue and unrescue https://review.openstack.org/416487 | |
| 05:46:57 | openstackgerrit | Andreas Jaeger proposed openstack/os-vif master: zuul: Enable functional tests in gate https://review.openstack.org/530961 | |
| 06:10:59 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: [placement] Add sending global request ID in get https://review.openstack.org/531299 | |
| 06:53:28 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Remove a duplicate line in a unit test https://review.openstack.org/531317 | |
| 07:04:05 | openstackgerrit | Yikun Jiang (Kero) proposed openstack/nova master: Fix 500 in test_resize_server_negative_invalid_state https://review.openstack.org/531117 | |
| 07:24:20 | openstackgerrit | Yikun Jiang (Kero) proposed openstack/nova master: Fix 500 in test_resize_server_negative_invalid_state https://review.openstack.org/531117 | |
| 07:37:10 | openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: Add ComputeNode.virt_type field https://review.openstack.org/531327 | |
| 07:37:11 | openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: Use virt_type in /os-hypervisors/details API https://review.openstack.org/531328 | |
| 07:37:16 | openstackgerrit | Chen Hanxiao proposed openstack/nova master: conf: hyperv: fix a comment typo https://review.openstack.org/531329 | |
| 07:48:34 | openstackgerrit | Marcin Juszkiewicz proposed openstack/nova master: libvirt: use 'host-passthrough' as default on AArch64 https://review.openstack.org/530965 | |
| 07:49:01 | hrw | stephenfin: added test for ^^ - can you take a look? | |
| 08:09:57 | openstackgerrit | Yikun Jiang (Kero) proposed openstack/nova master: Add index(instance_uuid, updated_at) on instance_actions table https://review.openstack.org/530429 | |
| 08:14:09 | kashyap | hrw: Oh, yes, I thought of the test in your change, but forgot to mention it. Cool that you've worked it out. | |
| 08:14:48 | hrw | kashyap: I was aware that test is needed. Just had no idea how to write it. Kevin wrote it for me. | |
| 08:15:16 | kashyap | I see, ah, missed the co-author bit. | |
| 08:15:32 | kashyap | hrw: I myself struggle with the unit tests as I don't look at it terribly often | |
| 08:15:52 | hrw | kashyap: Kevin knows Nova code. I don't | |
| 08:28:49 | hrw | mailed ML | |
| 08:57:50 | openstackgerrit | Yikun Jiang (Kero) proposed openstack/nova master: Add pagination and Changes-since filter support for os-migrations. https://review.openstack.org/330406 | |
| 09:18:33 | stephenfin | hrw: Sure | |
| 09:19:28 | hrw | stephenfin: thanks. | |
| 09:19:53 | hrw | stephenfin: hope that someone will take a look too. Not adding reviewers as I do not know who to add | |
| 09:29:15 | openstackgerrit | ChangBo Guo(gcb) proposed openstack/nova master: Use method validate_integer from oslo.utils https://review.openstack.org/528620 | |
| 09:50:42 | hrw | stephenfin: by 'link to your blog' you mean https://marcin.juszkiewicz.com.pl/2018/01/04/today-i-was-fighting-with-nova-no-idea-who-won/ link? | |
| 09:50:56 | openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: Fix hypervisor_type in /os-hypervisors/details https://review.openstack.org/531328 | |
| 09:50:57 | openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: [libvirt] Filter hypervisor_type by virt_type https://review.openstack.org/531347 | |
| 09:51:07 | stephenfin | hrw: Yup | |
| 09:51:14 | hrw | ok | |
| 09:53:19 | openstackgerrit | Marcin Juszkiewicz proposed openstack/nova master: libvirt: use 'host-passthrough' as default on AArch64 https://review.openstack.org/530965 | |
| 09:53:23 | hrw | reno updated, link added | |
| 09:54:27 | kashyap | hrw: Thanks for the quick feedback | |
| 09:54:35 | kashyap | s/feedback/response/ | |
| 09:55:24 | hrw | kashyap: it's my job ;D | |
| 09:55:58 | hrw | kashyap: and I prefer to have patches quickly updated to keep reviewers attention | |
| 09:56:12 | kashyap | hrw: Is this pressing? If so you have to state it, so reviewers calibrate their queue accordingly :-) | |
| 09:56:31 | kashyap | Those who can +2, i.e. | |
| 09:56:53 | hrw | kashyap: it would be lovely to get both my patches into Queens. That level of priority. | |
| 09:57:17 | kashyap | Yeah, it should. Upstream is not _that_ terribly slow for self-contained stuff like this. | |
| 09:57:34 | kashyap | This is a straight-forward one. No delay should be done in merging this. | |
| 09:57:50 | hrw | We have workaround for cpu_mode already merged into Kolla and I know that it is present in some other ways of deploying too. | |
| 09:59:09 | hrw | UEFI one can not be workarounded | |
| 10:01:48 | hrw | added some random core reviewers to both patches | |
| 10:03:43 | sean-k-mooney | stephenfin: o/ happy new year | |
| 10:03:51 | stephenfin | sean-k-mooney: o/ ditto | |
| 10:04:05 | sean-k-mooney | see my comment on https://review.openstack.org/#/c/530961/2 | |
| 10:04:35 | stephenfin | sean-k-mooney: Hahaha | |
| 10:04:41 | stephenfin | This is what I'm currently reading | |
| 10:04:46 | stephenfin | https://docs.python.org/2/library/subprocess.html#subprocess.Popen.communicate | |
| 10:05:04 | stephenfin | I was trying to capture stderr/stdout to see if that was indeed the issue (I suspected it was) | |
| 10:06:53 | stephenfin | sean-k-mooney: I'm thinking we just configure that to 'python -m privsep.[path_to_module]' | |
| 10:07:33 | sean-k-mooney | stephenfin: the test rodoflo is adding will need at least cap_net_admin to run so it will need to launch privesp with sudo to get that | |
| 10:09:19 | stephenfin | Right, I meant 'sudo python -m' as I thought we'd keep the virtualenv-configured Python, but we obviously won't actually | |
| 10:09:22 | sean-k-mooney | stephenfin: i think we can alter the config fuction here https://review.openstack.org/#/c/484386/30/os_vif/tests/functional/base.py | |
| 10:10:40 | sean-k-mooney | stephenfin: if we addin the a override for the oslo privsep helper_command form sudo privsep-helper to sudo -EH privsep-helper its shoudl resolve the issue | |
| 10:11:38 | sean-k-mooney | technicall i theink just the -E is need to keep the virtual env intact but i always use -EH just to be safe | |
| 10:12:08 | sean-k-mooney | my spelling is on point as always... | |
| 10:21:21 | stephenfin | sean-k-mooney: Turns out PATH isn't passed through via '-E'. Something about secure_path. Who knew! | |
| 10:32:16 | sean-k-mooney | stephenfin: really ok... im pulling down the patch in a clean vm and ill see if i can reporduce the gate issue | |
| 10:35:24 | sean-k-mooney | yep same issue | |
| 10:36:16 | sean-k-mooney | and the privsep-helper is in .tox/functional/bin/privsep-helper as i taught but not in /bin | |
| 10:38:05 | stephenfin | sean-k-mooney: This is as far as I've got http://paste.openstack.org/show/638995/ | |
| 10:40:14 | stephenfin | sean-k-mooney: But that's yielding this, ostensibly because we're using absolute paths to a package that is only installed in the virtualenv http://paste.openstack.org/show/638997/ | |
| 10:40:29 | sean-k-mooney | that worked for me | |
| 10:40:39 | stephenfin | It did? | |
| 10:40:45 | stephenfin | Failing for me on CentOS :( | |
| 10:40:48 | sean-k-mooney | well it passed all the tests | |
| 10:41:40 | sean-k-mooney | http://paste.openstack.org/show/638998/ | |
| 10:42:45 | stephenfin | How weird | |
| 10:43:07 | openstackgerrit | sahid proposed openstack/nova master: libvirt: disconnect volume from host during detach https://review.openstack.org/515008 | |
| 10:43:08 | stephenfin | You've other modifications though, to say that you have logging | |
| 10:43:32 | sean-k-mooney | no i just applied what you gave | |
| 10:43:52 | sean-k-mooney | http://paste.openstack.org/show/638999/ | |
| 10:44:16 | sean-k-mooney | that was applied directly on top of your zuul patch | |
| 10:44:20 | sean-k-mooney | 15264957c475a6ecc1648a9ca2f04c9beb95f98d | |
| 10:45:10 | sean-k-mooney | i can spin up a centos vm and test it there again if that helps | |
| 10:45:35 | stephenfin | If you wouldn't mind | |