| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2020-04-27 | |||
| 20:47:46 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Clarify the policy new defaults upgrade notes https://review.opendev.org/723645 | |
| 20:50:05 | gmann | dansmith: gibi stephenfin please check, i have added this upgrade notes for clarification on policy file things - https://review.opendev.org/#/c/723645/ | |
| 21:09:20 | melwitt | gmann: do we have people ready to review https://review.opendev.org/722551 ? wondering if I should wait on reviewing the nova change | |
| 21:10:31 | gmann | melwitt: i pinged few tempest core,may be we can get +A from masayukig once he wake up. | |
| 21:11:33 | melwitt | ok | |
| 21:12:50 | openstackgerrit | Artom Lifshitz proposed openstack/nova stable/queens: DNM: Partial cherry-pick of assertRequestMatchesUsage() https://review.opendev.org/723694 | |
| 21:12:51 | openstackgerrit | Artom Lifshitz proposed openstack/nova stable/queens: DNM: Add nova-manage placement heal_allocations CLI https://review.opendev.org/723696 | |
| 21:12:51 | openstackgerrit | Artom Lifshitz proposed openstack/nova stable/queens: DNM: Partial cherry-pick of _check_allocation_during_evacuate() https://review.opendev.org/723695 | |
| 21:12:52 | openstackgerrit | Artom Lifshitz proposed openstack/nova stable/queens: DNM: Partial cherry-pick of FakeResponse https://review.opendev.org/723698 | |
| 21:12:52 | openstackgerrit | Artom Lifshitz proposed openstack/nova stable/queens: DNM: Don't heal allocations for deleted servers https://review.opendev.org/723697 | |
| 21:12:53 | openstackgerrit | Artom Lifshitz proposed openstack/nova stable/queens: DNM: Heal allocations with incomplete consumer information https://review.opendev.org/723699 | |
| 21:14:10 | gmann | melwitt: or let's wait for these patches first to have ussuri branch setup properly - https://review.opendev.org/#/q/topic:qa-ussuri-release+status:open | |
| 21:14:55 | melwitt | gmann: ah k | |
| 22:48:22 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Clarify the policy new defaults upgrade notes https://review.opendev.org/723645 | |
| 23:14:59 | openstackgerrit | Artom Lifshitz proposed openstack/nova stable/queens: DNM: Add a placement audit command https://review.opendev.org/723751 | |
| 23:14:59 | openstackgerrit | Artom Lifshitz proposed openstack/nova stable/queens: DNM: Partial cherry-pick of report client changes https://review.opendev.org/723750 | |
| #openstack-nova - 2020-04-28 | |||
| 00:58:51 | songwenping_ | Hi, gibi. | |
| 01:00:01 | songwenping_ | i have a issue with filter_scheduler with accelerators. pls see the detail at https://etherpad.opendev.org/p/filter_scheduler_issue_with_accelerators | |
| 01:01:05 | songwenping_ | the relate fix patch is https://review.opendev.org/#/c/722651/ | |
| 01:02:33 | openstack | Launchpad bug 1874664 in OpenStack Compute (nova) "Boot more than one instances failed with accelerators in its flavor" [Medium,Confirmed] - Assigned to Wenping Song (wenping1) | |
| 01:02:33 | songwenping_ | the relate bug reported is https://bugs.launchpad.net/nova/+bug/1874664. pls help me have a look. | |
| 05:02:08 | openstackgerrit | Arthur Dayne proposed openstack/nova-specs master: new a test https://review.opendev.org/723794 | |
| 07:02:45 | gibi | good morning | |
| 07:02:57 | gibi | songwenping__: ack, I will look at it shortly | |
| 07:03:40 | songwenping__ | thanks, gibi. | |
| 07:11:16 | bauzas | good morning Nova | |
| 07:13:26 | aarents | good morning | |
| 07:14:59 | bauzas | fwiw, I'm a bit on and off this week due to children vacations at home | |
| 07:15:15 | bauzas | fortunately, it's after RC1 :) | |
| 07:18:26 | openstack | Launchpad bug 1874664 in OpenStack Compute (nova) "Boot more than one instances failed with accelerators in its flavor" [Medium,Confirmed] - Assigned to Wenping Song (wenping1) | |
| 07:18:26 | gibi | bauzas: a quick question. If we have a bug in the cyborg integration code https://bugs.launchpad.net/nova/+bug/1874664 and that code was added in Ussuri then Am I correct that such bug is an RC2 candidate? | |
| 07:19:48 | bauzas | gibi: MHO is that I don't think it's an urgent regression | |
| 07:20:05 | bauzas | gibi: given we weren't supporting Cyborg by Train | |
| 07:20:20 | gibi | bauzas: so it is a regression but it targets a small portion of a user base and it definetly does not break upgrade | |
| 07:20:48 | bauzas | gibi: do we have already a fix ? | |
| 07:20:57 | gibi | bauzas: there is a patch https://review.opendev.org/#/c/722651/ | |
| 07:21:05 | gibi | but I haven't looked it yet how complex it is | |
| 07:21:26 | bauzas | gibi: if not, we could just ask brinzhang to modify the documentation to say that Nova won't support more than one instance for Cyborg in Ussuri | |
| 07:21:49 | bauzas | and then we could backport the fix to a Ussuri later .z | |
| 07:21:56 | gibi | bauzas: good point. if the fix is problematic in RC timeframe then we can do a documentation patch | |
| 07:22:27 | gibi | bauzas: thanks for the consultation | |
| 07:23:22 | bauzas | gibi: anyway, it's just my opinion | |
| 07:23:32 | bauzas | brinzhang: are you around ? | |
| 07:25:12 | AJaeger | morning, any nova core available to review two tiny cleanups related to Babel/translations, please? https://review.opendev.org/#/c/723206/2 and https://review.opendev.org/#/c/720725/1 ? | |
| 07:26:12 | bauzas | AJaeger: ack, will look | |
| 07:26:51 | brinzhang | bauzas, gibi: ack | |
| 07:27:26 | brinzhang | bauzas, gibi: I can add that in accelerator support docs, to limit mutil create instances | |
| 07:28:18 | brinzhang | songwenping__ post the etherpad, I think that true, and he has some idea to fix this issue. | |
| 07:29:02 | brinzhang | I agree with bauzas, we can try to fix this issue, than consider to backport to U later ^ | |
| 07:29:18 | AJaeger | thanks, bauzas | |
| 07:30:19 | brinzhang | bauzas: That I just need to add the limit to https://docs.openstack.org/api-guide/compute/accelerator-support.html | |
| 07:30:35 | openstack | Launchpad bug 1874664 in OpenStack Compute (nova) "Boot more than one instances failed with accelerators in its flavor" [Medium,Confirmed] - Assigned to Wenping Song (wenping1) | |
| 07:30:35 | bauzas | brinzhang: yeah, I just provided a bug comment https://bugs.launchpad.net/nova/+bug/1874664 | |
| 07:31:22 | bauzas | brinzhang: fwiw, we did it like this for https://docs.openstack.org/nova/latest/admin/virtual-gpu.html | |
| 07:31:54 | bauzas | err, sorry this https://docs.openstack.org/nova/latest/admin/virtual-gpu.html#caveats | |
| 07:35:53 | bauzas | AJaeger: sent to the gate, thanks for clarifying | |
| 07:36:17 | AJaeger | thanks, bauzas | |
| 07:38:53 | AJaeger | could somebody import translations, please? https://review.opendev.org/722644 | |
| 07:39:40 | gibi | AJaeger: does this transaltion patch needs to be backported to stable/ussuri too? | |
| 07:41:47 | AJaeger | gibi: there will be a separte one proposed if available | |
| 07:41:53 | AJaeger | gibi: in other words: no | |
| 07:41:58 | AJaeger | translations are special | |
| 07:42:15 | AJaeger | gibi: and release notes are only translated on master | |
| 07:42:56 | gibi | AJaeger: thanks. | |
| 07:43:16 | AJaeger | thanks as well, gibi ;) | |
| 07:43:39 | gibi | AJaeger: does it mean that if we have a reno update merged after this point then we need to trigger a re-translation of the reno somehow? | |
| 07:46:41 | gibi | AJaeger: I mean we need to add at leat an upgrade warning to the ussuri reno due to a bug discovered after RC1 | |
| 07:57:51 | AJaeger | gibi: that happens automatically - after each merge, any new strings are send to the translation server and the translators translate at their leisure | |
| 07:58:15 | AJaeger | gibi: and if nobody translates, it shows up in English, so will be there for sure | |
| 07:58:32 | gibi | AJaeger: cool thanks | |
| 07:58:34 | AJaeger | gibi: so, no worries, continue adding releasenotes ;) | |
| 08:35:39 | openstackgerrit | OpenStack Proposal Bot proposed openstack/nova stable/ussuri: Imported Translations from Zanata https://review.opendev.org/723160 | |
| 08:43:37 | gibi | elod: ^^ | |
| 08:59:27 | openstack | Launchpad bug 1875418 in OpenStack Compute (nova) "Generated policy.json in Ussuri is broken by default" [Undecided,In progress] - Assigned to Ghanshyam Mann (ghanshyammann) | |
| 08:59:27 | gibi | zigo, gmann, dansmith, stephenfin: replied in https://bugs.launchpad.net/nova/+bug/1875418 let me know if you disagree or if my proposal is not feasible | |
| 08:59:44 | gibi | or if you have any other proposal as a way forward | |
| 09:01:25 | zigo | reading ... | |
| 09:03:50 | zigo | gibi: I very much agree with all you wrote, thanks you ! | |
| 09:04:00 | zigo | That's exactly what I would like to happen. | |
| 09:04:16 | gibi | zigo: let's see what others think about the feasibility | |
| 09:04:51 | gibi | as I have close to zero knowledge about the generator | |
| 09:18:49 | gibi | bauzas: how the vpgus are avoiding the problem what songwenping_ described for accelerators? | |
| 09:21:16 | gibi | songwenping_: do I understand correctly that your bug appers when you create two instances with one singe POST /servers request? | |
| 09:21:28 | bauzas | gibi: which problem, sorry ? | |
| 09:21:38 | songwenping_ | yes, gibi. | |
| 09:22:41 | bauzas | gibi: do I have a bug to lookup ? | |
| 09:23:19 | openstack | Launchpad bug 1874664 in OpenStack Compute (nova) "Boot more than one instances failed with accelerators in its flavor" [Medium,Confirmed] - Assigned to Wenping Song (wenping1) | |
| 09:23:19 | gibi | bauzas: same bug you looked at this morning https://bugs.launchpad.net/nova/+bug/1874664 | |
| 09:23:21 | gibi | songwenping_: thanks | |
| 09:23:28 | bauzas | hah | |
| 09:24:32 | bauzas | gibi: for vGPUs, we only ask for one RC | |
| 09:24:51 | bauzas | eg. resources:VGPU=2 | |
| 09:25:31 | bauzas | so, when calling Placement, it returns some allocations but all the VGPU ones are in the same RP | |
| 09:25:54 | gibi | bauzas: in case of instance multi create you need two separate allocation of one VGPUs | |
| 09:26:10 | gibi | bauzas: I mean instance multi create with min=2 max=2 | |
| 09:26:15 | bauzas | ah this | |
| 09:27:01 | openstack | Launchpad bug 1780225 in OpenStack Compute (nova) "Libvirt error when using --max > 1 with vGPU" [High,Confirmed] - Assigned to Sylvain Bauza (sylvain-bauza) | |
| 09:27:01 | bauzas | gibi: actually https://bugs.launchpad.net/nova/+bug/1780225 :-) | |
| 09:27:55 | gibi | bauzas: ohh but that leads to a different error in libvirt instead of placement | |
| 09:28:22 | bauzas | gibi: I should be testing it | |