| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2019-08-29 | |||
| 09:16:14 | sean-k-mooney | iphone uses png and raw filtes right | |
| 09:16:40 | alex_xu | emm...no, I don't remember which format | |
| 09:16:49 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Rename the nova-grenade-live-migration job to nova-grenade-multinode https://review.opendev.org/678841 | |
| 09:16:50 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Add cold migrate and resize to nova-grenade-multinode https://review.opendev.org/679210 | |
| 09:17:11 | alex_xu | .heic | |
| 09:17:26 | sean-k-mooney | i have never heared of that | |
| 09:17:40 | alex_xu | me too | |
| 09:18:58 | sean-k-mooney | oh is part of the h265/hvec standard form the MPEG body | |
| 09:20:47 | alex_xu | I guess that is used to support live photo | |
| 09:28:18 | openstackgerrit | Fan Zhang proposed openstack/nova master: Clear RequestSpec requested_destination and ignore_hosts in confirm/revert resize. https://review.opendev.org/679212 | |
| 10:02:08 | openstackgerrit | sean mooney proposed openstack/nova master: Libvirt: report storage bus traits https://review.opendev.org/666914 | |
| 10:02:09 | openstackgerrit | sean mooney proposed openstack/nova master: libvirt: use domain capabilities to get supported device models https://review.opendev.org/666915 | |
| 10:02:09 | openstackgerrit | sean mooney proposed openstack/nova master: Add transform_image_metadata request filter https://review.opendev.org/665775 | |
| 10:02:21 | openstackgerrit | Stephen Finucane proposed openstack/nova master: trivial: Rename 'nova.tests.unit.test_nova_manage' https://review.opendev.org/679095 | |
| 10:02:21 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Ensure online migrations have a unique name https://review.opendev.org/679096 | |
| 10:02:22 | openstackgerrit | Stephen Finucane proposed openstack/nova master: objects: Add online migration for legacy NUMA objects https://review.opendev.org/537414 | |
| 10:03:58 | stephenfin | bauzas: Fancy putting this trivial test-only patch and it's predecessor through? https://review.opendev.org/#/c/679096/ | |
| 10:15:44 | shilpasd | stephenfin: hi, going through your changes regarding Flatten 'ResourceRequest.from_extra_specs', ...https://github.com/openstack/nova/commit/7abe83f6460b053985c45d5b7af2b0ed5b17b66a | |
| 10:16:30 | shilpasd | stephenfin: accordingly making changes to https://review.opendev.org/#/c/671075/10/nova/scheduler/request_filter.py@69 | |
| 10:16:40 | shilpasd | so help me more to understand the same | |
| 10:16:47 | stephenfin | sure, shoot | |
| 10:17:52 | shilpasd | utils.ResourceRequest will hold all flavor and image related info | |
| 10:18:03 | shilpasd | no need to extract separately, right | |
| 10:18:42 | stephenfin | yeah, the idea is to do any translation from image metadata or flavor extra specs inside that class | |
| 10:18:53 | stephenfin | for example | |
| 10:19:43 | stephenfin | https://review.opendev.org/#/c/671801/32/nova/scheduler/utils.py@106 | |
| 10:20:03 | stephenfin | This avoids the need to set stuff on the request_spec.flavor.extra_specs field | |
| 10:24:48 | stephenfin | alex_xu: Can you elaborate on https://review.opendev.org/#/c/672693/13/nova/virt/libvirt/driver.py@a5938 ? | |
| 10:25:41 | shilpasd | ok got it, now in my changes i have to call utils.ResourceRequest(request_spec) | |
| 10:26:38 | stephenfin | shilpasd: In your case, yeah, that sounds correct | |
| 10:27:36 | shilpasd | stephenfin: tnx for clarification, will update and if further queries will ping you here | |
| 10:35:38 | aspiers | stephenfin: so that means I have to rewrite https://review.opendev.org/#/c/664420/44/nova/scheduler/request_filter.py@181 now? | |
| 10:35:52 | aspiers | I'm still in the process of grokking your change | |
| 10:36:29 | stephenfin | aspiers: yeah, probably, but (a) I'm happy to help out with that and (b) you're now in a runway | |
| 10:37:07 | stephenfin | aspiers: tl;dr: drop that change to nova/scheduler/request_filter.py and instead add a translation function inside the ResourceRequest object | |
| 10:37:22 | stephenfin | like we're doing here https://review.opendev.org/#/c/671801/32/nova/scheduler/utils.py@106 | |
| 10:37:41 | stephenfin | at least, I think that's the way we're going. Run it by efried in case I'm talking out of my ass :) | |
| 10:38:22 | stephenfin | I need to migrate the 'compute_status_filter' filter across to do the same thing | |
| 10:44:57 | openstackgerrit | Merged openstack/nova stable/stein: doc: remove confusing docs about aggregate allocation ratios https://review.opendev.org/678254 | |
| 10:47:26 | aspiers | stephenfin: are you about to move SEV into the etherpad's runways section? I see you updated the history at the bottom | |
| 10:47:36 | stephenfin | I am | |
| 10:47:40 | stephenfin | sorry, got distracted | |
| 10:47:41 | aspiers | kewl | |
| 10:47:44 | aspiers | haha np :) | |
| 10:47:46 | stephenfin | many plates | |
| 10:47:51 | aspiers | indeed | |
| 10:56:00 | stephenfin | kashyap: You asked for a review on https://review.opendev.org/#/c/673790/ but it looks like aspiers has some open comments. Are those resolved? It's not clear | |
| 10:56:36 | kashyap | stephenfin: Hey, yeah. Discussed with aspiers, too, yesterday on #virt. Just wanted a different set of eyes, along with ours | |
| 10:56:46 | stephenfin | ah, gotcha | |
| 10:56:51 | stephenfin | sure, I'll look after lunch | |
| 10:57:08 | kashyap | Sure, thx for remembering :-) | |
| 10:59:57 | aspiers | stephenfin, kashyap: just added some more comments | |
| 11:00:13 | openstackgerrit | Brin Zhang proposed openstack/nova master: Filter migrations by user_id/project_id https://review.opendev.org/674243 | |
| 11:00:16 | kashyap | aspiers: /me looks; thank you | |
| 11:04:29 | kashyap | aspiers: Agreed in general. But as a background, I used the "features" term, as in, they're loader's features. | |
| 11:04:42 | kashyap | aspiers: But happy to of course make the change to enums. If that's clearer | |
| 11:12:46 | openstackgerrit | Kashyap Chamarthy proposed openstack/nova master: libvirt: Parse the 'os' element from domainCapabilities https://review.opendev.org/673790 | |
| 11:13:56 | kashyap | aspiers: --^ Better? Two changes: (a) rename to 'enums; (b) in the tests, use variables for the assertEqual() express. | |
| 11:28:04 | aspiers | kashyap: very close :) | |
| 11:28:39 | kashyap | aspiers: Yeah, you're right. The damn variable | |
| 11:28:45 | kashyap | Let me remove it :-) | |
| 11:30:38 | aspiers | kashyap: I understand why you used features first time around. But if I was being pedantic (and I am, because that's what I do), none of "type" / "yes" / "no" are (conceptual) features, whereas they *are* parts of enumerations of allowed components in a schema. | |
| 11:30:40 | kashyap | I "hearby order" the unit tests to succeed | |
| 11:30:45 | aspiers | Hehe | |
| 11:31:00 | kashyap | [No? You won't budge? Okay, I'll fix it :D] | |
| 11:32:36 | brinzhang | Who can review https://review.opendev.org/#/c/665136/ Specifying AZ to unshelve the server, already +2 by Matt. | |
| 11:32:43 | kashyap | aspiers: Okay, you saw why I did. And yes, I agree with the pedantry and renaming is the Right Thing. In future, too, don't hesitate to think out loud and persist that way. Appreciate it | |
| 11:33:11 | brinzhang | Today is the deadline in runway, thanks :P | |
| 11:43:27 | sean-k-mooney | i didnt know we used runways for nova clinent review but i guess that makes sense | |
| 11:47:00 | brinzhang | beacuse of it belongs to the bp, does need to remove the part of the bp with novaclient or openstackclient patch? | |
| 11:48:43 | sean-k-mooney | brinzhang: no its fine | |
| 11:49:21 | sean-k-mooney | its just most people dont get around to doing the client update when they add an new api | |
| 11:50:29 | brinzhang | sean-k-mooney: yeah, IMO, they belong to the same part of nova and will only reset the bp state if they are all completed, so I add the novalient and OSC | |
| 11:51:15 | brinzhang | sean-k-mooney: If anyone canot necessary, I can remove it from the runway, reduce the work for reviewer :P | |
| 11:51:49 | sean-k-mooney | the patch is close so i would leave it | |
| 11:52:06 | sean-k-mooney | stephenfin: can are you core on nova-client | |
| 11:53:08 | sean-k-mooney | yep nova-core is in the group | |
| 11:53:17 | sean-k-mooney | https://review.opendev.org/#/c/665136/ is short | |
| 11:53:22 | brinzhang | Yeah | |
| 11:53:53 | sean-k-mooney | when you get back form luch that is | |
| 11:54:23 | brinzhang | nova-core has the +1 W permission for novaclient | |
| 11:55:27 | brinzhang | sean-k-mooney: Oh, no, I will go home, maybe tomorrow can see it :) | |
| 11:55:41 | brinzhang | sean-k-mooney: Thanks again. | |
| 11:58:26 | kashyap | aspiers: Hey, before I hit `git review`, you cool with this additional 'diff' for tests? -- http://paste.openstack.org/show/766973/ | |
| 11:58:47 | kashyap | (The multiple entries are for different arches...) | |
| 12:04:18 | sean-k-mooney | kashyap: do all those arch support it | |
| 12:04:38 | sean-k-mooney | the current static data was taken form vrish | |
| 12:05:06 | sean-k-mooney | if mipsel or sparc does not support secure boot we shoudl not add it | |
| 12:05:31 | kashyap | sean-k-mooney: From my look up all of them seem to do | |
| 12:05:39 | sean-k-mooney | ok | |
| 12:05:53 | aspiers | I had the same question as sean-k-mooney | |
| 12:06:03 | aspiers | kashyap: so the diff looks good to me | |
| 12:06:29 | kashyap | sean-k-mooney: I lied, I'm not sure of PPC -- let me check | |
| 12:07:15 | sean-k-mooney | PPC at lest the 64bit versions definetly supports uefi secure boot | |
| 12:08:17 | aspiers | kashyap: you could always add a conditional in the tests to exclude 'secure' for some archs if necessary to maintain realistic fixtures | |
| 12:08:20 | kashyap | Ah, okay | |
| 12:08:53 | sean-k-mooney | i would prefer to keep the files correct | |
| 12:08:59 | sean-k-mooney | but yes | |
| 12:09:01 | kashyap | aspiers: True. I don't want to keep fiddling with it, though | |
| 12:10:16 | kashyap | sean-k-mooney: And SPARC calls it "verified boot" (https://docs.oracle.com/cd/E37838_01/html/E61021/sysauth-vb.html) | |