| *** slaweq has joined #openstack-sdks | 00:11 | |
| *** ricolin__ has joined #openstack-sdks | 00:13 | |
| *** slaweq has quit IRC | 00:15 | |
| *** ricolin_ has quit IRC | 00:16 | |
| *** whoami-rajat has quit IRC | 00:34 | |
| *** mriedem has quit IRC | 00:52 | |
| *** whoami-rajat has joined #openstack-sdks | 01:32 | |
| *** slaweq has joined #openstack-sdks | 02:11 | |
| *** slaweq has quit IRC | 02:15 | |
| *** altlogbot_2 has quit IRC | 02:44 | |
| *** altlogbot_1 has joined #openstack-sdks | 02:45 | |
| *** redrobot has quit IRC | 03:22 | |
| *** gtema has joined #openstack-sdks | 03:37 | |
| *** gtema has quit IRC | 03:41 | |
| *** ricolin__ is now known as ricolin | 04:09 | |
| *** slaweq has joined #openstack-sdks | 04:11 | |
| *** factor has quit IRC | 04:12 | |
| *** factor has joined #openstack-sdks | 04:12 | |
| *** slaweq has quit IRC | 04:16 | |
| *** dave-mccowan has quit IRC | 04:25 | |
| *** slaweq has joined #openstack-sdks | 05:11 | |
| *** slaweq has quit IRC | 05:15 | |
| *** dave-mccowan has joined #openstack-sdks | 05:34 | |
| *** gtema has joined #openstack-sdks | 05:38 | |
| *** e0ne has joined #openstack-sdks | 05:40 | |
| *** gtema has quit IRC | 05:42 | |
| *** e0ne has quit IRC | 05:59 | |
| *** slaweq has joined #openstack-sdks | 06:11 | |
| *** slaweq has quit IRC | 06:15 | |
| *** dtantsur|afk is now known as dtantsur | 06:28 | |
| *** slaweq has joined #openstack-sdks | 06:42 | |
| *** tosky has joined #openstack-sdks | 07:23 | |
| *** jpich has joined #openstack-sdks | 07:49 | |
| *** ralonsoh has joined #openstack-sdks | 08:06 | |
| openstackgerrit | Loan Harrouin proposed openstack/openstacksdk master: Add get_actions for server resource https://review.opendev.org/675113 | 08:29 |
|---|---|---|
| openstackgerrit | Brin Zhang proposed openstack/python-openstackclient master: Add server migration list CLI https://review.opendev.org/675304 | 08:45 |
| *** jpich has quit IRC | 09:09 | |
| *** jpich has joined #openstack-sdks | 09:09 | |
| *** e0ne has joined #openstack-sdks | 09:24 | |
| *** cdent has joined #openstack-sdks | 09:37 | |
| *** gtema has joined #openstack-sdks | 09:40 | |
| *** gtema has quit IRC | 09:44 | |
| *** gtema has joined #openstack-sdks | 09:46 | |
| openstackgerrit | Loan Harrouin proposed openstack/openstacksdk master: Add get_actions for server resource https://review.opendev.org/675113 | 09:48 |
| *** gtema has quit IRC | 09:51 | |
| openstackgerrit | Loan Harrouin proposed openstack/openstacksdk master: Add get_actions for server resource https://review.opendev.org/675113 | 10:04 |
| *** jpich has quit IRC | 10:13 | |
| *** jpich has joined #openstack-sdks | 10:14 | |
| frickler | kmalloc: mordred: I've refreshed the logs on my devstack patch dropping the admin endpoint, this is the error in h-eng that I'm failing to resolve https://logs.opendev.org/92/651492/7/check/openstacksdk-functional-devstack/afeaec0/controller/logs/screen-h-eng.txt.gz#_Aug_09_08_44_55_817151 | 10:54 |
| frickler | the traceback looks slightly similar to what cmurphy fixed for https://review.opendev.org/651790 | 10:56 |
| mordred | frickler: yeah - I think it means the devstack patch should also add interface=public to the middleware config - maybe that's just a thing that needs to be done for heat? | 11:15 |
| frickler | mordred: I tested that locally for all different heat config sections I have found, but it didn't seem to make any difference. | 11:28 |
| mordred | frickler: boo | 11:28 |
| mordred | frickler: maybe cmurphy will know the magical answers whence she awakens | 11:28 |
| *** gtema has joined #openstack-sdks | 11:52 | |
| openstackgerrit | Merged openstack/openstacksdk master: Initalize pool_executor so close works https://review.opendev.org/675120 | 11:56 |
| *** gtema has quit IRC | 11:57 | |
| openstackgerrit | Shubham Potale proposed openstack/openstacksdk master: Add support for nfv-orchestration service https://review.opendev.org/675604 | 12:12 |
| openstackgerrit | Shubham Potale proposed openstack/python-openstackclient master: Add python-openstackclient support to vnf package APIs https://review.opendev.org/675609 | 12:17 |
| openstackgerrit | Shubham Potale proposed openstack/python-openstackclient master: Add commands for create and delete vnfpackage APIs https://review.opendev.org/675610 | 12:17 |
| openstackgerrit | Shubham Potale proposed openstack/python-openstackclient master: Add commands for list and show vnf package API's https://review.opendev.org/675611 | 12:17 |
| openstackgerrit | Shubham Potale proposed openstack/python-openstackclient master: Add command for upload vnf package API https://review.opendev.org/675612 | 12:17 |
| openstackgerrit | Shubham Potale proposed openstack/python-openstackclient master: Add python-openstackclient support to vnf package APIs https://review.opendev.org/675609 | 12:23 |
| openstackgerrit | Shubham Potale proposed openstack/python-openstackclient master: Add commands for create and delete vnfpackage APIs https://review.opendev.org/675610 | 12:23 |
| openstackgerrit | Shubham Potale proposed openstack/python-openstackclient master: Add commands for list and show vnf package API's https://review.opendev.org/675611 | 12:23 |
| openstackgerrit | Shubham Potale proposed openstack/python-openstackclient master: Add command for upload vnf package API https://review.opendev.org/675612 | 12:23 |
| *** cdent has quit IRC | 12:48 | |
| *** cdent has joined #openstack-sdks | 12:48 | |
| *** cdent has quit IRC | 12:49 | |
| *** gtema has joined #openstack-sdks | 12:52 | |
| openstackgerrit | Bence Romsics proposed openstack/openstacksdk master: Add router add/remove route operations https://review.opendev.org/674324 | 12:59 |
| *** Guest1857 has joined #openstack-sdks | 12:59 | |
| openstackgerrit | Bence Romsics proposed openstack/python-openstackclient master: fixup! Add command: router add/remove route --route https://review.opendev.org/675618 | 13:01 |
| openstackgerrit | Bence Romsics proposed openstack/python-openstackclient master: Add command: router add/remove route --route https://review.opendev.org/674325 | 13:04 |
| *** cdent has joined #openstack-sdks | 13:09 | |
| *** mriedem has joined #openstack-sdks | 13:10 | |
| *** stephenfin is now known as finucannot | 13:46 | |
| *** lbragstad has joined #openstack-sdks | 13:46 | |
| *** gtema_ has joined #openstack-sdks | 13:48 | |
| openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Rename cloud_config to cloud_region https://review.opendev.org/675627 | 13:53 |
| mordred | dtantsur: ^^ I did the variable renaming in a followup | 13:54 |
| *** lbragstad has quit IRC | 13:58 | |
| dtantsur | cool! | 14:03 |
| *** lbragstad has joined #openstack-sdks | 14:04 | |
| openstackgerrit | Artem Goncharov proposed openstack/openstacksdk master: Rework statistics reporting https://review.opendev.org/659841 | 14:06 |
| *** lbragstad has quit IRC | 14:14 | |
| cmurphy | mordred: frickler no idea | 14:35 |
| *** cdent has quit IRC | 14:42 | |
| *** cdent has joined #openstack-sdks | 14:45 | |
| *** ricolin has quit IRC | 14:57 | |
| *** ricolin has joined #openstack-sdks | 15:01 | |
| openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Replace catalog-v3.json with keystoneauth fixture https://review.opendev.org/675187 | 15:10 |
| openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Remove container-infra.example.com hack https://review.opendev.org/675378 | 15:10 |
| openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Rename cloud_config to cloud_region https://review.opendev.org/675627 | 15:10 |
| *** gtema has quit IRC | 15:14 | |
| openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Skip most service_description for unknown services https://review.opendev.org/675135 | 15:28 |
| mordred | Shrews, efried, mriedem: ^^ this time with a test | 15:28 |
| mordred | (which even tests that no warnings are thrown) | 15:28 |
| openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Allow services to override proxy creation. https://review.opendev.org/674110 | 15:35 |
| mordred | that has a test now too | 15:36 |
| mordred | Shrews: that one should avoid the annoying warnings we get from swift about not being able to talk to the root url | 15:36 |
| openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Skip most service_description for unknown services https://review.opendev.org/675135 | 15:42 |
| openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Allow services to override proxy creation. https://review.opendev.org/674110 | 15:42 |
| openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Fix discovery cache sharing https://review.opendev.org/675121 | 15:42 |
| openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Add support for global_request_id https://review.opendev.org/674807 | 15:42 |
| openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Rationalize endpoint_for and get_endpoint_from_catalog https://review.opendev.org/675130 | 15:42 |
| openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Validate that connect_as connects as the project https://review.opendev.org/675178 | 15:42 |
| openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Replace catalog-v3.json with keystoneauth fixture https://review.opendev.org/675187 | 15:42 |
| openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Remove container-infra.example.com hack https://review.opendev.org/675378 | 15:42 |
| openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Rename cloud_config to cloud_region https://review.opendev.org/675627 | 15:42 |
| mordred | rebased the stack | 15:42 |
| *** gtema_ has quit IRC | 15:43 | |
| *** gtema has joined #openstack-sdks | 15:44 | |
| *** e0ne has quit IRC | 15:44 | |
| *** cmurphy is now known as cmorpheus | 15:45 | |
| *** gtema has quit IRC | 15:46 | |
| openstackgerrit | Dean Troyer proposed openstack/python-openstackclient master: Add 'openstack server resize (confirm|revert)' commands https://review.opendev.org/665357 | 15:48 |
| *** cdent has quit IRC | 15:57 | |
| openstackgerrit | Dean Troyer proposed openstack/python-openstackclient master: Add openstack server create --boot-from-volume option https://review.opendev.org/674111 | 15:58 |
| openstackgerrit | Dean Troyer proposed openstack/python-openstackclient master: Fix compute service set handling for 2.53+ https://review.opendev.org/672577 | 16:00 |
| *** Guest1857 is now known as redrobot | 16:17 | |
| *** cdent has joined #openstack-sdks | 16:22 | |
| openstackgerrit | Dean Troyer proposed openstack/python-openstackclient master: Support type=image with --block-device-mapping option https://review.opendev.org/673340 | 16:36 |
| dtroyer | mriedem: I've mucked about with your bdm series (release notes), and likely in the wrong order so a bit of rebasing may be necessary. This one ^^^ I wondered about including the vda bit in the release note, will leave it to your discretion. | 16:38 |
| *** ricolin_ has joined #openstack-sdks | 16:41 | |
| mriedem | dtroyer: replied | 16:42 |
| dtroyer | mriedem: thanks, agreed | 16:42 |
| *** ricolin has quit IRC | 16:44 | |
| openstackgerrit | Matt Riedemann proposed openstack/python-openstackclient master: Fix <id> description for --block-device-mapping https://review.opendev.org/674105 | 16:44 |
| openstackgerrit | Matt Riedemann proposed openstack/python-openstackclient master: Add openstack server create --boot-from-volume option https://review.opendev.org/674111 | 16:44 |
| mriedem | $ openstack server create --image $IMAGE_ID --block-device-mapping vda=$IMAGE_ID:image:1:true --flavor m1.tiny --wait test-vol-backed-vm-from-image The volume cannot be assigned the same device name as the root device vda (HTTP 400) (Request-ID: req-b4569b5f-4415-4911-9119-7afd856e2714) | 16:45 |
| mriedem | from my earlier testing | 16:45 |
| dtroyer | yup, that's a pretty clear error message | 16:45 |
| *** irclogbot_0 has quit IRC | 16:47 | |
| *** irclogbot_3 has joined #openstack-sdks | 16:49 | |
| *** irclogbot_3 has quit IRC | 16:49 | |
| *** dtantsur is now known as dtantsur|afk | 16:50 | |
| *** irclogbot_2 has joined #openstack-sdks | 16:50 | |
| *** jpich has quit IRC | 17:09 | |
| Shrews | mordred: Should we just delete the openstacksdk-functional-devstack-magnum test, or do you know a magnum expert who might want to fix it? It never passes. | 17:23 |
| mordred | Shrews: wow, and it fails "has service magnum" | 17:24 |
| mordred | Shrews: I'd say it's pretty useless currently | 17:24 |
| Shrews | mordred: yeah. i had a quick glance at it but couldn't figure out what, exactly, was wrong | 17:25 |
| openstackgerrit | Merged openstack/python-openstackclient master: Make configuration show not require auth https://review.opendev.org/673110 | 17:29 |
| efried | !SHAZAM! | 17:32 |
| openstack | efried: Error: "SHAZAM!" is not a valid command. | 17:32 |
| mordred | efried: it's so sad that !SHAZAM! isn't a valid command | 17:36 |
| efried | Maybe it doesn't work for everyone | 17:36 |
| efried | Because, you see, now I *look* like a superhero, but I'm still an immature kid on the inside. | 17:37 |
| Shrews | that movie disappointed me, particularly the ending | 17:39 |
| efried | I went in with expectations too high, because a friend told me he really liked it. | 17:41 |
| efried | mordred, Shrews: This seems... wrong: https://opendev.org/openstack/openstacksdk/src/branch/master/openstack/proxy.py#L129 | 17:42 |
| efried | The *args part | 17:42 |
| efried | I don't think there's a way you can call that such that `args` has anything in it. | 17:43 |
| efried | (slightly surprising that python even lets you define a param list that way, but I've seen it before) | 17:43 |
| mordred | efried: I *think* if you were to call it like Proxy(session, statsd_client, statsd_prefix, prometheus_counter, prometheus_historgram, foo) that foo would wind up in *args | 17:46 |
| efried | mordred: But then the super() call would blow up, nah? | 17:47 |
| mordred | yeah - totally. | 17:47 |
| mordred | we should get rid of that *args - I don't think we want people calling things with positional args anyway tbh | 17:47 |
| efried | I always ask people to name kwargs | 17:47 |
| efried | even in unit tests | 17:47 |
| mordred | ++ | 17:48 |
| efried | but it's not especially enforceable | 17:48 |
| mordred | in 3.6 it is ... foo(self, *, bar=None) | 17:48 |
| mordred | defines bar as a keyword only argument | 17:48 |
| mordred | so once we drop 2.7 - bob's our uncle | 17:48 |
| efried | neat. | 17:49 |
| efried | I had *two* uncle Bobs. | 17:49 |
| mordred | HA bob | 17:50 |
| efried | well, one of them failed, and hasn't been replaced, so... | 17:52 |
| mordred | :( | 17:53 |
| efried | As my first superhero act, I'm sending https://review.opendev.org/#/c/675135/ | 17:58 |
| *** factor has quit IRC | 18:06 | |
| *** icarusfactor has joined #openstack-sdks | 18:06 | |
| Shrews | mordred: Do you know if swift converts the casing of any object metadata we supply? Looks like we store in lowercase, but it comes back camel cased. This is messing with our dict.get() calls | 18:10 |
| *** cdent has quit IRC | 18:15 | |
| Shrews | hrm, seems so. wow, none of the object code dealing with metadata allows for this | 18:17 |
| Shrews | also, the autocreate header we put on the objects has a different header value??? | 18:26 |
| Shrews | HOW IS ANYTHING WORKING?!?!? | 18:26 |
| Shrews | :) | 18:26 |
| Shrews | returned header is: x-object-meta-x-object-meta-x-sdk-autocreated | 18:29 |
| Shrews | the header we save is: x-object-meta-x-sdk-autocreated | 18:29 |
| Shrews | an extra "x-object-meta" is prepended somewhere | 18:29 |
| Shrews | doesn't seem to happen on the checksum headers though | 18:30 |
| Shrews | can open... worms everywhere | 18:30 |
| mordred | Shrews: see - I look away for a few minutes and you discover that the entire world is broken | 18:32 |
| Shrews | mordred: stop letting me do things | 18:32 |
| mordred | Shrews: no- do more things! the worms - they're everywhere?! | 18:33 |
| Shrews | serves me right for trying to do a real-world test on my object cleanup code | 18:34 |
| Shrews | i guess this means we are also missing functional tests of all this code | 18:35 |
| efried | Shrews: Would you please have a look at https://review.opendev.org/#/c/674675/ when you get a chance? TIA | 18:36 |
| efried | mordred: I think your patch may get rid of the warnings, but IMO we still want that ^ yah? | 18:36 |
| *** efried is now known as efried_pto | 18:42 | |
| Shrews | efried_pto: yeah, i'm gonna tackle this swift problem first while i have it paged in | 18:47 |
| * Shrews has inferior context switching h/w | 18:48 | |
| mordred | efried_pto: yeah - might as well | 18:51 |
| *** ricolin_ has quit IRC | 19:02 | |
| Shrews | AHA! found the header bug | 19:23 |
| openstackgerrit | David Shrewsbury proposed openstack/openstacksdk master: Add option to lowercase object meta keys https://review.opendev.org/675682 | 19:31 |
| openstackgerrit | David Shrewsbury proposed openstack/openstacksdk master: bug: avoid unnecessary object meta prefix https://review.opendev.org/675683 | 19:31 |
| openstackgerrit | David Shrewsbury proposed openstack/openstacksdk master: bug: Consider case when comparing object values https://review.opendev.org/675684 | 19:31 |
| Shrews | mordred: enjoy all the things | 19:31 |
| Shrews | mordred: two of those probably need tests | 19:32 |
| timburke | Shrews, mordred fwiw, swift will title case header names going out, but those are case-insensitive per the rfc. i know on py2, something (mimetools? rfc822?) would lower-case them when reading the response, while py3 preserves the casing from the wire | 19:32 |
| timburke | header values should preserve whatever casing you sent with the PUT/POST | 19:32 |
| Shrews | timburke: well, looks like we get them back in camelcase, even though we put in lower | 19:33 |
| Shrews | i'm not sure where or why that's happening | 19:33 |
| Shrews | unless we camel case our debug output just to be weird? /me checks | 19:34 |
| openstackgerrit | Merged openstack/openstacksdk master: Skip most service_description for unknown services https://review.opendev.org/675135 | 19:35 |
| openstackgerrit | David Shrewsbury proposed openstack/openstacksdk master: Retry large object manifest upload https://review.opendev.org/675134 | 19:39 |
| mordred | Shrews: so ... | 19:39 |
| mordred | Shrews: the headers object on the requests.Response here: https://review.opendev.org/#/c/675682/1/openstack/cloud/_object_store.py ... should be of type CaseInsensitiveDict | 19:39 |
| mordred | Shrews: that's super weird that you're having to lowercase them (although I totally believe you) | 19:40 |
| *** e0ne has joined #openstack-sdks | 19:41 | |
| Shrews | mordred: oh? i not sure that i've even heard of CaseInsensitiveDict. If that's the case, my failed dict.get() calls were probably just the extra prefix thing | 19:42 |
| mordred | Shrews: it's an object from the requests library | 19:43 |
| Shrews | mordred: YOU'RE an object from the requests library | 19:43 |
| mordred | because http headers are theoretically case insensitive | 19:43 |
| mordred | Shrews: some days I certainlky feel that way | 19:43 |
| Shrews | mordred: ok. i'll get rid of the lowercase stuff then | 19:44 |
| *** odyssey4me has left #openstack-sdks | 19:45 | |
| openstackgerrit | David Shrewsbury proposed openstack/openstacksdk master: bug: avoid unnecessary object meta prefix https://review.opendev.org/675683 | 19:47 |
| openstackgerrit | David Shrewsbury proposed openstack/openstacksdk master: Retry large object manifest upload https://review.opendev.org/675134 | 19:47 |
| Shrews | mordred: ok, that should be what we need now | 19:48 |
| *** ralonsoh has quit IRC | 19:57 | |
| mordred | Shrews: that looks fantastic! | 20:04 |
| Shrews | mordred: oh, oops. found a nit to fix in that | 20:09 |
| mordred | doh | 20:09 |
| openstackgerrit | David Shrewsbury proposed openstack/openstacksdk master: Retry large object manifest upload https://review.opendev.org/675134 | 20:09 |
| Shrews | just minor cleanup i forgot to do | 20:10 |
| *** whoami-rajat has quit IRC | 20:12 | |
| *** e0ne has quit IRC | 20:18 | |
| *** slaweq has quit IRC | 21:10 | |
| *** e0ne has joined #openstack-sdks | 21:21 | |
| *** slaweq has joined #openstack-sdks | 21:26 | |
| *** slaweq has quit IRC | 21:32 | |
| *** e0ne has quit IRC | 21:32 | |
| *** e0ne has joined #openstack-sdks | 21:36 | |
| *** e0ne has quit IRC | 21:45 | |
| *** slaweq has joined #openstack-sdks | 21:51 | |
| *** slaweq has quit IRC | 21:56 | |
| *** mriedem has quit IRC | 22:17 | |
| openstackgerrit | Merged openstack/openstacksdk master: Allow limiting Connection service_types from oslo.config https://review.opendev.org/674675 | 23:30 |
| *** tosky has quit IRC | 23:32 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!