| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-cyborg - 2020-03-19 | |||
| 03:12:24 | Sundar | NP, chenke. Sure, let's get started | |
| 03:12:25 | Yumeng | #info Yumeng | |
| 03:12:29 | chenke | ok | |
| 03:12:34 | Yumeng | Hi all, sorry for late. | |
| 03:12:55 | brinzhang_ | https://review.opendev.org/#/c/711869/3/doc/source/index.rst | |
| 03:13:18 | brinzhang_ | Hi all, let talking about this docs change, any suggestions? | |
| 03:14:05 | brinzhang_ | chenke, I donot know another perfact tittle to change "Writing to the API" | |
| 03:14:47 | brinzhang_ | if someone can give a good idea, I will be appreviate | |
| 03:15:14 | chenke | How about Docs related API | |
| 03:15:16 | Sundar | How about: "Using the API"? | |
| 03:15:19 | brinzhang_ | And is there others link should I add in the latest page? | |
| 03:15:32 | chenke | Using the API +1 | |
| 03:16:17 | brinzhang_ | If I add the api_microversion_history to the index page, it sound not good yet | |
| 03:16:25 | Sundar | Line 89,90: Who knows what genondex and modindex do? | |
| 03:16:43 | brinzhang_ | but Using the API is ok for me | |
| 03:17:37 | brinzhang_ | Sundar, I dont know what is and I dont why we need Line 89,90 | |
| 03:17:45 | chenke | I think we can remove | |
| 03:18:40 | chenke | LIne 89,90 just a link to the useless link.:https://2346739f9f4d66197c40-6f04b689d71be05d52bf14ef78b20a44.ssl.cf2.rackcdn.com/711869/2/check/openstack-tox-docs/ed5106b/docs/ | |
| 03:18:45 | Sundar | agreed | |
| 03:19:36 | brinzhang_ | agree | |
| 03:20:38 | Sundar | Anything else on this subject? | |
| 03:20:54 | Yumeng | agree both. using the API is good for understanding. remove useless doc also makes it cleaer. | |
| 03:21:08 | brinzhang_ | If anything need to add this, I think it's ok | |
| 03:21:52 | Sundar | OK Sounds good. | |
| 03:21:53 | s_shogo | Using the API seems to be natural,agree. | |
| 03:22:44 | brinzhang_ | Ok | |
| 03:22:58 | Sundar | On Nova status, I have been off for a few days for personal reasons, but hope to get back to it soon. | |
| 03:23:10 | Sundar | Any other topic to discuss? | |
| 03:23:49 | brinzhang_ | xinranwang: do we need to talked of the microversion change? | |
| 03:24:34 | brinzhang_ | https://review.opendev.org/#/c/696860/5 | |
| 03:25:17 | xinranwang | yes | |
| 03:25:23 | xinranwang | Thanks Brin | |
| 03:25:29 | brinzhang_ | np | |
| 03:25:39 | xinranwang | Now we only support 2.0 version | |
| 03:26:41 | brinzhang_ | My mainly concern is https://review.opendev.org/#/c/696860/5/cyborg/api/controllers/v2/versions.py@24, I think we should use completed version nubmer to help Cyborg control the microversion change | |
| 03:28:49 | openstackgerrit | Brin Zhang proposed openstack/cyborg master: Add Cyborg API Reference to Cyborg index https://review.opendev.org/711869 | |
| 03:29:04 | Sundar | I suggest we follow the precedent of placement or nova. Should we get cdent or somebody to review this? | |
| 03:29:05 | xinranwang | yes, Cyborg also use completed version number | |
| 03:29:28 | xinranwang | which locate L29 L30 in version.py | |
| 03:29:33 | xinranwang | brinzhang_: | |
| 03:30:17 | xinranwang | It is not the same framework, maybe ironic and watcher is more similiar | |
| 03:30:19 | brinzhang_ | xinranwang: yeah, for my comments, do we need L24 and L26? | |
| 03:30:53 | xinranwang | brinzhang_: it is easy for recording every microversion history | |
| 03:30:54 | brinzhang_ | xinranwang: And I think we need to change the _MIN_VERSION_STRING to _MIN_API_VERSION | |
| 03:31:20 | xinranwang | brinzhang_: how | |
| 03:32:16 | brinzhang_ | xinranwang: If we will change microversion to 3.0, this way seems not easy to convert, right? | |
| 03:32:44 | brinzhang_ | xinranwag: maybe it will be have a long time | |
| 03:32:49 | xinranwang | the version 3 will be implemented in v3 repo, not here | |
| 03:33:09 | xinranwang | and in controller/v3 repo, it will has its own versions.py file | |
| 03:33:58 | brinzhang_ | xinranwang: Ok, if so, threre is just some nits inline for me, just the defined parameters ^^ | |
| 03:34:56 | xinranwang | Ok | |
| 03:35:16 | xinranwang | This is pecan specific, which is diffirent from nova ;) | |
| 03:35:48 | brinzhang_ | I know, just want to have a good way to maintain ^^ | |
| 03:36:26 | xinranwang | Yes, so do we all agree to maintain like this? | |
| 03:37:13 | brinzhang_ | aree, but I would like you can change the defined parameter name (also just a nit) | |
| 03:37:26 | brinzhang_ | xinranwang:s/aree/agree | |
| 03:38:01 | xinranwang | Ok, thanks | |
| 03:38:44 | brinzhang_ | Sundar, chenke, Yumeng, s_shogo, do you all have some idea of this ? | |
| 03:38:52 | Sundar | For me too, it is just those minor points about variable names. | |
| 03:39:02 | chenke | @xinran Overall LGTM. I will. review it again. | |
| 03:39:15 | chenke | Yes | |
| 03:40:04 | Yumeng | yes. features are good enough. | |
| 03:40:18 | brinzhang_ | thanks xinranwang and all | |
| 03:40:23 | brinzhang_ | Sundar, pls do next agenda | |
| 03:40:41 | xinranwang | There are other projects naming with release name. If you guys prefer other ways like ADD_X_PARAM, I will update it. It is flexible. | |
| 03:41:24 | xinranwang | btw, please review deployabel API patch when you got time, I think it is almost ready. | |
| 03:41:49 | chenke | +1 | |
| 03:41:50 | Sundar | I don't have any specific next agenda item. | |
| 03:42:03 | chenke | how about your nova patches? | |
| 03:42:08 | chenke | Sundar | |
| 03:42:31 | brinzhang_ | xinranwang: just need change _MIN_VERSION_STRING to _MIN_API_VERSION, and use current_api_version or thers you thinks instead of "Version.sting" | |
| 03:42:51 | Sundar | xinranwang: Could you add a pointer to those projects's code that uses release names? If there is any precedent code that I can look at, for microversions in general, that would be useful. | |
| 03:43:29 | xinranwang | sure, let me find it. | |
| 03:43:37 | Sundar | chenke: I have got some review comments on Novapatches. I was off for a few days but will resume again. | |
| 03:44:30 | chenke | Okay. | |
| 03:45:32 | Sundar | xinranwang: You can provide those references in Gerrit. | |
| 03:45:41 | Sundar | Anything else, folks? | |
| 03:45:46 | s_shogo | Mere Reporting , the client topic mentioned in the last IRC : the cyborg client from pip doesn't include sdk implementation now.(That doesn't be released yet.) | |
| 03:45:47 | brinzhang_ | https://review.opendev.org/#/q/status:open+project:openstack/cyborg+branch:master+topic:story/2007397 | |
| 03:46:02 | brinzhang_ | pls review this change, I think it's ready to go | |
| 03:47:36 | xinranwang | Sundar: ok | |
| 03:47:53 | brinzhang_ | s_shogo: so cyborgclinet can support "pip install" and the "SDK" doesnot, right? | |
| 03:48:31 | brinzhang_ | I have used pip install to install the cyborgclient, but not try to test SDK yet | |
| 03:48:56 | s_shogo | brinzhang_ sdk is ok , but client is not yet. | |
| 03:49:22 | s_shogo | (by pip) | |
| 03:50:10 | brinzhang_ | s_shogo: get it, but I can install python-cyborgclinet to my local env (devstack). I will check the client version again | |
| 03:51:41 | s_shogo | brinzhang_ thanks .The client installed pip is probably old one,without sdk version. | |
| 03:51:43 | brinzhang_ | root@ubuntu-OpenStack:~# cyborg --version | |
| 03:51:44 | brinzhang_ | 1.0.0 | |
| 03:52:04 | brinzhang_ | Ah, 1.0.0 | |
| 03:52:22 | Sundar | brinzhang_: I think https://review.opendev.org/#/q/status:open+project:openstack/cyborg+branch:master+topic:story/2007397 needs to be updated, because the top patches are based on PS 4/6 of the bottom patch, instead of the latest 6/6. | |
| 03:53:08 | brinzhang_ | Sundar, it's not any impact | |
| 03:53:48 | brinzhang_ | the base patch just update the json data | |
| 03:54:00 | Sundar | ok | |
| 03:55:37 | brinzhang_ | Sundar: Thanks | |
| 03:56:40 | Sundar | Any thoughts or reviews of https://review.opendev.org/#/c/712231/ ? | |
| 03:57:24 | Sundar | Oh, I see some comments, but no -1. WIll respond to the comments | |
| 03:57:29 | brinzhang_ | Sundar I left some comments inline^^ | |
| 03:57:50 | Sundar | ok | |
| 03:58:03 | Yumeng | I was reviewing it yesterday. but not finished. | |