Earlier  
Later
New Search
Posted Nick Remark
#openstack-sdks - 2017-04-03
21:38:40 dtroyer jamielennox: when you get a sec have a look at https://review.openstack.org/#/c/452711/ and tell me "I told you that long ago" if this is what I think it is :)
21:50:39 openstackgerrit Merged openstack/python-openstackclient master: Change noauth strategy for plugin loading https://review.openstack.org/450453
22:16:13 openstackgerrit Nakul Dahiwade proposed openstack/python-openstacksdk master: Introduce Base for Octavia (load balancing) https://review.openstack.org/428414
23:04:54 jamielennox dtroyer: i was going to push this up but realized you weren't the author
23:05:01 jamielennox all you really need is:
23:05:02 jamielennox - if not self._auth_ref:
23:05:03 jamielennox - self.setup_auth()
23:05:03 jamielennox - LOG.debug("Get auth_ref")
23:05:03 jamielennox - self._auth_ref = self.auth.get_auth_ref(self.session)
23:05:03 jamielennox - return self._auth_ref
23:05:04 jamielennox + self.setup_auth()
23:05:06 jamielennox + return self.auth.get_access(session)
23:05:30 jamielennox get_access will mean it's reused if possible and there is already a guard that means setup_auth won't execute twice
23:13:07 dtroyer jamielennox: ok, thanks
#openstack-sdks - 2017-04-04
02:27:36 openstackgerrit Merged openstack/python-openstackclient master: Release notes cleanup for 3.10.0 release https://review.openstack.org/452965
02:38:59 openstackgerrit Merged openstack/python-openstackclient master: Introduce neutron flavor associate, disassociate to OSC https://review.openstack.org/403907
03:28:16 openstackgerrit Dean Troyer proposed openstack/python-openstackclient master: Help/docs cleanups: marker, limit, ip-address metavars https://review.openstack.org/452961
03:32:16 reedip o/
03:56:23 openstackgerrit Steve Martinelli proposed openstack/python-openstackclient master: Help/docs cleanups: marker, limit, ip-address metavars https://review.openstack.org/452961
04:37:21 openstackgerrit Sean McCully proposed openstack/keystoneauth master: KeystoneAuth should default to system CAFile. https://review.openstack.org/452585
04:55:58 openstackgerrit Sean McCully proposed openstack/keystoneauth master: KeystoneAuth should default to system CAFile. https://review.openstack.org/452585
06:13:33 openstackgerrit Reedip proposed openstack/python-openstackclient master: Structure FindFloatingIP() to work without ip_cache https://review.openstack.org/447938
06:38:44 reedip RuiChen : ^^
08:23:08 openstackgerrit Stephen Finucane proposed openstack/python-openstackclient master: WIP! Start using 'cliff.sphinxext' https://review.openstack.org/452861
08:32:24 openstackgerrit Cedric Brandily proposed openstack/osc-lib master: Avoid to authenticate twice https://review.openstack.org/452711
08:32:50 openstackgerrit Stephen Finucane proposed openstack/cliff master: Add 'cliff-command' Sphinx directive https://review.openstack.org/450322
08:42:24 openstackgerrit Cedric Brandily proposed openstack/osc-lib master: Avoid to authenticate twice https://review.openstack.org/452711
12:32:55 openstackgerrit Merged openstack/python-openstackclient master: Help/docs cleanups: marker, limit, ip-address metavars https://review.openstack.org/452961
13:13:55 openstackgerrit Ed Leafe proposed openstack/api-wg master: Recommend the correct HTTP method for tags https://review.openstack.org/451536
14:19:47 reedip dtroyer , stevemar : need your inputs on https://review.openstack.org/#/c/447938/
14:26:26 openstackgerrit Samriddhi proposed openstack/keystoneauth master: Updated inconsistent value of scope parameter https://review.openstack.org/452652
14:35:59 openstackgerrit Sean McCully proposed openstack/keystoneauth master: KeystoneAuth should default to system CAFile. https://review.openstack.org/452585
15:02:20 openstackgerrit Nakul Dahiwade proposed openstack/python-openstacksdk master: Introduce Base for Octavia (load balancing) https://review.openstack.org/428414
15:12:12 openstackgerrit Nakul Dahiwade proposed openstack/python-openstacksdk master: Introduce Listener for Octavia (load balancing) https://review.openstack.org/451574
15:39:30 openstackgerrit Nakul Dahiwade proposed openstack/python-openstacksdk master: Introduce Listener for Octavia (load balancing) https://review.openstack.org/451574
15:43:45 openstackgerrit Shashank Kumar Shankar proposed openstack/python-openstackclient master: Correct flavor associate/disassociate commands in Mapping Guide https://review.openstack.org/453232
15:45:50 openstackgerrit Shashank Kumar Shankar proposed openstack/python-openstackclient master: Correct flavor associate/disassociate commands in Mapping Guide https://review.openstack.org/453232
16:12:12 openstackgerrit Sindhu Devale proposed openstack/python-openstacksdk master: Introduce HealthMonitor for Octavia (load balancing) https://review.openstack.org/453243
16:15:44 openstackgerrit Dean Troyer proposed openstack/python-openstackclient master: Enable to specify which fixed-ip to add to a vm. https://review.openstack.org/452235
16:27:25 openstackgerrit Sindhu Devale proposed openstack/python-openstackclient master: Structure FindFloatingIP() to work without ip_cache https://review.openstack.org/447938
17:25:22 openstackgerrit Sindhu Devale proposed openstack/python-openstackclient master: "floating ip set/unset port" for OSC https://review.openstack.org/383025
18:00:53 sindhu dtroyer: stevemar : https://review.openstack.org/#/c/447938/ is ready :)
18:19:00 openstackgerrit Merged openstack/python-openstackclient master: Correct flavor associate/disassociate commands in Mapping Guide https://review.openstack.org/453232
19:12:12 openstackgerrit Nakul Dahiwade proposed openstack/python-openstacksdk master: Introduce Pool for Octavia (load balancing) https://review.openstack.org/449264
20:10:27 openstackgerrit ayoung proposed openstack/python-openstackclient master: Implied Roles https://review.openstack.org/290253
20:16:11 openstackgerrit Shashank Kumar Shankar proposed openstack/python-openstacksdk master: Introduce Pool for Octavia (load balancing) https://review.openstack.org/449264
20:31:15 openstackgerrit Merged openstack/python-openstackclient master: Structure FindFloatingIP() to work without ip_cache https://review.openstack.org/447938
20:32:46 sindhu dtroyer: stevemar : https://review.openstack.org/#/c/383025/ is ready as well :)
20:40:19 openstackgerrit Nakul Dahiwade proposed openstack/python-openstacksdk master: Introduce Member for Octavia (loadbalancing) https://review.openstack.org/451488
20:54:52 openstackgerrit ayoung proposed openstack/python-openstackclient master: Implied Roles https://review.openstack.org/290253
21:16:56 dtroyer sindhu: in 383025, what is the SDK dependency mentioned?
21:18:55 sindhu dtroyer: Rui introduced this change in sdk https://review.openstack.org/#/c/449879/2
21:19:10 sindhu dtroyer: which is needed by 383025
21:19:29 dtroyer has it been released yet?
21:19:42 sindhu dtroyer: but, that won't be available till it is released
21:19:49 sindhu dtroyer: nope
21:20:28 sindhu dtroyer: so, reedip's patch which got merged now helps
21:20:47 sindhu dtroyer: it removes the use of ip_cache
21:21:08 sindhu dtroyer: once sdk is released, this work around has to be fixed
21:22:00 dtroyer can you make those dependencies clear in the commit message?
21:22:14 dtroyer I can't tell right now what needs to happen in what oder
21:22:30 dtroyer I'll have a couple of other comments in a few minutes to post
21:22:46 sindhu dtroyer: sure, i'll add it :)
21:31:17 dtroyer sindhu: ok, comments in
21:32:03 sindhu dtroyer: great, thank you :)
22:35:39 openstackgerrit Nakul Dahiwade proposed openstack/python-openstacksdk master: [WIP] Introduce Member for Octavia (loadbalancing) https://review.openstack.org/451488
23:54:26 openstackgerrit Stephen Finucane proposed openstack/python-openstackclient master: WIP! Start using 'cliff.sphinxext' https://review.openstack.org/452861
23:55:04 openstackgerrit Stephen Finucane proposed openstack/python-openstackclient master: WIP! Start using 'cliff.sphinxext' https://review.openstack.org/452861
#openstack-sdks - 2017-04-05
00:02:08 openstackgerrit Stephen Finucane proposed openstack/cliff master: Add 'cliff-command' Sphinx directive https://review.openstack.org/450322
00:02:08 openstackgerrit Stephen Finucane proposed openstack/cliff master: Add 'cliff-commands' Sphinx directive https://review.openstack.org/453383
01:13:59 reedip o/
01:49:50 openstackgerrit yangweiwei proposed openstack/python-openstackclient master: Update endpoint-delete and endpoint-show method exception. https://review.openstack.org/453429
02:32:33 openstackgerrit yangweiwei proposed openstack/python-openstackclient master: Update endpoint-delete and endpoint-show method exception. https://review.openstack.org/453429
02:55:12 openstackgerrit Merged openstack/python-openstackclient master: Enable to specify which fixed-ip to add to a vm. https://review.openstack.org/452235
07:03:41 rabel good morning
07:14:04 rabel someone can help me with a question about argparse?
07:15:25 rabel in https://review.openstack.org/#/c/444924/4/openstackclient/compute/v2/server.py@612 cedric proposes that --nic, --network and --port should all add their values to the same variable. but is this even possible? or doable in a clean way?
07:23:49 openstackgerrit Reedip proposed openstack/python-openstackclient master: Introduce quota unset command https://review.openstack.org/376311
07:37:04 reedip rabel : hi
07:37:35 rabel hi reedop
07:37:40 rabel *reedip. sry ;)
07:37:52 reedip lol , no worries .. o and I are very near
07:38:22 reedip rabel : why are we using different options here? Cant we use MutliKeyValue Action ?
07:38:38 reedip I mean network, port and nic, all can be optional
07:38:44 reedip in the MultiKeyValue Action
07:38:54 reedip and it allows the user to specify them as a dict
07:39:04 reedip and remove the trouble which you guys are facing, right ?
07:39:43 rabel i'm not sure if i understand. we can use MultiKeyValueAction for --network, --port and --nic and then have them all in one?
07:39:46 reedip we can have --network net=Network_ID,port=Port_Id,nic=Nic_ID
07:39:53 reedip yes
07:40:44 reedip so that ways, we get a dictionary , with {"net":Network_ID, "port":Port_ID, "nic":Nic_ID} and handle the internals ourselves as we see fit
07:41:05 reedip though merging things in the same variable is POSSIBLE
07:41:14 rabel but that's actually what we don't want.
07:41:21 reedip but I agree that there are cleaner options
07:41:39 reedip rabel : okay
07:41:48 rabel what you propose is basically the current solution with --nic net-id=,port=,...
07:41:59 rabel the patch tries to simplify that
07:42:26 rabel it is possible, but not in a clean way?
07:43:02 reedip then the second option is to keep it in different metavar and then handle it yourself
07:43:18 reedip I mean keep nic, port and network in separate metavars

Earlier   Later