| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2021-07-06 | |||
| 16:16:02 | gibi | cool, thanks | |
| 16:16:09 | gibi | any other stable business for today? | |
| 16:16:31 | elodilles | nothing from me | |
| 16:17:03 | gibi | #topic Sub/related team Highlights | |
| 16:17:06 | gibi | Libvirt (bauzas) | |
| 16:17:13 | bauzas | nothing to tell | |
| 16:17:25 | gibi | #topic Open discussion | |
| 16:17:35 | gibi | there is one topic on the agenda | |
| 16:17:57 | gibi | yonglihe: Hi, gibi and everyone, here is the smartnic patch set status: 6 of 7 patches got +2 from alex_xu, the functional testing is under reviewing. seems a bulk of core time needed, thanks anyone's help on it, sincerely. https://review.opendev.org/q/topic:%22bp%252Fsriov-smartnic-support%22+(status:open%20OR%20status:merged) | |
| 16:18:11 | gibi | so basicly yonglihe asking for review | |
| 16:18:57 | gibi | I'm pretty swamped these days so if somebody has time, then please go ahead | |
| 16:19:01 | sean-k-mooney | oh that reminds me ill respin the review priorty patch sorry forgot to do that | |
| 16:19:14 | gibi | sean-k-mooney: good point, we can start using that | |
| 16:19:36 | stephenfin | I have one other thing to add | |
| 16:19:41 | sean-k-mooney | ill do that after the meeting | |
| 16:19:45 | gibi | thanks | |
| 16:19:57 | gibi | I see no cores queueing up for reviewing the smart nic impl so I will keep it on my radar | |
| 16:20:02 | gibi | stephenfin: stage is yours then | |
| 16:20:21 | stephenfin | I brought up the changes to os-vif stable core membership on openstack-discuss a week and a half ago and didn't hear any objections http://lists.openstack.org/pipermail/openstack-discuss/2021-June/023305.html | |
| 16:20:34 | stephenfin | so I think we can proceed with that change now | |
| 16:21:05 | gibi | elodilles, lyarwood, sean-k-mooney: any last minute objection? | |
| 16:21:18 | sean-k-mooney | not form me | |
| 16:21:21 | gibi | or from anybody else? | |
| 16:21:41 | gibi | then lets do it | |
| 16:21:44 | elodilles | no objection from me | |
| 16:22:43 | gibi | cool | |
| 16:22:53 | gibi | then I will note the agreement in the review https://review.opendev.org/c/openstack/project-config/+/798071 | |
| 16:23:05 | gibi | anything else for today? | |
| 16:24:31 | gibi | if not then thanks for joining today. | |
| 16:24:41 | gibi | #endmeeting | |
| 16:24:41 | opendevmeet | Meeting ended Tue Jul 6 16:24:41 2021 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | |
| 16:24:41 | opendevmeet | Minutes: https://meetings.opendev.org/meetings/nova/2021/nova.2021-07-06-16.00.html | |
| 16:24:41 | opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/nova/2021/nova.2021-07-06-16.00.txt | |
| 16:24:41 | opendevmeet | Log: https://meetings.opendev.org/meetings/nova/2021/nova.2021-07-06-16.00.log.html | |
| 16:25:37 | sean-k-mooney | gibi: cool am i started reviewing the placment chagne by the way i got intrupted yesteday so just pushed what i have sofar. ill review it porperly again tomorow | |
| 16:25:41 | sean-k-mooney | https://review.opendev.org/c/openstack/placement/+/784020 | |
| 16:26:02 | sean-k-mooney | so far it look pretty good but i think you can simplfy it slightly | |
| 16:26:03 | gibi | sean-k-mooney: thanks a lot | |
| 16:27:17 | sean-k-mooney | tl;dr i dont think we need one branc to set it to none and another to set it to a different value although what you have now is still technially correct and will work | |
| 16:27:52 | stephenfin | bauzas: have I answered your questions on https://review.opendev.org/c/openstack/nova-specs/+/793199 ? Care to bump your +1 if so? | |
| 16:28:34 | gibi | sean-k-mooney: ack, I will look into that suggestion later this week | |
| 16:28:43 | gibi | my brain already fried for today | |
| 16:28:58 | gmann | stephenfin: gibi can you check policy warnings patches https://review.opendev.org/c/openstack/nova/+/799422 and placement which is already reviewed by stephenfin https://review.opendev.org/c/openstack/placement/+/799418 | |
| 16:28:59 | bauzas | stephenfin: just did +W | |
| 16:29:16 | sean-k-mooney | ack i was not leand towrad -1 more of a nit anyway | |
| 16:29:24 | gibi | gmann: ack | |
| 16:30:21 | gmann | thanks | |
| 16:34:41 | sean-k-mooney | brb | |
| 16:36:58 | stephenfin | gmann: Apologies for inflicting unnecessary pain on you there. The warning seemed like the only way to get people to use the new thing, but it is very noisy to be fair | |
| 16:37:16 | stephenfin | i.e. https://github.com/openstack/oslo.policy/blob/3.7.0/oslo_policy/policy.py#L1538 | |
| 16:37:57 | gmann | stephenfin: yeah, I am updating all the current usage for those warning https://review.opendev.org/q/topic:%22fix-oslo-policy-warnings%22+(status:open%20OR%20status:merged) | |
| 16:38:34 | gmann | stephenfin: and once we merge all these then I am thinking to convert it to error from oslo side so that all the DeprecatedRule usgae in new RBAC work will be proper | |
| 16:39:03 | opendevreview | Merged openstack/nova-specs master: Repropose "vm-boot-with-unaddressed-port" spec https://review.opendev.org/c/openstack/nova-specs/+/793199 | |
| 16:49:00 | sean-k-mooney | ralonsoh: stephenfin: https://review.opendev.org/c/openstack/nova/+/799680 so it looks that os-vif patch works well | |
| 16:49:25 | sean-k-mooney | i modified the nova live migration and grenade jobs to use it | |
| 16:50:16 | ralonsoh | sean-k-mooney, do you think we need to add more testing? to add more tempest tests? | |
| 16:50:21 | ralonsoh | just for the spec | |
| 16:51:29 | sean-k-mooney | i think the exising jobs can be configured to cover the different cases. e.g. we can test ovn with per port bridges in nova-livemigration but use ml2/ovs with iptables in nova-livemigration-ceph | |
| 16:52:02 | sean-k-mooney | ralonsoh: i would hope that neutron has a live migration jobs too where it can be enabled. | |
| 16:52:11 | ralonsoh | sean-k-mooney, btw, I think you should add neutron dependency to this patch | |
| 16:52:23 | sean-k-mooney | on the os-vif side i may add a grenade job | |
| 16:52:35 | sean-k-mooney | ralonsoh: against which patch? the spec? | |
| 16:52:43 | ralonsoh | this nova patch | |
| 16:53:09 | sean-k-mooney | sorry can you rephase that | |
| 16:53:12 | ralonsoh | changing when we send the plugged event | |
| 16:53:15 | ralonsoh | but of course | |
| 16:53:22 | ralonsoh | we need Nova to accept this | |
| 16:53:32 | ralonsoh | nevermind | |
| 16:53:34 | sean-k-mooney | oh right that is a sperate change | |
| 16:53:46 | ralonsoh | ok, I'll push Neutron patch first | |
| 16:54:10 | sean-k-mooney | the corrent nova behavior is to just not wait for ovn in prelive migration | |
| 16:54:26 | ralonsoh | exactly and wait for the port plug event | |
| 16:54:28 | sean-k-mooney | so once your neutron patch is up we can add a nova one that will make it start waiting | |
| 16:55:24 | sean-k-mooney | ping me when the neutorn patch is up if you would like me to write the nova one. its should not be that hard but i probaly will not get to it until monday | |
| 16:55:45 | ralonsoh | sean-k-mooney, sure, tomorrow | |
| 16:58:26 | sean-k-mooney | ralonsoh: no worries its getting late for you | |
| 17:26:53 | stephenfin | sean-k-mooney: https://review.opendev.org/c/openstack/os-vif/+/798055 looks pretty good. If you've time to address my comments tonight I'll +2 in the AM | |
| 17:28:19 | sean-k-mooney | stephenfin: am i want to poc something else this evening but ill adress it tomorow | |
| 17:35:43 | opendevreview | Merged openstack/placement master: Fix oslo policy DeprecatedRule warnings https://review.opendev.org/c/openstack/placement/+/799418 | |
| 18:16:30 | opendevreview | Merged openstack/nova master: Fix oslo policy DeprecatedRule warnings https://review.opendev.org/c/openstack/nova/+/799422 | |
| 18:52:39 | opendevreview | Merged openstack/os-vif master: update os-vif ci to account for devstack default changes https://review.opendev.org/c/openstack/os-vif/+/798038 | |
| #openstack-nova - 2021-07-07 | |||
| 03:31:52 | opendevreview | wangzhenmeng proposed openstack/nova master: Three CPU parameters, mode, model and vendor_id, are added to flavor, some guest os do not support new CPU, so you need to specify a specific CPU when starting. https://review.opendev.org/c/openstack/nova/+/796986 | |
| 03:34:35 | opendevreview | wangzhenmeng proposed openstack/nova master: Add CPU parameters to flavor https://review.opendev.org/c/openstack/nova/+/796986 | |
| 09:24:55 | ralonsoh | hi folks, do you know what I need to set in libvirt for live-migration? | |
| 09:25:00 | ralonsoh | I'm having this error: | |
| 09:25:01 | ralonsoh | libvirt.libvirtError: Secret not found: no secret with matching uuid '70447a0f-443f-4b30-a154-c75706973e08' | |
| 09:25:38 | sean-k-mooney[m] | are you using ceph | |
| 09:25:49 | sean-k-mooney[m] | normally you do not need to set anything in libvirt | |
| 09:26:00 | ralonsoh | yes I configured ceph | |
| 09:26:07 | ralonsoh | I'll check it again | |
| 09:26:27 | sean-k-mooney[m] | you need to configure the ceph key ring as a secret but the devstack plugin does that for you | |
| 09:26:27 | ralonsoh | (maybe not in the second compute node...) | |
| 09:27:18 | ralonsoh | yeah, I think I didn't configure ceph in the second compute node | |
| 09:27:20 | ralonsoh | thanks! | |
| 09:28:17 | sean-k-mooney[m] | https://opendev.org/openstack/devstack-plugin-ceph/src/branch/master/devstack/lib/ceph#L244 | |
| 09:28:41 | sean-k-mooney[m] | that is what adds the user key ring as a secret | |
| 09:28:59 | sean-k-mooney[m] | so you could just do that or restack the compute | |
| 09:29:41 | sean-k-mooney[m] | you also need to copy over the ceph.confirm | |
| 09:29:49 | sean-k-mooney[m] | *ceph.conf | |
| 09:30:01 | ralonsoh | sean-k-mooney[m], btw, I think slaweq made a good point on https://review.opendev.org/c/openstack/neutron-specs/+/799198/3/specs/xena/ovn-auxiliary-port-bridge-live-migration.rst#51 | |