Earlier  
Later
New Search
Posted Nick Remark
#openstack-sdks - 2019-02-27
13:28:50 openstackgerrit Merged openstack/openstacksdk master: Add agent property: resources-synced https://review.openstack.org/635143
14:31:53 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Add glance image import support https://review.openstack.org/639122
14:31:53 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Move image methods to sdk image proxy https://review.openstack.org/609684
14:31:56 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Use retriable_status_codes in image upload https://review.openstack.org/639679
14:32:33 mordred dtantsur|afk: 639679 is a great idea - thanks - but it's also not working with the mocks on one test
14:33:01 mordred cmurphy, kmalloc: ^^ pinging you as well, as it might be a keystoneauth issue - but it might just be an issue with the test mocks
14:33:33 gtema mordred: wow, lot to review to guys ;-)
14:34:01 mordred gtema: :)
14:34:16 gtema one choco would be not enough
14:36:12 gtema I would like us to complete image, so that it doesn't look like a mess to add missing bits to image (for OSC)
14:41:10 cmurphy mordred: o7
14:42:14 mordred gtema: ++
14:51:10 kmalloc Hmmm
15:21:01 kmalloc weird.
15:35:41 kmalloc mordred: uhm, do we need to explicity set a status_code_retries for 639679?
15:37:14 openstackgerrit Rajat Dhasmana proposed openstack/python-openstackclient master: Fix: Restore output 'VolumeBackupsRestore' object is not iterable https://review.openstack.org/624860
15:38:30 openstackgerrit Rajat Dhasmana proposed openstack/python-openstackclient master: Fix: Restore output 'VolumeBackupsRestore' object is not iterable https://review.openstack.org/624860
16:03:26 kmalloc mordred: i don't think it's a KSA side, but it totally could be something subtle
16:04:16 holser|lunch good morning
16:05:16 holser_ mordred could you have a look at https://review.openstack.org/#/c/637169/ I have addressed your concerns
16:06:12 holser_ thanks
16:43:41 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Use retriable_status_codes in image upload https://review.openstack.org/639679
16:43:42 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Add glance image import support https://review.openstack.org/639122
16:43:57 mordred kmalloc: found it - turns out you have to set status_code_retries :)
16:44:26 kmalloc :P
16:46:54 openstackgerrit Rodolfo Alonso Hernandez proposed openstack/openstacksdk master: Add "name" filter in "list" call when retrieving a single register https://review.openstack.org/637238
16:52:49 openstackgerrit Artem Goncharov proposed openstack/openstacksdk master: Add DNS support https://review.openstack.org/623009
17:05:56 openstackgerrit Artem Goncharov proposed openstack/openstacksdk master: Tweak find_image method to search in hidden images https://review.openstack.org/639613
17:10:19 openstackgerrit Artem Goncharov proposed openstack/openstacksdk master: Add DNS support https://review.openstack.org/623009
17:16:41 kmalloc mordred: I'm glad it was a "whoopse missed that" instead of a ksa bug.
17:20:20 mordred kmalloc: me too. it would have been a pretty bad bug if it had been a bug :)
17:44:48 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: WIP Add support for generating form-post signatures https://review.openstack.org/639760
18:13:49 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: WIP Add support for generating form-post signatures https://review.openstack.org/639760
18:33:00 cdent Is it the case that openstackclient is not always happy with python 3.5?
18:42:08 cdent ah appears to be a problem in the openstack module itself
18:45:24 dtroyer cdent: I am not aware of such an issue
18:47:16 cdent dtroyer: it seems to be only 3.5, and it looks like it might actually be coming from openstack.exception (as that's one of the few places I can find that is using json.loads directory and it is only when interpreting error responses)
18:47:58 cdent http://logs.openstack.org/17/639717/2/check/osc-placement-dsvm-functional-py3/fefe661/logs/devstack-gate-post_test_hook.txt.gz#_2019-02-27_17_20_52_120
18:48:23 cdent unfortunately I don't have a working python3.5 env to hand, so I'm casting about a bit
18:49:33 cdent and json.loads definitely changed between 3.5 and 3.6
18:52:09 cdent ayup, that's it
18:54:56 cdent ugh, but from where...
18:55:56 cdent got it, the problem is localized to osc-placement, that's good
19:23:39 openstackgerrit Merged openstack/keystoneauth master: Expose app creds and new attrs in fixtures https://review.openstack.org/636030
20:41:57 openstackgerrit Merged openstack/python-openstackclient master: Disabling c-backup service for osc-functional-devstack-tips job https://review.openstack.org/639410
20:49:41 openstackgerrit Sean McGinnis proposed openstack/python-openstackclient master: Fix: Restore output 'VolumeBackupsRestore' object is not iterable https://review.openstack.org/624860
21:56:52 openstackgerrit Merged openstack/python-openstackclient master: add python 3.7 unit test job https://review.openstack.org/637785
21:56:56 openstackgerrit Merged openstack/python-openstackclient master: Fix help message of image add project https://review.openstack.org/638113
22:15:42 openstackgerrit Merged openstack/python-openstackclient master: Remove str() when setting network objects names https://review.openstack.org/633508
22:15:47 openstackgerrit Merged openstack/python-openstackclient master: This fix removes an erroneous underscore found within the function named test_snapshot_delete within test_snapshot.py found in both volume v1 and v2 of python-openstackclient. https://review.openstack.org/636186
22:15:48 openstackgerrit Merged openstack/python-openstackclient master: Partially Revert "Add command to unset information from Subnet-pools" https://review.openstack.org/599633
22:22:14 openstackgerrit Merged openstack/osc-lib master: add python 3.7 unit test job https://review.openstack.org/637743
22:28:11 openstackgerrit Sean McGinnis proposed openstack/python-openstackclient master: Fix: Restore output 'VolumeBackupsRestore' object is not iterable https://review.openstack.org/624860
#openstack-sdks - 2019-02-28
01:38:03 openstackgerrit Kailun Qin proposed openstack/python-openstackclient master: Add network segment range command object https://review.openstack.org/625544
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

Earlier   Later