Earlier  
Later
New Search
Posted Nick Remark
#openstack-sdks - 2018-10-25
19:01:55 peschk_l and in the case we need to move on: is there a recommendation from your side ?
20:04:05 mriedem who besides dean is an osc core that i can bug for reviews on this old bug fix? https://review.openstack.org/545946
20:08:13 mriedem amotoki: ^?
20:21:48 mordred mriedem: looking
20:22:25 mriedem i'm sure it'll be the best thing you've looked at all day
20:22:58 mordred mriedem: I've been on a plane all day - so as long as it's better than Ocean's 8 - I'll be thrilled
20:23:49 mriedem thanks
23:09:59 kmalloc peschk_l: i would plan on moving off paste. I am in process of writing a compat bit for oslo.middleware to load middleware and the app w/o paste
23:10:15 kmalloc peschk_l: but paste is maintained minimally because we have projects leaning on it
#openstack-sdks - 2018-10-26
01:36:33 openstackgerrit Merged openstack/keystoneauth master: Add missing release note for ironic discovery fix https://review.openstack.org/612872
03:20:17 openstackgerrit Merged openstack/python-openstackclient master: Default --nic to 'auto' if creating a server with >= 2.37 https://review.openstack.org/545946
04:26:09 openstackgerrit Ian Wienand proposed openstack/openstacksdk master: Move pre/post run task calls to queue https://review.openstack.org/613503
04:26:10 openstackgerrit Ian Wienand proposed openstack/openstacksdk master: Add doc depends to tox releasenotes environment https://review.openstack.org/613504
04:28:19 openstackgerrit Ian Wienand proposed openstack/openstacksdk master: Add doc depends to tox releasenotes environment https://review.openstack.org/613504
04:28:19 openstackgerrit Ian Wienand proposed openstack/openstacksdk master: Call pre/post run task calls from TaskManager.submit_task() https://review.openstack.org/613503
08:35:55 openstackgerrit Ian Wienand proposed openstack/openstacksdk master: Add doc depends to tox releasenotes environment https://review.openstack.org/613504
08:35:55 openstackgerrit Ian Wienand proposed openstack/openstacksdk master: Call pre/post run task calls from TaskManager.submit_task() https://review.openstack.org/613503
13:13:13 openstackgerrit Sean McGinnis proposed openstack/python-openstackclient master: Add volume backup import/export commands https://review.openstack.org/612735
13:19:06 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Don't pass disk_format or container_format to image task upload https://review.openstack.org/613141
13:40:57 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Add doc depends to tox releasenotes environment https://review.openstack.org/613504
13:40:57 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Call pre/post run task calls from TaskManager.submit_task() https://review.openstack.org/613503
14:44:54 openstackgerrit Brian Haley proposed openstack/python-openstackclient master: Update the Neutron CLI decoder document https://review.openstack.org/613605
16:53:37 openstackgerrit Sean McGinnis proposed openstack/python-openstackclient master: Remove deprecated network args https://review.openstack.org/613644
17:05:01 openstackgerrit Sean McGinnis proposed openstack/python-openstackclient master: Remove invalid 'unlock-volume' migration arg https://review.openstack.org/558910
18:51:09 openstackgerrit Daniel Speichert proposed openstack/openstacksdk master: Added basic CRUD functionality around Host Aggregates https://review.openstack.org/609416
18:54:17 openstackgerrit Sean McGinnis proposed openstack/openstacksdk master: Fix incorrect use of flake8:noqa https://review.openstack.org/613667
#openstack-sdks - 2018-10-27
01:13:28 openstackgerrit Merged openstack/openstacksdk master: Add registered limit CRUD support https://review.openstack.org/609572
04:13:05 openstackgerrit Merged openstack/openstacksdk master: Add limit CRUD support https://review.openstack.org/609604
08:14:46 openstackgerrit Yuval Shalev proposed openstack/openstacksdk master: Added assign function to identity v3 proxy https://review.openstack.org/612006
14:47:26 openstackgerrit Merged openstack/openstacksdk master: Fix incorrect use of flake8:noqa https://review.openstack.org/613667
14:47:46 openstackgerrit Merged openstack/openstacksdk master: Don't pass disk_format or container_format to image task upload https://review.openstack.org/613141
14:53:42 openstackgerrit Merged openstack/openstacksdk master: Add vnic_type to create_port valid kwargs https://review.openstack.org/612162
15:53:15 openstackgerrit Merged openstack/openstacksdk master: Add the ability to extend a volume size https://review.openstack.org/595873
16:05:39 openstackgerrit Merged openstack/openstacksdk master: Stop running shade tests https://review.openstack.org/612407
20:22:06 openstackgerrit Maxim Babushkin proposed openstack/openstacksdk master: Add port_security_enabled to create_port valid kwargs https://review.openstack.org/613759
21:48:06 openstackgerrit Maxim Babushkin proposed openstack/openstacksdk master: Add port_security_enabled to create_port valid kwargs https://review.openstack.org/613759
#openstack-sdks - 2018-10-28
13:41:13 openstackgerrit Merged openstack/openstacksdk master: Use python3 format syntax for citycloud https://review.openstack.org/612536
14:52:25 openstackgerrit Merged openstack/openstacksdk master: Add port_security_enabled to create_port valid kwargs https://review.openstack.org/613759
14:52:26 openstackgerrit Merged openstack/openstacksdk master: openstackcloud.py: Implement FWaaS wrapper methods. https://review.openstack.org/604811
15:06:03 ml mordred: thank you
16:02:51 openstackgerrit Merged openstack/openstacksdk master: Fix upload of Swift object smaller than segment limit (create_object) https://review.openstack.org/612113
19:29:17 openstackgerrit Merged openstack/python-openstackclient master: Update the Neutron CLI decoder document https://review.openstack.org/613605
20:39:24 openstackgerrit Akihiro Motoki proposed openstack/python-openstackclient master: Remove deprecated network args https://review.openstack.org/613644
20:49:47 openstackgerrit Merged openstack/python-openstackclient master: Improve document 'openstack complete' https://review.openstack.org/611469
20:49:47 openstackgerrit Merged openstack/python-openstackclient master: Update release note version reference table https://review.openstack.org/612754
23:34:56 openstackgerrit Merged openstack/openstacksdk master: Add all_projects as a preferred alias for all_tenants https://review.openstack.org/540390
23:42:06 openstackgerrit Merged openstack/openstacksdk master: Add support for per-service rate limits https://review.openstack.org/604521
#openstack-sdks - 2018-10-29
03:36:04 openstackgerrit Mike Chen proposed openstack/python-openstackclient master: Compute: Add description support for server https://review.openstack.org/568549
13:04:15 nmimi Hi all again!
13:04:21 nmimi I have one qustion, if someone knows the answer....
13:04:28 nmimi I try to create a server with SDK with the following attributes:
13:04:37 nmimi instance = self.conn.compute.create_server(name=vm_name,image_id=image.id,flavor_id=flavor_obj.id,networks=[{"uuid": network.id}], key_name=keypair.name)
13:04:48 nmimi When the server is created i noticed that although i haven't set security groups attribute, "Default" sec_group is being added to the server:
13:04:57 nmimi security_groups | name='default'
13:05:05 nmimi Is there a way to create the server without security groups (even "Default")?
13:09:20 dtantsur nmimi: the API docs say "If you omit this attribute, the API creates the server in the default security group"
13:09:27 dtantsur so it's not even openstacksdk doing, I suspect
13:21:35 nmimi dtantsur: Thanks for the answer!
13:27:48 mordred yeah, that is a very true statement
13:46:27 frickler nmimi: iiuc the only possible workaround for your issue is to create a port in your network beforehand and pass that to the "create_server" call: "Requested security groups are not applied to pre-existing ports."
13:57:22 nmimi frickler: i will try it, thanks!
14:01:07 mordred frickler: ooh. that's a workaround we should maybe put into create_server
14:01:50 mordred like - make security_groups=False create a port and pass it in
14:03:05 mordred nmimi: also - you might want to try self.conn.create_server instead of self.conn.compute.create_server - it's a bit more complete, and its api is more stable
14:03:47 smcginnis There are probably those that expect that behavior, so if we did a workaround in create_server we would probably have to make it a --no-security-group arg or something.
14:03:48 frickler mordred: I don't think that we want to do that in the sdk, because you will not be able to clean up properly on deleting the server
14:06:08 mordred frickler: hrm. that is a good point
14:37:22 openstackgerrit Daniel Speichert proposed openstack/openstacksdk master: Added basic CRUD functionality around Host Aggregates https://review.openstack.org/609416
14:43:15 openstackgerrit Artem Goncharov proposed openstack/openstacksdk master: implement block-storage backup resource https://review.openstack.org/591445
15:11:02 wouterdb hi all, I'm getting started with openstacksdk. I wonder: how do I close the connection? each connection maintains a thread pool and I'm opening a LOT of connections.
15:11:32 wouterdb I couldn't find anything in the documentation or the source and I can't figure out the search function on the bug tracker
15:13:25 mordred wouterdb: I don't know that we have a close function - but the connection opening a threadpool piece will be going away soon, so that issue should solve itself
15:14:02 mordred wouterdb: that said- any reason you're opening a lot of connections instead of sharing a connection? you should be able to use one connection from many threads
15:14:46 mordred (we do that in nodepool at pretty large scale - it's built to handle that use case)
15:17:25 mordred wouterdb: for reference- this: https://review.openstack.org/#/c/604926/ is the upcoming patch to remove the internal creation of thread pools - as soon as I write some tests for the keystoneauth-side replacement we should be able to get that forward - the extra threads spawned are annoying even in the shared connection usecase, so using the existing parallelism of the calling context to handle
15:17:27 mordred concurrency should be an improvement for everybody
15:38:20 wouterdb mordred: I'm writing code that logs in to the same openstack using different users
15:48:52 mordred wouterdb: ah - gotcha. that makes sense
15:50:20 mordred wouterdb: well, this will get better in a few weeks. in the mean time, if it's killing you, you can pin back to < 0.18.0
15:50:58 mordred the thread-splosion was introduced in 0.18.0 - and should go away hopefully by 0.20.0
15:55:13 wouterdb ok, can I find the docs for that version somewhere?
15:57:12 wouterdb I try connection.identity.find_domain(domain) on the 0.17.2 and I get AttributeError: 'Proxy' object has no attribute 'find_domain'
16:00:06 mordred wouterdb: that sounds like you're getting the v2 version of the proxy - find_domain has been on the v3 proxy object since 2015
16:01:02 mordred wouterdb: you might want to set OS_IDENTIY_API_VERSION=3 - or in your clouds.yaml set identity_api_version: 3 - up until 0.18.0 this was all driven by config (0.18 switch to properly using discovery)
16:01:11 mordred (I'm sorry this is wonky for you right now)
16:08:57 wouterdb It is a bit unfortunate, I've been working at porting code from the old client to openstacksdk for a week, under the assumption they don't have a close function because they don't really consume any system resources. But now it turns out that they do. So I really can't use it, as I can't properly dispose of the threadpools (even in the 604926 patch set, there still is a thread pool and no close).
16:09:03 wouterdb
16:10:20 wouterdb I really liked th idea of the library, to have one (rather nice) interface, but it all seems a bit risky now.
16:14:01 mordred wouterdb: which threadpool is still there in 604926 ?
16:14:23 mordred wouterdb: also - we could add a close method for you pretty easily if that would help
16:19:30 wouterdb @mordred: close method would be great, threadpool is at https://review.openstack.org/#/c/604926/12/openstack/cloud/openstackcloud.py line 7848
16:19:58 wouterdb mordred: thanks for your help, I'll consider my options
16:20:51 kmalloc mordred: looking closely at the rate limit stuff in ksa today
16:21:01 kmalloc mordred: i think the code looks ok, but i want to run through it in depth
16:21:08 kmalloc make sure there are no surprises
16:21:17 kmalloc if it's good i think we should land/release this week
16:22:31 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Add close method to shutdown threadpool https://review.openstack.org/613976
16:22:31 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Replace TaskManager with a keystoneauth concurrency https://review.openstack.org/604926
16:22:57 mordred wouterdb: ^^ that should get you a close() that will also work with the new code
16:23:57 mordred wouterdb: also - that threadpool will only exist if you happen to upload swift LargeObjects

Earlier   Later