Wednesday, 2017-01-25

openstackgerritAdam Harwell proposed openstack/octavia: Implement sorting and pagination for octavia  https://review.openstack.org/38214700:23
rm_workwhoops, forgot to change a line, but yeah now it works, woo00:23
rm_workjohnsom: I guess you can check that out if you have a spare devstack, your stuff should work now00:23
rm_workjust need testing00:23
rm_worksort/filter/marker00:23
rm_workerr00:23
rm_worksort/limit/marker00:23
rm_workstill need to do filter00:23
rm_workprolly will do it in the same place, but as a followup00:24
johnsomOk.  I'm pretty focused on the centos amp stuff at the moment, but will keep it in mind.00:24
rm_workno worries then00:24
rm_worki'll do more testing first00:24
rm_workwhen is our freeze date, do you think?00:24
rm_workI can probably get the testing bits done tomorrow....00:25
johnsomI had hoped for tomorrow, but now I'm thinking more like Thursday00:25
rm_workbut that's cutting it short on time for reviewers to actually review/test00:25
rm_worknot sure if it matters if we get this in, honestly00:25
rm_worksince the API ones won't be00:25
johnsomRight00:26
rm_workso yeah, whatever00:26
rm_workI'm planning to run production off master anyway :P00:26
openstackgerritMichael Johnson proposed openstack/neutron-lbaas-dashboard: Allow hacking 0.12.x  https://review.openstack.org/41041600:27
*** ducttape_ has quit IRC00:47
*** amotoki has joined #openstack-lbaas00:49
*** nrado has quit IRC01:15
*** ducttape_ has joined #openstack-lbaas01:19
johnsomWell, that is neat, the py3x gate couldn't ssh into the cirros image: http://logs.openstack.org/90/424890/1/check/gate-octavia-v1-dsvm-py3x-scenario-ubuntu-xenial-nv/8761dcc/console.html01:25
johnsomSame with the multi-node gate01:25
*** kevo has quit IRC01:48
*** KunalGandhi has joined #openstack-lbaas01:49
harlowjajohnsom are u gonna be in the bay area anytime :-P02:01
johnsomUmm, yes actually, on vacation.  What is up?02:01
harlowjau are02:01
harlowjahmmmm, tempting02:01
johnsomMay-ish02:02
johnsomNot finalized02:02
harlowjakk02:02
johnsomWant to buy me a beer?02:02
harlowjaok, nothing to worry about yet, just might be interesting to meet up and talk some lbaas02:02
johnsomHa, ok02:02
johnsomYou know I'm always open for a hangout chat too02:02
johnsomWe could of course chat at the PTG too, but somebody is slacking off....  Grin02:03
harlowjahey man, family ski vacation02:03
harlowjagotta go, ha02:04
xgermanNice!02:04
johnsomExpecting fresh powder that week?02:04
harlowjaalways, ha02:04
rm_workyeah we're looking at doing FLIPs inside octavia and giving out a FLIP address as the "VIP"02:05
rm_workI forget how RAX handled that? because we were doing something similar, right?02:05
rm_workcreating FLIPs takes a network_id though instead of a subnet_id like we require on our VIP object during LB create02:06
*** ducttape_ has quit IRC02:06
johnsomThat is an easy lookup though, if it is not a property.02:06
openstackgerritMerged openstack/octavia: Updated from global requirements  https://review.openstack.org/42462002:06
harlowjai'll let u guys figure it out, gotta run02:07
harlowja:-P02:07
harlowjagoooo taskflow!02:07
harlowjaha02:07
johnsomHmm, I think there are some arguments against, but I'm a bit fried at the moment from staring at this centos stuff02:07
johnsomhaha02:07
rm_workuhhh hang on02:07
rm_worksooooo02:07
rm_workhttp://developer.openstack.org/api-ref/networking/v2/?expanded=create-a-load-balancer-detail02:07
rm_workwe're moving octavia to the n-lbaas API02:07
rm_workfor "vip_subnet_id" it says: The ID of the subnet on which to allocate the virtual IP (VIP) address. This option is required if no vip_network_id is given.02:08
johnsomwell, something like that02:08
rm_workI don't see vip_network_id as a param??02:08
johnsomYeah, don't trust the docs out there.  They have been scrambled02:08
johnsomTrust the code02:09
rm_workok so it's possible we're actually fine02:09
rm_workwe can take a network_id02:09
rm_workyeah octavia's current API is subnet_id only02:09
rm_workbut02:09
rm_workI'll take a look at neutron-lbaas again02:09
johnsomBut again, you can get network from subnet so, not a big deal02:09
johnsomPicking the right subnet from a network is another story02:10
*** ducttape_ has joined #openstack-lbaas02:10
rm_workjohnsom: kinda...02:11
rm_workso FLIPs are provisioned by network_id02:11
rm_workso if the user had to find the right FLIP network, then pick a random subnet from it, pass that to us, so we could just *not use it* and provision a FLIP on that subnet.network_id and have the FLIP end up on some other subnet...02:11
rm_worklol02:11
rm_workkinda dumb02:11
johnsomYeah, it's a NAT so, that makes sense there.  You have to tell it the subnet/IP to target02:11
rm_workso we'd like to let the user provide a subnet_id *or* a network_id for lb create02:12
rm_workfor the vip object02:12
rm_workdoes that seem reasonable?02:13
rm_workI'll prolly bring it up in the meeting tomorrow02:13
johnsomIf we allow network_id, which subnet on that network gets the VIP?02:13
rm_workin the case of FLIPs... it's up to the scheduler?02:14
rm_workelsewise... random.choice()?02:14
rm_workif the user *really* doesn't care... that should be fine02:14
rm_workif they do, then they should provide a subnet instead02:14
rm_workand if they provide the subnet we'd always default to that02:14
rm_worktechnically it's up to the drivers to handle it, right? so in a flip_driver it'd use the network_id and ignore subnet02:15
rm_workin the existing aap driver, it'd prefer subnet, and if none was provided it'd go down to random.choice(network.subnets)02:15
rm_workwhich in most cases might just be one <_<02:15
rm_workbut again, it's the user's choice if they supply a network_id instead of a subnet, so they should only be doing that if that is what they actually want to happen02:16
rm_workif you provide a network_id and get annoyed it got provisioned on subnetA instead of subnetB, you should have just provided subnetB >_>02:17
johnsomHmmm.  Ok, I need to go eat dinner.  We can talk about it more tomorrow.02:17
rm_workkk02:17
openstackgerritMichael Johnson proposed openstack/octavia: Fix the amphora-agent support for RH based Linux flavors  https://review.openstack.org/33184102:17
*** sanfern has quit IRC02:24
*** armax has quit IRC02:31
*** KunalGandhi has quit IRC02:45
*** eezhova has joined #openstack-lbaas03:32
openstackgerritMichael Johnson proposed openstack/octavia: Fix the amphora-agent support for RH based Linux flavors  https://review.openstack.org/33184103:34
*** anilvenkata has joined #openstack-lbaas03:50
*** ducttape_ has quit IRC03:57
*** armax has joined #openstack-lbaas04:10
*** sputnik13 has joined #openstack-lbaas04:21
*** eezhova has quit IRC04:22
*** ducttape_ has joined #openstack-lbaas04:24
openstackgerritMichael Johnson proposed openstack/octavia: Fix the amphora-agent support for RH based Linux flavors  https://review.openstack.org/33184104:27
*** armax has quit IRC04:30
*** sanfern has joined #openstack-lbaas04:41
*** ducttape_ has quit IRC04:46
*** ianychoi has joined #openstack-lbaas04:55
*** armax has joined #openstack-lbaas04:55
*** armax has quit IRC04:56
openstackgerritSindhu Devale proposed openstack/octavia: Octavia v2 API for listeners  https://review.openstack.org/42474405:10
*** saju_m has joined #openstack-lbaas05:15
openstackgerritMichael Johnson proposed openstack/octavia: Fix the amphora-agent support for RH based Linux flavors  https://review.openstack.org/33184105:17
*** gongysh has joined #openstack-lbaas05:18
*** Alex_Staf has joined #openstack-lbaas05:19
*** eezhova has joined #openstack-lbaas05:29
openstackgerritMichael Johnson proposed openstack/octavia: Fix the amphora-agent support for RH based Linux flavors  https://review.openstack.org/33184105:40
*** ducttape_ has joined #openstack-lbaas05:46
*** csomerville has quit IRC05:47
*** cody-somerville has joined #openstack-lbaas05:49
*** cody-somerville has quit IRC05:49
*** cody-somerville has joined #openstack-lbaas05:49
*** ducttape_ has quit IRC05:51
Alex_Stafnmagnezi, I have it in my TP. active standby and pool testing. I will send the TP to u if u want .05:56
*** kevo has joined #openstack-lbaas05:58
*** KunalGandhi has joined #openstack-lbaas05:59
openstackgerritMichael Johnson proposed openstack/octavia: Fix the amphora-agent support for RH based Linux flavors  https://review.openstack.org/33184106:11
*** armax has joined #openstack-lbaas06:13
*** ianychoi has quit IRC06:15
*** armax has quit IRC06:16
rm_workjohnsom: still at it? :/06:17
johnsomYeah06:17
johnsomRunning another test pass06:18
*** KunalGandhi has quit IRC06:19
rm_worki'm looking at implementing sort_dir and sort_key06:24
*** nrado has joined #openstack-lbaas06:24
johnsomJoy.  Didn't you get that from glance?06:25
rm_workit was at a higher level06:25
rm_workand they technically ONLY handle that way, I think06:25
rm_workit isn't actually that hard06:25
rm_workI just have to do it06:25
rm_workI think it'll give precedence to sort= and only use sort_key= as a backup?06:26
rm_workthat is fine, right?06:26
rm_workor should I try to combine06:26
rm_work<_<06:26
rm_workprobably just one or the other06:26
rm_workif sort= is present I won't use sort_key=06:26
johnsomI think one of the other is fine06:26
johnsomYeah, the new way, sort= (I think) should have precidence06:27
*** saju_m has quit IRC06:29
*** anilvenkata has quit IRC06:32
rm_workalright pretty much got it06:35
rm_worklooking at how much effort it'd be to slap filters into here too06:35
rm_workare sort keys supposed to be case-sensitive?06:38
rm_workI guess so?06:38
johnsomHa, I have no idea06:39
rm_workugh testing06:44
*** anilvenkata has joined #openstack-lbaas06:44
*** saju_m has joined #openstack-lbaas06:47
rm_workuuuuugh06:49
rm_workglance has so much better basic test infrastructure it seems06:50
rm_workwe're missing a ton of stuff for testing this properly06:50
rm_worklike they have a FakeDB thing06:50
openstackgerritAdam Harwell proposed openstack/octavia: Implement sorting and pagination for octavia  https://review.openstack.org/38214706:56
johnsomWe have a fake db too, for the functional tests06:56
rm_workhmmm07:01
rm_workthey use it for unit tests too O_o07:01
rm_workmaybe I can make use of it...07:02
johnsomYeah, I would expect most if not all of the tests for that will be functional API tests.07:02
rm_workuhhh maybe I can just make the tests for this stuff be functional tests <_<07:03
rm_workheh yeah k07:03
johnsomOk, calling it a night.  Catch you in the morning07:05
rm_worknight07:05
nmagnezijohnsom, good night :)07:06
rm_workummm i forgot what time the meeting is, but I should be awake :P07:06
johnsomNoon pacific07:07
openstackgerritOpenStack Proposal Bot proposed openstack/neutron-lbaas-dashboard: Imported Translations from Zanata  https://review.openstack.org/42501307:13
*** yamamoto has quit IRC07:17
*** anilvenkata has quit IRC07:17
*** ducttape_ has joined #openstack-lbaas07:17
*** ducttape_ has quit IRC07:22
*** pcaruana has joined #openstack-lbaas07:28
*** anilvenkata has joined #openstack-lbaas07:30
*** saju_m has quit IRC07:32
*** saju_m has joined #openstack-lbaas07:34
*** tesseract has joined #openstack-lbaas07:34
openstackgerritBrandon Logan proposed openstack/octavia: Remove v1 base type  https://review.openstack.org/42502607:37
*** saju_m has quit IRC07:46
rm_workblogan: what was the point of that even07:53
rm_workoh I see, it was a deprecation thing07:53
*** yamamoto has joined #openstack-lbaas08:00
*** yamamoto has quit IRC08:02
*** yamamoto has joined #openstack-lbaas08:03
*** ianychoi has joined #openstack-lbaas08:12
*** gcheresh has joined #openstack-lbaas08:14
*** amotoki has quit IRC08:47
*** ducttape_ has joined #openstack-lbaas08:48
*** ducttape_ has quit IRC08:52
openstackgerritAdam Harwell proposed openstack/octavia: Implement sorting and pagination for octavia  https://review.openstack.org/38214708:58
*** kevo has quit IRC09:09
*** yamamoto has quit IRC09:38
*** strigazi_AFK is now known as strigazi09:50
rm_workugh almost have it09:51
rm_workjust a few tests that I'm not clear on...09:51
rm_workputting it up as-is, leaving the three functional tests I added in a failing state until I figure out whether they're failing in the "correct way" or not <_<09:58
openstackgerritAdam Harwell proposed openstack/octavia: Implement sorting and pagination for octavia  https://review.openstack.org/38214710:00
*** openstackgerrit has quit IRC10:17
*** amotoki has joined #openstack-lbaas10:19
*** yamamoto has joined #openstack-lbaas10:25
*** yamamoto has quit IRC10:26
*** gcheresh_ has joined #openstack-lbaas10:54
*** gcheresh has quit IRC10:58
*** nrado has quit IRC11:05
*** nrado has joined #openstack-lbaas11:05
*** nrado has quit IRC11:08
*** nrado has joined #openstack-lbaas11:08
*** yamamoto has joined #openstack-lbaas11:18
*** yamamoto has quit IRC11:18
*** yamamoto has joined #openstack-lbaas11:33
*** yamamoto has quit IRC11:34
*** anilvenkata has quit IRC11:41
*** sanfern has quit IRC11:45
*** ducttape_ has joined #openstack-lbaas11:49
*** ducttape_ has quit IRC11:53
*** anilvenkata has joined #openstack-lbaas12:01
*** anilvenkata has quit IRC12:09
*** reedip_ has joined #openstack-lbaas12:11
*** anilvenkata has joined #openstack-lbaas12:21
*** nrado has left #openstack-lbaas12:22
*** catintheroof has joined #openstack-lbaas12:28
-openstackstatus- NOTICE: Gerrit is going to be restarted due to slow performance12:46
*** ducttape_ has joined #openstack-lbaas12:48
-openstackstatus- NOTICE: Gerrit has been successfully restarted12:49
*** eezhova has quit IRC12:55
*** eezhova has joined #openstack-lbaas13:01
*** sanfern has joined #openstack-lbaas13:13
*** ducttape_ has quit IRC13:34
*** openstackgerrit has joined #openstack-lbaas13:38
openstackgerritAbed Abu dbai proposed openstack/octavia: ACTIVE-ACTIVE Topology - Initial Cluster Manager  https://review.openstack.org/40523813:38
openstackgerritAbed Abu dbai proposed openstack/octavia: ACTIVE-ACTIVE Topology: Initial Distributor Noop Driver  https://review.openstack.org/31300613:38
openstackgerritAbed Abu dbai proposed openstack/octavia: ACTIVE-ACTIVE Topology : create distributor network flow  https://review.openstack.org/40976313:38
openstackgerritAbed Abu dbai proposed openstack/octavia: Active-Active Topology - register/uregister amphorae tasks  https://review.openstack.org/40976513:38
openstackgerritAbed Abu dbai proposed openstack/octavia: Active-Active Topology - Cluster DB Tasks  https://review.openstack.org/40976413:38
openstackgerritAbed Abu dbai proposed openstack/octavia: ACTIVE-ACTIVE Topology OVS-based Distributor Driver  https://review.openstack.org/31762913:38
openstackgerritAbed Abu dbai proposed openstack/octavia: ACTIVE-ACTIVE Topology OVS-based Distributor Backend  https://review.openstack.org/32042213:38
openstackgerritAbed Abu dbai proposed openstack/octavia: ACTIVE-ACTIVE Topology - Distributor related tasks  https://review.openstack.org/40695113:38
openstackgerritAbed Abu dbai proposed openstack/octavia: ACTIVE-ACTIVE - controller network tasks  https://review.openstack.org/32348113:38
openstackgerritAbed Abu dbai proposed openstack/octavia: ACTIVE-ACTIVE Topology - Distributor image creation  https://review.openstack.org/40359413:38
openstackgerritAbed Abu dbai proposed openstack/octavia: ACTIVE-ACTIVE - network driver related changes  https://review.openstack.org/32249413:38
openstackgerritAbed Abu dbai proposed openstack/octavia: ACTIVE-ACTIVE - distributor certificate tasks  https://review.openstack.org/40695213:38
openstackgerritAbed Abu dbai proposed openstack/octavia: ACTIVE-ACTIVE Topology - distributor creation flow  https://review.openstack.org/40695313:38
openstackgerritAbed Abu dbai proposed openstack/octavia: ACTIVE-ACTIVE Topology - create shared distributor  https://review.openstack.org/40695413:38
openstackgerritAbed Abu dbai proposed openstack/octavia: Active-Active Topology - added disable arp  https://review.openstack.org/42517513:38
*** eezhova has quit IRC13:42
*** pck has quit IRC13:52
*** pck has joined #openstack-lbaas13:53
*** ipsecguy has quit IRC13:53
*** ipsecguy has joined #openstack-lbaas14:01
*** ducttape_ has joined #openstack-lbaas14:06
*** matt-borland has joined #openstack-lbaas14:15
openstackgerritNir Magnezi proposed openstack/octavia: Fix the amphora-agent support for RH based Linux flavors  https://review.openstack.org/33184114:23
*** ducttape_ has quit IRC14:33
*** catinthe_ has joined #openstack-lbaas15:00
*** catintheroof has quit IRC15:02
*** fnaval has joined #openstack-lbaas15:05
*** ducttape_ has joined #openstack-lbaas15:08
*** jschwarz is now known as jschwarz|brb15:41
*** catintheroof has joined #openstack-lbaas15:43
*** catinthe_ has quit IRC15:46
*** gcheresh_ has quit IRC15:46
*** jschwarz|brb is now known as jschwarz15:48
openstackgerritAbed Abu dbai proposed openstack/octavia: Active-Active Topology - added disable arp  https://review.openstack.org/42517515:55
openstackgerritAbed Abu dbai proposed openstack/octavia: Active-Active Topology - register/uregister amphorae tasks  https://review.openstack.org/40976515:55
openstackgerritAbed Abu dbai proposed openstack/octavia: Active-Active Topology - Cluster DB Tasks  https://review.openstack.org/40976415:55
*** _ducttape_ has joined #openstack-lbaas15:58
*** KunalGandhi has joined #openstack-lbaas15:59
*** ducttape_ has quit IRC16:02
*** gongysh has quit IRC16:07
*** anilvenkata has quit IRC16:07
*** eezhova has joined #openstack-lbaas16:07
openstackgerritAbed Abu dbai proposed openstack/octavia: Active-Active Topology - added disable arp  https://review.openstack.org/42517516:12
openstackgerritAbed Abu dbai proposed openstack/octavia: Active-Active Topology - register/uregister amphorae tasks  https://review.openstack.org/40976516:12
openstackgerritAbed Abu dbai proposed openstack/octavia: ACTIVE-ACTIVE Topology : create distributor network flow  https://review.openstack.org/42525816:12
*** gcheresh_ has joined #openstack-lbaas16:13
*** KunalGandhi has quit IRC16:14
*** armax has joined #openstack-lbaas16:18
*** _ducttape_ has quit IRC16:23
*** ducttape_ has joined #openstack-lbaas16:23
*** ankur-gupta-f2 has joined #openstack-lbaas16:32
*** AlexeyAbashkin has quit IRC16:35
*** gcheresh_ has quit IRC17:15
openstackgerritNakul Dahiwade proposed openstack/octavia: Introduce Octavia v2 API for Members  https://review.openstack.org/40593217:16
*** reedip_ has quit IRC17:23
*** anilvenkata has joined #openstack-lbaas17:33
*** tesseract has quit IRC17:33
*** kevo has joined #openstack-lbaas17:36
openstackgerritNakul Dahiwade proposed openstack/octavia: Align Octavia v2 API for Members  https://review.openstack.org/42339317:44
*** cody-somerville has quit IRC17:48
*** cody-somerville has joined #openstack-lbaas17:48
*** cody-somerville has quit IRC17:48
*** cody-somerville has joined #openstack-lbaas17:48
*** KunalGandhi has joined #openstack-lbaas17:54
*** beagles is now known as beagles-biab17:59
*** eezhova has quit IRC18:08
*** rm_mobile has joined #openstack-lbaas18:09
*** rm_mobile has joined #openstack-lbaas18:09
rm_mobilejohnsom what time is the meeting today? Ideally in like... Time from now18:09
johnsomrm_mobile Two hours from now18:10
johnsom(minus the 10 minutes)18:10
rm_mobileK cool18:10
rm_mobileThanks18:10
*** matt-borland has quit IRC18:23
*** catinthe_ has joined #openstack-lbaas18:33
*** catintheroof has quit IRC18:35
xgermanwhat does tune2 do? in addition to brick my Mac?18:36
xgermantune2fs18:36
*** KunalGandhi has quit IRC18:40
*** gcheresh_ has joined #openstack-lbaas18:40
*** KunalGandhi has joined #openstack-lbaas18:40
*** KunalGandhi has quit IRC18:45
*** pcaruana has quit IRC18:53
rm_mobileChanging ext2/3/4 metadata18:53
rm_mobileWhy do you have that on your Mac? Lol18:54
johnsomYeah, be careful with tune2fs18:57
johnsomIt's used for configuring filesystems18:58
xgermanfor some reason when I run ./devstack in my vm it does this crap18:58
johnsomI think diskimage-builder runs it against the image it is building, but I can't think why devstack would use it otherwise18:58
xgermanand then why would it run amok18:59
xgermangetting another datapoint18:59
johnsomThis is the only use of it I see in devstack: http://logs.openstack.org/90/424890/1/check/gate-neutron-lbaasv2-dsvm-scenario-ubuntu-xenial-nv/b4533a6/logs/devstacklog.txt.gz#_2017-01-24_23_28_00_36919:00
johnsomWhich is the DIB config of the new image19:00
*** sshank_ has quit IRC19:00
rm_mobileYeah DIB USES it19:01
rm_mobileBut it should just be on the new VM's filesystem19:01
johnsombin/disk-image-create:    sudo tune2fs -U ${DIB_IMAGE_ROOT_FS_UUID} ${IMAGE_BLOCK_DEVICE}19:01
johnsomYeah, it sets a UUID on the FS19:01
rm_mobileNever had a problem on my Mac in a VM19:01
*** sshank has joined #openstack-lbaas19:02
johnsomI haven't had a problem with it either, but I'm running native ubuntu19:02
*** csomerville has joined #openstack-lbaas19:03
*** KunalGandhi has joined #openstack-lbaas19:03
*** cody-somerville has quit IRC19:06
*** anilvenkata has quit IRC19:15
rm_workah does availability_zone *not* take a list?19:25
rm_workI guess that is probably a downstream patch19:25
rm_workI pass: availability_zone="zone1,zone2,zone3"19:25
*** rm_mobile has quit IRC19:34
*** eezhova has joined #openstack-lbaas19:53
johnsomOctavia meeting starting soon on #openstack-meeting-alt19:57
nmagneziack19:59
*** KunalGandhi has quit IRC20:08
*** KunalGandhi has joined #openstack-lbaas20:08
*** KunalGandhi has quit IRC20:12
openstackgerritMerged openstack/neutron-lbaas-dashboard: Imported Translations from Zanata  https://review.openstack.org/42501320:13
openstackgerritMerged openstack/neutron-lbaas-dashboard: Allow hacking 0.12.x  https://review.openstack.org/41041620:13
*** beagles-biab is now known as beagles20:16
openstackgerritNakul Dahiwade proposed openstack/octavia: Align Octavia v2 API for Members  https://review.openstack.org/42339320:22
openstackgerritOpenStack Proposal Bot proposed openstack/neutron-lbaas-dashboard: Updated from global requirements  https://review.openstack.org/42539420:24
*** catinthe_ has quit IRC20:27
nmagnezixgerman, rm_work, johnsom, if and when any of you have a cycle to spare - it would be nice if you can have a look at this newton backport (no conflicts): https://review.openstack.org/#/c/425084/20:31
xgermanwe usually don’t add features to previous releases only bug/security foxes20:32
johnsomCorrect20:32
johnsomhttp://docs.openstack.org/project-team-guide/stable-branches.html20:32
dasmxgerman: does it mean, we could backport this security fox? http://www.myalberton.co.za/wp-content/uploads/2014/06/download-1.jpg20:41
dasm;)20:41
xgermantotally20:42
*** diltram has quit IRC20:46
*** diltram has joined #openstack-lbaas20:46
*** KunalGandhi has joined #openstack-lbaas20:51
openstackgerritShashank Kumar Shankar proposed openstack/octavia: Introduce Octavia v2 API for Pools with LBaaS align  https://review.openstack.org/42019521:07
rm_workso I'm not sure on paging/sorting how the API should respond in case of certain failures21:07
rm_workhttp://logs.openstack.org/47/382147/21/check/gate-octavia-tox-functional-ubuntu-xenial/5f223f8/testr_results.html.gz21:08
rm_workthe three fails there21:08
rm_workoctavia.common.exceptions.InvalidSortKey: Supplied sort key '[u'name', u'desc', u'asc']' is not valid.21:08
rm_workwill pecan handle that properly?21:08
rm_workor is that a 50021:08
*** yamamoto has joined #openstack-lbaas21:27
rm_workoh btw johnsom i figured out the functional test runner issue when running isolated tests21:29
rm_workhttps://review.openstack.org/#/c/382147/21/octavia/tests/functional/api/v1/base.py21:32
rm_workthat21:32
rm_workif we don't import config our config stuff never gets loaded21:33
rm_workso everything explodes in the setup21:33
*** eezhova has quit IRC21:33
rm_workwe don't even need to DO anything with it (thus the #noqa)21:33
xgermanjohnsom I have been told that with big tent you can backport features which land in 1, e.g. if a geature hits Pike-1 it could be backported to Ocata - doesn’t help nmagnezi though ;-)21:36
*** yamamoto has quit IRC21:36
johnsomrm_work You need to make those API exceptions with a HTTP status code I think21:38
johnsomThen they will be handled correctly21:38
rm_workhmmm21:38
rm_workright now the hook just raises an exception21:38
johnsomLook two classes above your exceptions21:38
rm_workso if I go into the *exception classes*?21:38
rm_workyeah k21:38
rm_workWhat if I can't get the exception to raise *during the hook* but it happens further down?21:39
rm_workI guess I'll find out21:39
johnsomAs long as it is based off APIException, it should go back correctly21:39
rm_workawesome21:40
rm_workthat was the last thing21:40
rm_workfixing the tests actually found one or two last bugs21:40
rm_workand now i think it should be good once i fix that21:40
rm_workapparently I missed in the commit message that he said it was based on oslo_db paging/sorting21:41
rm_workbut it was really broken regardless so i think replacing it with the code from glance was probably for the best21:41
rm_workwhelp my tmux just crashed and *disappeared*. awesome21:42
rm_workjohnsom: i'll look over the code for the centos amp patch today21:44
rm_worki think i looked over it before once too21:44
rm_workyou seem to have the testing handled?21:44
johnsomrm_work Yeah, there is still a bug I'm tracking down, but I am focused on manual real-world testing21:46
rm_workkk21:46
rm_worklet me know if you need help21:46
johnsomIt doesn't create the lb in it's current form.  Bug I introduced yesterdayy21:46
rm_worki'm working on paging/sorting and this az thing but i can context switch if it's useful21:46
johnsomThanks21:46
openstackgerritAdam Harwell proposed openstack/octavia: Add option to choose an availability_zone for amps  https://review.openstack.org/42481621:49
*** gcheresh_ has quit IRC21:49
openstackgerritShashank Kumar Shankar proposed openstack/octavia: Align Octavia API to n-lbaasv2 for L7Rules  https://review.openstack.org/40633622:03
*** _ducttape_ has joined #openstack-lbaas22:09
*** ducttape_ has quit IRC22:12
openstackgerritAdam Harwell proposed openstack/octavia: Add option to choose an availability_zone for amps  https://review.openstack.org/42481622:16
rm_workjohnsom: code 400 for all of those?22:19
johnsomYeah, I think that works22:20
johnsomrm_work BTW, if you have a minute: https://review.openstack.org/#/c/424851/22:21
rm_workkk need to run to the store but i will when i get back22:22
*** yamamoto has joined #openstack-lbaas22:22
openstackgerritShashank Kumar Shankar proposed openstack/octavia: Align Octavia API to n-lbaasv2 for L7Rules  https://review.openstack.org/40633622:24
openstackgerritAdam Harwell proposed openstack/octavia: Implement sorting and pagination for octavia  https://review.openstack.org/38214722:25
nmagnezijohnsom, any luck with the network dir path issue? I think I ran into this in some tests as well22:26
johnsomnmagnezi working on it now22:27
johnsomhttps://www.irccloud.com/pastebin/r069Vv9T/22:27
johnsomIt's osutils.py 366 running the wrong version22:29
*** _ducttape_ has quit IRC22:29
*** ducttape_ has joined #openstack-lbaas22:29
*** _ducttape_ has joined #openstack-lbaas22:53
nmagnezijohnsom, i have found something22:53
johnsomDo tell22:53
johnsomJust got out of meeting22:54
nmagnezijohnsom, https://review.openstack.org/#/c/331841/65/octavia/common/constants.py22:54
nmagnezijohnsom, you have unneeded 'y'22:54
nmagnezijohnsom, which leads to amphora-haproxyy22:55
johnsomAh, yep, that is a typo22:56
nmagnezijohnsom, this failed some functional tests22:56
*** ducttape_ has quit IRC22:56
nmagnezijohnsom, how they fail for something else :<22:56
*** _ducttape_ has quit IRC22:57
*** catintheroof has joined #openstack-lbaas23:00
johnsomOk, I fixed my issue23:00
nmagnezijohnsom, the folder path thing?23:01
johnsomBoth issues.23:01
johnsomCan I push up a new patch?23:01
nmagneziyes23:02
johnsomOk, I'm creating an LB now, will push in a minute.23:02
nmagneziok23:03
openstackgerritMichael Johnson proposed openstack/octavia: Fix the amphora-agent support for RH based Linux flavors  https://review.openstack.org/33184123:08
johnsomnmagnezi ^^^ that boots  a working centos amp23:09
johnsomSo I am going to go back to working on testing against my test matrix23:09
nmagnezijohnsom, oh, i understand the bug you witnessed now. i shouldn't have used super here: https://review.openstack.org/#/c/331841/65..66/octavia/amphorae/backends/agent/api_server/osutils.py23:11
nmagnezisorry about that23:11
nmagnezithe result of coding after midnight23:11
johnsomNP, that is what we are looking for.  Catch these now....23:11
nmagnezijohnsom, yup. so I'm going to push a small one to address you comment about the oslo_fixture23:12
johnsomOk, just make sure you rebase onto that new version23:13
nmagnezijohnsom, sure will23:15
nmagnezijohnsom, heads up both octavia.tests.functional.amphorae.backend.agent.api_server.test_server.TestUbuntuServerTestCase.test_plug_VIP4 and octavia.tests.functional.amphorae.backend.agent.api_server.test_server.TestUbuntuServerTestCase.test_plug_VIP6 will fail23:16
nmagnezijohnsom, first the failed because of the typo, now for something else23:16
johnsomok23:16
johnsomI plan to go through my tests before I look at working on unit/functional tests.23:16
nmagnezijohnsom, I understand23:17
nmagnezijohnsom, I'm starting to worry that I won't make it with all the tests by the cut. I wasted a lot of time debugging stuff23:18
johnsomYeah, keep working on it.  We will see how far we get23:18
*** KunalGandhi has quit IRC23:20
openstackgerritShashank Kumar Shankar proposed openstack/octavia: Introduce Octavia v2 API for Pools with LBaaS align  https://review.openstack.org/42019523:22
*** ducttape_ has joined #openstack-lbaas23:25
openstackgerritNir Magnezi proposed openstack/octavia: Fix the amphora-agent support for RH based Linux flavors  https://review.openstack.org/33184123:26
nmagnezijohnsom, ^^23:26
johnsomOk23:27
rm_workI mean, we can review as late as tomorrow night, right? :)23:57
*** catintheroof has quit IRC23:58
johnsomYes23:59
*** catintheroof has joined #openstack-lbaas23:59

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