| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2018-05-14 | |||
| 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. | |
| 18:10:37 | kmalloc | and get that out the door | |
| 18:49:11 | openstackgerrit | Merged openstack/openstacksdk master: Fixing bug where original and new dicts would always be the same https://review.openstack.org/566683 | |
| 20:09:04 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Pass default_microversion to adapter constructor https://review.openstack.org/557935 | |
| 20:09:05 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Rename service_key to service_type https://review.openstack.org/558049 | |
| 20:09:05 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Throw an error on missing support for default microversion https://review.openstack.org/557938 | |
| 20:11:18 | openstackgerrit | Monty Taylor proposed openstack/keystoneauth master: Expose version_between as a real function https://review.openstack.org/568640 | |
| 20:44:29 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Throw an error on missing support for default microversion https://review.openstack.org/557938 | |
| 20:44:29 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Pass default_microversion to adapter constructor https://review.openstack.org/557935 | |
| 20:44:30 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Rename service_key to service_type https://review.openstack.org/558049 | |
| 21:41:16 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Avoid globally modifying yaml library https://review.openstack.org/568705 | |
| #openstack-sdks - 2018-05-16 | |||
| 07:11:39 | 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:31 | frickler | OSC cores: in addition to slaweq's request, another reminder for these patches waiting in queue for a long time already: https://review.openstack.org/516701 https://review.openstack.org/558824 https://review.openstack.org/540633 https://review.openstack.org/559742 | |
| 09:26:12 | frickler | dtroyer: amotoki: dhellmann: ^^ | |
| 09:48:58 | 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 | |
| 12:42:57 | dtantsur | folks, who knows what magic to add to keystoneauth Session to make it work with custom SSL certificates that are installed in the system? | |
| 12:43:17 | mordred | dtantsur: morning! | |
| 12:43:22 | dtantsur | I know that openstackclient, ironicclient, etc work correctly, but a naively written application does not.. | |
| 12:43:30 | dtantsur | hey mordred | |
| 12:44:08 | dtantsur | (it's not impossible that tripleo sets up certificates in a weird way, but everything seems okay with it...) | |
| 12:44:26 | mordred | dtantsur: verify and cert are the arguments you want | |
| 12:44:42 | dtantsur | mordred: okay, but what do I set them to? | |
| 12:44:54 | mordred | dtantsur: they take weird argugments - one sec | |
| 12:45:44 | mordred | dtantsur: https://docs.openstack.org/keystoneauth/latest/api/keystoneauth1.html#keystoneauth1.session.Session | |
| 12:45:58 | mordred | dtantsur: verify takes either True or False of the path to a CA bundle | |