| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-cyborg - 2020-07-09 | |||
| 03:19:23 | ollivier | Does it depend on specific service Ussuri versions (e.g. nova)? | |
| 03:19:41 | brinzhang_ | ildikov: I think you can backport the v2 api to Train, but there are many patches you should backport to Train | |
| 03:20:24 | ildikov | brinzhang_: do you mean backporting downstream? | |
| 03:21:50 | brinzhang_ | I can be sure that the API for backport v2 downstream should work, but the function of interacting with nova may not be backport, it depends on nova version and other related packages | |
| 03:22:47 | brinzhang_ | If you want to use nova-cyborg-interaction feature, I suggest you upgrade your platform | |
| 03:22:48 | Yumeng | from cyborg's perspective, just backport V2 API to Train is theorically possible, but the nova-cyborg integration is not that easy if backport to train(which needs nova's help and may affect much). | |
| 03:22:58 | ollivier | Here I would consider first the backport in Cyborg stable/ussuri | |
| 03:23:32 | ollivier | Would it also ask for backports in Nova as well? | |
| 03:24:00 | ildikov | as CNTT is consuming upstream code but not maintaining patches locally at this point I'm not sure downstream backport is an option | |
| 03:24:00 | shaohe_feng | hi all | |
| 03:24:02 | shaohe_feng | sorry | |
| 03:24:18 | brinzhang_ | Unless you don’t want to use the nova-cybrog-interaction feature | |
| 03:24:25 | shaohe_feng | conflict with other meeting. | |
| 03:24:40 | Yumeng | brinzhang_:yes, +1 | |
| 03:25:09 | Yumeng | hi shaohe_feng: no pb. | |
| 03:25:33 | ildikov | brinzhang_: what's included in the nova-cybrog-interaction feature? | |
| 03:25:45 | ildikov | brinzhang_: I assume some API endpoints depend on it? | |
| 03:26:23 | Yumeng | ildikov: nova operations like creating a VM with accelerator(provisioned by cyborg) , this kind of function depends on nova/ussuri version | |
| 03:27:19 | brinzhang_ | ollivier: Whether it is nova or cyborg, I think it can be used from Ussuri backport to stable/Train, but it will definitely encounter problems, I think it needs to be seriously considered. | |
| 03:28:31 | ollivier | brinzhang_: It's just checking what's doable. It would ask for backports in both Cyborg and Nova which seem quite serious. | |
| 03:28:33 | brinzhang_ | ildikov: agree with Yumeng, if you dont want to use nova-cyborg-interaction, I think you dont worry about to backport the cyborg V2 api to stable/train | |
| 03:28:43 | Yumeng | nova-cybrog-interaction feature will affect creation, deletion, soft reboot, hard reboot, VM wih accelerators this kind of operations https://docs.openstack.org/api-guide/compute/accelerator-support.html | |
| 03:29:12 | ildikov | Yumeng: thanks for the clarification, I believe that feature would be very much in our interest | |
| 03:29:17 | Yumeng | pls check supported operations in the link above. | |
| 03:29:56 | ildikov | ollivier: I assume you mean backproting upstream, right? | |
| 03:30:20 | brinzhang_ | ollivier: yes, the nova-cyborg-interaction feature is big, you can see https://review.opendev.org/#/q/status:merged+project:openstack/nova+branch:master+topic:bp/nova-cyborg-interaction | |
| 03:30:26 | s_shogo | sorry for late | |
| 03:30:40 | shaohe_feng | ildikov what's your openstack version? | |
| 03:31:11 | ollivier | ildikov: yes as you mentioned, here it's only about the OpenStack projects. All operators are free to backport by themselves but it would be out of CNTT. | |
| 03:31:26 | Yumeng | ildikov: does CNTT wanna support full function now or like half a year later? | |
| 03:31:35 | ollivier | shaohe_feng: Train for our next release (September) | |
| 03:32:06 | Yumeng | ok. got it. | |
| 03:32:23 | ildikov | Yumeng: the aim is full support, but the question is if it is doable or not | |
| 03:32:25 | shaohe_feng | seems it is mor easy for Train | |
| 03:33:00 | ildikov | Yumeng: if upstream backport is not an option then our interest would be to fix up the documentation for Cyborg to be clear about what is available in Train | |
| 03:33:06 | shaohe_feng | seems it is easier for Train | |
| 03:33:33 | Yumeng | ildikov: yes, that would be easier as a start! I can help. | |
| 03:33:41 | xinranwang__ | Hi all. | |
| 03:33:42 | ollivier | Yumeng: yes we will keep rolling. As far as I understand, waiting CNTT to select U or newer seems the best option | |
| 03:34:13 | Yumeng | ollivier: agree! | |
| 03:34:27 | Yumeng | hi xinranwang__ | |
| 03:34:28 | xinranwang__ | Sorry for late. | |
| 03:34:50 | xinranwang__ | Just finish another meeting | |
| 03:35:39 | brinzhang_ | xinranwang_, shaohe_feng, s_shogo: Just right~ | |
| 03:35:55 | ollivier | 1 to fix https://releases.openstack.org/train/highlights.html#cyborg | |
| 03:36:31 | Yumeng | xinranwang__,s_shogo: no worries. | |
| 03:36:31 | ollivier | "Updated Cyborg APIs to version 2" should be removed as we just discussed, right? | |
| 03:37:25 | brinzhang_ | ollivier: yeah, thanks, you can update that, it's not very accurate | |
| 03:37:31 | ollivier | I would think that "https://docs.openstack.org/cyborg/train/api/api.html#v2-0" is possible wrong too. "see The Train release introduces version 2.0 APIs. Details of these APIs can be found here." | |
| 03:37:48 | Yumeng | we did update part of APIs to version2, do you wanna a detailed list saying what are removed to V2, what are not yet | |
| 03:37:50 | ildikov | ollivier: +1 on U or newer for CNTT | |
| 03:38:26 | ildikov | Yumeng: the Train docs refers to the spec I think when it comes to the API and it is misleading in terms of what is available in Train | |
| 03:38:36 | ollivier | Yumeng: yes please. I'm always lost when we say that an API version is divided into 2 releases | |
| 03:39:12 | brinzhang_ | ollivier: thanks point this. | |
| 03:39:21 | ildikov | Yumeng: it would be better I think to be explicit and have the proper API documentation for Train with the available functions | |
| 03:39:24 | brinzhang_ | Yumeng: We need to clarify which APIs in Train support V2 and which are completed in Ussuri. | |
| 03:39:25 | ildikov | ollivier: +1 | |
| 03:40:20 | ollivier | How can we somehow deprecate v1 if v2 is partial? | |
| 03:41:27 | brinzhang_ | As I know, deprecate the v1 API until we completed the v2 API in Ussuri | |
| 03:41:59 | brinzhang_ | In v1, some attribute cannot support to request | |
| 03:43:13 | xinranwang__ | v1 api does not match current data model. We have changed data model pretty much, that 's why we need a new set of api(v2) as well | |
| 03:43:14 | Yumeng | ildikov: here it is the list. Train supports limited V2 APIs : Device Profiles API and Accelerator Requests APIs. and Ussuri supports full V2 APIs: Device Profiles API and Accelerator Requests APIs and Devices APIs, Deployables APIs, | |
| 03:43:33 | xinranwang__ | I agree that we should clarify this in document | |
| 03:43:56 | Yumeng | I will update documentation today to clarify. | |
| 03:44:16 | ildikov | thanks Yumeng | |
| 03:44:21 | songwenping_ | ollivier you can find corresponding api in v2 for deprecate v1. | |
| 03:44:32 | brinzhang_ | Yumeng: thanks | |
| 03:44:37 | ollivier | ok v2 is well a superset of v1 (from operations point of view). | |
| 03:44:55 | ildikov | I think it will be a big improvement to have the documentation uodated | |
| 03:45:31 | ollivier | Just a last question: Device Profiles v2 API and Accelerator Requests v2 APIs were released in Train, right? | |
| 03:45:40 | brinzhang_ | ildikov: thanks, Improve the readability and accuracy of the document, this is what we have been doing. | |
| 03:45:58 | Yumeng | ollivier: yes. they are released in Train. | |
| 03:46:32 | ollivier | and the microversion support? | |
| 03:46:37 | ollivier | Train as well? | |
| 03:47:07 | ollivier | It should be | |
| 03:47:31 | Yumeng | ollivier: nope. microversion is released in Ussuri | |
| 03:47:41 | xinranwang__ | microversion is in U | |
| 03:49:55 | ollivier | ok it's very clear. Then your release notes are quite ok if we consider the time you wrote them (before ussuri). Mentioning the API capability (Device Profiles API and Accelerator Requests APIs) could enhance it a litlle bit. | |
| 03:51:25 | ollivier | About Functest, I could integrate your tempest plugin in Functest Kali (U-release) or Functest master (next Victoria) but I'm not clear how I can gate it (specific hardware?) | |
| 03:52:16 | ollivier | I will check the devstack configuration. It would allow integrating the testing part in advance until CNTT selects U or newer. | |
| 03:53:03 | Yumeng | ollivier: not necessary to use specific hardware, current tempest-plugin is using a fake driver(you can understand it as a fake device) to test the cyborg workflow | |
| 03:54:11 | ildikov | ollivier: I think testing on specific hardware would require third-party CIs, I don't think our infra has those type of resources available | |
| 03:54:35 | Yumeng | cyborg-tempest-plugin(U-release) should work fine. since it is complete to do test for the basic cyborg workflow and cyborg-nova integration workflow | |
| 03:54:45 | Yumeng | ildikov: yes, exactly | |
| 03:55:00 | ollivier | Yumeng: great. That shouldn't be a challenge in that case. I will have a look to your testing plugin before my holidays (in a couple of days) | |
| 03:55:38 | Yumeng | ok. cool | |
| 03:55:48 | ildikov | Looks like we have a plan :) | |
| 03:56:15 | ollivier | ildikov: at least we could in OPNFV (I think OpenStack as well). Here it's more about verifying the test integration in Functest SUT. If we can select fake drivers, it should work easily. | |
| 03:57:08 | ildikov | ollivier: yeah, it would be great to make it work in Functest so when CNTT gets to U and newer we already have that figured out | |
| 03:57:18 | ollivier | Yes very clear on my side. Thank you. I will give you my feedback about the testing part. | |
| 03:57:42 | Yumeng | great! ping me at this channel if you have more questions.^^ | |
| 03:57:51 | ollivier | Yumeng: sure | |
| 03:57:54 | ildikov | thanks Yumeng and team! | |
| 03:58:08 | ollivier | ildikov: yes that's the best model | |
| 03:58:32 | Yumeng | ildikov,ollivier: you are welcome :) | |
| 03:58:41 | ollivier | Thank you all | |
| 03:59:19 | ollivier | (Sorry for the second topic) | |
| 03:59:37 | Yumeng | haha. no pl. the rest topic from my side for today would be the review | |
| 03:59:55 | ildikov | ollivier: and thank you for getting up this early :) | |
| 04:00:55 | ollivier | ildikov: my pleasure. it's quite the same for you, right? | |