| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2017-03-28 | |||
| 19:41:55 | efried | nope, cause I'm a doofus. I was spelling 2.37 as 2.7. | |
| 19:51:08 | efried | ankur-gupta-f4 dtroyer Apparently novaclient will detect and use the latest possible version. But openstackclient will default to 2.1 if not otherwise specified via OS_COMPUTE_API_VERSION/--os-compute-api-version: https://github.com/openstack/python-openstackclient/blob/master/openstackclient/compute/client.py#L26 | |
| 20:00:49 | openstackgerrit | Merged openstack/os-client-config master: Remove out of date comment https://review.openstack.org/450871 | |
| 20:16:50 | efried | Philosophically, is there a reason openstackclient doesn't detect and use the latest? | |
| 20:18:04 | dtroyer | no, it should, but there is some trickiness with microversions, and with the upcoming novaclient 8.0 release that'll break us like it breaks Horizon. | |
| 20:20:29 | efried | Breaks who? The openstackclient... uh, client? | |
| 20:21:24 | dtroyer | yup, we'll be supporting nova-net for a lot longer than Novaclient | |
| #openstack-sdks - 2017-03-29 | |||
| 01:51:56 | openstackgerrit | Ethan Lynn proposed openstack/python-openstacksdk master: Add vlan_transparent property to network resource https://review.openstack.org/450080 | |
| 02:44:55 | openstackgerrit | Merged openstack/python-openstacksdk master: Add vlan_transparent property to network resource https://review.openstack.org/450080 | |
| 03:05:11 | reedip | dtroyer, stevemar there / | |
| 03:16:15 | openstackgerrit | Merged openstack/python-openstacksdk master: Find floating ip by ip address https://review.openstack.org/449879 | |
| 03:19:03 | openstackgerrit | Takashi NATSUME proposed openstack/python-openstackclient master: Add '--no-live' and '--force' option in 'server migrate' https://review.openstack.org/427565 | |
| 04:16:07 | openstackgerrit | Takashi NATSUME proposed openstack/python-openstackclient master: List/Show all server migration types https://review.openstack.org/450119 | |
| 06:53:16 | openstackgerrit | Takashi NATSUME proposed openstack/python-openstackclient master: List/Show all server migration types https://review.openstack.org/450119 | |
| 06:56:44 | rabel | good morning | |
| 07:14:50 | reedip | good afternoon | |
| 07:48:28 | rabel | reedip: ;) | |
| 07:49:05 | reedip | :) | |
| 07:52:24 | rabel | could someone have a look at https://review.openstack.org/#/c/444924/ ? It is not of high priority, but it lies idle for a week now. | |
| 07:57:42 | reedip | rabel hi | |
| 07:57:55 | reedip | i had a patch for smething you are doing in Line#614 | |
| 07:58:05 | reedip | but didnt push it again ... need to retake it | |
| 07:58:59 | reedip | the split on , and then dict key value pair logic | |
| 07:58:59 | reedip | you can put this logic in osc-lib | |
| 07:59:34 | reedip | https://review.openstack.org/#/c/425515/ | |
| 08:00:08 | openstackgerrit | Reedip proposed openstack/osc-lib master: [WIP]Add MultiKeyValueCommaAction to osc-lib https://review.openstack.org/425515 | |
| 08:00:34 | reedip | rabel : can you check this patch , it would avoid you repeating the same code ( probably ) | |
| 08:03:58 | rabel | reedip: i will have a look at it. it would make sense to use your class for my patch once it is merged. | |
| 08:04:29 | reedip | rabel yes . though I wont like to bliock you , so you can add a ToDo for yourself | |
| 08:04:51 | reedip | that ways, everything is perfect, I guess | |
| 08:05:54 | rabel | reedip: it's true that my patch could already be merged without yours. i use the same logic as the --nic implementation. so we could change both to use the new class in a separate patch. | |
| 08:06:13 | reedip | rabel : sounds good ... lets race :D | |
| 08:07:31 | reedip | just kidding, I will push mine a bit slower , but otherwise the code looks good , I will look at it once more though in the evening | |
| 08:09:09 | rabel | :D i added me as a reviewer for your patch. i will have a look at it now and once you finished it, i will review it in detail | |
| 08:13:55 | reedip | :D | |
| 08:32:45 | rabel | reedip: let's see if i understand it right: the difference between MultiKeyValueAction and MultiKeyValueCommaAction is the latter supports something like "--my_param key1=val,ue1,key2=val,ue3", is that right? | |
| 08:33:06 | reedip | something like that | |
| 08:33:15 | reedip | actually exatly liek that | |
| 08:35:56 | rabel | than i think --nic, --network and --port should use MultiKeyValueAction, since we don't have values containing commas or multiple values for a key, i think. | |
| 08:36:14 | rabel | by the way: your patch looks good to me | |
| 08:43:38 | reedip | --network can have IPV4 address, IPv6 address and Network ID | |
| 08:44:18 | reedip | as the addresses are one-to-one, so I guess yes, the multikeyvaluecomma action may not be required | |
| 09:19:39 | RuiChen | hi dtroyer: I add comments in your patch https://review.openstack.org/#/c/450452/ | |
| 12:08:03 | dtroyer | RuiChen: thanks. FWIW, if we have not already set neutron to be the default for help we should do that so we can still do most help wihtout auth. | |
| 12:09:40 | dtroyer | it looks like that is the case… with no auth info, openstack help lists neutron commands now | |
| 14:23:46 | openstackgerrit | Jens Rosenboom proposed openstack/python-openstackclient master: Fix block-device-mapping when volume_size is empty https://review.openstack.org/451432 | |
| 14:32:50 | reedip | o/ | |
| 14:43:11 | frickler | why does e.g. "openstack server show" use compute API v2.1 by default and not something newer? | |
| 14:43:48 | frickler | that way it is not showing some information, like delete_on_termination attribute for volume attachments | |
| 14:45:29 | frickler | seems that was already discussed yesterday, so I'd just say that I'm +1 on "use the latest" | |
| 14:47:49 | openstackgerrit | Nakul Dahiwade proposed openstack/python-openstacksdk master: [WIP] Introduce Pool resource in Octavia Load Balancer Base https://review.openstack.org/449264 | |
| 16:19:26 | openstack | Launchpad bug 1677252 in python-openstackclient ""network auto allocated topology create --check-resources" Fails with: "Request requires an ID but none was found"" [Undecided,New] | |
| 16:19:26 | ankur-gupta-f4 | reedip: can you verify or not this bug: https://bugs.launchpad.net/python-openstackclient/+bug/1677252 | |
| 16:20:12 | reedip | ankur-gupta-f4 : u r lucky .. I was just drowsing :D | |
| 16:20:24 | ankur-gupta-f4 | ah well then good night :P | |
| 16:20:52 | reedip | ankur-gupta-f4 : this seems to be a bug | |
| 16:21:21 | reedip | jsut checking on master | |
| 16:23:54 | reedip | ankur-gupta-f4 : assigning it to myself with a bit of homework done, rest in another 6 hours I guess | |
| 16:24:31 | reedip | if u have any opinion, please do lemme know | |
| 16:24:52 | ankur-gupta-f4 | K. I have a patch that is blocked till its resolved | |
| 16:25:03 | reedip | ohh then wait | |
| 16:25:22 | ankur-gupta-f4 | do you know where the bug resides? | |
| 16:25:27 | reedip | SDK | |
| 16:25:33 | reedip | request preparation | |
| 16:25:41 | ankur-gupta-f4 | too lazy today to dig in | |
| 16:25:43 | ankur-gupta-f4 | okay | |
| 16:26:25 | reedip | wait | |
| 16:26:57 | reedip | http://paste.openstack.org/show/604690/ | |
| 16:27:50 | reedip | The --check resources is | |
| 16:27:52 | reedip | Does not return a topology. | |
| 16:27:52 | reedip | --check-resources Validate the requirements for auto allocated topology. | |
| 16:28:19 | reedip | ankur-gupta-f4 : I guess the current tenant ID needs to be used up | |
| 16:28:33 | reedip | for aut-allocated-topology with --check-resource | |
| 16:29:11 | ankur-gupta-f4 | so its just not passing, getting from context, the tenant/project id | |
| 16:29:23 | reedip | guess so | |
| 16:29:40 | ankur-gupta-f4 | bah. annoying. | |
| 16:29:57 | reedip | welcome to my world :D | |
| 16:30:20 | reedip | that was the last patch on _proxy | |
| 16:30:25 | reedip | https://github.com/openstack/python-openstacksdk/commit/e5068ec46aa7bb6b1738b8a64c29e44307f4ba98 | |
| 16:30:59 | reedip | seems like it fixed some. broke some | |
| 16:31:03 | reedip | maybe | |
| 16:35:49 | ankur-gupta-f4 | hmmm | |
| 16:36:04 | ankur-gupta-f4 | its weird because the fix from terry was in the last release of the SDK | |
| 16:36:13 | ankur-gupta-f4 | :( | |
| 16:36:21 | reedip | did it have the check for --check-resources? | |
| 16:36:49 | ankur-gupta-f4 | yea --check-resources his the Validate... function in auto_allocated_topology and the changes are there | |
| 16:37:30 | reedip | hmm lem edebug | |
| 16:37:53 | reedip | wait I think my code is not up to date ! | |
| 16:39:00 | reedip | I have 0.9.13 | |
| 16:39:59 | reedip | Upgraded to 0.9.14 and now checking | |
| 16:40:05 | reedip | it works ! | |
| 16:40:10 | reedip | thats weird ! | |
| 16:40:38 | ankur-gupta-f4 | yea | |
| 16:40:53 | ankur-gupta-f4 | its a matter of bumping global reqs | |
| 16:42:04 | reedip | done! | |
| 16:42:34 | reedip | He may have pulled up the master for OSC but not SDK | |
| 16:42:50 | reedip | if he is on Neutron, then lemme drop him a ping | |
| 16:47:40 | ankur-gupta-f4 | briancurtin: here? | |
| 16:48:05 | reedip | ankur-gupta-f4; that bug is now closed ( or would be soon ) | |
| 16:50:45 | ankur-gupta-f4 | k thanks | |