| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2020-06-24 | |||
| 14:14:11 | mordred | (stupid up arrow) | |
| 14:14:35 | mordred | gtema: also - if you want a more fun review: https://review.opendev.org/#/c/737608/ | |
| 14:14:53 | gtema | wow, it actually really work in LimeChat XD | |
| 14:39:37 | dtantsur | mordred: hey, does OSC automatically pick up keystoneauth auth plugins? | |
| 14:39:45 | dtantsur | i.e. can we already do --auth-type http_basic ? | |
| 14:41:56 | mordred | dtantsur: should do | |
| 14:42:36 | dtantsur | nice! | |
| 14:42:57 | mordred | dtantsur: if it doesn't work, there are some hooks in osc to support entering password from a prompt if it's not given... it's possible we may need to think about those | |
| 14:43:25 | dtantsur | we haven't tried yet :) | |
| 14:43:25 | mordred | but - you know - should work | |
| 14:43:31 | mordred | let me know how it goes! | |
| 14:52:30 | dtantsur | sure | |
| 14:56:35 | mordred | johnsom: https://review.opendev.org/#/c/737731 is good to go I think - | |
| 15:03:17 | johnsom | mordred Will take a look, it's marked WIP but I can ping Carlos on that. | |
| 15:18:17 | mordred | johnsom: yeah- I think https://review.opendev.org/#/c/735858/ being green is what we're looking for | |
| 15:18:47 | openstackgerrit | Merged openstack/ansible-collections-openstack master: Add more useful information from exception https://review.opendev.org/736953 | |
| 15:27:34 | iurygregory | mordred, hey o/ sorry to bother you, if you have some time today or tomorrow https://review.opendev.org/#/c/718746/ =) | |
| 15:30:27 | openstackgerrit | Merged openstack/ansible-collections-openstack master: Update author lines https://review.opendev.org/736307 | |
| 15:34:30 | mordred | iurygregory: yay it's green! | |
| 15:34:37 | mordred | dtantsur: ^^ that seems like one for you too | |
| 15:34:42 | iurygregory | yup =) | |
| 15:35:22 | iurygregory | Im only pinging because the guy have a deadline for this >.< till end of this month... | |
| 15:37:47 | mordred | iurygregory: it looks reasonable to me | |
| 15:37:59 | iurygregory | cool \o/ | |
| 16:20:03 | elmiko | dtantsur, gtema_ , mordred , we've been sent an invitation to add a slide about the api sig for the upcoming openstack year in review slide deck, and this is the 10 year anniversary special. i'm not sure we have much to mention in a full slide about the work that the api sig has done in the last year, but i am wondering if the sdk sig is doing something and if perhaps we could combine efforts? | |
| 16:20:34 | dtantsur | elmiko: we can mention our precious microversioning as our achievement :) | |
| 16:21:02 | elmiko | hahaha | |
| 16:21:18 | elmiko | that and finally getting to merge the service discovery docs? | |
| 16:21:20 | gtema_ | it's not yet merged in it's last form ;-) | |
| 16:36:50 | dtantsur | yep | |
| 16:38:23 | elmiko | i plan on merging the service discovery stuff tomorrow if no -1's appear | |
| 16:38:55 | gtema_ | cool | |
| 16:41:48 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Add support for multiple image stores https://review.opendev.org/737845 | |
| 17:32:58 | frickler | mordred: would you know an answer to this? person sadly already has left, but might still be good to fix that link http://eavesdrop.openstack.org/irclogs/%23openstack/latest.log.html#t2020-06-24T16:04:51 | |
| 17:35:03 | mordred | hrm. we should ask ajaeger | |
| 17:39:02 | mordred | diablo_rojo_phon: ^^ do you know where teh first-app stuff has gone? we need to do two things: a) update the link from openstack.org/appdev to something real - b) make that thing use openstacksdk and not libcloud | |
| 17:39:57 | diablo_rojo_phon | I do not actually. | |
| 17:40:05 | diablo_rojo_phon | I can do some digging though. | |
| 17:40:09 | mordred | darn. I think firstapp may not exist anymore | |
| 17:40:25 | diablo_rojo_phon | It might not.. | |
| 17:40:37 | mordred | diablo_rojo_phon: maybe as a temporary thing we can get the website folks to update the link on openstack.org/appdev to point to https://docs.openstack.org/openstacksdk/latest/user/index.html instead | |
| 17:40:52 | mordred | and then at least it won't be broken until we can figure out an actual story | |
| 17:41:31 | diablo_rojo_phon | Yeah can definitely do that. | |
| 17:41:37 | diablo_rojo_phon | I'll go mention it now. | |
| 17:41:37 | mordred | diablo_rojo_phon: woot - thanks! | |
| 17:42:05 | mordred | diablo_rojo_phon: it's the thing that currently links to http://developer.openstack.org/firstapp-libcloud/getting_started.html?_ga=2.85215833.1144355264.1592939032-247414602.1591288216 | |
| 17:42:55 | diablo_rojo_phon | So.. when I go https://www.openstack.org/appdev it points to something.. but yeah ^ link does not. | |
| 17:43:46 | mordred | yah | |
| 17:43:48 | mordred | https://github.com/OpenStackweb/openstack-org/blob/c41955c5a6b283276978b991a7e34cf84eed1127/themes/openstack/templates/Layout/AppDevHomePage.ss#L89 | |
| 17:43:53 | mordred | diablo_rojo_phon: ^^ that's the bad link source line | |
| 17:44:17 | mordred | I suppose I could just submit a PR | |
| 17:44:31 | diablo_rojo_phon | So that should point to https://docs.openstack.org/openstacksdk/latest/user/index.html instead then? | |
| 17:44:56 | diablo_rojo_phon | Heh yeah you could if you want. Then I can still poke Jimmy and people to get it merged. | |
| 17:45:06 | diablo_rojo_phon | Up to you mordred :) | |
| 17:48:23 | mordred | diablo_rojo_phon: https://github.com/OpenStackweb/openstack-org/pull/169 | |
| 17:48:25 | mordred | thanks! | |
| 17:48:51 | diablo_rojo_phon | Passing it along now :) | |
| 17:49:02 | johnsom | mordred +W on the octavia patch to fix SDK. It's in the gate queue. | |
| 17:49:54 | mordred | johnsom: woot! thanks! | |
| 18:11:05 | frickler | mordred: seems you need to re-approve 735858 as octavia is not in the integrated queue | |
| 18:15:57 | mordred | frickler: yah - although I think I should wait on the octavia patch landing for that to work | |
| 18:16:02 | mordred | because it's not in the itegrated queue | |
| 18:16:18 | johnsom | yep | |
| 18:17:02 | frickler | ah, right | |
| 20:16:41 | johnsom | The patch is merged now | |
| #openstack-sdks - 2020-06-25 | |||
| 08:25:18 | openstackgerrit | Vishakha Agarwal proposed openstack/openstacksdk master: Add access rule CRUD support https://review.opendev.org/734549 | |
| 12:32:11 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Upload image via interop import if needed https://review.opendev.org/737608 | |
| 12:32:11 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Add support for multiple image stores https://review.opendev.org/737845 | |
| 12:33:46 | mordred | gtema: ^^ interop import patch there should be working now - I've checked https://review.opendev.org/#/c/734184/ - which is a nova patch that exercises it in the gate via openstackclient | |
| 12:34:25 | mordred | gtema: silly me, I thought type=list would turn a comma separated header into a list | |
| 12:34:25 | gtema | oki | |
| 12:34:38 | gtema | do we have our tests repaired? | |
| 12:35:10 | mordred | gtema: looks like you rechecked the slaweq patch | |
| 12:35:16 | gtema | yes | |
| 12:35:29 | gtema | I see in zuul the only remaining job is networking | |
| 12:35:37 | mordred | fingers crossed | |
| 12:35:41 | gtema | hopefully it will go through this time | |
| 12:36:43 | gtema | I see you now follow the formatting style of the black for new functions | |
| 12:36:45 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Drop python3.5 support https://review.opendev.org/736058 | |
| 12:36:46 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Upload image via interop import if needed https://review.opendev.org/737608 | |
| 12:37:05 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Add support for multiple image stores https://review.opendev.org/737845 | |
| 12:37:23 | mordred | gtema: I rebased those on top of the network fix | |
| 12:37:28 | mordred | gtema: yes - I really like it actually | |
| 12:37:31 | gtema | oki | |
| 12:37:35 | gtema | yeah, mee to | |
| 12:37:48 | mordred | I think it's great for later refactoring to touch fewer lines | |
| 12:37:56 | gtema | agree | |
| 12:38:07 | mordred | I *almost* want to reformat our codebase to black | |
| 12:38:11 | mordred | but - like - oy | |
| 12:38:19 | gtema | :D | |
| 13:20:04 | mordred | gtema: ok - so I just went down a time-wasting hole about linters and black - and wound up adding flake8-bugbear ... which actually did catch some new bugs :( | |
| 13:20:24 | gtema | oh, that's interesting | |
| 13:20:40 | mordred | yeah. we have a bunch of {} in argument lists | |
| 13:21:29 | gtema | okay | |
| 13:22:09 | frickler | mordred: did you see http://lists.openstack.org/pipermail/openstack-discuss/2020-June/015610.html ? | |
| 13:29:20 | mordred | frickler: no, I didn't - but thanks, that should be fixable | |
| 13:43:28 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Provide hypervisor_hostname in Hypervisor object https://review.opendev.org/738026 | |
| 14:20:11 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Add use of flake8-bugbear https://review.opendev.org/738041 | |
| 14:21:11 | mordred | gtema: ^^ enjoy | |
| 14:21:27 | gtema | thks, already noticed amount of files | |