| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2018-10-17 | |||
| 16:20:46 | dtruong | ok, i guess i didn't realize that those tests existed | |
| 16:21:21 | dtruong | those tests are more comprehensive so i guess we can drop my functional tests in favor of those | |
| 16:24:11 | dtruong | ok, i see now. the functional tests in cloud package came from shade integration | |
| 16:24:32 | dtruong | so they test a different code path than my functional tests | |
| 16:24:32 | ShilpaSD | please elaborate this point for me | |
| 16:25:19 | mordred | dtruong: yah - that's right. we really should go through and make one set of tests - but also the plan (that we've just started on) is to get the openstackcloud.py layer things to be built on top of the things in the Proxy layer | |
| 16:26:01 | mordred | but we're still figuring out exactly what that should look like | |
| 16:27:10 | dtruong | so i think until that is done, it is useful to keep the two sets of tests | |
| 16:27:27 | mordred | dtruong: I agree | |
| 16:27:40 | mordred | dtruong: while I've got you there -- this just happened: http://logs.openstack.org/84/609984/1/gate/openstacksdk-functional-devstack-senlin/a6a39d7/testr_results.html.gz | |
| 16:27:47 | mordred | does that mean anythig to you? | |
| 16:28:28 | mordred | oh - that's stable/rocky | |
| 16:29:09 | mordred | I think it needs https://review.openstack.org/#/c/610753/ - but that is failing test | |
| 16:29:26 | mordred | *phew* I was worried something extra bad had happened | |
| 16:31:43 | dtruong | i have to look into why https://review.openstack.org/#/c/610753/ is failing | |
| 16:32:02 | dtruong | i don't see anything obvious but i'll check it out | |
| 16:35:21 | mordred | me either - and thanks! | |
| 16:37:43 | dtruong | ok, i need to backport one of my changes to rocky: https://review.openstack.org/#/c/597225/ | |
| 16:37:52 | dtruong | then it should work | |
| 16:48:44 | mordred | dtruong: awesome. I rebased my senlin patch on top of your backport, then added a depends-on to that patch for the sdk patch | |
| 16:48:50 | mordred | maybe it'll all work this time :) | |
| 17:08:51 | openstackgerrit | Merged openstack/openstacksdk master: Rearrange shade image code https://review.openstack.org/609683 | |
| 21:50:11 | openstackgerrit | Merged openstack/openstacksdk master: Add some warnings and clarifications for discovery https://review.openstack.org/606113 | |
| 23:27:41 | eandersson | mordred, I created a bug for the issue I mentioned yesterday https://bugs.launchpad.net/shade/+bug/1798489 | |
| 23:27:41 | openstack | Launchpad bug 1798489 in Shade "v4-fixed-ip / v6-fixed-ip no longer supported in shade for new vms" [Undecided,New] | |
| 23:46:04 | mordred | eandersson: thanks! I'll take a peek at that when I wake up. are you using shade directly still or are you on sdk? (the bug will be in both - mostly just curious) | |
| 23:48:21 | eandersson | This is for ansible | |
| 23:59:01 | eandersson | (so ansible -> shade) | |
| #openstack-sdks - 2018-10-18 | |||
| 00:42:31 | openstackgerrit | Robin Cernin proposed openstack/python-openstackclient master: Improve document 'openstack complete' https://review.openstack.org/611469 | |
| 00:49:06 | openstackgerrit | Robin Cernin proposed openstack/python-openstackclient master: Improve document 'openstack complete' https://review.openstack.org/611469 | |
| 06:50:49 | openstackgerrit | wangxiyuan proposed openstack/openstacksdk master: Add registered limit CRUD support https://review.openstack.org/609572 | |
| 06:50:50 | openstackgerrit | wangxiyuan proposed openstack/openstacksdk master: Add limit CRUD support https://review.openstack.org/609604 | |
| 06:51:30 | openstackgerrit | wangxiyuan proposed openstack/openstacksdk master: Add limit CRUD support https://review.openstack.org/609604 | |
| 09:51:21 | openstackgerrit | Mike Chen proposed openstack/python-openstackclient master: Compute: Add description support for server https://review.openstack.org/568549 | |
| 09:52:12 | jayashri_bidwe | hi team, I have one query that is it possible to move magnum test which is inside openstacksdk/openstack/tests/functional/cloud package? please help. | |
| 13:22:11 | mordred | jayashri_bidwe: sure - where do you want to move it? | |
| 14:26:30 | openstackgerrit | Mike Chen proposed openstack/python-openstackclient master: Compute: Add description support for server https://review.openstack.org/568549 | |
| 15:15:46 | openstackgerrit | Merged openstack/os-api-ref master: Update sphinx extension logging https://review.openstack.org/611157 | |
| 15:30:22 | openstackgerrit | Monty Taylor proposed openstack-infra/shade master: Support v4-fixed-ip and v6-fixed-ip in create_server https://review.openstack.org/611651 | |
| 15:30:31 | mordred | eandersson: ^^ that should take care of your issue | |
| 15:31:01 | mordred | Shrews: if you get a sec, mind looking at that? | |
| 15:33:23 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Support v4-fixed-ip and v6-fixed-ip in create_server https://review.openstack.org/611652 | |
| 15:34:52 | Shrews | mordred: alrighty | |
| 15:38:13 | Shrews | mordred: found a problem | |
| 15:40:58 | mordred | Shrews: oh no! | |
| 15:41:12 | mordred | Shrews: you mean I'm not infallible? | |
| 15:41:12 | Shrews | oh yeah! | |
| 15:41:34 | Shrews | mordred: i was disheartened to discover this too | |
| 15:41:58 | Shrews | i mean... what's real? what do i believe? | |
| 15:42:31 | openstackgerrit | Monty Taylor proposed openstack-infra/shade master: Support v4-fixed-ip and v6-fixed-ip in create_server https://review.openstack.org/611651 | |
| 15:43:06 | Shrews | mordred: uh... that fix is actually worse? | |
| 15:43:12 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Support v4-fixed-ip and v6-fixed-ip in create_server https://review.openstack.org/611652 | |
| 15:43:18 | mordred | Shrews: it is? | |
| 15:43:42 | mordred | Shrews: both port and port-id apparently go to 'port' | |
| 15:44:00 | Shrews | mordred: oh | |
| 15:44:32 | mordred | Shrews: what if I just makeall the parameters go to a one called "parameter" | |
| 15:44:56 | Shrews | i like that | |
| 15:45:00 | Shrews | easy to code to | |
| 15:45:10 | mordred | RIGHT? much easier than all of these different names | |
| 15:46:11 | mordred | and you can tell it things by putting markers into the parameter content - so like value['parameter'] = '<key>network</key>value:{network-name}EOL' | |
| 15:46:46 | mordred | but to save memory we sohuld really gzip the value first | |
| 15:47:11 | dtantsur | base64, then gzip | |
| 15:47:25 | dtantsur | hmm, the other way around | |
| 15:47:31 | dtantsur | oh both? | |
| 15:47:35 | Shrews | def both | |
| 15:48:21 | mordred | both for sure | |
| 15:48:42 | mordred | actually - for extra security, I thin you want to base64, then gzip, then base64 - then rot13 for good measure | |
| 15:50:01 | dtantsur | and pack it all into SOAP for enterprise friendliness | |
| 15:50:12 | Shrews | and yubikey 2FA so the user has to continually interact with the dongle | |
| 15:50:14 | mordred | maybe take the base64, turn it into an array of uint64, then swap the endianness of each uint64 | |
| 15:50:31 | mordred | we should start a security consulting company | |
| 15:51:34 | openstackgerrit | Merged openstack/openstacksdk master: Add stackviz processing to functional tests https://review.openstack.org/610167 | |
| 16:00:17 | edleafe | The API-SIG Office Hour has officially started | |
| 16:00:25 | edleafe | (please hold your applause) | |
| 16:34:37 | elmiko | XD | |
| 17:35:55 | eandersson | Awesome thanks mordred | |
| 18:06:24 | mordred | kmalloc, lbragstad: either of you have a sec to look at https://review.openstack.org/#/c/609604 ? | |
| 18:06:57 | lbragstad | mordred oh - yes... i need to look at that anyway | |
| 18:07:00 | mordred | mostly - thepatch looks reasonable, but I don't know much about that api | |
| 18:07:13 | mordred | same with https://review.openstack.org/#/c/609572/ | |
| 18:07:50 | mordred | also - what the heck is a region_id? | |
| 18:08:10 | mordred | I don't want to know do I? | |
| 18:12:11 | lbragstad | mordred what questions do you have about that API? | |
| 18:13:44 | mordred | lbragstad: just - do those patches look right? | |
| 18:13:48 | mordred | in terms of exposing it? | |
| 18:21:00 | openstackgerrit | Merged openstack-infra/shade master: Support v4-fixed-ip and v6-fixed-ip in create_server https://review.openstack.org/611651 | |
| 18:49:55 | lbragstad | mordred sorry about that - apparently my network connection dies when i disconnect from power | |
| 18:51:09 | mordred | lbragstad: silly you | |
| 18:51:21 | mordred | lbragstad: have you considered not ever disconnecting from power? | |
| 18:51:58 | lbragstad | i am no | |
| 18:51:59 | lbragstad | now* | |
| 18:53:05 | lbragstad | mordred i might have to verify those patches manually | |
| 18:53:11 | lbragstad | but they do look correct | |
| 18:53:48 | mordred | lbragstad: I'm probably good with them looking correct for now - the interface seems reasonable enough | |
| 18:54:00 | lbragstad | one thing to note | |
| 18:54:09 | lbragstad | those APIs in keystone are still marked as experimental | |
| 18:54:22 | lbragstad | (the limits and registered limits APIs) | |
| 18:55:30 | lbragstad | i'm not sure if that impacts merging things to sdk | |
| 18:55:51 | lbragstad | fwiw - we have implemented support in python-keystoneclient and python-openstackclient | |
| 19:01:00 | kmalloc | mordred: on my list post lunch. | |