| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2018-04-25 | |||
| 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 | |
| #openstack-sdks - 2018-04-26 | |||
| 01:50:15 | adriant | mordred: it's not that I keep re-solving them, I just get curious how stuff works and then dig into it myself :P I need to get better at checking for existing patches and bug reports :/ | |
| 01:52:43 | adriant | although, mordred, I'm tempted to look at adding swift object chunking support to the SDK itself. We probably want to share that code, so either we find a common place for it, or the proxy uses the shade code, or we move that code into the service code. No clue what's nicer. | |
| 02:12:41 | adriant | that said, there is also the issue that shade doesn't let you pass in an open file stream so if the file NEEDS to be opened as binary to send it to swift... you can't: https://pastebin.com/BHMNUPF4 | |
| 02:13:06 | adriant | and doing it with the sdk is 3 lines... | |
| 02:13:42 | adriant | trying to upload a gzip'd tar file with shade throws: UnicodeDecodeError: 'utf-8' codec can't decode byte 0x8b in position 1: invalid start byte | |
| 02:17:31 | adriant | will file a bug with a easy step by step guide to reproduce. | |
| 02:36:53 | adriant | mordred: https://storyboard.openstack.org/#!/story/2001918 | |
| 11:03:18 | mordred | adriant: *awesome* | |
| 11:04:40 | mordred | adriant: re: chunking - (and other similar things where we have an advanced version of something in shade layer) - I've been thinking we should move the shade call into the sdk proxy layer and turn the shade call into a passthrough | |
| 11:05:32 | mordred | adriant: but I agree on no clue what's nicer - I figured trying one or two of them and seeing what we like ... | |
| 11:12:15 | mordred | adriant: hrm. looking at your bug - that's really weird - we use the shade code in nodepool to upload qcow images to rackspace ... | |
| 11:22:17 | openstackgerrit | Monty Taylor proposed openstack/keystoneauth master: Turn normalize_status into a class https://review.openstack.org/564110 | |
| 11:22:19 | openstackgerrit | Monty Taylor proposed openstack/keystoneauth master: Infer version from old versioned service type aliases https://review.openstack.org/564299 | |
| 11:22:20 | openstackgerrit | Monty Taylor proposed openstack/keystoneauth master: Make VersionData class https://review.openstack.org/564469 | |
| 13:03:55 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Don't assume a full config dict https://review.openstack.org/564493 | |
| 13:03:56 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Defer all endpoint discovery to keystoneauth https://review.openstack.org/564494 | |
| 13:04:33 | openstackgerrit | Monty Taylor proposed openstack/keystoneauth master: Infer version from old versioned service type aliases https://review.openstack.org/564299 | |
| 13:04:34 | openstackgerrit | Monty Taylor proposed openstack/keystoneauth master: Allow tuples and sets in interface list https://review.openstack.org/564495 | |
| 13:12:55 | pabelanger | mordred: mind looking at https://review.openstack.org/564247/ again, volume limits for shade | |
| 13:16:57 | mordred | pabelanger: NO! NO REVIEW FOR YOU | |
| 13:18:04 | mordred | pabelanger: jk. +A | |
| 13:18:17 | pabelanger | Yay | |
| 13:23:26 | mordred | pabelanger: thanks for that | |
| 13:23:28 | umbSublime_ | when using the sdk's compute proxy compute.servers() method. Is there a way to get the count returned in the generator object without iterating over it ? | |
| 13:24:27 | mordred | umbSublime_: not really - the generator itself doesn't known a total count because of server-side pagination | |
| 13:24:37 | umbSublime_ | ahh I see | |
| 13:24:56 | mordred | umbSublime_: it would be a nice thing to have if it was possible :) | |
| 13:25:31 | umbSublime_ | indeed! | |
| 13:31:35 | pabelanger | mordred: np, the other issue we might need to do is some process to check for leaked volumes, like we did for FIPs in nodepool. I am sure you might have ideas on how to handle that | |
| 13:32:26 | mordred | pabelanger: I havethe beginnings of some | |
| 13:32:44 | pabelanger | cool | |
| 13:33:10 | mordred | pabelanger: do you know if our leaked volumes in vexxhost are still there? | |
| 13:33:58 | pabelanger | mordred: I think so, I haven't cleaned them up yet | |
| 13:37:36 | mordred | pabelanger: awesome. let me look at them as examples | |
| 13:38:03 | mordred | pabelanger: my biggest question is whetheror not they still show the old server id in their metadata | |
| 13:38:31 | mordred | pabelanger: (figuring out how to identify volumes that are safe to automatically delete is ... terrifying | |
| 13:38:45 | mordred | pabelanger: cause, you know, get that wrong and you could give someone a giant foot-cannon | |
| 13:40:28 | pabelanger | Agree, I too am hoping for some sort of metadata | |
| 13:42:33 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Add get_volume_limits() support https://review.openstack.org/564507 | |
| 13:42:43 | mordred | pabelanger: ^^ there, that's cherry-picked to sdk | |
| 13:45:06 | pabelanger | mordred: ah, right. Development on openstack-infra/shade is ending? | |
| 13:48:04 | mordred | pabelanger: wel - not quite yet - but yeah, for now we're cherry-picking patches | |
| 13:48:35 | mordred | pabelanger: hopefully we'll be able to get far enough along this cycle to make shade a thin wrapper | |
| 13:48:57 | pabelanger | ack | |