Earlier  
Later
New Search
Posted Nick Remark
#openstack-sdks - 2019-03-29
15:57:36 mordred very much so
16:07:45 brtknr mordred: please ship the updated version... just thought that the check should only be made if there a tag specified at all
16:11:57 brtknr gtema: to be fair, only 50 of 1664 open PRs are openstack related :P
16:12:42 gtema yes, but they are very long pending
16:12:57 gtema some of them are even mine
16:18:17 brtknr gtema: yes I was just looking at os_server_tags.. which another person also appears to have implemented... i wonder how many are dupes
16:18:36 gtema that's why we need to clean things up
16:18:56 brtknr gtema: what happens after shipit tag is applied?
16:18:58 gtema I would need to reimplement os_server_tags to support currently implemented SDK bindings
16:19:07 gtema there should be 2 shipits
16:19:20 brtknr gtema: oh okay
16:19:26 gtema and then it is automatically merged. But only special guys can apply shipit
16:20:29 brtknr gtema: gtema ok cool
17:10:08 larsks Whinge of the day: I wish that the output of the get_* functions in openstacksdk could be used as input to the set_* functions. I'm trying to migrate quotas between two environments, and I wish I could simply new.set_compute_quotas(old.get_compute_quotas()), but of course that doesn't work.
17:26:12 dtroyer larsks: ++ I've wanted to do that in some cases with OSC, the REST APIs themselves do not always work that way
17:42:36 dtruong mordred Can we get https://review.openstack.org/#/c/648334/ merged and then get a new openstacksdk release?
19:18:39 openstackgerrit Jeremy Houser proposed openstack/python-openstackclient master: Moved skiptest to setUpClass in order to avoid failures in test_snapshot and test_volume_transfer due to presence of openstack commands within setUpClass https://review.openstack.org/648760
21:19:16 mordred dtruong: I was sort of hoping to get some sort of feedback on that plan from the release team - have you heard anything further?
21:21:29 mordred larsks: that's the goal. it will actually work with the resource layer objects - but we've got some unwinding to do and we haven't ported quota support down into the resource layer
21:21:55 mordred larsks: which is to say, I agree with you and I am sorry it's not there yet
21:26:06 dtruong no, i haven't heard anything
21:27:08 mordred hrm. well - yes - in theory I'm fine moving forward with that plan. let's go poke people again real quick
21:29:58 dtruong ok, let me ping them in the release IRC channel
#openstack-sdks - 2019-03-30
00:21:05 larsks mordred: eh, no worries :). Thanks for taking the time to respond to my whinging.
10:24:46 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Add support for generating form-post signatures https://review.openstack.org/639760
10:24:47 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Move set_temp_url_key logic into resource objects https://review.openstack.org/648347
10:24:47 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Remove and rearrange object bonged proxy tests https://review.openstack.org/648352
#openstack-sdks - 2019-04-01
08:31:06 openstackgerrit Rico Lin proposed openstack/keystoneauth master: Update auth plugin name list in document https://review.openstack.org/648920
09:23:34 openstackgerrit Cao Xuan Hoang proposed openstack/openstacksdk master: Expose locked status for Server https://review.openstack.org/648931
09:58:36 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Add support for generating form-post signatures https://review.openstack.org/639760
09:58:37 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Move set_temp_url_key logic into resource objects https://review.openstack.org/648347
09:58:38 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Remove and rearrange object bonged proxy tests https://review.openstack.org/648352
10:31:29 openstackgerrit Merged openstack/openstacksdk master: Cleanup split of openstackcloud https://review.openstack.org/648435
12:41:19 openstackgerrit Cao Xuan Hoang proposed openstack/openstacksdk master: Expose locked status for Server https://review.openstack.org/648931
15:06:54 openstackgerrit Cao Xuan Hoang proposed openstack/openstacksdk master: Expose locked status for Server https://review.openstack.org/648931
15:39:54 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Add support for generating form-post signatures https://review.openstack.org/639760
15:39:54 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Move set_temp_url_key logic into resource objects https://review.openstack.org/648347
15:39:55 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Remove and rearrange object bonged proxy tests https://review.openstack.org/648352
15:43:07 openstackgerrit Stephen Finucane proposed openstack/os-api-ref master: trivial: Remove unused attribute https://review.openstack.org/649079
16:32:49 cyberpear It seems like newer versions of opentack sdk barf if a domain is not provided, but older versions accept no domain just fine. Is this an accurate assessment? Is this expected?
16:33:11 cyberpear (I noticed because my ansible playbooks broke on the step of 'os_user')
19:56:42 openstackgerrit Jim Rollenhagen proposed openstack/python-openstackclient master: Ignore case in security group rule --ethertype https://review.openstack.org/649174
19:57:48 jroll ^ fairly easy review :)
20:03:21 cyberpear looks good to me
20:06:01 dtroyer jroll: Really, we didn't already do that? /me shakes head…
20:06:20 jroll right? I was surprised
#openstack-sdks - 2019-04-02
07:46:09 openstackgerrit Cao Xuan Hoang proposed openstack/openstacksdk master: Expose locked status for Server https://review.openstack.org/648931
08:51:42 pas-ha hi all, plz review https://review.openstack.org/#/c/564200 (add user password locking capabilities to openstack CLI, supported by Keystone since Rocky)
10:21:34 openstackgerrit Artem Goncharov proposed openstack/openstacksdk master: Move Orchestration methods into Proxy https://review.openstack.org/649273
10:40:34 mordred gtema: nice call on importing resource into normalize :)
10:41:17 mordred gtema: I think you probably already have thought this - but once we're done squishing all of these things together, I hope _normalize.py goes away and we just use the Resource definitions to define the data models
11:20:33 openstackgerrit Merged openstack/os-api-ref master: trivial: Remove unused attribute https://review.openstack.org/649079
13:40:19 openstackgerrit Dean Troyer proposed openstack/python-openstackclient master: Volume backup functional test tweak https://review.openstack.org/649327
13:41:53 openstackgerrit Artem Goncharov proposed openstack/openstacksdk master: Allow dynamic properties on Resource https://review.openstack.org/649329
14:44:08 openstackgerrit Artem Goncharov proposed openstack/openstacksdk master: Move Orchestration methods into Proxy https://review.openstack.org/649273
16:24:22 Shrews mordred: i was thinking a bit about the statsd key formatting issue (wrt to nodepool). it'd be convenient if the caller could pass a method to use to format the key. but looking at how we pass the statsd info all the way down to Proxy... geesh that's a mess. adding a new param for the formatting func is painful.
16:25:10 mordred Shrews: yeah - I thought about a similar thing and had a similar self response
16:25:11 Shrews somehow factoring that portion out would be ideal....
16:25:50 mordred like - we COUDL support pushing a formatting filter function - but oy
16:26:01 Shrews yyeah
17:24:01 openstackgerrit Stephen Finucane proposed openstack/python-openstackclient master: Make 'destination' optional for live migration https://review.openstack.org/626949
17:24:16 stephenfin I apologize in advance for ^ but it's the best I could do
17:45:50 openstackgerrit Merged openstack/python-openstackclient master: Ignore case in security group rule --ethertype https://review.openstack.org/649174
17:56:54 mordred stephenfin: I am going to defer to mriedem on that - he has opinions - I have pinged him in nova channel
18:05:12 openstackgerrit Artem Goncharov proposed openstack/openstacksdk master: Move Orchestration methods into Proxy https://review.openstack.org/649273
18:35:51 Shrews mordred: i'm not seeing any reason why statsd clients seem to be instantiated per-CloudRegion. the clouds.yaml section for statsd appears to be at the top level, yeah? (i couldn't actually find examples)
18:36:51 Shrews i think maybe we could move instantiation to Connection and maybe use a singleton for stats reporting
18:37:31 Shrews *then* it would be easier to add a custom key formatter
18:38:37 Shrews maybe... /me tosses around words and ideas like wet cats
22:13:13 mordred Shrews: hrm. good point re: clouds.yaml - although the client object can be passed in per-connection. - so yeah, maybe a singleton something something I like where you're thinking
#openstack-sdks - 2019-04-03
04:28:08 openstackgerrit Merged openstack/openstacksdk master: Expose locked status for Server https://review.openstack.org/648931
06:50:00 openstackgerrit Artem Goncharov proposed openstack/openstacksdk master: Move Orchestration methods into Proxy https://review.openstack.org/649273
07:01:56 mordred dtantsur: if you get a minute - I believe I have addressed your review concerns: https://review.openstack.org/#/c/639760/
09:46:35 openstackgerrit Artem Goncharov proposed openstack/openstacksdk master: Move Orchestration methods into Proxy https://review.openstack.org/649273
09:52:26 openstackgerrit Artem Goncharov proposed openstack/openstacksdk master: Move Orchestration methods into Proxy https://review.openstack.org/649273
10:30:52 openstackgerrit Stephen Finucane proposed openstack/python-openstackclient master: Make 'destination' optional for live migration https://review.openstack.org/626949
10:30:52 openstackgerrit Stephen Finucane proposed openstack/python-openstackclient master: Allow specifying destination for cold migrations https://review.openstack.org/649574
11:06:14 openstackgerrit Artem Goncharov proposed openstack/openstacksdk master: Move Orchestration methods into Proxy https://review.openstack.org/649273
12:26:28 Shrews mordred: i have some in-flux code i'll post when i'm more coffeed up
13:25:31 openstackgerrit David Shrewsbury proposed openstack/openstacksdk master: WIP: Stats singleton https://review.openstack.org/649610
13:26:26 Shrews mordred: that ^^ is where I got to yesterday with unwinding the stats pass-thru stuff before I hit CloudRegion. but that's sort of what I was thinking (which may still be bad)
13:32:14 Shrews mordred: not being very familiar with the new keystoneauth task management, i'm also not sure if there may be badness around multiple thread things and a singleton we define/use
13:33:30 mordred Shrews: shouldn't be. the keystoneauth approach is to manage a sempahore and let concurrency come from whatever app-level concurrency exists
13:33:56 mordred Shrews: we would just need to make sure that whatever singleton we defined was itself thread-safe
13:35:48 openstackgerrit Jeremy Houser proposed openstack/python-openstackclient master: Moved skiptest to setUpClass in order to avoid failures in test_snapshot and test_volume_transfer due to presence of openstack commands within setUpClass https://review.openstack.org/648760
15:29:09 tosky dtroyer: hi! Back to our discussion about openstackclient-check-plugins failing on stable branches from few weeks ago... is there anything that can be done?
15:57:08 dtroyer cancel my meetings?
15:58:00 dtroyer tosky: I'm sure there is, I don't have that in my head at the moment but yeah, this is a good time to clean that up
16:02:09 tosky dtroyer: I can't help with your meetings :) but I can help with the implementation of a technical solution, if any; I'm just not sure about the right fix
16:23:41 openstackgerrit Merged openstack/openstacksdk master: Add support for generating form-post signatures https://review.openstack.org/639760
18:49:29 johnsom Is there a reason we don't have http://git.openstack.org/cgit/openstack/python-openstackclient/tree/openstackclient/network/v2/_tag.py in osc_lib?
18:50:41 johnsom Octavia is also doing tags and would like to share that code.
18:50:48 openstackgerrit Jeremy Houser proposed openstack/python-openstackclient master: Prevent setUpClass failures in volume v1 tests https://review.openstack.org/648760
18:52:19 johnsom I will sign up to "make that happen" if there are no objections.
19:14:47 dtroyer johnsom: probably because it originated there and nobody else has asked yet. I look forward to watching you make that happen :)
19:15:24 johnsom Ok, just wanted to check before effort was expended.
19:15:39 dtroyer np, and thank you
19:42:49 openstackgerrit Merged openstack/openstacksdk master: Move set_temp_url_key logic into resource objects https://review.openstack.org/648347
19:42:50 openstackgerrit Merged openstack/openstacksdk master: Remove and rearrange object bonged proxy tests https://review.openstack.org/648352
#openstack-sdks - 2019-04-04
09:49:54 dtantsur dustinc, mordred, I've started an etherpad on missing bare metal features in SDK: https://etherpad.openstack.org/p/openstacksdk-baremetal-gaps

Earlier   Later