Wednesday, 2018-05-09

*** rlandy has quit IRC00:07
ianwis zuul_site_mirror_fqdn not exported to all jobs?00:18
clarkbianw: it is set in our zuul site variables which I think means it should be in all jobs00:20
clarkbif it isn't that could possibly be another config bug ?00:20
ianwit may well be me ... putting in some debugging now.   http://git.openstack.org/cgit/openstack/kayobe/tree/roles/docker-engine/tasks/main.yml#n7 was just a weird comment too00:23
ianwmrhillsman: re your arm64 cross-build question, dan radez may have done something like that00:29
mrhillsmanthx ianw00:30
ianwclarkb: ok, it's there ... must be something in my tasks00:32
*** harlowja has quit IRC00:51
*** username_ has joined #zuul01:23
*** username_ is now known as username__01:23
*** username__ has quit IRC02:35
*** myoung|ruck|bbl is now known as myoung|ruck02:38
*** pleia2 has quit IRC03:04
*** pleia2 has joined #zuul03:18
*** threestrands has joined #zuul03:38
*** swest has joined #zuul04:29
*** swest has quit IRC04:33
*** swest has joined #zuul04:48
*** snapiri- has quit IRC05:20
*** snapiri has joined #zuul05:20
*** snapiri has quit IRC05:28
*** threestrands has quit IRC05:54
*** pcaruana has joined #zuul05:57
*** gtema has joined #zuul06:36
openstackgerritTobias Henkel proposed openstack-infra/zuul master: Improve logging around project branches  https://review.openstack.org/56712806:44
*** electrofelix has joined #zuul07:51
*** jpena|off is now known as jpena07:51
*** ssbarnea_ has joined #zuul08:47
*** snapiri has joined #zuul10:46
*** jpena is now known as jpena|lunch11:44
*** pcaruana has quit IRC12:21
*** rlandy has joined #zuul12:32
*** jpena|lunch is now known as jpena12:49
*** pwhalen_ has joined #zuul12:59
*** pwhalen has quit IRC13:01
*** pwhalen_ is now known as pwhalen13:07
*** pwhalen has quit IRC13:07
*** pwhalen has joined #zuul13:07
*** yolanda has joined #zuul13:16
*** yolanda_ has quit IRC13:18
*** dkranz has joined #zuul13:23
*** myoung|ruck is now known as myoung13:36
*** pcaruana has joined #zuul13:40
*** myoung is now known as myoung|ruck13:58
*** pcaruana has quit IRC14:05
openstackgerritTobias Henkel proposed openstack-infra/zuul master: Gracefully handle corrupt local git repositories  https://review.openstack.org/56722614:06
openstackgerritTobias Henkel proposed openstack-infra/zuul master: Gracefully handle broken .gitmodule files  https://review.openstack.org/56722714:06
*** pcaruana has joined #zuul14:08
tobiashthat fixes some problems we had that needed manual deletion of git repos on mergers/executors ^14:11
*** pcaruana has quit IRC14:25
*** ssbarnea_ has quit IRC14:26
*** pcaruana has joined #zuul14:26
fungias a reminder, i know a bunch of you are probably going to be attending opendev in vancouver... if you're also around for thursday, i'm moderating a forum session at the openstack summit (different floor in the same building) where we'll be talking about governance topics which reach across projects under the openstack foundation. anyone interested in zuul governance should try to be there14:38
fungihttps://www.openstack.org/summit/vancouver-2018/summit-schedule/events/21746/cross-community-governance-between-osf-projects14:38
fungiif you're around and want to sit in on that but are only registered for opendev and not the summit, just let me know and i can get you in14:39
mordredtobiash: the second failed tests, but it appears unrelated14:49
mordredShrews: if you have a sec, could I get a +A on https://review.openstack.org/#/c/566387/ ? (or anybody else)15:03
Shrewslooking15:03
Shrewsmordred: and the test failure there?15:04
Shrewsah, that was the async-timeout thing15:04
mordredShrews: shouldn't be related ... it's about async-timeout thing15:05
mordredyah15:05
corvusmordred: -1 on that15:06
corvusmordred: maybe just fix in a followup?15:07
Shrewsmordred: corvus: i removed the +W15:07
mordredcorvus: yah - I can fix in followup15:12
openstackgerritMonty Taylor proposed openstack-infra/nodepool master: Remove LIBS_FROM_GIT  https://review.openstack.org/56725315:13
mordredcorvus, Shrews ^^15:13
openstackgerritMerged openstack-infra/nodepool master: Add openstacksdk to nodepool-functional-py35-src  https://review.openstack.org/56638715:15
corvusclarkb, tobiash: do you think we've stabilized enough to make a release?15:23
clarkbcorvus: I'm happy with the config situation. Haven't heard or seen evidence of additional unfixed bugs15:24
clarkbcacti doesn't show major leaking either15:24
tobiashcorvus: running the config refactor since yesterday and didn't see any blocker today15:25
tobiash(in prod)15:26
openstackgerritDavid Shrewsbury proposed openstack-infra/nodepool master: Make manage_images a property of ProviderConfig  https://review.openstack.org/56726215:38
ShrewsI know that chain of changes (currently ending with 567262) may not seem to be related to multi-label support, but they really are15:40
ShrewsDigging down to the labels objects has revealed some issues15:41
*** acozine1 has joined #zuul15:46
mordredShrews: in 566963 - any reason to not just define the pools attribute in the base class? (curious)16:12
openstackgerritMerged openstack-infra/nodepool master: Remove LIBS_FROM_GIT  https://review.openstack.org/56725316:12
Shrewsmordred: if we did that, then the attr could potentially just be totally ignored by a driver. The abstract property sort of forces a driver to deal with it.16:15
mordredkk16:16
mordredstack looks good to me16:17
Shrewsi mean, it *is* defined in the base class, just not implemented16:17
Shrewsmordred: i responded in the review too16:21
mordredcool16:21
*** acozine1 has quit IRC16:25
Shrewsmordred: fwiw, the way i've been approaching this is by pretending to be a n00b to the driver api and trying to figure out what I need to do to create my fancy new driver w/o reading any documentation (since we don't have that).16:25
ShrewsIf python can tell me, 'hey dummy, you must implement this', that helps16:26
Shrewsbut also, we need docs16:26
mordred++16:28
*** sshnaidm|rover is now known as sshnaidm|off16:36
tobiashmordred: ah yes, that's unrelated16:40
tobiashjust rechecked16:41
*** gtema has quit IRC17:11
*** hashar has joined #zuul17:15
*** jpena is now known as jpena|off17:17
AJaegerzuul experts, my change https://review.openstack.org/#/c/548082/ for zuul-jobs has two +2s - shall we merge it now?17:19
*** hashar is now known as hasharBBQ17:23
*** gouthamr has quit IRC17:27
*** dmellado has quit IRC17:29
corvusAJaeger: done17:40
AJaegerthanks, corvus17:41
openstackgerritJames E. Blair proposed openstack-infra/zuul-website master: Update tagline  https://review.openstack.org/56729517:43
Shrewswoo tag lines17:46
openstackgerritMerged openstack-infra/zuul-website master: Add logo and move menu to top right  https://review.openstack.org/56590917:53
openstackgerritMerged openstack-infra/zuul-website master: Add community page  https://review.openstack.org/56617917:53
openstackgerritMerged openstack-infra/zuul-jobs master: Add simple run-test-command job  https://review.openstack.org/54808217:54
corvusShrews: put the spray paint away17:54
openstackgerritMerged openstack-infra/nodepool master: Clean up configuration file loading  https://review.openstack.org/56688618:03
openstackgerritMerged openstack-infra/nodepool master: Force driver provider configs to define pool attr  https://review.openstack.org/56696318:09
openstackgerritMerged openstack-infra/nodepool master: Make manage_images a property of ProviderConfig  https://review.openstack.org/56726218:09
openstackgerritMerged openstack-infra/nodepool master: Add systemd drop-in file for CentOS 7  https://review.openstack.org/56487218:09
clarkbShrews: totally not ignoring you but realized that I need to get summit prep stuff done so am forcing myself to focus on that today. I still want to review the label stuff but don't wait on me if others are reviewing it18:11
openstackgerritMerged openstack-infra/nodepool master: Remove debian-jessie from nodepool dsvm testing  https://review.openstack.org/56632718:11
Shrewsclarkb: all good. the ones i have up have been +A'd18:13
Shrewsand has mostly been driver cleanup up to this point anyway18:13
clarkbcool18:13
clarkbI now have a schedule without conflicts for the sessions I am leading18:13
corvusoh, sorry missed that18:14
clarkbeverything else is still full of conflicts woo18:14
*** gtema has joined #zuul18:36
*** dmellado has joined #zuul18:58
*** gouthamr has joined #zuul18:59
*** gouthamr has quit IRC19:13
*** gouthamr has joined #zuul19:17
clarkbcorvus: commented on https://review.openstack.org/#/c/566185/119:18
*** dkranz has quit IRC19:25
corvusclarkb: huh, that does seem backwards19:33
*** hasharBBQ has quit IRC19:38
openstackgerritJames E. Blair proposed openstack-infra/zuul-website master: Switch to Open Sans font  https://review.openstack.org/56618519:41
openstackgerritJames E. Blair proposed openstack-infra/zuul-website master: Change tagline to dark blue  https://review.openstack.org/56618619:41
openstackgerritJames E. Blair proposed openstack-infra/zuul-website master: Update tagline  https://review.openstack.org/56729519:41
corvusclarkb: yep, that was backwards, but the incorrect url was causing it to not have any effect and so it was falling back on the system "sans" font.  correcting the url did indeed cause normal/bold to switch like you would expect, and then swapping those in the css fixed it.19:42
corvusand you can see the difference now: http://logs.openstack.org/85/566185/2/check/zuul-website-build/7b8fa9a/html/  (correct)  vs  http://logs.openstack.org/85/566185/1/check/zuul-website-build/e71c042/html/  (incorrect)19:49
gtemawho can suggest (private zuul): in the job-output.txt output of tox is missing (and periodically for long running jobs "Log Stream did not terminate"), but in the job-output.json it is clearly present. Role start-zuul-console is present in the base job20:21
gtemain addition: ansible output in executor log: "[WARNING]: Failure using method (v2_runner_item_on_ok) in callback plugin"20:26
*** dkranz has joined #zuul20:42
clarkbgtema: the callback issue is likely related. YOu may need someone to check the executor logs to see why ^ failed?20:44
gtemaclarkb: ok will do a very deep analysis. It is already in the debug and ansible -vvv mode and still doesn't give really any hint so far20:48
*** gtema has quit IRC20:49
*** pcaruana has quit IRC20:59
*** dkranz has quit IRC21:02
*** dmellado has quit IRC21:15
*** gouthamr has quit IRC21:16
*** dmellado has joined #zuul21:23
*** dmellado has quit IRC21:27
*** gouthamr has joined #zuul21:42
*** myoung|ruck is now known as myoung|ruck|bbl21:59
*** gouthamr has quit IRC21:59
*** rlandy has quit IRC22:09
*** zigo has quit IRC22:11
*** zigo has joined #zuul22:11
*** gouthamr has joined #zuul22:14
*** threestrands has joined #zuul22:33
*** threestrands_ has joined #zuul22:36
*** threestrands_ has quit IRC22:36
*** threestrands_ has joined #zuul22:36
*** threestrands has quit IRC22:38
*** gouthamr has quit IRC22:52
*** gouthamr has joined #zuul22:55
*** myoung|ruck|bbl is now known as myoung|ruck|off22:55
*** myoung|ruck|off is now known as myoung|ruck22:57
openstackgerritMerged openstack-infra/nodepool master: Update logging format for devstack jobs  https://review.openstack.org/56694223:19

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