03:06:39 <Yumeng> #startmeeting openstack-cyborg
03:06:40 <openstack> Meeting started Thu Jul 23 03:06:39 2020 UTC and is due to finish in 60 minutes.  The chair is Yumeng. Information about MeetBot at http://wiki.debian.org/MeetBot.
03:06:41 <openstack> Useful Commands: #action #agreed #help #info #idea #link #topic #startvote.
03:06:43 <openstack> The meeting name has been set to 'openstack_cyborg'
03:06:45 <Yumeng> #topic Roll call
03:06:52 <Yumeng> #info Yumeng
03:07:00 <xinranwang__> #info xinranwang__
03:07:01 <brinzhang_> #info brinzhang_
03:07:05 <chenke> #info chenke
03:07:19 <Yumeng> #topic Agenda
03:07:32 <songwenping__> #info songwenping__
03:07:49 <Yumeng> #topic policy default refresh
03:08:27 <Yumeng> I updated this morning https://review.opendev.org/#/c/740542/
03:08:59 <Yumeng> now this patch submmited the basic policies and device_profile policies, and basic framwork
03:10:18 <brinzhang_> Yumeng: I was reviewed just now, but I am sorry not looked into. in general, LGTM, but that need to add the UT, I left comment inline.
03:11:05 <Yumeng> both the old and new policies will both work during development process
03:11:20 <brinzhang_> Yumeng: after I compeleted thing in my hands, I will check again about the implemention details.
03:11:51 <brinzhang_> sound good.
03:12:42 <brinzhang_> we should keep old policies and new policies are all run good, thanks
03:12:46 <Yumeng> brinzhang_: yes we need test. but not as easy as a unit test update in current API tests. that should be another tests for only policies, we should add new policy test framwork, that would be a big work.
03:13:34 <brinzhang_> Yumeng: I know, but in the patch if you dont add the UT, I dont think it's good to go.
03:13:39 <Yumeng> I will submit the policy test in another patch, I just don't want to make the first patch too big.
03:14:24 <brinzhang_> you can splite the UT for add new policies, another patch is to add the new policies to the interface
03:15:04 <Yumeng> for this patch, I have tested the function by current unit test, the test_create case of device_profile, I have pasted the detailed debug info to the commit message: http://paste.openstack.org/show/796229/
03:15:09 <xinranwang__> Policy test framework is a big work. We can just add test in API test first
03:16:21 <xinranwang__> For example, add test in device_profile API,  test create dp by non admin user, and it fails
03:16:32 <brinzhang_> xinranwang__: yeah, we can. But the better way is add as the same time.
03:16:39 <xinranwang__> Do you think this make sense
03:17:17 <brinzhang_> It make sense to me. But I also hope that it is not a TODO.
03:18:32 <xinranwang__> We can add api test in this patch. Policy test framwork can be in a new patch.
03:18:42 <xinranwang__> What do you think Yumeng
03:19:50 <Yumeng> xinranwang__: yes! agree, current API tests will go through the policy check in each test. I will start in the next patch very soon to submit policy test for device_profile
03:20:49 <brinzhang_> thanks
03:21:44 <Yumeng> but for this patch, I think the detailed debug info has shown the policy check process, which can show that this patch basically works, so we can start review.
03:23:26 <Yumeng> this patch can be merged after the next policy test patch is passed and submitted, but can start review now. what do you think?
03:24:05 <brinzhang_> Yumeng, do you test the older policies? I saw you deprecated the old policies, I am not sure it's ok for all apis
03:24:06 <brinzhang_> https://review.opendev.org/#/c/740542/10/cyborg/policies/base.py@95
03:25:20 <brinzhang_> Yumeng: this patch I think we (xinranwang__) dont reviewed into, after checked again, we can decide whether let it to go, do you think so?
03:25:48 <brinzhang_> is there a milestone?
03:27:10 <Yumeng> the old policies are "not actually" deprecated, they won't actually deprecated until we set [oslo_policy] enforce_policy=True in cyborg.conf
03:27:53 <Yumeng> the old policies work for now, they just marked as deprecated, not actually deprecated.
03:28:54 <Yumeng> emmm. I think we should add tests for new policies, for the old policies, we will deprecate them after all the new policies are implemented
03:29:36 <brinzhang_> agree, so I think this change should have UT to cover the old policies and the new policies, and checked the enforce_policy config (of course you can do in another patch, but it will be not easy to review).
03:30:15 <brinzhang_> no, after you completed the new policies, we also need to keep the old policies run until W or X release
03:30:46 <brinzhang_> To ensure backward compatibility, we should not remove it so quickly.
03:31:05 <brinzhang_> What do you think? Yumeng, xinranwang__, songwenping__
03:31:20 <songwenping__> Yumeng: you can fix the ut and pep8 first, ths.
03:32:05 <Yumeng> brinzhang_: yes, that's what I planed. I have NOTE them in the code. deprecate_since W
03:32:25 <xinranwang__> Sorry, I am back
03:32:39 <xinranwang__> brinzhang_:  yes, agree
03:33:08 <Yumeng> xinranwang__: oPS, will see the pep8 error. don't know why. works fine in my env.
03:35:32 <brinzhang_> ok. I have nothing to say of this patch now
03:35:45 <brinzhang_> I have two patches need to go, it was submited for a looong time. https://review.opendev.org/#/c/734406/ and https://review.opendev.org/#/c/734403/
03:35:59 <Yumeng> brinzhang_: anyway. this patch defines the overall framwork, I wanna you guys help to review it.
03:36:07 <brinzhang_> Yumeng, xinranwang__, and all: Hope you can review all, thanks
03:36:42 <brinzhang_> Yumeng: of course ^^, it's my pleasure
03:36:48 <songwenping__> bingzhang_: will look into this patch.
03:36:52 <Yumeng> brinzhang_: about the milestone, that was set to milestone-2, but we will see if we need postpone
03:36:55 <xinranwang__> https://www.irccloud.com/pastebin/z94K8fNV/
03:37:32 <Yumeng> brinzhang_: yse, will review
03:37:40 <brinzhang_> thanks
03:38:25 <songwenping__> brinzhang_: sorry for error spell.
03:38:45 <brinzhang_> xinranwang__: IRC cloud need to be jump the firewall, otherwise we cannot connect to it
03:39:07 <Yumeng> xinranwang__: aha. thanks xinran. I will fix it.
03:39:47 <xinranwang__> Oh, I thought is works for PRC, never mind
03:40:30 <Yumeng> Inspur FPGA driver spec: https://review.opendev.org/#/c/730760/
03:40:32 <Yumeng> songwenping__: just reviewed this morning. pls see my reply.
03:40:56 <Yumeng> we can merge this before milestone-2
03:40:57 <songwenping__> Yumeng: yeah, i see it.
03:41:12 <Yumeng> also Intel QAT driver ready for review:https://review.opendev.org/#/c/728014/
03:41:20 <Yumeng> I will review again today.
03:41:32 <xinranwang__> thanks a lot
03:41:33 <songwenping__> thanks for review, i will check again.
03:42:05 <Yumeng> And shogo's is also milestone-2. Programming support ready for review: https://review.opendev.org/#/c/698190/
03:42:17 <Yumeng> a big work.
03:42:26 <brinzhang_> songwenping__, xinranwang__:will review later
03:42:39 <xinranwang__> thanks brinzhang_
03:43:24 <Yumeng> xinranwang__: can you also ask haibing to help review this, I'm not expert in programming.
03:43:30 <songwenping__> thanks brinzhang__
03:43:45 <xinranwang__> Yes, sure
03:45:08 <Yumeng> thanks!
03:45:22 <Yumeng> I have nothing else to mention for today.
03:45:50 <Yumeng> Do you guys want to bring up anything else?
03:46:44 <brinzhang_> nothings by me
03:46:44 <xinranwang__> Just noticed that cyborg-tempest-plugins has no doc
03:47:08 <xinranwang__> shall we add some docs for it?
03:48:08 <brinzhang_> if  necessagr we can
03:48:26 <Yumeng> not sure if we need.
03:48:35 <brinzhang_> May there will be add in the future
03:48:48 <Yumeng> I can take a look, we can decide later.:)
03:48:58 <brinzhang_> yeah
03:49:18 <Yumeng> ok. sounds good.
03:49:41 <Yumeng> let's wrap up thisĀ meetingĀ and have a good lunch.
03:49:46 <xinranwang__> Ok, it's not urgent. Just mention it here.lol
03:50:20 <xinranwang__> Ok, thanks. Bye
03:50:22 <Yumeng> see you next week. Thank you for attending! bye
03:50:43 <songwenping__> bye
03:51:00 <Yumeng> #endmeeting