| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-01-31 | |||
| 19:35:37 | edleafe | mriedem: you wear pants? | |
| 19:35:45 | mriedem | still do yup | |
| 19:35:48 | mriedem | and shave and get haircuts | |
| 19:35:54 | mriedem | hasn't been a year of working at home yet | |
| 19:35:59 | dansmith | mriedem: I was going to say.. my pants still get laid out for me for important pants-wearing occasions | |
| 19:36:26 | edleafe | that's why I only have one pair. Prevents all the pants-related arguments | |
| 19:37:12 | dansmith | business in the front, party in the back? | |
| 19:38:27 | dansmith | not the kind that require specific pant selection, that's for sure | |
| 20:02:01 | openstackgerrit | Ameed Ashour proposed openstack/nova master: add "--until-complete" option for nova-manage db archive_deleted_rows man page https://review.openstack.org/539654 | |
| 20:05:14 | openstackgerrit | Matt Riedemann proposed openstack/nova master: doc: placement upgrade notes for queens https://review.openstack.org/539658 | |
| 20:09:36 | mriedem1 | need a final +2 on the reno for versioned notifications new in queens https://review.openstack.org/#/c/518018/ | |
| 20:11:30 | mriedem | cfriesen: did your perf testing show any issues with the counting quotas stuff that was new in pike? | |
| 20:24:19 | cfriesen | mriedem: don't know if we've gotten that far. I'm pretty sure it was on a mostly idle system so I wouldn't expect that to be a major factor. | |
| 20:35:10 | openstackgerrit | Ameed Ashour proposed openstack/nova master: detach instance volumes when VM creation fails https://review.openstack.org/528385 | |
| 20:37:58 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Re-arrange cells docs for modern deployments https://review.openstack.org/539674 | |
| 20:38:25 | mriedem | dansmith: melwitt: we had talked at one point a few weeks ago about re-arranging the cells docs to move the cells v2 setup stuff to the top since that's mostly what people care about now when upgrading to ocata ^ | |
| 20:38:47 | mriedem | tried to move that stuff around and still leave the historical manifesto and cells v1 stuff in there to still make sense | |
| 20:39:18 | melwitt | ack | |
| 20:44:37 | efried | mriedem: "the in-tree powervm driver doesn't support gpu passthrough right?" Not yeeeeeetttt.... | |
| 20:49:00 | hrw | does someone uses hw_pointer_model option? with something else than 'usbtablet' there? | |
| 20:49:38 | hrw | and if you do then does it work for you? | |
| 21:05:35 | ameeda | mriedem: replied | |
| 21:05:52 | mriedem | hrw: i thought it was end of day for you? | |
| 21:07:05 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Re-arrange cells docs for modern deployments https://review.openstack.org/539674 | |
| 21:17:18 | mriedem | dansmith: can you hit this backport so i can get the one below it? https://review.openstack.org/#/c/527560/ | |
| 21:17:24 | ameeda | mriedem: please check the patch there is important comment there https://review.openstack.org/#/c/507486 | |
| 21:18:14 | dansmith | mriedem: doneski | |
| 21:23:09 | mriedem | ameeda: i'm not sure what you want me to look at in that change | |
| 21:24:55 | mriedem | ugh i guess we can't backport this fix now https://review.openstack.org/#/c/524755/4/nova/objects/host_mapping.py@152 | |
| 21:25:47 | mriedem | even though nothing uses it over rpc? | |
| 21:27:49 | dansmith | mriedem: well like I said I don't think that needed to be a bump anyway | |
| 21:28:03 | dansmith | mriedem: you could backport without the version bump and be fine I think.. seems scary, but should be alright | |
| 21:28:37 | openstackgerrit | Claudiu Belu proposed openstack/nova master: hyper-v: autospec classes before they are instantiated https://review.openstack.org/342211 | |
| 21:32:05 | mriedem | efried: see what you think about my 'experimental' wording idea in https://review.openstack.org/#/c/539266/ | |
| 21:32:43 | mriedem | dansmith: might not be necessary, i was just thinking about it b/c the cells faq page says you can use this to list hosts in a cell, but there are other options for doing that as well, so not super important | |
| 21:36:57 | ameeda | mriedem: Surya Seetharaman -> The same change of adding "--until-complete" is been done here I think ? : https://review.openstack.org/#/c/507486/29/doc/source/cli/nova-manage.rst | |
| 21:37:17 | ameeda | mriedem: I will upload new patch now | |
| 21:37:59 | mriedem | ameeda: your patch is much more targeted to just updating the man page | |
| 21:38:15 | openstackgerrit | Ameed Ashour proposed openstack/nova master: add "--until-complete" option for nova-manage db archive_deleted_rows. https://review.openstack.org/539654 | |
| 21:38:57 | ameeda | mriedem: I uploaded new patch. | |
| 21:39:56 | ameeda | mriedem: it seems that I shouldn't write tab when I add description ? | |
| 21:41:16 | efried | mriedem: Responded. Thanks. | |
| 21:41:20 | openstackgerrit | Ameed Ashour proposed openstack/nova master: add "--until-complete" option for nova-manage db archive_deleted_rows. https://review.openstack.org/539654 | |
| 21:41:34 | mriedem | ameeda: everything should be spaces, not tabs | |
| 21:42:35 | ameeda | mriedem: oh, ok I will upload new patch, sorry | |
| 21:46:07 | openstackgerrit | Ameed Ashour proposed openstack/nova master: add "--until-complete" option for nova-manage db archive_deleted_rows. https://review.openstack.org/539654 | |
| 21:47:13 | ameeda | mriedem: uploaded (Y) | |
| 21:52:00 | ameeda | mriedem: Thank you!! now I need to get Ocata release and add the change for it ? I think that I has pike version right ? | |
| 21:53:20 | mriedem | ameeda: you're on the master branch, which is currently queens | |
| 21:53:31 | mriedem | ameeda: review process is generally 2 +2s to approve a patch | |
| 21:53:43 | mriedem | ameeda: so you might as well hold off until someone actually approves the change | |
| 21:55:05 | efried | mriedem: FYI, I seem to have a decent delay in receiving email from the mailing lists. (By way of explaining why I appeared to say the same thing you had "already" said.) | |
| 21:55:54 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Provide support matrix and doc for VGPU https://review.openstack.org/539266 | |
| 21:56:51 | mriedem | efried: that is no excuse, you need to do better at your precognition of issues to come up in the ML in order to beat me to the punch | |
| 21:57:04 | efried | clearly | |
| 21:57:06 | mriedem | better call tom cruise | |
| 21:57:19 | efried | or that bald chick | |
| 21:59:26 | ameeda | mriedem: its okay, Thank you for your time and helps. | |
| 21:59:43 | ameeda | mriedem: please don't forget to review this for me https://review.openstack.org/#/c/528385/ | |
| 22:01:29 | mriedem | ameeda: at this point i dream about the opportunity to review that patch :) | |
| 22:01:33 | mriedem | it is my destiny | |
| 22:05:12 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Remove old flavor_access_get_by_flavor_id db api method https://review.openstack.org/539700 | |
| 22:05:52 | ameeda | mriedem: lol, I appreciate that from you,I know that its hard to you to find time to check all reviews | |
| 22:06:38 | ameeda | mriedem: lastly I see +2 on one of my patches ^_^, lol | |
| 22:07:44 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Remove old flavor_extra_specs_delete db api method https://review.openstack.org/539702 | |
| 22:13:30 | melwitt | mriedem: on this backport, we're only concerned about holding it from ocata because of the possible related network-vif-plugged event timeout, but not for pike? https://review.openstack.org/#/c/531422 | |
| 22:13:53 | mriedem | melwitt: i am concerned for pike, but we have more time for pike | |
| 22:13:56 | mriedem | so less concerned | |
| 22:14:01 | melwitt | okay | |
| 22:14:27 | mriedem | i thought about dropping my +2 | |
| 22:14:32 | mriedem | but i didn't think about it very hard | |
| 22:14:46 | melwitt | fair enough | |
| 22:15:27 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Remove old flavor_extra_specs_delete db api method https://review.openstack.org/539702 | |
| 22:15:28 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Remove old flavor_extra_specs_get db api method https://review.openstack.org/539706 | |
| 22:17:45 | Roamer` | hm, has anybody reported problems with the "libvirt: QEMU native LUKS decryption for encrypted volumes" change recently? | |
| 22:18:36 | melwitt | not that I know of. have you experienced problems with it? | |
| 22:18:50 | Roamer` | the StorPool Cinder CI started failing each and every test run with the same problem: test_encrypted_cinder_volumes_luks fails and there is a nova-compute exception: Failed to detach volume 645fd643-89fc-4b3d-9ea5-59c764fc39a2 from /dev/vdb: AttributeError: 'NoneType' object has no attribute 'format_dom' | |
| 22:19:08 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Remove old flavor_extra_specs_delete db api method https://review.openstack.org/539702 | |
| 22:19:09 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Remove old flavor_extra_specs_get db api method https://review.openstack.org/539706 | |
| 22:19:11 | Roamer` | just a minute, let me point you at a stack trace | |
| 22:19:57 | Roamer` | http://logs.ci-openstack.storpool.com/18/539318/1/check/dsvm-tempest-storpool/c3daf58/logs/screen-n-cpu.txt.gz#_Jan_31_18_07_27_971552 and a couple of lines further down | |
| 22:20:50 | Roamer` | melwitt: ^^ - it looks like the LibvirtConfigGuestDiskEncryption is missing a "secret" (not initialized; maybe not passed somewhere?) | |
| 22:20:56 | Roamer` | and this happens on every run | |
| 22:21:30 | melwitt | Roamer`: okay. could you please open a bug against nova with a link to the log and excerpt of the stack trace? | |
| 22:21:54 | Roamer` | melwitt, sure, I'll open it right now | |
| 22:21:56 | melwitt | we'll have lyarwood take a look at it | |
| 22:22:01 | melwitt | thank you | |
| 22:24:46 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Remove old flavor_access_remove db api method https://review.openstack.org/539709 | |
| 22:34:09 | Roamer` | lyarwood, melwitt, https://bugs.launchpad.net/nova/+bug/1746609 | |
| 22:34:10 | openstack | Launchpad bug 1746609 in OpenStack Compute (nova) "test_boot_server_from_encrypted_volume_luks cannot detach an encrypted StorPool-backed volume" [Undecided,New] | |
| 22:34:23 | Roamer` | it's a bit brief, but it has links to the logs... | |
| 22:34:35 | openstackgerrit | Eric Fried proposed openstack/nova master: Avoid inventory DELETE API (no conflict detection) https://review.openstack.org/539712 | |
| 22:34:48 | efried | jaypipes: How about them apples ^ | |
| 22:35:12 | efried | Note approx -400LOC net. | |
| 22:35:13 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Remove old flavor_access_add db api methods https://review.openstack.org/539714 | |
| 22:35:30 | melwitt | Roamer`: thanks. we'll use this to investigate what's going wrong | |
| 22:35:38 | openstackgerrit | Eric Berglund proposed openstack/nova master: Use dict.get() when accessing capabilities dict https://review.openstack.org/539715 | |
| 22:41:23 | mriedem | Roamer`: is the storpool volume driver passing a secret in it's connection_info dict when using an encrypted volume type | |
| 22:42:01 | mriedem | i'm not actually sure if that's required | |
| 22:42:27 | mriedem | should probably check lee's patches to see if he had to change anything in devstack for this stuff to work, like with the key manager across both nova and cinder | |