Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2019-05-23
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
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 :)

Earlier   Later