| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2017-03-14 | |||
| 15:14:20 | ynirk | I first thought of a keystoneclient problem (with the bug https://bugs.launchpad.net/python-keystoneclient/+bug/1671179) but after deeper analysis I think the problem is in find_resource from osc-lib/utils.py | |
| 15:14:21 | ynirk | What do you think ? | |
| 15:16:45 | breton | ynirk: > All get requests (in all openstack components) made with a numeric name and kwargs might return a TypeError if the method's prototype doesn't handle kwargs. | |
| 15:16:51 | breton | ynirk: why is that? | |
| 15:21:50 | dtroyer | ynirk: I think you may be right there | |
| 15:22:22 | dtroyer | I am trying to recall what exceptions I thought should be raised at that point | |
| 15:23:49 | dtroyer | this function is a combination of 3 or 4 of these from individual libs, so working back to that may be fun. Changing it now may also change things in unexpected ways | |
| 15:24:33 | dtroyer | the choices are to add a type(ex) check for TypeError or to remove them altogether | |
| 15:25:31 | ynirk | dtroyer yes exactly but I was not sure of the impact | |
| 15:25:33 | dtroyer | I'm going to have to think on that a bit. if you have time, you may want to try proposing temoving the if in lines 120-124 and just doing a pass there and see what tests/jobs fail. | |
| 15:26:20 | dtroyer | it would be safer to add the TypeError check though | |
| 15:28:15 | ynirk | Ok I also think it is safer and shouldn't break anything | |
| 15:28:28 | ynirk | I was wondering of the reason for the raise (compared to the other try/except) | |
| 15:31:57 | ynirk | I will try to add the TypeError check and see if some tests fail | |
| 15:32:09 | ynirk | Thanks for the feedback | |
| 15:51:11 | josecastroleon | hi | |
| 15:51:35 | josecastroleon | the gate in openstackclient is failing since devstack removed glance v1 | |
| 15:51:58 | ankur-gupta-f4 | yes | |
| 15:53:05 | josecastroleon | shall we remove the glance v1 test? | |
| 15:53:47 | ankur-gupta-f4 | https://www.irccloud.com/pastebin/p3o3Cgo2/ | |
| 15:54:05 | dtroyer | no, we need to continue to support v1 for some time, for now I'd prefer to re-enable Glance v1 in DevStack | |
| 15:54:20 | dtroyer | at least for a time | |
| 15:54:36 | ankur-gupta-f4 | stevemar suggested doing a check first. If exception raised then skip tests | |
| 15:54:45 | josecastroleon | fine, but then we may need to enable glance v1 in the gate | |
| 15:54:48 | josecastroleon | right? | |
| 15:55:14 | dtroyer | yes, that's on my list, I'm fixing cliff's gate ATM | |
| 15:55:23 | josecastroleon | ok thx | |
| 16:04:59 | openstackgerrit | Dean Troyer proposed openstack/cliff master: Update cmd2 fix to still work with 0.6.7 https://review.openstack.org/445552 | |
| 16:06:10 | dtroyer | josecastroleon, ankur-gupta-f1: https://review.openstack.org/#/c/445553/ re-enables Image v1 for OSC functional jobs | |
| 16:18:55 | openstackgerrit | Merged openstack/golang-client master: Update for CTI ps4 https://review.openstack.org/444499 | |
| 16:27:55 | openstackgerrit | Bence Romsics proposed openstack/python-openstacksdk master: Add port property: trunk_details https://review.openstack.org/445501 | |
| 16:30:59 | ynirk | dtroyer tests pass with the two modifications. To be sure I will add the TypeError check | |
| 16:43:36 | reedip | sindhu : hi | |
| 16:44:45 | sindhu | reedip: hey wass up | |
| 16:45:12 | reedip | nothin much sindhu : just wanted to know about the floating ip patch | |
| 16:45:30 | sindhu | reedip: it's all ready right? | |
| 16:45:53 | reedip | no comments from anyone else, I guess | |
| 16:45:56 | reedip | :| | |
| 16:46:53 | sindhu | reedip: yeah! waiting for reviews/getting merged :) | |
| 16:47:47 | reedip | stevemar , can you come to our rescue ???? :D | |
| 18:52:40 | openstackgerrit | Merged openstack/python-openstacksdk master: Add port property: trunk_details https://review.openstack.org/445501 | |
| 18:54:37 | openstackgerrit | Merged openstack/cliff master: Update cmd2 fix to still work with 0.6.7 https://review.openstack.org/445552 | |
| 21:53:59 | ZZelle | dtroyer, hi | |
| 23:26:44 | openstackgerrit | Julien Lavesque proposed openstack/osc-lib master: Fix find_resource exception handling on numeric names with kwargs https://review.openstack.org/445679 | |
| #openstack-sdks - 2017-03-15 | |||
| 07:27:34 | openstackgerrit | Rui Chen proposed openstack/python-openstackclient master: [Compute]Make column content readable for both human and machine https://review.openstack.org/445794 | |
| 08:46:22 | openstackgerrit | Rui Chen proposed openstack/python-openstackclient master: [Compute]Make column content readable for both human and machine https://review.openstack.org/445794 | |
| 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 | |