| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2019-05-23 | |||
| 09:15:28 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Ensure controllers all call super https://review.opendev.org/660950 | |
| 09:49:36 | kashyap | stephenfin: Hi | |
| 09:50:03 | kashyap | stephenfin: I want to submit this doc to one of the upstream Nova admin guides -- which guide do you suggest: https://kashyapc.fedorapeople.org/Nova-Mitigation-for-Intel-MDS-Flaws.txt | |
| 09:52:20 | stephenfin | kashyap: probably the reference guide (doc/source/reference) | |
| 09:58:45 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Remove global state from the FakeDriver https://review.opendev.org/656709 | |
| 09:59:28 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Enhance service restart in functional env https://review.opendev.org/512552 | |
| 10:02:11 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Reproduce bug 1724172 in the functional test env https://review.opendev.org/512553 | |
| 10:02:12 | openstack | bug 1724172 in OpenStack Compute (nova) "Allocation of an evacuated instance is not cleaned on the source host if instance is not defined on the hypervisor" [Medium,In progress] https://launchpad.net/bugs/1724172 - Assigned to Balazs Gibizer (balazs-gibizer) | |
| 10:02:26 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: cleanup evacuated instances not on hypervisor https://review.opendev.org/512623 | |
| 10:02:38 | openstackgerrit | Arnaud Morin proposed openstack/nova master: Force refresh instance network info on deletion https://review.opendev.org/660761 | |
| 10:44:51 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Enhance service restart in functional env https://review.opendev.org/512552 | |
| 10:44:51 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Remove global state from the FakeDriver https://review.opendev.org/656709 | |
| 10:44:52 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Reproduce bug 1724172 in the functional test env https://review.opendev.org/512553 | |
| 10:44:53 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: cleanup evacuated instances not on hypervisor https://review.opendev.org/512623 | |
| 10:44:53 | openstack | bug 1724172 in OpenStack Compute (nova) "Allocation of an evacuated instance is not cleaned on the source host if instance is not defined on the hypervisor" [Medium,In progress] https://launchpad.net/bugs/1724172 - Assigned to Balazs Gibizer (balazs-gibizer) | |
| 11:06:29 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Do not start nova-network in the notification func test https://review.opendev.org/660975 | |
| 11:22:16 | hrw | morning | |
| 11:38:46 | openstackgerrit | Lee Yarwood proposed openstack/nova master: Use os-brick locking for volume attach and detach https://review.opendev.org/614190 | |
| 11:43:33 | openstackgerrit | Hamdy Khader proposed openstack/os-vif master: OVS DPDK port representors support https://review.opendev.org/658786 | |
| 11:59:40 | openstackgerrit | Arnaud Morin proposed openstack/nova master: Use force_refresh=True on network info update https://review.opendev.org/660982 | |
| 12:50:44 | openstackgerrit | Adam Spiers proposed openstack/nova master: Move patch_exists() to nova.test.TestCase for reuse https://review.opendev.org/660500 | |
| 13:22:16 | kashyap | stephenfin: Thx | |
| 13:27:24 | efried | kashyap: Did you see my patch answering the "how do we support renamed arch-specific cpu flag traits" question? | |
| 13:28:07 | kashyap | efried: Afraid, not yet. I'm at a one-day conference in the Netherlands, so my connectivity is a bit spotty today. | |
| 13:28:33 | kashyap | efried: Got the URL handy? | |
| 13:28:50 | efried | kashyap: https://review.opendev.org/#/c/660515/ | |
| 13:29:21 | efried | kashyap: it's all red in the CI because it depends on your os-traits patch, but the dep chain doesn't work easily | |
| 13:29:51 | efried | kashyap: But once your patch is in an os-traits release, we put that in requirements.txt/lower-constraints.txt in this patch and it should go green. | |
| 13:30:16 | efried | anyway, point is it's not super hard to make it work, and we're probably going to have to do it eventually anyway, so might as well. | |
| 13:30:37 | efried | what this means to you is: no reason to hold off "moving" those traits in your patch. | |
| 13:31:51 | openstackgerrit | Adam Spiers proposed openstack/nova master: Move patch_exists() to nova.test.TestCase for reuse https://review.opendev.org/660500 | |
| 13:32:08 | aspiers | efried: ^^^ this feels like the cleanest approach to me | |
| 13:33:01 | kashyap | efried: Thank you ... "For backward compatibility reasons, we have to continue exposing those versions of those traits" :-) | |
| 13:33:05 | aspiers | efried: I'm working on something similar for open() | |
| 13:33:17 | aspiers | but for now I need to go vote and get lunch. biab | |
| 13:33:35 | kashyap | efried: So my patch, shall I go back to making it a new PS of the older PS to preserve history? (Sorry for not doing that to begin with) | |
| 13:33:53 | kashyap | Or shall I just mention it in this new one, saying: "please read here for history"? | |
| 13:34:06 | efried | I would like someone libvirt-y, like stephenfin and aspiers, to have a look at that solution and make sure it's not insane. | |
| 13:34:25 | efried | kashyap: I would like it all kept in the original patch please. | |
| 13:34:42 | efried | aspiers: Well, you were the last one to touch the CPU_TRAITS_MAPPING anyway :) | |
| 13:34:47 | aspiers | true | |
| 13:35:08 | aspiers | reminder to self: voting and lunching means not being here :-/ | |
| 13:35:10 | kashyap | efried: Will keep it in the original (then you need to update your PS, too) | |
| 13:35:37 | efried | kashyap: Thanks. Yeah, at this point I've proven the concept; I'll just wait until your patch is merged and released and then update mine properely. | |
| 13:35:39 | efried | properly | |
| 13:35:44 | efried | (sean-k-mooney moment) | |
| 13:36:18 | kashyap | efried: Yeah, noted. Tomm I'm off the afternoon. Can get to it only next week. Since there's no urgency here, hope that's OK | |
| 13:36:39 | efried | kashyap: Sure, that's fine, no hurry from my end | |
| 13:36:45 | kashyap | efried: Also you might want to have a gander at this (which I'll submit as a doc to Nova): https://kashyapc.fedorapeople.org/Nova-Mitigation-for-Intel-MDS-Flaws.txt | |
| 13:37:58 | openstackgerrit | Dan Smith proposed openstack/nova master: Make nova-next archive using --before https://review.opendev.org/661002 | |
| 13:39:35 | efried | kashyap: lgtm | |
| 13:39:49 | efried | I'll have a few comments when it's up in gerrit | |
| 13:39:58 | efried | but overall clear and concise | |
| 13:40:23 | kashyap | efried: Thx; sure. Posting a variant of it now...before I forget. | |
| 13:40:43 | kashyap | Anything that improves clarity. | |
| 13:42:02 | mfeder | # as regular user : | |
| 13:42:02 | mfeder | E.g. | |
| 13:42:02 | mfeder | I would like to access regular-user's key pair using admin user. | |
| 13:42:02 | mfeder | Is there any option how to execute command as another user (linux way : "sudo -u user `command`") ? | |
| 13:42:02 | mfeder | Hi all, | |
| 13:42:03 | mfeder | # as admin user: | |
| 13:42:03 | mfeder | `openstack keypair create kp1` | |
| 13:42:04 | mfeder | `openstack keypair show kp1 ` | |
| 13:42:38 | mfeder | Thanks for any hint ! | |
| 13:44:49 | mfeder | There seems to be an option in the nova API to specify a user ID for keypair requests: https://developer.openstack.org/api-ref/compute/?expanded=list-keypairs-detail,create-or-import-keypair-detail#keypairs-keypairs But I don't know how to do this via `openstack` command | |
| 13:55:06 | frickler | mfeder: there's a patch proposed for that, so currently it isn't possible yet, see https://review.opendev.org/576507 . you may want to ask dtroyer or mordred in #openstack-sdks for help with getting this merged and released | |
| 13:56:01 | mfeder | frickler: Thank you ! | |
| 14:09:07 | openstackgerrit | Merged openstack/nova stable/stein: Skip _exclude_nested_providers() if not nested https://review.opendev.org/659206 | |
| 14:36:21 | mordred | frickler: wow. that's 10 months old | |
| 14:37:10 | frickler | mordred: yeah, progress in osc land is ... not fast | |
| 14:37:12 | efried | aspiers: +2, some tweaks if you feel so inclined. | |
| 14:38:17 | openstackgerrit | Merged openstack/python-novaclient master: Set the lower limit of api_version for volume_type https://review.opendev.org/660861 | |
| 14:43:07 | mriedem | get'ya simple perf optimizations hey! https://review.opendev.org/#/c/651647/ | |
| 14:54:21 | openstackgerrit | Brin Zhang proposed openstack/python-novaclient stable/stein: Set the lower limit of api_version for volume_type https://review.opendev.org/661024 | |
| 14:57:06 | openstackgerrit | Adam Spiers proposed openstack/nova master: Move patch_exists() to nova.test.TestCase for reuse https://review.opendev.org/660500 | |
| 14:57:21 | aspiers | efried: ^^^ | |
| 14:58:01 | aspiers | doh | |
| 14:58:04 | aspiers | docstring error | |
| 14:58:58 | openstackgerrit | Adam Spiers proposed openstack/nova master: Move patch_exists() to nova.test.TestCase for reuse https://review.opendev.org/660500 | |
| 14:59:27 | efried | yeah, you could even remove the word 'static' entirely to convey the meaning. | |
| 14:59:36 | efried | as the call doesn't change regardless | |
| 14:59:43 | aspiers | true | |
| 14:59:49 | aspiers | well, it says "static method" now | |
| 14:59:54 | aspiers | that should be good enough | |
| 14:59:58 | efried | yup | |
| 15:00:09 | efried | stephenfin: easy and fun if you're so inclined --^ | |
| 15:00:58 | aspiers | I hadn't previously realised that @contextlib.contextmanager gives you something which can be both a decorator and a context manager. Pretty nice | |
| 15:05:35 | efried | I didn't know that either. TIL | |
| 15:13:09 | aspiers | this is comically wrong https://www.stackalytics.com/?metric=commits&release=train&project_type=all&module=nova | |
| 15:13:41 | aspiers | efried: if there were only 16 commits so far this release, I guess you'd be feeling a lot more relaxed :) | |
| 15:14:16 | aspiers | I've already reported the issue, BTW | |
| 15:20:40 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Avoid unnecessary joins in InstanceGroup.get_hosts https://review.opendev.org/661032 | |
| 15:20:43 | mriedem | another easy optimization ^ | |
| 15:26:06 | efried | aspiers: Yup, saw the thread, thanks for reporting. | |
| 15:26:26 | mriedem | mlavalle: sean-k-mooney: btw i fixed the revert cross-cell cold migration port binding thing from the other day - activating the source host binding before deleting the dest host binding did the trick as expected | |
| 15:29:42 | mlavalle | mriedem: good to know :-) | |
| 15:35:42 | sean-k-mooney | mriedem: cool good to know. i need to add that review serise back to my review queue | |
| 15:36:29 | sean-k-mooney | mriedem:are you are only using the mulitple port binding flow for cross cell by the way or will it now be ussed for all resizes/cold migrations too | |
| 15:36:44 | mriedem | the former | |
| 15:36:50 | sean-k-mooney | ok | |