Earlier  
Later
New Search
Posted Nick Remark
#openstack-cyborg - 2020-04-23
03:45:21 xinranwang it is also depends on which version of OPAE we installed, right ?
03:45:23 Yumeng Sundar, brinzhang and all: agree with double check uuid to make sure it's a str, and I will double check the unittest
03:45:25 s_shogo I tried get OPAE version from "fpgainfo fme", but that format seems to have difference between some OPAE version
03:45:43 shaohe_feng we can push OPAE team to provide a unify API for program
03:45:50 brinzhang The depend-on, just only ensure the latest patch cannot merged earlier than the base patach, so it can add in the same project, in nova there is a common case
03:45:57 Sundar I had once thought of using the image suffix, whether it is ".gbs" or not, to indeitfy the command. However, AFAIK, the bitstreams that need fpgaconf or fpgasupdate may both have the same suffix
03:46:36 shaohe_feng Let's them distinguish to different bitstream format and call the right cmd
03:47:19 shaohe_feng In my former demo, I did not change any cyborg code.
03:47:20 Sundar xinranwang: Yes, it depends on OPAE version too. However, if we let the operator install OPAE libraries and we remove that OPAE version setting from devstack -- which would be a good thing -- we would not be able to use the version.
03:47:50 Sundar May be this can be a goal for Victoria release :)
03:48:01 shaohe_feng I just write a shell wrap to call fpgasupdate or fpgaconf.
03:48:38 shaohe_feng I can do, OPAE team can also distinguish them=D
03:48:38 Sundar shaohe_feng: In a production env, with different types of FPGAs from different vendors, we would need a better mechanism.
03:48:59 Sundar It is not just OPAE -- but hopefully other vendors too
03:50:36 shaohe_feng Yes, but that's hard. No now know all vendors product well. So different vendors maintainer their special code :')
03:51:52 Sundar The special code must be inside their resp. FPGA drivers. The image file need only have metadata that indicates the image type, maybe 'accel:bitstream_type=gbs' in Glance metadata.
03:52:46 shaohe_feng For image upload, it is necessary to call the vendors special tools to parser the metadata of their image
03:53:12 shaohe_feng and format glance metadata
03:53:46 shaohe_feng we are tying to do it. This can make user more easy to user cyborg
03:53:56 shaohe_feng and avoid error prone
03:54:40 Sundar shaohe_feng: That would be useful. Nova/Glance developers told me that it is better if Cyborg does not call Glance API for that. Whoever does the image upload should just add the metadata, and Cyborg can check whether it is present.
03:55:15 xinranwang It is better to let operator choose the right version, and configure it in conf file, cyborg just need to read this file and choose the right commands.
03:55:56 shaohe_feng Sundar, agree. I'm make the planing to improve it.
03:56:14 Sundar xinranwang: The conf file approach is fine too, as long as there are not too many image types. Right now, there is only 1, so no problem :)
03:57:49 s_shogo (IMO, multiple OPAE in single env is little difficult to operate.)
03:58:58 Sundar s_shogo: Agreed. However, unfortunately, different FPGA cards require different versions of OPAE today. If you need to use more than one in the same cluster, maybe you can put them in different compute nodes, so that each node has only 1 OPAE version.
04:00:11 xinranwang shaohe_feng Sundar does OPAE team plan to unify a set of API for different cards?
04:00:21 s_shogo Sundar, Thanks, I missed that possibility.that's right.
04:00:50 shaohe_feng xinranwang let us feedback to them and push them -_-
04:02:03 shaohe_feng I guess no. we can user, but they are not.
04:02:07 Sundar xinranwang: I can explain that offline, because we are hitting the top of the hour.
04:02:32 shaohe_feng so they do not know the pain-point
04:03:04 Sundar Any other topic for today?
04:04:31 Sundar Tomorrow is the last day for RC release. We may not be able to merge any patches for Ussuri after that.
04:04:56 Sundar Thanks a lot to all of you. Have a good day. Bye.
04:05:01 Sundar #endmeeting
04:05:02 xinranwang Sundar: ok
04:05:03 openstack Meeting ended Thu Apr 23 04:05:00 2020 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)
04:05:04 xinranwang byebye
04:05:05 openstack Minutes: http://eavesdrop.openstack.org/meetings/openstack_cyborg/2020/openstack_cyborg.2020-04-23-03.01.html
04:05:06 openstack Minutes (text): http://eavesdrop.openstack.org/meetings/openstack_cyborg/2020/openstack_cyborg.2020-04-23-03.01.txt
04:05:07 openstack Log: http://eavesdrop.openstack.org/meetings/openstack_cyborg/2020/openstack_cyborg.2020-04-23-03.01.log.html
04:05:15 s_shogo bye
04:05:33 shaohe_feng do we still need to refactor the accelerator driver
08:20:21 openstackgerrit YumengBao proposed openstack/cyborg master: Fix bandit error: [B108:hardcoded_tmp_directory] https://review.opendev.org/720143
08:20:43 openstackgerrit YumengBao proposed openstack/cyborg master: Fix bandit error: [B108:hardcoded_tmp_directory] https://review.opendev.org/720143
08:47:07 openstackgerrit YumengBao proposed openstack/cyborg master: Fix bandit error: [B108:hardcoded_tmp_directory] https://review.opendev.org/720143
08:50:22 openstackgerrit YumengBao proposed openstack/cyborg master: Fix bandit error: [B108:hardcoded_tmp_directory] https://review.opendev.org/720143
10:19:14 openstackgerrit Merged openstack/cyborg master: Fix bandit error: [B108:hardcoded_tmp_directory] https://review.opendev.org/720143
11:34:01 openstackgerrit zhurong proposed openstack/cyborg master: Fix bandit error: Ascend driver:[B602:subprocess_popen_with_shell_equals_true] https://review.opendev.org/720456
11:36:26 openstackgerrit zhurong proposed openstack/cyborg master: Fix bandit error: [B104:hardcoded_bind_all_interfaces] https://review.opendev.org/720149
11:39:36 openstackgerrit YumengBao proposed openstack/cyborg master: Fix bandit error: Ascend driver:[B602:subprocess_popen_with_shell_equals_true] https://review.opendev.org/720456
12:55:10 openstackgerrit Merged openstack/cyborg master: Fix bandit error: [B104:hardcoded_bind_all_interfaces] https://review.opendev.org/720149
#openstack-cyborg - 2020-04-24
02:02:41 openstackgerrit Merged openstack/cyborg master: Fix bandit error: SPDK driver:[B602:subprocess_popen_with_shell_equals_true] https://review.opendev.org/720475
02:35:08 openstackgerrit Merged openstack/cyborg master: Fix bandit error: Ascend driver:[B602:subprocess_popen_with_shell_equals_true] https://review.opendev.org/720456
02:35:09 openstackgerrit Merged openstack/cyborg master: Change bandit job from non-voting to voting https://review.opendev.org/720479
08:47:34 openstackgerrit OpenStack Release Bot proposed openstack/cyborg master: Update master for stable/ussuri https://review.opendev.org/722636
08:47:36 openstackgerrit OpenStack Release Bot proposed openstack/cyborg master: Add Python3 victoria unit tests https://review.opendev.org/722637
#openstack-cyborg - 2020-04-26
01:26:21 openstackgerrit Merged openstack/cyborg-tempest-plugin master: Fix a typo https://review.opendev.org/696252
01:34:05 openstackgerrit Merged openstack/cyborg-specs master: Replacing the HTTP protocal with HTTPS in cyborg-driver-proposal.rst. https://review.opendev.org/617781
08:08:20 openstackgerrit Brin Zhang proposed openstack/python-cyborgclient master: Remove future imports https://review.opendev.org/723167
08:09:58 openstackgerrit Brin Zhang proposed openstack/python-cyborgclient master: Use unittest.mock instead of third party mock https://review.opendev.org/720981
08:17:39 openstackgerrit Brin Zhang proposed openstack/cyborg master: Use unittest.mock instead of third party mock https://review.opendev.org/716916
08:30:17 openstackgerrit Brin Zhang proposed openstack/python-cyborgclient master: Remove future imports https://review.opendev.org/723167
08:30:17 openstackgerrit Brin Zhang proposed openstack/python-cyborgclient master: Use unittest.mock instead of third party mock https://review.opendev.org/720981
09:12:52 openstackgerrit Brin Zhang proposed openstack/cyborg master: hacking: force explicit import of python's mock https://review.opendev.org/716920
09:14:14 openstackgerrit Brin Zhang proposed openstack/cyborg master: hacking: force explicit import of python's mock https://review.opendev.org/716920
09:18:48 openstackgerrit Brin Zhang proposed openstack/cyborg master: hacking: force explicit import of python's mock https://review.opendev.org/716920
10:11:13 openstackgerrit Brin Zhang proposed openstack/cyborg master: Add py38 package metadata https://review.opendev.org/723201
10:15:17 openstackgerrit Brin Zhang proposed openstack/python-cyborgclient master: Add py38 package metadata https://review.opendev.org/723203
10:20:38 openstackgerrit Brin Zhang proposed openstack/cyborg-tempest-plugin master: Add py38 package metadata https://review.opendev.org/723204
10:41:07 openstackgerrit Brin Zhang proposed openstack/python-cyborgclient master: Add py38 package metadata https://review.opendev.org/723203
11:25:37 openstackgerrit Brin Zhang proposed openstack/python-cyborgclient master: Update docs building https://review.opendev.org/720804
11:29:59 openstackgerrit Brin Zhang proposed openstack/python-cyborgclient master: Remove translation sections from setup.cfg https://review.opendev.org/723207
11:41:02 openstackgerrit Brin Zhang proposed openstack/cyborg master: Update docs building https://review.opendev.org/723209
11:45:33 openstackgerrit Brin Zhang proposed openstack/cyborg master: Remove translation sections from setup.cfg https://review.opendev.org/723211
#openstack-cyborg - 2020-04-27
00:17:50 openstackgerrit Brin Zhang proposed openstack/python-cyborgclient master: Add py38 package metadata https://review.opendev.org/723203
01:08:34 openstackgerrit Brin Zhang proposed openstack/python-cyborgclient master: Update master for stable/stein https://review.opendev.org/723287
01:11:56 openstackgerrit Brin Zhang proposed openstack/python-cyborgclient master: Update master for stable/train https://review.opendev.org/723288
01:16:59 openstackgerrit Brin Zhang proposed openstack/python-cyborgclient master: Update master for stable/stein https://review.opendev.org/723287
01:22:14 openstackgerrit Brin Zhang proposed openstack/python-cyborgclient master: Update master for stable/stein https://review.opendev.org/723287
01:22:35 openstackgerrit Brin Zhang proposed openstack/python-cyborgclient master: Update master for stable/train https://review.opendev.org/723288
01:30:18 openstackgerrit Brin Zhang proposed openstack/python-cyborgclient master: Add py38 package metadata https://review.opendev.org/723203
01:49:27 openstackgerrit Brin Zhang proposed openstack/python-cyborgclient master: Update master for stable/train https://review.opendev.org/723288
02:01:11 openstackgerrit Merged openstack/python-cyborgclient master: Update docs building https://review.opendev.org/720804
02:44:38 openstackgerrit Merged openstack/cyborg master: Update master for stable/ussuri https://review.opendev.org/722636
03:09:49 openstackgerrit Merged openstack/cyborg master: Add Python3 victoria unit tests https://review.opendev.org/722637
03:10:29 openstackgerrit Merged openstack/cyborg master: Update docs building https://review.opendev.org/723209
03:21:47 openstackgerrit Merged openstack/python-cyborgclient master: Update master for stable/ussuri https://review.opendev.org/719283
03:22:26 openstackgerrit Merged openstack/python-cyborgclient master: Add Python3 victoria unit tests https://review.opendev.org/719286
05:06:41 openstackgerrit Arthur Dayne proposed openstack/cyborg master: Add NVMe SSD driver https://review.opendev.org/723310
05:35:07 openstackgerrit Arthur Dayne proposed openstack/cyborg master: Add NVMe SSD driver https://review.opendev.org/723310
05:40:26 openstackgerrit Brin Zhang proposed openstack/python-cyborgclient master: Update master for stable/stein https://review.opendev.org/723287
05:42:59 openstackgerrit Brin Zhang proposed openstack/python-cyborgclient master: Update master for stable/train https://review.opendev.org/723288
06:00:40 openstackgerrit Arthur Dayne proposed openstack/cyborg master: WIP: Add NVMe SSD driver https://review.opendev.org/723310
06:09:57 openstackgerrit Arthur Dayne proposed openstack/cyborg master: WIP: Add NVMe SSD driver https://review.opendev.org/723310
06:15:27 openstackgerrit Merged openstack/python-cyborgclient master: Update master for stable/train https://review.opendev.org/723288
06:34:57 openstackgerrit Brin Zhang proposed openstack/python-cyborgclient master: Update master for stable/stein https://review.opendev.org/723287
07:18:22 openstackgerrit Qiu Fossen proposed openstack/python-cyborgclient master: Bump default tox env from py37 to py38 https://review.opendev.org/723329
07:53:52 openstackgerrit Brin Zhang proposed openstack/python-cyborgclient master: Revert "Update master for stable/train" https://review.opendev.org/723334

Earlier   Later