| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2018-05-11 | |||
| 03:13:18 | openstackgerrit | huangshan proposed openstack/openstackclient master: Update http links in doc https://review.openstack.org/567732 | |
| 06:44:56 | openstackgerrit | chenyb4 proposed openstack/openstacksdk master: Add function examples test for clustering https://review.openstack.org/555762 | |
| 08:21:09 | openstackgerrit | Merged openstack/openstacksdk master: Defer all endpoint discovery to keystoneauth https://review.openstack.org/564494 | |
| 11:06:13 | 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 | |
| 14:00:02 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Reenable osc-functional-devstack-tips and neutron-grenade https://review.openstack.org/549309 | |
| 14:00:16 | mordred | slaweq, Shrew, rods: if you get bored today, I'd love reviews on https://review.openstack.org/#/q/topic:service-type-aliases+status:open and https://review.openstack.org/#/q/topic:test-updates+status:open | |
| 14:00:45 | slaweq | mordred: since I'm in RH I'm never bored | |
| 14:00:50 | slaweq | mordred: but I will take a look :) | |
| 14:00:52 | mordred | slaweq: hehe | |
| 14:05:55 | slaweq | mordred: https://review.openstack.org/#/c/547225/1 is for stable/queens and it doesn't look like cherry-pick, is it intentional? | |
| 14:08:51 | mordred | slaweq: I can't remember if it was intentional - but it's definitely mislabeled | |
| 14:09:46 | mordred | slaweq: yah - was a cherry-pick ... but I don't think we need it anymore - I'm just going to abandon | |
| 14:09:59 | slaweq | one less to check :) | |
| 14:10:02 | slaweq | thx | |
| 14:10:08 | mordred | \o/ | |
| 14:25:44 | Shrews | mordred: reviewed the test series, but the other is going to take more brain power than i can spare at the moment while figuring out multi-label stuff | |
| 14:42:43 | Shrews | mordred: ok, i did review the first couple of the other series until i found an issue. | |
| 14:47:39 | mordred | Shrews: kk. thanks! (also yeah - multi-label) | |
| 15:45:25 | openstackgerrit | Merged openstack/openstacksdk master: Add nodepool-functional-py35-src job https://review.openstack.org/567254 | |
| 15:51:35 | openstackgerrit | Merged openstack/openstacksdk master: Add python 3.6 jobs https://review.openstack.org/567255 | |
| 17:52:26 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Add get_volume_limits() support https://review.openstack.org/564507 | |
| 19:00:50 | openstackgerrit | Merged openstack/openstacksdk master: Reenable osc-functional-devstack-tips and neutron-grenade https://review.openstack.org/549309 | |
| #openstack-sdks - 2018-05-12 | |||
| 15:32:53 | openstackgerrit | Monty Taylor proposed openstack-infra/shade master: Fix recent pep8 issues https://review.openstack.org/567996 | |
| 17:43:47 | shellfishers | TESTING TESTING | |
| 17:49:50 | shellfishers | TESTING TESTING | |
| #openstack-sdks - 2018-05-13 | |||
| 13:14:12 | openstackgerrit | Merged openstack-infra/shade master: Fix recent pep8 issues https://review.openstack.org/567996 | |
| 13:25:52 | openstackgerrit | XiaojueGuan proposed openstack/api-wg master: Trivial: update url to new url https://review.openstack.org/568075 | |
| 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 :) | |