Thursday, 2017-01-26

rm_work.. having issues posting to gerrit00:03
rm_workseems to LOAD fine00:03
*** catintheroof has quit IRC00:03
rm_workthere we go00:05
rm_workgot  few 500s but it works now00:05
johnsomMaybe it's running low on disk space again...00:06
*** ducttape_ has quit IRC00:08
*** ducttape_ has joined #openstack-lbaas00:24
rm_workwtf00:40
rm_workhttps://gist.github.com/rm-you/9105ff3a83106caa37097aaf07afc2a500:42
rm_workdifferent behavior for ONE of the exceptions...00:42
rm_workI tested all 4 and only InvalidSortKey does it that way00:42
rm_workeverything else is the proper way00:42
rm_workor, what I assume is the proper way00:42
johnsomUmm, looking at your posted code, that is super strange00:43
rm_workI lied, it's 50/5000:43
johnsomThe first one looks right to me00:44
rm_workupdated the gist00:44
rm_workrefresh00:44
rm_workI would assume the first two are right00:44
rm_workthere's no difference in the actual exception code00:45
rm_work*where* they're called is ... maybe different? tracking it down00:45
johnsomYeah, the string is right.  The exception code looks right00:46
*** armax has quit IRC00:50
*** armax has joined #openstack-lbaas00:56
*** fnaval has quit IRC00:57
rm_workyeah don't worry about it01:00
rm_workhopefully you aren't :P01:01
rm_workit's lower priority01:01
johnsomI am testing away01:01
johnsomPeeked in between01:01
rm_workmy guess is that it depends on WHERE I throw the exception01:01
rm_worklike, what layer01:02
rm_workit's only marker and sort-key that are different01:02
rm_workand those two are the ones that can be thrown in the apply()01:02
rm_workthough ... i'd EXPECT the sort-key one to be thrown above that :/01:02
johnsomYeah, maybe something in between is messing with it, like a reraise or something01:02
johnsomhttps://www.irccloud.com/pastebin/aJ5JNRYQ/01:03
johnsomSo existing code isn't broken01:04
rm_workyep it's the ones inside the apply() that don't work the same01:04
rm_workso i'll have to trace that up01:04
*** csomerville has quit IRC01:04
rm_worknice01:04
rm_workso we're good?01:04
rm_workoh01:04
rm_workthat's just the ubuntu half of the matric01:04
rm_work*matrix01:04
johnsomNope, I need to run through the same with the centos image01:04
*** ducttape_ has quit IRC01:06
rm_workin controllers/load_balancer.py get_all() which is the wsme_exposed method01:06
rm_workthere's no catches01:06
rm_workall the way down to where the error is raised01:06
rm_workso i dunno01:06
rm_workmaybe it's that there SHOULD be?01:06
rm_work:/01:06
rm_workbut that's definitely it... if it gets raised during the hook processing, it gives what I assume is the "correct" type of 400 response01:08
rm_workif it gets raised during the repository processing, it gives the other one01:08
rm_workso, either way it DOES return status 400... it's just the body of the response that's different01:10
rm_workand I don't think that's my problem? maybe01:10
rm_workanywho, fixed one last bug and I guess I'll sit on it for now01:10
openstackgerritMerged openstack/octavia: Fix active/standby under python3  https://review.openstack.org/42485101:13
openstackgerritAdam Harwell proposed openstack/octavia: Implement sorting and pagination for octavia  https://review.openstack.org/38214701:13
rm_workah should have rebased :P01:14
rm_workok i'm out for a bit, might be on later johnsom so feel free to leave me notes01:14
rm_workand by might, i mean... i'll be on later01:14
johnsomOk, enjoy01:14
rm_workmade some pressure cooker ribs :P #winning01:15
johnsomNICE, garlic shrimp with pasta for me tonight01:15
*** kevo has quit IRC01:32
openstackgerritHa Dung proposed openstack/octavia: Fixes some current typo errors on octavia project.  https://review.openstack.org/42548501:43
*** catintheroof has joined #openstack-lbaas01:44
*** gongysh has joined #openstack-lbaas01:49
*** catintheroof has quit IRC01:49
johnsomUgh, why can't the distros agree on anything02:03
johnsomcentos ifup doesn't have an -a option02:03
*** armax has quit IRC02:12
*** ducttape_ has joined #openstack-lbaas02:19
*** ducttape_ has quit IRC02:24
*** fnaval has joined #openstack-lbaas02:37
*** csomerville has joined #openstack-lbaas02:39
johnsomhttps://www.irccloud.com/pastebin/K4Oblals/02:49
openstackgerritMichael Johnson proposed openstack/octavia: Fix the amphora-agent support for RH based Linux flavors  https://review.openstack.org/33184102:50
*** csomerville has quit IRC02:52
*** reedip_ has joined #openstack-lbaas03:00
*** kevo has joined #openstack-lbaas03:01
*** ducttape_ has joined #openstack-lbaas03:40
*** fnaval has quit IRC03:56
*** fnaval has joined #openstack-lbaas03:57
*** ducttape_ has quit IRC04:00
*** fnaval has quit IRC04:01
*** armax has joined #openstack-lbaas04:10
*** ducttape_ has joined #openstack-lbaas04:22
rm_workis that a GNU thing or...04:24
rm_worksometimes I'm not sure which is actually more "standard" and which isn't04:24
rm_worklike, I thought OSX was non-standard on some utils but turns out it's actually that ubuntu had a bunch of extra gnu stuff that wasn't standard04:25
*** armax has quit IRC04:34
*** ducttape_ has quit IRC04:34
*** reedip_ has quit IRC05:05
*** sputnik13 has quit IRC05:35
openstackgerritMichael Johnson proposed openstack/octavia: Fix the amphora-agent support for RH based Linux flavors  https://review.openstack.org/33184105:38
*** ducttape_ has joined #openstack-lbaas05:50
*** eezhova has joined #openstack-lbaas05:52
*** ducttape_ has quit IRC05:55
*** gcheresh_ has joined #openstack-lbaas05:58
*** sputnik13 has joined #openstack-lbaas06:00
johnsomhttps://www.irccloud.com/pastebin/IJJEcKJh/06:47
johnsomSo, down to one issue in my testing.  We will have to be creative to deal with the lack of ifup -a.06:50
*** tesseract has joined #openstack-lbaas07:11
*** diltram has quit IRC07:14
*** eezhova has quit IRC07:14
*** ducttape_ has joined #openstack-lbaas07:21
*** AlexeyAbashkin has joined #openstack-lbaas07:22
*** ducttape_ has quit IRC07:26
*** Alex_Staf has quit IRC07:28
*** yamamoto has quit IRC07:41
*** eezhova has joined #openstack-lbaas07:41
*** yamamoto has joined #openstack-lbaas08:16
openstackgerritAbed Abu dbai proposed openstack/octavia: Active-Active Topology - Cluster DB Tasks  https://review.openstack.org/42557808:31
openstackgerritAbed Abu dbai proposed openstack/octavia: Active-Active Topology - Cluster DB Tasks.  https://review.openstack.org/40976408:46
*** ducttape_ has joined #openstack-lbaas08:48
*** kevo has quit IRC08:50
openstackgerritAbed Abu dbai proposed openstack/octavia: Active-Active Topology - added disable arp  https://review.openstack.org/42517508:56
openstackgerritAbed Abu dbai proposed openstack/octavia: Active-Active Topology - register/uregister amphorae tasks.  https://review.openstack.org/40976508:56
*** Alex_Staf has joined #openstack-lbaas08:57
*** gongysh has quit IRC09:10
*** AlexeyAbashkin has quit IRC09:10
*** amotoki has quit IRC09:27
*** amotoki has joined #openstack-lbaas09:28
*** amotoki has quit IRC09:32
*** amotoki has joined #openstack-lbaas09:39
rm_workjohnsom: why do we even need to run that? do netns not honor auto network configs?09:52
rm_workIE, when we move the interface into the netns we can't just set "auto <interfacename>" at the same time?09:52
rm_workoh, what... that's *for" bringing up auth interfaces09:55
rm_workwhy doesn't that "just happen"09:55
rm_workI have some thoughts but I need to spin that patch10:04
rm_worki'll start it running tonight and look at it tomorrow10:04
*** amotoki has quit IRC10:18
*** Brenda has quit IRC10:21
*** Brenda has joined #openstack-lbaas10:22
nmagnezirm_work, i think i've found a solution for what johnsom mentioned. 'ifup *' works on centos, testing if Ubuntu will not spoil the party10:23
rm_workhah10:24
nmagnezi:)10:24
rm_workis it very specific?10:24
nmagnezirm_work, ifup * ?10:25
rm_workugh10:27
rm_workmacbook just kernel paniced10:27
nmagnezirm_work, yay for Sierra10:27
rm_workI'm still on El Capitan10:28
rm_worksomething corporate installed is my guess...10:28
nmagnezirm_work, yay for not upgrading :D10:28
nmagneziso you cannot upgrade until IT allows it?10:28
rm_worklol not sure10:30
rm_workprolly could? I think the VPN would still work10:30
rm_workand that's all that matters10:30
rm_workbut it's annoying to redo everything <_<10:30
rm_workcosts me at least a day or two just to get everything set up again the way i want10:31
rm_workaaanywho out for a bit10:32
rm_workhopefully your fix works10:32
rm_workit can't pick based on if -a is valid?10:32
rm_worklike, try -a and if it's bad, do *10:32
nmagnezi# ip netns exec amphora-haproxy ifdown *10:36
nmagneziUnknown interface *10:36
nmagnezirm_work, ^^ that makes me really sad..10:36
nmagnezisame for ifup10:36
nmagnezijohnsom, ^^ I will replay with findings in the patch10:38
*** openstackgerrit has quit IRC11:17
*** catintheroof has joined #openstack-lbaas12:27
*** catintheroof has quit IRC12:38
*** catintheroof has joined #openstack-lbaas12:38
*** sferna1_ has joined #openstack-lbaas12:41
*** sanfern has quit IRC12:45
*** Alex_Staf has quit IRC12:49
*** catinthe_ has joined #openstack-lbaas12:57
*** catintheroof has quit IRC12:59
*** mugsie_ has quit IRC13:00
*** mugsie_ has joined #openstack-lbaas13:02
*** bedis has quit IRC13:05
*** bedis has joined #openstack-lbaas13:05
*** amotoki has joined #openstack-lbaas13:24
*** openstackgerrit has joined #openstack-lbaas13:43
openstackgerritAbed Abu dbai proposed openstack/octavia: Active-Active Topology - added disable arp  https://review.openstack.org/42517513:43
openstackgerritAbed Abu dbai proposed openstack/octavia: Active-Active Topology - register/uregister amphorae tasks  https://review.openstack.org/40976513:43
openstackgerritAbed Abu dbai proposed openstack/octavia: Active-Active Topology - Cluster DB Tasks  https://review.openstack.org/40976413:43
*** Alex_Staf has joined #openstack-lbaas13:45
*** ducttape_ has quit IRC13:53
*** ducttape_ has joined #openstack-lbaas14:13
*** matt-borland has joined #openstack-lbaas14:21
*** _ducttape_ has joined #openstack-lbaas14:31
*** ducttape_ has quit IRC14:32
openstackgerritSindhu Devale proposed openstack/octavia: Introduce Octavia v2 API for listeners  https://review.openstack.org/40506014:35
*** gcheresh_ has quit IRC14:45
*** coreycb has joined #openstack-lbaas14:46
coreycbhello, anyone familiar with neutron-lbaas-dashboard?  i'm curious why it only has stable/mitaka and master branches, and no stable/newton branch.14:46
dasmjohnsom: ^14:47
xgermancoreycb there was no developement in newton so no release14:47
*** beardedeagle has joined #openstack-lbaas14:47
*** _ducttape_ has quit IRC14:47
coreycbxgerman, ok but how do you provide stable support for newton?14:48
xgermanwe will definitely cut a release if there are bug fixes which have been backported14:48
xgermanbut the dashboard got developed by people who since left the dashboard project so support is slim14:49
xgermanwe anticipate an O release though to catch up with horizon changes14:49
coreycbxgerman, ok so it sounds like it is still supported, if only in maintenance mode though.14:50
xgermanyeah, horizon itself is short handed…14:51
xgermanI actually installed stable-mitaka on master LBaaS on devstack and it worked… so YMMV14:51
coreycbxgerman, appreciate the info, thanks14:52
dasmxgerman: but, afaik ankur-gupta-f1 tried to do the same, and had some issues. although i don't know the details14:52
xgermanyeah, it’s really YMMV14:52
dasmTrue :)14:53
*** sferna1_ has quit IRC14:55
openstackgerritAbed Abu dbai proposed openstack/octavia: ACTIVE-ACTIVE Topology - Initial Cluster Manager  https://review.openstack.org/40523814:56
openstackgerritAbed Abu dbai proposed openstack/octavia: Active-Active Topology - added disable arp  https://review.openstack.org/42517514:56
openstackgerritAbed Abu dbai proposed openstack/octavia: ACTIVE-ACTIVE Topology - Distributor related tasks  https://review.openstack.org/40695114:56
openstackgerritAbed Abu dbai proposed openstack/octavia: ACTIVE-ACTIVE Topology - Distributor image creation  https://review.openstack.org/40359414:56
openstackgerritAbed Abu dbai proposed openstack/octavia: ACTIVE-ACTIVE Topology : create distributor network flow  https://review.openstack.org/40976314:56
openstackgerritAbed Abu dbai proposed openstack/octavia: Active-Active Topology - register/uregister amphorae tasks  https://review.openstack.org/40976514:56
openstackgerritAbed Abu dbai proposed openstack/octavia: ACTIVE-ACTIVE - distributor certificate tasks  https://review.openstack.org/40695214:56
openstackgerritAbed Abu dbai proposed openstack/octavia: Active-Active Topology - Cluster DB Tasks  https://review.openstack.org/40976414:56
openstackgerritAbed Abu dbai proposed openstack/octavia: ACTIVE-ACTIVE Topology - distributor creation flow  https://review.openstack.org/40695314:56
openstackgerritAbed Abu dbai proposed openstack/octavia: ACTIVE-ACTIVE Topology - create shared distributor  https://review.openstack.org/40695414:56
*** jamespage has joined #openstack-lbaas14:56
coreycbjamespage, hey was just chatting with xgerman and dasm about neutron-lbaas-dashboard14:56
* jamespage waves at xgerman and dasm14:57
dasmo/14:57
jamespageso....14:57
jamespagesounds like we don't really have a lbaas v2 horizon past mitaka atm?14:58
jamespageis that accurate?14:58
coreycbjamespage, grepping the code it looks like there is lbaas v2 support14:59
jamespagecoreycb, directly in horizon ?14:59
xgermanjamespage just we didn’t release it doesn’t mean it doesn’t work15:00
coreycbjamespage, i'm just looking at the lbaas-dashboard code15:00
jamespagexgerman, sorry I did not mean to imply its not functional15:01
coreycbxgerman, you did mention ymmv15:01
xgermanyep15:01
coreycbxgerman, thanks again for the info15:07
xgermany.w.15:07
jamespagexgerman, is it likely that the 1.0.0 release will just work with newton/ocata?15:07
xgermanyep15:08
jamespageawesome15:08
jamespagexgerman, thanks for the info :-)15:08
xgermanour API is the same we just need to make sure Horizon doesn’t make too many breaking changes15:08
* jamespage crosses fingers15:12
johnsom jamespage There were not enough true changes to cut a newton release, just use the mitaka release on newton.  I will be releasing for ocata however15:24
johnsomcoreycb: ^^^15:24
coreycbjohnsom, ok thanks15:25
nmagnezimorning johnsom15:25
johnsomMorning15:26
nmagnezijohnsom, I think I've found a solution to the "ifup -a" issue15:26
nmagnezireplied in the patch15:26
johnsomPretty good test results.  I did fix a few things to get there, but looking good15:26
nmagnezijohnsom, we can iron those out today15:27
johnsomThat would be good.  I will take a look when I get to the office15:27
nmagneziit is indeed too early :)15:27
nmagneziI plan to submit a PS, let me know if that will conflict with stuff that you do15:28
johnsomNo, I pushed my changes up last night, so go for it15:30
nmagneziack15:31
*** fnaval has joined #openstack-lbaas15:34
*** armax has joined #openstack-lbaas15:37
*** catintheroof has joined #openstack-lbaas15:39
*** catinthe_ has quit IRC15:42
*** openstackgerrit has quit IRC15:48
*** sanfern has joined #openstack-lbaas16:01
*** trevormc has joined #openstack-lbaas16:02
*** csomerville has joined #openstack-lbaas16:05
*** eezhova has quit IRC16:11
*** Alex_Staf has quit IRC16:24
johnsomnmagnezi so ifup * works on centos?16:25
johnsomI will test that and come up with a plan.  I think we can just put a conditional in the template rather than create another separate template16:26
*** cody-somerville has joined #openstack-lbaas16:28
*** cody-somerville has quit IRC16:28
*** cody-somerville has joined #openstack-lbaas16:28
*** csomerville has quit IRC16:30
*** openstackgerrit has joined #openstack-lbaas16:47
openstackgerritNakul Dahiwade proposed openstack/octavia: Align Octavia API to n-lbaasv2 for L7Policy  https://review.openstack.org/40632816:47
openstackgerritShashank Kumar Shankar proposed openstack/octavia: Align Octavia API to n-lbaasv2 for L7Rules  https://review.openstack.org/40633616:52
*** tesseract has quit IRC16:55
openstackgerritNakul Dahiwade proposed openstack/octavia: Align Octavia API to n-lbaasv2 for L7Policy  https://review.openstack.org/40632816:59
openstackgerritAnkur proposed openstack/octavia: Add v2 load balancer type and controllers  https://review.openstack.org/42354317:02
johnsomnmagnezi Process: 880 ExecStartPre=/sbin/ip netns exec amphora-haproxy ifup * (code=exited, status=1/FAILURE)17:02
johnsom[root@amphora-64e1bb17-0272-47ef-a774-bdf74fce2579 centos]# ifup *17:03
johnsom/sbin/ifup: configuration for * not found.17:03
johnsomUsage: ifup <configuration>17:03
*** ducttape_ has joined #openstack-lbaas17:12
*** eezhova has joined #openstack-lbaas17:21
*** _ducttape_ has joined #openstack-lbaas17:26
*** ducttape_ has quit IRC17:30
johnsomI have a plan17:35
nmagnezijohnsom, worked for me when I tested it17:37
nmagnezijohnsom, but didn't work on Ubuntu17:37
johnsomIt didn't work for me on centos 7.  That is the paste from above.17:37
johnsomBut I have something that does work, coding it up now17:37
*** kevo has joined #openstack-lbaas17:37
johnsomnmagnezi are you going to post a patch?17:38
johnsomI am going to conditionally run this:17:38
johnsomExecStartPre=-/bin/awk '{ system("/sbin/ip netns exec amphora-haproxy ifup " $2)}' /var/lib/octavia/plugged_interfaces17:38
*** _ducttape_ has quit IRC17:55
*** ducttape_ has joined #openstack-lbaas17:55
*** _ducttape_ has joined #openstack-lbaas17:56
*** _ducttape_ has quit IRC17:58
*** ducttape_ has quit IRC18:00
*** SumitNaiksatam has joined #openstack-lbaas18:00
*** eezhova has quit IRC18:03
xgermanjohnsom you recall how our encrypted ramfs worked18:04
xgerman?18:04
johnsomThe original version I wrote, yes, kind of18:04
xgermanI installed the Lunke image and amp comes up but no ramdisk in /var/lib/octavia/certs18:04
xgermanor do I need to switch to the namespace to see it?18:05
johnsomhttps://review.openstack.org/#/c/375905/18:07
johnsomI don't think it was tied to the namespace, I think it was global.18:08
johnsomYeah, looking back at my upstart script for it, it was global, not namespace.18:09
xgermanyeah, that’s what I thought18:09
johnsomAh, but I do remember there is a flag you need to use to see it18:09
xgermanso I would expect to see it somewhere mounted18:09
johnsomTry df -a18:10
xgermanhttps://www.irccloud.com/pastebin/7y9Fq7It/18:10
xgermannope18:10
openstackgerritShashank Kumar Shankar proposed openstack/octavia: Introduce Octavia v2 API for Pools with LBaaS align  https://review.openstack.org/42019518:11
johnsomxgerman root18:11
xgermansame18:15
xgermanhttps://www.irccloud.com/pastebin/0jbgYQYg/18:15
xgermanwell, at least this LUKE change doesn’t break anything ;-)18:16
xgermanit probably is suppoed to come up with systemd but this eludes me18:18
johnsomYeah, it's there, I wonder if this is an startup order issue with xenial18:21
johnsomservice cert-ramfs-ecrypt status18:21
johnsom● cert-ramfs-ecrypt.service18:21
johnsom   Loaded: loaded (/usr/lib/systemd/system/cert-ramfs-ecrypt.service; disabled;18:21
johnsom   Active: inactive (dead)18:21
xgermanmmh, so we can merge Luke but that would be a Noop since the fs doesn’t work18:25
xgermanarrgh18:25
johnsomLet's not merge that.18:25
xgermanagreed18:25
johnsomI really want to look at it and understand why I decided to not use it the first time18:25
xgermanbut do we mark it -1?18:25
*** SumitNaiksatam has quit IRC18:25
johnsomYeah, it looks like the existing code has some xenial fallout18:26
johnsomxgerman so his patch doesn't work either?  It's a very different systemd script18:27
xgermanwell, you saw my df-a18:28
xgermanbut it has the systemv servcie18:28
johnsomjournalctl -xel --no-pager | less18:29
johnsomThat should show you the error18:29
johnsomOk, yeah, it needs updating for the order anyway18:29
*** rcernin has joined #openstack-lbaas18:32
*** amotoki has quit IRC18:33
xgermancan’t see anyhting but I am also not sure what I am looking for18:37
johnsomOk, let's hold it for deeper inspection18:37
xgerman+118:37
xgermanyeah, I though it would be as easy as maing an amp, and running mount18:38
johnsomYeah, I would expect it to be visible via either df -a or mount18:39
openstackgerritSindhu Devale proposed openstack/octavia: Octavia v2 API for listeners  https://review.openstack.org/42474418:49
*** harlowja has quit IRC19:02
johnsomnmagnezi I am ready to commit a patch.  Is that ok?19:11
*** eezhova has joined #openstack-lbaas19:19
openstackgerritMichael Johnson proposed openstack/octavia: Fix the amphora image support for RH Linux flavors  https://review.openstack.org/33184119:29
johnsomnmagnezi ^^^19:29
nmagnezijohnsom, yes sir19:30
johnsomFYI, I pushed up a patch with fixes for syslog and the reboot issue19:31
*** amotoki has joined #openstack-lbaas19:34
nmagnezijohnsom, "Co-Authored-By:" you read my mind :)19:44
rm_workoh what, we're supposed to handle &page_reverse=False19:47
rm_workand I don't think this does the atom tokens for navigation yet... >_>19:48
rm_workyeah this spec is totally not what anyone does yet i think19:48
rm_workjohnsom: i'm cleaning up the priority review list a little19:57
johnsomreverse?  Haven't heard of that one20:01
johnsomrm_work cool.  I just asked the folks working on API stuff to update as well20:01
blogansshank: ill come by in like 30 mins20:01
johnsomI am going to run to lunch, then take a last pass on patches and start putting in WIP release patches for O-320:02
rm_workk, we still trying to merge this centos patch first?20:05
*** harlowja has joined #openstack-lbaas20:09
nmagnezijohnsom, may i send a ps?20:18
johnsomYes, just make sure you rebase on the one I just pushed20:19
nmagnezijohnsom, i did. I just want to make sure I fixed the failing functional tests20:19
openstackgerritNir Magnezi proposed openstack/octavia: Fix the amphora image support for RH Linux flavors  https://review.openstack.org/33184120:20
johnsomrm_work: yeah, I think we can get the RH patch in.  It is now passing my tests, it is just down to unit tests20:20
rm_workjohnsom: how long until we can merge "features" again?20:21
sshankblogan, Cool.20:25
johnsomDepends how testing goes and if the patch would make backports hard.  Worst case 13th.  Best case next week20:25
rm_workk20:25
nmagnezijohnsom, I'm writing a release notes for this one20:25
openstackgerritShashank Kumar Shankar proposed openstack/octavia: Introduce Octavia v2 API for Pools with LBaaS align  https://review.openstack.org/42019520:26
johnsomGrabbing lunch, back in a while20:26
nmagnezijohnsom, will also remove this which was added my mistake https://review.openstack.org/#/c/331841/71/octavia/compute/drivers/nova_driver.py20:26
nmagnezijohnsom, enjoy20:26
nmagnezi:)20:26
*** gcheresh_ has joined #openstack-lbaas20:34
*** gcheresh_ has quit IRC20:43
openstackgerritSindhu Devale proposed openstack/octavia: Octavia v2 API for listeners  https://review.openstack.org/42474420:53
*** trevormc has quit IRC21:05
openstackgerritNir Magnezi proposed openstack/octavia: Fix the amphora image support for RH Linux flavors  https://review.openstack.org/33184121:28
nmagnezijohnsom, ^^21:29
*** yamamoto has quit IRC21:30
blogansindhu, ndahiwade, sshank: i don't see why there are 2 reviews for each resource21:36
*** gcheresh_ has joined #openstack-lbaas21:36
bloganthere's one for introducing, that just keeps the old v1 references, and then one that just changes those references to v221:36
bloganwhy not just do all that in the same patch?21:37
sshankblogan, We made Pools separate since members come under it. This way 2 can work on them independently.21:39
ndahiwadeSame would be for L7 policy and rules21:40
sshankblogan, Old pools is parent of old members and new pools.21:40
blogansshank: no that i udnerstand21:40
blogani mean like tehse 2 reviews: https://review.openstack.org/#/c/40592221:41
bloganhttps://review.openstack.org/#/c/420195/21:41
sshankblogan, The first one is used by https://review.openstack.org/#/c/405932/21:41
bloganso why can't you just combine the 2 reviews i linked and have the members on depend on that one?21:42
blogani'm not understanding why there are 2 pools reviews21:42
bloganand listeners, and members of course, but i'm using pools as an example21:43
johnsomOk, I am going to pull down the latest RH patch, make sure I didn't do something dumb with a quick test of those changes, then move on to starting the release patches21:46
*** catintheroof has quit IRC21:47
nmagnezijohnsom, anything I can do to help you?21:47
*** gcheresh_ has quit IRC21:47
johnsomArgh, too much of my C/C++/C# code coming out...21:48
johnsomnmagnezi So other than my goofs, the tests that are there are passing?21:49
nmagnezijohnsom, so far so good. including functional tests.21:49
nmagnezijohnsom, i'm watching Zuul.21:50
johnsomOk.  So are you good to stop for the day?  I can take it from here.  I want to add to the release notes a bit, then I will work to open some bugs about missing tests.  If we pass the gate tests and my quick test, I think we can merge21:51
johnsomThen we have some time to finish the tests21:53
johnsomSound like a plan?21:53
johnsomWe passed sonar, so that is a good sign.21:54
openstackgerritShashank Kumar Shankar proposed openstack/octavia: Introduce Octavia v2 API for pools  https://review.openstack.org/40592221:57
nmagnezijohnsom, yes. also if you file bugs feel free to assign those to me.22:07
johnsom+122:07
nmagnezijohnsom, will we have time to push those to Ocata?22:07
nmagnezijohnsom, btw this failed (non-voting), but seem to be unrelated http://logs.openstack.org/41/331841/72/check/gate-octavia-v1-dsvm-py3x-scenario-ubuntu-xenial-nv/a74912d/logs/testr_results.html.gz22:08
*** beardedeagle has quit IRC22:08
johnsomYes, we have time on the bug fixes, but should try to get them in as soon as possible.22:09
johnsomYes, ignore the py3x tests22:09
*** yamamoto has joined #openstack-lbaas22:11
*** eezhova has quit IRC22:11
nmagnezijohnsom, I'm looking at https://releases.openstack.org/ocata/schedule.html but i'm not so sure, when is the final date for those tests? just want to know how much time I have left22:12
johnsomnmagenzi The absolute last week is Feb 13th, but we really want them in sooner than that.22:13
johnsomDang, we have a bug again.22:16
johnsomhaproxy config didn't verify22:16
nmagnezijohnsom, I'll prioritize accordingly.22:16
nmagnezijohnsom, Ubuntu or Centos?22:17
johnsomcentos22:17
nmagnezijohnsom, any specific type of configuration?22:18
johnsomNot sure yet, I have to dig into it22:18
johnsomJan 26 22:14:33 host-192-168-0-9 amphora-agent: 2017-01-26 22:14:33.233 1668 ERR22:18
johnsomOR octavia.amphorae.backends.agent.api_server.listener [-] Failed to verify haproxy file: Command '['haproxy', '-c', '-L', 'hB918ZoSmqBl4oQFSdWWUCEouy0', '-f', '/var/lib/octavia/c01377e1-7f73-42f8-a621-485b7438de9d/haproxy.cfg.new']' returned non-zero exit status 122:18
nmagnezijohnsom, i will look into this as well but i need to respin my devstack so it will take me few min22:19
johnsomnmagnezi My mistake, the user group change didn't take22:23
* nmagnezi note to self: auto-detection to user_group22:23
nmagnezijohnsom, i'll do ^^ right after the tests, which are more important.22:24
johnsomYeah, do the tests.22:24
johnsomWe need to discuss options on the user_group.  I will add that to the release notes.22:24
johnsomuser_group is a little complicated since we render the config on the controller.  So, we just need to decide how best to handle that22:25
*** matt-borland has quit IRC22:26
nmagnezijohnsom, we have +1 from jenkins22:39
johnsomOk cool.  I still have a reboot bug.  It didn't detect it was RH for some reason22:39
johnsomlooking at that now22:39
nmagnezijohnsom, is it related to the ifup action for all NICs? or is it something else?22:40
johnsomYes, it's just the conditional not working right22:41
rm_workjohnsom: i'm out tomorrow... but I can hop on and do a review if you need me to22:41
rm_workbut i won't be able to do much complex testing22:41
johnsomOk, how late-ish are you on today?22:41
rm_workummm22:53
rm_workbasically, not after now22:53
rm_workbut22:53
rm_workemail me or SMS me22:53
rm_workI'll be around much later22:53
rm_worklike 9pm PST prolly22:53
johnsomDArn22:53
johnsomhttps://review.openstack.org/42469522:53
johnsomhttps://review.openstack.org/42364922:53
rm_workLGTM22:53
johnsomI was just finallizing the lbaas list22:54
rm_workwhy did our functional fail on the first one22:54
johnsomfunctional is broken on nlbaas22:54
rm_workaugh22:54
johnsomIt does *nothing*22:54
rm_workyeah that perms but22:54
rm_work*bug22:54
johnsomHas been for a long time.  I think Doug said it was started, but never finished22:54
rm_work+A'd22:55
rm_workboth look sane22:55
rm_workand no major testing issues it seems22:55
rm_workbrb in 15m and then out for a while22:55
openstackgerritMerged openstack/neutron-lbaas-dashboard: Updated from global requirements  https://review.openstack.org/42539423:00
nmagnezijohnsom, it is still stacking so i have yet to debug this. any luck so far?23:06
johnsomI had to jump to a release task, will be back to it soon23:06
nmagneziok23:07
openstackgerritMerged openstack/neutron-lbaas-dashboard: Update for ui bootstrap changes  https://review.openstack.org/41821823:10
openstackgerritMichael Johnson proposed openstack/octavia: Fix the amphora image support for RH Linux flavors  https://review.openstack.org/33184123:15
johnsomThat will fix the issue23:15
* nmagnezi looks23:18
* johnsom thinks he is doing too many things at once...23:19
nmagnezijohnsom, I think you reverted my last changes23:19
johnsomCrumb, I just saw that.  Yeah, fixing...23:20
nmagnezijohnsom, sorry to be useless about this reboot debug. it takes my machine forever to stack for some reason23:21
*** ipsecguy has quit IRC23:24
nmagnezijohnsom, anything else I can help with?23:28
nmagnezijohnsom, if not, I'm calling it a night.23:29
johnsomNo, I think I am good.  Just updating the release notes now.  Have a good night!23:29
openstackgerritMerged openstack/neutron-lbaas: Replaces uuid.uuid4 with uuidutils.generate_uuid()  https://review.openstack.org/40676823:33
openstackgerritMichael Johnson proposed openstack/octavia: Fix the amphora image support for RH Linux flavors  https://review.openstack.org/33184123:34
*** ipsecguy has joined #openstack-lbaas23:35
openstackgerritMerged openstack/neutron-lbaas: Use neutron-lib portbindings api-def  https://review.openstack.org/42282223:51
openstackgerritMerged openstack/neutron-lbaas: Remove psutil dependency  https://review.openstack.org/42469523:52
openstackgerritMerged openstack/neutron-lbaas: Updated from global requirements  https://review.openstack.org/42364923:56
openstackgerritMerged openstack/neutron-lbaas: The lb namespace can not be deleted  https://review.openstack.org/34396323:56
*** rcernin has quit IRC23:56
*** rcernin has joined #openstack-lbaas23:57
*** rcernin has quit IRC23:59

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