*** ducttape_ has joined #openstack-lbaas | 00:25 | |
*** diogogmt has joined #openstack-lbaas | 00:37 | |
*** gongysh has joined #openstack-lbaas | 01:12 | |
*** bana_k has quit IRC | 01:21 | |
*** kevo has quit IRC | 01:25 | |
*** amotoki has joined #openstack-lbaas | 01:29 | |
*** amotoki has quit IRC | 01:33 | |
*** amotoki has joined #openstack-lbaas | 01:46 | |
*** ducttape_ has quit IRC | 01:57 | |
*** ducttape_ has joined #openstack-lbaas | 01:59 | |
*** amotoki has quit IRC | 02:05 | |
*** amotoki has joined #openstack-lbaas | 02:20 | |
*** amotoki has quit IRC | 02:28 | |
*** amotoki has joined #openstack-lbaas | 02:41 | |
*** amotoki has quit IRC | 02:48 | |
*** bana_k has joined #openstack-lbaas | 02:52 | |
*** Appa has joined #openstack-lbaas | 02:57 | |
*** ducttape_ has quit IRC | 03:05 | |
*** bana_k has quit IRC | 03:14 | |
*** ducttape_ has joined #openstack-lbaas | 03:26 | |
*** ducttape_ has quit IRC | 03:39 | |
*** Alex_Stef has joined #openstack-lbaas | 03:45 | |
*** links has joined #openstack-lbaas | 03:45 | |
openstackgerrit | Michael Johnson proposed openstack/neutron-lbaas: Test to enable kvm for neutron-lbaas/Octavia gates https://review.openstack.org/366472 | 03:55 |
---|---|---|
*** Alex_Stef has quit IRC | 04:01 | |
*** amotoki has joined #openstack-lbaas | 04:20 | |
*** Alex_Stef has joined #openstack-lbaas | 04:31 | |
*** 17WAAKU0S has joined #openstack-lbaas | 04:37 | |
*** bana_k has joined #openstack-lbaas | 04:38 | |
*** 17WAAKU0S has quit IRC | 04:38 | |
Appa | Any ref. document for haproxy configuration with lbaas in openstack | 04:38 |
*** eezhova has joined #openstack-lbaas | 04:56 | |
*** gcheresh has joined #openstack-lbaas | 05:02 | |
*** Appa has quit IRC | 05:09 | |
*** openstackgerrit has quit IRC | 05:18 | |
*** openstackgerrit has joined #openstack-lbaas | 05:19 | |
*** Appa has joined #openstack-lbaas | 05:22 | |
*** ducttape_ has joined #openstack-lbaas | 05:39 | |
*** ducttape_ has quit IRC | 05:43 | |
*** eezhova has quit IRC | 05:45 | |
*** eezhova has joined #openstack-lbaas | 05:46 | |
*** Appa has quit IRC | 06:05 | |
johnsom | Hot Dog! Check out the scenario test time on this patch: https://review.openstack.org/#/c/366472/ | 06:07 |
johnsom | That is the difference having nested virtualization enabled.... | 06:07 |
johnsom | gate-neutron-lbaasv2-dsvm-scenarioSUCCESS in 33m 38s (non-voting) | 06:07 |
johnsom | It almost takes an hour off of that test. | 06:10 |
*** Alex_Stef has quit IRC | 06:19 | |
*** Appa has joined #openstack-lbaas | 06:20 | |
*** pcaruana has joined #openstack-lbaas | 06:34 | |
*** ducttape_ has joined #openstack-lbaas | 06:40 | |
*** Alex_Stef has joined #openstack-lbaas | 06:41 | |
*** sticker has quit IRC | 06:42 | |
*** ducttape_ has quit IRC | 06:45 | |
*** bana_k has quit IRC | 06:48 | |
*** Appa has left #openstack-lbaas | 06:49 | |
*** tesseract- has joined #openstack-lbaas | 07:02 | |
*** nmagnezi_ has joined #openstack-lbaas | 07:08 | |
rm_work | johnsom: sweet monkey on fire | 07:25 |
rm_work | that's excellent | 07:25 |
*** eezhova has quit IRC | 07:27 | |
*** kobis has joined #openstack-lbaas | 07:31 | |
*** armax has quit IRC | 07:31 | |
*** woodster_ has quit IRC | 07:39 | |
*** ducttape_ has joined #openstack-lbaas | 07:41 | |
*** ducttape_ has quit IRC | 07:45 | |
*** ihrachys has joined #openstack-lbaas | 07:52 | |
*** eezhova has joined #openstack-lbaas | 08:02 | |
*** nmagnezi_ is now known as nmagnezi | 08:32 | |
*** openstackgerrit has quit IRC | 08:34 | |
*** openstackgerrit has joined #openstack-lbaas | 08:35 | |
*** ihrachys has quit IRC | 08:36 | |
*** Alex_Stef has quit IRC | 08:40 | |
*** links has quit IRC | 08:53 | |
*** links has joined #openstack-lbaas | 08:55 | |
*** ihrachys has joined #openstack-lbaas | 09:19 | |
*** ihrachys has quit IRC | 09:26 | |
eezhova | johnsom: wow, that's just lovely | 09:34 |
*** jsheeren has joined #openstack-lbaas | 09:49 | |
*** jsheeren has left #openstack-lbaas | 09:49 | |
*** ihrachys has joined #openstack-lbaas | 10:05 | |
*** jsheeren has joined #openstack-lbaas | 10:06 | |
*** jsheeren has quit IRC | 10:06 | |
*** jsheeren has joined #openstack-lbaas | 10:06 | |
*** saju_m has joined #openstack-lbaas | 10:25 | |
nmagnezi | ihrachys, ping re: https://review.openstack.org/#/c/299998/ | 10:38 |
nmagnezi | ihrachys, question about your last comment | 10:38 |
nmagnezi | ihrachys, so should i provide q-svc both neutron_lbaas.conf and services_lbaas.conf or just the latter? | 10:39 |
ihrachys | nmagnezi: I think lbaas folks should decide what's their strategy regarding config files, and why they have services_* and neutron_* in the first place. Once you decide what's the end goal, you can adopt the approach in devstack code. | 10:41 |
ihrachys | nmagnezi: from the look of it, I would say, both should be loaded because both are shipped with the repo | 10:41 |
ihrachys | nmagnezi: but since neutron_* contains only service provider option so far, they were happy so far to rely on auto-magic loading by neutron of the file | 10:42 |
nmagnezi | ihrachys, that was my assumption. fair enough. I will make sure to point it out in the patch | 10:42 |
ihrachys | nmagnezi: since the magic is gone in Ocata, we should really load both in lbaas devstack code | 10:42 |
nmagnezi | ihrachys, ack. testing locally. patch will last soon. thanks! :) | 10:43 |
ihrachys | nmagnezi: I am happy to help with config part at least because it seems like I know how it actually works. | 10:44 |
*** saju_m has quit IRC | 10:51 | |
*** jsheeren has quit IRC | 10:55 | |
*** saju_m has joined #openstack-lbaas | 11:04 | |
*** amotoki has quit IRC | 11:06 | |
*** nmagnezi_ has joined #openstack-lbaas | 11:12 | |
*** nmagnezi_ has quit IRC | 11:13 | |
*** nmagnezi_ has joined #openstack-lbaas | 11:14 | |
*** nmagnezi_ has quit IRC | 11:15 | |
*** jsheeren has joined #openstack-lbaas | 11:22 | |
*** gongysh has quit IRC | 11:26 | |
openstackgerrit | Nir Magnezi proposed openstack/neutron-lbaas: Adds neutron_lbaas.conf and services_lbaas.conf to q-svc command line https://review.openstack.org/366690 | 11:32 |
nmagnezi | ihrachys, https://review.openstack.org/#/c/366690 | 11:33 |
nmagnezi | ihrachys, will add our discussion from ^^ as a comment | 11:34 |
*** amotoki has joined #openstack-lbaas | 11:37 | |
ihrachys | nmagnezi: if it's needed for context, could make it into the commit message then | 11:40 |
ihrachys | nmagnezi: assuming the CI will prove the required CLI is passed, I am good. | 11:42 |
*** amotoki has quit IRC | 11:50 | |
*** eandersson has joined #openstack-lbaas | 11:57 | |
*** amotoki has joined #openstack-lbaas | 11:58 | |
*** Frito has joined #openstack-lbaas | 12:04 | |
*** links has quit IRC | 12:04 | |
*** chlong has joined #openstack-lbaas | 12:05 | |
nmagnezi | ihrachys, re: "auto-magic loading by neutron of the file" , if i need to added this to the commit msg i will need some reference | 12:07 |
*** amotoki has quit IRC | 12:07 | |
openstackgerrit | Nir Magnezi proposed openstack/neutron-lbaas: Adds neutron_lbaas.conf and services_lbaas.conf to q-svc command line https://review.openstack.org/366690 | 12:12 |
*** ducttape_ has joined #openstack-lbaas | 12:16 | |
*** links has joined #openstack-lbaas | 12:16 | |
*** ducttape_ has quit IRC | 12:18 | |
*** ducttape_ has joined #openstack-lbaas | 12:18 | |
*** gongysh has joined #openstack-lbaas | 12:24 | |
nmagnezi | ihrachys, btw i looked at service_lbaas.conf again (here is an example of it: http://paste.openstack.org/show/567430/ ) and it seem like the 'allow_automatic_lbaas_agent_failover | 12:25 |
nmagnezi | ihrachys, btw i looked at service_lbaas.conf again (here is an example of it: http://paste.openstack.org/show/567430/ ) and it seem like the 'allow_automatic_lbaas_agent_failover' option fits neutron_lbaas.conf better | 12:26 |
nmagnezi | since neutron_lbaas.conf contains non driver specific scheduler options | 12:26 |
*** ducttape_ has quit IRC | 12:33 | |
*** amotoki has joined #openstack-lbaas | 12:33 | |
*** jsheeren has quit IRC | 12:40 | |
*** woodster_ has joined #openstack-lbaas | 12:42 | |
*** saju_m has quit IRC | 13:01 | |
*** bcafarel is now known as bcafarel_ | 13:06 | |
*** gongysh has quit IRC | 13:07 | |
*** bcafarel_ is now known as bcafarel | 13:07 | |
ihrachys | nmagnezi: maybe, no idea. :) I am fine if lbaas folks are fine. :) | 13:10 |
*** links has quit IRC | 13:12 | |
*** Alex_Stef has joined #openstack-lbaas | 13:13 | |
*** matt-borland has joined #openstack-lbaas | 13:17 | |
*** ducttape_ has joined #openstack-lbaas | 13:18 | |
*** Alex_Stef has quit IRC | 13:19 | |
*** ducttape_ has quit IRC | 13:20 | |
*** ducttape_ has joined #openstack-lbaas | 13:45 | |
*** rcernin has joined #openstack-lbaas | 13:57 | |
*** ddaskal has joined #openstack-lbaas | 14:00 | |
*** ducttape_ has quit IRC | 14:02 | |
*** ddaskal has quit IRC | 14:05 | |
*** nmagnezi_ has joined #openstack-lbaas | 14:09 | |
*** nmagnezi_ has quit IRC | 14:10 | |
*** ajo_ has joined #openstack-lbaas | 14:12 | |
*** nmagnezi_ has joined #openstack-lbaas | 14:16 | |
*** BjoernT has joined #openstack-lbaas | 14:16 | |
*** diogogmt has quit IRC | 14:17 | |
*** bcafarel has quit IRC | 14:18 | |
*** bcafarel has joined #openstack-lbaas | 14:18 | |
openstackgerrit | Phillip Toohill proposed openstack/octavia: WIP: Octavia: Basic LoadBalancer Scenario Test https://review.openstack.org/172199 | 14:20 |
*** eandersson has quit IRC | 14:20 | |
*** darrenc has quit IRC | 14:20 | |
*** jidar has quit IRC | 14:20 | |
*** nmagnezi has quit IRC | 14:21 | |
*** kobis has quit IRC | 14:24 | |
diltram | morning | 14:31 |
*** pglass has joined #openstack-lbaas | 14:38 | |
*** eandersson has joined #openstack-lbaas | 14:39 | |
*** ducttape_ has joined #openstack-lbaas | 14:39 | |
*** darrenc has joined #openstack-lbaas | 14:42 | |
*** jidar has joined #openstack-lbaas | 14:42 | |
*** amotoki has quit IRC | 14:56 | |
*** _ducttape_ has joined #openstack-lbaas | 15:00 | |
*** ducttape_ has quit IRC | 15:03 | |
*** gongysh has joined #openstack-lbaas | 15:04 | |
*** diogogmt has joined #openstack-lbaas | 15:04 | |
*** nmagnezi_ has quit IRC | 15:13 | |
*** diogogmt has quit IRC | 15:15 | |
*** diogogmt has joined #openstack-lbaas | 15:19 | |
*** _ducttape_ has quit IRC | 15:22 | |
*** ducttape_ has joined #openstack-lbaas | 15:22 | |
*** armax has joined #openstack-lbaas | 15:27 | |
*** Lakshman has joined #openstack-lbaas | 15:31 | |
Lakshman | Hi All, | 15:32 |
Lakshman | I have a query on L7Policy | 15:33 |
Lakshman | Is this the right place to dicuss? | 15:33 |
Lakshman | *discuss | 15:33 |
openstackgerrit | Lubosz Kosnik (diltram) proposed openstack/octavia: Backend Keystone authentication https://review.openstack.org/364655 | 15:39 |
diltram | Lakshman: yes | 15:39 |
*** chlong has quit IRC | 15:41 | |
Lakshman | When I create l7policy from openstack, I don't see any impact on status of loadbalancer. | 15:41 |
Lakshman | Is this expected behaviour? | 15:41 |
Lakshman | Or am I missing something in the setup? | 15:42 |
johnsom | You need L7 rules as well | 15:43 |
Lakshman | Yes I have created them as well | 15:44 |
Lakshman | If am not wrong there should be change in status of loadbalancer from ACTIVE to PENDING_UPDATE on create of l7policy or l7rule. | 15:45 |
*** gongysh has quit IRC | 15:45 | |
*** pglass has quit IRC | 15:46 | |
*** pglass has joined #openstack-lbaas | 15:54 | |
sbalukoff | Creating L7policies and rules is usually a very fast operation. (As in, can usually happen in less than 1 second.) How long are you waiting between the time you create the policy / rule and check the status of the load balancer? | 16:00 |
*** pcaruana has quit IRC | 16:01 | |
Lakshman | The moment L7Policy create command returns, I issue a command to list lbs. And I always see lb in ACTIVE state. | 16:02 |
sbalukoff | Has the L7 Policy finished being created at that point? (How are you determining this?) | 16:07 |
*** bana_k has joined #openstack-lbaas | 16:07 | |
*** tesseract- has quit IRC | 16:09 | |
Lakshman | I am working on supporting L7switching for Netscaler. | 16:10 |
Lakshman | I could see calls reaching my services from openstack via my driver. | 16:11 |
Lakshman | I could see an entry in l7policies table in neutron db | 16:13 |
*** ajo_ has quit IRC | 16:15 | |
Lakshman | Is there any way I can introduce some sleep in l7policy operation, so that lb will stay in PENDING_UPDATE for long time? | 16:16 |
openstackgerrit | Lubosz Kosnik (diltram) proposed openstack/octavia: Backend Keystone authentication https://review.openstack.org/364655 | 16:16 |
*** ajo_ has joined #openstack-lbaas | 16:16 | |
diltram | ^^ Ready to review finally :) | 16:17 |
openstackgerrit | Lubosz Kosnik (diltram) proposed openstack/octavia: Backend Keystone authentication https://review.openstack.org/364655 | 16:18 |
johnsom | Lakshman I would hope we don't add sleep to the code. Is this just a situation where the Netscaler driver is returning success too fast or running async? | 16:23 |
johnsom | rm_work I see you +2 | 16:24 |
johnsom | 'd https://review.openstack.org/#/c/365382/ | 16:24 |
rm_work | yeah | 16:24 |
rm_work | that seems right to me | 16:24 |
johnsom | I agree the code is fine, but are we using it? | 16:24 |
johnsom | Or are we just adding dead code? | 16:25 |
rm_work | well | 16:25 |
rm_work | if someone added the __eq__ | 16:25 |
rm_work | for consistency we SHOULD have the NE just in case someone tries it | 16:25 |
rm_work | whether it's called or not | 16:25 |
rm_work | i'm imagining someone doing NE and it behaving oddly and how much of their life we'd waste with them trying to debug it | 16:26 |
rm_work | when we could just correct it now | 16:26 |
johnsom | Hmm, ok. The dead code argument was my I was sitting on that one trying to decide. | 16:26 |
rm_work | there's dead code, and then there's landmine code | 16:26 |
johnsom | Yeah, true | 16:26 |
johnsom | I'll have a look at why scenario failed and push it through | 16:27 |
rm_work | i feel like not fixing this inconsistency just because no one tried to do it yet is like leaving a landmine in place :P | 16:27 |
johnsom | This one is still open for review BTW: https://review.openstack.org/#/c/364707/ | 16:27 |
johnsom | Fix the admin state bug | 16:27 |
*** matt-borland has quit IRC | 16:28 | |
rm_work | i'll look today | 16:29 |
*** Lakshman has quit IRC | 16:29 | |
*** Lakshman has joined #openstack-lbaas | 16:30 | |
Lakshman | Yes, its a async call to Netscaler. But it does not impact the status of L7Policy instantaneously. Driver will collect status of PENDING_* entities frequently and then update the status of such entities in Openstack DB. | 16:38 |
Lakshman | That's how it is currently working for entities like lb, listener, pool and other entities | 16:40 |
Lakshman | I mean creation of listener or pool or pool_member or healthmonitor would push lb into PENDING_UPDATE state. Later on driver will collect the status of all child entities of lb which is in PENDING_UPDATE state and eventually update the status in Openstack DB. | 16:44 |
*** diogogmt has quit IRC | 16:46 | |
*** nmagnezi has joined #openstack-lbaas | 16:48 | |
*** Lakshman has quit IRC | 16:51 | |
*** daneyon has joined #openstack-lbaas | 16:57 | |
*** bana_k has quit IRC | 16:57 | |
*** daneyon has quit IRC | 16:58 | |
xgerman | johnsom: https://review.openstack.org/#/c/251168/ | 17:04 |
xgerman | + rm_work | 17:04 |
*** ajo_ has quit IRC | 17:14 | |
*** ajo_ has joined #openstack-lbaas | 17:14 | |
*** links has joined #openstack-lbaas | 17:15 | |
*** matt-borland has joined #openstack-lbaas | 17:18 | |
*** ducttape_ has quit IRC | 17:18 | |
johnsom | Cool stuff | 17:19 |
*** kevo has joined #openstack-lbaas | 17:22 | |
*** links has quit IRC | 17:29 | |
*** ihrachys has quit IRC | 17:30 | |
*** ducttape_ has joined #openstack-lbaas | 17:38 | |
*** Alex_Stef has joined #openstack-lbaas | 17:39 | |
*** harlowja has quit IRC | 17:46 | |
*** csomerville has joined #openstack-lbaas | 17:48 | |
*** bana_k has joined #openstack-lbaas | 17:49 | |
*** harlowja has joined #openstack-lbaas | 17:49 | |
*** saju_m has joined #openstack-lbaas | 17:52 | |
*** ajo_ has quit IRC | 17:53 | |
*** ajo_ has joined #openstack-lbaas | 17:53 | |
carrbs | is 'docs' expected to fail when I run tox in a dev environment? | 17:58 |
*** diogogmt has joined #openstack-lbaas | 17:59 | |
johnsom | No | 17:59 |
johnsom | specs is broken at the moment, but docs should run | 17:59 |
*** barclaac|2 has joined #openstack-lbaas | 18:01 | |
*** barclaac|2 has quit IRC | 18:01 | |
openstackgerrit | Lubosz Kosnik (diltram) proposed openstack/octavia: Add keystone authentication of token https://review.openstack.org/265065 | 18:09 |
*** matt-borland has quit IRC | 18:10 | |
diltram | where I can find info that patch which I send to gerrit is depended of smth? | 18:11 |
diltram | ok, found it | 18:12 |
diltram | we have finally keystone auth completely working :) | 18:12 |
diltram | we need to add policy.json and we're good to go :) | 18:12 |
*** csomerville has quit IRC | 18:14 | |
*** matt-borland has joined #openstack-lbaas | 18:18 | |
*** kevo has quit IRC | 18:22 | |
diltram | we're planning to use low-hanging-fruit tags on launchpad? | 18:30 |
diltram | I just added a bug with this dot file compilation and I used that tah | 18:30 |
diltram | tag | 18:30 |
johnsom | Yeah, that is a fine tag | 18:31 |
carrbs | johnsom: this is the error I'm getting, maybe it's just a missing dep? http://paste.openstack.org/show/567589/ | 18:32 |
diltram | I don't know why but I have the same error sometimes | 18:33 |
carrbs | there's a file in that directory, but it has a different hash | 18:33 |
carrbs | it's almost in there: http://paste.openstack.org/show/567591/ | 18:38 |
openstackgerrit | Merged openstack/octavia: Add __ne__ built-in function https://review.openstack.org/365382 | 18:39 |
*** saju_m has quit IRC | 18:39 | |
rm_work | yeah... watching a recording of a DIFFERENT display | 18:42 |
rm_work | not really getting the color gamut thing | 18:42 |
rm_work | need to see it IRL | 18:42 |
rm_work | err, wrong channel :P | 18:42 |
rm_work | (watching the iphone stream lol) | 18:43 |
carrbs | fixed by killing docs/build/ and re-running | 18:44 |
rm_work | yeah that's usually what i do | 18:44 |
carrbs | is this an error anyone is familiar with running tox? http://paste.openstack.org/show/567593/ | 18:50 |
*** Alex_Stef has quit IRC | 18:57 | |
diltram | yeah I see them all the time | 18:57 |
rm_work | is the test actually failing? | 18:57 |
rm_work | it prints out expected errors as well | 18:58 |
carrbs | diltram: haha | 18:59 |
diltram | johnsom: In the next few days I'm planning to write that API docs and after that I will work on policy.json, ok? | 19:00 |
diltram | rm_work: yeah, it resulted errors | 19:00 |
carrbs | rm_work, yes it fails with: `ERROR: py34: InvocationError: '/home/carrbs/octavia/.tox/py34/bin/python /home/carrbs/octavia/setup.py --name'` | 19:00 |
diltram | expected* | 19:00 |
diltram | ok, so it not suppose to fail :P | 19:00 |
johnsom | By API docs you mean the spec or pulling in the oslo API doc engine? | 19:03 |
diltram | the spec | 19:03 |
johnsom | Ok, cool | 19:03 |
johnsom | Sounds great to me | 19:03 |
johnsom | That tox error with mock, I'm not sure I have seen that one | 19:04 |
diltram | and also that docs use os-api-ref | 19:04 |
johnsom | Hahaha, adding scope eh? | 19:04 |
johnsom | I think I have the policy bug assigned to me on launchpad, feel free to grab it. I didn't get beyond research during the mid-cycle | 19:05 |
diltram | ok, I will ping you when I will start working on that | 19:11 |
openstackgerrit | Phillip Toohill proposed openstack/octavia: Fix No sql_connection parameter is established https://review.openstack.org/366913 | 19:12 |
*** rcernin has quit IRC | 19:21 | |
*** ducttape_ has quit IRC | 19:23 | |
*** ducttape_ has joined #openstack-lbaas | 19:27 | |
*** _ducttape_ has joined #openstack-lbaas | 19:30 | |
*** ducttape_ has quit IRC | 19:34 | |
diltram | ptoohill: there is Toggle CI button on review | 19:36 |
diltram | and it's showing all jenkins logs | 19:36 |
diltram | you can grab in that way the logs | 19:36 |
*** _ducttape_ has quit IRC | 19:39 | |
*** ducttape_ has joined #openstack-lbaas | 19:40 | |
ptoohill | diltram: I dont see the button but I updated the bug report. | 19:40 |
diltram | ok | 19:40 |
diltram | thx | 19:40 |
diltram | ptoohill: +1 | 19:46 |
ptoohill | Thanks for the review diltram ! | 19:47 |
diltram | np :) | 19:47 |
Frito | meeting soon in #openstack-meeting-alt right? | 19:58 |
*** _ducttape_ has joined #openstack-lbaas | 19:59 | |
openstackgerrit | Trevor Vardeman proposed openstack/octavia: Add quota support to Octavia https://review.openstack.org/360794 | 19:59 |
sbalukoff | Frito: Yes. | 20:00 |
Frito | sbalukoff: Thx. I always forget what channel it is for some reason | 20:00 |
*** ducttape_ has quit IRC | 20:02 | |
*** openstackgerrit has quit IRC | 20:04 | |
*** BjoernT has quit IRC | 20:04 | |
*** openstackgerrit has joined #openstack-lbaas | 20:05 | |
*** BjoernT has joined #openstack-lbaas | 20:37 | |
*** _ducttape_ has quit IRC | 20:40 | |
*** ducttape_ has joined #openstack-lbaas | 20:41 | |
pglass | so we're running `bandit-baseline` in tox.ini. if you just run `bandit`, it doesn't compare issues between git refs. and if I do run plain bandit, it lists a bunch of issues that don't show up with bandit-baseline. | 20:41 |
*** BjoernT has quit IRC | 20:41 | |
pglass | so like, we've implicitly accepted all bandit issues that existed in the code at the time we introduced bandit to our tests. | 20:42 |
pglass | because that will only ever catches new issues. | 20:42 |
pglass | catch* | 20:42 |
sbalukoff | pglass: I think this was a compromise to get bandit testing going to start with. Yes, we should go through and fix existing issues. But in the mean time, we shouldn't block new code that doesn't add new issues. | 20:45 |
*** kevo has joined #openstack-lbaas | 20:46 | |
pglass | sbalukoff: sure. my point is if you do work toward running plain bandit (rather than stifling the error from bandit-baseline), then the dev experience is better and you actually have fewer issues. | 20:47 |
*** Frito has quit IRC | 20:47 | |
*** gcheresh has quit IRC | 20:47 | |
sbalukoff | pglass: Feel free to submit patches to get us there. :) | 20:48 |
*** gongysh has joined #openstack-lbaas | 20:49 | |
openstackgerrit | Nir Magnezi proposed openstack/neutron-lbaas: Auto reschedule loadbalancers from dead agents https://review.openstack.org/299998 | 20:51 |
*** _ducttape_ has joined #openstack-lbaas | 20:55 | |
*** ducttape_ has quit IRC | 20:59 | |
nmagnezi | balukoff, Depends-On: I652ab029b7427c8783e4b2f0443a89ee884bf064 --> like that? | 21:00 |
sbalukoff | Yes, like that! | 21:00 |
sbalukoff | Ok, folks, need to go AFK for about an hour; Will be back afterward. | 21:01 |
nmagnezi | sbalukoff, updated. | 21:01 |
nmagnezi | sbalukoff, bye :) i'm going offline. and thanks! | 21:01 |
sbalukoff | nmagnezi: Cool beans. Add that release note, and I'll put a +2 on it (with a note that it won't get merged until Newton is cut.) | 21:02 |
diltram | johnsom: we have a bug for support oslo.config generator? | 21:02 |
nmagnezi | johnsom, i covered all of my topics. thanks for allowing me to raise those in the meeting. | 21:02 |
johnsom | nmagnezi Sure, no problem. Sorry we didn't get those into newton | 21:03 |
diltram | yeah, we really not merged too much in N release | 21:03 |
johnsom | A lot of bug fixes....\ | 21:04 |
johnsom | diltram https://bugs.launchpad.net/octavia/+bug/1604584 | 21:04 |
openstack | Launchpad bug 1604584 in octavia "Octavia should use the oslo config sphinx integration for confiig docs" [Wishlist,Fix released] | 21:04 |
diltram | but still it's not end of world, we can work on that in next release | 21:04 |
sbalukoff | Feature-wise, that's true. I feel like we have a lot of bug fixes / stability improvements though. | 21:04 |
nmagnezi | johnsom, yeah that's too bad. that could really be beneficial for this release | 21:04 |
diltram | true | 21:04 |
sbalukoff | Ok, really going AFK. See y'all later, eh! | 21:05 |
nmagnezi | johnsom, i was under the impression that since I had those in gerrit some time ago that we can still merge those. | 21:05 |
johnsom | resolved 64 bugs | 21:05 |
diltram | johnsom: but we still need to add tool which is generating conf file and configuration for it | 21:06 |
johnsom | nmagnezi Past the feature freeze date you can request a Feature Freeze Extension (FFE) if it doesn't have any -1's and is complete, i.e. docs, cli, etc. | 21:06 |
*** _ducttape_ has quit IRC | 21:06 | |
johnsom | diltram That was the config docs work. The auto generation of a config file is already in the repo, we just aren't using it. I think because we annotate the config more than the tool would allow | 21:07 |
johnsom | diltram https://github.com/openstack/octavia/blob/master/tox.ini#L53 | 21:08 |
diltram | johnsom: ok, but that config looks like very very small in comparison to our real config | 21:09 |
diltram | :P | 21:09 |
johnsom | Yeah, I don't know that we have looked at that in a while because we maintain the commented one. | 21:10 |
diltram | I know ;) | 21:10 |
johnsom | diltram Is this patch still needed with your new one? https://review.openstack.org/265065 | 21:10 |
diltram | johnsom: yeah, it's second one in chain | 21:11 |
nmagnezi | johnsom, is it still possible to ask for it for those? or is it a final verdict. | 21:11 |
*** ducttape_ has joined #openstack-lbaas | 21:12 | |
johnsom | It's my final answer but you can try to convince dougwig our neutron lieutenant as we are under neutron still. | 21:13 |
*** minwang2 has joined #openstack-lbaas | 21:13 | |
*** minwang2 has left #openstack-lbaas | 21:14 | |
*** bana_k has quit IRC | 21:14 | |
*** bank_ has joined #openstack-lbaas | 21:14 | |
johnsom | It really comes down to risk for the release and time to find and patch any bugs. | 21:14 |
openstackgerrit | Lubosz Kosnik (diltram) proposed openstack/octavia: Backend Keystone authentication https://review.openstack.org/364655 | 21:17 |
*** matt-borland has quit IRC | 21:17 | |
openstackgerrit | Lubosz Kosnik (diltram) proposed openstack/octavia: Add keystone authentication of token https://review.openstack.org/265065 | 21:17 |
*** BjoernT has joined #openstack-lbaas | 21:19 | |
diltram | ok, cu guys | 21:22 |
diltram | I'm going home | 21:22 |
*** ducttape_ has quit IRC | 21:31 | |
dougwig | nmagnezi: having your review in gerrit is a pre-req for an FFE, but it does not guarantee one. in this case, it's not the ref driver, so you'd need several cores to agree to review and for johnsom to agree to add it to the release load/risk. i'm sorry that it sat in gerrit so long, but i'd consider his no to be final. | 21:35 |
*** ducttape_ has joined #openstack-lbaas | 21:36 | |
openstackgerrit | Paul Glass proposed openstack/octavia: Stop using bandit-baseline https://review.openstack.org/366997 | 21:40 |
nmagnezi | dougwig, i'm sad to hear this is your opinion, but I accept it. I hope nothing will prevent us from merging this in Ocata. thanks you for the answer. | 21:42 |
*** ducttape_ has quit IRC | 21:43 | |
dougwig | nmagnezi: i understand, and please do solicit reviews in ocata to get things moving. | 21:44 |
nmagnezi | dougwig, i sure will | 21:44 |
openstackgerrit | Paul Glass proposed openstack/octavia: Stop using bandit-baseline https://review.openstack.org/366997 | 21:52 |
*** nmagnezi has quit IRC | 21:56 | |
openstackgerrit | Paul Glass proposed openstack/octavia: Stop using bandit-baseline https://review.openstack.org/366997 | 21:59 |
openstackgerrit | Michael Johnson proposed openstack/neutron-lbaas: Enable kvm for neutron-lbaas/Octavia gates https://review.openstack.org/366472 | 22:01 |
*** BjoernT has quit IRC | 22:02 | |
*** ducttape_ has joined #openstack-lbaas | 22:03 | |
*** gongysh has quit IRC | 22:03 | |
*** ducttape_ has quit IRC | 22:12 | |
*** chlong has joined #openstack-lbaas | 22:12 | |
*** pglass has quit IRC | 22:18 | |
*** ducttape_ has joined #openstack-lbaas | 22:19 | |
openstackgerrit | Phillip Toohill proposed openstack/octavia: Fix No sql_connection parameter is established https://review.openstack.org/366913 | 22:26 |
*** BjoernT has joined #openstack-lbaas | 22:26 | |
*** BjoernT has quit IRC | 22:26 | |
*** gongysh has joined #openstack-lbaas | 22:27 | |
*** amuller has joined #openstack-lbaas | 22:39 | |
amuller | sbalukoff: respun https://review.openstack.org/#/c/299998/ with a release note | 22:39 |
openstackgerrit | Assaf Muller proposed openstack/neutron-lbaas: Auto reschedule loadbalancers from dead agents https://review.openstack.org/299998 | 22:39 |
amuller | ... now I did | 22:40 |
amuller | sbalukoff: if you have any notes I can respin it again this evening | 22:40 |
*** ducttape_ has quit IRC | 22:48 | |
*** ducttape_ has joined #openstack-lbaas | 22:55 | |
*** armax has quit IRC | 22:58 | |
sbalukoff | amuller: Looks good! Added my +2. | 22:58 |
amuller | sbalukoff: I just ran tox -e releasenotes locally and it causes an issue, hold on... | 22:58 |
sbalukoff | Ok. | 22:58 |
openstackgerrit | Assaf Muller proposed openstack/neutron-lbaas: Auto reschedule loadbalancers from dead agents https://review.openstack.org/299998 | 23:02 |
amuller | sbalukoff: sorry, should be good now | 23:02 |
sbalukoff | amuller: Ok! | 23:03 |
*** harlowja has quit IRC | 23:03 | |
*** gongysh has quit IRC | 23:08 | |
amuller | sbalukoff: thanks :) | 23:14 |
*** amuller has quit IRC | 23:14 | |
*** ducttape_ has quit IRC | 23:17 | |
*** diogogmt has quit IRC | 23:24 | |
*** ducttape_ has joined #openstack-lbaas | 23:46 | |
*** ducttape_ has quit IRC | 23:48 | |
*** ducttape_ has joined #openstack-lbaas | 23:48 | |
*** armax has joined #openstack-lbaas | 23:50 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!