| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2019-08-06 | |||
| 22:27:22 | efried | but anyway, not today. | |
| 22:37:59 | openstackgerrit | Matt Riedemann proposed openstack/nova master: docs: update 2.23 REST API version history https://review.opendev.org/674948 | |
| 22:42:42 | openstackgerrit | Matt Riedemann proposed openstack/nova master: api-ref: document valid GET /os-migrations?migration_type values https://review.opendev.org/674949 | |
| 22:45:49 | openstackgerrit | Matt Riedemann proposed openstack/python-novaclient master: docs: clarify nova migration-list --host option https://review.opendev.org/674950 | |
| 22:46:48 | openstackgerrit | Matt Riedemann proposed openstack/python-novaclient master: docs: clarify nova migration-list --host option https://review.opendev.org/674950 | |
| 23:51:51 | openstackgerrit | Huachang Wang proposed openstack/nova-specs master: Use PCPU and VCPU in one instance https://review.opendev.org/668656 | |
| #openstack-nova - 2019-08-07 | |||
| 02:09:12 | openstackgerrit | Li Liu proposed openstack/nova master: WIP: Create and bind Cyborg ARQs. https://review.opendev.org/631244 | |
| 02:11:37 | openstackgerrit | Eric Fried proposed openstack/nova master: trivial: Remove unused '_instance_to_allocations_dict' function https://review.opendev.org/674893 | |
| 02:56:06 | fungi | i directly reenqueued 674821,1 back into the gate after that last failure, so no need to recheck it | |
| 02:56:46 | fungi | i'd promote it but the only thing ahead is a cinder change which is eta 28 minutes to merge anyway | |
| 02:56:58 | openstackgerrit | ZhengMa proposed openstack/nova master: Check container_format on instance creation https://review.opendev.org/673407 | |
| 02:59:29 | openstackgerrit | ZhengMa proposed openstack/nova master: Check container_format on instance creation https://review.opendev.org/673407 | |
| 03:01:15 | fungi | the failure was a TimeoutException in test_walk_versions on py36, for the record | |
| 03:12:20 | openstackgerrit | Li Liu proposed openstack/nova master: WIP: Create and bind Cyborg ARQs. https://review.opendev.org/631244 | |
| 03:36:22 | openstackgerrit | YAMAMOTO Takashi proposed openstack/nova master: Revert "Revert resize: wait for events according to hybrid plug" https://review.opendev.org/675021 | |
| 03:53:36 | openstackgerrit | Li Liu proposed openstack/nova master: WIP: Create and bind Cyborg ARQs. https://review.opendev.org/631244 | |
| 04:11:26 | openstackgerrit | Li Liu proposed openstack/nova master: WIP: Added Cyborg External Events https://review.opendev.org/675022 | |
| 04:14:41 | openstackgerrit | Brin Zhang proposed openstack/python-novaclient master: Microversion 2.76: Add user_id/project_id to migration-list API https://review.opendev.org/675023 | |
| 04:15:35 | openstackgerrit | Alex Xu proposed openstack/nova-specs master: Update the virtual persistent memory spec based on discussion https://review.opendev.org/675024 | |
| 04:17:20 | openstackgerrit | Jing Zhang proposed openstack/nova master: Remove the current host from cold migration target host list https://review.opendev.org/675025 | |
| 05:20:53 | openstackgerrit | Merged openstack/nova master: Replace non-nova server fault message https://review.opendev.org/674821 | |
| 05:40:27 | openstackgerrit | ZhengMa proposed openstack/nova master: Check container_format on instance creation https://review.opendev.org/673407 | |
| 07:01:13 | openstackgerrit | Merged openstack/nova master: trivial: Remove unused '_instance_to_allocations_dict' function https://review.opendev.org/674893 | |
| 07:14:33 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Fix non-existent method of Mock https://review.opendev.org/675041 | |
| 07:55:50 | brinzhang | takashi: Why abandon this patch https://review.opendev.org/#/c/450119? | |
| 07:57:35 | brinzhang | takashi: I have a bp, it's codes need to add parameters to support for migration-list command, but there isnot the ``openstack server migration-list`` CLI | |
| 08:01:53 | brinzhang | takashin: :) | |
| 08:02:13 | takashin | brinzhang: Hi | |
| 08:02:55 | brinzhang | https://review.opendev.org/#/c/450119 this patch, looks good, but why are you abandon? | |
| 08:03:38 | takashin | brinzhang: There was a subsequnt BP. It was the function to abort cold migration. | |
| 08:04:33 | takashin | brinzhang: But in community discussin, it is not necessary to abort cold migration. So https://review.opendev.org/#/c/450119 was abondaned. | |
| 08:05:39 | brinzhang | takashin: Now in the openstackclient, there is not has the 'opesntack server migration-list', I was reviewed this patch, found you added it | |
| 08:06:19 | brinzhang | takashin: so I asked for that, if so, I will push a new patch for the migration-list | |
| 08:06:41 | takashin | brinzhang: You can take over it. | |
| 08:08:38 | brinzhang | takashin: cool, I will have a try, and remove the cold migration deal, and add my requirement. | |
| 08:23:18 | openstackgerrit | Lee Yarwood proposed openstack/nova master: Use os-brick locking for volume attach and detach https://review.opendev.org/614190 | |
| 09:22:18 | openstackgerrit | Chris Dent proposed openstack/nova master: Add a document that describes profiling eventlet services https://review.opendev.org/674865 | |
| 09:28:09 | bhagyashris | stephenfin: Hi , Good morning :) | |
| 09:29:19 | stephenfin | bhagyashris: Evening o/ | |
| 09:29:49 | bhagyashris | stephenfin: I saw you have uploaded patches including the upgrade changes, But I found few issues after applying all the patches. #link: http://paste.openstack.org/show/755606/ | |
| 09:30:24 | bhagyashris | stephenfin: noted issues in above #link path | |
| 09:34:47 | bhagyashris | stephenfin: As per discussion and mentioned in spec we are not going to support both the PCPU and VCPU in single boot request so IMO we will need to add check some where to reject such request with proper error message to user. | |
| 09:34:56 | stephenfin | bhagyashris: Yeah, I should have a WIP flag on that. There's a few corner cases I still need to cover including that one | |
| 09:35:54 | stephenfin | You're correct about the pinning though. I guess that's because we need to set the 'cpu_policy' field on the 'InstanceNUMATopology' object | |
| 09:36:59 | bhagyashris | stephenfin:yeah | |
| 09:37:50 | bhagyashris | stephenfin: And also I would like to know about are we going tp supporting new syntax of flavor extra specs like resources:PCPU and resources:VCPU in Train? | |
| 09:38:05 | bhagyashris | S/tp/to | |
| 09:39:18 | stephenfin | That would require changes to the InstanceNUMATopology object, but like we discussed previously, they don't need to be done yet | |
| 09:40:42 | bhagyashris | stephenfin: if yes then in that case we will also need to take care the first issue that I have mentioned above ... gust xml will give wrong result if user will create the instance using flavor extra specs resources:PCPU=1 | |
| 09:41:46 | stephenfin | Yup. Indeed. That needs to be done regardless. | |
| 09:42:09 | bhagyashris | stephenfin: OK | |
| 09:42:14 | bhagyashris | stephenfin: and one more thing I would like to ask about the translation of flavor extra specs and image meta data .. are you going to use my change https://review.opendev.org/#/c/672223/1 or you are going to continue on your change? | |
| 09:44:11 | stephenfin | I think I'm going to keep going with https://review.opendev.org/#/c/671801/. I don't like the idea of rewriting the flavor behind the users back and this change doesn't require that. I might change my mind again, but I'll go with this for now | |
| 09:45:20 | bhagyashris | stephenfin: OK No problem. | |
| 09:45:52 | stephenfin | bhagyashris: Thanks for testing this stuff, btw. Really good to have as many eyes on it as possible, as we're seeing :) | |
| 09:47:14 | bhagyashris | stephenfin: If you are OK? then I will take up adding the functional test cases and will fix the existing test that you have added for https://review.opendev.org/#/c/674895/2 | |
| 09:48:07 | stephenfin | bhagyashris: Yes, please. I kept the Author the same on that. I'm going to be working on the previous two patches and adding support for reporting the 'HW_CPU_HYPERTHREADING' trait today | |
| 09:48:58 | bhagyashris | stephenfin: OK then I will take up this task and will upload the patch soon | |
| 10:01:33 | bhagyashris | stephenfin: Thank you :) | |
| 11:40:47 | sean-k-mooney | stephenfin: gibi alex_xu https://review.opendev.org/#/c/621646/23 has been around for a while and it hasnt gotten any core review. i have review this several times at this point, would ye be able to take a look? | |
| 11:48:06 | gibi | sean-k-mooney: I opened it now on a tab but I cannot promies to reach there today | |
| 11:48:42 | sean-k-mooney | gibi: no worries i was pingged to review it yesterday so i did it this morning | |
| 11:49:32 | sean-k-mooney | i just wanted to flag it to some people since it was opened back in december and they have been rebasing it since | |
| 12:03:59 | alex_xu | sean-k-mooney: I added it to my review list, will reach it when I do review work this week | |
| 12:05:04 | sean-k-mooney | alex_xu: cool thanks | |
| 12:07:58 | stephenfin | sean-k-mooney: sure | |
| 12:10:59 | sean-k-mooney | speaking of reveiw where are we at with https://review.opendev.org/#/c/668656/7/specs/train/approved/use-pcpu-vcpu-in-one-instance.rst | |
| 12:11:19 | sean-k-mooney | oh wrong version | |
| 12:11:53 | sean-k-mooney | all the discusstion seams to be on 7 but the patch is on version 11 | |
| 12:14:19 | sean-k-mooney | i think the table for the setting in the image vs flavor is wrong by the way | |
| 12:26:35 | openstackgerrit | Alex Xu proposed openstack/nova master: Add functional tests for resize with VGPU https://review.opendev.org/675106 | |
| 12:27:55 | openstackgerrit | Stephen Finucane proposed openstack/nova master: scheduler: Flatten 'ResourceRequest.from_extra_specs', 'from_image_props' https://review.opendev.org/674894 | |
| 12:27:55 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Add support for translating CPU policy extra specs, image meta https://review.opendev.org/671801 | |
| 12:27:56 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Add reshaper for PCPU https://review.opendev.org/674895 | |
| 12:40:31 | alex_xu | sean-k-mooney: dansmith also has a point about whether we should let the user specific which vcpu is dedicated | |
| 12:43:37 | openstackgerrit | Shilpa Devharakar proposed openstack/nova master: Nova object changes for forbidden aggregates request filter https://review.opendev.org/671072 | |
| 12:45:01 | openstackgerrit | Shilpa Devharakar proposed openstack/nova master: DB API changes to get non-matching aggregates from metadata https://review.opendev.org/671074 | |
| 12:46:18 | openstackgerrit | Shilpa Devharakar proposed openstack/nova master: Add a new request filter to forbidden aggregates https://review.opendev.org/671075 | |
| 12:47:11 | sean-k-mooney | alex_xu: dansmith the reason for being able to define which cpus are dedicated is the same for why we allow it for realtime cpus. we want to be able to say that core x is low performace and can be used for the os/manament interfaces vs core y is high perfomace and can be used for the application | |
| 12:48:03 | openstackgerrit | Shilpa Devharakar proposed openstack/nova master: Enable request filter forbidden_aggregates https://review.opendev.org/667952 | |
| 12:48:28 | sean-k-mooney | alex_xu: dansmith: so yes we do need to allow the user to pass a mask in my opipion and i dont think nova should do any magic to try and calualte it. | |
| 12:48:56 | alex_xu | sean-k-mooney: hint by you, if I want the realtime cpu to be dedicated, then I still need a way to define that. | |
| 12:49:03 | sean-k-mooney | if the user has multiple numa nodes and/or a specific cpu toplogy then they shoudl understand that mapping | |
| 12:49:40 | sean-k-mooney | well that is simple. use the same mask for both | |
| 12:49:50 | alex_xu | yea, otherwise, we will have a magic to mapping realtime vcpu to dedicated vcpu inside nova | |
| 12:49:55 | sean-k-mooney | but really i dont think we should allow realtime core to not be dedicated | |
| 12:50:57 | alex_xu | ah, that make sense. but it still will be a magic | |
| 12:51:02 | sean-k-mooney | if you are declaring a core as realtime it does not make sense for it to be multiplex to a shared host core that other vms can execute on as that will violate any realtiem logic | |
| 12:51:21 | sean-k-mooney | well we just and the bitmasks | |
| 12:51:56 | sean-k-mooney | actuuly xor the bits and check the result is 0 | |
| 12:52:30 | sean-k-mooney | actully no that is also wrong | |
| 12:53:05 | sean-k-mooney | but you get the idea we just need to validate each bit set in the realtime mask is set in the dedicated mask | |
| 12:53:19 | alex_xu | yea | |
| 12:53:23 | alex_xu | i got it | |
| 12:55:18 | sean-k-mooney | to dansmith point however i do belive we discusssed the idea of not exposing a mask in the past and instead doing what he say and just having tally counts of how many of each type you want | |
| 12:55:54 | sean-k-mooney | then evenly distubting them over the numa nodes and ensureing within a numa node the shared cpus are first followed by the pinned ones | |
| 12:56:16 | sean-k-mooney | that fits the paradime where core 0 is often reserved for the os | |
| 12:56:53 | sean-k-mooney | and in a multi numa works the first core on each numa node is often reserved for the os | |
| 12:57:17 | sean-k-mooney | but that does seam like magic that would easily break | |
| 13:00:39 | alex_xu | good to know | |