Earlier  
Posted Nick Remark
#openstack-nova - 2022-09-27
16:29:10 bauzas #topic Review priorities
16:29:18 bauzas #link https://review.opendev.org/q/status:open+(project:openstack/nova+OR+project:openstack/placement+OR+project:openstack/os-traits+OR+project:openstack/os-resource-classes+OR+project:openstack/os-vif+OR+project:openstack/python-novaclient+OR+project:openstack/osc-placement)+(label:Review-Priority%252B1+OR+label:Review-Priority%252B2)
16:29:43 bauzas nothing to tell about those
16:30:53 bauzas moving on
16:31:04 bauzas #topic Stable Branches
16:31:08 bauzas elodilles: time is yours
16:31:14 elodilles #info stable/yoga is unblocked openstacksdk fix has merged
16:31:22 elodilles #info stable/stein (and older) are blocked: grenade and other devstack based jobs fail with the same timeout issue as stable/train was previously
16:31:30 elodilles #info stable branch status / gate failures tracking etherpad: https://etherpad.opendev.org/p/nova-stable-branch-ci
16:31:42 elodilles and that is all from me
16:32:16 bauzas ok
16:32:39 bauzas thanks all for unblocking the yoga branch
16:33:37 elodilles yepp, thanks to openstacksdk team
16:34:04 bauzas ok, anything to add ?
16:34:41 bauzas ok, eventually the last topic then
16:34:51 bauzas #topic Open discussion
16:35:02 bauzas #link https://review.opendev.org/c/openstack/nova-specs/+/859290 Add support of Napatech SmartNICs
16:35:08 bauzas justas_napa: tell us more
16:35:18 justas_napa So maybe I'll start with a quick intro
16:35:35 justas_napa I'm Justas Poderys, product architect working on Napatech LinkVirtualization products
16:36:05 justas_napa We have been running OpenStack on our SmartNICs for over a year now, and internally support (via patches) everything from Victoria to Zed
16:36:41 justas_napa Going forward we'd like to enable OpenStack to work with Napa's smartnics out of the box
16:36:56 justas_napa for that we would like to add a VIF type corresponding to Napatech SmartNIC
16:37:08 justas_napa ein a simmilar fashion like Netronomes Agillio
16:37:33 bauzas do you work with Neutron ?
16:37:33 justas_napa But at the same time we do not need a new VNIC type
16:37:38 justas_napa Yes
16:37:46 bauzas so I guess you also have a neutron spec
16:38:15 justas_napa We have extremely small update to Neutron, but yes, we will do a Neutron spec as well
16:38:55 bauzas this doesn't look a lot of change for nova, right?
16:39:11 bauzas we already support *some* smartnic experiments
16:39:13 justas_napa no, I think it's less than 50 lines
16:40:13 bauzas justas_napa: I guess you already know about what Nova supports by the Wallaby release ?
16:40:16 justas_napa We will publish all changes to Nova and Os-Vif on OpenDev in the next few days
16:40:33 justas_napa bauzas What do you mean?
16:41:08 bauzas sorry, wrong release
16:41:13 bauzas justas_napa: I was talking of https://specs.openstack.org/openstack/nova-specs/specs/xena/implemented/sriov-smartnic-support.html
16:41:39 bauzas oh wait no, my bad
16:41:45 bauzas again, wrong spec
16:42:15 bauzas https://specs.openstack.org/openstack/nova-specs/specs/yoga/implemented/integration-with-off-path-network-backends.html
16:42:57 bauzas anyway, I just pointed out the existing specs
16:43:10 bauzas afaics, this is just a review ask
16:43:28 bauzas so, we'll review it
16:43:43 bauzas justas_napa: would you want to discuss your topic during the PTG ?
16:43:51 bauzas in case we have concerns
16:43:52 justas_napa Sure, I can do that
16:44:15 bauzas thanks
16:44:19 bauzas if you can't do, no worriezs
16:44:27 justas_napa Re: off-path networking backed - we are in the progress of integrating Intel Big Spring Canyon platform to du just that
16:44:32 bauzas justas_napa: and you don't need to be on the whole PTG times
16:44:57 bauzas justas_napa: we could just ping you when we are at the topic
16:45:01 justas_napa sure
16:45:15 bauzas thanks
16:45:30 justas_napa So what's the path from here: 1. We puiblish code changes and ask for review in channel?
16:45:35 justas_napa and then PTG?
16:45:38 bauzas anyone having questions or thoughts for this ?
16:45:49 bauzas justas_napa: indeed
16:45:56 justas_napa cool.
16:46:17 bauzas we will review your spec first, but if you have implementation changes, that would be nice
16:46:29 bauzas as if we have concerns, we could just look at your changes
16:46:36 justas_napa Yes, we have implemetation changes as well, since we are running it internally anyway
16:46:38 gibi I agree with bauzas
16:46:40 bauzas to understand what you need
16:47:21 justas_napa btw - is it OK to bring one more topick to PTG. Specifically, how to support Packed Ring Libvirt option (packed=on) and which project should implement that.
16:47:27 bauzas justas_napa: I guess you know how to depend on neutron changes ?
16:47:38 bauzas justas_napa: yeah, no worries
16:47:46 bauzas the PTG is here for discussing
16:48:08 justas_napa Yes, I'm aware that we need changes in Nova *and* Neutron to make this work
16:48:18 bauzas justas_napa: my only concern is about how to test your nova changes if you need an os-vif release
16:48:33 bauzas as you can't Depends-On
16:48:52 bauzas gibi: correct for os-vif, right?
16:49:17 bauzas I meant, can we depend-on os-vif patches ?
16:49:29 gibi I'm not sure either
16:49:49 justas_napa I'll put this also as TBD item for our dev team
16:49:59 bauzas justas_napa: do you understand about my question ?
16:50:18 bauzas justas_napa: if you upload three changes, each for a different repo
16:50:32 gibi bauzas: it depends on how the zuul job is set up. Are we testing with released os-vif lib or are we taking master all the time?
16:50:37 bauzas justas_napa: in general, you can depend a gerrit cvhange on another one
16:50:58 justas_napa Yes, I understand the issue of cross-project dependancy
16:51:14 gibi bauzas: if the former, then the a DNM nova patch on top of the nova series could configure zuul to use the later method + add a Depends-On to the os-vig patch
16:51:18 justas_napa and that our change is not atomic across the projects
16:51:24 bauzas justas_napa: but here, the question is that for some libraries, we maybe use a specific release and not using master
16:51:52 bauzas gibi: yeah, that would work
16:52:06 bauzas justas_napa: do you understand what gibi is explaining ?
16:52:08 justas_napa OK, I understand the issue, but I'm not sure I can offer a solution.
16:52:44 bauzas justas_napa: well, we maybe have a solution, that's the point
16:52:45 justas_napa One way is to use our ci/cd setup where we runn all regression tests and have control of releases of all projects
16:53:21 bauzas anyway, we should be discussing this in the spec
16:53:43 bauzas justas_napa: are you saying you could provide a third-party CI for nova/neutron ? :D
16:53:58 bauzas this would be lovely
16:54:16 justas_napa not sure what you mean 3rd party
16:54:44 justas_napa but we have our own ci/cd pipeline running integration tests with our changes
16:54:55 bauzas ok, then we'll discuss this in the spec and we can also provide you some links if you want
16:55:09 justas_napa to ensure we do not break anything with our OS patches
16:55:19 bauzas justas_napa: are you running tempest tests ?
16:55:22 justas_napa yes
16:55:44 bauzas cool, then that's something we need to discuss then
16:55:55 bauzas anyway, I guess we're done about your topic
16:56:01 bauzas we'll continue discussing
16:56:21 justas_napa sure, thanks for all questions and comments

Earlier   Later