| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2018-04-25 | |||
| 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 | |
| 13:59:19 | umbSublime_ | I remember talking here a few months ago about making a cutsom api request not supported by the SDK, at the time the work-around was to use the <ressource>._action method and pass the connection.session object. I was discussed that new standard way would be eventualy implemented in the SDK. Was this added since ? | |
| 13:59:29 | mugsie | is there any easy way to take a path to openstack-client-config file in openstack-sdk? | |
| 13:59:54 | mugsie | (not using a env var) | |
| 14:33:49 | mordred | mugsie: you want to override the location of the clouds.yaml file? | |
| 14:34:23 | mugsie | yeah - I am writing a certbot plugin, and the location of the clouds.yaml file may be $RANDOM_DIR | |
| 14:34:48 | mordred | mugsie: the config_files parameter to openstack.config.loader.OpenStackConfig is what you want | |
| 14:35:20 | mordred | mugsie: there's some examples of usage in the test suite - one sec, lemme find you a reference | |
| 14:35:33 | mugsie | mordred: sweet - thanks | |
| 14:35:51 | mordred | mugsie: actualy - openstack/cloud/inventory.py and look for config_files= | |
| 14:37:07 | mordred | mugsie: but you;ll need to do config = loader.OpenStackConfig( ... then cloud_region = config.get_one( ... then conn = connection.Connection(config=cloud_region) | |
| 14:37:35 | mugsie | yup, looks doable - thanks :) | |
| 14:50:08 | openstackgerrit | Merged openstack/openstacksdk master: Fix openstack-inventory https://review.openstack.org/563754 | |
| 14:58:27 | mordred | samP: remote: https://review.openstack.org/564527 Release openstacksdk 0.13.0 | |
| 15:00:07 | Shrews | pabelanger: did you ping someone about the bifrost jobs fix? can't merge your shade change w/o it | |