Earlier  
Later
New Search
Posted Nick Remark
#openstack-sdks - 2017-03-20
21:04:21 reedip sindhu : hey do you know the VPNaaS patch for OSC ?
21:05:06 reedip got it sindhu
21:09:04 sindhu reedip: ok
21:09:25 dtroyer ZZelle: about the option name, we have been consistent with how we use —fixed-ip, with the kv pairs as you mentioned, but I'd rather not create a new option name for this. We've used —fixed-ip-address in the floating IP commands and it takes only an IP address as an argument. How do you feel about that?
21:11:53 openstackgerrit Adrian Otto proposed openstack/service-types-authority master: WIP: Add coe-cluster service type https://review.openstack.org/447694
21:12:14 ZZelle dtroyer, honestly i have no opinion if fixed-ip-address has been somewhere else let's go for it
21:14:47 dtroyer cool…leaving comments
21:16:23 dtroyer heh, on huanxuan's other comment about the relnote name, I do find the 'bug-####' naming helpful as I am going through the release notes cleaning things up. no need to change, just thought I'd point out where I do find them meaningful.
21:22:36 openstackgerrit Cedric Brandily proposed openstack/python-openstackclient master: Enable to specify which vm fixed-ip to publish https://review.openstack.org/371786
21:25:46 ZZelle dtroyer, ^^
21:31:54 openstackgerrit Merged openstack/python-openstackclient master: Introduce Neutron DHCP agent commands to OSC https://review.openstack.org/387611
21:32:12 dtroyer ZZelle: could you make the help string match the doc? "Fixed IP address to associate with this floating IP address"
21:34:07 openstackgerrit Cedric Brandily proposed openstack/python-openstackclient master: Enable to specify which vm fixed-ip to publish https://review.openstack.org/371786
21:34:08 ZZelle dtroyer, ^^
21:34:33 dtroyer cool, thanks
21:56:20 openstackgerrit Dean Troyer proposed openstack/python-openstackclient master: Simplify logic around option lists in port set https://review.openstack.org/439048
23:20:53 openstackgerrit Hongbin Lu proposed openstack/service-types-authority master: Add container service type for Zun https://review.openstack.org/447725
#openstack-sdks - 2017-03-21
01:50:17 openstackgerrit Merged openstack/python-openstackclient master: Jsonify meter and meter rule functional tests https://review.openstack.org/439854
01:55:51 openstackgerrit Merged openstack/python-openstackclient master: Enable to specify which vm fixed-ip to publish https://review.openstack.org/371786
01:57:25 openstackgerrit Qiming Teng proposed openstack/python-openstacksdk master: Add StackFiles resource to orchestration v1 https://review.openstack.org/446904
02:40:22 openstackgerrit Huanxuan Ao proposed openstack/python-openstackclient master: Support to add/remove multi users for "group add/remove user" https://review.openstack.org/446848
02:41:00 reedip stevemar, sindhu :ping ?
02:43:26 openstackgerrit Reedip proposed openstack/python-openstackclient master: Fix Trivial Changes in [1] https://review.openstack.org/447764
03:32:48 openstackgerrit Merged openstack/python-openstackclient master: OSC Network Flavor Profile https://review.openstack.org/396783
03:56:05 openstackgerrit Merged openstack/python-openstackclient master: OSC Quota List https://review.openstack.org/379813
04:44:24 openstackgerrit qtlu proposed openstack/osc-lib master: Remove log translations https://review.openstack.org/447793
05:57:50 openstackgerrit wangzhenyu proposed openstack/python-openstackclient master: Remove log translations https://review.openstack.org/447838
06:44:01 openstackgerrit yfzhao proposed openstack/python-openstackclient master: Remove log translations https://review.openstack.org/447888
07:38:35 openstackgerrit Reedip proposed openstack/python-openstackclient master: "floating ip set/unset port" for OSC https://review.openstack.org/383025
07:49:36 reedip RuiChen : there>
08:09:00 openstackgerrit Reedip proposed openstack/python-openstackclient master: TrivialFix : Introduce FindFloatingIP() without ip_Cache https://review.openstack.org/447938
08:09:07 reedip RuiChen : new patch , just for you :) ^^
08:12:27 openstackgerrit Reedip proposed openstack/python-openstackclient master: TrivialFix : Introduce FindFloatingIP() without ip_Cache https://review.openstack.org/447938
08:15:37 openstackgerrit Reedip proposed openstack/python-openstackclient master: "floating ip set/unset port" for OSC https://review.openstack.org/383025
08:26:10 openstackgerrit wangzhenyu proposed openstack/python-openstackclient master: Remove log translations https://review.openstack.org/447838
08:29:28 openstackgerrit Merged openstack/python-openstackclient master: Fix Trivial Changes in [1] https://review.openstack.org/447764
09:23:27 openstackgerrit yfzhao proposed openstack/python-openstackclient master: Remove log translations https://review.openstack.org/447888
10:08:16 openstackgerrit Stephen Finucane proposed openstack/python-openstackclient master: Don't duplicate commands https://review.openstack.org/447592
13:39:31 openstackgerrit Monty Taylor proposed openstack/os-client-config master: Add designateclient to constructors list https://review.openstack.org/448100
14:19:36 openstackgerrit Sindhu Devale proposed openstack/python-openstackclient master: "floating ip set/unset port" for OSC https://review.openstack.org/383025
14:21:17 sindhu reedip: hey, u pinged yday?
14:21:48 reedip sindhu : I dont remember ...
14:22:00 reedip too many things were happening
14:22:06 reedip OSC, neutron etc etc etc :F
14:22:07 reedip :D
14:22:18 sindhu lol :D
14:43:51 openstackgerrit Sindhu Devale proposed openstack/python-openstackclient master: OSC Extension Show https://review.openstack.org/382023
14:46:28 reedip sindhu : why is the Blueprint named extension-show ?? when it tackles almost everything for extension ^^
14:46:33 openstackgerrit Shashank Kumar Shankar proposed openstack/python-openstackclient master: Introduce neutron flavor associate, disassociate to OSC https://review.openstack.org/403907
14:50:25 sindhu reedip: It handles only show
14:50:49 sindhu reedip: everything?
14:50:51 reedip aah.... 2 meetings and code review dont work well !
14:50:58 reedip I thought it handled list as well
14:51:25 sindhu reedip: :)
15:19:11 openstackgerrit Merged openstack/python-openstacksdk master: Avoid imports in openstack/__init__.py https://review.openstack.org/446823
15:19:27 openstackgerrit Merged openstack/python-openstacksdk master: Expose ha_state property from HA enabled L3 Agents. https://review.openstack.org/447050
15:23:43 openstackgerrit Merged openstack/python-openstacksdk master: Add StackFiles resource to orchestration v1 https://review.openstack.org/446904
17:20:48 openstackgerrit Merged openstack/python-openstackclient master: Support to add/remove multi users for "group add/remove user" https://review.openstack.org/446848
17:30:04 openstackgerrit David Rabel proposed openstack/python-openstackclient master: Add --network and --port to server create https://review.openstack.org/444924
21:00:02 sindhu dtroyer: stevemar : https://review.openstack.org/#/c/382023/ is ready for review :)
23:47:02 openstackgerrit Merged openstack/osc-lib master: Fix find_resource exception handling on numeric names with kwargs https://review.openstack.org/445679
23:53:51 openstackgerrit Merged openstack/osc-lib master: Updated from global requirements https://review.openstack.org/440084
23:56:50 reedip stevemar , sindhu : u there
23:57:09 reedip ankur-gupta-f4 : u too
#openstack-sdks - 2017-03-22
00:00:48 reedip stevemar, sindhu, ankur-gupta-f4 , RuiChen : need ur reviews on https://review.openstack.org/#/c/447938/2
00:01:57 ankur-gupta-f4 reedip: I agree with Sindhu adding that patchset within the FIP/port patch
00:02:12 reedip I brought it out of the Floating IP associate/disassociate patch bcz the change here is not related with that patch
00:02:39 reedip Its a separate piece of code which would also impact other locations where the code is used.
00:02:57 ankur-gupta-f4 okay makes sense
00:03:04 reedip If it would have been directly related to the floating ip associate/disassociate, I would have been the first one to put it there :)
00:07:41 sindhu reedip: hey, I still feel it can be added in the fip patch, then there is no need of any dependency and even fip patch can get merged soon rather than wait for this one. anyway this method is currently not being used anywhere else in the code
00:08:15 reedip sindhu : no not the new method, but there is a good amount of code which uses ip_cache but ignores it
00:08:32 reedip sindhu : like show floating ip, delete floating ip etc
00:08:53 reedip sindhu : the code you are working on is dependent on this change but this change also spawns other code changes
00:09:06 reedip sindhu :thats why I avoided mixing the 2
00:09:50 reedip the old method has some issues, and if you do a find, you will see that there are code sections which use the old function, get the ip_Cache, but then do not use it
00:10:45 sindhu reedip: are u planning to make changes to show and delete in the same patch?
00:11:52 reedip sindhu : I think I can do it in this patch or in a follow up patch . I wanted to get this patch on road so that the floating IP patch can move forward. But if you think its corret then I can make the changes in Show and Delete as well
00:12:50 reedip thats not a big issue, I guess. The point which dtroyer and stevemar had was that this function was not actually pretty good, but it can still be used where ip_cache is actually required
00:13:32 reedip therefore I spun up a new function so that old code isnt hampered, the ip_Cache function still exists and we can just move on
00:13:43 sindhu reedip: I feel, it makes sense to have a new patch only if ur changing the rest of the code base (show and delete). it its just defining the new method, then might as well do it in the fip patch so it gets merged together
00:14:19 reedip sindhu : okay, got your point. Wait, lemme spin the new version
00:15:33 sindhu reedip: gtg. will look at it again tom
00:15:44 reedip Okay ... have a great day :)
00:17:03 openstackgerrit Reedip proposed openstack/python-openstackclient master: TrivialFix : Introduce FindFloatingIP() without ip_Cache https://review.openstack.org/447938
00:17:10 reedip sindhu : jfyi , done :P
01:22:47 openstackgerrit Merged openstack/os-client-config master: Add support for bailing on invalid service versions https://review.openstack.org/439119
02:28:07 openstackgerrit qtlu proposed openstack/osc-lib master: Using assertIsNone() instead of assertEqual(None) https://review.openstack.org/448346
03:52:53 openstackgerrit Reedip proposed openstack/python-openstackclient master: TrivialFix : Introduce FindFloatingIP() without ip_Cache https://review.openstack.org/447938
05:55:54 reedip o/
10:07:38 openstackgerrit Reedip proposed openstack/python-openstackclient master: Introduce FindFloatingIP() without ip_Cache https://review.openstack.org/447938
12:32:17 reedip__ o/
13:24:02 openstack Launchpad bug 1674192 in openstack-api-wg "Mention max length limit of tags in api-wg" [Undecided,New]
13:24:02 cdent elmiko, edleafe: some work in progress we might be able to halt sooner than later: https://bugs.launchpad.net/openstack-api-wg/+bug/1674192
13:52:57 elmiko cdent: added a comment agreeing with you and Sergey on the review
13:53:17 cdent thans elmiko
13:58:16 elmiko i'm curious though, why 60 char limit. why not 255?
14:00:58 cdent elmiko: 60 is what nova is using, so I think it was just a matter of codifying what's there
14:07:07 edleafe cdent: elmiko: added my two cents
14:07:37 edleafe I think it was my discussion with lbragstad that started this ball rolling

Earlier   Later