| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2018-05-13 | |||
| 13:45:48 | openstackgerrit | Merged openstack/openstacksdk master: Add get_volume_limits() support https://review.openstack.org/564507 | |
| 14:54:24 | openstackgerrit | Merged openstack/api-wg master: Trivial: update url to new url https://review.openstack.org/568075 | |
| #openstack-sdks - 2018-05-14 | |||
| 06:33:36 | openstackgerrit | Lingyong Xu proposed openstack/cliff master: Trivial: Fix the pep8 warning The yaml should start with "---" https://review.openstack.org/568205 | |
| 09:15:18 | openstackgerrit | Slawek Kaplonski proposed openstack/python-openstackclient master: Make max_burst_kbps option as optional for bw limit QoS rule https://review.openstack.org/567828 | |
| 09:23:07 | openstackgerrit | Dongcan Ye proposed openstack/python-openstackclient master: Network: Add tag support for QoS policy https://review.openstack.org/547810 | |
| 09:24:22 | openstackgerrit | Dongcan Ye proposed openstack/python-openstackclient master: Network: Add tag support for security group https://review.openstack.org/547733 | |
| 12:15:47 | openstackgerrit | Daniel Speichert proposed openstack/openstacksdk master: Added few image properties to Image class https://review.openstack.org/544049 | |
| 14:17:18 | rcarrillocruz | mordred , Shrews : out of curiosity, how you filter the jobs for ansible shade ? i.e. i don't see a 'files' attribute on the jobs def, curious how you trigger them on just-openstack-modules PRs | |
| 14:20:46 | mordred | rcarrillocruz: it's in project pipeline definition | |
| 14:21:04 | mordred | rcarrillocruz: because we only want to skip things on ansible prs ... one sec | |
| 14:21:26 | rcarrillocruz | ah ok... | |
| 14:21:47 | mordred | rcarrillocruz: https://github.com/openstack-infra/project-config/blob/master/zuul.d/projects.yaml#L4-L16 | |
| 14:23:57 | rcarrillocruz | i'm helping out gundalow to put jobs for ansible/ansible network modules | |
| 14:24:13 | rcarrillocruz | and wanted to link him the 'files' thingy in a reall world example | |
| 15:40:33 | openstackgerrit | Daniel Speichert proposed openstack/openstacksdk master: Added image properties to Image class https://review.openstack.org/544049 | |
| 15:42:49 | openstackgerrit | Daniel Speichert proposed openstack/openstacksdk master: Added few image properties to Image class https://review.openstack.org/544049 | |
| 15:45:12 | openstackgerrit | Daniel Speichert proposed openstack/openstacksdk master: Added image properties to Image class https://review.openstack.org/544544 | |
| 15:46:32 | dasp | mordred: sorry about the spam above, https://review.openstack.org/#/c/544049/ that you previously CR+2'ed now passes testing | |
| 15:49:55 | dasp | I'm also unsure what's the best way to implement arbitrary properties on the Image object, I've updated https://review.openstack.org/#/c/544544/ about that. It uses Image.properties as a bag for those unexpected properties but is it worth packing/unpacking them from the API object? | |
| 16:19:47 | mordred | dasp: I think it is - on the shade side we put known/official properties on the main Image and the others into a properties - I'd eventually like for both the shade and sdk sides of openstacksdk to be working with the same objects... so I think that's a step in the right direction - I've got thatpatch queued up for review here in just a little bit | |
| 16:21:16 | dasp | mordred: okay, I think my latest iteration correctly gathers those unexpected properties and allows for updating them, had to modify some tests. I'm looking forward to your review and comments if I can make it better. | |
| 17:32:10 | mnaser | it would be nice if 'openstack image create' ported over the '--progress' option that exists in glance | |
| 17:41:12 | mordred | mnaser: we've got a todo-list item for that sdk-side | |
| 17:41:28 | mnaser | mordred: cool, i try to mention things i see missing when i do :) | |
| 17:41:31 | mordred | mnaser: ianw started poking at some generalized callback mechanisms for tracking progress of long-running calls | |
| 17:41:52 | mnaser | i'd push up a patch but i figured it was something a bit more complicated in regards to a standarized progress tracking method | |
| 17:58:56 | openstackgerrit | Daniel Speichert proposed openstack/openstacksdk master: Added image properties to Image class https://review.openstack.org/544544 | |
| 18:17:53 | openstackgerrit | Pavlo Shchelokovskyy proposed openstack/python-openstackclient master: Resolve images and flavors one by one in server list https://review.openstack.org/568345 | |
| 18:17:53 | openstackgerrit | Pavlo Shchelokovskyy proposed openstack/python-openstackclient master: Skip calls to glance and nova when got no servers https://review.openstack.org/568344 | |
| 19:13:20 | mordred | mnaser: yah - the tricky part is that v1 PUT and v2 PUT are both just single realy long HTTP PUT calls - so we'll likely need a virtual file wrapper object that can fire callbacks ever X bytes read/transferred or something insane like that | |
| 19:13:40 | mordred | BUT - once we do it once, since we'll do it at the low-level adapter layer, we should actually be able to apply it to all calls | |
| 19:52:52 | dasp | is openstackclient.tests.functional.compute.v2.test_aggregate.AggregateTests known to fail or is it a random error? | |
| 19:57:34 | mordred | dasp: I don't know that I've noticed it a bunch - I see floating ip and volume tests most frequently with random errors - but looking at that one I'd say that's pretty fair for a recheck | |
| 20:00:07 | mordred | dasp: found one thing I think you probably want to change, but otherwise I thnk it looks good | |
| 20:00:53 | mordred | adriant: ^^ if you have a sec, https://review.openstack.org/544544 has a behavior change in it and would be nice to have you chime in on | |
| 20:07:56 | openstackgerrit | Daniel Speichert proposed openstack/openstacksdk master: Added image properties to Image class https://review.openstack.org/544544 | |
| 20:25:13 | openstackgerrit | Brian Rosmaita proposed openstack/keystoneauth master: WIP - fix logging of encoded headers https://review.openstack.org/568365 | |
| 20:35:17 | openstackgerrit | Brian Rosmaita proposed openstack/keystoneauth master: WIP - fix logging of encoded headers https://review.openstack.org/568365 | |
| 22:53:13 | adriant | mordred: will look | |
| 23:48:50 | adriant | mordred: Oh I see, so glance kind of does something similar with extra properties like Keystone. | |
| 23:53:07 | adriant | actually that's reminds me... I should test the Keystone extra properties stuff. I know we sort of have a few like 'email' (which isn't a real field in Keystone), but Keystone's extra json blob lets you add arbitrary values. | |
| #openstack-sdks - 2018-05-15 | |||
| 00:15:48 | adriant | mordred: left a review on the patch. It works, but also doesn't when getting back the values. | |
| 00:49:39 | mnaser | mordred: if you have extra cycles (lol) i think we need to work on the "openrc" files delivered by horizon | |
| 00:49:54 | mnaser | our queens horizon deployment still has a "OpenStack RC File v2" download option | |
| 01:25:26 | adriant | mnaser: in what capacity do you mean work on openrc files? | |
| 01:25:35 | adriant | as in get rid of them, or get rid of v2 ones? | |
| 01:25:43 | mnaser | adriant: as in, we shouldn't be giving away "RC File v2" when keystone v2 api is gone | |
| 01:25:57 | adriant | that is a config option in horizon now ;) | |
| 01:26:01 | adriant | lemme find a link | |
| 01:26:12 | mnaser | i'd assume queens release of keystone should just hide it automagically? | |
| 01:26:23 | mnaser | also the format they're delivered needs work with workarounds for v2/v3 and pinned versions in the openrc file (ex OS_AUTH_URL with trailing /v3) | |
| 01:27:00 | adriant | mnaser: horizon is backwards compatible with older versions of services even up to queens | |
| 01:27:26 | mnaser | adriant: i've had experience otherwise (maybe it was just magnum or cant remember which service) where it required a higher microversion by default | |
| 01:27:33 | mnaser | but that could be a a plugin and not native horizon | |
| 01:27:52 | adriant | we tend to run horizon off the latest release and our cloud is running a weird mix of services | |
| 01:28:23 | adriant | mnaser: https://docs.openstack.org/horizon/latest/configuration/settings.html#show-keystone-v2-rc | |
| 01:28:37 | adriant | ^ we can propose the default is false from Rocky onwards | |
| 01:29:47 | adriant | also related: https://docs.openstack.org/horizon/latest/configuration/settings.html#user-menu-links but the first value does remove the v2 link if present | |
| 03:18:15 | openstackgerrit | chenyb4 proposed openstack/openstacksdk master: Add function examples test for clustering https://review.openstack.org/555762 | |
| 08:38:29 | Vishal_ | I am not able to create tenant vm inside a project(tenant) using openstack-sdk | |
| 08:38:35 | Vishal_ | Getting error like | |
| 08:38:47 | Vishal_ | RESP BODY: {"badRequest": {"message": "Invalid input for field/attribute server. Value: {u'name': u'tenant-300-net-1-vm-1', u'imageRef': u'54f7b0a7-e3d3-4265-8342-f469ed444db2', u'tenant_id': u'19f5c2522fbd4cf8b37a89d0758bdd68', u'flavorRef': u'1', u'networks': [{u'uuid': u'70e7f644-45c6-4006-a187-59d0c9d6ab5e'}], u'security_groups': [{u'name': u'39f8598e-dd23-4ea1-873c-665f7348cce9'}, {u'name': u'34d84a57-8b63-4d4f-ba9b-298ad | |
| 08:40:14 | Vishal_ | Can somebody please help...is it an allowed workflow...if not how can I spawn a vm in a different tenant using sdk | |
| 10:46:53 | openstackgerrit | Chen proposed openstack/python-openstackclient master: Compute: Add description support for server https://review.openstack.org/568549 | |
| 12:15:51 | pooja-jadhav | cdent: Hi | |
| 12:16:50 | pooja-jadhav | cdent: Do you know difference between LEGACY_NOTIFIER and NOTIFIER in Nova? | |
| 12:17:40 | cdent | pooja_jadhav: NOTIFIER uses newer versioned notifications that are based on oslo versioned objects | |
| 12:18:42 | pooja_jadhav | cdent: Ohhk.. :) | |
| 12:19:04 | cdent | pooja_jadhav: what are you trying to figure out? | |
| 12:20:00 | pooja_jadhav | cdent: Actually, as nova is using versioned notification, I am trying same thing in cinder. | |
| 12:23:14 | pooja_jadhav | cdent: I am understanding the nova code, but in that they have created group for notification and conf options in that group. In cinder, what is the correct place where I should add group for notifications and the related conf options..? can u please guide me for the same? | |
| 12:24:10 | cdent | pooja_jadhav: I'm not familiar with the cinder code | |
| 12:24:32 | pooja_jadhav | cdent: Ohhh, no problem :) | |
| 12:24:42 | pooja_jadhav | cdent: Thanks for ur inputs | |
| 12:25:00 | cdent | you're welcome, sorry i'm not more help. good luck. | |
| 12:25:30 | pooja_jadhav | cdent: :) | |
| 12:35:41 | openstackgerrit | Chen proposed openstack/python-openstackclient master: Compute: Add description support for server https://review.openstack.org/568549 | |
| 13:15:00 | openstackgerrit | Pavlo Shchelokovskyy proposed openstack/python-openstackclient master: Skip calls to glance and nova when got no servers https://review.openstack.org/568344 | |
| 13:15:01 | openstackgerrit | Pavlo Shchelokovskyy proposed openstack/python-openstackclient master: Add --name-lookup option to server list https://review.openstack.org/568345 | |
| 13:30:46 | mordred | mnaser: I added an ability for horizon to serve clouds.yaml files ... I'd prefer to spend more effort on just handing those out ... I've been meaning to write an osc command for "add this clouds.yaml file to my existing file" | |
| 13:31:11 | mnaser | yeah that is a nicer alterantive | |
| 13:52:07 | Vishal_ | As an admin tenant I want to create a vm on some other tenant...but when I am doing this using openstack sdk I am getting this error http://paste.openstack.org/show/721003/ | |
| 13:52:26 | Vishal_ | Any pointers?? thanks | |
| 14:10:42 | openstackgerrit | Monty Taylor proposed openstack/keystoneauth master: Fix logging of encoded headers https://review.openstack.org/568365 | |
| 14:11:01 | dasp | Vishal_: per https://developer.openstack.org/api-ref/compute/#create-server tenant_id is not an allowed field, you should get a token for that project and use that to make the request | |
| 14:13:15 | mordred | Vishal_: what dasp said. you may want to try the 'connect_as' method on connection which will return you a new connection object ... | |
| 14:13:56 | mordred | so like "new_conn = conn.connect_as(project_id='19f5c2522fbd4cf8b37a89d0758bdd68')" | |
| 14:19:34 | openstackgerrit | Monty Taylor proposed openstack/keystoneauth master: Fix logging of encoded headers https://review.openstack.org/568365 | |
| 14:29:56 | openstackgerrit | Monty Taylor proposed openstack/keystoneauth master: Fix logging of encoded headers https://review.openstack.org/568365 | |
| 15:25:42 | slaweq | hi OSC cores, can You add to Your review list: https://review.openstack.org/#/c/567828/ and https://review.openstack.org/#/c/567620/ ? | |
| 15:25:45 | slaweq | thx in advance | |
| 16:43:49 | openstackgerrit | Monty Taylor proposed openstack/keystoneauth master: Expose version_between as a real function https://review.openstack.org/568640 | |
| 17:01:10 | openstackgerrit | Slawek Kaplonski proposed openstack/python-openstackclient master: Make max_burst_kbps option as optional for bw limit QoS rule https://review.openstack.org/567828 | |
| 17:44:10 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Pass default_microversion to adapter constructor https://review.openstack.org/557935 | |
| 17:44:10 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Honor service-type aliases in config https://review.openstack.org/566929 | |
| 17:44:11 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Rename service_key to service_type https://review.openstack.org/558049 | |
| 17:44:11 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Throw an error on missing support for default microversion https://review.openstack.org/557938 | |
| 17:44:55 | mordred | kmalloc: we should release a ksa with the new function before https://review.openstack.org/557938 gets landed/released (the current version of the patch uses the private function, which is obviously not great) | |
| 18:10:05 | kmalloc | mordred: ++ | |
| 18:10:32 | kmalloc | mordred: working on the sphinx issue shortly for the header fix, then we can do the next round of things. | |