| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-cyborg - 2019-05-29 | |||
| 03:40:53 | Sundar__ | Or you cold make it a patch on top of this patch series, kinda like what Li_Liu plans to do | |
| 03:41:23 | Sundar__ | xinranwang: ^ | |
| 03:42:05 | xinranwang | Ok, it's fine for me to submit patch on your patch series. | |
| 03:42:16 | Sundar__ | Thanks, xinranwang. | |
| 03:42:23 | xinranwang | NP :) | |
| 03:42:49 | Coco_gao | 9-12am every day? | |
| 03:43:23 | Sundar__ | Coco_gao: :) As much as we can. Or perhaps you post on IRC and ping the other person on WeChat | |
| 03:43:45 | ikuo_ | I think it is not mandatory. Sundar's plan +1 | |
| 03:43:55 | Sundar__ | This is just to facilitate communication -- not meant to be a binding rule | |
| 03:44:05 | Coco_gao | got that | |
| 03:44:27 | xinranwang | 9-12am sounds good | |
| 03:44:32 | Sundar__ | Li_Liu: For developing patches in a series, please look at https://docs.openstack.org/contributors/code-and-documentation/patch-series-tutorial.html | |
| 03:44:46 | Li_Liu | Thanks a lit | |
| 03:44:46 | xinranwang | Sundar__: got it, thanks | |
| 03:44:49 | Li_Liu | lot* | |
| 03:45:54 | Sundar__ | Sorry that the pilot code is still not merged. I feel like I should write more UT and implement all comments. That has been a stretch. But I'll make it a priority | |
| 03:46:04 | xinranwang | I remember there is a tools that can run irc background all the time, which nova guys usually use. I will check it | |
| 03:46:35 | Sundar__ | irccloud can notify when offline, but only in paid version, right? | |
| 03:47:09 | Coco_gao | I see, 5$ for permanet use. | |
| 03:47:13 | xinranwang | I am not sure, cause i didn't pay for it. lol | |
| 03:47:16 | ikuo_ | I want to know such a tool, if possible, for free :) | |
| 03:47:47 | xinranwang | I think the tools i mentioned before is a kind of extension | |
| 03:48:32 | xinranwang | ikuo_: I will let you know if I find it :) | |
| 03:48:44 | ikuo_ | Thanks, xinranwang! | |
| 03:48:54 | Sundar__ | xinranwang: Please share after you find out about this. Thanks! | |
| 03:49:02 | Li_Liu | I paid for Irccloud | |
| 03:49:02 | xinranwang | Sure | |
| 03:49:07 | Sundar__ | #topic Cyborg patches | |
| 03:49:09 | Li_Liu | it works great so far | |
| 03:49:45 | Sundar__ | https://review.opendev.org/659233 -- xinranwang, can you rewrite this to use our own placement calling code? | |
| 03:50:07 | ikuo_ | I see. thanks Li_Liu! | |
| 03:51:52 | xinranwang | Sundar__: Yes, will do it. I am working on a cyborg DEMO which required by customers recently, and do not have much time on this. I will update it ASAP. | |
| 03:54:28 | Sundar__ | Thanks | |
| 03:54:46 | Sundar__ | I have provided some comments on other specs and patches | |
| 03:55:09 | Sundar__ | Any questions or comments on specs or patches? | |
| 03:55:17 | ikuo_ | I post some comments and questions to the spec yesterday, pls see them. | |
| 03:55:27 | ikuo_ | in here: https://review.opendev.org/#/c/659233/ | |
| 03:55:58 | ikuo_ | soryy | |
| 03:56:00 | ikuo_ | https://review.opendev.org/#/c/626057/ | |
| 03:57:09 | Sundar__ | Thanks, ikuo_ and all. Yes, I'll respond and fix them as much as practical. | |
| 03:57:37 | Sundar__ | #topic AoB | |
| 03:57:46 | Sundar__ | Anything else for today? | |
| 03:58:02 | ikuo_ | fine. thanks. | |
| 03:58:26 | Sundar__ | Have a good day (or night), everybody! | |
| 03:58:31 | Sundar__ | #endmeeting | |
| 03:58:33 | openstack | Meeting ended Wed May 29 03:58:31 2019 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | |
| 03:58:34 | openstack | Minutes: http://eavesdrop.openstack.org/meetings/openstack_cyborg/2019/openstack_cyborg.2019-05-29-03.03.html | |
| 03:58:35 | openstack | Minutes (text): http://eavesdrop.openstack.org/meetings/openstack_cyborg/2019/openstack_cyborg.2019-05-29-03.03.txt | |
| 03:58:36 | openstack | Log: http://eavesdrop.openstack.org/meetings/openstack_cyborg/2019/openstack_cyborg.2019-05-29-03.03.log.html | |
| 03:59:14 | Li_Liu | Have a good night Sundar | |
| 03:59:24 | ikuo_ | Bye, Sundar and all | |
| 04:58:59 | xinranwang | Hi guys, I have asked about the IRC tools. It's called ZNC. Here is the tutorial: https://fedoramagazine.org/never-leave-irc-znc/ | |
| 06:46:08 | Li_Liu | Wanna ask a CI/CD related question, do we have plan to have dummy device drivers for CI/CD? | |
| 12:22:17 | edleafe | I was reading the minutes from the meeting last night (well, last night for me), and xinranwang asked about a tool for continuous IRC. These are called "bouncers", and There is a free tool called 'znc' that works great. You set it up on a machine that's always on. It connects to IRC, and your IRC client connects to the znc instance. It will play back any messages you missed while disconnected. | |
| #openstack-cyborg - 2019-05-30 | |||
| 03:10:10 | Sundar__ | @Li_Liu: Re. fake drivers aka dummy devices, I have had different feedback. In the Nova spec, I put in both. However, we haven't got an owner yet. So, I am thinking of focusing on real hardware | |
| 03:11:37 | Sundar__ | #link https://etherpad.openstack.org/p/cyborg-ptg-train Liness 157-160 | |
| 03:12:43 | Sundar__ | @edleafe: Thanks. Do you know of any good free IRC bouncers for Android? | |
| 12:36:12 | edleafe | Sundar__: you don't run the bouncer on your client; you run it on a server. You would set your Android client to connect to the server running the bouncer instead of connecting to Freenode. | |
| #openstack-cyborg - 2019-05-31 | |||
| 00:44:35 | Sundar | @all, There is a discussion about whether we should use asserts in https://review.opendev.org/#/c/626057/10/cyborg/accelerator/drivers/fpga/intel/driver.py | |
| 00:45:51 | Sundar | First, raising an exception is appropriate f something is wrong with the user input or other subsystem's behavior, but asserts are good to catch internal program logic, invariants that should always ok, etc. Using asserts for these will prevent proliferation of exceptions. | |
| 00:46:08 | Sundar | Second, there are many precedents for using exceptions. E.g. | |
| 00:46:19 | Sundar | [1] https://github.com/openstack/nova/blob/master/nova/rpc.py#L97 | |
| 00:46:28 | Sundar | [2] https://github.com/openstack/nova/blob/master/nova/virt/disk/mount/api.py#L187 | |
| 00:46:37 | Sundar | [3] https://github.com/openstack/nova/blob/master/nova/objects/aggregate.py#L441 | |
| 01:09:46 | Sundar | @wangzhh I don't understand your response in https://review.opendev.org/#/c/659014/1/cyborg/tests/unit/db/utils.py Line 106 | |
| 01:10:08 | Sundar | The other comment referred to using the lspci command. | |
| 01:15:06 | wangzhh | For we use string as cpid_info, if we split it an convert to json, {"device": "2", "bus": "00", "function": "01"} is not enough. | |
| 01:15:59 | wangzhh | It should looks like {"device": "2", "bus": "00", "function": "01", "domain": "0000"} | |
| 01:17:28 | wangzhh | Am I clear? | |
| 01:19:33 | wangzhh | The cpid_info looks like 0000:00:02.0. The json_str in this patch lost domain. | |
| 01:20:14 | Coco_gao | yes, if cpid_info is a json string, what about the attach_info? | |
| 01:22:10 | wangzhh | I think it is the same. | |
| 01:24:05 | Coco_gao | And cpid_info is used to identify some of the devices, if its a json_str, then it will be hard to identify, because it' s hard to compare json_str. | |
| 01:25:26 | Coco_gao | what do you think? | |
| 01:25:39 | Coco_gao | @Sundar | |
| 01:27:23 | wangzhh | Sundar said he'll BRB in 30 min. | |
| 01:31:43 | Coco_gao | so will I | |
| 02:08:13 | Sundar | @Coco_gao @wangzhh Yes, domain should be included. attach_id also should be fixed in the same way: I'll do that. All good? | |
| 02:12:37 | Sundar | @Coco_gao: "if its a json_str, then it will be hard to identify, because it' s hard to compare json_str" Are you saying that we can store it as individual PCI BDF fields (domain, bus, etc.) in the db? That would not work for PowerVM, which represents PCI devices differently. | |
| 02:14:43 | wangzhh | Fine. Sundar. | |
| 02:16:38 | wangzhh | And do u install znc which xinran mentioned? I don't find windows version. | |
| 02:17:18 | Sundar | wangzhh: znc is linux only | |
| 02:18:07 | wangzhh | OK. Thx. | |
| 02:43:18 | Sundar | @yikun, @wangzhh, @Coco_gao and all, please respond to http://eavesdrop.openstack.org/irclogs/%23openstack-cyborg/%23openstack-cyborg.2019-05-31.log.html#t2019-05-31T00:45:51 | |
| 07:29:24 | wangzhh | @yikun Is generic driver ready? New GPU driver depends on it. Or u just responsible for spec, I'm not sure. | |
| #openstack-cyborg - 2019-06-02 | |||
| 06:46:45 | openstackgerrit | Brin Zhang proposed openstack/cyborg-specs master: Change openstack-dev to openstack-discuss https://review.opendev.org/622031 | |
| 19:53:10 | Sundar | FYI: https://review.opendev.org/#/c/662643/ "Categorize Cyborg under Accelerators and update description" | |
| #openstack-cyborg - 2019-06-04 | |||
| 05:12:52 | Sundar | @all, Shall we have a Zoom call this Thursday China time (Wed in Americas)? Agenda: Review the pending specs and patches. | |
| 05:17:21 | Sundar | Do we need the spec https://review.opendev.org/#/c/593726? We have already implemented discovery, right? | |
| 05:27:47 | ikuo_o | I can attend Thursday zoom meeting. | |
| 05:33:00 | Sundar | Thanks, ikuo_o | |
| 13:41:39 | Li_Liu | should be fine for me to attend the zoom meeting | |
| 15:13:06 | Sundar | Thanks, Li_Liu | |
| 15:30:38 | Li_Liu | Sundar, do we still have the irc meeting tonight then? | |
| 15:56:06 | Sundar | Li_Liu: Yes | |
| 18:50:27 | openstackgerrit | Donny Davis proposed openstack/cyborg master: Updating Index to reflect dependent projects and provide more information on Cyborg https://review.opendev.org/663121 | |
| #openstack-cyborg - 2019-06-05 | |||
| 03:00:48 | Sundar | Hi Biwei | |
| 03:00:55 | Biwei | Hi Sundar | |
| 03:01:09 | Sundar | Hi wangzhh | |
| 03:01:16 | wangzhh | Hi Sundar. | |