Tuesday, 2018-03-13

« Monday, 2018-03-12 Index Wednesday, 2018-03-14 »
*** fnaval has quit IRC00:13
*** Swami has quit IRC00:45
*** JudeC has joined #openstack-lbaas00:49
openstackgerritMichael Johnson proposed openstack/octavia master: Reduce stestr concurrency to 1 for tempest jobs  https://review.openstack.org/55180100:50
*** harlowja has quit IRC01:10
openstackgerritOpenStack Proposal Bot proposed openstack/neutron-lbaas master: Updated from global requirements  https://review.openstack.org/55153301:14
openstackgerritOpenStack Proposal Bot proposed openstack/octavia master: Updated from global requirements  https://review.openstack.org/55219301:17
openstackgerritMerged openstack/octavia master: Rename q- to neutron- services  https://review.openstack.org/54428101:21
*** JudeC has quit IRC01:24
*** rcernin has quit IRC02:04
*** jaff_cheng has joined #openstack-lbaas02:04
*** annp has joined #openstack-lbaas02:13
*** rcernin has joined #openstack-lbaas03:07
*** jaff_cheng has quit IRC03:23
*** jaff_cheng has joined #openstack-lbaas03:24
*** harlowja has joined #openstack-lbaas03:48
*** links has joined #openstack-lbaas03:50
openstackgerritwangqi proposed openstack/octavia-tempest-plugin master: Switch to tempest.common.utils.services  https://review.openstack.org/55228704:58
*** jaff_cheng has quit IRC05:30
*** jaff_cheng has joined #openstack-lbaas05:30
*** gcheresh has joined #openstack-lbaas05:34
*** irenab has quit IRC05:36
*** oanson has quit IRC05:37
*** gcheresh has quit IRC05:41
*** harlowja has quit IRC05:59
*** oanson has joined #openstack-lbaas06:38
*** irenab has joined #openstack-lbaas06:38
*** oanson has quit IRC06:38
*** irenab has quit IRC06:38
*** kobis has joined #openstack-lbaas06:39
*** oanson has joined #openstack-lbaas06:39
*** irenab has joined #openstack-lbaas06:40
*** JudeC has joined #openstack-lbaas06:57
*** rcernin has quit IRC07:24
openstackgerritOpenStack Proposal Bot proposed openstack/python-octaviaclient master: Updated from global requirements  https://review.openstack.org/54957007:27
*** JudeC has quit IRC07:29
openstackgerritOpenStack Proposal Bot proposed openstack/octavia-dashboard master: Imported Translations from Zanata  https://review.openstack.org/55037107:30
*** pcaruana has joined #openstack-lbaas07:42
*** AlexeyAbashkin has joined #openstack-lbaas07:50
*** links has quit IRC07:52
*** links has joined #openstack-lbaas07:59
*** tesseract has joined #openstack-lbaas08:19
*** tesseract-RH has joined #openstack-lbaas08:20
*** tesseract-RH has quit IRC08:20
*** sticker has joined #openstack-lbaas08:34
*** mburrows has quit IRC09:30
*** dmellado has joined #openstack-lbaas09:47
*** salmankhan has joined #openstack-lbaas10:12
*** jaff_cheng has quit IRC10:38
*** pcaruana has quit IRC10:45
*** rcernin has joined #openstack-lbaas11:02
*** yamamoto has quit IRC11:46
*** yamamoto has joined #openstack-lbaas11:50
*** rcernin has quit IRC12:00
*** yamamoto has quit IRC12:01
*** yamamoto has joined #openstack-lbaas12:01
openstackgerritDaniel Mellado proposed openstack/neutron-lbaas master: Adapt plugin to new q- n- syntax  https://review.openstack.org/55249312:03
*** yamamoto has quit IRC12:06
*** annp has quit IRC12:07
rm_work[02:57:22]  <rm_work>though i bet 99% what is going to happen is that this whole test will just get dropped12:08
rm_work[04:37:31]  <cgoncalves>so I propose to keep it open until the neutron patch gets merged, and then we can abandon this one12:09
rm_work^^ exactly what I meant :P12:09
rm_workwhich is probably fine12:09
cgoncalves:)12:11
*** salmankhan has quit IRC12:12
rm_workso on https://review.openstack.org/#/c/551796/4/tox.ini12:24
rm_workwas reading scrollback here, and wasn't johnsom saying parallel mode was wrong there?12:25
rm_workah but we're not going to try to do specifically named files so it doesn't matter?12:25
*** salmankhan has joined #openstack-lbaas12:27
*** atoth has joined #openstack-lbaas12:29
rm_workwait, why aren't the coverage tests running?12:29
rm_workcgoncalves: i don't see the coverage test running in that patch? do we not have a gate for it? >_>12:32
cgoncalvesrm_work: heh, I thought we had12:50
cgoncalvesrm_work: want to propose it or shall I?12:51
rm_workyou've been doing gate patches recently, you go for it12:53
rm_worki'm working through some logic12:53
rm_worktrying to decide how to fix this barbican ACL dumbness12:54
*** fnaval has joined #openstack-lbaas12:54
*** HW-Peter has joined #openstack-lbaas13:08
*** HW-Peter has quit IRC13:10
*** HW-Peter has joined #openstack-lbaas13:11
*** yamamoto has joined #openstack-lbaas13:34
johnsomrm_work: doing the API add the ACL thing?13:38
rm_workyes13:38
johnsomSweet13:39
*** yamamoto has quit IRC13:39
rm_workwell after i finish the patch that fixes our API usage of barbican :/13:39
rm_workwe forgot to actually... do an API check for the cert13:39
rm_worklol13:39
rm_workwe did it on the n-lbaas side but never on the octavia api side13:39
rm_workthat patch is pending but updating it to do more correct checks, one sec13:39
johnsomOk, I am not “in the office yet”, just picked up my phone for a bit.13:40
rm_workk13:40
*** yamamoto has joined #openstack-lbaas13:53
*** yamamoto has quit IRC13:55
*** kobis has quit IRC14:01
openstackgerritMerged openstack/neutron-lbaas master: Reduce stestr concurrency to 1 for tempest jobs  https://review.openstack.org/55179614:01
rm_workthere we go14:04
openstackgerritAdam Harwell proposed openstack/octavia master: Properly test access to tls_refs in the API layer  https://review.openstack.org/55030314:04
rm_worknow doing it *right* i think14:04
rm_workso now it's possible to do the ACLs but it's not ... explicitly clear to me whether the session we have is the user token one14:05
openstackgerritHengqing Hu proposed openstack/octavia-dashboard master: Add l7 support  https://review.openstack.org/55194714:07
rm_workor rather, context14:11
openstackgerritDaniel Mellado proposed openstack/neutron-lbaas master: Add ml2_conf.ini default value in settings  https://review.openstack.org/55252714:14
*** kobis has joined #openstack-lbaas14:22
rm_workhmmm i think there may be too many gotchas with this...14:49
rm_worki can kinda do it, but there's a lot of places where it kinda falls apart14:49
*** kobis has quit IRC14:50
*** kobis has joined #openstack-lbaas14:53
johnsomrm_work need a second set of eyes?15:03
rm_workgonna push up some code without any testing that's kinda ... a first go at mapping it out15:03
rm_workin a moment15:03
johnsomOk15:04
*** kobis has quit IRC15:08
openstackgerritAdam Harwell proposed openstack/octavia master: [WIP] Automatically set Barbican ACLs  https://review.openstack.org/55254915:17
*** sticker has quit IRC15:17
rm_workit's messy for a few reasons15:17
rm_workjohnsom: second set of eyes is good, but you may have more important things to focus on still, this soon back from your trip :)15:18
johnsomYeah, I have a few internal things to do today, but I will find a minute or two to take a look15:18
johnsomFYI: Another DVR bug that can impact us: https://bugs.launchpad.net/neutron/+bug/175343415:21
openstackLaunchpad bug 1753434 in neutron "Unbound ports floating ip not working with address scopes in DVR HA " [Undecided,Confirmed]15:21
rm_work<_<15:22
*** yamamoto has joined #openstack-lbaas15:28
*** beagles is now known as beagles_food15:36
*** yamamoto has quit IRC15:38
*** JudeC has joined #openstack-lbaas15:43
*** kobis has joined #openstack-lbaas15:45
*** harlowja has joined #openstack-lbaas15:46
*** JudeC has quit IRC15:50
*** links has quit IRC15:55
*** harlowja has quit IRC16:06
johnsomrm_work Gave you a few comments16:06
rm_workk16:07
rm_workhammering out backup members really quick16:07
johnsomlol16:07
*** links has joined #openstack-lbaas16:08
*** beagles_food is now known as beagles16:16
*** kobis has quit IRC16:22
*** kobis has joined #openstack-lbaas16:23
*** links has quit IRC16:30
*** gcheresh has joined #openstack-lbaas16:38
*** kobis has quit IRC16:40
*** gcheresh has quit IRC16:45
*** JudeC has joined #openstack-lbaas16:49
andreykurilinrm_work: hi! I know that openstack/neutron-lbaas is deprecated, but do you know anythin about plans to make it compatible with a new devstack neutron naming?)16:54
rm_workyes, we actually have mostly finished that work16:55
rm_workI think it merged in the past couple of days16:55
rm_workthere's one more patch possibly outstanding there16:55
andreykurilincool, thanks16:55
andreykurilinPS: we had legacy job with a bunch of neutron extensions - networking-bgpvpn, neutron-fwaas, neutron-lbaas, neutron-vpnaas, octavia (it worked together!). But as for new naming only octavia and vpnaas work16:56
*** kobis has joined #openstack-lbaas16:59
rm_workyes, because octavia is not tightly coupled with neutron, we are an API consumer only17:00
rm_workbut neutron-lbaas is ... a little wonky17:01
*** kobis has quit IRC17:01
*** Swami has joined #openstack-lbaas17:06
openstackgerritAdam Harwell proposed openstack/octavia master: Allow members to be set as "backup"  https://review.openstack.org/55263217:09
rm_workthere17:10
rm_workdeciding whether i go to sorry-page or not17:20
*** kobis has joined #openstack-lbaas17:32
*** AlexeyAbashkin has quit IRC17:34
*** imacdonn has quit IRC17:44
*** imacdonn has joined #openstack-lbaas17:44
openstackgerritAdam Harwell proposed openstack/octavia master: Devstack plugin image build option quoting fix  https://review.openstack.org/55264117:51
rm_workjohnsom: ^^ that one is annoying17:53
rm_workkeeps making my stacks fail >_<17:54
johnsomI'm looking at this docs thing now17:54
*** kobis has quit IRC18:05
*** kobis has joined #openstack-lbaas18:06
*** kobis has quit IRC18:06
*** kobis has joined #openstack-lbaas18:06
*** kobis has quit IRC18:07
*** kobis has joined #openstack-lbaas18:07
*** kobis has quit IRC18:08
*** salmankhan has quit IRC18:08
*** kobis has joined #openstack-lbaas18:08
*** kobis has quit IRC18:08
*** kobis has joined #openstack-lbaas18:09
*** kobis has quit IRC18:09
rm_workthe sphinx change?18:14
rm_workerr i thought we could just ignore the requirements update until we fixed it as our requirements specifically have a "less than" that should block it18:16
rm_workbut it seems like it's affecting everything anyway somehow? <_<18:16
rm_workalso, wow gate queue is long O_o18:17
johnsomhttps://www.irccloud.com/pastebin/rf4FJuQB/18:27
*** harlowja has joined #openstack-lbaas18:30
*** harlowja has quit IRC18:30
johnsomYeah, so it looks like shpinx > 1.6.5 is broken18:46
*** AlexeyAbashkin has joined #openstack-lbaas18:58
johnsomSo, this is back to the dumb thing in sphinx that it doesn't index folders with empty __init__.py and only sub-folders19:01
*** AlexeyAbashkin has quit IRC19:03
*** tesseract has quit IRC19:11
rm_work<_<19:16
*** igageis has joined #openstack-lbaas19:35
*** igageis has quit IRC19:37
rm_workjohnsom: wait a minute ... so the context we have in the API layer... is that actually a user context?19:38
rm_workso is our original cert lookup ALREADY using the user's token?19:38
johnsomBut the code that pulls the cert to install on the amp doesn't have the user context, it is async19:41
rm_workright19:44
rm_workbut i knew that19:44
rm_worki just thought the API side context was still "ours"19:44
rm_workin that if I used it to perform actions it would be our actions, not the users19:44
rm_workthat changes things a lot19:45
johnsomSo, the docs issue is because the upper-constraint got bumped. It's not related to the G-R change. Everything is going to break. I have restored Jacky's patch: https://review.openstack.org/#/c/53476519:51
rm_workyeah sad19:58
rm_workah this is all different sessions anyway, nm, maybe doesn't change19:59
rm_workso i think basically it's even more ridiculous:20:08
openstackgerritAdam Harwell proposed openstack/octavia master: [WIP] Automatically set Barbican ACLs  https://review.openstack.org/55254920:08
johnsomrm_work Still not sure I get this.  Isn't get_cert only called from the backend?20:14
johnsomo-cw?20:14
rm_worksee the patch it depends on20:15
rm_workyou missed my explanation of that bug earlier i think20:15
rm_workit was like the first thing i mentioned when you came back20:15
johnsomAh, got it, yeah missed the other patch20:16
rm_workwe (I) totally forgot to put the check in the API layer20:16
rm_workbecause... i dunno, dumb20:16
rm_workneutron-lbaas did it properly20:16
rm_workwe did not20:16
rm_workpossibly because at the time it was API v1 and i didn't want to have it pull from barbican FOUR TIMES or something silly just to get the certs deployed20:17
johnsomWe had jsut a little bit going on at the time....20:17
rm_workyeah20:17
johnsomHmm, I thought we fixed all of the py35 issues with the certs...20:22
rm_workthis is one i added20:23
rm_worki'm working on i20:23
rm_work*it20:23
rm_workof course at the last minute i decided to add one more test and neglected to also run py3520:24
rm_workwas trying to improve coverage <_<20:25
rm_worki see why no one bothered to cover this line20:25
rm_worki think it's not worth it >_<20:25
*** harlowja has joined #openstack-lbaas20:37
rm_workwhich is better:20:48
rm_workisinstance(passphrase, six.text_type)20:48
rm_worktype(passphrase) == six.text_typetype(passphrase) == six.text_type20:48
rm_workerr20:48
rm_worktype(passphrase) == six.text_type20:48
rm_workisinstance(passphrase, six.text_type)20:48
rm_workthose two20:48
rm_worki am unsure20:48
*** mburrows has joined #openstack-lbaas20:52
openstackgerritAdam Harwell proposed openstack/octavia master: Properly test access to tls_refs in the API layer  https://review.openstack.org/55030321:08
rm_workk fixed21:08
openstackgerritAdam Harwell proposed openstack/octavia master: [WIP] Automatically set Barbican ACLs  https://review.openstack.org/55254921:09
rm_workwhat i would give just to have our gates work for a week21:11
rm_workall it took was three days of broken to make me throw my hands up and just start merging stuff that seemed to make it better <_<21:12
rm_workjohnsom: that patch still has a pep8 error (well, lots of them)21:17
rm_workneed to disable H10421:17
rm_workummm and somehow a bunch of other errors, wtf21:18
johnsomHmm, yeah, the logs were gone so I wasn't sure what was up with that.21:22
rm_workyeah wtf these are valid21:22
rm_workthis is so weird21:22
rm_workit's like flake8 maybe also wasn't checking stuff correctly sometimes?21:22
rm_workbut this fixes i??21:22
rm_workdunno21:22
rm_workfixing all this suff21:22
rm_work*stuff21:22
johnsomWere we missing __init__.py files?21:23
johnsomNo, those are all updates21:23
rm_workit's weird, they're all over the place21:26
rm_workit's mostly E126 and E12321:26
rm_workso, indentation errrs21:26
rm_workuhg i would be done by now but my macbook is lagging to all hell for some reason21:27
rm_workhmmm docs test is also still failing for me though with the toctree thing i saw elsewhere21:29
rm_workoh nm it's different21:29
johnsomThat is what we are fixing with the __init__ patch21:29
rm_workoctavia/amphorae/backends/agent/agent_jinja_cfg.py:docstring of octavia.amphorae.backends.agent.agent_jinja_cfg.AgentJinjaTemplater:0:duplicate object description of octavia.amphorae.backends.agent.agent_jinja_cfg.AgentJinjaTemplater, other instance in octavia/doc/source/contributor/modules/octavia.amphorae.backends.agent.rst, use :noindex: for one of them21:29
rm_workthis is breaking ON the init patch21:30
rm_worklooking into it now21:31
*** mburrows has quit IRC21:31
*** mburrows has joined #openstack-lbaas21:31
rm_workis this breaking just because it's the first thing alphabetically to be checked21:32
rm_workand they will all break?21:32
rm_workregardless, pushing up the fixes21:32
rm_workor21:32
rm_worki guess i can wait for it to finish21:32
rm_workhmmm the sphinx docs build works tho in gate so whatever21:35
rm_workmust be me21:35
*** AlexeyAbashkin has joined #openstack-lbaas21:36
rm_workugh, johnsom can you copy off links to the runs you want from that patch21:37
rm_workso i can push up the fix for the pep8 stuff21:37
rm_workbecause it's got like an hour left i think21:38
johnsomGo ahead.21:38
johnsomThe docs job passed21:38
rm_workyeah k21:38
openstackgerritAdam Harwell proposed openstack/octavia master: Add license for empty __init__.py  https://review.openstack.org/53476521:39
rm_workmy schedule is moved back a bit now, so mostly i'm working early early morning21:40
*** AlexeyAbashkin has quit IRC21:41
johnsomHmm, we can override the checks in those files, but I'm not sure H104 really benefits us anyway.21:44
rm_workit does not, IMO21:52
rm_worki'm still perplexed by how those bad indentations ever got past flake821:53
rm_worki was tempted to assume something had gone wrong and they were false positives locally that i should ignore... but... I can SEE that they were actually wrong21:54
rm_workah i think the docs fail locally was because of how it built the modules (in the source dir, wtf) and that i couldn't tell because it's in gitignore ... but nothing cleans it up21:55
rm_workwiping them and re-running21:55
rm_workyep <_< lame21:58
rm_workwiping that helped21:58
rm_workmaybe should put that into the docs run21:58
*** rcernin has joined #openstack-lbaas21:59
johnsomremoving the modules path?22:02
rm_workjohnsom: i'm gonna add that22:02
rm_workdoc/source/contributor/modules22:02
johnsomI thought about it too22:02
johnsomyeah22:02
rm_workto the existing rm -rf command in the docs22:02
rm_workk22:02
rm_workit was still queued anyway22:03
rm_work... man, what is up today22:03
rm_worksuch queues22:03
openstackgerritAdam Harwell proposed openstack/octavia master: Add license for empty __init__.py  https://review.openstack.org/53476522:03
rm_workhours of queues22:03
rm_worki need to stop making new patches for a bit, i'm juggling like 6 outstanding now i think <_<22:05
rm_workonce the gates are better i will make an actual clear list of them, prioritized22:07
rm_workdo we have a review priority etherpad or something? I liked the one we had before22:07
johnsomNo, I don't have one going right now22:08
nmagnezi_rm_work, o/ re: https://review.openstack.org/#/c/548729 , I'm still not convinced Oracle should skip my suggestions.. including dib elements :P22:09
* nmagnezi_ goes back to OOO22:09
*** fnaval has quit IRC22:27
openstackgerritMerged openstack/neutron-lbaas-dashboard master: Imported Translations from Zanata  https://review.openstack.org/54884822:50
*** yamamoto has joined #openstack-lbaas23:08
openstackgerritMerged openstack/python-octaviaclient master: Updated from global requirements  https://review.openstack.org/54957023:16
openstackgerritGerman Eichberger proposed openstack/octavia master: Periodic job to build + publish diskimage  https://review.openstack.org/54925923:21
xgerman_ts. ts -1 for merge conflict as if gerrit isn’t his own man…23:21
johnsomHe's just trying to balance out the diversity tag23:22
johnsomgrin23:22
xgerman_I thought they got rid of it23:23
openstackgerritMerged openstack/python-octaviaclient master: fix error url  https://review.openstack.org/54092323:36
*** AlexeyAbashkin has joined #openstack-lbaas23:36
openstackgerritMerged openstack/octavia-dashboard master: Imported Translations from Zanata  https://review.openstack.org/55037123:37
*** AlexeyAbashkin has quit IRC23:41
openstackgerritMerged openstack/neutron-lbaas master: Updated from global requirements  https://review.openstack.org/55153323:42
openstackgerritMerged openstack/neutron-lbaas master: Imported Translations from Zanata  https://review.openstack.org/54877023:42
*** Swami has quit IRC23:56
« Monday, 2018-03-12 Index Wednesday, 2018-03-14 »

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