| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2019-06-20 | |||
| 20:51:26 | aspiers | efried: first pass at series restructuring done | |
| 20:51:41 | aspiers | I'm sure you'll find things you still don't like :) | |
| 20:51:43 | efried | aspiers: cool. Sorry to be a PITA about this | |
| 20:51:48 | aspiers | np | |
| 20:52:07 | efried | it's the 'master switch' that's the most important thing. | |
| 20:52:36 | aspiers | I may not entirely disagree with your commit boundary preferences, but I agree 100% with the master switch thing, and 1000% with a pedantic approach which achieves high quality results ;-) | |
| 20:52:44 | aspiers | s/disagree/agree/ | |
| 20:53:06 | aspiers | OK, that's enough (actually too much) work for today | |
| 20:53:14 | aspiers | 10pm and still haven't eaten | |
| 20:53:30 | efried | there must be a magic triangle of pedantry/quality/productivity | |
| 20:53:32 | aspiers | haha | |
| 20:53:44 | aspiers | screw productivity, it's all about pedantry/quality ;-) | |
| 20:54:02 | aspiers | BTW tomorrow may be my last day working on SEV for ~3 weeks since next week is SUSE hackweek, and then I'm off to the USA for 2 weeks holiday | |
| 20:54:13 | efried | ight. | |
| 20:54:24 | aspiers | I think Boris is back around now, so *maybe* he can help progress things while I'm gone | |
| 20:54:34 | efried | aspiers: In your absence, if small things come up, permission to address them without you? | |
| 20:54:39 | aspiers | sure! | |
| 20:54:59 | efried | so like you might (*might*) come back to find this merged | |
| 20:55:03 | aspiers | haha | |
| 20:55:15 | aspiers | well that would be a lovely welcome back, but I'm not expecting it | |
| 20:55:18 | efried | yes, laughter appropriate | |
| 20:55:43 | aspiers | BTW it's looking like I'm being more officially allocated to upstream work from now on | |
| 20:55:55 | aspiers | so this is very good news (for me at least, maybe not for you) | |
| 20:56:42 | efried | nice | |
| 20:56:47 | efried | it is good news for me | |
| 20:56:51 | aspiers | :) | |
| 20:56:55 | efried | been getting kinda breezy around here | |
| 20:57:00 | aspiers | more excuses to be a pedant ;-p | |
| 20:58:00 | aspiers | would be great if I got time to hack on that requested_resources stuff, for example | |
| 20:58:16 | aspiers | at the moment I'm slightly feeling the heat on the SEV stuff, as we have some internal deadlines | |
| 20:58:47 | aspiers | not for forcing stuff down upstream's throat, but for reaching something that is functionally near complete downstream | |
| 21:00:00 | aspiers | OK, dinner time. Hopefully zuul CI won't shit all over my latest patchsets :) | |
| 21:00:04 | aspiers | Cya tomorrow o/ | |
| 21:00:47 | mriedem | having a meeting today? | |
| 21:01:00 | efried | oh, crap, I am completely unprepared. | |
| 21:32:42 | efried | hmph, probably no meeting next week due to idempotence day | |
| 21:34:27 | efried | week after next, rather | |
| 21:35:53 | mriedem | i read that as impotence | |
| 21:35:59 | mriedem | which is not something i celebrate in my house | |
| 21:36:11 | mriedem | and on that note, i'm out of here, o/ | |
| 23:30:44 | aspiers | efried: you can abandon https://review.opendev.org/#/c/662557/13 too | |
| #openstack-nova - 2019-06-21 | |||
| 00:42:51 | openstackgerrit | Dustin Cowles proposed openstack/nova-specs master: WIP: Spec: Use OpenStack SDK in Nova https://review.opendev.org/662881 | |
| 02:25:29 | alex_xu | ls | |
| 02:25:31 | alex_xu | oops | |
| 02:26:49 | alex_xu | sean-k-mooney: if we want to put the vpmem on the numa node rp, then it looks like we still need the reshape API, so the upgrade won't be easy with separated rp for vpmem now. But I can't get explict reason we should put the vpmem on numa node rp, or the child rp of numa node rp. | |
| 02:27:59 | alex_xu | if we prefer to the vpmem as the child rp of numa node rp, then we can go to that way, and separate the rp for now | |
| 02:33:09 | sean-k-mooney | ya im not sure what the preference is | |
| 02:34:08 | sean-k-mooney | i really dont mind too much either way but i just tought it might be simpler in the long run if we started with them seperate | |
| 02:34:24 | sean-k-mooney | but its not a point im really going to rathole on | |
| 02:35:06 | yonglihe | mriedem: HI. | |
| 02:42:13 | sean-k-mooney | yonglihe: while its less of an insane time of mriedem then me its still well after 22:40 for him so he wont be around for about 8-10 hours | |
| 02:51:49 | alex_xu | sean-k-mooney: ok, I see | |
| 02:54:49 | alex_xu | sean-k-mooney: another concern is people is asking us for a simple version for now, I'm afraid to people feel we add more in the current propsoal. actually, I don't have a preference also, happy to do the thing people like one | |
| 02:56:17 | sean-k-mooney | alex_xu: ya i know its frustrating that placments lack of support for numa has been holding back feature like this for 3 or 4 releases | |
| 02:57:02 | sean-k-mooney | we are cripleing the feature to make it simpeler because of the lack fo placmenent freature which created techdebt that need to fix next cycle | |
| 02:57:28 | alex_xu | sean-k-mooney: yea, agree with that | |
| 02:57:36 | sean-k-mooney | but doing it properly this cycle create tech debt when placement finally does support numa | |
| 02:57:37 | alex_xu | I understand your point | |
| 03:04:32 | openstackgerrit | sean mooney proposed openstack/nova master: Add transform_image_metadata request filter https://review.opendev.org/665775 | |
| 03:54:53 | openstackgerrit | Boxiang Zhu proposed openstack/nova master: Update AZ admin doc to mention the new way to specify hosts https://review.opendev.org/666767 | |
| 04:22:39 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Fix default values in update_cell command https://review.opendev.org/662383 | |
| 04:22:56 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Fix cleaning up console tokens https://review.opendev.org/637716 | |
| 04:29:49 | openstackgerrit | karim proposed openstack/nova-specs master: Support VM memory encryption using Intel's MKTME https://review.opendev.org/666769 | |
| 07:48:33 | openstackgerrit | huichen proposed openstack/nova stable/queens: Suppot paginaton for server groups by sql https://review.opendev.org/666782 | |
| 07:57:31 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Defaults missing group_policy to 'none' https://review.opendev.org/657796 | |
| 08:07:04 | kashyap | efried: Heya, fully agreed, on jroll's spec for _physical_ TPM passthrough. Maybe I misphrased... | |
| 08:11:03 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Defaults missing group_policy to 'none' https://review.opendev.org/657796 | |
| 08:22:08 | openstackgerrit | Merged openstack/nova master: Delete resource providers for all nodes when deleting compute service https://review.opendev.org/657016 | |
| 08:32:42 | openstackgerrit | Merged openstack/nova master: Avoid unnecessary joins in delete_resource_provider https://review.opendev.org/657021 | |
| 08:35:59 | openstackgerrit | Bence Romsics proposed openstack/os-vif master: Insert osprofiler trace info as external_ids to the bridge table https://review.opendev.org/665715 | |
| 08:39:25 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Remove global state from the FakeDriver https://review.opendev.org/656709 | |
| 08:43:30 | openstackgerrit | Surya Seetharaman proposed openstack/nova master: API microversion 2.74: Add 'power-update' external event https://review.opendev.org/645611 | |
| 08:46:35 | gibi | melwitt: commented https://review.opendev.org/#/c/631294 I don't know if my suggestion would lead to a simpler test or not. :/ | |
| 08:54:43 | openstackgerrit | Surya Seetharaman proposed openstack/python-novaclient master: API microversion 2.74: Add 'power-update' external event https://review.opendev.org/666792 | |
| 09:06:02 | openstackgerrit | Adam Spiers proposed openstack/nova master: Add extra spec parameter and image property for memory encryption https://review.opendev.org/664420 | |
| 09:06:02 | openstackgerrit | Adam Spiers proposed openstack/nova master: Use fake flavor instead of empty dict in test https://review.opendev.org/662555 | |
| 09:06:03 | openstackgerrit | Adam Spiers proposed openstack/nova master: Pass extra_specs to flavor in vif tests https://review.opendev.org/662556 | |
| 09:06:03 | openstackgerrit | Adam Spiers proposed openstack/nova master: Extract SEV-specific bits on host detection https://review.opendev.org/636334 | |
| 09:06:04 | openstackgerrit | Adam Spiers proposed openstack/nova master: Add |
|
| 09:06:04 | openstackgerrit | Adam Spiers proposed openstack/nova master: Apply SEV-specific guest config when SEV is required https://review.opendev.org/644565 | |
| 09:06:05 | openstackgerrit | Adam Spiers proposed openstack/nova master: Enable booting of libvirt guests with AMD SEV memory encryption https://review.opendev.org/666616 | |
| 09:36:59 | openstackgerrit | sean mooney proposed openstack/nova master: Add transform_image_metadata request filter https://review.opendev.org/665775 | |
| 09:40:21 | cdent | It's interesting to read this in the context of OpenStack, especially Nova: https://techbeacon.com/app-dev-testing/forget-monoliths-vs-microservices-cognitive-load-what-matters | |
| 09:48:56 | sean-k-mooney | cdent: the Germane cognitive load of nova is quite high by its nature | |
| 09:49:16 | cdent | it didn't have to be that way, but yes | |
| 09:50:02 | sean-k-mooney | well its partly becasue nova is ofthen the theing that end up driving interaction with other services | |
| 09:50:41 | sean-k-mooney | even if you precreate teh neuton ports for exampls nova still had sto notify nova when a vm uses them | |
| 09:51:42 | sean-k-mooney | some of that is intrisi to being the compute service wich is the main service that consumes other services | |
| 09:51:52 | sean-k-mooney | but ya its an interesting read | |
| 09:53:29 | cdent | the nova implementation is very tightly coupled with itself in a weirdly recursive kind of way | |
| 09:56:40 | sean-k-mooney | well not entirly the api, conductor, compute agent and schduler more or less work as miscroservices | |
| 09:57:56 | sean-k-mooney | or well services they are not that small | |
| 09:59:20 | cdent | that's not really true | |
| 09:59:30 | sean-k-mooney | how so | |
| 09:59:43 | cdent | interactions via rpc are insufficiently hard boundaries | |
| 09:59:43 | sean-k-mooney | the api does not care what virt dirver you use | |
| 10:00:35 | sean-k-mooney | am im not sure i would agree | |
| 10:01:09 | stephenfin | gibi++ Thanks for hitting https://review.opendev.org/#/c/609460/ again. Much obliged :) | |
| 10:01:20 | sean-k-mooney | the could be more orbust in some ways but i would not consier it to less stict then something like grpc | |
| 10:02:28 | sean-k-mooney | its still versioned using ovo | |