Wednesday, 2016-01-27

sbalukoffOh gerrit, why you no worky?00:02
sbalukoffIs it just me or did gerrit fall over for other people as well?00:03
sbalukoffOk...  it's back. Hmmm.00:06
johnsomIt's been doing that to me for the last two weeks.  Dies and then comes back00:07
*** Aish has quit IRC00:09
*** Aish has joined #openstack-lbaas00:11
*** _cjones_ has quit IRC00:12
*** _cjones_ has joined #openstack-lbaas00:12
*** ducttape_ has quit IRC00:14
openstackgerritStephen Balukoff proposed openstack/octavia: Shared pools support  https://review.openstack.org/25636900:17
sbalukoffTrevorV: As most of your comments had to do with... well, comments... I'm hoping that I can get that +1 soon, eh. ;)00:18
sbalukoff(er... referring the the new shared pools patchset that just went up.)00:18
*** yamamoto_ has quit IRC00:39
*** dnovosel1_ has quit IRC00:39
openstackgerritDoug Wiegley proposed openstack/neutron-lbaas: WIP - DO NOT MERGE - ci test, 1  https://review.openstack.org/26489800:58
openstackgerritDoug Wiegley proposed openstack/neutron-lbaas: WIP - DO NOT MERGE - ci test, 1  https://review.openstack.org/26489800:58
openstackgerritDoug Wiegley proposed openstack/neutron-lbaas: Migrate some stuff to neutron-lib  https://review.openstack.org/25326301:00
openstackgerritMerged openstack/octavia: Change the length of name in the algorithm  https://review.openstack.org/27117801:01
*** dnovosel__ has joined #openstack-lbaas01:03
openstackgerritMerged openstack/octavia: Add APP_COOKIE session persistence type  https://review.openstack.org/27234501:07
johnsomCRITICAL octavia [-] error: [Errno 98] Address already in use01:09
johnsomWell, that is a new gate failure....01:09
johnsomINFO octavia.cmd.api [-] Starting API server on 0.0.0.0:987601:09
xgermanfun, fun01:09
johnsomun-related change too01:09
xgermanwondering if sbalukoff asserting IBMs copyright on each file is kosher… never added HP copyright to a file which already had copyright01:10
*** mgarza has quit IRC01:10
johnsomIf we make changes, we should be adding them01:10
sbalukoffxgerman: TrevorV recommended I do that. :/01:10
xgermanlet’s ask the great dougwig if there is some policy01:11
dougwigsigh.01:11
johnsomThough I have to say....  Adding a copyright for changing the constants names, oye01:11
* sbalukoff rolls his eyes.01:12
sbalukoffI honestly don't care about that-- it's all apache 2 anyway.01:12
dougwigif you significantly change a file, you can add a copyright line. however, adding copyright lines is now discouraged, as it's not necessary.01:12
sbalukoffI just do whatever policy says I should do in that case. :P01:12
johnsomhttps://review.openstack.org/#/c/256369/18/octavia/controller/worker/flows/amphora_flows.py01:12
dougwighttps://wiki.openstack.org/wiki/LegalIssuesFAQ#Copyright_Headers01:12
xgermanyeah, I have never seen two copyrights in a single file01:12
sbalukoffxgerman: Really? I've definitely seen that before.01:13
dougwigi just don't put in the line anymore. makes life easier.  :)01:13
xgermanI only do if the file is new01:13
sbalukoffI don't really care--  but I don't like be run around in circles. I only added those because someone complained that I hadn't.01:14
sbalukoffAre y'all going to go back and say I have to remove them all now?01:14
xgermanok, I guess we are getting into yak shaving01:14
sbalukoffYep.01:14
xgermanso if you want to keep them we are living in 2016 not 201501:15
johnsomYeah, I try not to pick on other company's copyright stuff too much.  Just if it's a new file01:15
xgermanand for 2015 ones you might want to write 2015-201601:15
sbalukoffOctavia consists of a lot of files that have been updated many times over the last few years.01:15
johnsomYep01:16
xgermanyep, but if you add a new (c) line please do it right ;-)01:16
xgermanand use the right year01:16
sbalukoffIf we want to go and fix *dates* on the copyright notices (as well as, perhaps, make them compliant with whatever the openstack standard is), i suggest someone file a bug report and get on that.01:16
dougwigsbalukoff: read the faq. if you're in compliance, push back.01:16
sbalukoffI tihnk that's entirely orthogonal to the actual stuff the shared pools stuff does.01:16
xgermanI am saying if you add a new (c) line it should have the correct date01:16
dougwigno no no, do not go retroactively edit.01:17
xgermanyeah, only where you added a new (c)01:17
dougwigthe maintenance of said headers is the main reason they recommend against them now.01:17
xgermane.g. https://review.openstack.org/#/c/256369/18/octavia/api/v1/controllers/member.py01:17
johnsom+1 on not retroactively edit01:17
xgermandougwig, johnsom +101:18
sbalukoff+1 on this is a really annoying conversation to have...  seriously? copyright notices?01:18
* sbalukoff sighs01:18
xgermanthat’s why I don;’t touch them ;-)01:18
xgermanbut putting up the wrong year, I got -1’d for less01:18
sbalukoffWell, more than likely the file you're looking at was actually written in 2015.01:19
sbalukoffThis patch has been open for a while.01:19
dougwigoh snap.01:19
johnsomsbalukoff Not rubbing salt in a wound I didn't create, but are all of those copyrights the only change in that patchset?  Could you just revert and make this all go away?01:19
xgermanwell, HPE’s legal team recommends 2015-2016 in that case01:20
sbalukoffjohnsom: There's a function rename, but that would be easy to duplicate.01:20
xgermanbut yeah, there is a reason I don’t touch (c)01:20
*** minwang2 has quit IRC01:20
johnsomOk, just a thought....01:20
sbalukoffSo...01:20
sbalukoffI cannot express in words how inane I find this subject to be. Given, especially that we're producing open-source apache-2 licensed code.  So!  just tell me what you want me to do and I will do it. But don't change your mind on this and make me shave more yaks, or I will murder you.01:21
sbalukoff<301:22
openstackgerritAishwarya Thangappa proposed openstack/octavia: Adding "region and endpoint_type" parameters to barbican_acl.py  https://review.openstack.org/27286101:22
johnsomdougwig My vote is to revert and be done with it, but I must defer this one to my lieutenant01:23
*** yamamoto_ has joined #openstack-lbaas01:23
xgermanI don;t care but recommend to use current dates in the added (c)01:23
sbalukoffI need an answer, y'all.01:25
sbalukoffAnd you have to mean it!01:25
sbalukoffRevert and be done with it would be *really* easy... but then I don't want to see complaints about copyright lines in this patch again...01:27
johnsomI would only comment if it was a new file with no copyright.01:28
sbalukoffHeh... the FAQ on this:  "We do not yet have guidance for when to add or remove a copyright header in source files."01:29
sbalukoffSweet. Clear as mud.01:29
johnsomI don't see any files that are new in that patch that you don't have covered, so fine by me01:30
sbalukoffjohnsom: OK, so can I take what you have said to mean "Please revert the copyright changes from your latest patchset and I will no longer bug you about this stupid subject again unless you do something stupid or really rude here. Like changing others' copyright notices."01:31
sbalukoffMr. PTL?01:32
johnsomYes sir.  As long as that meets your needs for copyright compliance.01:32
*** Aish has quit IRC01:32
sbalukoffUntil someone at IBM tells me "I done bad," I'll go with the convention here. Which seems to be "add it for new files or *major* changes"01:33
sbalukoffI have no problem claiming ignorance on this.01:33
dougwigi think we've killed enough bits on this one.  my vote would be that it's between sbalukoff and his lawyers what he wants to claim, as long as it doesn't violate that faq.01:35
sbalukoffdougwig: The faq is almost entirely unhelpful:  "We do not yet have guidance for when to add or remove a copyright header in source files. "01:36
dougwigsbalukoff: right, so you can 1) leave it out, 2) add a line when you see fit, 3) update the date as you see fit.  what you can't do: 1) add someone else's, edit someone else's.01:37
sbalukoffdougwig: Then...  right. I was effectively following that guideline in the first place.01:37
sbalukoffBah! Whatever... I'm almost done with this revert anyway.01:38
dougwigsbalukoff: so..  enough bits killed.  carry on.  :)01:38
sbalukoff:)01:38
johnsomsbalukoff I think you were following the guideline in the first place.01:38
sbalukoffYep, I was.01:39
*** jaff_cheng has joined #openstack-lbaas01:40
*** paco20151113 has joined #openstack-lbaas01:43
*** diogogmt has quit IRC01:48
*** _cjones_ has quit IRC01:50
*** _cjones_ has joined #openstack-lbaas01:50
*** _cjones_ has quit IRC01:50
*** _cjones_ has joined #openstack-lbaas01:51
*** _cjones_ has quit IRC01:55
*** bochi-michael has joined #openstack-lbaas01:59
openstackgerritStephen Balukoff proposed openstack/octavia: Shared pools support  https://review.openstack.org/25636902:00
*** bana_k has quit IRC02:17
openstackgerritStephen Balukoff proposed openstack/octavia: Add L7 database structures  https://review.openstack.org/26543002:40
*** ducttape_ has joined #openstack-lbaas02:49
openstackgerritStephen Balukoff proposed openstack/octavia: Update repos for L7 objects / methods  https://review.openstack.org/26552902:58
*** diogogmt has joined #openstack-lbaas02:58
*** fnaval has quit IRC02:58
*** ducttape_ has quit IRC02:59
*** ducttape_ has joined #openstack-lbaas03:00
*** bana_k has joined #openstack-lbaas03:07
*** fnaval has joined #openstack-lbaas03:13
openstackgerritStephen Balukoff proposed openstack/octavia: WIP: Add L7 api  https://review.openstack.org/26569003:15
*** ducttape_ has quit IRC03:17
*** yuanying has quit IRC03:21
*** Aish has joined #openstack-lbaas03:23
*** Aish has quit IRC03:23
*** yuanying has joined #openstack-lbaas03:23
openstackgerritStephen Balukoff proposed openstack/octavia: WIP: Tests for L7 API  https://review.openstack.org/26571803:26
*** yuanying has quit IRC03:28
*** yuanying has joined #openstack-lbaas03:33
*** yuanying has quit IRC03:40
*** yuanying has joined #openstack-lbaas03:40
*** doug-fish has joined #openstack-lbaas03:49
*** yuanying has quit IRC03:56
*** yuanying has joined #openstack-lbaas03:57
*** yuanying_ has joined #openstack-lbaas03:58
*** yuanying has quit IRC04:01
*** ducttape_ has joined #openstack-lbaas04:11
*** madhu_ak has quit IRC04:42
*** ducttape_ has quit IRC04:56
*** fnaval has quit IRC05:06
openstackgerritPhillip Toohill proposed openstack/octavia: Fixes Octavia Amphora RHEL builds  https://review.openstack.org/27290505:21
*** hang has joined #openstack-lbaas05:29
*** jaff_cheng has quit IRC05:33
*** hang has quit IRC05:41
openstackgerritStephen Balukoff proposed openstack/octavia: Fix minor typo in warning message  https://review.openstack.org/27291005:55
rm_workxgerman: are you editing the wiki now or did i just get that email from earlier06:08
rm_worki will add that to my long list of thing to look at tomorrow, there are some notes I had on changes to the TLS howto page06:10
*** minwang2 has joined #openstack-lbaas06:10
xgermanYep, fixed a typo06:19
xgermanThe sni scenario wasn't right06:20
*** minwang2 has quit IRC06:32
*** minwang2 has joined #openstack-lbaas06:38
openstackgerritTrevor Vardeman proposed openstack/octavia: Amphora Create now fails when amphora goes into ERROR status  https://review.openstack.org/26713706:40
*** numans has joined #openstack-lbaas06:43
*** minwang2 has quit IRC06:48
*** nmagnezi_ has joined #openstack-lbaas07:21
*** rcernin has joined #openstack-lbaas07:27
openstackgerritLIU Yulong proposed openstack/neutron-lbaas: Restrict delete pool if it has binding healthmonitor  https://review.openstack.org/17968407:28
*** chlong_zzz is now known as chlong07:31
*** jaff_cheng has joined #openstack-lbaas08:27
*** lijie has joined #openstack-lbaas08:30
*** lijie has left #openstack-lbaas08:30
*** lijie has joined #openstack-lbaas08:31
*** lijie has left #openstack-lbaas08:33
openstackgerritStephen Balukoff proposed openstack/octavia: Update repos for L7 objects / methods  https://review.openstack.org/26552908:36
*** bana_k has quit IRC08:38
openstackgerritStephen Balukoff proposed openstack/octavia: WIP: Add L7 api  https://review.openstack.org/26569008:44
*** armax has quit IRC08:45
openstackgerritStephen Balukoff proposed openstack/octavia: WIP: Tests for L7 API  https://review.openstack.org/26571808:47
*** nmagnezi_ has quit IRC08:58
*** bdrich has joined #openstack-lbaas09:02
*** armax has joined #openstack-lbaas09:04
*** nmagnezi_ has joined #openstack-lbaas09:11
*** woodster_ has quit IRC09:16
*** Alex_Stef has joined #openstack-lbaas09:17
*** armax_ has joined #openstack-lbaas09:19
*** armax has quit IRC09:21
*** armax_ is now known as armax09:21
*** bdrich has quit IRC09:22
*** amotoki has joined #openstack-lbaas09:41
*** nmagnezi_ has quit IRC10:18
*** paco20151113 has quit IRC10:18
*** jaff_cheng has quit IRC10:21
*** bochi-michael has quit IRC10:28
*** armax has quit IRC10:30
*** nmagnezi_ has joined #openstack-lbaas10:31
*** prabampm1 has quit IRC10:50
*** pcaruana has joined #openstack-lbaas11:44
*** prabampm has joined #openstack-lbaas11:59
*** ajmiller has quit IRC12:07
*** ajmiller has joined #openstack-lbaas12:07
*** rtheis has joined #openstack-lbaas12:13
*** pcaruana has quit IRC12:37
*** yamamoto_ has quit IRC12:49
*** chlong has quit IRC13:15
*** chlong has joined #openstack-lbaas13:28
*** numans has quit IRC13:55
*** numans has joined #openstack-lbaas13:56
*** prabampm has quit IRC14:00
*** diogogmt has quit IRC14:01
*** diogogmt has joined #openstack-lbaas14:03
*** yamamoto has joined #openstack-lbaas14:10
*** yamamoto_ has joined #openstack-lbaas14:11
*** neelashah has joined #openstack-lbaas14:12
*** yamamoto has quit IRC14:15
*** ajmiller_ has joined #openstack-lbaas14:18
*** diogogmt has quit IRC14:21
openstackgerritOpenStack Proposal Bot proposed openstack/octavia: Updated from global requirements  https://review.openstack.org/27280814:23
*** ajmiller_ has quit IRC14:34
*** yamamoto_ has quit IRC14:41
*** yamamoto has joined #openstack-lbaas14:48
*** nmagnezi_ is now known as nmagnezi14:52
*** ducttape_ has joined #openstack-lbaas14:56
*** yamamoto has quit IRC14:57
*** yamamoto has joined #openstack-lbaas15:08
*** yamamoto has quit IRC15:19
*** yamamoto has joined #openstack-lbaas15:39
*** TrevorV has joined #openstack-lbaas15:46
*** Alex_Stef has quit IRC15:57
*** nmagnezi has quit IRC15:59
*** Aish has joined #openstack-lbaas16:00
*** mgarza_ has joined #openstack-lbaas16:03
*** numans has quit IRC16:04
*** rcernin has quit IRC16:09
ajmillerdoug-fish ping16:11
doug-fishajmiller: hi16:11
ajmillerGood Morning.  I've been through the loadbalancer/listener/pool/member/monitor patches.  They all work as expected.16:12
doug-fishexcellent!16:13
ajmillerI gave +2's, on the member/monitor patches, and also the npm dep patch.16:14
doug-fishI'll wrap those up - I've been keeping up with them pretty well anyway. Hopefully we can get them merged today!16:14
ajmillerNow I'd like to move on to testing the remaining patches but am running into merge conflicts when I try to cherry-pick them onto my repo.16:14
ajmillerIs there an implied order, or have they just gotten out of sync?16:15
doug-fishpretty sure they are actually out of sync16:15
ajmillerOK.  We16:15
ajmillerWould be good to get someone to look at them and get them usable.16:16
doug-fishI'll harass Lucas and Justin to get those back up to date16:16
ajmillerExcellent!16:16
doug-fishlikewise - great to see these moving! and glad to hear they do what they ought!16:16
ajmillerI did leave a comment about some rendering issues, and I will try to get an actual example/screenshot of the problem/16:18
ajmillerIts something that needs to get fixed before release, but I feel OK merging them as is now.16:18
doug-fishok, I'll take a look as I re-rereview16:18
*** minwang2 has joined #openstack-lbaas16:18
doug-fishyeah, that's my attitude now as well - not quite polished code is okay to get merged until we have the basic set going16:18
ajmillerIt feels like some HTML or CSS is making bad layout assumptiosn somewhere.16:19
*** bdrich has joined #openstack-lbaas16:19
*** woodster_ has joined #openstack-lbaas16:21
*** ducttape_ has quit IRC16:23
*** rtheis_ has joined #openstack-lbaas16:23
*** minwang2 has quit IRC16:24
*** rtheis has quit IRC16:24
*** ducttape_ has joined #openstack-lbaas16:30
*** bana_k has joined #openstack-lbaas16:45
*** piet has joined #openstack-lbaas16:52
pietMorning folks16:52
pietOne of the UI devs that was implementing the LBaaS panels in Horizon had a question for the group16:52
pietIs anyone available to answer a few questions?16:53
johnsompiet I can try16:54
xgermansure16:54
ajmillerme too16:54
xgermanpiet how’s life at the big I?16:54
*** manjeets1 has joined #openstack-lbaas16:55
pietHe16:55
*** manjeets1 has left #openstack-lbaas16:56
pietGood!  But the honeymoon is over16:56
piethttps://openstack.invisionapp.com/share/V4507TP2S#/screens/11555802516:56
*** matt-borland has joined #openstack-lbaas16:56
*** jpomeroy has joined #openstack-lbaas16:56
*** diogogmt has joined #openstack-lbaas16:56
pietCan you take a look at this link?  We have a question around what options are displayed when HTTPS is selected as a monitor16:57
johnsomYes16:57
johnsomI'm on the health monitor panel16:57
pietIs the yellow note accurate?16:58
johnsomHTTPS does not have the last three as we are not doing health monitors over HTTPS yet, it is just checking that the TCP port is functional16:58
*** _cjones_ has joined #openstack-lbaas16:58
*** _cjones_ has quit IRC16:58
johnsomYes, I think it is correct16:58
matt-borlandgotcha, so does the back-end just to some standard GET to / or such?16:58
*** _cjones_ has joined #openstack-lbaas16:58
xgermanyep16:59
johnsomWhen HTTPS monitor is selected it currently uses the same mechanism as TCP16:59
matt-borlandah, does a TCP handshake?16:59
johnsomSo, no GET or /16:59
johnsomYes.16:59
matt-borlandcool, thanks :)16:59
xgermanyeah, we only do get on HTTP16:59
johnsomTo monitor with HTTPS using HTTPS we would have to deal with the certificate issues on backend connections, which we have not yet addressed17:00
matt-borlandmakes sense, thanks!17:00
*** matt-borland has left #openstack-lbaas17:00
*** bdrich has quit IRC17:01
*** jpomeroy has left #openstack-lbaas17:02
johnsomAny other questions we can answer?17:02
johnsomThanks for moving forward with this stuff.  We really appreciate it.17:03
*** yamamoto has quit IRC17:07
*** numans has joined #openstack-lbaas17:07
pietThanks for your help johnsom17:08
*** neelashah has quit IRC17:17
*** minwang2 has joined #openstack-lbaas17:19
*** minwang2 has quit IRC17:23
openstackgerritMichael Johnson proposed openstack/neutron-lbaas: Don't always run Octavia amphroa image build  https://review.openstack.org/26779817:24
*** minwang2 has joined #openstack-lbaas17:25
*** minwang2 has quit IRC17:28
*** neelashah has joined #openstack-lbaas17:31
*** bdrich has joined #openstack-lbaas17:35
johnsomAgenda for today's Octavia IRC meeting: https://wiki.openstack.org/wiki/Octavia/Weekly_Meeting_Agenda#Meeting_2016-01-2717:41
*** armax has joined #openstack-lbaas17:42
*** minwang2 has joined #openstack-lbaas17:42
*** minwang2 has quit IRC17:44
*** minwang2 has joined #openstack-lbaas17:44
*** neelashah has quit IRC17:45
*** minwang2 has quit IRC17:46
*** minwang2 has joined #openstack-lbaas17:58
*** Aish has quit IRC17:58
*** ducttape_ has quit IRC17:59
*** rcernin has joined #openstack-lbaas18:00
*** neelashah has joined #openstack-lbaas18:01
*** ajmiller has quit IRC18:02
*** armax has quit IRC18:02
*** ajmiller has joined #openstack-lbaas18:02
*** ducttape_ has joined #openstack-lbaas18:04
*** Aish has joined #openstack-lbaas18:10
*** bana_k has quit IRC18:11
*** doug-fish has quit IRC18:16
*** madhu_ak has joined #openstack-lbaas18:18
*** fnaval has joined #openstack-lbaas18:18
*** doug-fish has joined #openstack-lbaas18:22
*** bana_k has joined #openstack-lbaas18:23
*** neelashah has quit IRC18:26
*** doug-fish has quit IRC18:27
*** doug-fish has joined #openstack-lbaas18:35
*** doug-fish has quit IRC18:40
*** ducttape_ has quit IRC18:53
*** doug-fish has joined #openstack-lbaas18:55
*** doug-fish has quit IRC19:00
*** doug-fish has joined #openstack-lbaas19:01
*** doug-fis_ has joined #openstack-lbaas19:02
*** doug-fi__ has joined #openstack-lbaas19:03
TrevorVjohnsom should I add to the review list with my bugfix reviews in the agenda or just bring it up in progress?19:05
*** doug-fish has quit IRC19:06
*** numans has quit IRC19:06
johnsomTrevorV Up to you.  I'm trying to highlight important reviews (I.e. things likely to conflict) or will have long dependency chains.19:07
*** doug-fis_ has quit IRC19:07
johnsomEither is fine with me19:07
TrevorVAlright, I'll wait to ask for reviews in the meeting19:07
*** ducttape_ has joined #openstack-lbaas19:09
*** _cjones_ has quit IRC19:11
*** piet has quit IRC19:15
*** jpomeroy has joined #openstack-lbaas19:15
jpomeroyanyone know what it means to have a health monitor delay value of 0?19:16
jpomeroylbaasv2 API19:17
jpomeroyjust sends another probe immediately when the previous one returns?19:21
*** _cjones_ has joined #openstack-lbaas19:24
*** _cjones_ has quit IRC19:24
*** _cjones_ has joined #openstack-lbaas19:24
johnsomjpomeroy Looking, give me a minute19:27
jpomeroysure, thanks19:29
johnsomIt's not called out in the docs.  My guess would be it goes straight into another check with no delay.19:30
johnsomIf you really need an answer we can dig into the haproxy code, but ...19:30
doug-fi__johnsom: the source of the concern is from matt's review comment in https://review.openstack.org/#/c/257574/19:32
jpomeroyjohnsom, just had a comment from someone that this is confusing so we should add something about it to the new ui19:32
*** bdrich has quit IRC19:33
johnsomhttp://cbonte.github.io/haproxy-dconv/configuration-1.5.html#5.2-inter19:34
johnsomThat is the HAProxy configuration those settings get applied.19:35
*** piet has joined #openstack-lbaas19:36
johnsomI thought we defined some validation in the mocks.  Maybe we missed that one19:36
johnsomI commented on the question in the patch19:40
*** neelashah has joined #openstack-lbaas19:40
openstackgerritMerged openstack/octavia: Updated from global requirements  https://review.openstack.org/27280819:41
*** neelashah has quit IRC19:47
*** neelashah has joined #openstack-lbaas19:51
johnsomOctavia meeting starting on #openstack-meeting-alt19:59
*** purandarkakde_ has joined #openstack-lbaas20:01
*** purandarkakde_ is now known as PK20:01
*** PK has quit IRC20:02
*** PK has joined #openstack-lbaas20:04
*** barra204 has quit IRC20:06
*** ducttape_ has quit IRC20:11
a2hillhttps://review.openstack.org/#/c/272905/20:12
*** crc32 has joined #openstack-lbaas20:13
*** neelashah1 has joined #openstack-lbaas20:16
*** neelashah has quit IRC20:16
*** blallau has joined #openstack-lbaas20:22
*** madhu_ak has quit IRC20:23
*** ducttape_ has joined #openstack-lbaas20:23
*** blallau has quit IRC20:24
a2hillAnyone want to try this out? https://review.openstack.org/#/c/272905/20:25
sbalukoffA RHEL amphora?20:25
sbalukoffI'm honestly not sure how to build one... :P20:26
a2hill:)20:26
TrevorVsbalukoff I think I'm more concerned about the reviews than to run that down :)20:26
sbalukoffJust set the AMP_BASEOS environment variable when stacking?20:26
a2hillI dont see the other guy in here that was concerned with it, but really its just the disk image builder at this point20:26
sbalukoffOk, I can have a look at the code and give you my opinion, eh.20:27
a2hillThink we would have to tweak the plugin a bit, but you can run manually ./diskimage-build.sh -i fedora20:27
*** doug-fi__ is now known as doug-fish20:27
sbalukoffThough I'm less inclined to actually build the damned thing as I've got a lot of other stuff on my plate at the moment.20:27
a2hillindeed20:28
rm_worklol evgeny signed off IRC about 2 seconds before I PM'd him20:28
a2hillsomone wanted this, theyll look at it at some point20:28
rm_workah well20:28
sbalukoffIs the guy who wants it a redhat employee?20:28
a2hillprobably20:28
sbalukoffCan we convince them to add their own 3rd party CI?20:28
sbalukoffBecause without tests, this is likely to break again.20:29
a2hilli have no idea, i just know i was going to test something with it, and someone wanted it so I figured it out20:29
sbalukoffAah. Ok.20:29
a2hillBut thinking about it, even is uwsgi restarts normally in fedora i still have to figure it out for ubuntu. bleugh20:31
a2hillmengezi? I think is the one that was testing things, anyone know when he gets on?20:33
*** madhu_ak has joined #openstack-lbaas20:40
*** bdrich has joined #openstack-lbaas20:46
johnsomNot sure, but he is from RedHat20:56
*** PK has quit IRC21:01
a2hillWell that should be working for him, or anyone else that wants to give it a shot. If anything its atleast in a better state in that patch then master.21:01
bharathmNeed reviews on https://review.openstack.org/#/c/272708/  . To fix the openssl - csr issue.21:03
rm_workbharathm: was this same issue already fixed in master I assume?21:08
*** PK has joined #openstack-lbaas21:11
*** rcernin has quit IRC21:16
bharathmrm_work: yup.. I see this is missing only in Liberty21:22
rm_workI don't see anything like this merging into master at any point21:30
rm_workthere's a .sign() operation now, but that wasn't introduced until we switched crypto libraries21:30
rm_workI don't know that adding sign() works the same way with pyOpenSSL21:30
rm_workit might, I need to test21:31
a2hillAlso, im getting push back for the backport of openssl removal.21:31
a2hillNot sure if related21:32
bharathmrmwork: I didnt get a chance to test the Master on latest openssl but looking at the code, the issue would arise in Liberty because of using Crypto library and not signing the csr.. The .sign() operation you mentioned in Master should prevent this error. I guess.21:36
*** piet has quit IRC21:40
fnavalsbalukoff: could you point me to any documentation on shared pools pleae?21:40
fnavalplease21:40
rm_workbharathm: in master we use pyca/cryptography to generate the CSR, which does have a .sign() but in liberty, it is still using pyOpenSSL to generate the CSR21:44
rm_workcompletely different code21:44
bharathmIndeed it's a different library. Thats why I mentioned the fix is needed only in Liberty. :-)21:46
*** piet has joined #openstack-lbaas21:51
*** TrevorV has quit IRC21:55
*** jpomeroy has left #openstack-lbaas22:00
*** kobis has joined #openstack-lbaas22:06
*** rtheis_ has quit IRC22:23
*** piet has quit IRC22:28
openstackgerritMadhusudhan Kandadai proposed openstack/neutron-lbaas: Set config values in lbaas scenario tests and Fix casual failing of nc service for loadbalancer check when using cirros 0.3.3 image  https://review.openstack.org/27326222:30
*** kobis has quit IRC22:35
*** neelashah1 has quit IRC22:42
*** PK has quit IRC22:45
*** yuanying_ has quit IRC23:02
*** doug-fish has quit IRC23:03
*** harlowja has quit IRC23:03
*** harlowja has joined #openstack-lbaas23:04
*** davidlenwell has quit IRC23:06
*** davidlenwell has joined #openstack-lbaas23:08
*** doug-fish has joined #openstack-lbaas23:10
*** yuanying has joined #openstack-lbaas23:12
dougwigI'm beginning to think amphora should be built periodic and consumed as images.  Like node pool.23:16
johnsomHa.  Right as I have that darn build/no-build thing working....23:16
*** armax has joined #openstack-lbaas23:17
openstackgerritMadhusudhan Kandadai proposed openstack/neutron-lbaas: Set config values in lbaas scenario tests and Fix casual failing of nc service for loadbalancer check when using cirros 0.3.3 image  https://review.openstack.org/27326223:17
*** doug-fish has quit IRC23:18
*** chlong has quit IRC23:18
johnsomdougwig In theory amphora agent could be it's own project and build images on merge23:18
*** doug-fish has joined #openstack-lbaas23:21
*** bdrich has quit IRC23:22
*** doug-fish has quit IRC23:24
*** doug-fish has joined #openstack-lbaas23:25
*** doug-fish has quit IRC23:25
*** doug-fish has joined #openstack-lbaas23:25
sbalukofffnaval: Sorry! Was AFK for a while there.23:27
*** doug-fish has quit IRC23:27
sbalukofffnaval: The best place to find the shared pools documentation is here, for now: https://review.openstack.org/#/c/256369/19/doc/source/api/octaviaapi.rst23:27
sbalukoff(I recommend avoiding the deprecated URLs. :) )23:27
*** doug-fish has joined #openstack-lbaas23:27
ajmillerdoug-fish +A on https://review.openstack.org/#/c/257574/1023:28
*** barclaac has joined #openstack-lbaas23:30
fnavalkk thanks sbalukoff23:30
sbalukoffNo problemo!23:30
*** chlong has joined #openstack-lbaas23:31
*** doug-fish has quit IRC23:32
*** _cjones_ has quit IRC23:42
*** ducttape_ has quit IRC23:43
*** _cjones_ has joined #openstack-lbaas23:43
*** _cjones__ has joined #openstack-lbaas23:45
*** frzn_ has joined #openstack-lbaas23:46
*** _cjones__ has quit IRC23:46
*** _cjones_ has quit IRC23:46
*** _cjones__ has joined #openstack-lbaas23:46
*** PK has joined #openstack-lbaas23:53

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