| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2017-03-15 | |||
| 10:07:17 | openstackgerrit | lihaijing proposed openstack/python-openstackclient master: [1/3]Replace six.iteritems() with .items() https://review.openstack.org/445856 | |
| 10:32:22 | openstackgerrit | lihaijing proposed openstack/python-openstackclient master: [2/3]Replace six.iteritems() with .items() https://review.openstack.org/445865 | |
| 11:13:22 | openstackgerrit | lihaijing proposed openstack/python-openstackclient master: [3/3]Replace six.iteritems() with .items() https://review.openstack.org/445878 | |
| 11:58:39 | frickler | so it seems like the project-config patch did not fix the gate: http://logs.openstack.org/75/443075/2/check/gate-osc-dsvm-functional-ubuntu-xenial/5eb7ede/console.html#_2017-03-15_11_12_51_996706 | |
| 12:01:04 | openstackgerrit | Chris Dent proposed openstack/api-wg master: Refactor and re-validate api change guidelines https://review.openstack.org/421846 | |
| 12:06:05 | frickler | I guess https://review.openstack.org/445900 would work better | |
| 16:29:29 | benoit | hello | |
| 16:30:42 | benoit | i have a strange behavior using openstacksdk (network class) : i want to retrieve project_id for a network but it always return None | |
| 16:31:49 | benoit | using **query parameters with project_id also seems to not be taken in account | |
| 16:32:28 | benoit | is this parameter set ? or do i miss something ? | |
| 16:36:38 | cdent | sdague: you have an opinion on what should happen here: https://review.openstack.org/#/c/434999/ there's a conflict between the value of the service type and already existing documentation | |
| 17:03:09 | reedip | benoit : whats the request you are sending and whats the response u r getting | |
| 17:05:25 | benoit | reedip, just digging into RESP BODY, it has tenant_id not project_id | |
| 17:05:53 | benoit | reedip, on a more recent openstack, project_id is set and sdk is working as expected | |
| 17:08:07 | benoit | (first openstack is a liberty, second a newton) not sure how to use openstacksdk with liberty then | |
| 17:41:04 | dtroyer | benoit: what you are seeing is a difference between Identity v2 and v3 behaviour | |
| 17:41:14 | dtroyer | re tenant vs project | |
| 17:41:37 | dtroyer | the SDK is supposed to hide much of that from you | |
| 17:42:39 | briancurtin | benoit: can you share some code? what you’re talking about doesn’t really make sense to me to figure out what’s going on | |
| 17:44:41 | briancurtin | the things that come back in a response body aren’t names you’re going to use, that’s raw data out of the server | |
| 17:48:58 | briancurtin | also, project_id actually has to be a query parameter on the REST end of it in order to send it, and we have to have added support for this, so you can’t arbitrarily add query parameters. if something is documented to accept project_id as a query parameter we can add it, of course | |
| 18:23:10 | openstackgerrit | Pepijn Oomen proposed openstack/python-openstacksdk master: Expose OS-EXT-SRV-ATTR:{hypervisor_hostname,instance_name} for Server. https://review.openstack.org/446104 | |
| 18:44:48 | kfowler | where can I find good documentation on the openstack sdks | |
| 18:46:28 | kfowler | I've been able to authenticate to our openstack environment using token auth and query nova successfully. I'm having trouble using the heat client though to create new stacks | |
| 19:28:56 | openstackgerrit | Jim Rollenhagen proposed openstack/api-wg master: Microversions: add next_min_version field in version body https://review.openstack.org/446138 | |
| 19:30:21 | briancurtin | kfowler: you’re best off going to the channel for the project you’re having a problem with, i think #openstack-heat probably knows | |
| 19:57:55 | kfowler | thanks, I'll try there | |
| 20:29:25 | rabel | hi everyone. any news from the broken functional tests? | |
| 20:35:29 | openstackgerrit | Jim Rollenhagen proposed openstack/api-wg master: Microversions: add next_min_version field in version body https://review.openstack.org/446138 | |
| 20:39:24 | sshank | Are the client gates unstable? | |
| 20:41:54 | frickler | rabel: sshank: should have been fixed now, feel free to recheck. glance v1 has been reenabled for them | |
| 20:42:17 | sshank | frickler, Cool. Thanks, | |
| 20:42:33 | rabel | frickler thanks | |
| 20:50:53 | openstackgerrit | Jim Rollenhagen proposed openstack/api-wg master: Microversions: add next_min_version field in version body https://review.openstack.org/446138 | |
| 21:01:57 | openstackgerrit | Merged openstack/python-openstacksdk master: Expose OS-EXT-SRV-ATTR:{hypervisor_hostname,instance_name} for Server. https://review.openstack.org/446104 | |
| 21:04:48 | pepijn | \o/ | |
| #openstack-sdks - 2017-03-16 | |||
| 02:00:20 | openstackgerrit | lihaijing proposed openstack/python-openstackclient master: [3/3]Replace six.iteritems() with .items() https://review.openstack.org/445878 | |
| 03:21:46 | openstackgerrit | Reedip proposed openstack/python-openstackclient master: "floating ip set/unset port" for OSC https://review.openstack.org/383025 | |
| 03:22:13 | reedip | ankur-gupta-f4 : Nit-picked :D ^^ | |
| 03:25:11 | ankur-gupta-f4 | reedip: I try. Depends on which side of the bed I wake up in the morning :P | |
| 03:25:49 | reedip | ankur-gupta-f4 : I hope you wake up straight tomorrow than on any side :D | |
| 06:56:29 | openstackgerrit | Rui Chen proposed openstack/python-openstackclient master: Replace "Display Name" by "Name" in volume list https://review.openstack.org/423081 | |
| 07:03:01 | rabel | hi there. image create functional test is working, but now other functional tests are failing, that seem to have nothing to do with the change. could someone help with this issue? https://review.openstack.org/#/c/444924/ | |
| 07:04:14 | reedip | recheck rabel ? | |
| 07:04:55 | rabel | reedip: yes ok. i just thought, maybe the tests are still broken | |
| 07:06:42 | reedip | It happened with me as well rabel , i just rechecked | |
| 07:07:51 | openstackgerrit | Rui Chen proposed openstack/python-openstackclient master: Replace "Display Name" by "Name" in volume list https://review.openstack.org/423081 | |
| 07:08:09 | rabel | reedip: ok, i do the same then. thanks | |
| 07:34:19 | openstackgerrit | Rui Chen proposed openstack/python-openstackclient master: Make block-device-mapping more stable and clear https://review.openstack.org/438141 | |
| 08:13:50 | openstackgerrit | Rui Chen proposed openstack/osc-lib master: Add formattable column base classes https://review.openstack.org/444760 | |
| 08:58:49 | openstackgerrit | Merged openstack/python-openstackclient master: Add the bgp agent type to network agent command https://review.openstack.org/443075 | |
| 09:49:55 | benoit | briancurtin, you can find some code with execution here : http://pastebin.com/QxFPn2Mp - tell me if you need more | |
| 10:00:44 | frickler | So I was too optimistic earlier, devstack did put mysql on a diet causing it to starve when being fed OSC food, but now things should finally been green again | |
| 10:16:18 | frickler | dtroyer: there is this error happening sometimes now, maybe your other ds variables also need to be moved into DEVSTACK_LOCAL_CONFIG? http://logs.openstack.org/41/438141/3/check/gate-osc-dsvm-functional-ubuntu-xenial/661985e/logs/screen-c-vol.txt.gz#_2017-03-16_08_13_50_158 | |
| 10:22:12 | rabel | meeting would be today at 19:00 UTC, right? | |
| 10:23:42 | rabel | i mean osc meeting | |
| 11:33:56 | RuiChen | rabel: I saw the latest IRC meeting log is in 2017-02-02, seems no meeting in the past few weeks, I'm not sure today would be :( | |
| 11:40:37 | RuiChen | reedip, rabel : volume always fall into error status randomly in osc functional tests, I face it several times, it's same in your patch? | |
| 11:41:00 | reedip | Seems to be RUiChen | |
| 11:41:11 | reedip | RuiChen* | |
| 11:42:09 | RuiChen | not sure the root reason, but I think might we can rework for volume test cases, and add some wait_for or status check | |
| 11:46:40 | rabel | RuiChen: it would be great to have less random test fails | |
| 11:49:17 | RuiChen | rabel: agree, but I think we can't insure the volume creating success everytime in osc side :( | |
| 11:50:35 | rabel | RuiChen: I wonder why it does not work all the time. Do you know that? It seems not to be a good test, if it fails randomly from time to time. ;) | |
| 11:52:49 | RuiChen | rabel: not sure the root reason, I just free it happen frequently in recent | |
| 11:53:06 | rabel | yes | |
| 11:57:36 | frickler | rabel: RuiChen: I think this is due to an error in project-config, see my comment to dtroyer earlier | |
| 11:58:25 | rabel | frickler: ah ok | |
| 11:59:49 | RuiChen | thank you, frickler | |
| 12:05:17 | openstackgerrit | Bence Romsics proposed openstack/python-openstackclient master: Make MAC address of port updatable https://review.openstack.org/442543 | |
| 12:48:03 | RuiChen | Thank you frickler and rabel , functional tests would be happy when the patch merged https://review.openstack.org/446501 : ) | |
| 12:49:32 | rabel | RuiChen: nice, thank you | |
| 12:50:57 | RuiChen | rabel: np, I should go home, good night. | |
| 12:51:21 | rabel | RuiChen: what time is it at your place? | |
| 12:51:39 | RuiChen | PM9:00 | |
| 12:51:57 | rabel | yeah, than you should go home ;) | |
| 12:52:13 | rabel | good night. and thanks for the patch :) | |
| 12:52:29 | RuiChen | bye : ) | |
| 12:55:30 | amotoki | hi, questions on OSC command convention. most OSC commands take <name> as required optional parameters even though 'name' is optional in API. | |
| 12:56:15 | amotoki | is it a recommended way in OSC convention? I personally agree this from user perspective, but some *developers* says it is optional so would like to use --name. | |
| 12:56:22 | amotoki | I'd like to follow the convention. | |
| 12:57:13 | amotoki | one more question on similar topic. is there any guideline on required options. there are two choices: positional arg and required --xxxx arg. | |
| 12:58:15 | amotoki | IMHO it is better to use required --xxx arg since it is better to understand. Having several positional args is not easy to understand / remember. | |
| 13:01:49 | openstackgerrit | Merged openstack/python-openstackclient master: Switch to use cleanup method in functional test https://review.openstack.org/444090 | |
| 13:01:56 | openstackgerrit | Merged openstack/python-openstackclient master: Add sort support to project list https://review.openstack.org/335204 | |
| 13:02:03 | openstackgerrit | Merged openstack/python-openstackclient master: Narrow expected responses for CheckUserInGroup https://review.openstack.org/445257 | |
| 13:02:17 | openstackgerrit | Merged openstack/python-openstackclient master: Fix "security group list" command to display project ID properly https://review.openstack.org/437320 | |
| 13:29:09 | dtroyer | amotoki: there are guidelines on the formation of commands in https://docs.openstack.org/developer/python-openstackclient/commands.html. To answer your questions directly: | |
| 13:30:04 | dtroyer | a) in create commands 'name' is the only positional argument and is not optional, unless there is absolutley no place to map a resource name (compute flavor) in which case the positional is an ID | |
| 13:30:42 | dtroyer | everywhere else the resource is identified name or ID is accepted and OSc does the lookup if the API is not capable of that | |
| 13:32:17 | dtroyer | b) the only commands thus far that take multiple positional arguments are add/remove where two resources need to be identified. The pattern is maintained as in 'XXX add YYY' command uses the name/id of XXX and YYY as the positionals, in that order | |
| 13:33:26 | dtroyer | everything else uses options, and 'required option' is strange (an oxymoron) but maintains the pattern. | |
| 13:34:34 | dtroyer | also, the OSC Human Interface Guide is at https://docs.openstack.org/developer/python-openstackclient/humaninterfaceguide.html which has more details on all of the OSC UI | |
| 13:36:08 | dtroyer | frickler, RuiChen: thanks for beating me to the fixup of the functional jobs | |
| 14:06:42 | amotoki | dtroyer: thanks for the detail clarification. I already read the human interface guide but my understanding was not enough. I will visit it again for better understanding. | |
| 15:52:50 | openstackgerrit | Merged openstack/python-openstackclient master: Add a validation about options for server migrate command https://review.openstack.org/439451 | |
| 16:31:42 | openstackgerrit | Merged openstack/python-openstackclient master: Functional test for ip_availability https://review.openstack.org/444135 | |
| 16:33:14 | edleafe | cdent: elmiko: newsletter is ready for your edits: https://etherpad.openstack.org/p/api-wg-newsletter | |
| 16:44:42 | elmiko | edleafe: ack, looking | |
| 16:45:24 | elmiko | edleafe: +1 ship it! | |