Earlier  
Later
New Search
Posted Nick Remark
#openstack-sdks - 2018-07-03
20:28:53 openstackgerrit David Shrewsbury proposed openstack-infra/shade master: Fix for passing dict for get_* methods https://review.openstack.org/579974
20:29:12 Shrews that probably could use a test
20:29:43 mordred yah. also - I'm pretty sure pep8 is going to complain about the whitespace
20:32:06 openstackgerrit David Shrewsbury proposed openstack-infra/shade master: Fix for passing dict for get_* methods https://review.openstack.org/579974
20:32:18 Shrews pep8 passed for me
20:33:56 mordred Shrews: cool
20:35:31 Shrews mordred: you want a release note for that?
20:38:51 wacuuu yyy question: are there any plans for sdk in go?
20:42:17 mordred wacuuu: the gophercloud project is basically that
20:42:22 mordred Shrews: nah
20:42:38 mordred wacuuu: https://github.com/gophercloud/gophercloud
20:47:37 Shrews oh that's interesting
20:47:55 Shrews mordred: my fav part: "Backwards-Compatibility Guarantees - None. Vendor it and write tests covering the parts you use."
20:49:22 wacuuu thanks
#openstack-sdks - 2018-07-04
02:09:42 openstackgerrit Fan Zhang proposed openstack/python-openstackclient master: Add --key-name and --key-unset option for server rebuild API. https://review.openstack.org/576422
03:00:32 openstackgerrit tianhui proposed openstack/python-openstackclient master: Compute:Support keypair set key-type https://review.openstack.org/577768
03:03:00 openstackgerrit tianhui proposed openstack/python-openstackclient master: Compute:Support keypair set key-type https://review.openstack.org/577768
07:57:49 openstackgerrit Josephine Seifert proposed openstack/openstacksdk master: Implement signature generation functionality https://review.openstack.org/580080
08:21:48 openstackgerrit Josephine Seifert proposed openstack/python-openstackclient master: [WIP] osc-included image signing (using openstacksdk) https://review.openstack.org/580086
10:21:45 openstackgerrit Mohammed Naser proposed openstack/python-openstackclient master: Allow setting volume attach status https://review.openstack.org/574719
11:17:06 openstackgerrit Monty Taylor proposed openstack/osc-lib master: Use openstack.config for config and Session objects https://review.openstack.org/579139
12:05:06 dtantsur mordred: got it. this munch vs resource thing does give me a headache..
12:05:32 dtantsur also I realized that we probably need this redundant GET now: the microversion to do an action may depend on the current state
12:07:45 dtantsur mordred: also, how does self.assert_calls() order calls? it insist PUT is run the first, while I'm pretty sure GET happens ealier..
12:13:16 dtantsur never mind me. I might have been wrong from the beginning about this redundant GET
12:42:29 mordred dtantsur: yay, glad I could help
12:42:51 mordred dtantsur: (and yes, I will be VERY happy once we've rationalized Resource and shade/munch)
13:07:02 openstackgerrit Dmitry Tantsur proposed openstack/openstacksdk master: [WIP] set_provision_state and wait_for_provision_state for baremetal Node https://review.openstack.org/570895
13:07:07 dtantsur okay, this could use more unit tests, but is reviewable ^^^
13:09:21 dtantsur TheJulia: if you have a minute ^^^
13:14:55 openstackgerrit Dmitry Tantsur proposed openstack/openstacksdk master: [WIP] set_provision_state and wait_for_provision_state for baremetal Node https://review.openstack.org/570895
#openstack-sdks - 2018-07-05
09:37:26 frickler mordred: I'm still affected by the network SSL issue, maybe you or someone else can take a look and check whether they can at least reproduce to confirm I'm not doing something dumb https://storyboard.openstack.org/#!/story/1756723
13:30:47 openstackgerrit Monty Taylor proposed openstack/osc-lib master: Update sphinx lower-constraint to match test-requirements https://review.openstack.org/580380
13:30:47 openstackgerrit Monty Taylor proposed openstack/osc-lib master: Use openstack.config for config and Session objects https://review.openstack.org/579139
14:00:47 mordred frickler: I'll look in to it - there is still a decent amount of unwinding we need to do
15:33:40 mordred dtroyer: woot! https://review.openstack.org/#/c/579139 passed tests
15:51:30 dtroyer \o/
16:26:04 mordred mriedem: does https://review.openstack.org/#/c/578960/ look ok to you now?
16:29:54 mriedem looking
16:30:38 mriedem hells yeah!
16:33:02 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Add task manager parameter to Connection https://review.openstack.org/577616
16:38:57 mordred mriedem: woot!
16:56:19 cdent dtantsur, edleafe, elmiko : a bit boring but https://etherpad.openstack.org/p/api-sig-newsletter
16:56:28 cdent also, I forgot to mention: I won't be at next week's meeting
16:58:02 dtantsur oh, nor will I
16:58:39 cdent jroll: thanks for that "that's the job of the server" response on web server headers
16:58:45 jroll :)
17:02:17 elmiko cdent dtantsur ack, might just be me and edleafe then
17:03:04 elmiko cdent: lgtm
17:03:16 edleafe cdent: ship it
17:03:19 cdent i'll wait on edleafe a bit longer, as he....
17:03:20 cdent omg
17:03:29 elmiko haha
17:22:46 openstackgerrit Merged openstack-infra/shade master: Fix for passing dict for get_* methods https://review.openstack.org/579974
18:36:27 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Import rate limiting TaskManager from nodepool https://review.openstack.org/574285
18:55:00 openstackgerrit Merged openstack/openstacksdk master: Add connection backreference to proxy instances https://review.openstack.org/579167
19:06:46 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Add task manager parameter to Connection https://review.openstack.org/577616
19:11:54 openstackgerrit Merged openstack/openstacksdk master: Only send force parameter to live migration if supported https://review.openstack.org/578960
19:15:42 openstackgerrit Monty Taylor proposed openstack-infra/shade master: Make OpenStackCloud a subclass of Connection https://review.openstack.org/577647
19:29:14 mordred wacuuu: great patch - I just clicked the approve button - but also left you some comments on things to know for next time
19:33:51 openstackgerrit Monty Taylor proposed openstack-infra/shade master: Make OpenStackCloud a subclass of Connection https://review.openstack.org/577647
19:33:52 openstackgerrit Monty Taylor proposed openstack-infra/shade master: Use the openstacksdk manager constructor argument https://review.openstack.org/580475
19:36:08 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Fix for passing dict for get_* methods https://review.openstack.org/580477
19:36:15 openstackgerrit Monty Taylor proposed openstack-infra/shade master: Make OpenStackCloud a subclass of Connection https://review.openstack.org/577647
19:36:16 openstackgerrit Monty Taylor proposed openstack-infra/shade master: Use the openstacksdk manager constructor argument https://review.openstack.org/580475
19:36:44 mordred Shrews: cherry-picked your shade patch to sdk
19:53:54 wacuuu mordred: anything else on my side to do? i'm used to magic submit button on my work gerrit that i don't see here
19:55:39 mordred wacuuu: nope - it'll merge once it passes the tests
19:56:12 mordred wacuuu: Zuul presses the submit button for us - once we leave a "Workflow +1" vote, it runs tests again and then merges
19:57:09 wacuuu ok, good to know. thanks alot for help with this
19:58:32 mordred wacuuu: sure thing! happy to have you contributing
20:28:47 openstackgerrit Merged openstack-infra/shade master: Remove shade-ansible-devel job https://review.openstack.org/570682
20:50:46 openstackgerrit Merged openstack/openstacksdk master: Implementing solution for 2002563 issue from story board https://review.openstack.org/579679
21:06:39 openstackgerrit Merged openstack/osc-lib master: Update sphinx lower-constraint to match test-requirements https://review.openstack.org/580380
22:51:48 openstackgerrit Merged openstack/openstacksdk master: Fix for passing dict for get_* methods https://review.openstack.org/580477
#openstack-sdks - 2018-07-06
00:05:15 openstackgerrit Merged openstack/osc-lib master: Adding api_version to FakeAPP https://review.openstack.org/579523
08:25:21 openstackgerrit tianhui proposed openstack/python-openstackclient master: Compute: Add tag support for server add volume https://review.openstack.org/580579
09:34:51 openstackgerrit Dmitry Tantsur proposed openstack/openstacksdk master: [WIP] set_provision_state and wait_for_provision_state for baremetal Node https://review.openstack.org/570895
10:25:08 openstackgerrit Jens Harbott (frickler) proposed openstack/python-openstackclient master: Retry floating IP tests https://review.openstack.org/578961
10:26:16 frickler dtroyer: ^^ amended according to my comments. this is currently causing me to do a lot of rechecks for other patches, so would be great to get in soon, assuming you approve my modifications
12:36:38 dtroyer frickler: thank you, +W
13:47:14 openstackgerrit Ryan Brady proposed openstack-infra/shade master: Add support for static routes https://review.openstack.org/580661
14:04:38 openstackgerrit Merged openstack/python-openstackclient master: Retry floating IP tests https://review.openstack.org/578961
15:05:03 dtantsur mordred: hi! we have to decide on the critical question of a title for our talk :) I'm out next week, so we should probably submit it today, if you have time
15:32:30 openstackgerrit Merged openstack/osc-lib master: Use openstack.config for config and Session objects https://review.openstack.org/579139
15:46:43 openstackgerrit Chris Dent proposed openstack/api-wg master: Expand schema for error.codes to reflect reality https://review.openstack.org/580703
18:16:20 openstackgerrit Dean Troyer proposed openstack/python-openstackclient master: Slow down and retry aggregate create/delete to lessen race https://review.openstack.org/580732
20:25:06 dtroyer mordred: I think https://review.openstack.org/#/c/579139 broke OSC's —insecure option, all of the shell integration tests with —insecure are failing since that merged
#openstack-sdks - 2018-07-07
12:35:56 mordred dtroyer: piddle. I'll look and see what I can figure out
12:36:01 mordred how did we miss it on the osc patch?
12:36:10 mordred osc-lib that is
12:36:46 mordred ah - osc unittests
12:36:52 mordred awesome- that's at least easy to replicate
13:17:42 mordred dtroyer: ok. I found the issue - the overriding of the config options happens in the osc-lib ClientManager constructor, but then in osc prepare_to_run_command we make a new CloudRegion
13:18:17 mordred dtroyer: there is, happily, an easy solution - which is just to add support to openstack.config for the insecure option in the first place
13:35:08 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Add support for processing insecure https://review.openstack.org/580795
13:45:22 openstackgerrit Monty Taylor proposed openstack/osc-lib master: Stop processing requests verify args https://review.openstack.org/580796
13:46:12 mordred dtroyer: the openstacksdk patch fixes the osc unittests. I've got a release request for sdk up anyway, but I'll put a WIP on it and we can land that and get it in the release
13:53:03 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Add support for processing insecure https://review.openstack.org/580795
13:53:19 mordred dtroyer: the osc-lib patch is not strictly required - but while we're thinking about it, it seems like a good thing to land once the sdk patch is out

Earlier   Later