sbalukoff | Oh gerrit, why you no worky? | 00:02 |
---|---|---|
sbalukoff | Is it just me or did gerrit fall over for other people as well? | 00:03 |
sbalukoff | Ok... it's back. Hmmm. | 00:06 |
johnsom | It's been doing that to me for the last two weeks. Dies and then comes back | 00:07 |
*** Aish has quit IRC | 00:09 | |
*** Aish has joined #openstack-lbaas | 00:11 | |
*** _cjones_ has quit IRC | 00:12 | |
*** _cjones_ has joined #openstack-lbaas | 00:12 | |
*** ducttape_ has quit IRC | 00:14 | |
openstackgerrit | Stephen Balukoff proposed openstack/octavia: Shared pools support https://review.openstack.org/256369 | 00:17 |
sbalukoff | TrevorV: 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 IRC | 00:39 | |
*** dnovosel1_ has quit IRC | 00:39 | |
openstackgerrit | Doug Wiegley proposed openstack/neutron-lbaas: WIP - DO NOT MERGE - ci test, 1 https://review.openstack.org/264898 | 00:58 |
openstackgerrit | Doug Wiegley proposed openstack/neutron-lbaas: WIP - DO NOT MERGE - ci test, 1 https://review.openstack.org/264898 | 00:58 |
openstackgerrit | Doug Wiegley proposed openstack/neutron-lbaas: Migrate some stuff to neutron-lib https://review.openstack.org/253263 | 01:00 |
openstackgerrit | Merged openstack/octavia: Change the length of name in the algorithm https://review.openstack.org/271178 | 01:01 |
*** dnovosel__ has joined #openstack-lbaas | 01:03 | |
openstackgerrit | Merged openstack/octavia: Add APP_COOKIE session persistence type https://review.openstack.org/272345 | 01:07 |
johnsom | CRITICAL octavia [-] error: [Errno 98] Address already in use | 01:09 |
johnsom | Well, that is a new gate failure.... | 01:09 |
johnsom | INFO octavia.cmd.api [-] Starting API server on 0.0.0.0:9876 | 01:09 |
xgerman | fun, fun | 01:09 |
johnsom | un-related change too | 01:09 |
xgerman | wondering if sbalukoff asserting IBMs copyright on each file is kosher… never added HP copyright to a file which already had copyright | 01:10 |
*** mgarza has quit IRC | 01:10 | |
johnsom | If we make changes, we should be adding them | 01:10 |
sbalukoff | xgerman: TrevorV recommended I do that. :/ | 01:10 |
xgerman | let’s ask the great dougwig if there is some policy | 01:11 |
dougwig | sigh. | 01:11 |
johnsom | Though I have to say.... Adding a copyright for changing the constants names, oye | 01:11 |
* sbalukoff rolls his eyes. | 01:12 | |
sbalukoff | I honestly don't care about that-- it's all apache 2 anyway. | 01:12 |
dougwig | if 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 |
sbalukoff | I just do whatever policy says I should do in that case. :P | 01:12 |
johnsom | https://review.openstack.org/#/c/256369/18/octavia/controller/worker/flows/amphora_flows.py | 01:12 |
dougwig | https://wiki.openstack.org/wiki/LegalIssuesFAQ#Copyright_Headers | 01:12 |
xgerman | yeah, I have never seen two copyrights in a single file | 01:12 |
sbalukoff | xgerman: Really? I've definitely seen that before. | 01:13 |
dougwig | i just don't put in the line anymore. makes life easier. :) | 01:13 |
xgerman | I only do if the file is new | 01:13 |
sbalukoff | I 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 |
sbalukoff | Are y'all going to go back and say I have to remove them all now? | 01:14 |
xgerman | ok, I guess we are getting into yak shaving | 01:14 |
sbalukoff | Yep. | 01:14 |
xgerman | so if you want to keep them we are living in 2016 not 2015 | 01:15 |
johnsom | Yeah, I try not to pick on other company's copyright stuff too much. Just if it's a new file | 01:15 |
xgerman | and for 2015 ones you might want to write 2015-2016 | 01:15 |
sbalukoff | Octavia consists of a lot of files that have been updated many times over the last few years. | 01:15 |
johnsom | Yep | 01:16 |
xgerman | yep, but if you add a new (c) line please do it right ;-) | 01:16 |
xgerman | and use the right year | 01:16 |
sbalukoff | If 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 |
dougwig | sbalukoff: read the faq. if you're in compliance, push back. | 01:16 |
sbalukoff | I tihnk that's entirely orthogonal to the actual stuff the shared pools stuff does. | 01:16 |
xgerman | I am saying if you add a new (c) line it should have the correct date | 01:16 |
dougwig | no no no, do not go retroactively edit. | 01:17 |
xgerman | yeah, only where you added a new (c) | 01:17 |
dougwig | the maintenance of said headers is the main reason they recommend against them now. | 01:17 |
xgerman | e.g. https://review.openstack.org/#/c/256369/18/octavia/api/v1/controllers/member.py | 01:17 |
johnsom | +1 on not retroactively edit | 01:17 |
xgerman | dougwig, johnsom +1 | 01:18 |
sbalukoff | +1 on this is a really annoying conversation to have... seriously? copyright notices? | 01:18 |
* sbalukoff sighs | 01:18 | |
xgerman | that’s why I don;’t touch them ;-) | 01:18 |
xgerman | but putting up the wrong year, I got -1’d for less | 01:18 |
sbalukoff | Well, more than likely the file you're looking at was actually written in 2015. | 01:19 |
sbalukoff | This patch has been open for a while. | 01:19 |
dougwig | oh snap. | 01:19 |
johnsom | sbalukoff 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 |
xgerman | well, HPE’s legal team recommends 2015-2016 in that case | 01:20 |
sbalukoff | johnsom: There's a function rename, but that would be easy to duplicate. | 01:20 |
xgerman | but yeah, there is a reason I don’t touch (c) | 01:20 |
*** minwang2 has quit IRC | 01:20 | |
johnsom | Ok, just a thought.... | 01:20 |
sbalukoff | So... | 01:20 |
sbalukoff | I 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 | <3 | 01:22 |
openstackgerrit | Aishwarya Thangappa proposed openstack/octavia: Adding "region and endpoint_type" parameters to barbican_acl.py https://review.openstack.org/272861 | 01:22 |
johnsom | dougwig My vote is to revert and be done with it, but I must defer this one to my lieutenant | 01:23 |
*** yamamoto_ has joined #openstack-lbaas | 01:23 | |
xgerman | I don;t care but recommend to use current dates in the added (c) | 01:23 |
sbalukoff | I need an answer, y'all. | 01:25 |
sbalukoff | And you have to mean it! | 01:25 |
sbalukoff | Revert 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 |
johnsom | I would only comment if it was a new file with no copyright. | 01:28 |
sbalukoff | Heh... the FAQ on this: "We do not yet have guidance for when to add or remove a copyright header in source files." | 01:29 |
sbalukoff | Sweet. Clear as mud. | 01:29 |
johnsom | I don't see any files that are new in that patch that you don't have covered, so fine by me | 01:30 |
sbalukoff | johnsom: 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 |
sbalukoff | Mr. PTL? | 01:32 |
johnsom | Yes sir. As long as that meets your needs for copyright compliance. | 01:32 |
*** Aish has quit IRC | 01:32 | |
sbalukoff | Until 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 |
sbalukoff | I have no problem claiming ignorance on this. | 01:33 |
dougwig | i 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 |
sbalukoff | dougwig: 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 |
dougwig | sbalukoff: 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 |
sbalukoff | dougwig: Then... right. I was effectively following that guideline in the first place. | 01:37 |
sbalukoff | Bah! Whatever... I'm almost done with this revert anyway. | 01:38 |
dougwig | sbalukoff: so.. enough bits killed. carry on. :) | 01:38 |
sbalukoff | :) | 01:38 |
johnsom | sbalukoff I think you were following the guideline in the first place. | 01:38 |
sbalukoff | Yep, I was. | 01:39 |
*** jaff_cheng has joined #openstack-lbaas | 01:40 | |
*** paco20151113 has joined #openstack-lbaas | 01:43 | |
*** diogogmt has quit IRC | 01:48 | |
*** _cjones_ has quit IRC | 01:50 | |
*** _cjones_ has joined #openstack-lbaas | 01:50 | |
*** _cjones_ has quit IRC | 01:50 | |
*** _cjones_ has joined #openstack-lbaas | 01:51 | |
*** _cjones_ has quit IRC | 01:55 | |
*** bochi-michael has joined #openstack-lbaas | 01:59 | |
openstackgerrit | Stephen Balukoff proposed openstack/octavia: Shared pools support https://review.openstack.org/256369 | 02:00 |
*** bana_k has quit IRC | 02:17 | |
openstackgerrit | Stephen Balukoff proposed openstack/octavia: Add L7 database structures https://review.openstack.org/265430 | 02:40 |
*** ducttape_ has joined #openstack-lbaas | 02:49 | |
openstackgerrit | Stephen Balukoff proposed openstack/octavia: Update repos for L7 objects / methods https://review.openstack.org/265529 | 02:58 |
*** diogogmt has joined #openstack-lbaas | 02:58 | |
*** fnaval has quit IRC | 02:58 | |
*** ducttape_ has quit IRC | 02:59 | |
*** ducttape_ has joined #openstack-lbaas | 03:00 | |
*** bana_k has joined #openstack-lbaas | 03:07 | |
*** fnaval has joined #openstack-lbaas | 03:13 | |
openstackgerrit | Stephen Balukoff proposed openstack/octavia: WIP: Add L7 api https://review.openstack.org/265690 | 03:15 |
*** ducttape_ has quit IRC | 03:17 | |
*** yuanying has quit IRC | 03:21 | |
*** Aish has joined #openstack-lbaas | 03:23 | |
*** Aish has quit IRC | 03:23 | |
*** yuanying has joined #openstack-lbaas | 03:23 | |
openstackgerrit | Stephen Balukoff proposed openstack/octavia: WIP: Tests for L7 API https://review.openstack.org/265718 | 03:26 |
*** yuanying has quit IRC | 03:28 | |
*** yuanying has joined #openstack-lbaas | 03:33 | |
*** yuanying has quit IRC | 03:40 | |
*** yuanying has joined #openstack-lbaas | 03:40 | |
*** doug-fish has joined #openstack-lbaas | 03:49 | |
*** yuanying has quit IRC | 03:56 | |
*** yuanying has joined #openstack-lbaas | 03:57 | |
*** yuanying_ has joined #openstack-lbaas | 03:58 | |
*** yuanying has quit IRC | 04:01 | |
*** ducttape_ has joined #openstack-lbaas | 04:11 | |
*** madhu_ak has quit IRC | 04:42 | |
*** ducttape_ has quit IRC | 04:56 | |
*** fnaval has quit IRC | 05:06 | |
openstackgerrit | Phillip Toohill proposed openstack/octavia: Fixes Octavia Amphora RHEL builds https://review.openstack.org/272905 | 05:21 |
*** hang has joined #openstack-lbaas | 05:29 | |
*** jaff_cheng has quit IRC | 05:33 | |
*** hang has quit IRC | 05:41 | |
openstackgerrit | Stephen Balukoff proposed openstack/octavia: Fix minor typo in warning message https://review.openstack.org/272910 | 05:55 |
rm_work | xgerman: are you editing the wiki now or did i just get that email from earlier | 06:08 |
rm_work | i 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 page | 06:10 |
*** minwang2 has joined #openstack-lbaas | 06:10 | |
xgerman | Yep, fixed a typo | 06:19 |
xgerman | The sni scenario wasn't right | 06:20 |
*** minwang2 has quit IRC | 06:32 | |
*** minwang2 has joined #openstack-lbaas | 06:38 | |
openstackgerrit | Trevor Vardeman proposed openstack/octavia: Amphora Create now fails when amphora goes into ERROR status https://review.openstack.org/267137 | 06:40 |
*** numans has joined #openstack-lbaas | 06:43 | |
*** minwang2 has quit IRC | 06:48 | |
*** nmagnezi_ has joined #openstack-lbaas | 07:21 | |
*** rcernin has joined #openstack-lbaas | 07:27 | |
openstackgerrit | LIU Yulong proposed openstack/neutron-lbaas: Restrict delete pool if it has binding healthmonitor https://review.openstack.org/179684 | 07:28 |
*** chlong_zzz is now known as chlong | 07:31 | |
*** jaff_cheng has joined #openstack-lbaas | 08:27 | |
*** lijie has joined #openstack-lbaas | 08:30 | |
*** lijie has left #openstack-lbaas | 08:30 | |
*** lijie has joined #openstack-lbaas | 08:31 | |
*** lijie has left #openstack-lbaas | 08:33 | |
openstackgerrit | Stephen Balukoff proposed openstack/octavia: Update repos for L7 objects / methods https://review.openstack.org/265529 | 08:36 |
*** bana_k has quit IRC | 08:38 | |
openstackgerrit | Stephen Balukoff proposed openstack/octavia: WIP: Add L7 api https://review.openstack.org/265690 | 08:44 |
*** armax has quit IRC | 08:45 | |
openstackgerrit | Stephen Balukoff proposed openstack/octavia: WIP: Tests for L7 API https://review.openstack.org/265718 | 08:47 |
*** nmagnezi_ has quit IRC | 08:58 | |
*** bdrich has joined #openstack-lbaas | 09:02 | |
*** armax has joined #openstack-lbaas | 09:04 | |
*** nmagnezi_ has joined #openstack-lbaas | 09:11 | |
*** woodster_ has quit IRC | 09:16 | |
*** Alex_Stef has joined #openstack-lbaas | 09:17 | |
*** armax_ has joined #openstack-lbaas | 09:19 | |
*** armax has quit IRC | 09:21 | |
*** armax_ is now known as armax | 09:21 | |
*** bdrich has quit IRC | 09:22 | |
*** amotoki has joined #openstack-lbaas | 09:41 | |
*** nmagnezi_ has quit IRC | 10:18 | |
*** paco20151113 has quit IRC | 10:18 | |
*** jaff_cheng has quit IRC | 10:21 | |
*** bochi-michael has quit IRC | 10:28 | |
*** armax has quit IRC | 10:30 | |
*** nmagnezi_ has joined #openstack-lbaas | 10:31 | |
*** prabampm1 has quit IRC | 10:50 | |
*** pcaruana has joined #openstack-lbaas | 11:44 | |
*** prabampm has joined #openstack-lbaas | 11:59 | |
*** ajmiller has quit IRC | 12:07 | |
*** ajmiller has joined #openstack-lbaas | 12:07 | |
*** rtheis has joined #openstack-lbaas | 12:13 | |
*** pcaruana has quit IRC | 12:37 | |
*** yamamoto_ has quit IRC | 12:49 | |
*** chlong has quit IRC | 13:15 | |
*** chlong has joined #openstack-lbaas | 13:28 | |
*** numans has quit IRC | 13:55 | |
*** numans has joined #openstack-lbaas | 13:56 | |
*** prabampm has quit IRC | 14:00 | |
*** diogogmt has quit IRC | 14:01 | |
*** diogogmt has joined #openstack-lbaas | 14:03 | |
*** yamamoto has joined #openstack-lbaas | 14:10 | |
*** yamamoto_ has joined #openstack-lbaas | 14:11 | |
*** neelashah has joined #openstack-lbaas | 14:12 | |
*** yamamoto has quit IRC | 14:15 | |
*** ajmiller_ has joined #openstack-lbaas | 14:18 | |
*** diogogmt has quit IRC | 14:21 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/octavia: Updated from global requirements https://review.openstack.org/272808 | 14:23 |
*** ajmiller_ has quit IRC | 14:34 | |
*** yamamoto_ has quit IRC | 14:41 | |
*** yamamoto has joined #openstack-lbaas | 14:48 | |
*** nmagnezi_ is now known as nmagnezi | 14:52 | |
*** ducttape_ has joined #openstack-lbaas | 14:56 | |
*** yamamoto has quit IRC | 14:57 | |
*** yamamoto has joined #openstack-lbaas | 15:08 | |
*** yamamoto has quit IRC | 15:19 | |
*** yamamoto has joined #openstack-lbaas | 15:39 | |
*** TrevorV has joined #openstack-lbaas | 15:46 | |
*** Alex_Stef has quit IRC | 15:57 | |
*** nmagnezi has quit IRC | 15:59 | |
*** Aish has joined #openstack-lbaas | 16:00 | |
*** mgarza_ has joined #openstack-lbaas | 16:03 | |
*** numans has quit IRC | 16:04 | |
*** rcernin has quit IRC | 16:09 | |
ajmiller | doug-fish ping | 16:11 |
doug-fish | ajmiller: hi | 16:11 |
ajmiller | Good Morning. I've been through the loadbalancer/listener/pool/member/monitor patches. They all work as expected. | 16:12 |
doug-fish | excellent! | 16:13 |
ajmiller | I gave +2's, on the member/monitor patches, and also the npm dep patch. | 16:14 |
doug-fish | I'll wrap those up - I've been keeping up with them pretty well anyway. Hopefully we can get them merged today! | 16:14 |
ajmiller | Now 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 |
ajmiller | Is there an implied order, or have they just gotten out of sync? | 16:15 |
doug-fish | pretty sure they are actually out of sync | 16:15 |
ajmiller | OK. We | 16:15 |
ajmiller | Would be good to get someone to look at them and get them usable. | 16:16 |
doug-fish | I'll harass Lucas and Justin to get those back up to date | 16:16 |
ajmiller | Excellent! | 16:16 |
doug-fish | likewise - great to see these moving! and glad to hear they do what they ought! | 16:16 |
ajmiller | I did leave a comment about some rendering issues, and I will try to get an actual example/screenshot of the problem/ | 16:18 |
ajmiller | Its something that needs to get fixed before release, but I feel OK merging them as is now. | 16:18 |
doug-fish | ok, I'll take a look as I re-rereview | 16:18 |
*** minwang2 has joined #openstack-lbaas | 16:18 | |
doug-fish | yeah, that's my attitude now as well - not quite polished code is okay to get merged until we have the basic set going | 16:18 |
ajmiller | It feels like some HTML or CSS is making bad layout assumptiosn somewhere. | 16:19 |
*** bdrich has joined #openstack-lbaas | 16:19 | |
*** woodster_ has joined #openstack-lbaas | 16:21 | |
*** ducttape_ has quit IRC | 16:23 | |
*** rtheis_ has joined #openstack-lbaas | 16:23 | |
*** minwang2 has quit IRC | 16:24 | |
*** rtheis has quit IRC | 16:24 | |
*** ducttape_ has joined #openstack-lbaas | 16:30 | |
*** bana_k has joined #openstack-lbaas | 16:45 | |
*** piet has joined #openstack-lbaas | 16:52 | |
piet | Morning folks | 16:52 |
piet | One of the UI devs that was implementing the LBaaS panels in Horizon had a question for the group | 16:52 |
piet | Is anyone available to answer a few questions? | 16:53 |
johnsom | piet I can try | 16:54 |
xgerman | sure | 16:54 |
ajmiller | me too | 16:54 |
xgerman | piet how’s life at the big I? | 16:54 |
*** manjeets1 has joined #openstack-lbaas | 16:55 | |
piet | He | 16:55 |
*** manjeets1 has left #openstack-lbaas | 16:56 | |
piet | Good! But the honeymoon is over | 16:56 |
piet | https://openstack.invisionapp.com/share/V4507TP2S#/screens/115558025 | 16:56 |
*** matt-borland has joined #openstack-lbaas | 16:56 | |
*** jpomeroy has joined #openstack-lbaas | 16:56 | |
*** diogogmt has joined #openstack-lbaas | 16:56 | |
piet | Can you take a look at this link? We have a question around what options are displayed when HTTPS is selected as a monitor | 16:57 |
johnsom | Yes | 16:57 |
johnsom | I'm on the health monitor panel | 16:57 |
piet | Is the yellow note accurate? | 16:58 |
johnsom | HTTPS 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 functional | 16:58 |
*** _cjones_ has joined #openstack-lbaas | 16:58 | |
*** _cjones_ has quit IRC | 16:58 | |
johnsom | Yes, I think it is correct | 16:58 |
matt-borland | gotcha, so does the back-end just to some standard GET to / or such? | 16:58 |
*** _cjones_ has joined #openstack-lbaas | 16:58 | |
xgerman | yep | 16:59 |
johnsom | When HTTPS monitor is selected it currently uses the same mechanism as TCP | 16:59 |
matt-borland | ah, does a TCP handshake? | 16:59 |
johnsom | So, no GET or / | 16:59 |
johnsom | Yes. | 16:59 |
matt-borland | cool, thanks :) | 16:59 |
xgerman | yeah, we only do get on HTTP | 16:59 |
johnsom | To monitor with HTTPS using HTTPS we would have to deal with the certificate issues on backend connections, which we have not yet addressed | 17:00 |
matt-borland | makes sense, thanks! | 17:00 |
*** matt-borland has left #openstack-lbaas | 17:00 | |
*** bdrich has quit IRC | 17:01 | |
*** jpomeroy has left #openstack-lbaas | 17:02 | |
johnsom | Any other questions we can answer? | 17:02 |
johnsom | Thanks for moving forward with this stuff. We really appreciate it. | 17:03 |
*** yamamoto has quit IRC | 17:07 | |
*** numans has joined #openstack-lbaas | 17:07 | |
piet | Thanks for your help johnsom | 17:08 |
*** neelashah has quit IRC | 17:17 | |
*** minwang2 has joined #openstack-lbaas | 17:19 | |
*** minwang2 has quit IRC | 17:23 | |
openstackgerrit | Michael Johnson proposed openstack/neutron-lbaas: Don't always run Octavia amphroa image build https://review.openstack.org/267798 | 17:24 |
*** minwang2 has joined #openstack-lbaas | 17:25 | |
*** minwang2 has quit IRC | 17:28 | |
*** neelashah has joined #openstack-lbaas | 17:31 | |
*** bdrich has joined #openstack-lbaas | 17:35 | |
johnsom | Agenda for today's Octavia IRC meeting: https://wiki.openstack.org/wiki/Octavia/Weekly_Meeting_Agenda#Meeting_2016-01-27 | 17:41 |
*** armax has joined #openstack-lbaas | 17:42 | |
*** minwang2 has joined #openstack-lbaas | 17:42 | |
*** minwang2 has quit IRC | 17:44 | |
*** minwang2 has joined #openstack-lbaas | 17:44 | |
*** neelashah has quit IRC | 17:45 | |
*** minwang2 has quit IRC | 17:46 | |
*** minwang2 has joined #openstack-lbaas | 17:58 | |
*** Aish has quit IRC | 17:58 | |
*** ducttape_ has quit IRC | 17:59 | |
*** rcernin has joined #openstack-lbaas | 18:00 | |
*** neelashah has joined #openstack-lbaas | 18:01 | |
*** ajmiller has quit IRC | 18:02 | |
*** armax has quit IRC | 18:02 | |
*** ajmiller has joined #openstack-lbaas | 18:02 | |
*** ducttape_ has joined #openstack-lbaas | 18:04 | |
*** Aish has joined #openstack-lbaas | 18:10 | |
*** bana_k has quit IRC | 18:11 | |
*** doug-fish has quit IRC | 18:16 | |
*** madhu_ak has joined #openstack-lbaas | 18:18 | |
*** fnaval has joined #openstack-lbaas | 18:18 | |
*** doug-fish has joined #openstack-lbaas | 18:22 | |
*** bana_k has joined #openstack-lbaas | 18:23 | |
*** neelashah has quit IRC | 18:26 | |
*** doug-fish has quit IRC | 18:27 | |
*** doug-fish has joined #openstack-lbaas | 18:35 | |
*** doug-fish has quit IRC | 18:40 | |
*** ducttape_ has quit IRC | 18:53 | |
*** doug-fish has joined #openstack-lbaas | 18:55 | |
*** doug-fish has quit IRC | 19:00 | |
*** doug-fish has joined #openstack-lbaas | 19:01 | |
*** doug-fis_ has joined #openstack-lbaas | 19:02 | |
*** doug-fi__ has joined #openstack-lbaas | 19:03 | |
TrevorV | johnsom 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 IRC | 19:06 | |
*** numans has quit IRC | 19:06 | |
johnsom | TrevorV 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 IRC | 19:07 | |
johnsom | Either is fine with me | 19:07 |
TrevorV | Alright, I'll wait to ask for reviews in the meeting | 19:07 |
*** ducttape_ has joined #openstack-lbaas | 19:09 | |
*** _cjones_ has quit IRC | 19:11 | |
*** piet has quit IRC | 19:15 | |
*** jpomeroy has joined #openstack-lbaas | 19:15 | |
jpomeroy | anyone know what it means to have a health monitor delay value of 0? | 19:16 |
jpomeroy | lbaasv2 API | 19:17 |
jpomeroy | just sends another probe immediately when the previous one returns? | 19:21 |
*** _cjones_ has joined #openstack-lbaas | 19:24 | |
*** _cjones_ has quit IRC | 19:24 | |
*** _cjones_ has joined #openstack-lbaas | 19:24 | |
johnsom | jpomeroy Looking, give me a minute | 19:27 |
jpomeroy | sure, thanks | 19:29 |
johnsom | It's not called out in the docs. My guess would be it goes straight into another check with no delay. | 19:30 |
johnsom | If 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 |
jpomeroy | johnsom, just had a comment from someone that this is confusing so we should add something about it to the new ui | 19:32 |
*** bdrich has quit IRC | 19:33 | |
johnsom | http://cbonte.github.io/haproxy-dconv/configuration-1.5.html#5.2-inter | 19:34 |
johnsom | That is the HAProxy configuration those settings get applied. | 19:35 |
*** piet has joined #openstack-lbaas | 19:36 | |
johnsom | I thought we defined some validation in the mocks. Maybe we missed that one | 19:36 |
johnsom | I commented on the question in the patch | 19:40 |
*** neelashah has joined #openstack-lbaas | 19:40 | |
openstackgerrit | Merged openstack/octavia: Updated from global requirements https://review.openstack.org/272808 | 19:41 |
*** neelashah has quit IRC | 19:47 | |
*** neelashah has joined #openstack-lbaas | 19:51 | |
johnsom | Octavia meeting starting on #openstack-meeting-alt | 19:59 |
*** purandarkakde_ has joined #openstack-lbaas | 20:01 | |
*** purandarkakde_ is now known as PK | 20:01 | |
*** PK has quit IRC | 20:02 | |
*** PK has joined #openstack-lbaas | 20:04 | |
*** barra204 has quit IRC | 20:06 | |
*** ducttape_ has quit IRC | 20:11 | |
a2hill | https://review.openstack.org/#/c/272905/ | 20:12 |
*** crc32 has joined #openstack-lbaas | 20:13 | |
*** neelashah1 has joined #openstack-lbaas | 20:16 | |
*** neelashah has quit IRC | 20:16 | |
*** blallau has joined #openstack-lbaas | 20:22 | |
*** madhu_ak has quit IRC | 20:23 | |
*** ducttape_ has joined #openstack-lbaas | 20:23 | |
*** blallau has quit IRC | 20:24 | |
a2hill | Anyone want to try this out? https://review.openstack.org/#/c/272905/ | 20:25 |
sbalukoff | A RHEL amphora? | 20:25 |
sbalukoff | I'm honestly not sure how to build one... :P | 20:26 |
a2hill | :) | 20:26 |
TrevorV | sbalukoff I think I'm more concerned about the reviews than to run that down :) | 20:26 |
sbalukoff | Just set the AMP_BASEOS environment variable when stacking? | 20:26 |
a2hill | I dont see the other guy in here that was concerned with it, but really its just the disk image builder at this point | 20:26 |
sbalukoff | Ok, I can have a look at the code and give you my opinion, eh. | 20:27 |
a2hill | Think we would have to tweak the plugin a bit, but you can run manually ./diskimage-build.sh -i fedora | 20:27 |
*** doug-fi__ is now known as doug-fish | 20:27 | |
sbalukoff | Though 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 |
a2hill | indeed | 20:28 |
rm_work | lol evgeny signed off IRC about 2 seconds before I PM'd him | 20:28 |
a2hill | somone wanted this, theyll look at it at some point | 20:28 |
rm_work | ah well | 20:28 |
sbalukoff | Is the guy who wants it a redhat employee? | 20:28 |
a2hill | probably | 20:28 |
sbalukoff | Can we convince them to add their own 3rd party CI? | 20:28 |
sbalukoff | Because without tests, this is likely to break again. | 20:29 |
a2hill | i have no idea, i just know i was going to test something with it, and someone wanted it so I figured it out | 20:29 |
sbalukoff | Aah. Ok. | 20:29 |
a2hill | But thinking about it, even is uwsgi restarts normally in fedora i still have to figure it out for ubuntu. bleugh | 20:31 |
a2hill | mengezi? I think is the one that was testing things, anyone know when he gets on? | 20:33 |
*** madhu_ak has joined #openstack-lbaas | 20:40 | |
*** bdrich has joined #openstack-lbaas | 20:46 | |
johnsom | Not sure, but he is from RedHat | 20:56 |
*** PK has quit IRC | 21:01 | |
a2hill | Well 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 |
bharathm | Need reviews on https://review.openstack.org/#/c/272708/ . To fix the openssl - csr issue. | 21:03 |
rm_work | bharathm: was this same issue already fixed in master I assume? | 21:08 |
*** PK has joined #openstack-lbaas | 21:11 | |
*** rcernin has quit IRC | 21:16 | |
bharathm | rm_work: yup.. I see this is missing only in Liberty | 21:22 |
rm_work | I don't see anything like this merging into master at any point | 21:30 |
rm_work | there's a .sign() operation now, but that wasn't introduced until we switched crypto libraries | 21:30 |
rm_work | I don't know that adding sign() works the same way with pyOpenSSL | 21:30 |
rm_work | it might, I need to test | 21:31 |
a2hill | Also, im getting push back for the backport of openssl removal. | 21:31 |
a2hill | Not sure if related | 21:32 |
bharathm | rmwork: 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 IRC | 21:40 | |
fnaval | sbalukoff: could you point me to any documentation on shared pools pleae? | 21:40 |
fnaval | please | 21:40 |
rm_work | bharathm: 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 CSR | 21:44 |
rm_work | completely different code | 21:44 |
bharathm | Indeed it's a different library. Thats why I mentioned the fix is needed only in Liberty. :-) | 21:46 |
*** piet has joined #openstack-lbaas | 21:51 | |
*** TrevorV has quit IRC | 21:55 | |
*** jpomeroy has left #openstack-lbaas | 22:00 | |
*** kobis has joined #openstack-lbaas | 22:06 | |
*** rtheis_ has quit IRC | 22:23 | |
*** piet has quit IRC | 22:28 | |
openstackgerrit | Madhusudhan 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/273262 | 22:30 |
*** kobis has quit IRC | 22:35 | |
*** neelashah1 has quit IRC | 22:42 | |
*** PK has quit IRC | 22:45 | |
*** yuanying_ has quit IRC | 23:02 | |
*** doug-fish has quit IRC | 23:03 | |
*** harlowja has quit IRC | 23:03 | |
*** harlowja has joined #openstack-lbaas | 23:04 | |
*** davidlenwell has quit IRC | 23:06 | |
*** davidlenwell has joined #openstack-lbaas | 23:08 | |
*** doug-fish has joined #openstack-lbaas | 23:10 | |
*** yuanying has joined #openstack-lbaas | 23:12 | |
dougwig | I'm beginning to think amphora should be built periodic and consumed as images. Like node pool. | 23:16 |
johnsom | Ha. Right as I have that darn build/no-build thing working.... | 23:16 |
*** armax has joined #openstack-lbaas | 23:17 | |
openstackgerrit | Madhusudhan 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/273262 | 23:17 |
*** doug-fish has quit IRC | 23:18 | |
*** chlong has quit IRC | 23:18 | |
johnsom | dougwig In theory amphora agent could be it's own project and build images on merge | 23:18 |
*** doug-fish has joined #openstack-lbaas | 23:21 | |
*** bdrich has quit IRC | 23:22 | |
*** doug-fish has quit IRC | 23:24 | |
*** doug-fish has joined #openstack-lbaas | 23:25 | |
*** doug-fish has quit IRC | 23:25 | |
*** doug-fish has joined #openstack-lbaas | 23:25 | |
sbalukoff | fnaval: Sorry! Was AFK for a while there. | 23:27 |
*** doug-fish has quit IRC | 23:27 | |
sbalukoff | fnaval: The best place to find the shared pools documentation is here, for now: https://review.openstack.org/#/c/256369/19/doc/source/api/octaviaapi.rst | 23:27 |
sbalukoff | (I recommend avoiding the deprecated URLs. :) ) | 23:27 |
*** doug-fish has joined #openstack-lbaas | 23:27 | |
ajmiller | doug-fish +A on https://review.openstack.org/#/c/257574/10 | 23:28 |
*** barclaac has joined #openstack-lbaas | 23:30 | |
fnaval | kk thanks sbalukoff | 23:30 |
sbalukoff | No problemo! | 23:30 |
*** chlong has joined #openstack-lbaas | 23:31 | |
*** doug-fish has quit IRC | 23:32 | |
*** _cjones_ has quit IRC | 23:42 | |
*** ducttape_ has quit IRC | 23:43 | |
*** _cjones_ has joined #openstack-lbaas | 23:43 | |
*** _cjones__ has joined #openstack-lbaas | 23:45 | |
*** frzn_ has joined #openstack-lbaas | 23:46 | |
*** _cjones__ has quit IRC | 23:46 | |
*** _cjones_ has quit IRC | 23:46 | |
*** _cjones__ has joined #openstack-lbaas | 23:46 | |
*** PK has joined #openstack-lbaas | 23:53 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!