Earlier  
Later
New Search
Posted Nick Remark
#openstack-sdks - 2018-11-28
15:44:50 dtantsur|mtg mordred: do you think you could release the SDK with the just merged patch? I'd give the metalsmith migration another try.
15:45:04 mordred dtantsur|mtg: sure thing
15:46:26 mordred dtantsur|mtg: remote: https://review.openstack.org/620639 Release 0.20.0 of openstacksdk
15:47:32 dtantsur|mtg thnx!
15:47:47 mordred larsks: ^^ release request I think interests you too
16:05:06 openstackgerrit Monty Taylor proposed openstack/keystoneauth master: Add support for client-side rate limiting https://review.openstack.org/605043
16:07:28 openstackgerrit Monty Taylor proposed openstack/keystoneauth master: Add support for client-side rate limiting https://review.openstack.org/605043
16:38:28 openstackgerrit Merged openstack/osc-lib master: Remove pypy job https://review.openstack.org/617336
16:46:11 larsks mordred: ooo, nifty. Thanks!
17:39:54 openstackgerrit Merged openstack/openstacksdk master: Update link address for vendor support https://review.openstack.org/619830
18:01:24 nicolasochem__ Hello, I would like to use the sdk to validate a heat stack update. It's possible with the CLI (--dry-run command).
18:01:32 nicolasochem__ This is now the CLI does it : https://github.com/openstack/python-heatclient/blob/master/heatclient/v1/stacks.py#L185-L189
18:02:08 nicolasochem__ But I don't see this call anywhere in the openstacksdk source code. Am I missing anything, or is it missing from sdk altogether ?
18:02:38 nicolasochem__ Note that you can do a dry-run of stack creation with the sdk. My question is about stack update.
18:21:06 mordred nicolasochem__: doesn't look like it's in the sdk - but we should definitely add it
18:51:00 nicolasochem__ mordred: thank you. I have opened a ticket and I am happy to QA it if needed. https://storyboard.openstack.org/#!/story/2004450
18:58:39 mordred awesome! thanks
#openstack-sdks - 2018-11-29
07:54:08 openstackgerrit Merged openstack/python-openstackclient master: Updated the take_actions for unified limits https://review.openstack.org/612283
09:19:01 openstackgerrit Merged openstack/openstacksdk master: Add missing seperator between words https://review.openstack.org/618704
09:25:17 openstackgerrit Merged openstack/openstacksdk master: implement block-storage backup resource https://review.openstack.org/591445
14:33:08 openstackgerrit Radoslaw Smigielski proposed openstack/python-openstackclient master: Fix --limit option in image list sub-command https://review.openstack.org/617139
14:38:36 gtema oh god zuul is slow today - one change recheck is already since 5 hours open and one task still queued
15:57:17 edleafe API-SIG Office Hour begins at the top of the hour. I will be here, but also in another meeting, so don't everyone ask your questions at once
15:58:00 elmiko i'll be here too
15:58:04 elmiko but ask edleafe first XD
16:02:06 cdent the lack of formality with you two is why I had to stop coming to these things
16:02:08 cdent :D
16:03:14 elmiko fair
16:58:43 elmiko catch ya next week edleafe, if not sooner o/
16:59:07 edleafe elmiko: Leaving a minute early? Where's your dedication, man??
16:59:25 elmiko haha
18:55:11 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: DNM Strawman stats abstraction https://review.openstack.org/620990
19:05:34 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: DNM Strawman stats abstraction https://review.openstack.org/620990
19:31:47 openstackgerrit Merged openstack/openstacksdk master: Add CRUD methods for Neutron Port Forwarding https://review.openstack.org/616710
20:25:08 mriedem just wanted to come by and say https://docs.openstack.org/keystoneauth/latest/using-sessions.html is nice
20:25:09 mriedem thanks
20:28:32 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: DNM Strawman stats abstraction https://review.openstack.org/620990
20:32:30 mordred mriedem: \o/
21:13:25 openstackgerrit Merged openstack/openstacksdk master: Check result of server metadata operation https://review.openstack.org/619478
#openstack-sdks - 2018-11-30
06:52:08 openstackgerrit Rabi Mishra proposed openstack/osc-lib master: Fix formatter handling for python 3.7 https://review.openstack.org/618137
07:13:53 openstackgerrit Filippo Inzaghi proposed openstack/python-openstackclient master: Add Python 3.6 classifier to setup.cfg https://review.openstack.org/621096
07:36:08 openstackgerrit Radoslaw Smigielski proposed openstack/python-openstackclient master: Fix --limit option in image list sub-command https://review.openstack.org/617139
08:26:15 openstackgerrit Artem Goncharov proposed openstack/openstacksdk master: Refactor tag support https://review.openstack.org/618993
08:27:48 openstackgerrit Artem Goncharov proposed openstack/openstacksdk master: Refactor tag support https://review.openstack.org/618993
09:23:03 gtema dtantsur|msg, are you here?
09:24:34 dtantsur|mtg gtema: I am
09:25:09 gtema dtantsur|msg: as a continue to removal of xxxDetails I have found similar issue in orchestrate
09:25:19 gtema but the difference is that it is not list, but create/update
09:25:30 dtantsur|mtg I think all Details are broken more or less
09:25:36 gtema basically StackPreview
09:26:14 gtema I feel it would be good to apply the same approach as for Details here, but this would mean extension for also other request types
09:26:30 gtema and extending basic proxy/resource/test_proxy
09:26:43 gtema this is however becoming relatively big
09:26:55 gtema I mean everywhere adding optional base_path
09:27:08 gtema wanted to ask, what is your opinion here?
09:28:04 dtantsur|mtg I need to think about it, I'm quite sleepy right now :) and I'm not familiar with orchestration enough
09:28:54 gtema it's not about orchestration itself. I am asking more about adding base_path optional parameter not only for "list", what you did for Details, but also for other request types
09:30:00 gtema use case: upon some parameter to proxy (i.e. preview) the endpoint for "create" is different
09:44:28 dtantsur|mtg gtema: this can be done in _prepare_request (used by everything except for list)
09:45:12 gtema sure, this is what I have done. But you would first need to also extend Proxy._create (and _update) and pass this optional parameter into Resource
09:45:46 gtema and this mean that test_proxy_base should be also extended with additional check of base_path for such calls
09:46:35 gtema that's what I ment "is getting big"
09:47:29 gtema and wanted to check before I land a patch not to face resistence
09:47:54 gtema s/resistence/resistance/
09:58:21 dtantsur|mtg the approach seems good to me at first glance, but I really need to see the code
09:58:37 gtema ok, than I will work further
09:58:46 gtema thanks
13:02:50 openstackgerrit Artem Goncharov proposed openstack/openstacksdk master: Add possibility to override base_path for resource operations https://review.openstack.org/621153
13:04:43 openstackgerrit Artem Goncharov proposed openstack/openstacksdk master: Rework orchestration to add update preview https://review.openstack.org/621154
14:11:36 openstackgerrit Artem Goncharov proposed openstack/openstacksdk master: Add possibility to override base_path for resource operations https://review.openstack.org/621153
14:46:49 openstackgerrit Artem Goncharov proposed openstack/openstacksdk master: Rework orchestration to add update preview https://review.openstack.org/621154
16:39:11 openstackgerrit Artem Goncharov proposed openstack/openstacksdk master: Rework orchestration to add update preview https://review.openstack.org/621154
16:41:06 openstackgerrit Artem Goncharov proposed openstack/openstacksdk master: Add possibility to override base_path for resource operations https://review.openstack.org/621153
16:48:43 openstackgerrit Artem Goncharov proposed openstack/openstacksdk master: Rework orchestration to add update preview https://review.openstack.org/621154
16:57:41 openstackgerrit Artem Goncharov proposed openstack/openstacksdk master: Rework orchestration to add update preview https://review.openstack.org/621154
18:40:11 openstackgerrit Monty Taylor proposed openstack/keystoneauth master: WIP Fix version discovery for rackspace public cloud https://review.openstack.org/621257
18:49:27 kmalloc mordred: ^ that makes me sad.
18:49:44 Shrews kmalloc: yah, he's working on a better thilng
18:49:51 kmalloc i figured
18:49:53 kmalloc but man.
18:50:00 kmalloc gross.
18:50:54 kmalloc cmurphy: ^ cc
18:58:50 openstackgerrit Monty Taylor proposed openstack/keystoneauth master: Fix version discovery for clouds with int project_ids https://review.openstack.org/621257
19:09:35 openstackgerrit Monty Taylor proposed openstack/keystoneauth master: Fix version discovery for clouds with int project_ids https://review.openstack.org/621257
19:35:36 openstackgerrit Nathan Kinder proposed openstack/osc-lib master: Default to using public interface for all clients https://review.openstack.org/621276
20:00:23 mordred kmalloc: ^^ care to weigh in on that one?
20:01:32 kmalloc the most recent one?
20:01:50 kmalloc or the nkinder change?
20:02:11 kmalloc cause YES PUBLIC INTERFACES
20:03:05 kmalloc mordred: +1 (since i don't have +2 on that repo)
20:03:11 kmalloc but i'd +2 that if i could
20:18:38 mordred kmalloc: sweet. perhaps dhellmann or dtroyer will do the +3
20:21:24 dhellmann lgtm
21:14:40 larsks mordred: https://review.openstack.org/#/c/618519/ has two +2s and +verified, but it hasn't merged yet. Is something stuck?
21:15:04 mordred larsks: it's marked cannot merge
21:15:09 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: syntax errors and undefined exceptions in service_description.py https://review.openstack.org/618519
21:15:20 mordred larsks: I rebased it - let's see if that fixes the issue
21:16:09 larsks mordred: I must not have been looking at the right spot. Thanks!
21:16:31 mordred larsks: fingers crossed gerrit likes us this time
21:40:48 openstackgerrit Monty Taylor proposed openstack/promstat master: Add initial code https://review.openstack.org/621295

Earlier   Later