Friday, 2015-03-20

*** Aish has joined #openstack-lbaas00:05
*** vivek-eb_ has quit IRC00:14
*** banix has quit IRC00:23
*** mlavalle has quit IRC00:31
*** jamiem has quit IRC00:40
*** mwang2 has quit IRC00:45
*** fnaval has quit IRC00:54
*** fnaval has joined #openstack-lbaas01:01
*** banix has joined #openstack-lbaas01:06
*** apuimedo has quit IRC01:12
*** vivek-ebay has joined #openstack-lbaas01:20
*** banix has quit IRC01:22
*** vivek-ebay has quit IRC01:22
*** vivek-ebay has joined #openstack-lbaas01:23
*** vivek-ebay has quit IRC01:36
*** banix has joined #openstack-lbaas01:40
*** sbfox has joined #openstack-lbaas01:44
*** woodster_ has quit IRC02:00
*** sbfox has quit IRC02:21
*** sbfox has joined #openstack-lbaas02:36
*** woodster_ has joined #openstack-lbaas02:39
*** sbfox has quit IRC02:42
*** BrianShang has quit IRC02:50
*** BrianShang has joined #openstack-lbaas02:52
*** amotoki has joined #openstack-lbaas02:53
*** xgerman has joined #openstack-lbaas02:55
openstackgerritMadhusudhan Kandadai proposed openstack/neutron-lbaas: Added negative public API tempest tests  https://review.openstack.org/16562703:06
*** amotoki has quit IRC03:11
*** xgerman has quit IRC03:18
*** xgerman_ has joined #openstack-lbaas03:18
*** sbfox has joined #openstack-lbaas03:27
*** vivek-ebay has joined #openstack-lbaas03:36
*** xgerman_ has quit IRC04:02
*** mwang2 has joined #openstack-lbaas04:06
*** amotoki has joined #openstack-lbaas04:09
*** vivek-eb_ has joined #openstack-lbaas04:12
*** vivek-ebay has quit IRC04:14
*** banix has quit IRC04:27
*** mwang2 has quit IRC04:45
*** BrianShang_ has joined #openstack-lbaas04:46
*** BrianShang has quit IRC04:49
*** amotoki_ has joined #openstack-lbaas05:00
*** chlong has quit IRC05:02
*** Aish has quit IRC05:16
*** chlong has joined #openstack-lbaas05:19
openstackgerritmin wang proposed stackforge/octavia: Add the config file for nova compute drive  https://review.openstack.org/16324005:26
*** chlong has quit IRC06:01
*** vivek-eb_ has quit IRC06:07
*** chlong has joined #openstack-lbaas06:13
*** Santosh_NS_ has joined #openstack-lbaas06:34
*** fnaval has quit IRC07:02
*** fnaval has joined #openstack-lbaas07:03
*** fnaval has quit IRC07:07
*** chlong has quit IRC07:45
*** sbfox has quit IRC08:10
*** apuimedo has joined #openstack-lbaas08:20
*** openstackgerrit has quit IRC08:22
*** openstackgerrit has joined #openstack-lbaas08:22
*** vivek-ebay has joined #openstack-lbaas09:08
*** vivek-ebay has quit IRC09:12
*** amotoki has quit IRC09:37
*** mwang2 has joined #openstack-lbaas10:41
*** mwang2 has quit IRC10:46
*** BrianShang_ has quit IRC10:48
*** BrianShang has joined #openstack-lbaas10:49
*** enikanorov has quit IRC10:54
*** enikanorov has joined #openstack-lbaas10:55
*** BrianShang has quit IRC11:01
*** BrianShang has joined #openstack-lbaas11:02
*** enikanorov has quit IRC11:04
*** enikanorov has joined #openstack-lbaas11:05
*** enikanorov has quit IRC11:06
*** enikanorov has joined #openstack-lbaas11:07
*** enikanorov has quit IRC11:26
*** enikanorov has joined #openstack-lbaas11:27
*** enikanorov has quit IRC11:30
*** enikanorov has joined #openstack-lbaas11:31
*** enikanorov has quit IRC11:39
*** enikanorov has joined #openstack-lbaas11:40
*** mestery is now known as mestery_afk11:45
*** Santosh_NS_ has quit IRC12:04
*** enikanorov has quit IRC13:09
*** enikanorov has joined #openstack-lbaas13:11
*** amotoki_ has quit IRC13:35
*** amotoki has joined #openstack-lbaas13:35
*** raginbajin has joined #openstack-lbaas13:55
*** localloop127 has joined #openstack-lbaas14:07
*** TrevorV|Home has joined #openstack-lbaas14:08
*** Aish has joined #openstack-lbaas14:14
*** mestery_afk has quit IRC14:19
*** enikanorov has quit IRC14:47
*** fnaval has joined #openstack-lbaas14:47
*** enikanorov has joined #openstack-lbaas14:48
*** vivek-ebay has joined #openstack-lbaas14:54
*** xgerman_ has joined #openstack-lbaas15:09
sballeMorning15:14
*** mestery has joined #openstack-lbaas15:15
openstackgerritAishwarya Thangappa proposed openstack/neutron-lbaas: Included more api test cases for neutron-lbaas  https://review.openstack.org/16571615:16
rm_workmorning15:18
*** ajmiller has joined #openstack-lbaas15:19
*** fnaval has quit IRC15:21
*** openstackgerrit has quit IRC15:21
*** openstackgerrit has joined #openstack-lbaas15:22
*** fnaval has joined #openstack-lbaas15:35
*** mlavalle has joined #openstack-lbaas15:50
*** vivek-eb_ has joined #openstack-lbaas15:56
*** vivek-ebay has quit IRC15:58
*** banix has joined #openstack-lbaas16:04
*** Aish has quit IRC16:05
*** amotoki has quit IRC16:07
*** amotoki has joined #openstack-lbaas16:07
*** banix has quit IRC16:12
*** banix has joined #openstack-lbaas16:16
*** kbyrne has quit IRC16:18
*** banix has quit IRC16:25
*** BrianShang_ has joined #openstack-lbaas16:25
*** sbfox has joined #openstack-lbaas16:27
*** sbfox has quit IRC16:28
*** sbfox has joined #openstack-lbaas16:28
*** BrianShang has quit IRC16:28
*** madhu__ has joined #openstack-lbaas16:34
*** amotoki has quit IRC16:36
*** vivek-eb_ has quit IRC16:36
*** mlavalle has quit IRC16:51
*** mwang2 has joined #openstack-lbaas17:05
openstackgerritAishwarya Thangappa proposed openstack/neutron-lbaas: Included more api test cases for neutron-lbaas  https://review.openstack.org/16571617:07
*** Aish has joined #openstack-lbaas17:08
*** openstackgerrit has quit IRC17:21
*** openstackgerrit has joined #openstack-lbaas17:22
*** banix has joined #openstack-lbaas17:25
*** bharath has joined #openstack-lbaas17:38
Aishblogan: Is there any rule for giving a listener name? max length, invalid characters ??17:41
xgerman_Aish you cna check in the database/datamodel how the tables are set up17:50
*** sbfox has quit IRC17:50
Aishxgerman_: thnks17:52
xgerman_rm_work: When you have a chance can you please review https://review.openstack.org/#/c/160034/17:54
rm_worksure17:55
xgerman_thanks a dozen17:55
xgerman_I am expeiclaly interested if I overlooked some TLS util functions17:56
*** sbfox has joined #openstack-lbaas17:57
*** jorgem has joined #openstack-lbaas18:07
*** banix has quit IRC18:19
*** mlavalle has joined #openstack-lbaas18:19
*** jorgem has quit IRC18:20
*** banix has joined #openstack-lbaas18:24
*** crc32 has joined #openstack-lbaas18:35
*** dougwig has quit IRC18:39
*** jorgem has joined #openstack-lbaas18:46
*** banix has quit IRC18:50
bloganAish: what xgerman said18:53
*** madhu__ has quit IRC18:58
*** banix has joined #openstack-lbaas19:01
*** jorgem has quit IRC19:10
Aishblogan: yup.. got it.. thnks :)19:20
openstackgerritmin wang proposed stackforge/octavia: Add the config file for nova compute drive  https://review.openstack.org/16324019:20
openstackgerritTrevor Vardeman proposed stackforge/octavia: Amphora SSH Driver  https://review.openstack.org/16096419:25
*** TrevorV|Home has quit IRC19:26
*** banix has quit IRC19:27
*** xgerman_ has quit IRC19:30
*** Trevor_V has joined #openstack-lbaas19:36
*** apuimedo has quit IRC19:41
*** dougw2 has joined #openstack-lbaas19:44
*** ajmiller_ has joined #openstack-lbaas19:56
*** ajmiller has quit IRC20:00
*** madhu_ has joined #openstack-lbaas20:05
*** dougwig has joined #openstack-lbaas20:13
*** ajmiller_ is now known as ajmiller20:14
*** jorgem has joined #openstack-lbaas20:23
*** Trevor_V has quit IRC20:32
*** localloop127 has quit IRC20:44
*** xgerman has joined #openstack-lbaas20:45
*** xgerman has quit IRC20:47
*** xgerman has joined #openstack-lbaas20:48
*** enikanorov has quit IRC20:50
johnsomblogan dougwig : I guess we have to get busy http://sched.co/2qbg20:51
*** enikanorov has joined #openstack-lbaas20:51
rm_workxgerman: hey20:51
xgermanhey20:51
rm_workxgerman: so how do you feel about ''.format()? :P20:51
bloganjohnsom: ill just do a dance routine20:52
xgermannot very fondly - never worked for me20:52
rm_workT_T20:52
xgermanbut I probably should learn it20:52
rm_workI got distracted while reviewing that CR by the huge number of %substitutions20:52
rm_worklol20:52
bloganrm_work gets distracted by different approaches20:54
rm_work%s substitution is supposed to be deprecated T_T20:54
rm_workEVENTUALLY <_<20:54
rm_workand .format() has many advantages20:54
xgermanmaybe, one thing I noticed was that it wouldn't substitute stuff20:55
rm_workwhich wouldn't?20:55
xgermanit's just "bla %s".format('bla')20:56
*** localloop127 has joined #openstack-lbaas20:56
xgermanand when you pass that as an argument it doesn't get substituted right away and causes errors20:57
xgermanwhereas magiclaly % gets substituted right away20:57
xgermannow after two minutes I gave up on that20:57
xgermanprobably should push myse;lf harder ;-)20:58
rm_workheh20:58
rm_work"blah {0}".format('bla')20:58
rm_workbut more importantly20:58
xgermanah, thyat's what I did wrong20:58
xgerman\me wonders where is dougwig and Ruby when you need him20:59
rm_work"The {object_type} with IP {ip} was not found -- when making a(n) {object_type}, please use care!".format(object_type=a, ip=b)21:00
rm_workand that it auto-casts21:00
rm_workso you don't need to do str(a)\21:01
rm_workor str(b)21:01
dougwig"The #{object_type} with IP #{ip} was not found"   <- ruby.21:01
xgermanyep21:01
dougwigpython string interpolation blows baby goats in comparison.21:01
xgerman+ 121:01
xgermanno wonder I got confused21:01
rm_work$mystring = "This is a string about $object_type and $ip";21:01
rm_workthanks PHP21:01
rm_workfor being dumb but still having decent string support21:02
rm_workalso BASH21:02
dougwigwhere would the world be without print("hello %s world %s", "foo"); style security bugs?21:02
openstackgerritAishwarya Thangappa proposed openstack/neutron-lbaas: Included more API tests for listener creation/updation  https://review.openstack.org/16571621:02
rm_workdougwig: does that print "hello foo world foo" or cause an exception or ???21:03
dougwigit reads and inserts random garbage from the stack.21:03
rm_workawesome21:03
*** jorgem has quit IRC21:04
xgermanrm_work flag that and I will fix21:07
*** jorgem has joined #openstack-lbaas21:08
rm_workheh, I had originally flagged EVERY SINGLE ONE, but I decided to remove those comments, i just have a single comment per file about it21:08
rm_workthere were like 15 usages21:08
xgermanyep21:08
xgermanthanks though21:08
rm_work"If you use .format() you don't need to do an explicit str() cast here. I recommend using .format() everywhere actually, but I don't think we ever came to an official consensus, and it would be good to be consistent. Right now I believe there are significant uses of both styles in our codebase."21:08
rm_workis what I put21:08
*** localloop127 has quit IRC21:11
xgermanI can change it everywhere...21:11
rm_workI mean, I'm all for that! but I thought it might actually merit a discussion21:11
johnsomHmm, I have gone off of this doc in the past: http://docs.openstack.org/developer/oslo.i18n/guidelines.html21:14
johnsomSeems to encourage the %(name)s syntax21:14
rm_workwell it makes sense for21:15
rm_workLOG.info(_LI('some message: variable=%s'), variable)21:15
rm_worksince this has actual distinct advantage21:15
rm_workbut raise ValueError(_('some message: v1=%(v1)s v2=%(v2)s') % {'v1': v1, 'v2': v2}) might as well use .format()21:16
rm_workit's interesting that their docs still use %s tho21:16
rm_workbut being recommended by Oslo -- not sure whether that is a pro or a con <_<21:17
johnsomhahah21:17
*** xgerman has quit IRC21:18
*** xgerman has joined #openstack-lbaas21:19
dougwigif you really want to be on a the warpath, add a HACKING check.21:30
rm_workheh21:30
rm_workI might just :P21:30
*** banix has joined #openstack-lbaas21:31
rm_workI have been staring at this patchset for like 2 hours now21:35
rm_workxgerman ate my productivity today entirely :P21:35
rm_workalmost done21:35
xgermanthanks a ton21:36
rm_worklooks pretty good on the whole21:36
rm_workyou had a regex bug21:36
*** jorgem has quit IRC21:39
rm_workxgerman: review up, will get to the tests on Monday21:39
*** banix has quit IRC21:39
xgermanthanks21:40
madhu_blogan, dougwig: what is the max character length can the user give in for name/description while creating loadbalancer?21:50
*** enikanorov has quit IRC21:53
*** enikanorov has joined #openstack-lbaas21:54
madhu_fnaval : ^^21:57
*** banix has joined #openstack-lbaas21:58
*** banix has quit IRC22:00
*** banix has joined #openstack-lbaas22:03
*** sbfox has quit IRC22:10
xgermanI thought Aish looke dit up in DB22:18
*** madhu_ has quit IRC22:25
Aishmadhu_:22:29
Aishname = sa.Column(sa.String(255))22:29
Aish    description = sa.Column(sa.String(255))22:29
*** jorgem has joined #openstack-lbaas22:34
*** mlavalle has quit IRC22:52
*** enikanorov has quit IRC22:55
*** enikanorov has joined #openstack-lbaas22:56
*** jorgem has quit IRC23:01
xgermanafter I stole rmworks productivity... can we get a +2 for https://review.openstack.org/163240 blogan?23:03
dougwigxgerman: i crushed your hopes and dreams.23:21
xgermannotthe first time23:22
*** dougw2 has quit IRC23:22
*** enikanorov has quit IRC23:22
*** enikanorov has joined #openstack-lbaas23:23
*** banix has quit IRC23:24
*** bharath has quit IRC23:32
*** Aish has quit IRC23:32
*** ajmiller has quit IRC23:46
*** xgerman has quit IRC23:57

Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!