| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2019-02-28 | |||
| 11:10:19 | dtantsur | gtema: you can check how glances reacts to some non-sense parameter | |
| 11:10:28 | dtantsur | I guess older glance will react the same to is_hidden | |
| 11:11:03 | gtema | as I wrote - in my case it ignores it. Basically Glance allows millions of "non-standart" properties | |
| 11:11:10 | dtantsur | \o/ | |
| 11:11:15 | dtantsur | then the code is probably okay | |
| 11:11:19 | gtema | that is "doch" our problem | |
| 11:11:44 | gtema | ok, will change order for return | |
| 11:12:00 | dtantsur | no worries, this is just a nit | |
| 11:12:08 | gtema | oki | |
| 11:12:23 | openstackgerrit | Artem Goncharov proposed openstack/openstacksdk master: Tweak find_image method to search in hidden images https://review.openstack.org/639613 | |
| 11:28:06 | openstackgerrit | Artem Goncharov proposed openstack/openstacksdk master: Add DNS support https://review.openstack.org/623009 | |
| 12:11:07 | openstackgerrit | David Rabel proposed openstack/python-openstackclient master: Add --attached / --detached parameter to volume set https://review.openstack.org/638671 | |
| 13:28:27 | dtantsur | mordred: morning! will it be possible to release the SDK with https://review.openstack.org/#/c/639149/ soon(ish)? | |
| 13:34:01 | openstackgerrit | Merged openstack/openstacksdk master: Fix minor issues in the baremetal proxy docs https://review.openstack.org/638136 | |
| 13:34:49 | mordred | dtantsur: absolutely! | |
| 13:35:13 | dtantsur | mordred: cool! if it happens, I may still be able to use it in an ironic change :) | |
| 13:36:04 | mordred | Shrews_: when you awaken / are online (assuming you did not break any wrists while snowboarding this year) feel like reviewing https://review.openstack.org/#/c/609684/ and its children? | |
| 13:37:24 | mordred | dtantsur: also - wanna re-review https://review.openstack.org/#/c/639613/3 - gtema updated it | |
| 13:38:08 | dtantsur | mordred: approved | |
| 13:40:13 | mordred | \o/ | |
| 13:40:58 | mordred | I just +Ad 3 of his other patches that had been previous +A'd but rebased. when all of those land, I'l put in a release request | |
| 13:42:58 | Shrews_ | mordred: here now. i did not break anything (i think) but hospitals were involved | |
| 13:43:25 | Shrews_ | i can review though | |
| 13:43:50 | mordred | Shrews: have you considered NOT going to the hospital while snowboarding | |
| 13:44:11 | Shrews | mordred: i am officially giving up snowboarding instead | |
| 13:45:37 | mordred | Shrews: I have yet to get injured while scuba diving | |
| 14:16:50 | openstackgerrit | Artem Goncharov proposed openstack/openstacksdk master: Add DNS support https://review.openstack.org/623009 | |
| 14:47:59 | brtknr | I hey all | |
| 14:48:05 | brtknr | I have an urgent question | |
| 14:48:28 | brtknr | How do you access user_id and project_id of the active session from openstacksdk? | |
| 14:48:57 | brtknr | Surely this data is available in the current context right? | |
| 14:49:29 | brtknr | Or do I need to make another call to get those bits? | |
| 14:51:05 | brtknr | wait i found it | |
| 14:51:12 | brtknr | its .current_project.id | |
| 14:51:16 | brtknr | and .current_user_id | |
| 14:51:49 | brtknr | Thanks a billion ;) x | |
| 14:54:16 | openstackgerrit | Merged openstack/openstacksdk master: baremetal: support network_data when building configdrive https://review.openstack.org/639149 | |
| 15:06:47 | openstackgerrit | Merged openstack/openstacksdk master: Add image.task resource https://review.openstack.org/637913 | |
| 15:08:27 | dtantsur | merge conflicts \o/ | |
| 15:10:44 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Use retriable_status_codes in image upload https://review.openstack.org/639679 | |
| 15:10:44 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Move image methods to sdk image proxy https://review.openstack.org/609684 | |
| 15:10:45 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Add glance image import support https://review.openstack.org/639122 | |
| 15:10:55 | mordred | dtantsur: yeah. sigh ^^ fixed | |
| 15:11:10 | mordred | brtknr: happy we could help! :) | |
| 15:13:40 | dtantsur | :D | |
| 15:13:42 | dtantsur | mordred: reapproved | |
| 15:13:50 | mordred | thanks! | |
| 15:21:06 | edleafe | dtantsur: elmiko: I may be late to the office hour, so feel free to start the fun without me | |
| 15:28:58 | elmiko | edleafe: ack, i will be here for the duration =) | |
| 15:29:38 | mrhillsman | sorry for missing quite a few but will be here as well for office hours | |
| 15:29:47 | elmiko | ++ | |
| 15:30:03 | elmiko | mrhillsman: it's usually pretty quiet, but always happy to have more folks tuning in =) | |
| 15:34:08 | mrhillsman | well i started the thread on sdk validation hehe | |
| 15:34:35 | mrhillsman | had a few things at work that needed to be adjusted so good now to continue moving forward | |
| 15:38:13 | openstackgerrit | Artem Goncharov proposed openstack/openstacksdk master: Add image.service_info resources https://review.openstack.org/638227 | |
| 15:44:26 | gtema | hey elmiko, edleafe. I was recently chatting for mordred (http://eavesdrop.openstack.org/irclogs/%23openstack-sdks/%23openstack-sdks.2019-02-19.log.html#t2019-02-19T12:24:20) about my query in the maillist | |
| 15:44:58 | gtema | he suggested, that it might be possible to make a API-SIG doc to describe "good" way. What do you think about that? | |
| 15:46:54 | gtema | s/for/with/ | |
| 15:48:46 | openstackgerrit | Dean Troyer proposed openstack/python-openstackclient master: Add --attached / --detached parameter to volume set https://review.openstack.org/638671 | |
| 15:51:05 | elmiko | gtema: taking a look | |
| 15:53:26 | elmiko | gtema: an interesting topic, i definitely think recording that guidance /somewhere/ is a good thing. i am curious to hear where it might fit in the current api-sig guidance, i don't think it's a topic that i am deeply familiar with | |
| 15:54:10 | gtema | elmiko: understand | |
| 15:56:01 | elmiko | that said, i am definitely open to the idea of including this information into the guidance, if we can find an appropriate location | |
| 15:56:52 | elmiko | gtema: like, maybe in here somewhere? http://specs.openstack.org/openstack/api-sig/guidelines/consuming-catalog.html | |
| 15:58:35 | elmiko | API SIG office hours have begun! (a little early too) | |
| 15:59:13 | gtema | elmiko: might be an option | |
| 15:59:48 | elmiko | gtema: totally open to suggestions too | |
| 15:59:53 | gtema | since it is under api-sig - for sure. On the other hand it is more related to configuring catalog and not consuming | |
| 16:00:11 | elmiko | right | |
| 16:00:23 | elmiko | that's why i'm a little stumped about where we should put it | |
| 16:00:42 | elmiko | should this be part of the keystone docs? | |
| 16:01:06 | gtema | well... | |
| 16:01:11 | gtema | might be | |
| 16:01:59 | mrhillsman | proposal: forum session on sdk validation discussion | |
| 16:02:32 | gtema | however if "consuming" page guide describes how to treat uncertancies or strict requirements - is also good | |
| 16:04:52 | openstackgerrit | Artem Goncharov proposed openstack/openstacksdk master: Add DNS support https://review.openstack.org/623009 | |
| 16:04:58 | gtema | mrhillsman: unfortunately not going to the summit | |
| 16:04:58 | mrhillsman | apologies for not being around, elmiko you volunteered to help lead sdk validation stuff right? | |
| 16:05:03 | elmiko | gtema: ack, i'm good with that | |
| 16:05:16 | mrhillsman | or was that gtema | |
| 16:05:23 | gtema | yupp | |
| 16:05:30 | elmiko | mrhillsman: i'm definitely willing to help as i can, i won't be at the forum though | |
| 16:05:36 | gtema | but was also terribly busy | |
| 16:05:38 | mrhillsman | ok cool, no worries, i will be there | |
| 16:06:39 | gtema | elmiko: shoul I the suggest a change to "consuming catalog" guide to adress the issue? | |
| 16:08:53 | gtema | hmm, current state doesn't disallow bad region-name (refering to current keystone) | |
| 16:09:10 | openstackgerrit | Dean Troyer proposed openstack/osc-lib master: Add release note for --os-user-id fix https://review.openstack.org/640082 | |
| 16:09:28 | gtema | but it is exactly a problem for me with current keystone, so we might "re-phrase" requirements here | |
| 16:38:14 | elmiko | doh, gtema disappeared | |
| 16:39:04 | elmiko | gtema: if you read back later, i think making a PR to the guide is definitely welcome and a good place to further this discussion | |
| 17:23:33 | openstackgerrit | Merged openstack/osc-lib master: Add release note for --os-user-id fix https://review.openstack.org/640082 | |
| 17:43:44 | dtroyer | mordred: I'm going to release osc-lib 1.12.1 at that point ^^^ | |
| 17:44:04 | dtroyer | there's only 1 bug fix for this cycle, this should be it | |
| 17:45:34 | mordred | dtroyer: sounds great! | |
| 17:49:30 | kmalloc | mordred: lets revisit landing the fair semaphore and just lean on SDK consuming it as the functional test. | |
| 17:49:42 | kmalloc | mordred: looking into why the SDK consuming patch is unhappy/failing now. | |
| 17:49:47 | kmalloc | (had to recheck) | |
| 17:50:01 | mordred | kmalloc: ++ I think that's likely a good idea | |
| 17:50:21 | mordred | kmalloc: it's possible that the consuming patch isn't properly actually installing the right versions of things | |
| 17:50:26 | kmalloc | yeah that is my guess. | |