| *** tetsuro has joined #openstack-cyborg | 00:13 | |
| *** TxGirlGeek has quit IRC | 00:22 | |
| *** openstackstatus has joined #openstack-cyborg | 00:43 | |
| *** ChanServ sets mode: +v openstackstatus | 00:43 | |
| *** chenke has joined #openstack-cyborg | 01:18 | |
| *** Sundar has joined #openstack-cyborg | 02:10 | |
| Sundar | Hello all | 02:11 |
|---|---|---|
| brinzhang | \O | 02:11 |
| Sundar | Hi brinzhang. Anybody else here? | 02:12 |
| brinzhang | I think they are waiting 3:00UTC, right? | 02:13 |
| brinzhang | Few people joined irc | 02:13 |
| *** s_shogo has joined #openstack-cyborg | 02:14 | |
| *** xinranwang has joined #openstack-cyborg | 02:14 | |
| Sundar | Oh yes, with daylight savings, my meeting time has shifted again | 02:14 |
| brinzhang | ah, nothing, wait 45min later | 02:16 |
| *** Yumeng has joined #openstack-cyborg | 02:16 | |
| chenke | brinzhang https://review.opendev.org/#/c/710911/ need your reply. | 02:20 |
| brinzhang | chenke: will check | 02:21 |
| chenke | ok. thanks. | 02:22 |
| Sundar | chenke, brinzhnag: I addressed your comments in https://review.opendev.org/#/c/712231/ | 02:25 |
| Sundar | brinzhang: ^ | 02:26 |
| chenke | Ye. I reply you. seems UT passed. | 02:26 |
| brinzhang | Sundar: hi | 02:29 |
| brinzhang | Sundar: thanks add that test case of bound exception, that what I want ^^ | 02:31 |
| openstackgerrit | chenker proposed openstack/cyborg master: Bugfix: Set the appropriate status code for ServiceUnavailable https://review.opendev.org/712283 | 02:32 |
| Sundar | brinzhang: That's what I was mentioning yesterday. If I apply the UT to self.Patch_json, it would fail. I solved it by factoring out _check_if_bound, and applying UT to it separately. | 02:33 |
| Sundar | We still need to solve the problem of handling the HTTP error for exception test cases for post/patch etc. | 02:34 |
| chenke | Your latest approach is appropriate, UT should only test this function. | 02:34 |
| Sundar | chenke, brinzhang: You may want to read the new commit message. I added lots more info in there. | 02:34 |
| brinzhang | Sundar: I will check again your latest patch at afternoon. | 02:35 |
| Sundar | Sure, np | 02:35 |
| brinzhang | "We still need to solve the problem of handling the HTTP error for exception test cases for post/patch etc.", I hasn't concern this issue, in future, I think I will meet | 02:36 |
| brinzhang | If anyone can get this clearly, can push a commit that we can check together | 02:37 |
| chenke | Sundar cool. The latest commit message is clear. | 02:37 |
| openstackgerrit | YumengBao proposed openstack/cyborg master: Fix GPU's improper cpid_type allowed value and attach_handle_type value https://review.opendev.org/709433 | 02:38 |
| openstackgerrit | YumengBao proposed openstack/cyborg master: Fix GPU's improper cpid_type allowed value and attach_handle_type value https://review.opendev.org/709433 | 02:38 |
| brinzhang | Sundar, chenke, xinranwang, Yumeng, I want to start use launchpad, do you think all? | 02:39 |
| Sundar | brinzhang: launchpad for blueprints? | 02:40 |
| brinzhang | I think it's easier to manager bug,blueprint, as I know cinder,nova,neutron all use it | 02:40 |
| Sundar | I have no objection either way. But we would need to migrate all stories in Storyboard to launchpad. | 02:41 |
| brinzhang | Any tools can do? | 02:42 |
| Sundar | brb in 10 min | 02:49 |
| *** shaohe_feng has joined #openstack-cyborg | 02:50 | |
| chenke | Didn't it have been discussed in the email before? According to the opinions of community leaders, cyborg has already used storyborad, so there is no need to change it back. I think storyborad is good. | 02:51 |
| Sundar | back | 02:57 |
| brinzhang | chenke: I removed -1 from that patch, but I think Sundar's concern is right, how about change the fpga_program_v2 to fpga_program in this patch? | 02:59 |
| Sundar | #startmeeting openstack-cyborg | 03:00 |
| chenke | Ok. thanks. I replied, in my opinion, that's not what this patch should do. I think the theme should be consistent. | 03:00 |
| openstack | Meeting started Thu Mar 12 03:00:37 2020 UTC and is due to finish in 60 minutes. The chair is Sundar. Information about MeetBot at http://wiki.debian.org/MeetBot. | 03:00 |
| openstack | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 03:00 |
| *** openstack changes topic to " (Meeting topic: openstack-cyborg)" | 03:00 | |
| openstack | The meeting name has been set to 'openstack_cyborg' | 03:00 |
| Sundar | o/ Sundar | 03:00 |
| chenke | #info chenke | 03:01 |
| xinranwang | Hi all | 03:01 |
| Yumeng | #info Yumeng | 03:01 |
| brinzhang | #info brinzhang | 03:01 |
| s_shogo | #info s_shogo | 03:01 |
| s_shogo | Hi all | 03:01 |
| xinranwang | #info xinranwang | 03:01 |
| Sundar | First off, I am happy t o propose s_shogo and brinzhang as core reviewers: http://lists.openstack.org/pipermail/openstack-discuss/2020-March/013230.html | 03:02 |
| Sundar | It will take effect next Friday (March 20) barring any objection. | 03:03 |
| brinzhang | Sundar: It's my honor, thanks, thanks all | 03:03 |
| Sundar | brinzhang: welcome, you have earned it. | 03:04 |
| chenke | Congratulations Brin, Shogo. | 03:04 |
| Yumeng | congrats! brinzhang,and s_shogo | 03:04 |
| s_shogo | Thanks all! | 03:04 |
| xinranwang | congratulations! | 03:05 |
| brinzhang | Thanks Sundar, chenke, Yumeng, xinranwang, s_shogo ^^ | 03:06 |
| Sundar | Secondly, we should take stock of our progress in the U release. | 03:06 |
| Sundar | Here's the U release planning etherpad: https://etherpad.openstack.org/p/cyborg-ptg-ussuri | 03:07 |
| Sundar | Nova integration | 03:08 |
| Sundar | Nova integration is making good progress, still hope to finish it by March. | 03:08 |
| Sundar | Cyborg client: OpenStackSDK part is done. Hereafter it is incremental improvement, right? | 03:09 |
| Sundar | s_shogo, all: ^ | 03:09 |
| s_shogo | Yes, I'll implement filter function and so on >> client | 03:11 |
| Sundar | Would the client be mostly done by Ussuri? Or do you think it'll go to V release? | 03:12 |
| s_shogo | The SDK including cyborg part was released yesterday. Thus that could be released in U release,IMO. | 03:13 |
| s_shogo | ( SDK's release cycle is independent from OpenStack's one) | 03:13 |
| Sundar | The reason I am asking is, the Cyborg client has been released midcycle. If most functionality is in, we could consider an end-of-cycle release instead of waiting for V. | 03:13 |
| xinranwang | client is based on v2 API, I think we don't have v2 deployable API now, right ? | 03:15 |
| s_shogo | OK I got it. Release in U could be chosen | 03:15 |
| s_shogo | Yes >> xinranwang | 03:15 |
| brinzhang | The SDK cannot match Cyborg API now? | 03:16 |
| s_shogo | brinzhang No, the Latest SDK supports many Cyborg API, but that is not perfect like xinranwang mentioned. | 03:17 |
| brinzhang | I know the v2 microversion in progress, if so, the SDK/cyborgclient maybe cannot works good, rirht? | 03:17 |
| brinzhang | s_shogo: Ok, later, while I am free, I will test in my evn ^^ | 03:18 |
| xinranwang | s_shogo: got it. | 03:18 |
| s_shogo | brinzhang thanks:) | 03:18 |
| xinranwang | brinzhang: yes, we should support microversion in client side too | 03:18 |
| Sundar | brinzhang: The microversion support in the API server side means that the client must specific headers with the microversion number. That is probably the next big infrastructure in the client. Right, s_shogo? | 03:18 |
| brinzhang | xinranwang: necessary. in clinet we must support microversion to request the Cyborg api | 03:19 |
| Sundar | *must send | 03:19 |
| xinranwang | brinzhang: yes, exactly | 03:19 |
| brinzhang | s_shogo: I installed cyborgclient by "pip install python-cyborgclient", is that ok by this way? | 03:20 |
| s_shogo | Yes >>Sundar , xinranwang and me start to investigate a method of supporting them with sdk's functions. | 03:20 |
| Sundar | Great. Thanks, s_shogo | 03:20 |
| Sundar | xinranwang: Support for deployables in the client is an incremental feature. That brings me to the 2nd item above. /v2/devices API for inventory is done. But /v2/deployables for programming is open. | 03:21 |
| brinzhang | s_shogo: we should add --os-cyborg-api-version <cyborg-api-ver> in your client implemation | 03:21 |
| Sundar | So, our top 3 priorities for U are still the same as above: Nova integration | 03:21 |
| Sundar | The last one includes microversion support. | 03:22 |
| xinranwang | Sundar: yes, device API is done except upgrade firmware, and deployable v2 is missing | 03:22 |
| s_shogo | brinzhang Sorry , recently I don't have test the "pip install python-cyborg client" , so I'll test that. I've checked "pip install openstacksdk" includes Cyborg part , yesterday. | 03:23 |
| brinzhang | s_shogo: ok, hoping get your feedback | 03:24 |
| Yumeng | s_shogo: so , for now we should install from source code, right? | 03:25 |
| s_shogo | Yumeng That is certain way, now. | 03:26 |
| brinzhang | Yumeng: I think pip install is ok, but I think s_shogo should check whether it works fine for all cli | 03:26 |
| Yumeng | s_shogo: Alright. got it. | 03:27 |
| s_shogo | I'll check the latest pip status and report that. | 03:27 |
| Yumeng | great.! thanks | 03:28 |
| chenke | great] | 03:28 |
| Sundar | So, our top 3 priorities for U are still the same as above: Nova integration | 03:30 |
| Sundar | All agree? | 03:30 |
| brinzhang | how about default policy refresh? | 03:30 |
| xinranwang | what is the other 2? | 03:30 |
| brinzhang | nova-cyborg-interaction, v2 microversion, policy-default-refresh? | 03:31 |
| Sundar | brinzhang: Policy refresh is important. I suppose we should add that as 4th | 03:32 |
| xinranwang | I think completing v2 API is also important | 03:32 |
| brinzhang | I think anther is SDK/cyborgclient | 03:32 |
| Sundar | xinranwang: Other 2 are: Add /v2/devices API for inventory and /v2/deployables for programming | 03:32 |
| Sundar | As i said above, and also in Ussuri etherpad | 03:33 |
| Sundar | We should add policy refresh because that is important for security. | 03:33 |
| Sundar | Activities like improving UT are important too. However, when we publish our cycle highlights, it is usually features | 03:33 |
| xinranwang | what about microversion, the spec is already merged. | 03:34 |
| xinranwang | and it is listed in etherpad too. | 03:34 |
| xinranwang | I think s_shogo is working on other v2 APIs | 03:35 |
| Sundar | I effectively included that in the API enhancements above. Adding/changing APIs requires microversion support. | 03:35 |
| Sundar | May be I should make it explicit. | 03:35 |
| s_shogo | I'm working on "programming in deployable" and "enable/disable in device" | 03:36 |
| Yumeng | policy-default-refresh is based on v2 API. maybe not enough time for all APIs to be implemented in U. but we can implement some, I mean, at least one sample API policy check. | 03:36 |
| Yumeng | And its spec has been merged. | 03:36 |
| xinranwang | s_shogo: Oh, do you want me to help on list/show APIs, or you will do it together? I think we can just implement them based on v1 code | 03:37 |
| Sundar | Let me restate the priorities like this: Nova integration | 03:38 |
| s_shogo | xinranwang oh,Thanks! could you implement them is OK? > list/show APIs. Now, I'm simply copy them from v1 code in my env. | 03:38 |
| xinranwang | s_shogo: I think we just need adapt v1 code to v2 model ;) | 03:39 |
| xinranwang | I can help on that if needed | 03:40 |
| *** tetsuro has quit IRC | 03:41 | |
| s_shogo | xinranwang: May I ask your help that? It may be simple. | 03:41 |
| xinranwang | s_shogo: sure, I am willing to help. | 03:42 |
| s_shogo | Thanks , xinranwang | 03:44 |
| Sundar | Great. Anything else, folks? | 03:44 |
| xinranwang | yes, please review microversion patch if you got time | 03:45 |
| xinranwang | Thanks all | 03:45 |
| Sundar | #topic AoB | 03:45 |
| *** openstack changes topic to "AoB (Meeting topic: openstack-cyborg)" | 03:45 | |
| *** tetsuro has joined #openstack-cyborg | 03:45 | |
| Sundar | xinrnwang: Do you need review from outside, like from cdent or Nova folks? | 03:45 |
| xinranwang | I think we can start review internally | 03:46 |
| xinranwang | because cyborg use pecan which is different from nova | 03:46 |
| Sundar | ok | 03:47 |
| Sundar | Anything else? | 03:47 |
| brinzhang | me | 03:47 |
| Sundar | Yes, brinzhang? | 03:48 |
| brinzhang | I would like to improve the cyborg docs, such as the https://docs.openstack.org/cyborg/latest/ | 03:48 |
| brinzhang | Many mainly links mostly hard to found, such as the api reference | 03:49 |
| brinzhang | I think many people cannot remember the all links, so I would like to make it perfect | 03:49 |
| brinzhang | https://review.opendev.org/#/c/711869/ | 03:49 |
| brinzhang | I mainly add the api reference here, if there are anything else that I missing, please point and leave comments | 03:50 |
| Sundar | Maybe we should have some blueprint, story or doc that describes all the doc changes needed? if we do it one at a time, it may get confusing or we may miss something. | 03:51 |
| brinzhang | blueprint, story links maybe should in README | 03:52 |
| brinzhang | of course, maybe we should add a document about how to contribute to Cyborg such as nova | 03:52 |
| brinzhang | like this one: https://review.opendev.org/#/c/712420/1/doc/source/contributor/contributing.rst | 03:53 |
| brinzhang | but looks like Cyborg doesnot have these thing need to be record | 03:54 |
| brinzhang | suggestion step by step to do, after all, we lack so many docs | 03:54 |
| Sundar | Could we resume this discussion next week? I need to leave a bit early today. | 03:55 |
| brinzhang | yeah, nothing else ^^ | 03:55 |
| Sundar | Added this to next week's agenda: https://wiki.openstack.org/wiki/Meetings/CyborgTeamMeeting#Agenda | 03:56 |
| Sundar | Thanks | 03:56 |
| Sundar | Thanks a lot, everybody. Good progress so far in U. Let's keep it up! Have a good day. | 03:56 |
| Sundar | #endmeeting | 03:56 |
| *** openstack changes topic to "Pending patches (Meeting topic: openstack-cyborg)" | 03:56 | |
| openstack | Meeting ended Thu Mar 12 03:56:57 2020 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 03:56 |
| chenke | bye | 03:56 |
| openstack | Minutes: http://eavesdrop.openstack.org/meetings/openstack_cyborg/2020/openstack_cyborg.2020-03-12-03.00.html | 03:57 |
| openstack | Minutes (text): http://eavesdrop.openstack.org/meetings/openstack_cyborg/2020/openstack_cyborg.2020-03-12-03.00.txt | 03:57 |
| openstack | Log: http://eavesdrop.openstack.org/meetings/openstack_cyborg/2020/openstack_cyborg.2020-03-12-03.00.log.html | 03:57 |
| Yumeng | bye | 03:57 |
| brinzhang | bye | 03:57 |
| s_shogo | bye | 03:57 |
| *** Sundar has quit IRC | 03:57 | |
| *** s_shogo has quit IRC | 04:07 | |
| *** tetsuro_ has joined #openstack-cyborg | 05:23 | |
| *** tetsuro has quit IRC | 05:27 | |
| *** links has joined #openstack-cyborg | 05:58 | |
| *** xinranwang has quit IRC | 06:04 | |
| *** Yumeng has quit IRC | 06:12 | |
| *** chenke has quit IRC | 07:22 | |
| *** tetsuro has joined #openstack-cyborg | 07:25 | |
| *** tetsuro_ has quit IRC | 07:29 | |
| *** tetsuro_ has joined #openstack-cyborg | 07:35 | |
| *** tetsuro has quit IRC | 07:37 | |
| *** tetsuro_ has quit IRC | 08:44 | |
| *** tetsuro has joined #openstack-cyborg | 08:47 | |
| *** tetsuro_ has joined #openstack-cyborg | 08:53 | |
| *** tetsuro has quit IRC | 08:57 | |
| *** tetsuro has joined #openstack-cyborg | 09:14 | |
| *** tetsuro_ has quit IRC | 09:17 | |
| *** tetsuro_ has joined #openstack-cyborg | 09:20 | |
| *** tetsuro has quit IRC | 09:23 | |
| *** tetsuro_ has quit IRC | 10:39 | |
| *** tetsuro has joined #openstack-cyborg | 11:28 | |
| *** tetsuro has quit IRC | 12:22 | |
| *** shaohe_feng has quit IRC | 13:01 | |
| *** jraju__ has joined #openstack-cyborg | 14:01 | |
| *** links has quit IRC | 14:02 | |
| *** TxGirlGeek has joined #openstack-cyborg | 14:48 | |
| *** TxGirlGeek has quit IRC | 15:45 | |
| *** TxGirlGeek has joined #openstack-cyborg | 15:47 | |
| *** jraju__ has quit IRC | 17:16 | |
| *** gmann is now known as gmann_lunch | 19:00 | |
| *** gmann_lunch is now known as gmann | 20:01 | |
| *** TxGirlGeek has quit IRC | 20:32 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!