| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2020-04-27 | |||
| 16:16:05 | sean-k-mooney | nova and neuton have completely different approchs to self.assert* methods | |
| 16:16:13 | gmann | sean-k-mooney: true, that is why many projects cap it | |
| 16:16:27 | sean-k-mooney | neutron blocked the use of any not in py27 and nova used mock the lib | |
| 16:18:03 | gmann | gibi: dansmith lbragstad in addition to release note for nova ussuri, should we have a clear doc from oslo/keystone or somewhere generic on 'how to generate and use policy file and how deployer can be broken for xyz cases'. i mean a single recommended way instead of supporting all possible way deployment doing ? | |
| 16:19:15 | sean-k-mooney | gmann: well that would be in oslo.policy correct | |
| 16:19:22 | sean-k-mooney | or rather should be | |
| 16:19:41 | sean-k-mooney | e.g. discribing how the lib should be used by developers | |
| 16:20:13 | gmann | I am not sure, we have. but lbragstad or bnemec can point to if there is any. | |
| 16:21:09 | gmann | i meant explicitly saying, 'this way of re-genrating policy file or having not-override rule in policy file etc etc can break you if you do not carefully audit on upgrades' | |
| 16:27:47 | openstackgerrit | Takashi Natsume proposed openstack/nova master: Fix list rendering in the accelerator support doc https://review.opendev.org/721846 | |
| 16:28:00 | lbragstad | gmann we have this | |
| 16:28:02 | lbragstad | https://bugs.launchpad.net/oslo.policy/+bug/1853170 | |
| 16:28:02 | openstack | Launchpad bug 1853170 in oslo.policy "Need documentation on recommended operator workflow for deprecated policies" [High,Triaged] | |
| 16:28:02 | openstackgerrit | Takashi Natsume proposed openstack/nova master: Update contributor guide for Victoria https://review.opendev.org/722647 | |
| 16:30:05 | lbragstad | gmann i don't think there is anything in review for that, yes | |
| 16:30:07 | lbragstad | yet* | |
| 16:32:06 | gmann | lbragstad: i see, thanks | |
| 16:32:33 | stephenfin | melwitt: confident enough to bump your +1 to +2 now? https://review.opendev.org/#/c/720725/ | |
| 16:35:47 | gibi | gmann: I have to stop for today. If you start writing a reno update for the policy thing then please link it to me and I will read it first thing in the morning | |
| 16:37:16 | gmann | gibi: ok, I will update the upgrade section for now to mention the re-generated policy file case. and later we can work on some generic doc (bug/1853170). | |
| 16:37:44 | gibi | gmann: ack, thanks | |
| 17:43:53 | sean-k-mooney | stephenfin: can you take a look at https://review.opendev.org/#/c/722407/ | |
| 17:44:02 | sean-k-mooney | stephenfin: it needt to merge before your change can merge | |
| 17:48:58 | AJaeger | any nova core available for two tiny cleanups related to Babel/translations, please? https://review.opendev.org/#/c/723206/2 and https://review.opendev.org/#/c/720725/1 ? | |
| 18:01:28 | openstackgerrit | Merged openstack/python-novaclient master: Use unittest.mock instead of third party mock https://review.opendev.org/723152 | |
| 18:19:56 | openstackgerrit | Merged openstack/nova master: Add placeholder migrations for Ussuri backports https://review.opendev.org/722546 | |
| 18:51:11 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Clarify the policy new defaults upgrade notes https://review.opendev.org/723645 | |
| 18:51:41 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Clarify the policy new defaults upgrade notes https://review.opendev.org/723645 | |
| 19:59:04 | openstackgerrit | Merged openstack/nova master: Fix list rendering in the accelerator support doc https://review.opendev.org/721846 | |
| 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: Partial cherry-pick of _check_allocation_during_evacuate() https://review.opendev.org/723695 | |
| 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:52 | openstackgerrit | Artom Lifshitz proposed openstack/nova stable/queens: DNM: Don't heal allocations for deleted servers https://review.opendev.org/723697 | |
| 21:12:52 | openstackgerrit | Artom Lifshitz proposed openstack/nova stable/queens: DNM: Partial cherry-pick of FakeResponse https://review.opendev.org/723698 | |
| 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: Partial cherry-pick of report client changes https://review.opendev.org/723750 | |
| 23:14:59 | openstackgerrit | Artom Lifshitz proposed openstack/nova stable/queens: DNM: Add a placement audit command https://review.opendev.org/723751 | |
| #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 | songwenping_ | the relate bug reported is https://bugs.launchpad.net/nova/+bug/1874664. pls help me have a look. | |
| 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) | |
| 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 | 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: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: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 | bauzas | brinzhang: yeah, I just provided a bug comment https://bugs.launchpad.net/nova/+bug/1874664 | |
| 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: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 ;) | |