Earlier  
Later
New Search
Posted Nick Remark
#openstack-sdks - 2019-02-21
21:15:52 elmiko mordred: definitely giving me a case of the sads
21:16:02 elmiko i can work around it, just curious why it happens
21:17:14 mordred elmiko: I'm trying to think of a good explanation of why that would happen - if you have the git repo locally, that should Just Work (although fwiw I tend to just do a pip install -e . in my source dir and then months later forget that I'm running off of my local branch :) )
21:25:32 elmiko mordred: ack, i am working locally and usually this works. i think i must have futzed with the versions or something.
21:26:14 elmiko what does the `-e` flag do?
21:27:46 mordred elmiko: it does an 'editable' install - it's like python setup.py develop
21:28:00 elmiko mordred: interesting, thanks!
21:28:03 mordred elmiko: so, basically, it installs links to your source dir - and as you change python files it's immediately reflected
21:28:13 elmiko that is super useful
21:28:29 mordred if you add new entrypoints in setup.cfg you'll need to re-run pip install -e .
21:28:37 mordred just fwiw
21:33:38 elmiko ack, thanks again =)
21:33:43 elmiko mordred ++
21:44:29 njohnston Hi! I have a quick question about adding functionality into the openstack sdk. I want to create functionality in the sdk that will use the neutron bulk port creation method, so that you can supply an array of port information in a single request and all of those ports will be created at once and in streamlined manner. Will I need to use something other than the generic _create, or can it handle a list of inputs in and a
21:44:29 njohnston list of objects returned?
21:49:18 mordred njohnston: that is a a great question! I don't think the generic _create will do what you want - so you might need to just do your own magic method
21:49:47 mordred njohnston: that said - maybe once you've got that we can abstract out a _create_list or something like that
21:50:17 njohnston mordred: Great, that is what I thought! I'll start working on it. Thanks!
21:51:40 mordred \o/
22:11:45 njohnston mordred: If I am understanding this code right, it sounds like I might need a different fundamental class than Resource, something like BulkResource that understands that instead of getting **attrs and returning Resource will be getting [attrs, attrs, attrs] and returning [Resource, Resource, Resource]. Does that sound like a sane proposition?
22:18:33 mordred njohnston: maybe? but also - maybe it's just a method on the proxy? (like, making a BulkResource might be a significant engineering effort - and the main bits of Resource are largely about expressing the remote objects
22:19:05 mordred njohnston: that said - maybe you do want a BulkResource object that takes another resource class as a parameter and does like you said
22:20:16 mordred njohnston: so that instead of conn.network.create_ports([args, args, args]) returning [Port, Port, Port] it returns BulkResource which wraps a list of Ports?
22:20:43 mordred njohnston: I could go either way - might be one of those "get in to it and see which sucks less" sorts of things
22:24:48 njohnston yeah, sounds like it. :-)
22:37:08 johnsom Thank you for the rechecks. Was just going to check on the gate status.
22:41:29 mordred johnsom: I was so excited about the gate status
23:03:48 openstackgerrit Merged openstack/openstacksdk master: Add Octavia (load_balancer) load balancer failover https://review.openstack.org/634010
23:52:50 openstackgerrit Merged openstack/openstacksdk master: Add Octavia (load_balancer) provider API support https://review.openstack.org/634488
23:52:52 openstackgerrit Merged openstack/openstacksdk master: Add Octavia (load_balancer) flavor profile API https://review.openstack.org/634518
23:52:54 openstackgerrit Merged openstack/openstacksdk master: Add Octavia (load_balancer) flavor API https://review.openstack.org/634532
#openstack-sdks - 2019-02-22
01:14:24 openstackgerrit Paul Belanger proposed openstack/openstacksdk master: Fix syntax error with exception handling https://review.openstack.org/638564
02:15:08 openstackgerrit Merged openstack/openstacksdk master: Add Octavia (load_balancer) amphora API https://review.openstack.org/634538
02:31:12 openstackgerrit Merged openstack/openstacksdk master: Add support for bodyless commits https://review.openstack.org/634558
04:56:49 openstackgerrit Merged openstack/openstacksdk master: Fix syntax error with exception handling https://review.openstack.org/638564
11:09:30 openstackgerrit David Rabel proposed openstack/python-openstackclient master: Only allow project ID in 'image add project' https://review.openstack.org/638495
14:22:56 openstackgerrit David Rabel proposed openstack/python-openstackclient master: Add --attach-status parameter to image set https://review.openstack.org/638671
14:24:11 mordred dtantsur, Shrews: https://review.openstack.org/#/c/637563/ is happy now - and has tests!
14:25:28 Shrews mordred: https://review.openstack.org/#/c/637563/3/openstack/service_description.py line 197... that return will never happen
14:25:44 Shrews will it? can you both raise and return?
14:25:54 Shrews is this some secret python black magic?
14:26:19 mordred Shrews: you are right - that return will never happen
14:26:48 Shrews aww shoot. i was hoping for the voodoo stuff
14:27:28 mordred sorry bout that - let me rev that real quick
14:28:14 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Bail earlier on a version mismatch for a supported service https://review.openstack.org/637563
14:28:19 mordred Shrews: thereyago
14:28:57 Shrews +2
14:29:51 frickler openstacksdk-ansible-devel-functional-devstack seems to be failing for some time and the error looks consistently like this http://logs.openstack.org/38/637238/3/check/openstacksdk-ansible-devel-functional-devstack/ca428a7/job-output.txt.gz#_2019-02-21_20_47_46_673039
14:32:37 mordred frickler: I think gtema had identified an ansible-side issue the other day
14:32:52 gtema yupp
14:33:11 gtema port_security_enabled was set to false by default
14:33:20 gtema in the os_network module
14:33:32 mordred and it probably wants to be None right?
14:33:34 gtema but I didn't manage to create PR in ansible for that
14:33:39 gtema yes, should be none
14:33:42 mordred I can do that real quick
14:36:13 frickler yeah, port_security_enabled=False is a very bad default
14:40:06 mordred frickler, gtema, Shrews, mnaser: https://github.com/ansible/ansible/pull/52821
14:41:09 frickler mordred: lgtm, thx. odyssey4me ^^
14:41:45 mordred frickler, odyssey4me: we should be able to see that -devel job go green on that PR
14:46:23 frickler mordred: is it expected that the details for that job on github only point to https://zuul.openstack.org/ and not to specific results?
14:47:33 frickler makes fixing issues probably a bit difficult. see e.g. https://github.com/ansible/ansible/pull/52699
14:47:59 frickler the shippable check points to results instead
14:52:29 mordred frickler: yeah, eventually we want to point that to the build page
15:22:35 openstackgerrit David Rabel proposed openstack/python-openstackclient master: Only allow project ID in 'image add project' https://review.openstack.org/638495
16:25:46 cmorpheus mordred: hi is sdk broken? https://review.openstack.org/636074 please halp
16:27:17 mordred cmorpheus: oh - that's shade - and is probably related to the dogpile.cache stuff morgan fixed in sdk a little bit ago
16:27:45 mordred cmorpheus: why don't we actually drop the shade jobs from the ksa repo - we've got the sdk jobs there now
16:28:32 openstackgerrit Monty Taylor proposed openstack/keystoneauth master: Remove shade jobs https://review.openstack.org/638704
16:28:37 mordred cmorpheus: ^^
16:30:05 cmorpheus mordred: +2 ty
16:31:49 openstackgerrit Monty Taylor proposed openstack-infra/shade master: Fix dogpile.cache 0.7.0 interaction https://review.openstack.org/638708
16:32:04 mordred cmorpheus, kmalloc: ^^ fwiw, that should fix the error ksa triggered
16:38:17 gtema frickler, odyssey4me, mordred, Shrews, mnaser: https://github.com/ansible/ansible/pull/52821 is ready. Leave a shipit who can
16:42:30 openstackgerrit David Rabel proposed openstack/python-openstackclient master: Add --attach-status parameter to volume set https://review.openstack.org/638671
16:52:21 gtema mordred: I'm now preparing move object logic from cloud to proxy (similar to image) and again see problem with custom resource properties
16:52:23 kmalloc mordred: ah cool
16:52:57 kmalloc mordred: aha, yeah that is dogpile just being better about not being sloppy... it is an unfortunate behavior break
16:52:59 gtema how can we handle this better, than doing always direct calls (not relying to regular resource methods)
16:53:00 kmalloc but... yeah
17:29:07 openstackgerrit David Rabel proposed openstack/python-openstackclient master: Add --attach-status parameter to volume set https://review.openstack.org/638671
17:35:30 mnaser mordred: just found a fun weird thing i guess
17:35:39 mnaser the nested profile thing, it makes a request and doesnt set the user agent
17:35:54 mnaser so it sends with python-requests which was being blocked by our waf poop
17:46:27 openstackgerrit Artem Goncharov proposed openstack/openstacksdk master: Move object methods to object_store proxy https://review.openstack.org/638731
17:49:39 openstackgerrit Artem Goncharov proposed openstack/openstacksdk master: Move object methods to object_store proxy https://review.openstack.org/638731
18:25:43 openstackgerrit Merged openstack/openstacksdk master: handle "paginated" argument in test_list properly https://review.openstack.org/638401
18:25:43 openstackgerrit Merged openstack/openstacksdk master: Make all resource locations process project_id https://review.openstack.org/634572
18:25:45 openstackgerrit Merged openstack/openstacksdk master: Rename compute.service.zone to availability_zone https://review.openstack.org/634573
18:25:46 openstackgerrit Merged openstack/openstacksdk master: Stop mocking method in fwaas test https://review.openstack.org/634574
19:03:29 openstackgerrit Merged openstack-infra/shade master: Fix dogpile.cache 0.7.0 interaction https://review.openstack.org/638708
19:38:43 elmiko mordred: happy friday, but also i tried `pip install -e .` and i still got the error! https://paste.fedoraproject.org/paste/oX3trx~rHGypkI3HdyLo2g
19:38:59 elmiko i think this must be because i have tags in my git history or something
19:39:20 elmiko and i haven't update the version in setup.cfg
20:30:34 openstackgerrit Merged openstack/keystoneauth master: Remove shade jobs https://review.openstack.org/638704
23:09:06 openstackgerrit Amit Oren proposed openstack/openstacksdk master: WIP: Add support for shared file systems (manila) https://review.openstack.org/638782
#openstack-sdks - 2019-02-23
00:38:21 openstackgerrit Merged openstack/keystoneauth master: Expose application credentials in AccessInfoV3 https://review.openstack.org/636074
07:42:46 openstackgerrit Amit Oren proposed openstack/openstacksdk master: WIP: Add support for shared file systems (manila) https://review.openstack.org/638782
15:07:31 mordred elmiko: that's really weird. I mean - I shouldn't ever need a version in setup.cfg
15:07:43 mordred elmiko: is that repo something I can clone and poke at locally?

Earlier   Later