Friday, 2019-03-08

*** rascasoft has joined #openstack-infra00:02
*** wolverineav has quit IRC00:02
*** wolverineav has joined #openstack-infra00:02
openstackgerritMerged openstack-infra/system-config master: Send readme parameter when creating projects  https://review.openstack.org/64185100:03
*** rascasoft has quit IRC00:08
corvusstill no change in the status as reported via openstackclient00:11
fungiclarkb: i didn't see anyone volunteer to add a `rm -rf /etc/exim4/paniclog` near the end of the launch script, no00:14
clarkbcorvus: did you do a hard reboot? that is with the --hard flag iirc00:15
clarkbits possible we might need to do that instead of the acpi version00:15
corvusclarkb: no00:15
corvusoh, the status changed back to active, let's see if it's better00:16
fungiyeah, i think the normal reboot just sends ctrl-alt-del to the console and hopes for the best00:16
corvusstill looks squirrely to me.  shall i do --hard?00:16
clarkbcorvus: ++00:16
corvusdone00:16
corvusi have a shell now00:17
corvusi don't see any obvious problems in syslog00:20
clarkbcorvus: it is running the services I expect it to run too00:21
fungianybody happen to know whether our ubuntu-bionic images include python2.7 by default? (i know bionic itself has made it optional)00:22
clarkbfungi: pretty sure they do00:22
clarkbwe don't have special logic for python versioning the zuul cloner shim so python2 it across the baord iirc00:23
fungithanks, i'll only prepare a fix for that if upcoming testing proves it an invalid assumption00:23
clarkbonce centos8 happesn we can python300:23
openstackgerritJeremy Stanley proposed openstack-infra/openstack-zuul-jobs master: Use ubuntu-xenial explicitly in py35 jobs  https://review.openstack.org/64187700:24
fungithere may be similar places in other repos i need to do the same00:24
clarkbfungi: the end of that diff has py35 and py36 listed won't that try to run all of them on the same host then tox will skip whichever python version isn't present?00:25
clarkbif that is the case, is that intentional?00:26
fungiclarkb: that's bindep profiles not tox envs00:26
clarkboh00:26
fungithe lower-constraints job uses default python3 and the default nodeset00:26
*** bgmccollum has joined #openstack-infra00:26
*** owalsh has joined #openstack-infra00:27
fungiso if the default nodeset switches to a platform where default python3 is 3.6 then including any available py36 bindep profile is warranted00:27
openstackgerritZane Bitter proposed openstack-infra/openstack-zuul-jobs master: Add Python3 project templates for Train release  https://review.openstack.org/64187800:27
*** owalsh_ has quit IRC00:27
openstackgerritZane Bitter proposed openstack-infra/openstack-zuul-jobs master: Add Python3 project templates for Train release  https://review.openstack.org/64187800:28
fungii guess to run a job with one nodeset on certain branches and a different nodeset on other branches i need two job variants one of which has the inverse of the other's branch matchers, right?00:29
fungi(for a job defined in a single-branch repository, that is)00:30
corvusw00t, zuul install on zuul01 has caught up with master00:30
fungii wonder if i need to care about branches older than stable/newton00:32
corvusi'm going to go ahead and restart all of zuul, mostly because i *think* this'll be the commit we release00:33
diablo_rojoclarkb, I finally went through and changed the topic for all the patches I was sure were from outreachy applicants to 'outreachy'00:33
fungithanks corvus! that sounds marvellous00:34
diablo_rojoIts like 7 patches or so? in both webclient and regular storyboard00:35
corvuswell, apparently i'm not because there is a bunch of release activity happening now.00:35
*** rlandy has quit IRC00:35
fungiahh00:35
openstackgerritClark Boylan proposed openstack-infra/system-config master: Cleanup exim paniclog on newly launched hosts  https://review.openstack.org/64188000:35
clarkbfungi: ^ tthere we go00:35
fungithanks clarkb!00:36
*** eharney has quit IRC00:37
*** roman_g has quit IRC00:38
corvusi'll do the restart tomorrow morning00:39
*** roman_g has joined #openstack-infra00:40
fungiclarkb: corvus: looking more closely at the errors we get and moreso the timestamps, i *think* they're coming at log rotation when the time between initial launch and addition to inventory spans one or more logrotate pulses00:42
fungiso it's possible deleting at the end of the launch is too early to help00:43
corvusi think that error should stop when our aliases file is in place00:43
fungi"User 0 set for local_delivery transport is on the never_users list"00:43
fungiyeah, do we not add the aliases until the server is in the inventory:?00:43
clarkbI would expect the base playbook to do that, but it is possible that puppet does it/00:44
corvusbase playbook does it00:44
clarkbif base playbook does it then we do apply that during launch00:44
corvusthen... "hrm" i say.00:45
fungia few of the examples i found had entries in them spanning the first several days the server was online00:47
*** wolverineav has quit IRC00:49
fungiokay, i think it logs an entry for that error each time the server needs to send e-mail00:49
fungithe wiki-dev02.openstack.org server i've booted but not yet added to our static inventory most assuredly does *not* have our aliases added to /etc/aliases yet00:49
fungiits ip address (which is not in dns yet) is 23.253.253.20800:49
*** markvoelker has joined #openstack-infra00:50
openstackgerritGhanshyam Mann proposed openstack-infra/openstack-zuul-jobs master: Move zuul-legacy-jobs.yaml jobs to Bionic  https://review.openstack.org/64188600:52
*** mriedem_away is now known as mriedem00:54
*** ricolin has joined #openstack-infra00:59
*** wolverineav has joined #openstack-infra01:03
*** mriedem has quit IRC01:04
fungigmann: ahh, cool, i was just about to start hacking on a similar change for the openstack-tox abstract job so i'll replicate your pattern from 63909601:06
gmannfungi: +1. moving towards projects side nodeset removal and few testing patches. then i will shoot those on ML.01:09
fungii've prepped copies of the base-test and base job changes about to push in a bit, and 641877 is what will keep the py35 jobs running on xenial indefinitely since those will assuredly break on bionic01:11
gmannk01:12
*** gyee has quit IRC01:15
*** whoami-rajat has joined #openstack-infra01:23
*** markvoelker has quit IRC01:23
*** sthussey has quit IRC01:24
openstackgerritJeremy Stanley proposed openstack-infra/openstack-zuul-jobs master: Explicitly pin openstack-tox to xenial for <=rocky  https://review.openstack.org/64189501:32
openstackgerritJeremy Stanley proposed opendev/base-jobs master: Test changing the default nodeset to ubuntu-bionic  https://review.openstack.org/64189601:33
openstackgerritJeremy Stanley proposed opendev/base-jobs master: Switch default nodeset to ubuntu-bionic  https://review.openstack.org/64189701:33
fungigmann: there ^ are the other pieces01:33
fungithat last one will be wip until cutover01:34
*** rascasoft has joined #openstack-infra01:34
gmannnice.01:35
fungithe others should in theory be a complete no-op01:37
fungiuntil the last one merges01:37
*** wolverineav has quit IRC01:39
*** wolverineav has joined #openstack-infra01:40
*** rascasoft has quit IRC01:40
gmannyeah. *-py35 has ubuntu-xenial as explicitly mentioned as nodeset.01:41
openstackgerritGhanshyam Mann proposed openstack-infra/openstack-zuul-jobs master: Add legacy 3 nodes nodeset based on Ubuntu bionic  https://review.openstack.org/64189801:41
*** wolverineav has quit IRC01:44
*** bhavikdbavishi has joined #openstack-infra02:11
*** markvoelker has joined #openstack-infra02:20
*** hwoarang has quit IRC02:28
*** jamesmcarthur has joined #openstack-infra02:30
*** hwoarang has joined #openstack-infra02:30
*** rascasoft has joined #openstack-infra02:32
*** hongbin has joined #openstack-infra02:42
*** roman_g has quit IRC02:43
*** rascasoft has quit IRC02:43
*** jhesketh has quit IRC02:45
*** jhesketh has joined #openstack-infra02:47
*** markvoelker has quit IRC02:53
*** rkukura has quit IRC02:54
*** ramishra has joined #openstack-infra02:55
*** bhavikdbavishi has quit IRC02:58
*** roman_g has joined #openstack-infra03:07
*** apetrich has quit IRC03:16
*** ianw_pto has quit IRC03:23
*** ianw has joined #openstack-infra03:23
*** jbadiapa has quit IRC03:24
*** jbadiapa has joined #openstack-infra03:25
*** jamesmcarthur has quit IRC03:27
*** armax has quit IRC03:30
*** armax has joined #openstack-infra03:33
*** ykarel|away has joined #openstack-infra03:36
*** ykarel|away is now known as ykarel03:37
*** rascasoft has joined #openstack-infra03:40
*** rascasoft has quit IRC03:44
*** bhavikdbavishi has joined #openstack-infra03:48
*** bhavikdbavishi1 has joined #openstack-infra03:48
*** markvoelker has joined #openstack-infra03:50
*** bhavikdbavishi has quit IRC03:52
*** bhavikdbavishi1 is now known as bhavikdbavishi03:52
*** rascasoft has joined #openstack-infra03:59
*** rascasoft has quit IRC04:09
*** markvoelker has quit IRC04:23
*** udesale has joined #openstack-infra04:24
*** whoami-rajat has quit IRC04:33
openstackgerritGhanshyam Mann proposed openstack-infra/openstack-zuul-jobs master: Move legacy-periodic-package-stackviz-element to stackviz repo  https://review.openstack.org/64192304:44
*** whoami-rajat has joined #openstack-infra04:53
*** hwoarang has quit IRC05:02
*** hwoarang has joined #openstack-infra05:08
*** markvoelker has joined #openstack-infra05:20
*** janki has joined #openstack-infra05:34
*** tkajinam has quit IRC05:38
*** tkajinam has joined #openstack-infra05:39
*** hongbin has quit IRC05:42
*** kjackal has joined #openstack-infra05:44
AJaegerconfig-core, could you review https://review.openstack.org/#/c/639018/ , please?05:45
openstackgerritMerged openstack-infra/openstack-zuul-jobs master: Use ubuntu-xenial explicitly in py35 jobs  https://review.openstack.org/64187705:48
*** kjackal has quit IRC05:51
*** markvoelker has quit IRC05:53
openstackgerritRiju Khatri proposed openstack-infra/storyboard-webclient master: Make task ordering in Story view selectable  https://review.openstack.org/64193005:59
openstackgerritRiju Khatri proposed openstack-infra/storyboard-webclient master: Make task ordering in Story view selectable  https://review.openstack.org/64193006:04
openstackgerritRiju Khatri proposed openstack-infra/storyboard-webclient master: Make task ordering in Story view selectable  https://review.openstack.org/64193006:18
*** kjackal has joined #openstack-infra06:47
*** markvoelker has joined #openstack-infra06:51
*** Bhujay has joined #openstack-infra06:52
openstackgerritTristan Cacqueray proposed openstack-infra/zuul master: tests: remove debugging prints  https://review.openstack.org/64194207:00
*** armax has quit IRC07:01
*** wolverineav has joined #openstack-infra07:09
*** ramishra has quit IRC07:10
*** kjackal has quit IRC07:13
*** kjackal has joined #openstack-infra07:16
openstackgerritTristan Cacqueray proposed openstack-infra/zuul master: web: switch jobs list to a tree view  https://review.openstack.org/63343707:20
*** cjloader has quit IRC07:22
*** kjackal has quit IRC07:22
*** markvoelker has quit IRC07:23
openstackgerritTristan Cacqueray proposed openstack-infra/zuul master: web: add jobs list filter  https://review.openstack.org/63365207:23
*** hwoarang has quit IRC07:25
*** ramishra has joined #openstack-infra07:25
*** pcaruana has joined #openstack-infra07:26
*** pgaxatte has joined #openstack-infra07:26
*** hwoarang has joined #openstack-infra07:30
*** kjackal has joined #openstack-infra07:32
*** ykarel_ has joined #openstack-infra07:45
*** ykarel has quit IRC07:46
*** Bhujay has quit IRC07:50
*** Bhujay has joined #openstack-infra07:57
*** tkajinam has quit IRC08:00
*** ginopc has joined #openstack-infra08:01
*** wolverineav has quit IRC08:04
*** wolverineav has joined #openstack-infra08:05
*** rpittau|afk is now known as rpittau08:06
openstackgerritTristan Cacqueray proposed openstack-infra/zuul master: web: switch jobs list to a tree view  https://review.openstack.org/63343708:11
openstackgerritTristan Cacqueray proposed openstack-infra/zuul master: web: add jobs list filter  https://review.openstack.org/63365208:11
*** markvoelker has joined #openstack-infra08:20
*** xek_ has joined #openstack-infra08:26
openstackgerritAdam Coldrick proposed openstack-infra/storyboard-webclient master: Add a subcontroller for Team projects  https://review.openstack.org/64196308:26
openstackgerritAdam Coldrick proposed openstack-infra/storyboard-webclient master: Add UI for making security teams related to projects  https://review.openstack.org/64196408:26
*** xek_ has quit IRC08:27
*** xek has joined #openstack-infra08:28
*** Bhujay has quit IRC08:31
*** jpich has joined #openstack-infra08:31
*** rfolco has joined #openstack-infra08:36
*** rfolco|ruck has quit IRC08:38
*** iurygregory|home is now known as iurygregory08:40
*** kjackal has quit IRC08:43
*** kjackal has joined #openstack-infra08:43
*** wolverineav has quit IRC08:46
*** markvoelker has quit IRC08:53
openstackgerritAdam Coldrick proposed openstack-infra/storyboard master: Add a table mapping Teams to Projects  https://review.openstack.org/64171808:54
openstackgerritAdam Coldrick proposed openstack-infra/storyboard master: Add an endpoint for getting Projects related to a Team  https://review.openstack.org/64171908:54
openstackgerritAdam Coldrick proposed openstack-infra/storyboard master: Allow Teams to be filtered by project_id  https://review.openstack.org/64172008:54
*** Wryhder has joined #openstack-infra09:02
*** mcgiggler has joined #openstack-infra09:03
*** pcaruana has quit IRC09:12
*** ccamacho has joined #openstack-infra09:16
*** ykarel_ is now known as ykarel|lunch09:16
*** ginopc has quit IRC09:18
*** kjackal has quit IRC09:18
*** ginopc has joined #openstack-infra09:24
*** pcaruana has joined #openstack-infra09:27
*** fdegir has quit IRC09:34
*** ginopc has quit IRC09:40
*** kjackal has joined #openstack-infra09:42
*** ykarel_ has joined #openstack-infra09:43
*** ykarel_ is now known as ykarel09:43
*** ginopc has joined #openstack-infra09:44
*** ykarel|lunch has quit IRC09:45
*** ykarel is now known as ykarel|lunch09:46
*** rcernin has quit IRC09:47
*** fdegir has joined #openstack-infra09:48
*** markvoelker has joined #openstack-infra09:51
*** wolverineav has joined #openstack-infra09:57
*** wolverineav has quit IRC10:02
*** rascasoft has joined #openstack-infra10:05
*** ramishra has quit IRC10:05
*** ykarel|lunch has quit IRC10:15
*** ykarel has joined #openstack-infra10:16
*** jistr is now known as jistr|biab10:21
*** ccamacho has quit IRC10:21
*** markvoelker has quit IRC10:23
*** Wryhder has quit IRC10:23
*** Wryhder has joined #openstack-infra10:24
*** ccamacho has joined #openstack-infra10:30
*** gfidente has joined #openstack-infra10:32
*** electrofelix has joined #openstack-infra10:36
*** ricolin has quit IRC10:39
*** yamamoto has quit IRC10:43
*** ramishra has joined #openstack-infra10:48
*** bhavikdbavishi has quit IRC10:51
*** jistr|biab has quit IRC10:51
*** jistr has joined #openstack-infra10:54
*** udesale has quit IRC11:03
*** udesale has joined #openstack-infra11:04
*** Wryhder has quit IRC11:06
*** luizbag has joined #openstack-infra11:06
*** dtantsur|afk is now known as dtantsur11:08
*** yamamoto has joined #openstack-infra11:18
*** markvoelker has joined #openstack-infra11:20
*** jaosorior has quit IRC11:20
*** yamamoto has quit IRC11:25
*** yamamoto has joined #openstack-infra11:26
*** jaosorior has joined #openstack-infra11:32
*** bhavikdbavishi has joined #openstack-infra11:43
*** raissa has joined #openstack-infra11:44
*** raissa has quit IRC11:46
*** markvoelker has quit IRC11:54
*** panda|rover is now known as panda|rover|lunc11:54
*** gfidente has quit IRC12:07
*** EmilienM is now known as EvilienM12:08
*** tosky has joined #openstack-infra12:12
*** mcgiggler has quit IRC12:18
kashyapAJaeger: Hi, wonder if you know how can I find the ".zuul.yaml in the base job"?  (Referring to: https://review.openstack.org/#/c/638757/)12:24
AJaegerWhich part do you have problems with?12:37
AJaegerThe file you touch is run as part of a job, that job is defined in the file .zuul.yaml in the devstack repo12:38
AJaegerlet me give you a link...12:38
AJaegerkashyap, see http://git.openstack.org/cgit/openstack-dev/devstack/tree/.zuul.yaml#n18712:39
AJaegerthat's the job...12:39
kashyapAh, sorry, missed to check here (distraction)12:39
kashyapAJaeger: Ah, so instead of the main.yaml, I should add it to the .zuul.yaml12:40
kashyapOr _both_ places?12:40
kashyap(By "it", I mean the content of the patch)12:40
AJaegerkashyap: your patch as is is part 1 - part 2 is adding to zuul.yaml, see the list with files to copy12:41
AJaegerkashyap: line 227, zuul_copy_output is the variable12:41
* kashyap looks12:41
AJaegeryou need *both* changes12:41
AJaegerwait...12:42
* kashyap waits12:42
AJaegerI think you can add /var/log/audit/audit.log  directly to zuul_copy_output; that single line change might be all that's needed. Please try it - and if you have further questions, about devstack, the #openstack-qa channel is a better place12:43
*** gfidente has joined #openstack-infra12:45
*** zbr|ssbarnea has quit IRC12:46
kashyapYeah, noted.12:46
kashyapAJaeger: So the current change main.yaml might not even be required, after all?12:47
*** zbr has joined #openstack-infra12:47
kashyapI'll first just put in audit.log to 'zuul_copy_output' variable12:47
kashyapSo I've added:12:49
kashyap+        '{{ stage_dir }}/audit.log': logs12:49
kashyapTo .zuul.yaml; /me posts the patch to see if it gets captured12:49
*** kgiusti has joined #openstack-infra12:52
*** iurygregory is now known as iurygregory_mtg12:53
*** rh-jelabarre has joined #openstack-infra13:04
*** bhavikdbavishi has quit IRC13:07
*** janki has quit IRC13:07
openstackgerritMerged openstack-infra/openstack-zuul-jobs master: Add bionic nodeset for legacy job to switch to bionic  https://review.openstack.org/63901813:09
*** eharney has joined #openstack-infra13:17
*** frickler has quit IRC13:17
*** frickler has joined #openstack-infra13:18
AJaegerkashyap: yes, taht was my suggestion.13:18
*** agopi has joined #openstack-infra13:19
kashyapNoted, thanks13:20
*** iurygregory_mtg is now known as iurygregory13:24
*** panda|rover|lunc is now known as panda|rover13:25
*** ramishra has quit IRC13:27
*** yamamoto has quit IRC13:27
*** pcaruana has quit IRC13:30
*** jcoufal has joined #openstack-infra13:30
*** dave-mccowan has joined #openstack-infra13:30
*** rlandy has joined #openstack-infra13:32
*** dave-mccowan has quit IRC13:34
*** mriedem has joined #openstack-infra13:38
*** janki has joined #openstack-infra13:40
*** derekh has joined #openstack-infra13:53
*** yamamoto has joined #openstack-infra13:54
*** bhavikdbavishi has joined #openstack-infra13:55
openstackgerritMatt Riedemann proposed openstack-infra/elastic-recheck master: Add query for nova evacuate --force bug 1819166  https://review.openstack.org/64202014:00
openstackbug 1819166 in OpenStack Compute (nova) "nova-live-migration evacuation tests fail with "error: unrecognized arguments: --force"" [Critical,In progress] https://launchpad.net/bugs/1819166 - Assigned to sean mooney (sean-k-mooney)14:00
*** ccamacho has quit IRC14:01
openstackgerritGhanshyam Mann proposed openstack-infra/openstack-zuul-jobs master: Add legacy 3 nodes nodeset based on Ubuntu bionic  https://review.openstack.org/64189814:06
*** rfolco is now known as rfolco|ruck14:08
AJaegerconfig-core, let's merge the change above as well for bionic move ^ - please review14:09
*** bhavikdbavishi has quit IRC14:10
mordredAJaeger: legacy-ubuntu-bionic-3-node is funny14:11
AJaegeryeah - legacy job for current OS ;(14:15
*** kjackal has quit IRC14:19
*** markvoelker has joined #openstack-infra14:20
fungi"funny" isn't the word i would have chosen14:27
fungidisappointing is closer to it14:27
openstackgerritMerged openstack-infra/elastic-recheck master: Add query for nova evacuate --force bug 1819166  https://review.openstack.org/64202014:28
openstackbug 1819166 in OpenStack Compute (nova) "nova-live-migration evacuation tests fail with "error: unrecognized arguments: --force"" [Critical,In progress] https://launchpad.net/bugs/1819166 - Assigned to sean mooney (sean-k-mooney)14:28
openstackgerritNatal Ngétal proposed openstack/diskimage-builder master: [Configuration] Switch to stestr.  https://review.openstack.org/62941414:35
sean-k-mooneyfor what its worth i plan to port the os-vif legacy job soon14:35
sean-k-mooneyi recently added a non legacy tempest-base jobs so it should be fairly trivail to swap over os-vif-ovs14:36
sean-k-mooneyhttps://github.com/openstack/os-vif/blob/master/.zuul.yaml#L1-L5614:36
*** pcaruana has joined #openstack-infra14:37
openstackgerritMerged openstack-infra/openstack-zuul-jobs master: Add legacy 3 nodes nodeset based on Ubuntu bionic  https://review.openstack.org/64189814:37
sean-k-mooneyone slightly disapoint thing i found was that devstack with linux bridge seamed to not work right on python3 but updating all of them and refining the test they run is on my too list for before the ptg14:38
*** jchhatbar has joined #openstack-infra14:40
*** ykarel is now known as ykarel|away14:40
openstackgerritNatal Ngétal proposed openstack/diskimage-builder master: [Configuration] Switch to stestr.  https://review.openstack.org/62941414:41
*** mcgiggler has joined #openstack-infra14:42
*** jchhatba_ has joined #openstack-infra14:42
*** janki has quit IRC14:42
fungisean-k-mooney: any idea what about python 3 breaks devstack with linuxbridge?14:44
fungii wouldn't have expected any interaction there14:45
*** jchhatbar has quit IRC14:45
openstackgerritNatal Ngétal proposed openstack/diskimage-builder master: [Configuration] Switch to stestr.  https://review.openstack.org/62941414:47
sean-k-mooneyit looked like it installed all the deps in python3 and tried to execute the linux bridge agent with python214:47
sean-k-mooneylet me see if the logs are still there14:47
fungiahh, in that case it might be an easy fix in devstack14:49
sean-k-mooneyoh it was releated to privsep14:49
mordredfungi, AJaeger: https://review.openstack.org/#/c/632532/ is green again - if you have a sec for a quick review14:50
sean-k-mooneyfungi: http://logs.openstack.org/32/639732/3/check/os-vif-linuxbridge/9f6f380/controller/logs/screen-q-agt.txt.gz14:50
*** efried is now known as fried_rice14:51
sean-k-mooneyso the privsep helper defaults to python2 even when we install with python314:51
*** wolverineav has joined #openstack-infra14:51
fungiianw: ^ if you're still around, were you involved in the devstack python3 conversion?14:52
*** cjloader has joined #openstack-infra14:52
sean-k-mooneythe privesep helpt is created by setup tools right14:52
sean-k-mooneylooks like it hardcodes the system python path14:54
*** markvoelker has quit IRC14:54
sean-k-mooney[stack@devstack2 nova]$ cat /usr/bin/privsep-helper14:54
sean-k-mooney#!/usr/bin/python14:54
fungiahh, yeah, if it's being installed globally and has an explicit shebang, that'll cause trouble14:55
sean-k-mooneywell that is still proably easy to fix with sed14:55
*** wolverineav has quit IRC14:55
sean-k-mooneyat least in devstack14:56
fungithat either needs to be /usr/bin/python3 or just invoke the helper under an interpreter like `python3 /usr/bin/privsep-helper` instead14:56
sean-k-mooneyya i think there is a config optin for the privsep deamon so we could jsut set it in the python3 jobs14:56
fungii'm doing a bit of searching to see where it gets called14:57
fungioh, yeah calling it under the interpreter isn't an option14:58
fungisince the idea is that the privsep-helper executable is whitelisted for running as root14:59
fungito call it under an interpreter as root you'd need to whitelist the interpreter, and then you could really run just about anything (or at least it becomes even harder to lock down)14:59
sean-k-mooneyright so either devstack need to change the path in the privsep-helper15:00
sean-k-mooneyor we could pass the interperter as an arg to it15:00
fungii'm surprised that the shebang references python(2) when installed for python315:00
funginormally our entrypoints arrangement should take care of setting the correct interpreter in the wrapper15:01
sean-k-mooneywell it reference system python15:01
fungias opposed to system python3, yes, but the entrypoints wrapper should set the correct one on installation15:01
sean-k-mooneyon ubunut that would be python3 right15:01
funginope15:01
sean-k-mooneyno i taught ubuntu 18.04 was python 3 only by defualt15:02
fungi/usr/bin/python should *always* refer to python2. for python3 you have to call it explicitly15:02
sean-k-mooneyeven on distors that dont have python315:02
fungiubuntu 18.04 doesn't provide a python2 by default unless you install it, but they don't link python to python315:02
sean-k-mooneyoh ok15:02
fungithe only distro i'm aware of which has made that choice is arch linux, and it was against the recommendation of the entire python community15:03
sean-k-mooneyim just used to looking at it in tox where python links to the one you asked for15:03
fungianyway, i'm going to take a quick look at how devstack is installing privsep-helper, and then maybe dig into the packaging of privsep-helper to see what's causing that15:03
fungiyeah, inside a virtualenv which is created with one and only one python interpreter the bin/python is symlinked to whatever interpreter was used to create it even if that's python315:04
sean-k-mooneysorry to create more work but i guess its better then this biting use when we drop python 2 support15:04
openstackgerritTristan Cacqueray proposed openstack-infra/zuul master: web: add flatten checkbox  https://review.openstack.org/64204715:04
fungiabsolutely, this is a great catch. thanks for raising it!15:04
sean-k-mooneyit was on my todo list to try and fix when i got around to optimising the os-vif jobs next week.15:05
fungiso /usr/local/bin/privsep-helper is what's ending up in the path, looks like, indicating it's probably coming from a pip install15:05
sean-k-mooneyit got deferend due to ff15:05
sean-k-mooneyyes it comes from privsep15:06
sean-k-mooneyhttps://github.com/openstack/oslo.privsep/blob/master/setup.cfg#L4015:07
fungioslo.privsep==1.32.0 in http://logs.openstack.org/32/639732/3/check/os-vif-linuxbridge/9f6f380/controller/logs/pip2-freeze.txt.gz15:07
sean-k-mooneyso it generate by setup tools15:07
fungioslo.privsep==1.32.015:07
fungialso in http://logs.openstack.org/32/639732/3/check/os-vif-linuxbridge/9f6f380/controller/logs/pip3-freeze.txt.gz15:07
fungii suspect what's happening is it's getting installed twice and the second entrypoint wrapper (for python2) is overwriting the first (for python3)15:08
sean-k-mooney it might be related to swift15:08
sean-k-mooneyi didnt deisable it but dont they still run on python 215:08
sean-k-mooneyevent when devstack is python 315:08
sean-k-mooneyoh i have disabled swift already15:09
sean-k-mooneyoh its comming form the installatyion of os-vif form git15:12
sean-k-mooneyhttp://logs.openstack.org/32/639732/3/check/os-vif-linuxbridge/9f6f380/controller/logs/devstacklog.txt.gz#_2019-02-27_16_42_48_51415:13
sean-k-mooneyfrom this http://logs.openstack.org/32/639732/3/check/os-vif-linuxbridge/9f6f380/controller/logs/devstacklog.txt.gz#_2019-02-27_16_42_33_78215:15
*** jchhatba_ has quit IRC15:15
*** janki has joined #openstack-infra15:16
*** jamesmcarthur has joined #openstack-infra15:16
fungigreat catch, so need to make it install os-vif under python315:17
sean-k-mooneyfungi: ya but its been install via libs form git due to the require project15:17
sean-k-mooneyin the zuul job15:17
sean-k-mooneyalso just before that15:18
sean-k-mooneyhttp://logs.openstack.org/32/639732/3/check/os-vif-linuxbridge/9f6f380/controller/logs/devstacklog.txt.gz#_2019-02-27_16_42_32_85915:18
fungi"Installing os-vif again without Python 3 enabled"15:18
sean-k-mooney2019-02-27 16:42:32.854 | + inc/python:python3_enabled:600           :   [[ True == \T\r\u\e ]]15:18
sean-k-mooney2019-02-27 16:42:32.857 | + inc/python:python3_enabled:601           :   return 015:18
fungisee comment block at http://git.openstack.org/cgit/openstack-dev/devstack/tree/inc/python#n44115:20
*** udesale has quit IRC15:21
*** udesale has joined #openstack-infra15:21
fungithis should probably move to #openstack-qa, the pattern there means any packages installing entrypoint wrappers are going to end up running under python2 instead of python315:21
sean-k-mooney right... so  that is not logically correct due to the gloabl privsep-helper15:21
sean-k-mooneyya i can bring it up there or you can if you like. im also on another call at the moment15:22
fungibecause the libraries themselves can be installed for multiple python interpreters, but the executables they provide are not going to be globally unique names and so will overwrite each other15:22
fungisean-k-mooney: no worries, i'll bring it up (though i also have to disappear to run errands in about 5 minutes)15:23
*** janki has quit IRC15:23
sean-k-mooneythe ovs job proably would have also exploed for the same reason but i i had other issue in the ovs job so it didint get that far15:24
openstackgerritAdam Coldrick proposed openstack-infra/storyboard master: Add a security flag to Stories  https://review.openstack.org/64205615:25
openstackgerritAdam Coldrick proposed openstack-infra/storyboard master: Add RESTful endpoints for Story permissions  https://review.openstack.org/64205715:25
*** jangutter has quit IRC15:39
*** armax has joined #openstack-infra15:40
*** fried_rice is now known as efried_afk15:48
*** yamamoto has quit IRC15:48
*** yamamoto has joined #openstack-infra15:49
*** TheJulia is now known as needssleep15:50
*** markvoelker has joined #openstack-infra15:51
*** yamamoto has quit IRC15:51
*** dtantsur is now known as creepy_owlet15:52
*** ginopc has quit IRC15:57
*** ccamacho has joined #openstack-infra15:58
*** yamamoto has joined #openstack-infra16:01
*** cmurphy is now known as cmorpheus16:02
*** yamamoto has quit IRC16:07
*** bhavikdbavishi has joined #openstack-infra16:12
*** efried_afk is now known as fried_rice16:16
*** sthussey has joined #openstack-infra16:19
*** markvoelker has quit IRC16:23
*** sean-k-mooney has quit IRC16:23
*** roman_g has quit IRC16:30
*** roman_g has joined #openstack-infra16:30
*** adriancz has quit IRC16:36
mriedemeverything in the gate that is co-gating with nova is probably getting kicked out right now, fyi16:37
mriedemhttp://status.openstack.org/elastic-recheck/#181916616:37
mriedemi don't know if we want to talk about bumping https://review.openstack.org/#/c/641986 to the top?16:37
mriedemthe nova-live-migration job is 100% fail on master16:38
mriedemwithout that fix16:38
openstackgerritAdam Coldrick proposed openstack-infra/storyboard-webclient master: Add support for Story permission endpoints  https://review.openstack.org/64207016:40
openstackgerritAdam Coldrick proposed openstack-infra/storyboard-webclient master: WIP: Automatically add security teams to security stories  https://review.openstack.org/64207116:40
*** pgaxatte has quit IRC16:42
*** cmoura has quit IRC16:46
clarkbmriedem: ps2 on that change failed testing16:53
clarkbmriedem: do we want to see it pass on ps3 before pushing to the top of the gate?16:53
clarkb(not sure what our confidence is in that change on ps3)16:53
*** iurygregory has quit IRC16:55
mriedemclarkb: yeah ps2 failed because the --os-compute-api-version option has to come *before* the "evacuate" sub-command, which is what is fixed in PS316:56
mriedemso i'm 99.5% sure it'll be ok16:57
mriedemlike the usage here says http://logs.openstack.org/86/641986/2/check/nova-live-migration/6f6b678/job-output.txt.gz#_2019-03-08_14_32_57_20451716:57
*** ccamacho has quit IRC16:57
clarkbmriedem: ok looks like there aer no nova changes ahead in the gate so as long as you haven't approved other changes I think we are set16:58
mriedemok16:58
clarkbunless the neutron failures are related to this but I don't think they are16:58
mriedemtempest changes run the nova-live-migration job in the check queue..17:00
mriedemoh but non-voting :/17:00
*** jamesmcarthur_ has joined #openstack-infra17:00
*** jpich has quit IRC17:01
*** e0ne has joined #openstack-infra17:03
*** jamesmcarthur has quit IRC17:03
*** udesale has quit IRC17:04
openstackgerritClark Boylan proposed openstack-infra/system-config master: Only install puppet 4 if not already installed  https://review.openstack.org/64208017:07
clarkbcmorpheus: ^ fyi I think we want that due to the side effects of puppet4 install on the review-dev server17:07
mordredclarkb, corvus: https://review.openstack.org/#/c/632532/ is green - have a sec for a +3?17:08
*** e0ne has quit IRC17:08
clarkbmordred: looking17:08
clarkbmordred: can you check my ansible in the change just above?17:08
cmorpheusclarkb: it's /opt/puppetlabs/bin/puppet17:08
clarkbcmorpheus: it seems like it is actually both17:09
cmorpheusoh17:09
cmorpheusTIL17:09
clarkband /opt/puppetlabs/bin/puppet is al ink to the other poath17:09
clarkb(so I went with the actual file not the link but both would work)17:09
cmorpheusgotcha17:09
clarkblrwxrwxrwx 1 root root 20 Jun 12  2018 /opt/puppetlabs/bin/puppet -> ../puppet/bin/puppet17:09
*** xek has quit IRC17:11
*** xek has joined #openstack-infra17:11
*** rlandy is now known as rlandy|brb17:12
*** pcaruana has quit IRC17:12
mordredclarkb: looking17:13
mordredclarkb: do I want to know why installing puppet removes gerrit rc.d files?17:13
mordred+2 from me17:14
clarkbmordred: I didn't want to know myself :/ my hunch is that they call some deb helper function to install rc.d things and it helpfully clears out things it thinks are redundant17:15
clarkbmordred: we have a S04gerrit script from system sysv init compat and puppet installs a S90gerrit17:15
clarkbsomething sees S90gerrit as redundant in that process and removes it. We shoudl go back through and systemdify that module if we stick with it long enough but for not installing puppet 4 once seems like a good approach17:16
mordredclarkb: fun. I'm guessing no amount of digging further will improve happiness17:16
cmorpheusnot rerunning the install_puppet script is a good thing17:17
clarkb++17:17
*** markvoelker has joined #openstack-infra17:20
*** gyee has joined #openstack-infra17:22
*** gfidente has quit IRC17:24
clarkbfungi: corvus looking at the root never_users paniclog thing. aliases does have an entry for root pointing to us on health01 (one of the servers that complained). never_users is set to root ( and that is compiled in via an even stronger directive according to the comment above that). Reading our exim role we write the aliases file, then the exim conf, then set the exim restart handler17:27
clarkbmy two theories for what is happening are we write an empty root aliases entry on launch because our vars aren't properly available for some reason then the cron job fixes it (more likely given timestamps) or the restart handler runs long enough after exim is installed that for a time we are doing tasks that trigger root emails until that handler runs17:28
*** creepy_owlet is now known as dtantsur|afk17:29
clarkbmordred: ^ for the first theory I'm trying to understand how system-config/playbooks/group_vars makes it into our ansible runs. Is that purely due to the relative path relationship to our playbooks?17:31
*** rpittau is now known as rpittau|afk17:31
mordredyes17:31
clarkbok os.path.join(SCRIPT_DIR, '..', 'playbooks', playbook) is how we refer to the playbooks in launch node. I wonder if that .. or cwd confuses it17:32
clarkbbut if not, that should just work (tm)17:33
corvusclarkb: my experience is that ansible does things relative to the playbook path regardless of the CWD used when running ansible17:34
clarkbexim_sysadmins: "{{ ','.join(sysadmins|default([])) }}" is how that var is defined and sysadmins i defined in /etc/ansible/hosts/group_vars/all.yaml17:34
clarkbpossible it is the lookup of sysadmins that is failing17:35
clarkbmordred: we pass an entire config directory via the ansible-playbook -i argument with host and groups and so on. I wonder if that overrides the global settings?17:40
mordredclarkb: they merge17:41
mordredclarkb: so the host and group vars from the inventory sources are merged with the host and group vars adjacent to the playbooks17:41
clarkbok, the docs totally ignore that use case. And only talk about a single file17:41
clarkbmordred: ya but we have a third set of vars, the global ones in /etc/ansible17:42
clarkbby passing a dir to the -i flag are we replacing /etc/ansible?17:42
*** mriedem is now known as mriedem_burgers17:43
clarkbOne way we could test this is to move the sysadmins var into system-config/playbooks/group_vars/all.yaml17:43
mordredI'm sorry - I've stopped being able to follow this - can we back up?17:43
clarkbmordred: sure17:43
mordredwhat are we passing via -i ?17:43
clarkbmordred: a tmpdir with a hosts file and a groups file (and maybe some other stuff)17:43
*** fried_rice is now known as fried_rolls17:43
mordredclarkb: where are we doing this?17:44
clarkbmordred: in system-config/launch/launch-node.py17:44
mordredthanks. one sec17:44
*** derekh has quit IRC17:44
mordredclarkb: yes - that is almost certainly overriding the existence of /etc/ansible17:45
clarkbmordred: if we switch it to -i $inventory_file we should get back the behavior we want I assume?17:45
clarkbrun with a different inventory against our global config17:45
mordredno - I don't believe so17:45
clarkboh17:46
mordredwhat we need is a list of inventory sources17:46
*** asettle is now known as away17:46
*** away is now known as asettle17:46
mordredclarkb: /etc/ansible/ansible.cfg has:17:46
mordredinventory=/opt/system-config/inventory/openstack.yaml,/opt/system-config/inventory/groups.yaml,/etc/ansible/hosts/emergency.yaml17:46
clarkbfwiw I don't really want to move sysadmins into the system-cofngi repo as then we get emails from random people's servers set up with our tooling17:46
mordred-i is setting a new value of inventory17:46
mordredwe're doing this so that we can run on the new host before adding it to the global inventory - right?17:47
clarkbmordred: yes17:47
mordredhow about if we add ,/opt/system-config/inventory/openstack.yaml,/opt/system-config/inventory/groups.yaml,/etc/ansible/hosts/emergency.yaml to the -i17:47
corvusi'm going to restart zuul now17:47
mordredcorvus: ++17:47
mordredclarkb: I haven't tested that the command line will take comma separated list like the config file does - but I don't see any reason why it wouldn't17:48
*** panda|rover is now known as panda|rover|baby17:48
clarkbmordred: that seems like a reasonable thing at first glance. Any concerns with side effects? I think our use of the target var for the hosts entry and only running a small subset of things like base and a cleanup or two should make it safe17:48
clarkbmordred: I don't think base respects {{ target }} though17:48
clarkbwe might end up running base against everything and maybe we need to add a --limit?17:48
mordredyeah - I think we should add a limit17:49
mordredwhich we already have actually17:49
*** xek has quit IRC17:49
clarkbthe -l ? ya17:49
*** xek has joined #openstack-infra17:49
mordredyeah17:50
*** agopi is now known as agopi|afk17:50
mordredoh - --help says comma separated will work17:50
clarkbcool feel good to get to the bottom of that :)17:50
clarkbmordred: do you want to push the change or should I?17:50
mordredclarkb: I can push one up17:51
*** mcgiggler has quit IRC17:51
*** rlandy|brb is now known as rlandy17:51
*** markvoelker has quit IRC17:54
clarkbfungi: ^ fyi if you don't want to read all the scrollback the fix is to add more ansible environment stuff to our launch ansible runs17:54
openstackgerritMonty Taylor proposed openstack-infra/system-config master: Add global inventory to launch_node  https://review.openstack.org/64209617:54
clarkbfungi: we weren't setting the root alias as the sysadmins var isn't availalbe to our launch script. We then set if after regular ansible cron runs17:54
clarkbmordred: I'm thinking we may still want my change at https://review.openstack.org/#/c/641880/ to clear out any panic log that might happen during our setup? though I suppose it should be totally unnecesasry if we set alias before exim config and exim restart17:55
corvus#status log restarted all of zuul at commit 603ce6f474ef70439bfa3adcaa27d806c23511f717:57
openstackstatuscorvus: finished logging17:57
corvusre-enqueing17:57
mordredclarkb: yeah - I think maybe let's see if we're still getting paniclogs with the aliases set properly17:58
mordredclarkb: it seems like they're currently serving a purpose - indicating that there is an issue we should solve :)17:59
clarkbmordred: ++17:59
mordredclarkb: but - if we're still getting them and they're bogus, let's land that for sure17:59
clarkbya I think the case we'd want that is if there is a race between exim being installed and configuring it where we get a paniclog due to something triggering a root email18:00
clarkbthat case may not exist at all18:00
clarkbmordred: also we should get ansible to document this behavior18:00
clarkb"specify inventory host path or comma separated host list. –inventory-file is deprecated" is all the docs say18:01
*** mattw4 has joined #openstack-infra18:01
mordredlovely :)18:02
clarkbcorvus: we may want to enqueue https://review.openstack.org/#/c/641986 to the gate since I think our restart kicked it out of check (and its broken all of nova). I don't think it needs to be promoted as there are no other nova changes gating.18:02
corvusclarkb: it's in check now18:04
*** electrofelix has quit IRC18:04
clarkbinfra-root can I get a second revoew on https://review.openstack.org/#/c/642080/ thats my last major todo out of putting puppet4 on our dev servers18:07
corvusclarkb: do you know what the test hole is for that nova change?18:08
clarkbcorvus: my guess is that the test there only runs on nova changes and not novaclient changes18:09
clarkbthen novaclient changed behavior and it broke18:09
clarkbbut I don't kow for sure18:09
clarkbmriedem_burgers: ^18:09
*** rkukura has joined #openstack-infra18:11
corvushttps://review.openstack.org/#/c/635131/ did not run nova-live-migration but it also merged 3 weeks ago18:12
openstackgerritPaul Belanger proposed openstack-infra/project-config master: Add Zuul project-config repo  https://review.openstack.org/64210018:12
openstackgerritPaul Belanger proposed openstack-infra/project-config master: Define a specific Zuul tenant for Zuul project  https://review.openstack.org/64210118:12
pabelangercorvus: toabctl: mordred: clarkb: ^my first attempt at creating zuul specific tenant in zuul18:13
clarkbcorvus: we just made releases yesterday18:13
clarkbcorvus: possible novaclient was in that big round of release activity you noticed yesterday?18:13
pabelangerI suspect we'll also want an untrusted 'openstack-zuul-jobs' repo there too, at some point18:13
jrollcorvus: clarkb python-novaclient was released recently, yeah18:13
jroll(per the comments)18:14
jrollthe test must use novaclient from pip, the release started using the new microversion and broke it18:14
corvusclarkb, jroll: that makes sense.  in which case, it might be good to put a nova-live-migration-src job on both repos, so we can catch those before release18:14
clarkb++18:14
jrollmakes sense to me18:15
jrollmriedem_burgers: ^^18:15
*** wolverineav has joined #openstack-infra18:17
openstackgerritJames E. Blair proposed openstack-infra/system-config master: Add a soft dependency between gitea build and run  https://review.openstack.org/64210318:19
corvusclarkb, mordred, tobiash, pabelanger, dmsimard: ^18:20
tobiashyay18:21
pabelanger+218:21
*** roman_g has quit IRC18:22
*** ykarel|away has quit IRC18:22
clarkbcorvus: fwiw I think that is the feature that tripleo was asking for (for completely different reasons)18:23
corvusclarkb: yeah, it's generally useful in situations where you may or may not build a binary artifact18:24
*** eernst has joined #openstack-infra18:25
mordredcorvus: that's super awesome18:25
clarkbin their case they want a soft dependency of their integration jobs on their non integration jobs so that they can optionally not run the not integration jobs aiui18:26
corvusi've rechecked 640903 and 905; if they are green, then i think we're ready for a release18:26
clarkbI don't know why you'd want to run integration jobs if not running unittests but whatever18:26
corvusthat is not something we should encourage or support18:27
*** eernst has quit IRC18:29
*** eernst has joined #openstack-infra18:29
openstackgerritJames E. Blair proposed openstack-infra/system-config master: Add a soft dependency between gitea build and run  https://review.openstack.org/64210318:29
*** eernst has quit IRC18:30
corvusclarkb, pabelanger, tobiash, mordred: ^ revised -- there's a caveat there we'll want to keep in mind.18:30
clarkbya thats the difference with how tripleo wants to use the feature18:31
corvusi wonder if that's another thing we should consider adding to zuul -- an option to a dependency relationship which says, basically, "if job A runs, forget about the files matcher on job B, always run job B"18:31
*** wolverineav has quit IRC18:31
*** jamesmcarthur_ has quit IRC18:32
corvusor we could look into doing artifact dependencies...18:32
mordredcorvus: yeah - that sounds useful/important18:32
*** betherly has joined #openstack-infra18:34
*** wolverineav has joined #openstack-infra18:35
*** eernst has joined #openstack-infra18:36
*** rkukura has quit IRC18:36
openstackgerritMerged openstack-infra/nodepool master: docker: don't daemonize when starting images  https://review.openstack.org/63558418:37
*** betherly has quit IRC18:39
*** eernst has quit IRC18:39
openstackgerritMerged openstack-infra/zuul master: tests: remove debugging prints  https://review.openstack.org/64194218:48
*** markvoelker has joined #openstack-infra18:51
*** bhavikdbavishi has quit IRC18:55
corvushuzzah!  903 failed and 905 succeeded as expected!18:55
corvushttps://review.openstack.org/640905 is a fully working cross-repo-dependency speculative container execution18:56
corvusi'm going to take a break, and then start working on a release18:56
mordredcorvus: huzzah indeed! that's super exciting18:57
clarkbcorvus: neat.18:57
*** wolverineav has quit IRC19:03
*** wolverineav has joined #openstack-infra19:03
*** wolverineav has quit IRC19:06
*** wolverineav has joined #openstack-infra19:06
clarkbalso I'm happy that my assertion about the command not existing causes a failure is true :)19:06
clarkbfungi: have a sec to review https://review.openstack.org/#/c/642080/1 ? cleans up some puppet-4 stuff19:06
*** rfolco|ruck is now known as rfolco|ruck|brb19:11
*** jamesmcarthur has joined #openstack-infra19:11
openstackgerritMerged openstack-infra/zuul master: SQL: only create tables in scheduler  https://review.openstack.org/61069619:12
*** rkukura has joined #openstack-infra19:13
openstackgerritAnkita Bansal proposed openstack-infra/storyboard-webclient master: Adds number of active stories beside repo list in the project_group/id page. A function storyCountActive is written in project_story_list_controller.js and called while fetching the projects list. A HTML tag added in project_search_item.html for the activ  https://review.openstack.org/64212119:15
*** mriedem_burgers is now known as mriedem19:16
fungiokay, i'm back from extended friday lunch errands and almost caught up on irc scrollback (though not e-mail yet)19:18
*** diablo_rojo has quit IRC19:20
mriedemjroll: correct the job uses novaclient from pypi / devstack19:20
fungiclarkb: in regard to the exim paniclog and our /etc/aliases, have you yet looked at a newly-launched server that's not in the static inventory yet? i have one handy (the soon relpacement wiki-dev02.openstack.org isn't in dns yet but is reachable at 23.253.253.208)19:21
fungiif you peek in the aliases file you'll see we don't actually write it during launch19:21
fungibut i'm still reading scrollback and it looks like you may have looked deeper into things19:22
clarkbfungi: yup https://review.openstack.org/#/c/642096/ should be the fix19:22
clarkbbasically we don't have our global ansible vars availalbe to launch currently so the sysadmins value isn't found19:22
fungiand now i see a fix has been suggested19:22
fungimakes sense based on what i've seen up to this point19:22
*** markvoelker has quit IRC19:24
fungiapproved 642096 and i agree in retrospect 641880 is likely entirely a no-op19:24
*** jamesmcarthur has quit IRC19:30
fungiokay, all caught up in here19:31
*** wolverineav has quit IRC19:34
*** wolverineav has joined #openstack-infra19:35
clarkbwe probably want to watch the next launch to be sure that we don't run base on all the things but the limit should address that19:35
mnaserfungi: https://review.openstack.org/#/c/641896/1 do we wanna merge this now?19:36
clarkbmnaser: yes I think we do19:36
clarkbI can approve it now19:36
mnasercool, i think https://review.openstack.org/#/c/641895/ as well might be good19:37
clarkbthen jobs can reparent to base-test and check the results19:37
fungimnaser: clarkb: yes, the sooner the better19:37
fungiall of the topic:bionic-transition changes should be safe to merge except the one i marked wip19:37
clarkbfungi: all are approved now except that wip change19:38
fungithough 641895 could likely be expanded by puppet, ansible and other teams to add similar branch pins to some of their jobs19:38
fungii figure between the openstack-tox abstract job and the legacy-base job gmann is working on we should have most stuff covered (the new-style devstack jobs are already taken care of)19:39
*** wolverineav has quit IRC19:40
*** jamesmcarthur has joined #openstack-infra19:43
openstackgerritMerged opendev/base-jobs master: Test changing the default nodeset to ubuntu-bionic  https://review.openstack.org/64189619:49
openstackgerritMerged openstack-infra/openstack-zuul-jobs master: Explicitly pin openstack-tox to xenial for <=rocky  https://review.openstack.org/64189519:49
clarkbhttps://ivanceras.github.io/svgbob-editor/ is a neat diagramming tool19:50
clarkbinfra-root ^ will probably aprpeciate that given our use of ascii art19:50
fungithe name makes me think of microsoft bob19:50
corvusmordred, clarkb, fungi: i just thought of a refactor for the buildset registry jobs that will make things simpler; i don't think it requires any changes to zuul, so i'll continue with zuul release prep after lunch, but then i'll work on that refactor.  i don't think it's going to be very complex, just rearranging some role rearranging19:51
fungiclarkb: but yes, that looks like an amazing tool19:51
clarkbI'm going to check my zuul logo ascii art in it19:51
corvusclarkb: that is awesome19:51
corvustobiash: ^ :)19:52
fungigranted some of their examples seem to take advantage of characters outside the 7-bit ascii set19:52
clarkbhrm it doesn't seem to do the inferring with the ascii lines19:52
clarkbso would have to convert to | and _ and so on19:53
fungii find the circuit diagramming example at the end very compelling19:54
corvusit's a lot faster if you clear out all the stuff from the example19:55
fungigranted the use of + and - for line art lends some confusion in that particular context19:55
corvushttps://github.com/ivanceras/svgbob has a cli tool and written in rust19:57
openstackgerritGhanshyam Mann proposed openstack-infra/openstack-zuul-jobs master: Move zuul-legacy-jobs.yaml jobs to Bionic  https://review.openstack.org/64188619:57
corvusfungi: apparently you're supposed to think "bob ross" not "microsoft bob"19:57
corvusthis might be interesting as a sphinx plugin19:58
*** jbadiapa has quit IRC19:59
fungiso true20:00
clarkbI can't make anyting for zuul that looks better than what I've already got. But probably could if I spent more than 5 minutes on it20:00
*** luizbag has quit IRC20:01
openstackgerritMerged openstack-infra/system-config master: Add global inventory to launch_node  https://review.openstack.org/64209620:03
openstackgerritMerged openstack-infra/system-config master: Only install puppet 4 if not already installed  https://review.openstack.org/64208020:06
*** wolverineav has joined #openstack-infra20:06
*** wolverineav has quit IRC20:11
*** irclogbot_1 has quit IRC20:13
*** markvoelker has joined #openstack-infra20:21
*** rfolco|ruck|brb is now known as rfolco|ruck20:21
*** e0ne has joined #openstack-infra20:24
pabelangercorvus: clarkb: are we also considering a nodepool release?20:26
clarkbthere wad a full nodepool restart not long ago to check openstack sdk compatobility20:29
clarkband to support provider affinity20:29
clarkbI think it would be ok to release nodepool too20:29
corvusya20:29
pabelangergreat20:29
clarkbIm off to lunch to get out of the house. back after to followup on puppet and python thingd20:30
*** xek has quit IRC20:37
*** xek has joined #openstack-infra20:38
openstackgerritJames E. Blair proposed openstack-infra/nodepool master: Remove prelude from AWS release note  https://review.openstack.org/64214820:42
*** yamamoto has joined #openstack-infra20:49
*** markvoelker has quit IRC20:53
mordredcorvus: wow. we really can't manage to get the python-base patch landed can we?20:54
*** yamamoto has quit IRC20:54
*** kjackal has joined #openstack-infra20:55
mordredcorvus, clarkb: http://logs.openstack.org/32/632532/5/gate/system-config-run-gitea/ca78bee/ara-report/ is looking like it's trying to actually run against the actual gitea hosts20:56
openstackgerritJames E. Blair proposed openstack-infra/zuul-jobs master: Revert "Docker: use the buildset registry if defined"  https://review.openstack.org/64215020:56
*** fried_rolls is now known as fried_rice20:56
openstackgerritJames E. Blair proposed opendev/base-jobs master: In docker build jobs, run the registry if not running  https://review.openstack.org/64215120:56
corvusmordred: cool, maybe we can continue to use it to test those changes ^ :)20:56
mordred\o/20:56
corvusmordred: i'll start digging into your link20:57
*** slaweq has quit IRC20:57
corvusoh, sorry, one more patch to write, then i'll dig in20:57
*** slaweq has joined #openstack-infra20:58
clarkbI'll review after lunch21:00
openstackgerritJames E. Blair proposed openstack-infra/system-config master: Use the opendev docker build jobs  https://review.openstack.org/64215221:00
corvusthere it is ^21:00
corvusmordred: wow yeah, there should be no gitea07 in there.21:01
corvusmordred: ah i think i get it.  this runs after the base playbook, which probably re-bootstrapped fake-bridge enough to overwrite the config and use the real inventory21:03
*** e0ne has quit IRC21:03
*** agopi|afk is now known as agopi21:03
mordredcorvus: oh! right21:04
corvusmordred: didn't we land a fix to that though which was supposed to be self testing?21:04
mordredmaybe that fix didn't run test gitea?21:04
* mordred looking21:04
mordredcorvus: oh - no - the only thing we landed in this area was related to launch-node21:06
corvusmordred: oh, another hypothesis -- that it's not the base playbook run in run-base.yaml that resets the configuration, but rather, the cron job.  if this worked once and failed once, that might explain the difference -- whether the cron job ran21:06
mordredcorvus: yes. it does in fact make sense21:06
mordredand makes more sense than a patch we landed impacting it21:06
corvusmordred: no i mean -- haven't we seen the test gitea playbook run?21:06
corvusif it has ever worked correctly, then it must be the second hypothesis21:07
mordredcorvus: yeah - sorry - that's what I meant21:07
mordredcorvus: first hypothesis only makes sense if we landed a patch that changed behavior AND that had a files exclusion causing gitea job to not run21:08
mordredbut neither of those things happened - so I'm with 2nd hypothesis21:08
mordredis there a good way to disable cron?21:08
corvushere's a successful run: http://logs.openstack.org/03/642103/2/check/system-config-run-gitea/b9a8f9d/ara-report/result/d4311d2a-b5b5-4326-84da-4af0ee607a2d/21:09
corvusmordred: hrm.  normally it's only a minor nuisance; it shows up in output but doesn't affect results.  and it's kind of nice not to have to debug it.21:09
corvuswe're walking the walk on testing production here.21:10
corvuser, i mean it's kind of nice not to have to *disable* it21:10
corvusbut i guess we could21:10
corvusput in a hostvar to set whether or not to enable cron21:11
corvushttp://logs.openstack.org/32/632532/5/gate/system-config-run-gitea/ca78bee/hosts/bridge.openstack.org/ara-report/21:12
corvusthat's from the failing run, and cron did run21:12
corvusthe only other idea that springs to mind would be to make a cleaner environment for the test run21:12
corvusmordred: i think you've convinced me that just disabling cron is ok21:12
corvusmordred is a skilled rhetorician21:13
*** jamesmcarthur has quit IRC21:13
*** jamesmcarthur has joined #openstack-infra21:14
corvusmordred: should we do that?  you want to patch or shall i?21:14
mordredcorvus: yeah - let's do that ... I think while testing real production is important ...21:15
mordredcorvus: we're testing a thing here which the cron is bonging21:15
mordredcorvus: if you have your eyes on where to put the flag already it would be great if you could - but if you're going to have to dig for it too I can21:16
corvusmordred: i don't, so if you can take it, i'd appreciate it21:16
mordredkk. on it21:16
corvusmordred: just that you'll probably want to put the setting for the gate in system-config/playbooks/zuul/templates/host_vars/bridge.openstack.org.yaml.j221:17
*** bnemec is now known as beekneemech21:18
openstackgerritMonty Taylor proposed openstack-infra/system-config master: Add flag to disable cron for test jobs  https://review.openstack.org/64215721:19
mordredcorvus: maybe that'll fix it21:19
corvusmordred: lgtm21:21
corvusmordred: do you think we need to do similar for the cloud-launcher cron?21:21
mordredcorvus: I don't think so - it shouldn't be polluting the host21:21
corvusk.  couldn't remember if it did any 'pull project-config' or similar things21:22
mordredit might attempt to run some things against some clouds - but it won't have the creds for it to work21:22
openstackgerritMerged openstack-infra/nodepool master: Remove prelude from AWS release note  https://review.openstack.org/64214821:25
*** Wryhder has joined #openstack-infra21:26
corvusmordred: would you mind reviewing https://review.openstack.org/#/q/status:open+topic:registry ?21:26
*** betherly has joined #openstack-infra21:26
*** jamesmcarthur has quit IRC21:27
*** jamesmcarthur has joined #openstack-infra21:28
*** wolverineav has joined #openstack-infra21:28
mordredcorvus: stack looks great to me21:28
corvusmordred: thanks!21:29
*** wolverineav has quit IRC21:30
*** betherly has quit IRC21:31
*** wolverineav has joined #openstack-infra21:32
*** wolverineav has quit IRC21:32
*** wolverineav has joined #openstack-infra21:32
openstackgerritDavid Shrewsbury proposed openstack-infra/zuul-preview master: Port in changes from the c++ version  https://review.openstack.org/64108921:37
*** betherly has joined #openstack-infra21:47
*** diablo_rojo has joined #openstack-infra21:47
*** jamesmcarthur has quit IRC21:50
*** markvoelker has joined #openstack-infra21:51
*** betherly has quit IRC21:52
*** jamesmcarthur has joined #openstack-infra21:53
*** jamesmcarthur has quit IRC21:54
*** Lucas_Gray has joined #openstack-infra21:58
corvusfungi, clarkb: are you around to +3 https://review.openstack.org/642157  ?21:59
clarkbcorvus: I just pulled up your topic:registry changes22:00
clarkblooks like zuul was unhappy with the third change there22:00
corvusclarkb: yeah, that's actually likely fixed by 64215722:00
*** rh-jelabarre has quit IRC22:00
clarkbcool I've approved that one22:01
clarkbnow to review the other stack22:01
*** Wryhder has quit IRC22:01
corvusclarkb: so i think we should land 642157 and the first 2 registry changes, then recheck 642152, and my zuul-preview changes, and 63253222:01
clarkbcorvus: whihc zuul-preview changes? I know about the DNM changes22:01
corvusclarkb: those22:01
corvusjust to exercise the new jobs22:02
clarkbgotit22:02
*** rh-jelabarre has joined #openstack-infra22:03
clarkbok I think things are headed to the gate now and we just have to wait before we recheck22:03
corvus++22:03
*** kgiusti has left #openstack-infra22:04
clarkbhrm something is uninstalling puppet4 in addition to installing it22:07
clarkbso my don't install puppet 4 all the time fix isn't working right22:07
fungicorvus: looking22:07
clarkbbecause puppet 4 has actually been removed22:07
corvusokay, i think docker wants me to buy a new hard drive22:07
fungiseems clarkb already beat me to it22:07
*** betherly has joined #openstack-infra22:08
fungicorvus: what's the provenance of http://files.openstack.org/project/git-test/www/.htaccess (is it in a git repo somewhere i can copy history from, or should i just list you as co-author on a patch introducing it)?22:08
*** kjackal has quit IRC22:08
*** jamesmcarthur has joined #openstack-infra22:08
*** wolverineav has quit IRC22:08
mordredcorvus: running rust builds in a docker container via docker build DEFINITELY wants me to buy a new hard drive22:09
openstackgerritMerged openstack-infra/zuul-jobs master: Revert "Docker: use the buildset registry if defined"  https://review.openstack.org/64215022:09
corvusfungi: entirely me in emacs directly in afs, so co-author is the way to go22:10
fungiwill do, thanks!22:10
corvuswho makes the best internal ssd's these days?  it's been a while since i've bought one22:11
*** betherly has quit IRC22:13
*** EvilienM is now known as EmilienM22:16
fungigood question. my most recent purchase was a "mushkin pilot" (1tb pcie m.2 ssd)22:16
fungiobtained in october for ~us$24022:17
*** wolverineav has joined #openstack-infra22:18
fungiapparently their hq is in austin, tx22:18
fungianyway, it's been performing well so far in a gaming rig which is also doing some virtualization. not sure how it would perform under more severe torture22:19
*** rlandy has quit IRC22:19
fungigranted i don't have any real benchmarks to compare either22:19
*** jamesmcarthur has quit IRC22:19
openstackgerritClark Boylan proposed openstack-infra/system-config master: Fix puppet 4 installations  https://review.openstack.org/64216922:21
clarkbcmorpheus: mordred fungi ^ you reviewed the last change and that is the next fix. I expect this will actually fix it22:21
fungimy next-most-recent internal ssd purchase was in 2015, a 240gb "MyDigitalSSD" (msata ssd) for ~us$110 and it's still going strong in my primary workstation22:22
mordredclarkb: oh wow22:22
clarkbmordred: its good that we are working through this on a small number of hosts :)22:22
mordredclarkb: \o/22:22
clarkbmordred: I expect we'll be pretty confident in the process as soon as we get through that :)22:22
mordredclarkb: it's almost like we've learned our lesson on things in the past22:23
*** wolverineav has quit IRC22:23
clarkbright22:23
clarkbcorvus: samsung is who I've been going with lately22:23
clarkbcorvus: decent prices for what I'm told is good quality. Both on their sata drives and nvme22:23
clarkbnext host upgrade is going nvme for the zoom zooms22:24
*** markvoelker has quit IRC22:24
fungii've been doing external usb3-powered ssds on my media server, with kernel mdraid mirroring22:25
mtreinishI think all but one of my machines has samsung ssds at this point, I've never had a failure so far22:26
* mtreinish just doomed himself22:26
corvusmtreinish: i assume you have enough drives to have statistically significant numbers :)22:26
fungifound a manufacturer named "vectotech" making 2tb external usb3 ssds, got a couple in 2018 for ~us$400@22:27
dmsimardthere were huge price drops on SSDs and NVMEs around xmas/black friday22:27
mtreinishfor SSDs it's not nearly as many. Only 12 between all my boxes22:27
dmsimardI have too many drives now but I've been doing stuff like swapping out 5400rpm spindles out of old laptops for relatives :p22:27
clarkbI think intel still has the longest warranty but they cost a lot more22:28
mtreinishcompared to the >50 spinning disks spread throughout my house22:28
*** Swami has joined #openstack-infra22:28
openstackgerritMerged openstack-infra/system-config master: Add flag to disable cron for test jobs  https://review.openstack.org/64215722:28
dmsimardcorvus: https://ca.pcpartpicker.com/products/internal-hard-drive/ is a good place to look -- you see price trends at multiple vendors, $/GB, user reviews, etc22:30
*** wolverineav has joined #openstack-infra22:32
*** jtomasek has quit IRC22:34
*** rh-jelabarre has quit IRC22:35
openstackgerritJeremy Stanley proposed openstack-infra/system-config master: Set up legacy git redirect sites  https://review.openstack.org/64217322:35
*** betherly has joined #openstack-infra22:50
*** wolverineav has quit IRC22:54
*** betherly has quit IRC22:55
*** eernst has joined #openstack-infra22:58
*** irclogbot_1 has joined #openstack-infra23:01
*** wolverineav has joined #openstack-infra23:01
*** eernst has quit IRC23:02
*** diablo_rojo has quit IRC23:03
openstackgerritMerged opendev/base-jobs master: In docker build jobs, run the registry if not running  https://review.openstack.org/64215123:10
*** rascasoft has quit IRC23:10
*** jcoufal has quit IRC23:15
*** eernst has joined #openstack-infra23:20
*** markvoelker has joined #openstack-infra23:21
*** xek has quit IRC23:22
*** eernst has quit IRC23:23
*** eernst has joined #openstack-infra23:27
openstackgerritJames E. Blair proposed openstack-infra/system-config master: Use opendev logos  https://review.openstack.org/64217923:28
fungiooh! fancy23:28
corvushttps://screenshots.firefox.com/7uGZTC42mvVafmgz/localhost if you're curious23:28
fungii am!23:28
fungior at least i was...23:28
funginow i'm not though. thanks!23:28
corvusthe <h2> under there is too big, but we can fix that later23:29
fungilgtm23:29
corvusand still need to find where to put the favicon :)23:29
fungidon't ask a web designer23:30
fungiapparently there's all sorts of crazy things they do these days to create iphone-compliant app buttons or tiles or whatever they're called23:30
fungiand if you just stick browser-oriented icons in there, they make funny faces at you23:31
corvusoh yeah, i used an online tool to bundle all that up for zuul's website23:31
*** eernst has quit IRC23:31
fungiit's nuts, isn't it?23:31
*** betherly has joined #openstack-infra23:32
corvusayep23:32
*** eernst has joined #openstack-infra23:33
*** whoami-rajat has quit IRC23:33
fungifor my personal sites i just do something like <link rel="shortcut icon" href="_static/logo.svg"/> and give up23:33
fungiand assume some day in the distant future, all devices will support svg23:34
fungiand at least in the interim, firefox seems to be fine with that23:34
clarkbIm trying to rememberif the emails I sent had the png/svg type assets in them23:36
fungii assume so since there's svg in that change23:36
*** betherly has quit IRC23:36
fungiand apparently also a ping23:36
*** eernst has quit IRC23:37
*** wolverineav has quit IRC23:37
corvusclarkb: not that i found, but you sent a pdf, and i turned that into an svg23:37
corvusand then turned the svg into a png23:37
clarkbah we can probably get james to send the real things along if we want23:37
corvusclarkb: yeah.  we can replace them.  i suspect they'll be substantially similar, so we can probably land these and swap them out when we get 'em23:38
corvusthe pdf was vectorized, so there shouldn't have been any data loss.  but if it was an older revision or something, the image could be slightly different.23:38
clarkb++23:39
*** wolverineav has joined #openstack-infra23:39
*** wolverineav has quit IRC23:44
*** wolverineav has joined #openstack-infra23:44
*** eernst has joined #openstack-infra23:44
*** eernst has quit IRC23:44
openstackgerritJames E. Blair proposed openstack-infra/system-config master: Disable ansible cron even more  https://review.openstack.org/64218123:45
openstackgerritTristan Cacqueray proposed openstack-infra/zuul master: web: add flatten checkbox  https://review.openstack.org/64204723:45
corvusclarkb, fungi, mordred: ^ one more fix needed for the ansible cron (the run gitea job failed again with the same error, i think that'll get it)23:45
corvusclarkb, fungi: also if one of you could add a +2 to https://review.openstack.org/642152 that'd be great; i can continue rechecking it and pushing it through when ready.23:46
corvusi think it works at this point and only ran into the ansible cron issue23:46
openstackgerritJames E. Blair proposed openstack-infra/system-config master: Use the opendev docker build jobs  https://review.openstack.org/64215223:48
openstackgerritJames E. Blair proposed openstack-infra/system-config master: Add a soft dependency between gitea build and run  https://review.openstack.org/64210323:48
openstackgerritJames E. Blair proposed openstack-infra/system-config master: DNM: test broken zuul-preview dockerfile  https://review.openstack.org/64090323:48
openstackgerritJames E. Blair proposed openstack-infra/system-config master: DNM: test fixed zuul-preview dockerfile  https://review.openstack.org/64090523:48
openstackgerritJames E. Blair proposed openstack-infra/system-config master: Split python-base into its own Dockerfile  https://review.openstack.org/63253223:48
openstackgerritJames E. Blair proposed openstack-infra/system-config master: Run python with jemalloc in containers  https://review.openstack.org/63601723:48
corvusthat's all the changes rebased and serialized for maximum testing23:49
fungiguess i'll review the new patchest for that one. only got a few lines in23:49
corvusfungi: sorry, was only a rebase23:49
fungiyeah, looks the same. cool23:49
openstackgerritJames E. Blair proposed openstack-infra/system-config master: Split python-base into its own Dockerfile  https://review.openstack.org/63253223:50
openstackgerritJames E. Blair proposed openstack-infra/system-config master: Run python with jemalloc in containers  https://review.openstack.org/63601723:50
*** markvoelker has quit IRC23:54
openstackgerritJeremy Stanley proposed openstack-infra/system-config master: Set up legacy git redirect sites  https://review.openstack.org/64217323:55
*** fried_rice is now known as efried23:56
clarkbcorvus: fungi have a moment for other ansible things https://review.openstack.org/#/c/642169/123:56
fungija23:57
fungiweird, i could swear i already reviewed half of that one earlier. i must have gotten distracted before i finished23:58
*** agopi has quit IRC23:59

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