| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2018-04-23 | |||
| 13:45:17 | dtroyer_zz | frickler: I'm done travelling for a while, hope to get caught up Real Soon Now | |
| 13:55:51 | amotoki | frickler: ack. sorry for late. I have been busy due to the beginning of FY here. | |
| 14:26:00 | openstackgerrit | Mário Santos proposed openstack-infra/shade master: Adds kwargs support when creating a Neutron subnet https://review.openstack.org/563134 | |
| 14:49:31 | openstackgerrit | Merged openstack/api-wg master: Remove use of Python builtin name for variables https://review.openstack.org/560994 | |
| 20:19:48 | openstackgerrit | Ilya Margolin proposed openstack/openstacksdk master: Fix openstack-inventory https://review.openstack.org/563754 | |
| 20:40:51 | openstackgerrit | Ilya Margolin proposed openstack/openstacksdk master: Fix openstack-inventory https://review.openstack.org/563754 | |
| #openstack-sdks - 2018-04-24 | |||
| 02:33:20 | adriant | mordred: not sure if bug or something silly I'm doing... but list(conn.image.images()) against v2 throws me a not found error because it is trying to call "/v2/v2/images" | |
| 02:33:49 | adriant | also, I didn't realise this at first, but because of how my interpreter piggybacks on the OSC if you call it with --debug... it work! | |
| 02:33:55 | adriant | which is weird, but so very useful | |
| 02:34:24 | adriant | you get a shit load of debug! | |
| 02:53:52 | adriant | also GAH!!! WHY GLANCE! The API limits to 25 items by default... so the fucking glance client uses the pagination to get around that and 'list all' which means what you think is one API call without a limit set is like 20 | |
| 02:54:50 | adriant | with the sdk you have to explicitly set a crazy high limit to get around this, which is fine, but a failing of the API (which can't be changed) | |
| 02:55:17 | adriant | Am I wrong to assume that an API shouldn't limit list calls unless asked to? | |
| 03:17:03 | adriant | mordred: it's the pagination in the sdk | |
| 03:17:08 | adriant | that appears is to be my issue... | |
| 03:17:19 | adriant | and image() calls list with pagination true by default | |
| 03:24:10 | adriant | mordred: each iteration of the pagination is... prepending an extra version? What? | |
| 03:25:18 | adriant | Known bug, fix merged to master already nvm! | |
| 03:25:19 | adriant | https://review.openstack.org/#/c/560975/1/openstack/resource.py | |
| 03:25:24 | adriant | and now being backported | |
| 04:00:39 | adriant | Oh well, at least now I understand more of the code. | |
| 06:06:31 | samP | mordred: hi! | |
| 06:14:24 | samP | Hi! Any one can help me to have new version of openstack-sdk? | |
| 06:14:43 | samP | #link http://lists.openstack.org/pipermail/openstack-dev/2018-April/129578.html | |
| 06:15:21 | samP | Details are in ^^ | |
| 11:31:06 | openstackgerrit | Sergey Skripnick proposed openstack/openstacksdk master: Heat files parameter https://review.openstack.org/563035 | |
| 11:34:56 | heatuser | Hi all. I'm trying to create a stack (heat) but can't figure out how to provide "files" | |
| 11:36:05 | heatuser | There is nothing about files here https://docs.openstack.org/openstacksdk/latest/user/proxies/orchestration.html and here https://docs.openstack.org/openstacksdk/latest/user/resources/orchestration/v1/stack.html#openstack.orchestration.v1.stack.Stack | |
| 11:37:13 | heatuser | Is this even possible? | |
| #openstack-sdks - 2018-04-25 | |||
| 06:27:47 | openstackgerrit | Monty Taylor proposed openstack/keystoneauth master: Turn normalize_status into a class https://review.openstack.org/564110 | |
| 09:20:57 | mordred | samP: heya! sorry - I've been unexpectedly afk for a bit - yes, definitely want to cut an sdk release | |
| 09:30:33 | openstackgerrit | Merged openstack-infra/shade master: Trivial: Update pypi url to new url https://review.openstack.org/563412 | |
| 09:46:46 | rlupton20 | I'm trying to find where I should raise a bug against the openstack python sdk - the launchpad bug page seems inactive: https://bugs.launchpad.net/python-openstacksdk | |
| 09:46:56 | rlupton20 | where should I be raising issues? | |
| 09:51:19 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Fix bugtracker and documentation references https://review.openstack.org/564154 | |
| 09:51:36 | mordred | rlupton20: I *just* pushed up a patch fixing the docs on that ^^ :) | |
| 09:51:49 | mordred | rlupton20: https://storyboard.openstack.org/#!/project/972 | |
| 09:52:21 | rlupton20 | Thanks - so I should log in and raise the issue there? | |
| 09:52:38 | mordred | yes please | |
| 09:53:09 | rlupton20 | and I should add it as a story? | |
| 09:53:15 | mordred | Shrews, adriant: wanna quick/easy +2/+A - https://review.openstack.org/564154 (I think we can single-core that) | |
| 09:53:19 | mordred | rlupton20: yes | |
| 10:01:00 | rlupton20 | https://storyboard.openstack.org/#!/story/2001910 | |
| 10:22:48 | frickler | mordred: I think we should add a note about the storyboard migration to the channel topic, seen those questions multiple times now | |
| 10:37:11 | umbSublime | Starting at what version of openstack-sdk where the shade features merged in ? | |
| 11:08:21 | openstackgerrit | Merged openstack/openstacksdk master: Fix typo in README.rst https://review.openstack.org/562916 | |
| 11:08:21 | openstackgerrit | Merged openstack/openstacksdk master: Trivial: Update pypi url to new url https://review.openstack.org/563287 | |
| 11:08:35 | mordred | frickler: ++ | |
| 11:10:12 | mordred | umbSublime: 0.10 | |
| 11:17:22 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Fix bugtracker and documentation references https://review.openstack.org/564154 | |
| 11:17:36 | frickler | mordred: ^^ thus? or do you want more text around that? | |
| 11:17:38 | mordred | frickler: ^^ had to fix merge conflict | |
| 11:19:45 | mordred | frickler: I think that's great for now, let's seehow it goes? | |
| 11:21:21 | frickler | mordred: o.k. and +3 for your patch | |
| 11:22:01 | mordred | \o/ | |
| 11:22:28 | mordred | frickler: I think as soon as it lands we can cut a release and get those bugfixes out there so adriant doesn't have to keep re-solving them :) | |
| 11:24:35 | frickler | mordred: cool. do you also review OSC patches? I have a couple of them queued (as in "been nagging reviewers about them for some time now") | |
| 11:43:22 | mordred | frickler: not in any useful way (I'm not core or anything) - I just usually ping folks | |
| 11:58:22 | umbSublime | thanks mordred | |
| 12:40:25 | openstackgerrit | Monty Taylor proposed openstack/keystoneauth master: Turn normalize_status into a class https://review.openstack.org/564110 | |
| 13:00:44 | openstackgerrit | Pavlo Shchelokovskyy proposed openstack/python-openstackclient master: Support locking user password https://review.openstack.org/564200 | |
| 13:13:42 | Shrews | mordred: ohai | |
| 14:24:59 | Shrews | who lifted our olaph ban??? | |
| 15:11:52 | openstackgerrit | Paul Belanger proposed openstack-infra/shade master: Add get_volume_limits() support https://review.openstack.org/564247 | |
| 15:37:24 | openstackgerrit | Lance Bragstad proposed openstack/keystoneauth master: Use Status variables in tests https://review.openstack.org/564258 | |
| 15:53:23 | openstackgerrit | Lance Bragstad proposed openstack/keystoneauth master: Reference class variable in Status https://review.openstack.org/564262 | |
| 15:54:04 | Shrews | TheJulia: looks like the bifrost jobs on shade are failing. any idea what's going on there? | |
| 15:56:39 | Shrews | "msg": "An unhandled exception occurred while running the lookup plugin 'file'. Error was a <class 'ansible.errors.AnsibleError'>, original message: could not locate file in lookup: /tmp/baremetal.json.rest" | |
| 16:24:43 | mordred | samP: fwiw, patch going in to fix doc things and as soon as it lands we'll cut a release - but I'm about to AFK for the day, so I'll likely do it first thing in the morning | |
| 16:25:30 | mordred | Shrews: from the ironic channel... | |
| 16:25:37 | mordred | 15:37:14 TheJulia | anyway, stepping away for coffee and short brain break before resuming | |
| 16:26:06 | mordred | TheJulia: yay! | |
| 16:26:11 | TheJulia | Shrews: as of right now, or from a few days ago? | |
| 16:26:18 | TheJulia | hmmmmmm that is new | |
| 16:26:34 | Shrews | TheJulia: now and a few days ago | |
| 16:26:44 | TheJulia | mordred: oO | |
| 16:26:59 | Shrews | TheJulia: the latest https://review.openstack.org/564247 | |
| 16:27:12 | Shrews | hey, that's my box of rhinos | |
| 16:27:33 | mordred | Shrews: you'd be amazed at the pile of those boxes I found here | |
| 16:29:51 | TheJulia | Shrews: dude, I'm a little disturbed by these boxes of rhinos... unless we're talking about cookies. | |
| 16:30:23 | Shrews | TheJulia: looks like that review failed with a different error, actually | |
| 16:30:33 | pabelanger | I think I know what is happening on bifrost jobs, it has to do with removal of install-distro-packages.sh I believe | |
| 16:30:36 | Shrews | TheJulia: "Failed to start mariadb.service: Unit mariadb.service not found." | |
| 16:30:52 | pabelanger | bindep is using shade bindep.txt file, not bifrosts | |
| 16:31:11 | pabelanger | let me confirm | |
| 16:31:18 | TheJulia | we could just explicitly call it... | |
| 16:31:46 | pabelanger | yes, that will be the fix | |
| 16:31:49 | pabelanger | let me propose patch | |
| 16:32:55 | TheJulia | pabelanger: thanks! | |
| 16:38:54 | openstackgerrit | Paul Belanger proposed openstack-infra/shade master: Add get_volume_limits() support https://review.openstack.org/564247 | |
| 16:39:03 | pabelanger | okay, fixed and added depends-on to test | |
| 16:40:08 | openstackgerrit | Merged openstack/keystoneauth master: Expose version status in EndpointData https://review.openstack.org/559125 | |
| 16:43:06 | Shrews | pabelanger: oh, i was wondering how you identified the problem so quickly | |
| 16:45:01 | pabelanger | yah, my fault, sorry about that | |
| 16:45:52 | pabelanger | bifrost jobs are still legacy, so some oddities there | |
| 17:09:16 | openstackgerrit | Merged openstack/openstacksdk master: Fix bugtracker and documentation references https://review.openstack.org/564154 | |
| 17:21:45 | pabelanger | Shrews: TheJulia: patch looks to have fixed bifrost jobs from a shade patch | |
| 17:22:57 | Shrews | pabelanger: yep, look like it. awesome | |
| 17:34:23 | openstackgerrit | Sean McGinnis proposed openstack/python-openstackclient master: Remove invalid 'unlock-volume' migration arg https://review.openstack.org/558910 | |
| 17:38:12 | openstackgerrit | Monty Taylor proposed openstack/keystoneauth master: Infer version from old versioned service type aliases https://review.openstack.org/564299 | |