| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2019-02-28 | |||
| 05:35:27 | openstackgerrit | Vishakha Agarwal proposed openstack/keystoneauth master: Drop py35 jobs https://review.openstack.org/639915 | |
| 09:34:12 | openstackgerrit | Artem Goncharov proposed openstack/openstacksdk master: Add DNS support https://review.openstack.org/623009 | |
| 10:43:45 | gtema | dtantsur: I wrote in the change. From image api docs: | |
| 10:43:49 | gtema | dtantsur: When true, filters the response to display only “hidden” images. By default, “hidden” images are not included in the image-list response. (Since Image API v2.7) | |
| 10:44:26 | gtema | I faced this implementing this piece in OSC in functional tests | |
| 10:45:43 | dtantsur | gtema: oh. and there's no way to fetch both? | |
| 10:45:54 | gtema | nope, no known to me | |
| 10:46:08 | dtantsur | that's why we can't have nice things.. okay, will re-review | |
| 10:46:14 | gtema | thanks | |
| 10:47:31 | dtantsur | gtema: left a question inline | |
| 10:47:51 | gtema | thanks | |
| 10:53:55 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Use retriable_status_codes in image upload https://review.openstack.org/639679 | |
| 10:54:33 | mordred | dtantsur: ^^ fixed the pep8 | |
| 10:54:47 | dtantsur | w00t | |
| 10:58:18 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Add glance image import support https://review.openstack.org/639122 | |
| 10:58:41 | gtema | dtantsur: do you have possibility to check, whether this list with os_hidden and glance<2.7 returns any error. In my case (unfortunately not very representative cloud) - I get no exception, simply empty list | |
| 11:08:21 | dtantsur | gtema: I'm not sure I have access to older glance.. | |
| 11:09:22 | gtema | dtantsur: sad, then I am not able to ensure, that Glance return error, so your last comment might stay unfixed | |
| 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: Move image methods to sdk image proxy https://review.openstack.org/609684 | |
| 15:10:44 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Use retriable_status_codes in image upload https://review.openstack.org/639679 | |
| 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 | mrhillsman | apologies for not being around, elmiko you volunteered to help lead sdk validation stuff right? | |
| 16:04:58 | gtema | mrhillsman: unfortunately not going to the summit | |
| 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 | |