| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2017-12-29 | |||
| 14:41:56 | bruva242 | ▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄ IRC.SUPERNETS.ORG IS HOSTING THE DDOS AND FLOODING CHALLENGE..PLEASE SEE VAP0R IN #SUPERBOWL FOR DETAILS!! dutpibtvfa: DuncanT d0ugal edleafe samueldmq johnsom charz ldl rcarrillocruz tristanC thrash lifeless Shrews tobberydberg tonyb TheJulia mnaser rm_work Klumben csatari Kevin_Zheng irenab alex_x | |
| 14:42:00 | bruva242 | ▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄ IRC.SUPERNETS.ORG IS HOSTING THE DDOS AND FLOODING CHALLENGE..PLEASE SEE VAP0R IN #SUPERBOWL FOR DETAILS!! soqmfggh: dhellmann rm_work rcarrillocruz edleafe charz NobodyCam vdrok dtantsur|afk thrash DuncanT d0ugal baffle tristanC mnaser dayou Hazelesque_ Kevin_Zheng Jeffrey4l purplerbot Shrews csatari ameade openstackgerrit iren | |
| 14:42:04 | bruva242 | ▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄ IRC.SUPERNETS.ORG IS HOSTING THE DDOS AND FLOODING CHALLENGE..PLEASE SEE VAP0R IN #SUPERBOWL FOR DETAILS!! ityui: csatari yolanda DuncanT lbragstad vdrok zhongjun shu-mutou-AWAY openstack Jeffrey4l charz zacksh purplerbot openstackgerrit d0ugal Kevin_Zheng tonyb baffle calebb tob | |
| #openstack-sdks - 2017-12-31 | |||
| 06:01:23 | ldl | why openstack-sdk do not create specs for it? | |
| 13:00:27 | openstackgerrit | Merged openstack/python-openstackclient master: Allow port list to shown undefined attributes https://review.openstack.org/522901 | |
| #openstack-sdks - 2018-01-02 | |||
| 04:35:17 | openstackgerrit | Nguyen Hung Phuong proposed openstack/cliff master: Remove empty files https://review.openstack.org/530674 | |
| 10:51:03 | ldl | hi | |
| 10:52:01 | ldl_ | hi | |
| 13:26:58 | cdent | elmiko, edleafe, dtantsur|afk : did you receive a link to the pending foundation report where they want some input from api-sig on what we did? | |
| 13:46:03 | elmiko | cdent: i have not seen said link yet | |
| 15:45:11 | edleafe | cdent: I'm horribly distracted at the moment as a result of having to set up an office 2 hours from my home, so if I got something, I won't find it for a few hours | |
| 15:46:17 | cdent | edleafe: no worries, I think I can take care of it. And I'm pretty sure they sent it just to me. Probably alphabetical choice | |
| 15:46:49 | edleafe | cdent: or since you're more visibile, with the TC and all | |
| 16:02:19 | openstackgerrit | Merged openstack/cliff master: Add ability to sort data by columns for list commands https://review.openstack.org/521351 | |
| 16:34:44 | openstackgerrit | Monty Taylor proposed openstack/python-openstacksdk master: WIP: Make resource2.Resource inherit from Munch https://review.openstack.org/529592 | |
| 16:34:44 | openstackgerrit | Monty Taylor proposed openstack/python-openstacksdk master: Start using Connection in openstack.cloud https://review.openstack.org/523259 | |
| 16:34:45 | openstackgerrit | Monty Taylor proposed openstack/python-openstacksdk master: Use links dicts for pagination https://review.openstack.org/530769 | |
| 16:34:45 | openstackgerrit | Monty Taylor proposed openstack/python-openstacksdk master: Remove self argument from connect helper function https://review.openstack.org/530768 | |
| 16:34:46 | openstackgerrit | Monty Taylor proposed openstack/python-openstacksdk master: Use sdk for list_servers https://review.openstack.org/530770 | |
| 16:35:26 | mordred | rods_: ^^ I rebased your patch in with that stack - and then updated it just a smidge - it works now - but I think there is still some work needed to make it good | |
| 18:34:27 | openstackgerrit | Merged openstack/python-openstacksdk master: Set empty Tag list if Network Tag API extension not supported https://review.openstack.org/523834 | |
| 18:43:04 | openstackgerrit | Merged openstack-infra/shade master: Add supported method for checking the network exts https://review.openstack.org/529360 | |
| 19:59:12 | rods_ | mordred thx, I'm giving a look in a minute | |
| 20:07:13 | mordred | rods_: just wait - I've got a REALLY fun followup coming :) | |
| 20:07:23 | mordred | rods_: (I may have gotten myself nerd-sniped) | |
| 21:24:57 | openstackgerrit | Julia Kreger proposed openstack-infra/shade master: WIP: Add support for portgroups/vifs https://review.openstack.org/530803 | |
| #openstack-sdks - 2018-01-03 | |||
| 03:08:13 | openstackgerrit | Merged openstack/python-openstacksdk master: Remove self argument from connect helper function https://review.openstack.org/530768 | |
| 04:25:23 | openstackgerrit | Monty Taylor proposed openstack/python-openstacksdk master: Make resource a dict subclass usable by shade layer https://review.openstack.org/530835 | |
| 04:25:24 | openstackgerrit | Monty Taylor proposed openstack/python-openstacksdk master: Rename Resource.get to Resource.fetch https://review.openstack.org/530836 | |
| 04:25:38 | mordred | rods_: ^^ ok. there you go - enjoy! | |
| 10:34:50 | cmurphy | || ggrreepp ssddbb | |
| 10:35:03 | cmurphy | rename pike.rst to master.rst i think | |
| 10:35:05 | cmurphy | || ggrreepp ssddbb | |
| 10:37:21 | cmurphy | oh wow sorry | |
| 10:38:04 | cmurphy | terminal problems | |
| 10:38:21 | cdent | no need to apologize, a) I was curious, b) it wasn't clear you knew it happened | |
| 15:19:35 | mordred | cdent, cmurphy: that was my first morning irc communication to read - I think today is going to be a good day | |
| 15:19:56 | cdent | agreed | |
| 15:23:10 | cmurphy | always happy to entertain | |
| 15:23:51 | cmurphy | (fwiw terminator + weechat = bad) | |
| 15:26:33 | Shrews | cmurphy: yeah, i had issues with that combo | |
| 16:52:46 | openstackgerrit | Merged openstack/python-openstackclient master: Update new documentation PTI jobs https://review.openstack.org/530378 | |
| 16:55:52 | openstackgerrit | Merged openstack/osc-lib master: Fix sorting in Python 3 https://review.openstack.org/526046 | |
| 17:26:46 | openstackgerrit | Monty Taylor proposed openstack/python-openstacksdk master: Update for new docs PTI https://review.openstack.org/530978 | |
| 21:11:03 | openstackgerrit | Hongbin Lu proposed openstack/python-openstackclient master: Allow ports filtering with device_id https://review.openstack.org/526798 | |
| 21:13:34 | mordred | cdent: you know everything, right? | |
| 21:14:19 | mordred | cdent: do you happen to know if there are any openstack services that paginate resources with limit/marker but do not return a {foo}_links dict with next links in it? | |
| 21:17:58 | mordred | cdent: the question comes up because of https://review.openstack.org/#/c/530769 and briancurtin's comments on it - I'm gonna go look and see what I can learn, but I figured if anyone happened to know off the top of their head about outliers that it would be helpful | |
| 21:18:21 | mordred | edleafe, elmiko: ^^ in case you happen to know | |
| 21:19:52 | edleafe | mordred: I confess that I do not know of any | |
| 21:20:07 | edleafe | mordred: I also confess that I haven't researched that at all :) | |
| 21:53:12 | elmiko | mordred: i haven't looked into that, and it looks like we have not done a study across projects about it. if you end up poking around, any info would be appreciated =) | |
| 21:53:15 | elmiko | https://wiki.openstack.org/wiki/API_Working_Group/Current_Design | |
| 22:12:14 | openstackgerrit | Merged openstack/python-openstacksdk master: Start using Connection in openstack.cloud https://review.openstack.org/523259 | |
| 22:14:51 | openstackgerrit | Bar Elharar proposed openstack/osc-lib master: Suppress subTest() expected errors https://review.openstack.org/531025 | |
| 22:18:36 | openstackgerrit | Bar Elharar proposed openstack/osc-lib master: Suppress subTest() expected errors https://review.openstack.org/531025 | |
| 22:26:00 | openstackgerrit | Bar Elharar proposed openstack/osc-lib master: Suppress subTest() expected errors https://review.openstack.org/531025 | |
| 22:28:44 | mordred | edleafe, elmiko: thanks - I think I'm going to start with a mailing list message and see where it gets me | |
| 22:40:54 | cdent | mordred, edleafe, elmiko : I was elsewhere, catching up | |
| 22:42:25 | mordred | cdent, edleafe, elmiko: I've found at least one example of a service that does not return $foo_links with a next link to indicate that there are more results available ... swift | |
| 22:43:32 | mordred | although now the question is - is it *only* swift (and therefore something that could be overridden in the sdk just for swift) or are there more, in which case adding a pagination_type flag to Resource objects may be the right choice | |
| 22:43:47 | cdent | are you using _ as an indicator of object descent, like server.links | |
| 22:44:36 | cdent | because both the api-wg guidance and looking at nova I'm not seeing what you're talking about, mordred, but I'm also not sure I know what you're talking about | |
| 22:46:01 | mordred | cdent: I've found another - glance is also different ... BUT ... for instance, https://developer.openstack.org/api-guide/compute/paginated_collections.html | |
| 22:46:18 | cdent | and https://developer.openstack.org/api-guide/compute/links_and_references.html | |
| 22:46:31 | mordred | cdent: nova returns {"servers": [...], "servers_links": [..]} | |
| 22:46:56 | mordred | so for a server list one can check servers_links for a next link, and if it exists, follow it | |
| 22:47:13 | cdent | sigh | |
| 22:47:44 | cdent | nobody ever follows the guidance ( http://specs.openstack.org/openstack/api-wg/guidelines/links.html) and even the services aren't in sync with themselves | |
| 22:48:10 | mordred | cdent: but for glance, it's {"images": [...], "next": "http:..."} | |
| 22:48:18 | cdent | blargh | |
| 22:48:22 | mordred | cdent: well - these links are a little bit different than the per-object links | |
| 22:48:51 | cdent | the pagination guidance doesn't distinguish, it's suggest generic 'links': http://specs.openstack.org/openstack/api-wg/guidelines/pagination_filter_sort.html#pagination | |
| 22:48:52 | mordred | cdent: I actually *do* see the reference links pretty frequently and consistently - other than when they're broken because reverse-proxies aren't set up correctly :) | |
| 22:49:36 | cdent | mordred: oh, I didn't mean nobody follows links. I mean projects not developing consistently. | |
| 22:49:44 | mordred | oh - yah. | |
| 22:50:07 | cdent | the pagination guidance example on that last link (heh) uses just 'links' in a relatively sane collection example | |
| 22:50:10 | briancurtin | since approximately day one, and going on until approximately forever, unfortunately | |
| 22:50:12 | mordred | cdent: that api-wg guidance is nice - and at least structurally similar to what I see from cinder and nova (naming notwithstanding) | |
| 22:50:15 | cdent | so yeah, I would guess that briancurtin is right | |
| 22:50:27 | mordred | briancurtin wins | |
| 22:52:14 | mordred | briancurtin: so - since a quick look has shown at least 4 different mechanisms, I'm thinking I should rework that patch to add a resource-level config flag choose one, with the behavior defaulting to the current sdk inferance behavior | |
| 22:53:01 | mordred | briancurtin: so that as we know or understand that a given resource provides links guidance we can opt a resource in to that behavior, but there is a good default that should work in the other cases | |
| 22:53:05 | briancurtin | mordred: that’s probably reasonable. we originally thought to go something like that direction for times when we knew for sure we wouldn’t need that extra call, so go for it | |
| 22:53:12 | briancurtin | yep | |
| 22:53:19 | mordred | woot | |
| 22:53:49 | mordred | briancurtin, cdent: see - if all the API services were consistent, we'd be issing out on all of this fun! | |
| 23:03:01 | cdent | oh look at the time, goodnight | |
| 23:16:37 | mordred | briancurtin: while I'm bugging you - https://review.openstack.org/#/c/530769/1/openstack/resource2.py line 765 in the original code ... seems like a bug - it'll stop iteration if yielded is < limit - shouldn't that be yielded >= query_params['limit'] - or is my brain just too melty? | |
| 23:18:11 | mordred | briancurtin: also - line 748 - yielded is reset in each loop iteration - seems like it should be moved to above line 734 - and that line 767 should be query_params['limit'] = limit - yielded | |
| 23:19:14 | mordred | briancurtin: I'm going to keep it as is in this patch so we can fix it if needed in its own patch - but just wanted to double-check my reading of the logic there | |
| #openstack-sdks - 2018-01-04 | |||
| 01:08:08 | openstackgerrit | Jake Yip proposed openstack/python-openstackclient master: Fix error with image show when image name is None https://review.openstack.org/529464 | |
| 02:06:01 | liuyulong | hello python-openstackclient core team, can we get this https://review.openstack.org/#/c/517845/ merged, since neutron doc is depending on it. | |
| 02:06:29 | liuyulong | neutron doc patch: https://review.openstack.org/#/c/526033/ | |
| 03:07:47 | dtroyer | liuyulong: I'll be able to verify that my concerns were addressed in the morning, I'd like amotoki to check his too… | |
| 03:09:34 | liuyulong | dtroyer, OK, thank you. | |
| 06:56:21 | openstackgerrit | OpenStack Proposal Bot proposed openstack/python-openstackclient master: Updated from global requirements https://review.openstack.org/531089 | |
| 07:54:51 | amotoki | liuyulong: I am in new year holidays till today. I am aware of yours and will check it tomorrow as I mentioned in the neutron team meeting this week. | |
| 07:57:13 | liuyulong | amotoki: Great, thank you. | |
| 08:51:33 | dtantsur | cdent: no, I haven't received any links. also likely to skip today's meeting due to personal stuff | |