| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-cyborg - 2019-05-29 | |||
| 03:35:17 | Coco_gao | What do you suggest? | |
| 03:35:34 | Li_Liu | lol | |
| 03:35:37 | ikuo_ | Yes, my colleague cannot use WeChat. Thank you for proposing the topic. | |
| 03:35:41 | Sundar__ | It would be good to use IRC and ML by default. For IRC, does it make sense to have a window of time where we all login? For example, it could be morning 9-12 am by CHina time? | |
| 03:35:46 | Li_Liu | I think irc is our only option now | |
| 03:36:07 | Sundar__ | Li_Liu and all: yes | |
| 03:36:19 | Li_Liu | can we still talk in zoom meetings? | |
| 03:36:41 | xinranwang | Sundar__: Hi Sundar, I have pull your pilot branch in local, and find some bugs, for example, we cannot delete extended_arq when deleting vm, and some other. I have done some bug-fix. Do you want me to leave comments where bug occurs , or I can co-author with you if you want. Which way you prefer? | |
| 03:36:59 | Sundar__ | Li_Liu: My guidelines are that public meetings are ok. I have put the Zoom URL in wiki along with the times. SO it is open to all | |
| 03:37:14 | ikuo_ | That's sounds good staying 9-12 am. | |
| 03:37:44 | Sundar__ | ikuo_: Thanks. Others ok with that time? | |
| 03:38:09 | Li_Liu | yup, I will be on irc all the time now | |
| 03:38:20 | Sundar__ | Cool. | |
| 03:39:46 | Sundar__ | xinranwang_: The Cyborg code to delete ARQs exists. Bu the Nova code to call the DELETE for ARQs is not done yet. If you want to take that up, NP. But please be aware that this will be on the critical path. | |
| 03:40:27 | Sundar__ | If you want to leave comments, that is also fine | |
| 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. | |