| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2019-05-23 | |||
| 08:05:13 | ileixe | CPU load | |
| 08:05:24 | ileixe | alex_xu: I used stress and ssl encryption | |
| 08:05:32 | ileixe | tool called 'stress' | |
| 08:08:07 | alex_xu | ileixe: the vcpu and pcpu is 1:1 pinning, but in the same cpu socket, they still share cpu cache | |
| 08:08:50 | ileixe | I made them to be separated by using cpu_thread_policy | |
| 08:08:59 | ileixe | to 'require' | |
| 08:09:17 | ileixe | so I think no VMs share physical core for the test environment. | |
| 08:11:40 | ileixe | alex_xu: Have you ever expereince cpu pinning? Is it assure that VMs could not interfere each other? | |
| 08:12:03 | ileixe | (if there are only cpu loads) | |
| 08:13:21 | lyarwood | # git review | |
| 08:13:24 | lyarwood | Could not connect to gerrit. | |
| 08:13:34 | lyarwood | hmm anyone seeing this or have I just borked something locally? | |
| 08:15:35 | lyarwood | `RSA host key for review.openstack.org has changed and you have requested strict checking.` - nope it's remote | |
| 08:21:33 | alex_xu | ileixe: i only can suggestion you to check whether is system process running on the those pinned pcpu. and setting the emulator thread policy ensure the emulator thread not running on those pcpu. | |
| 08:23:14 | ileixe | alex_xu: Hm.. I double checked libvirt configuration and PID/stats at proc interface, and it was expected. Thanks anyway. | |
| 08:25:59 | alex_xu | ileixe: np, good luck | |
| 09:15:25 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Remove cells v1 parameter from 'ComputeTaskAPI.resize_instance' https://review.opendev.org/651311 | |
| 09:15:25 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Remove 'ComputeManager._reschedule' https://review.opendev.org/660774 | |
| 09:15:26 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Stop passing 'delete_type' to 'terminate_instance' https://review.opendev.org/651313 | |
| 09:15:26 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Stop passing 'kwargs' to 'rebuild_instance' https://review.opendev.org/651312 | |
| 09:15:27 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Remove nova.compute.*API() shims https://review.opendev.org/660527 | |
| 09:15:27 | openstackgerrit | Stephen Finucane proposed openstack/nova master: filters: Stop handling cells v1 https://review.opendev.org/651314 | |
| 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 | |