Friday, 2016-10-14

*** ducttape_ has joined #openstack-lbaas00:05
*** ducttape_ has quit IRC00:05
*** ducttape_ has joined #openstack-lbaas00:05
*** harlowja has joined #openstack-lbaas00:06
diltramsbalukoff: I'm gere00:08
sbalukoffdiltram: Sorry to bug you! johnsom already got it, eh.00:09
diltramok, np :)00:09
diltramsbalukoff, johnsom: heard about that? https://wiki.fd.io/view/VPP/What_is_VPP%3F00:19
johnsomYes00:20
johnsomThere is an openstack project proposed to use it00:22
*** bdeschenes has joined #openstack-lbaas00:22
johnsomhttp://osdir.com/ml/openstack-dev/2016-10/msg00258.html00:22
*** bana_k has quit IRC00:23
sbalukoffdiltram: That's cool, eh!00:24
diltramnice, didn't know that there is something using it already00:25
*** ducttape_ has quit IRC00:26
johnsomWell, it's proposed...00:28
openstackgerritMerged openstack/neutron-lbaas: Fix a gate issue with a mock and deprecated call  https://review.openstack.org/38627000:34
openstackgerritMichael Johnson proposed openstack/neutron-lbaas: Create a consolidated API test  https://review.openstack.org/38624000:34
*** bana_k has joined #openstack-lbaas00:52
*** yuanying has quit IRC01:03
*** gongysh_ has quit IRC01:06
*** yuanying has joined #openstack-lbaas01:07
*** fnaval has quit IRC01:09
*** ducttape_ has joined #openstack-lbaas01:13
*** fnaval has joined #openstack-lbaas01:19
*** yuanying_ has joined #openstack-lbaas01:23
*** bdeschenes has quit IRC01:25
*** bana_k has quit IRC01:26
*** yuanying has quit IRC01:26
*** derekjhyang has joined #openstack-lbaas01:49
*** gongysh_ has joined #openstack-lbaas02:09
*** cody-somerville has joined #openstack-lbaas02:24
*** cody-somerville has quit IRC02:24
*** cody-somerville has joined #openstack-lbaas02:24
*** gongysh_ has quit IRC02:37
*** ducttape_ has quit IRC02:43
*** yuanying_ has quit IRC02:51
*** yuanying has joined #openstack-lbaas02:52
*** armax has quit IRC03:09
*** tovin07 has joined #openstack-lbaas03:10
*** sputnik13 has joined #openstack-lbaas03:12
*** yuanying has quit IRC03:15
*** yuanying has joined #openstack-lbaas03:15
*** yuanying_ has joined #openstack-lbaas03:17
*** yuanying has quit IRC03:18
*** yuanying_ has quit IRC03:22
*** ducttape_ has joined #openstack-lbaas03:23
*** yamamoto has quit IRC03:30
*** ducttape_ has quit IRC03:39
*** yuanying has joined #openstack-lbaas03:52
*** links has joined #openstack-lbaas03:59
*** sputnik13 has quit IRC04:15
*** anilvenkata has joined #openstack-lbaas04:28
*** gongysh_ has joined #openstack-lbaas04:30
openstackgerritStephen Balukoff proposed openstack/octavia: Load system sysctl in amphora network namespace  https://review.openstack.org/38632804:36
*** ducttape_ has joined #openstack-lbaas04:39
*** yamamoto has joined #openstack-lbaas04:44
*** ducttape_ has quit IRC04:45
*** bana_k has joined #openstack-lbaas05:01
*** sputnik13 has joined #openstack-lbaas05:06
*** bana_k has quit IRC05:29
openstackgerritOpenStack Proposal Bot proposed openstack/octavia: Updated from global requirements  https://review.openstack.org/38637005:41
*** ducttape_ has joined #openstack-lbaas05:41
*** ducttape_ has quit IRC05:45
*** sputnik13 has quit IRC05:48
*** amotoki has joined #openstack-lbaas05:51
*** woodster_ has quit IRC05:55
*** numans has joined #openstack-lbaas06:12
*** pcaruana has joined #openstack-lbaas06:23
*** jsheeren has joined #openstack-lbaas06:40
*** ducttape_ has joined #openstack-lbaas06:42
*** ducttape_ has quit IRC06:46
*** jsheeren has quit IRC06:50
*** jsheeren has joined #openstack-lbaas06:51
*** tesseract- has joined #openstack-lbaas07:05
*** rcernin has joined #openstack-lbaas07:20
openstackgerritStephen Balukoff proposed openstack/octavia: Update haproxy logging configuration  https://review.openstack.org/38642707:21
*** amoralej|off is now known as amoralej07:27
*** ducttape_ has joined #openstack-lbaas07:42
*** ihrachys has joined #openstack-lbaas07:43
*** ihrachys has quit IRC07:45
*** ducttape_ has quit IRC07:47
*** ihrachys has joined #openstack-lbaas07:48
*** sputnik13 has joined #openstack-lbaas07:52
*** gtrxcb has joined #openstack-lbaas07:55
*** eezhova has joined #openstack-lbaas08:00
*** gtrxcb has quit IRC08:10
openstackgerritMiguel Angel Ajo proposed openstack/octavia: Enable support for multinode in the gate hook  https://review.openstack.org/37867708:15
openstackgerritMiguel Angel Ajo proposed openstack/octavia: WIP/DONOTMERGE: multinode ignore server_ca (checking)  https://review.openstack.org/38645008:17
*** ipsecguy has quit IRC08:28
*** eezhova has quit IRC08:31
*** ducttape_ has joined #openstack-lbaas08:43
*** ducttape_ has quit IRC08:48
*** ipsecguy has joined #openstack-lbaas08:50
*** eezhova has joined #openstack-lbaas09:01
openstackgerritMerged openstack/octavia: Updated from global requirements  https://review.openstack.org/38637009:09
*** ihrachys has quit IRC09:10
*** ihrachys has joined #openstack-lbaas09:13
*** ihrachys has quit IRC09:16
openstackgerritMiguel Angel Ajo proposed openstack/octavia: WIP/DONOTMERGE: multinode ignore server_ca (checking)  https://review.openstack.org/38645009:17
*** Kiall has joined #openstack-lbaas09:37
*** amotoki has quit IRC09:41
*** ihrachys has joined #openstack-lbaas09:43
*** ducttape_ has joined #openstack-lbaas09:44
*** ihrachys has quit IRC09:44
*** openstack has joined #openstack-lbaas09:48
*** ducttape_ has quit IRC09:48
*** amotoki has joined #openstack-lbaas09:55
*** tovin07 has quit IRC10:00
*** amotoki has quit IRC10:04
*** openstack has joined #openstack-lbaas10:17
*** sputnik13 has quit IRC10:20
*** jsheeren has quit IRC10:33
*** ducttape_ has joined #openstack-lbaas10:45
*** amotoki has joined #openstack-lbaas10:49
*** ducttape_ has quit IRC10:50
*** gongysh_ has quit IRC10:58
*** kobis has joined #openstack-lbaas11:05
openstackgerritMiguel Angel Ajo proposed openstack/octavia: Enable support for multinode in the gate hook  https://review.openstack.org/37867711:16
openstackgerritMiguel Angel Ajo proposed openstack/octavia: Increase the verbosity of haproxy.rest_api_driver failures  https://review.openstack.org/38654411:16
*** kobis has quit IRC11:20
*** kobis has joined #openstack-lbaas11:23
*** amoralej is now known as amoralej|lunch11:28
*** yamamoto has quit IRC11:29
*** ducttape_ has joined #openstack-lbaas11:46
*** strigazi is now known as strigazi_AFK11:48
*** ducttape_ has quit IRC11:51
*** kobis has quit IRC12:00
*** links has quit IRC12:02
*** links has joined #openstack-lbaas12:14
*** amotoki has quit IRC12:15
*** yamamoto has joined #openstack-lbaas12:20
*** ducttape_ has joined #openstack-lbaas12:20
*** yamamoto has quit IRC12:24
*** yamamoto has joined #openstack-lbaas12:26
*** anilvenkata is now known as anilvenkata_afk12:32
*** yamamoto has quit IRC12:36
*** yamamoto has joined #openstack-lbaas12:37
*** openstack has joined #openstack-lbaas12:47
*** ducttape_ has quit IRC12:49
*** ducttape_ has joined #openstack-lbaas13:04
*** reedip has quit IRC13:06
*** bdeschenes has joined #openstack-lbaas13:10
*** eezhova has quit IRC13:12
*** eezhova has joined #openstack-lbaas13:14
*** eezhova has quit IRC13:15
*** links has quit IRC13:16
*** ducttape_ has quit IRC13:17
*** eezhova has joined #openstack-lbaas13:18
*** eezhova has quit IRC13:21
*** portdirect has quit IRC13:24
*** portdirect has joined #openstack-lbaas13:24
*** eezhova has joined #openstack-lbaas13:26
*** eezhova has quit IRC13:26
*** yamamoto has quit IRC13:30
*** gongysh has joined #openstack-lbaas13:37
*** ducttape_ has joined #openstack-lbaas13:39
*** yamamoto has joined #openstack-lbaas13:42
*** eezhova has joined #openstack-lbaas13:42
*** yamamoto has quit IRC13:44
*** yamamoto has joined #openstack-lbaas13:44
openstackgerritMiguel Angel Ajo proposed openstack/octavia: Fixes pregenerated certificates validity to 50y  https://review.openstack.org/38662313:46
openstackgerritMiguel Angel Ajo proposed openstack/octavia: Enable support for multinode in the gate hook  https://review.openstack.org/37867713:47
openstackgerritMiguel Angel Ajo proposed openstack/octavia: Fixes pregenerated certificates validity to 50y  https://review.openstack.org/38662313:47
ajosorry for the excessive noise ;)13:47
*** amoralej|lunch is now known as amoralej13:50
*** fnaval has quit IRC13:57
*** fnaval has joined #openstack-lbaas14:08
*** ducttape_ has quit IRC14:17
ajo2016-10-14 14:22:01.475446 | + ./safe-devstack-vm-gate-wrap.sh:main:L616:   sudo mv /home/jenkins/workspace/gate-octavia-v1-dsvm-scenario-multinode-nv/devstack-gate-cleanup-host.txt /opt/stack/logs/14:24
ajo2016-10-14 14:22:01.481642 | + ./safe-devstack-vm-gate-wrap.sh:main:L618:   exit 014:24
ajo2016-10-14 14:22:01.482755 | [Zuul] Task exit code: 014:24
ajo2016-10-14 14:22:06.090446 | [Zuul] Job complete, result: SUCCESS14:24
ajooh yeah folks! :D14:24
ajologan-, blogan,  dougwig,   sbalukoff , ptoohill ^14:26
ajo:)14:26
ajomultinode scenario working ':D14:26
ajoin 27m, amazing :)14:27
ajohttps://review.openstack.org/#/c/378677/14:27
*** ducttape_ has joined #openstack-lbaas14:32
*** ducttape_ has quit IRC14:33
*** ducttape_ has joined #openstack-lbaas14:34
openstackgerritMiguel Angel Ajo proposed openstack/octavia: Enable support for multinode in the gate hook  https://review.openstack.org/37867714:39
*** amotoki has joined #openstack-lbaas14:41
*** bdeschenes has quit IRC14:48
*** amotoki has quit IRC14:50
*** portdirect has quit IRC14:57
*** portdirect has joined #openstack-lbaas14:58
*** portdirect has quit IRC14:58
*** portdirect_ has joined #openstack-lbaas14:58
*** portdirect_ is now known as portdirect14:59
*** rcernin has quit IRC15:02
*** strigazi_AFK is now known as strigazi15:04
*** pcaruana has quit IRC15:15
*** armax has joined #openstack-lbaas15:20
diltramajo: great job man :)15:31
*** bana_k has joined #openstack-lbaas15:40
bloganajo: awesome!15:40
diltrambest copyrights ever :P # Copyright 2012 Your Mom15:46
ajodiltram, blogan :)15:46
ajolol15:46
ajothanks diltram blogan I sweated a bit to get it up , but now it works :D I will propose it to non-voting check once it's merged :15:51
ajoso we can gather stats and compare to the singlenode15:51
blogandiltram: where is that copyright?15:51
diltramhttps://review.openstack.org/#/c/382147/315:51
bloganof course it'd be carlos15:52
diltramyeah :P15:52
ajolol15:52
*** bana_k has quit IRC15:52
bloganajo: yeah the multinode will be quite informative, for better or worse ;)15:53
diltram+215:53
johnsomblogan oye....15:54
*** tesseract- has quit IRC15:54
bloganjohnsom: the second i saw what diltram wrote, i knew who it'd be15:55
diltram:P15:55
ajothanks diltram, please note I have a depends-on https://review.openstack.org/#/c/381965/ (devstack fix for bug), other projects, like ironic, have included that in their gate_hook15:56
ajoso until that merges, I will include it in ours too, so we don't have to wait15:56
xgermanblogan what happened to octavia.io? RAX needs to save?15:56
ajodiltram, long history short, some of the multinode-capable clouds use the 10.1.x.x range for their nodes, and the neutron subnet pool in 10.0.0.0/8 by default breaks it15:56
bloganxgerman: ptoohill was maintainer of that, i only consumed it15:57
xgermanok, the waybackmachine is our only hope15:57
diltramajo: heard that, I saw e-mail on mailing list about that15:57
ajodiltram, please also note those two depended patches in octavia: https://review.openstack.org/#/c/386544/1 & https://review.openstack.org/#/c/386623/215:57
ajowell, actually first, is not a real dependency, I will remove as a base15:58
johnsomxgerman I see the domain still works: sonar.octavia.io15:58
ajothe 2nd one (regenerated pregenerated certs...) is15:58
ajoapparently, I cleated a client.pem for 10 years, and the CA was just for 1 month ':D15:58
xgermanwell, I was hoping for our specs and docs...15:58
ajocleated->created15:58
ajo':D15:58
johnsomxgerman This works well now: http://docs.openstack.org/developer/octavia/ or the docs job15:58
xgermanoh, we move don15:59
diltramyeah, we can even kill that old octavia.io because it leads to missunderstandings - someone is updating that website?16:00
johnsomIt auto updates16:00
diltramok16:00
diltramReview could not be loaded. - I can see this16:00
johnsomJust don't kill the domain!  We need it for our coverage gate16:01
blogandiltram: octavia.io reads all communication and understands all...by suggesting killing it you've activated it to take over the world!16:01
diltramhahahha :p16:01
diltramhttps://review.openstack.org/#/c/386328/ - this change is based on test results?16:03
*** gongysh has quit IRC16:04
johnsomIt is a problem.  I looked into it when they first raised it.16:05
openstackgerritMiguel Angel Ajo proposed openstack/octavia: Enable support for multinode in the gate hook  https://review.openstack.org/37867716:05
openstackgerritMiguel Angel Ajo proposed openstack/octavia: Fixes pregenerated certificates validity to 50y  https://review.openstack.org/38662316:05
johnsomThe created namespaces start blank16:05
ajook, those two are now the only patches missing for multinode ^ other deps removed :)16:06
*** bana_k has joined #openstack-lbaas16:07
*** yamamoto has quit IRC16:09
*** yamamoto has joined #openstack-lbaas16:11
rm_workTIL: python only flyweights -5 through 25616:24
rm_workso, don't rely on "is" for integer comparisons16:24
*** ihrachys has quit IRC16:28
*** numans has quit IRC16:38
*** bana_k has quit IRC16:41
*** amoralej is now known as amoralej|off17:00
*** cody-somerville has quit IRC17:06
*** yamamoto has quit IRC17:06
*** yamamoto has joined #openstack-lbaas17:07
*** ihrachys has joined #openstack-lbaas17:12
*** yamamoto has quit IRC17:12
*** yamamoto has joined #openstack-lbaas17:16
*** links has joined #openstack-lbaas17:18
*** ihrachys has quit IRC17:19
*** eezhova has quit IRC17:24
*** bana_k has joined #openstack-lbaas17:26
*** cody-somerville has joined #openstack-lbaas17:29
*** cody-somerville has quit IRC17:29
*** cody-somerville has joined #openstack-lbaas17:29
johnsomdougwig ping17:30
dougwigjohnsom: ack17:31
johnsomCan you look at Andreas comment here: https://review.openstack.org/#/c/386245/  Am I missing something when it comes to stable branches?  I'm leaving the old targets17:32
njohnstonjohnsom: I think he is saying that https://review.openstack.org/386240 sets up the consolidated API test for master.  But since there is no version constraint on the new test, it will run against master + stable/newton + stable/mitaka etc.  So you may need to use version constraints so that versions without https://review.openstack.org/386240 have working tests.17:37
johnsomHmm, ok, I will look into that17:38
johnsomThanks!17:38
njohnstonjohnsom: By way of example, I had to do something like that with the fwaas tempest tests, so I defined a 'legacy' job for old branches, and my new 'v1' and 'v2' test sets would run against everything else.  https://github.com/openstack-infra/project-config/blob/master/zuul/layout.yaml#L2214-L222117:39
johnsomnjohnston Thanks!17:39
njohnstonjohnsom: Any time!17:40
*** pcaruana has joined #openstack-lbaas17:48
*** anilvenkata_afk has quit IRC17:55
*** portdirect has quit IRC18:02
*** cody-somerville has quit IRC18:02
*** cody-somerville has joined #openstack-lbaas18:03
*** cody-somerville has quit IRC18:03
*** cody-somerville has joined #openstack-lbaas18:03
*** portdirect has joined #openstack-lbaas18:03
*** eezhova has joined #openstack-lbaas18:15
dougwigjohnsom: do the consolidated jobs just timeout on non-nested-virt clouds?18:27
johnsomI am consolidating the API test jobs which use the noop drivers18:27
johnsomThe scenario is the that spins up amps.   Currently our scenario test times out and fails18:28
*** ducttape_ has quit IRC18:31
openstackgerritAdam Harwell proposed openstack/octavia: Implement sorting and pagination for octavia  https://review.openstack.org/38214718:33
johnsomdougwig This would be an example of that: http://logs.openstack.org/40/386240/2/check/gate-neutron-lbaasv2-dsvm-scenario/b134378/18:34
johnsomUgh, in other scenario test issues, I'm not sure what to do about this: http://logs.openstack.org/40/386240/2/check/gate-neutron-lbaasv2-dsvm-scenario/cc6f667/logs/screen-o-cw.txt.gz#_2016-10-14_17_40_30_91918:36
johnsomI added a "give it one more try" but that seems to have not worked.18:36
johnsomNot happy with flask/werkz18:36
johnsomI can't keep retrying on 404's as that is a valid response in some cases.18:37
*** ihrachys has joined #openstack-lbaas18:37
*** ihrachys has quit IRC18:37
*** ihrachys has joined #openstack-lbaas18:38
*** ihrachys has quit IRC18:39
openstackgerritAdam Harwell proposed openstack/octavia: Add quota support to Octavia  https://review.openstack.org/36079418:40
rm_workyeah we should convert is to use gunicorn18:40
rm_workit isn't hard T_T18:40
rm_work*convert it18:40
rm_workthat would solve multiple problems18:40
*** links has quit IRC18:41
*** derekjhyang_ has joined #openstack-lbaas18:43
*** johnsom_ has joined #openstack-lbaas18:44
*** vmtyler_ has joined #openstack-lbaas18:45
*** Aju has joined #openstack-lbaas18:49
*** derekjhyang has quit IRC18:51
*** vmtyler has quit IRC18:51
*** johnsom has quit IRC18:51
*** afranc has quit IRC18:51
*** tries_ has quit IRC18:51
*** derekjhyang_ is now known as derekjhyang18:52
*** ducttape_ has joined #openstack-lbaas18:53
*** johnsom_ is now known as johnsom18:53
*** vmtyler_ is now known as vmtyler18:53
*** tries_ has joined #openstack-lbaas18:57
*** eezhova has quit IRC19:04
*** cody-somerville has quit IRC19:08
*** cody-somerville has joined #openstack-lbaas19:08
*** cody-somerville has quit IRC19:08
*** cody-somerville has joined #openstack-lbaas19:08
*** dlundquist has quit IRC19:09
*** dlundquist has joined #openstack-lbaas19:11
rm_workjohnsom: looking to see how tedious it will actually be to shove gunicorn in there19:11
rm_workit will kill the guru stuff, i think... not sure what else it'll interfere with19:11
rm_workthe interesting part may actually be reading the config values19:11
rm_workbecause it'll have to be done outside of python19:11
rm_workor ... hmm19:12
*** cody-somerville has quit IRC19:18
*** cody-somerville has joined #openstack-lbaas19:25
*** cody-somerville has quit IRC19:25
*** cody-somerville has joined #openstack-lbaas19:25
rm_workyeah ok I think this is actually pretty easy19:27
rm_workalmost done maybe?19:27
*** cody-somerville has quit IRC19:32
rm_worktook longer to get devstack up and an amphora created to test this, than it did to write it, i think19:33
rm_workbut, i should wait until I know it is working :P19:33
*** cody-somerville has joined #openstack-lbaas19:34
*** cody-somerville has quit IRC19:34
*** cody-somerville has joined #openstack-lbaas19:34
rm_workseems to work?19:51
openstackgerritAdam Harwell proposed openstack/octavia: Run amphora agent with gunicorn  https://review.openstack.org/38675819:51
rm_workjohnsom: ^^19:51
johnsomHmm cool!  I at lunch, but will take a look later19:53
rm_workOK19:53
rm_workIt doesn't do auto-reloading (though I could enable it?) but I think we explicitly reload the agent on changes, don't we?19:53
rm_workit's easy enough to add the reload option19:54
rm_work'reload': True,19:54
rm_workjohnsom: anywho... leave comments, probably it isn't perfect, but should be easy enough to make a few tweaks. i'm heading to bed19:55
rm_workbasically i am hoping it should pass the scenarios19:57
*** Aju has quit IRC20:05
rm_work... i lied about going to bed, i'm watching the test runs20:06
*** eezhova has joined #openstack-lbaas20:17
*** eezhova has quit IRC20:17
*** pcaruana has quit IRC20:18
rm_worklooking good20:20
rm_workif there's regressions, it should be on incidentals, like cert replacement reloading and such20:21
rm_workthe server itself seems to work fine20:21
rm_workhow were you able to reproduce issues with werkzeug?20:21
rm_workif you have a CR that causes issues to surface, can you rebase it on mine and see what happens?20:22
*** strigazi is now known as strigazi_AFK20:22
*** _ducttape_ has joined #openstack-lbaas20:22
diltramlistener, loadbalancer and minimal passed20:22
rm_workyeah pool is about to pass, watching it20:23
*** ducttape_ has quit IRC20:23
rm_workscenario is still setting up devstack >_>20:23
rm_worki think the ovh host is shitty20:23
rm_workwait, i think those all use noop >_>20:24
rm_workerg20:24
rm_workprolly need to wait for scenario20:25
rm_worklol scenario finally starting to actually run tests20:26
rm_worktook 35 minutes for devstack to install on that node T_T20:27
diltramhahahaha20:27
rm_worki'm not sure if this will even be definitive, might time out waiting for the amphora to come up, but because the host is bad, not because the change didn't work >_<20:29
rm_workdo me a favor and recheck it if it fails? I'm going to go to bed, can't wait another 30m for this to actually spin an amp >_<20:30
diltramyeah, I will be here to take a look :P20:31
rm_workah, just as i was about to lock my screen, first test passed20:32
rm_workso obviously it started the agent properly20:33
rm_workis johnsom back from lunch yet? :P20:34
johnsomYep, just catching up20:34
rm_workthe way I implemented gunicorn is the least disruptive to the current workflow20:35
diltramlooks working :P20:35
rm_workwe could go as deep as switching out internals of the config and upstart init to make it just run gunicorn directly20:36
rm_workbut that's a lot more work20:36
johnsomI know Brandon proposed gunicorn before20:36
rm_workyeah20:36
rm_worki did too20:36
rm_worki just never had time to do it20:36
rm_workand it looks like we waited the appropriate amount of time, because 19.0 added the ability to do what i'm doing20:36
johnsomYeah, I think we have fought with issues on the old platform long enough20:38
diltramsecond is ok20:38
diltramso it's working20:38
rm_workyep20:40
rm_workjust responded to dougwig on the CR20:40
rm_workbasically, re: global-reqs, IDGAF20:40
rm_workdoes anyone else care? :/20:42
openstackgerritStephen Balukoff proposed openstack/octavia: Load system sysctl in amphora network namespace  https://review.openstack.org/38632820:43
diltramrm_work: you're right in my opinion20:44
diltramso if go is not approved as programing language we can't use this new http go smth?20:44
openstackgerritStephen Balukoff proposed openstack/octavia: Update haproxy logging configuration  https://review.openstack.org/38642720:50
diltrampassed20:50
dougwigdiltram: that merged.  water under the bridge.20:55
diltramI don't understad what do you mean?20:55
dougwigrm_work: replied replied.20:56
dougwigdiltram: the test http server is merged; it's all good.20:56
rm_workdougwig: besides it being in g-r.txt or not, I still don't want to add gunicorn to our requirements.txt because it isn't needed for octavia to run, it is ONLY for the amphorae20:56
dougwigrm_work: right, see my reply20:56
rm_worki was about to say "i could maybe be convinced to add an amphorae-requirements.txt"20:56
diltramdougwig: +120:57
dougwigi will assume the "maybe" in that sentence is due to too much sake.20:57
rm_workthe issue with this is that now the packages that ARE in global-reqs aren't going to be tracked20:59
rm_workre: versions and such20:59
rm_work(I mean, the bot won't update them anymore)21:00
sbalukoffHmmm....21:00
rm_workso, see my new patchset, but i don't like it a whole lot21:00
rm_workerr, momentarily21:00
diltrambut even adding anything into dib elements will not be tracked by bot21:01
sbalukoffWhat's the license on gunicorn?21:01
rm_workyeah21:01
johnsomOk, just getting back into the conversation after a phone call.  So, gunicorn is not openstack approved?21:01
rm_workso, no difference?21:01
rm_workjohnsom: it's not in g-r but #infra seems to think there wouldn't be a fight to get it there21:01
dougwigbut we don't want it in requirements.txt if it's not required for the server to run, i'd think21:02
rm_workright21:02
rm_workso even if it were in g-r i wouldn't want to add it to requirements21:02
johnsomWhere did we put pyroute2?21:02
rm_workrequirements.txt21:03
rm_workwhich sucks21:03
rm_workbut i moved it21:03
rm_workurg connection issues21:03
rm_workok should post shortly21:03
openstackgerritAdam Harwell proposed openstack/octavia: Run amphora agent with gunicorn  https://review.openstack.org/38675821:03
johnsomYeah, so, it should be there.  We use that when we build the image I think21:03
rm_work^^ see that21:03
*** cody-somerville has quit IRC21:03
*** cody-somerville has joined #openstack-lbaas21:04
*** cody-somerville has quit IRC21:04
*** cody-somerville has joined #openstack-lbaas21:04
rm_workbut yeah, now none of those things are tracked by the openstack bot21:04
johnsomYeah, so why move it out?21:04
rm_workI've never been a fan of installing that stuff in requirements.txt21:05
rm_workbut if we're going to have an amphora-requirements.txt then that's where it should live21:05
johnsomFair, but that argues for moving the agent into it's own project21:05
dougwigi'd bet we could modify the bot to check for additional requirement files, if that's the only holdup21:05
rm_workhonestly i liked how patchset one went21:05
rm_workdougwig: that'd be interesting, worth investigating21:05
dougwigor a separate repo, as johnsom suggests.21:05
johnsomdougwig +121:06
dougwigrm_work: your first patchset was a disgusting hack21:06
rm_workyeah, and that's been long in the works21:06
rm_worki actually tried splitting it once but it was going to be a lot more work than i had time for21:06
johnsomHahaha, we are going to end up with octavia-lib21:06
*** matt-borland has quit IRC21:09
sbalukoffHeh! Probably.21:10
sbalukoffAnd whatever pattern we follow for the amphora image is going to follow for the distributor image. :P21:11
diltrambut even splitting amphora agent into new repo will be good21:11
rm_workyes but not in scope for this >_<21:11
sbalukoffTrue.21:11
sbalukoffdiltram: That's where we end up getting octavia-lib.21:11
diltramAnyone knows what part of code is required by base octavia and amphora agent?21:12
diltramas a shared code?21:12
*** cody-somerville has quit IRC21:13
sbalukoffNot off the top of my head, but I would guess many things from octavia/common.21:13
johnsomNot off their head, probably not.21:13
johnsomoctavia/amphorae/backends down would be a start21:13
rm_worki actually do21:13
rm_workmost of it21:13
rm_workit is a lot21:13
johnsomBut, I think we need to step back from that edge.21:13
rm_worki have an abandoned CR for it...21:13
rm_workthe rabbit-hole was deeeep21:14
johnsomI'm not sure now is the right time to try to do that split.21:14
rm_workIMO it is not21:14
johnsomI still don't see the advantage of the amphora-requirements if you are still installing them all in the amp.  I commented as such on the patch.21:16
rm_work?21:17
rm_workamphora-requirements.txt is for things that are ONLY required in the amphora21:17
rm_workwe absolutely need cryptography, but it's also needed by the main app21:17
johnsomSo we are saving a few packages from the controllers?21:18
rm_workyes :/21:18
rm_workmaybe this is not worth the optimization?21:18
rm_worksince the amp is really where optimizing matters, and we aren't21:18
johnsomIt seems like saving space in the amp is more important21:18
rm_workwell, as we're talking, i'm TRYING to propose gunicorn to g-r21:19
rm_workbut having environment issues21:19
dougwigwith the ssh driver, none.  :)21:21
rm_work^^ :P21:22
rm_workall this nonsense goes away21:22
rm_workah, maybe now IS a good time to resurrect that21:22
dougwigsince amps are meant to be black boxes, and we've intermingled this much already, that is arguing really well that we should separate those repos.  IMO.21:22
* johnsom glares at you people21:22
rm_workand do it the way i wanted to do it to begin with..21:22
* dougwig apologizes for the ssh troll, even if it is superior.21:22
rm_workaww, thought you were serious :( still really want the ssh driver back21:23
rm_workthough, we still need a process of some sort to do heartbeats21:23
diltramin my opinion if Adam has time and don't want to work on something more urgent he has a green light to make that split21:23
dougwigi want it back too, but the team has decided, so let it lie.21:23
rm_work:/21:25
johnsomdiltram my concern is it would be a distraction for reviewing, splitting in-flight patches (act/act), ...  I of course can't/will not stop him, but it has some downside to do it now.21:25
diltramtrue21:25
johnsomPlus, they are arguing for a new amp driver over SSH, which didn't really work the first time (yes, my opinion)21:26
diltramso like I said - if he don't want to help with merge :P21:26
rm_workFYI, I in no way want to deal with that split right now :P21:26
rm_work"new" lol21:26
rm_workI guess technically21:26
diltrambut what is value added of ssh driver?21:27
dougwigi mean, this entire argument is because we have two projects in one.  separate repos, or separate requirements files, or ... we end up with hacks in the bowels of scripts or just LCD.  pick your poison.21:27
johnsomHey, I said "my opinion"...  grin21:27
rm_workugh g-r is not working here21:27
rm_workbecause i don't have *system-level* python321:27
dougwigdiltram: the "image" becomes native ubuntu or cirros, and no code to update on the amp, ever.21:27
rm_workdougwig: so, technically not true unfortunately, unless we deploy and run a heartbeat/stats-sender via sftp21:28
diltramok - understand21:28
* johnsom thinks dougwig started his weekend early...21:28
rm_workcan someone else do this proposal? T_T21:28
rm_workglobal-requirements proposals require this constraints hackery and i can't get their stuff to run right on my box21:29
rm_workit REALLY wants system-level python2 and python3 and i don't have both (I use py3 via pyenv)21:29
dougwigrm_work: i can.  what's the g-r line, including license?21:31
*** portdirect_ has joined #openstack-lbaas21:31
*** portdirect has quit IRC21:31
*** portdirect_ is now known as portdirect21:31
rm_workgunicorn>=19.0  # MIT21:32
dougwigrm_work: https://review.openstack.org/38679021:37
dougwigwhy are the two of us, of all people, doing work to maintain the crappy rest agent?21:38
dougwigsorry, that slipped out21:38
openstackgerritAdam Harwell proposed openstack/octavia: Run amphora agent with gunicorn  https://review.openstack.org/38675821:38
* johnsom glares again21:38
rm_workdougwig: should I depends-on: I211dc771aa95147c0f1d9e6ac1a65a7e164b33c221:38
rm_work?21:38
johnsomYes21:39
openstackgerritAdam Harwell proposed openstack/octavia: Run amphora agent with gunicorn  https://review.openstack.org/38675821:40
rm_workdone21:40
rm_workhappy now? you monsters21:40
johnsomHa21:40
johnsomDo you want to expand on the text of Doug's patch?  It's, um, basic21:41
rm_workyeah uhhh21:41
rm_workwas thinking about it21:41
dougwigwait and see what happens.  more words is just more things to argue about.21:41
johnsomHa, true21:42
johnsomI hope this fixes all the strangeness we have had with ports getting lost, requests not getting responded to, or partial responses21:43
rm_workwhelp, already responded21:43
dougwigrm_work: but if you change it, do add a co-author.  forgot that.21:43
johnsomIf it does I will buy rm_work a tapas21:43
rm_workheh21:43
rm_workyeah, uhh... so, how's it going, people with corp cards? :P21:44
rm_workwho wants to get some dinner? ^_^21:44
rm_worki tried to keep my response pretty short but to the point21:45
rm_workwhat channel do g-r changes go to?21:45
dougwiginfra?21:47
rm_workI thought so but i don't see it there21:47
rm_workah well, now that it's 7am i should head to bed >_>21:47
sbalukoffSleep well, man.21:50
openstackgerritAdam Harwell proposed openstack/octavia: Add quota support to Octavia  https://review.openstack.org/36079421:54
rm_work^^ missed updating the migration downrevision >_>21:54
rm_workoh hey diltram it looks like it is ACTUALLY testing quotas now? :P22:00
rm_worksee failures http://sonar.octavia.io/archive/octavia/1c992690482cac6ae4ac18d555f2a23e13358e68/2394/console_log.txt.gz22:00
rm_workas you were saying, the test was using the wrong default22:00
rm_worki think that was tyou22:00
openstackgerritAdam Harwell proposed openstack/octavia: Add quota support to Octavia  https://review.openstack.org/36079422:03
diltramhere22:03
diltramhttps://review.openstack.org/#/c/360794/33/etc/octavia.conf22:03
diltramdefault quota is 2022:03
rm_workah for just the one22:03
rm_workok need to update again22:03
openstackgerritAdam Harwell proposed openstack/octavia: Add quota support to Octavia  https://review.openstack.org/36079422:04
rm_workthere we go22:04
rm_workcorrected22:05
rm_workand NOW sleep maybe T_T22:05
johnsomUgh, this is going to be a fun weekend.  The power is already blipping on and off22:10
dougwigjohnsom: got your generator and plenty of beer?22:38
johnsomBatteries, natural gas, and yes, beer22:39
johnsomwine, bourbon, etc.22:39
dougwigthen it's just like camping.  with more water and looting.22:39
johnsomJust the basics22:39
*** _ducttape_ has quit IRC23:05
openstackgerritMerged openstack/octavia: Load system sysctl in amphora network namespace  https://review.openstack.org/38632823:10
openstackgerritDustin Lundquist proposed openstack/octavia: Scenario: test with concurrent clients  https://review.openstack.org/38564023:20
*** afranc has joined #openstack-lbaas23:28
*** gongysh has joined #openstack-lbaas23:37
*** ducttape_ has joined #openstack-lbaas23:38
openstackgerritDustin Lundquist proposed openstack/octavia: Scenario: test with concurrent clients  https://review.openstack.org/38564023:41
*** bana_k has quit IRC23:59

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