Tuesday, 2019-06-11

clarkbianw: no objection from me but I've got to pop off here shortly00:00
clarkbthe dfw image cleanup is running in screen in dfw so that can run in the background and will take time00:00
*** jamesmcarthur has joined #openstack-infra00:01
ianwoh we still leaking?  i need to read scroll back00:01
fungisdk regression00:02
* mordred is also trying to figure that out00:02
ianwclarkb: i also missed the cut off just for meeting but i'd like to sneak in an item for my plans update backup server if ok, i'll add it in a sec00:04
clarkbianw: ok00:04
fungishould the announcements include the spf change?00:05
fungii was unsure00:05
clarkbfungi: announcements?00:05
fungiannouncements portion of the meeting00:06
fungioh, may also want to announce the service-announce ml00:06
clarkbre spf I'm not sure we need to announce it broadly considering we are only enforcing failures00:06
clarkbso doing what was asked of us00:06
clarkbannouncing the other mailing list would probably be a good idea00:07
fungiyup, that's why i was unsure00:07
fungiabout spf00:07
fungibeing announcement-worthy00:07
*** michael-beaver has quit IRC00:09
*** goldyfruit has joined #openstack-infra00:10
*** markvoelker has joined #openstack-infra00:16
*** nicolasbock has quit IRC00:16
*** weifan has quit IRC00:21
*** weifan has joined #openstack-infra00:22
*** lseki has quit IRC00:25
*** weifan has quit IRC00:26
*** rcernin_ has quit IRC00:41
*** rcernin has joined #openstack-infra00:41
*** smarcet has joined #openstack-infra00:44
*** yamamoto has joined #openstack-infra00:46
openstackgerritKevin Carter (cloudnull) proposed zuul/zuul-jobs master: Add sanity checks  https://review.opendev.org/66447100:48
*** markvoelker has quit IRC00:49
*** jamesmcarthur has quit IRC00:52
*** igordc has quit IRC00:56
*** mattw4 has quit IRC00:57
*** gregoryo has joined #openstack-infra00:58
ianwdmsimard: re ERROR! Unexpected Exception, this is probably a bug: Unable to import Django: the server dependencies can be installed with 'pip install ara[server]' errors in devel job, is django really required?00:59
*** icarusfactor has joined #openstack-infra01:00
ianwthe other thing -- "The TRANSFORM_INVALID_GROUP_CHARS settings is set to allow bad characters in group names by default, this will change, but still be user configurable on deprecation." sounds like a big mess even reading the github issue01:01
*** factor has quit IRC01:02
*** smarcet has quit IRC01:07
*** ricolin has joined #openstack-infra01:10
openstackgerritTristan Cacqueray proposed opendev/gerritbot master: Support missing daemon lib  https://review.opendev.org/66447201:12
ianwok, ara django changes make more sense if not looking at a super old checkout pre opendev.org change01:12
*** icarusfactor has quit IRC01:19
*** factor__ has joined #openstack-infra01:19
*** factor__ has quit IRC01:24
*** factor__ has joined #openstack-infra01:25
*** sshnaidm has quit IRC01:33
openstackgerritIan Wienand proposed opendev/system-config master: Add "server" extras for ARA  https://review.opendev.org/66447801:34
*** gyee has quit IRC01:34
*** sshnaidm has joined #openstack-infra01:35
*** jamesmcarthur has joined #openstack-infra01:35
*** jamesmcarthur has quit IRC01:37
*** markvoelker has joined #openstack-infra01:46
openstackgerritMerged opendev/system-config master: bridge.opendev.org: use Ansible 2.8.0 stable  https://review.opendev.org/66266201:48
openstackgerritMerged opendev/system-config master: Update to ansible-lint 4.1.0  https://review.opendev.org/66350101:48
*** yamamoto has quit IRC01:48
*** factor__ has quit IRC01:49
*** factor__ has joined #openstack-infra01:50
*** diablo_rojo has joined #openstack-infra02:03
*** bhavikdbavishi has joined #openstack-infra02:05
*** whoami-rajat has joined #openstack-infra02:09
*** bhavikdbavishi has quit IRC02:09
*** michael-beaver has joined #openstack-infra02:12
*** smarcet has joined #openstack-infra02:13
openstackgerritKevin Carter (cloudnull) proposed zuul/zuul-jobs master: Update the readme for new role options  https://review.opendev.org/66448402:14
*** factor__ has quit IRC02:17
*** factor__ has joined #openstack-infra02:18
*** factor__ has quit IRC02:18
*** markvoelker has quit IRC02:20
openstackgerritKevin Carter (cloudnull) proposed zuul/zuul-jobs master: Add variable to set the docker download url  https://review.opendev.org/66448502:22
*** hongbin has joined #openstack-infra02:25
openstackgerritKevin Carter (cloudnull) proposed zuul/zuul-jobs master: Update the readme for new role options  https://review.opendev.org/66448402:49
openstackgerritKevin Carter (cloudnull) proposed zuul/zuul-jobs master: Add variable to set the docker download url  https://review.opendev.org/66448502:49
*** yamamoto has joined #openstack-infra02:50
*** bhavikdbavishi has joined #openstack-infra02:56
*** bhavikdbavishi1 has joined #openstack-infra02:59
*** bhavikdbavishi has quit IRC03:01
*** bhavikdbavishi1 is now known as bhavikdbavishi03:01
*** bnemec has quit IRC03:07
*** bnemec has joined #openstack-infra03:08
openstackgerritKevin Carter (cloudnull) proposed zuul/zuul-jobs master: Update the readme for new role options  https://review.opendev.org/66448403:12
*** diablo_rojo has quit IRC03:14
*** markvoelker has joined #openstack-infra03:17
*** bhavikdbavishi1 has joined #openstack-infra03:18
*** bhavikdbavishi has quit IRC03:20
*** bhavikdbavishi1 is now known as bhavikdbavishi03:20
*** psachin has joined #openstack-infra03:39
openstackgerritIan Wienand proposed opendev/system-config master: Update to ARA 1.0.1  https://review.opendev.org/66447803:42
*** markvoelker has quit IRC03:50
*** ramishra has joined #openstack-infra03:59
openstackgerritIan Wienand proposed opendev/system-config master: Update to ARA 1.0.1  https://review.opendev.org/66447803:59
*** hongbin has quit IRC04:01
*** udesale has joined #openstack-infra04:05
*** janki has joined #openstack-infra04:10
*** apetrich has joined #openstack-infra04:13
*** verdurin has quit IRC04:19
openstackgerritIan Wienand proposed opendev/system-config master: Update to ARA 1.0.1  https://review.opendev.org/66447804:21
*** dave-mccowan has quit IRC04:23
*** verdurin has joined #openstack-infra04:28
openstackgerritIan Wienand proposed opendev/system-config master: Update to ARA 1.0.1  https://review.opendev.org/66447804:43
*** markvoelker has joined #openstack-infra04:47
*** michael-beaver has quit IRC05:02
*** e0ne has joined #openstack-infra05:15
*** bhavikdbavishi1 has joined #openstack-infra05:19
*** bhavikdbavishi has quit IRC05:19
*** bhavikdbavishi1 is now known as bhavikdbavishi05:19
*** kjackal has joined #openstack-infra05:19
*** weifan has joined #openstack-infra05:20
*** markvoelker has quit IRC05:21
*** bhavikdbavishi has quit IRC05:23
*** ykarel has joined #openstack-infra05:23
*** weifan has quit IRC05:26
*** lpetrut has joined #openstack-infra05:28
*** weifan has joined #openstack-infra05:35
*** udesale has quit IRC05:35
*** udesale has joined #openstack-infra05:36
*** ykarel_ has joined #openstack-infra05:36
openstackgerritIan Wienand proposed opendev/system-config master: Update to ARA 1.0.1  https://review.opendev.org/66447805:38
*** ykarel has quit IRC05:39
*** bhavikdbavishi has joined #openstack-infra05:41
*** e0ne has quit IRC05:42
*** lpetrut has quit IRC05:43
*** weifan has quit IRC05:44
openstackgerritTristan Cacqueray proposed zuul/nodepool master: static: enable using a single host with different user or port  https://review.opendev.org/65920905:47
openstackgerritMerged zuul/zuul master: Annotate logs around build states  https://review.opendev.org/66148905:51
*** threestrands has joined #openstack-infra05:52
*** threestrands has quit IRC05:53
openstackgerritTristan Cacqueray proposed zuul/nodepool master: static: document multiple labels shortcomming  https://review.opendev.org/66295405:55
*** dchen has quit IRC06:12
openstackgerritMerged zuul/zuul master: Annotate logs around reporting  https://review.opendev.org/66149006:12
*** dchen has joined #openstack-infra06:15
*** markvoelker has joined #openstack-infra06:18
*** smarcet has quit IRC06:22
*** ykarel__ has joined #openstack-infra06:29
*** pgaxatte has joined #openstack-infra06:31
*** ykarel_ has quit IRC06:31
*** udesale has quit IRC06:32
*** udesale has joined #openstack-infra06:32
openstackgerritMerged zuul/zuul master: Annotate logs around finished builds  https://review.opendev.org/66149106:33
*** ccamacho has joined #openstack-infra06:36
*** yamamoto has quit IRC06:40
*** lpetrut has joined #openstack-infra06:46
*** markvoelker has quit IRC06:51
*** ginopc has joined #openstack-infra06:54
*** pcaruana has joined #openstack-infra06:55
*** slaweq has joined #openstack-infra06:56
*** nicolasbock has joined #openstack-infra06:57
*** yamamoto has joined #openstack-infra06:58
*** jtomasek has joined #openstack-infra07:05
*** rpittau|afk is now known as rpittau07:08
*** rcernin has quit IRC07:08
*** tesseract has joined #openstack-infra07:08
*** rh-jelabarre has quit IRC07:11
*** ramishra has quit IRC07:13
*** tosky has joined #openstack-infra07:17
*** slaweq has quit IRC07:20
*** e0ne has joined #openstack-infra07:20
openstackgerritMark Meyer proposed zuul/zuul master: Extend event reporting  https://review.opendev.org/66213407:22
*** dayou has quit IRC07:23
*** ykarel__ is now known as ykarel|lunch07:24
*** ramishra has joined #openstack-infra07:30
*** dayou has joined #openstack-infra07:37
*** yboaron has joined #openstack-infra07:42
*** jpena|off is now known as jpena07:46
*** markvoelker has joined #openstack-infra07:48
*** kopecmartin|off is now known as kopecmartin07:51
*** udesale has quit IRC07:53
*** udesale has joined #openstack-infra07:53
*** jpich has joined #openstack-infra08:02
*** gfidente has joined #openstack-infra08:02
*** dtantsur|afk is now known as dtantsur08:02
*** nicolasbock has quit IRC08:04
*** dchen has quit IRC08:04
*** tkajinam has quit IRC08:05
openstackgerritMark Meyer proposed zuul/zuul master: Extend event reporting  https://review.opendev.org/66213408:07
*** lucasagomes has joined #openstack-infra08:07
*** ociuhandu has joined #openstack-infra08:07
openstackgerritIan Wienand proposed opendev/system-config master: Pin ARA on devel job to stable branch  https://review.opendev.org/66452208:08
*** ralonsoh has joined #openstack-infra08:09
*** gregoryo has quit IRC08:12
*** liuyulong has joined #openstack-infra08:16
*** ykarel|lunch is now known as ykarel08:18
evrardjpcorvus: I know I am painful to reiterate on https://review.opendev.org/#/c/662828/ -- but do you have time to chat about this today? Having an out of the box thinking/solution for example :)08:19
*** markvoelker has quit IRC08:22
openstackgerritMerged openstack/project-config master: Add api-ref for Qinling  https://review.opendev.org/66403008:25
*** dayou has quit IRC08:28
*** yboaron has quit IRC08:33
*** yboaron has joined #openstack-infra08:33
openstackgerritMark Meyer proposed zuul/zuul master: Extend event reporting  https://review.opendev.org/66213408:35
*** roman_g has joined #openstack-infra08:36
*** imacdonn has quit IRC08:38
*** imacdonn has joined #openstack-infra08:38
openstackgerritTobias Henkel proposed zuul/zuul master: Parallelize github event processing  https://review.opendev.org/66281808:39
*** derekh has joined #openstack-infra08:40
*** dayou has joined #openstack-infra08:41
openstackgerritTobias Henkel proposed zuul/zuul master: Allow to select the merge method in Github  https://review.opendev.org/61794908:52
openstackgerritTobias Henkel proposed zuul/zuul master: Support squash merge in Github  https://review.opendev.org/66109608:52
*** priteau has joined #openstack-infra08:57
*** udesale has quit IRC08:59
*** rcernin has joined #openstack-infra09:02
*** udesale has joined #openstack-infra09:07
*** liuyulong has quit IRC09:11
*** iurygregory has quit IRC09:17
*** dayou has quit IRC09:18
*** markvoelker has joined #openstack-infra09:19
*** tobberydberg has quit IRC09:22
openstackgerritMark Meyer proposed zuul/zuul master: Extend event reporting  https://review.opendev.org/66213409:25
*** iurygregory has joined #openstack-infra09:26
*** nicolasbock has joined #openstack-infra09:26
*** iurygregory has quit IRC09:29
*** tobberydberg has joined #openstack-infra09:31
*** dayou has joined #openstack-infra09:32
*** yamamoto has quit IRC09:32
*** yamamoto has joined #openstack-infra09:37
openstackgerritMark Meyer proposed zuul/zuul master: Extend event reporting  https://review.opendev.org/66213409:45
openstackgerritDivya K Konoor proposed opendev/irc-meetings master: Update PowerVM Meeting timing  https://review.opendev.org/66453609:47
*** panda has quit IRC09:47
*** slaweq has joined #openstack-infra09:47
*** panda has joined #openstack-infra09:49
*** markvoelker has quit IRC09:52
*** ykarel is now known as ykarel|afk09:57
openstackgerritMark Meyer proposed zuul/zuul master: Extend event reporting  https://review.opendev.org/66213409:59
*** yamamoto has quit IRC10:03
*** yamamoto has joined #openstack-infra10:07
openstackgerritTobias Henkel proposed zuul/zuul master: Allow to select the merge method in Github  https://review.opendev.org/61794910:08
openstackgerritTobias Henkel proposed zuul/zuul master: Support squash merge in Github  https://review.opendev.org/66109610:08
openstackgerritFabien Boucher proposed zuul/zuul master: Disable gc in test_scheduler.TestExecutor  https://review.opendev.org/66131610:13
openstackgerritFabien Boucher proposed zuul/zuul master: Disable gc in test_scheduler.TestExecutor  https://review.opendev.org/66131610:14
openstackgerritFabien Boucher proposed zuul/zuul master: Pagure driver - https://pagure.io/pagure/  https://review.opendev.org/60440410:14
*** pcaruana has quit IRC10:19
*** yamamoto has quit IRC10:24
*** tdasilva_ has quit IRC10:25
openstackgerritMark Meyer proposed zuul/zuul master: Extend event reporting  https://review.opendev.org/66213410:31
*** slaweq has quit IRC10:31
*** bhavikdbavishi has quit IRC10:39
*** slaweq has joined #openstack-infra10:40
*** iurygregory has joined #openstack-infra10:44
*** markvoelker has joined #openstack-infra10:49
*** jchhatbar has joined #openstack-infra10:55
*** yamamoto has joined #openstack-infra10:56
*** janki has quit IRC10:56
*** ykarel|afk is now known as ykarel10:57
openstackgerritStephen Finucane proposed openstack/openstack-zuul-jobs master: Add support for building PDFs  https://review.opendev.org/66455510:58
*** slaweq has quit IRC10:59
*** jchhatbar has quit IRC10:59
*** janki has joined #openstack-infra10:59
*** janki has quit IRC11:00
*** yamamoto has quit IRC11:00
*** janki has joined #openstack-infra11:01
*** pcaruana has joined #openstack-infra11:09
*** udesale has quit IRC11:20
*** markvoelker has quit IRC11:21
*** dave-mccowan has joined #openstack-infra11:22
openstackgerritMark Meyer proposed zuul/zuul master: Extend event reporting  https://review.opendev.org/66213411:31
*** jpena is now known as jpena|lunch11:38
*** Goneri has quit IRC11:39
*** rcernin has quit IRC11:43
*** bhavikdbavishi has joined #openstack-infra11:45
*** janki has quit IRC11:51
*** janki has joined #openstack-infra11:51
*** yamamoto has joined #openstack-infra11:56
*** rlandy has joined #openstack-infra11:58
*** rlandy is now known as rlandy|ruck11:59
openstackgerritMark Meyer proposed zuul/zuul master: Extend event reporting  https://review.opendev.org/66213412:02
*** yamamoto has quit IRC12:04
*** dave-mccowan has quit IRC12:05
*** jcoufal has joined #openstack-infra12:12
*** goldyfruit has quit IRC12:17
*** jchhatbar has joined #openstack-infra12:21
openstackgerritKevin Carter (cloudnull) proposed zuul/zuul-jobs master: Add variable to set the docker download url  https://review.opendev.org/66448512:24
*** pkopec has joined #openstack-infra12:24
*** pkopec has quit IRC12:24
*** janki has quit IRC12:24
openstackgerritMerged opendev/irc-meetings master: Update PowerVM Meeting timing  https://review.opendev.org/66453612:24
*** ricolin_ has joined #openstack-infra12:27
*** jchhatbar has quit IRC12:27
*** jchhatbar has joined #openstack-infra12:27
*** ricolin has quit IRC12:29
*** slaweq has joined #openstack-infra12:30
*** gfidente has quit IRC12:32
*** rfarr__ has joined #openstack-infra12:32
*** gfidente has joined #openstack-infra12:34
*** rfarr_ has joined #openstack-infra12:35
*** rfarr__ has quit IRC12:38
*** eharney has quit IRC12:39
*** smarcet has joined #openstack-infra12:41
*** jpena|lunch is now known as jpena12:43
*** zbr|ruck is now known as zbr|rover12:44
*** slaweq has quit IRC12:46
*** jamesmcarthur has joined #openstack-infra12:51
openstackgerritSorin Sbarnea proposed zuul/zuul-jobs master: Adds tox-molecule job  https://review.opendev.org/66423012:54
*** udesale has joined #openstack-infra12:55
*** ekultails has joined #openstack-infra12:56
openstackgerritSorin Sbarnea proposed openstack/openstack-zuul-jobs master: Adds openstack-tox-molecule template  https://review.opendev.org/66359912:56
openstackgerritSorin Sbarnea proposed openstack/openstack-zuul-jobs master: Adds openstack-tox-molecule template  https://review.opendev.org/66359913:01
*** aaronsheffield has joined #openstack-infra13:01
*** rh-jelabarre has joined #openstack-infra13:01
*** jamesmcarthur has quit IRC13:02
cloudnullif folks have a moment to review this set https://review.opendev.org/#/q/topic:docker-multi-distro+status:open - I would greatly appreciate it13:03
openstackgerritSorin Sbarnea proposed openstack/openstack-zuul-jobs master: Adds openstack-tox-molecule template  https://review.opendev.org/66359913:04
ildikovsaw this on a docs build job output: "ERROR: Sphinx requires Python '>=3.5' but the running Python is 2.7.15"13:05
ildikovfull log: http://logs.openstack.org/05/664405/3/check/build-tox-manuals-checkbuild/07cc6e2/job-output.txt.gz13:06
ildikovpatch: https://review.opendev.org/#/c/664405/13:06
ildikovcan anyone help figuring out if this is environment related or some messed up settings in the repo?13:07
ildikovif it was brought up already then apologies :)13:07
*** nicolasbock has quit IRC13:07
*** nicolasbock has joined #openstack-infra13:08
zbr|roverildikov: sphinx 2 dropped support for py27. only two workarounds: either change jobs to use py3 or limit sphinx version.13:09
openstackgerritTobias Henkel proposed zuul/zuul master: Evaluate CODEOWNERS settings during canMerge check  https://review.opendev.org/64455713:10
ildikovzbr|rover: ah ok, that was one of my guesses, but wasn't sure13:10
ildikovzbr|rover: thanks!13:10
*** lseki has joined #openstack-infra13:14
*** mriedem has joined #openstack-infra13:16
*** tdasilva has joined #openstack-infra13:17
ildikovzbr|rover: quick dumb question13:18
ildikovzbr|rover: I saw in tox.ini that there's a basepython set to python3: https://opendev.org/openstack/training-guides/src/branch/master/tox.ini13:18
*** markvoelker has joined #openstack-infra13:18
ildikovzbr|rover: if I need to fix up something with the jobs themselves (https://opendev.org/openstack/training-guides/src/branch/master/.zuul.yaml) where can I locate where the changes need to be made?13:19
zbr|roverildikov: you need to add basepython = python3 to the tox env that fails (did not check which one is used)13:22
*** markvoelker has quit IRC13:23
ildikovzbr|rover: is the problem that basepython is specified under [testenv:venv] as opposed to just [testenv]?13:23
ildikovzbr|rover: as I assumed there're some dependencies between those sections13:24
zbr|roverildikov: don't touch the default one, venv is useless for CI.13:24
ildikovzbr|rover: the one that fails is checkbuild, but others will fail eventually too13:24
zbr|roverildikov: no deps unless manually added13:24
*** Goneri has joined #openstack-infra13:25
ildikovzbr|rover: for openstack-manuals the basepython is defined under [testenv] and only there: https://opendev.org/openstack/openstack-manuals/src/branch/master/tox.ini13:25
ildikovzbr|rover: so I assumed we would need to follow that model as well13:25
*** goldyfruit has joined #openstack-infra13:26
*** jamesmcarthur has joined #openstack-infra13:26
*** signed8bit has joined #openstack-infra13:26
*** eharney has joined #openstack-infra13:26
zbr|roveri guess that for manuals nobody will complain if we force python3 on all environments.13:27
zbr|roverwe already have goal of python3 first,....13:27
fungiyou _can_ set basepython=python3 in the main [testenv] definition but you need ignore_basepython_conflict=True set in the [tox] section to still be able to run things for other python interpreter versions which aren't your default python313:28
zbr|roverbut unless you mention the basepython, tox may pick py27 on some platforms.13:28
openstackgerritMark Meyer proposed zuul/zuul master: Extend event reporting  https://review.opendev.org/66213413:28
fungipretty sure the official openstack projects already got their docs jobs switched to run in python3 a while back13:28
ildikovwe didn't pick up on that in training-guides13:28
fungipossible the training-guides repo is not though13:28
ildikovI assume people thought the [testenv:venv] section is going to cut it13:29
*** yamamoto has joined #openstack-infra13:29
fungiahh, yeah not many projects use the build-tox-manuals job, so that one may simply have been missed in the python3 transition13:29
fungiand the projects which do may still be pinning sphinx<213:29
ildikovfungi: what's the best way forward?13:29
ildikovI don't want to pin the sphinx version unless we really need to13:30
ildikovas it'll be forgotten just like the prep work for the python 3 transition...13:31
fungii believe there's a variable we set in job definitions if we want to tell tox to use python313:31
fungifinding an example now13:32
ildikovnow I only need a little guidance on where that might be :)13:32
*** bhavikdbavishi has quit IRC13:33
*** slittle1 has joined #openstack-infra13:34
*** sthussey has joined #openstack-infra13:34
slittle1Are there any infrastructure cores that can take a look at ... https://review.opendev.org/#/c/663954/ ... Thanks, Scott13:35
fungiildikov: i see in some job variants we're passing in "sphinx_python: python3" for vars, but not seeing it set on normal docs jobs13:37
ildikovfungi: do those use what's set in [testenv]?13:38
ildikovas I saw basepython set to python3 there13:38
*** ykarel is now known as ykarel|away13:39
fungiildikov: yeah, so from what i can tell the docs jobs are running `tox -e docs` and, in nova for example, the [testenv:docs] section is just inheriting basepython = python3 from [testenv]13:39
ildikovfungi: so setting it in [testenv] for training-guides might not hurt either then?13:39
fungijust make sure ignore_basepython_conflict = True is set in the [tox] section and also increase minversion to something like 3.1.113:39
fungi(i think that's the first version which added the ignore_basepython_conflict option anyway)13:40
ildikovnoted, thanks!13:40
fungiildikov: give it a try and see what happens, anyway13:40
openstackgerritMark Meyer proposed zuul/zuul master: Extend event reporting  https://review.opendev.org/66213413:42
*** electrofelix has joined #openstack-infra13:44
ildikovfungi: gave it an attempt: https://review.opendev.org/66459513:45
ildikovfungi: there's another attempt here, but I would hope it's not that complicated to fix this: https://review.opendev.org/#/c/664545/13:48
*** ChosSimbaOne has joined #openstack-infra13:48
*** ChosSimbaOne has quit IRC13:48
*** ChosSimbaOne has joined #openstack-infra13:49
fungiyeah, that's basically pinning it for python 2.713:51
fungithe two approaches can also be combined13:51
*** jchhatbar has quit IRC13:52
*** tdasilva has quit IRC13:54
*** tdasilva has joined #openstack-infra13:55
*** ykarel|away has quit IRC13:55
ildikovok, cool13:56
ildikovjust realized that patch is for contributor-guide :)13:56
*** slaweq has joined #openstack-infra13:56
ildikovgood to know that we can make it bullet proof :)13:57
ildikovthanks fungi and zbr|rover !!13:57
*** sreejithp has joined #openstack-infra13:58
*** pcaruana has quit IRC14:04
*** liuyulong has joined #openstack-infra14:04
openstackgerritMark Meyer proposed zuul/zuul master: Extend event reporting  https://review.opendev.org/66213414:10
*** michael-beaver has joined #openstack-infra14:10
openstackgerritMark Meyer proposed zuul/zuul master: Extend event reporting  https://review.opendev.org/66213414:11
*** jpena is now known as jpena|afk14:14
*** iurygregory_ has joined #openstack-infra14:15
*** yamamoto has quit IRC14:15
*** yamamoto has joined #openstack-infra14:16
*** yamamoto has quit IRC14:16
*** iurygregory has quit IRC14:18
*** chandankumar is now known as raukadah14:23
rpiosojamesmcarthur: Have you seen the email I sent you?14:26
rpiosojamesmcarthur: It's about a change to driverlog which merged early Friday morning EDT -- https://review.opendev.org/#/c/663460/14:27
jamesmcarthurrpioso: yes sir. On my list :)14:27
rpiosojamesmcarthur: Thank you, Jimmy!14:27
*** slaweq has quit IRC14:28
*** pkopec has joined #openstack-infra14:30
*** pcaruana has joined #openstack-infra14:30
*** dave-mccowan has joined #openstack-infra14:30
*** yamamoto has joined #openstack-infra14:30
*** pcaruana has quit IRC14:31
*** pcaruana has joined #openstack-infra14:31
corvusevrardjp, clarkb, pabelanger: i'm trying to think of alternate approaches to https://review.opendev.org/662828 -- i'm frustrated that the date/time is not available to an untrusted zero-node job.  that seems like it would be generally very useful.  perhaps we should consider modifying zuul to stick that into the executor's fact cache?  or, add a new zuul. variable?14:32
openstackgerritMark Meyer proposed zuul/zuul master: Extend event reporting  https://review.opendev.org/66213414:32
corvusactually, let me repeat that in #zuul14:32
evrardjpcorvus: ok let me join there14:33
*** iurygregory_ is now known as iurygregory14:40
slittle1Are there any infrastructure cores that can take a look at ... https://review.opendev.org/#/c/663954/ ... Thanks14:44
*** happyhemant has joined #openstack-infra14:44
openstackgerritMark Meyer proposed zuul/zuul master: Extend event reporting  https://review.opendev.org/66213414:45
*** yamamoto has quit IRC14:45
*** yamamoto has joined #openstack-infra14:49
*** jpena|afk is now known as jpena14:51
*** yamamoto has quit IRC14:53
*** yboaron_ has joined #openstack-infra15:00
*** pkopec has quit IRC15:01
clarkbcloudnull: on the docker instal role the readme update was abandoned but the base change was not updated15:02
*** diablo_rojo has joined #openstack-infra15:02
*** yboaron has quit IRC15:02
cloudnullI'm rebasing all of that right now15:03
cloudnullthere were a couple things folks wanted changed so I figured i'd just squash the commits15:03
openstackgerritKevin Carter (cloudnull) proposed zuul/zuul-jobs master: Add multi-distro support to install-docker  https://review.opendev.org/66445515:05
openstackgerritKevin Carter (cloudnull) proposed zuul/zuul-jobs master: Add variable to set the docker download url  https://review.opendev.org/66448515:05
cloudnullnow just the two prs ^15:05
cloudnullianw: that last one implements the mirror fqdn replacement variable you noted in review 115:06
*** slaweq has joined #openstack-infra15:08
*** Lucas_Gray has joined #openstack-infra15:09
openstackgerritArnaud Morin proposed openstack/project-config master: Remove glean config for OVH nodes  https://review.opendev.org/66461715:12
*** lpetrut has quit IRC15:12
amorinhey all, about ^15:13
amorinfeel free to test it before15:13
clarkbamorin: ooh did that get fixed? tyty15:13
amorinto be sure that glean is working correctly with OVH network metadata15:13
amorinyes clarkb :p15:13
clarkbWe can test it easily before we update the nodepool config (by booting the image by hand) so we should do that15:13
amorinyes, please15:13
*** yamamoto has joined #openstack-infra15:15
*** yamamoto has quit IRC15:15
*** yamamoto has joined #openstack-infra15:16
*** hamzy_ is now known as hamzy15:17
*** pcaruana has quit IRC15:17
*** yamamoto has quit IRC15:20
*** bhavikdbavishi has joined #openstack-infra15:21
*** yamamoto has joined #openstack-infra15:22
*** yamamoto has quit IRC15:22
openstackgerritPaul Belanger proposed zuul/zuul master: WIP: Remove fact cache restriction for localhost (zuul-executor)  https://review.opendev.org/66462015:24
*** ociuhandu_ has joined #openstack-infra15:25
*** ociuhand_ has joined #openstack-infra15:25
*** ociuhandu_ has quit IRC15:26
*** yamamoto has joined #openstack-infra15:26
*** yamamoto has quit IRC15:26
*** ociuhandu has quit IRC15:28
*** ociuhand_ has quit IRC15:29
*** ociuhandu has joined #openstack-infra15:30
*** tdasilva has quit IRC15:31
*** tdasilva has joined #openstack-infra15:31
*** pgaxatte has quit IRC15:33
clarkbjust waiting for clarkb-test to boot in BHS1 now15:33
*** e0ne has quit IRC15:36
fungiamorin: is there any hope of including ipv6 configuration included in metadata? or does that still require manual configuration to use?15:37
clarkbcloudnull: one thing on the fqdn override. let me know what you think15:41
amorinfungi: hum good question15:41
amorinI think ipv6 should be ok also15:41
mriedemlooks like this can be abandoned now - i think it was an osic effort https://review.opendev.org/#/c/394659/15:41
amorinneed to be tested15:41
fungiamorin: neat! i'm going to reboot one of my personal ovh instances and check that out15:42
*** udesale has quit IRC15:42
*** pcaruana has joined #openstack-infra15:43
clarkbmriedem: done15:44
mriedemthanks15:44
openstackgerritJames E. Blair proposed zuul/zuul-jobs master: download-artifacts: only consider the most recent build  https://review.opendev.org/66462415:44
*** igordc has joined #openstack-infra15:44
clarkbinfra-root 158.69.76.93 was booted in bhs1 without setting the metadata flag and it got its IP configured15:45
clarkbthe netmask is 255.255.255.255 too as expected15:46
openstackgerritMerged openstack/project-config master: Add ansible-playbook repo to starlingx  https://review.opendev.org/66395415:46
*** eharney has quit IRC15:46
clarkbI don't see ipv6 data in the network_data.json15:46
fungiyeah, not finding it myself either15:47
clarkb(so unless we need to wait for an RA I don't think that is ready yet)15:47
clarkbbut I think we can merge amorin's change safely /me +2's now15:47
fungiagreed15:47
fungithanks again, amorin!15:47
cloudnullclarkb so that would do a full string replacement for "https://download.docker.com/linux/fedora" with "{{ docker_mirror_base_url }}"15:48
cloudnulland each distro would contain a default?15:48
clarkbcloudnull: ya I think so15:49
cloudnullcool15:49
cloudnullwill do15:49
*** jcoufal has quit IRC15:50
clarkbfungi: are you done looking at the test instance? I'll delete it if so15:50
fungiclarkb: yep, i was looking at a personal instance anyway15:52
clarkbdone15:52
*** pcaruana has quit IRC15:55
mriedemclarkb: this one as well https://review.opendev.org/#/c/278415/15:55
openstackgerritMerged openstack/project-config master: Remove glean config for OVH nodes  https://review.opendev.org/66461715:56
clarkbmriedem: ack15:57
clarkbI'm going to try and get through project-config reviews this morning as we've had several pings about them recently (people notice when ajaeger has vacation time)15:57
*** tdasilva_ has joined #openstack-infra15:58
*** eharney has joined #openstack-infra15:58
clarkbfungi: https://review.opendev.org/#/c/636574/ needs a rebase? should we just go ahead and do that and reapprove?16:00
openstackgerritMichal Nasiadka proposed opendev/system-config master: Add bionic to deb-docker releases in mirror_update  https://review.opendev.org/66462516:00
*** tdasilva has quit IRC16:00
*** gfidente has quit IRC16:01
fungiclarkb: i'll add a project group for that if we're rebasing anyway16:02
*** iurygregory has quit IRC16:02
*** yboaron_ has quit IRC16:03
*** gfidente has joined #openstack-infra16:03
openstackgerritJeremy Stanley proposed openstack/project-config master: Update karbor to use storyboard  https://review.opendev.org/63657416:08
yoctozeptoguys, we are seeing strange failures in kolla, related to tox job for lower constraints, examples:16:11
yoctozeptohttp://logs.openstack.org/16/642416/11/check/openstack-tox-lower-constraints/c713b37/ara-report/16:11
yoctozeptohttp://logs.openstack.org/80/664580/2/check/openstack-tox-lower-constraints/53dcd2d/ara-report/16:11
clarkbfungi: care to review the stack starting at https://review.opendev.org/#/c/663854/ ? I don't think we want to approve that one until the child changes merge and new mirrors are booted, but that change has the pointers to the children16:11
*** ykarel|away has joined #openstack-infra16:11
yoctozeptonothing related is changed, the errors seems to be irrelevant16:11
yoctozepto(success with a failure)16:11
*** ykarel|away has quit IRC16:12
clarkbyoctozepto: I think that means there is utf8 (or other unicode) output from the installation of packages perofrmed by tox and it can't print that16:13
*** lucasagomes has quit IRC16:13
openstackgerritKevin Carter (cloudnull) proposed zuul/zuul-jobs master: Add variable to set the docker download url  https://review.opendev.org/66448516:13
clarkbI would expect that to be locally reproduceable16:13
*** ykarel has joined #openstack-infra16:13
*** ykarel is now known as ykarel|away16:13
yoctozeptoclarkb: thanks, unfortunately it is not16:13
*** ginopc has quit IRC16:14
yoctozeptoseems like our master suddenly stopped passing this16:14
openstackgerritMerged zuul/zuul-jobs master: download-artifacts: only consider the most recent build  https://review.opendev.org/66462416:14
yoctozeptoI will try your hint manually to find the culprit16:14
yoctozeptobut it does not solve the issue really16:14
clarkbyoctozepto: see http://logs.openstack.org/16/642416/11/check/openstack-tox-lower-constraints/c713b37/tox/lower-constraints-1.log16:15
*** gfidente has quit IRC16:15
fungiyoctozepto: last successful build started at 17:03:40 yesterday, first failure started at 09:14:49 today according to http://zuul.opendev.org/t/openstack/builds?project=openstack%2Fkolla&job_name=openstack-tox-lower-constraints16:16
*** pkopec has joined #openstack-infra16:17
fungimy guess is updated image builds brought in some new behavior for libssl16:17
*** tesseract has quit IRC16:18
corvusand smart quotes16:18
*** tdasilva has joined #openstack-infra16:18
fungiyeah, fun that16:18
fungiyeah, maybe it's gcc switching to non-ascii quoting characters in warning messages?16:19
clarkbb'conflicting types for \xe2\x80\x98BIO_callback_ctrl\xe2\x80\x99\n   l' is the first substring it is breaking on16:19
fungithat seems like a strange behavior for a tls distro16:19
fungier, lts16:19
clarkb(I loaded the log file into python then decoded as ascii and boom with an index)16:19
fungi(ssl on the brain!_16:19
yoctozeptofungi, clarkb: very strange indeed, thank you guys for help16:20
fungibut yeah, that seems more like it would be a gcc thing16:20
clarkbdoes tox not respect our locale which is en_US.utf8 on our ubuntu systems?16:20
*** tdasilva_ has quit IRC16:20
yoctozeptoclarkb: then it respects it16:21
fungipossible it's missing from the environment16:21
yoctozeptoutf8 makes gcc unicode :-)16:21
yoctozeptoI believe LANG=C would help here, though there is something broken anyway16:21
fungitox does strip out a lot of envvars16:21
clarkbyoctozepto: no, tox itself is trying to encode the data from the exec as ascii16:22
clarkbyoctozepto: which isn't respecting that16:22
yoctozeptoclarkb: yeah, tox is not, I meant gcc16:22
yoctozeptoit should not produce unicode output with non-UTF8 locale16:22
yoctozepto(unless another bug)16:22
fungiyeah, i expect gcc sees a utf8-based locale or it wouldn't be generating non-ascii output... but tox is clearly expecting ascii for some reason16:24
*** pkopec has quit IRC16:25
openstackgerritSorin Sbarnea proposed zuul/zuul-jobs master: Adds tox-molecule job  https://review.opendev.org/66423016:25
openstackgerritSorin Sbarnea proposed zuul/zuul-jobs master: Adds tox-molecule job  https://review.opendev.org/66423016:25
clarkbyoctozepto: fungi it could be a new tox bug16:26
clarkbor at least the hiding of the data16:26
clarkbthe compile issue is likely unrelated to tox itself :)16:27
fungimaybe fixed in https://github.com/tox-dev/tox/pull/1237 (tox 3.8.4 and later)? are we using latest tox?16:27
*** pkopec has joined #openstack-infra16:28
fungithough i guess comparing to a successful build like http://zuul.opendev.org/t/openstack/build/8bd457005a29437fb67623b59d0b3f0f may help16:28
*** smarcet has quit IRC16:28
yoctozeptoclarkb: the compile obviously not, thanks for spotting, the creepy output scared me :-)16:28
clarkbfungi: I think dib is supposed to install latest tox16:29
clarkber actually no we don't install tox there anymore do we16:29
fungihttp://logs.openstack.org/06/634706/3/check/openstack-tox-lower-constraints/8bd4570/tox/lower-constraints-1.log does not show the compile errors for pyca/cryptography dependencies16:29
fungiso that could be what changed16:29
*** pkopec_ has joined #openstack-infra16:31
*** eernst has joined #openstack-infra16:31
fungiand since kolla's lower constraint for cryptography presumably has not been updated, this brings us back to new ubuntu packages in a recent image upload16:32
*** jamesmcarthur has quit IRC16:32
*** pkopec has quit IRC16:33
*** pkopec__ has joined #openstack-infra16:33
*** jpich has quit IRC16:33
fungithe successful build compiled extensions for cryptography-1.9 and the failing build was also using the cryptography-1.9 sdist16:33
yoctozeptofungi: yeah16:34
yoctozeptojust let know other kollars :-)16:34
yoctozeptoscrewed overnight16:34
openstackgerritMerged openstack/project-config master: Update karbor to use storyboard  https://review.opendev.org/63657416:34
fungihttp://changelogs.ubuntu.com/changelogs/pool/main/o/openssl/openssl_1.1.1-1ubuntu2.1~18.04.1/changelog doesn't indicate any changes to the openssl source package since december16:35
*** yamamoto has joined #openstack-infra16:36
*** ccamacho has quit IRC16:36
*** pkopec_ has quit IRC16:36
yoctozeptofungi: it did not try to compile16:37
yoctozeptothen16:37
yoctozeptodamned manylinux116:37
yoctozeptoit brought binary package16:37
fungior the failures are being introduced elsewhere and not by a change to openssl16:37
yoctozeptoseems like ubuntu stopped being recognized as compatible16:37
fungithough i just checked libffi-dev and it's also not seen changes since january16:38
yoctozeptofor manylinux116:38
*** whoami-rajat has quit IRC16:38
*** ociuhandu_ has joined #openstack-infra16:39
*** ociuhandu has quit IRC16:39
yoctozeptoor well, maybe not16:39
yoctozeptoit was building wheel16:39
yoctozeptobut succeeded16:39
fungiright16:39
yoctozeptoso many question, so little answers16:39
*** Lucas_Gray has quit IRC16:39
fungiwhich is why i'm suspecting an image update changing something about the c toolchain or libs16:39
fungijust trying to work out what that might have been16:40
clarkbif this wasn't lower constraints I would suspect a new sdist without a whl16:40
clarkbwe've seen that from cryptography and friends where an sdist gets uploaded up to a day or two in advance of the wheels16:40
clarkbbut with lower constraints we should be using older package versions16:41
yoctozeptowe get msgpack and cffi from manylinux116:41
fungithere won't be a manylinux1 wheel in this case16:41
yoctozeptoand then did so too16:41
fungiremember, lower constraints generally means "old"16:42
fungihttps://pypi.org/project/cryptography/1.9/#files16:42
fungino manulinux116:42
yoctozeptoyeah, I confirmed that, my bad there above16:42
clarkbaha so predates their wheel builds16:42
fungithis is from 2 years ago16:42
clarkb(for linux at least)16:42
*** gfidente has joined #openstack-infra16:42
*** yamamoto has quit IRC16:42
*** roman_g has quit IRC16:44
corvusi'm not having great success using paste16:45
*** Lucas_Gray has joined #openstack-infra16:45
*** ykarel|away has quit IRC16:49
*** slaweq has quit IRC16:49
*** pcaruana has joined #openstack-infra16:49
fungias in getting it to load? or getting it to save? or...?16:49
*** pkopec__ has quit IRC16:50
*** dtantsur is now known as dtantsur|afk16:51
slittle1How do I get added to group starlingx-ansible-playbooks-core ?16:51
slittle1Just created via https://review.opendev.org/#/c/663954/16:52
*** slaweq has joined #openstack-infra16:52
fungislittle1: the fastest way is by asking in here ;)16:54
slittle1yes please16:54
slittle1kinda assumed I'd be a member by default, being the one that requested it's creation   :)16:54
*** rpittau is now known as rpittau|afk16:55
fungiwe lack automation for setting initial group members, so it's a manual process. just a sec16:55
slittle1great, thanks16:55
*** slaweq has quit IRC16:56
fungislittle1: done. lmk if you need anything else16:57
slittle1another question if I may16:57
fungisure!16:57
fungiask away16:57
slittle1I'm trying to add the new project to a storyboard16:58
slittle1And a first task, assigned to myself16:59
slittle1it just seems to hang when I hit 'save'16:59
slittle1I'm logged in17:00
fungithis one? https://storyboard.openstack.org/#!/story/200469517:00
slittle1Same result from firefox and Chrome17:00
slittle1yes17:00
*** derekh has quit IRC17:01
fungithe task without the assignee is the one hanging when you try to assign it to yourself?17:01
yoctozeptoclarkb, fungi: https://www.ubuntuupdates.org/package/core/bionic/main/updates/openssl looks like they bumped up version17:01
yoctozeptoeh17:01
fungislittle1: i guess all of the17:02
fungier17:02
slittle1I'm trying to create the task with myself as assignee17:02
fungislittle1: i guess all of the "Initial Zuul and TOX setup" tasks repeated there are from multiple attempts?17:02
slittle1yes, probably17:02
*** weifan has joined #openstack-infra17:03
fungiso i guess the creation task call to the api is at least making it in17:03
openstackgerritMerged zuul/zuul-jobs master: Adds tox-molecule job  https://review.opendev.org/66423017:03
slittle1saw a pop up complaining about title='u' on first attempt17:03
slittle1had a '/' in the task title on the first try17:04
*** whoami-rajat has joined #openstack-infra17:05
fungiahh, i'm surprised / is disallowed on task titles17:05
slittle1not sure if that buggered up some behind the scenes database or scripting17:06
fungianyway, i just added a task on that story for myself (via firefox), though i did have to wait a few seconds... we're trying to improve some of the sql queries as they can take a lot of time to return17:06
*** jcoufal has joined #openstack-infra17:07
fungithe task with the / in its title is also showing up so i doubt it's disallowed17:07
slittle1oh!!!!17:07
*** ramishra has quit IRC17:07
slittle1it reload, now I see all my failed atempts... perhaps not failed after all17:07
fungiit's possible something went sideways with that first addition and caused the webclient to stop updating for you17:08
slittle1the browser just never scrolled or reformatted to show the new entries I had created after hitting 'save'17:08
fungiit did for me when i added a task, though it took 2-3 seconds17:09
slittle1perhaps it wasn't hung after all17:09
slittle1so I should expect it to scroll to the new entry ?17:10
*** kopecmartin is now known as kopecmartin|off17:10
*** jamesmcarthur has joined #openstack-infra17:10
fungiare you adding it via the "Add task affecting this project" link below the existing tasks?17:11
slittle1So now I have unwanted entries to clean up17:11
slittle1No,  first attempt was via '+ Affects other project'17:11
fungiahh, right, so it's possible the problem is coming from that widget17:12
slittle1when that through an error, I retired the same on each retry.17:12
fungion the unwanted tasks you can expand themm with the little red triangle to their left and there should be a delete button (which will pop up a confirmation modal for you to accept)17:12
slittle1so yah, no scroll or obvious redraw after '+ affects other projects' on success or fail17:14
fungii have a feeling this particular story is pushing the limits of usability with our current slow sql query issues, given there are a couple dozen tasks and hundreds of timeline entries17:15
fungicorvus helped us get slow query logging turned on and mordred pitched in with some analysis tools and we've got an outreachy intern coming up to speed to work on query optimization for the api17:16
fungiso hopefully this will speed up soon for largeish stories like that one17:17
*** jpena is now known as jpena|off17:17
slittle1did it again, sort of17:18
slittle1'+ Affects other project' to add starlingx/manifest, then fill in a first task and assign to myself, then click 'save'.17:19
slittle1No scroll, or redraw, or any other indicatin that the 'save' did anything17:20
fungicould be the save is still trying to finish. did the modal go away?17:20
slittle1but if I scroll up i see my new project and task, but it's unassigned17:21
fungioh, right, there's no separate modal, it's just in-list input controls17:21
slittle1I hadn't gone away when I scrolled up17:21
*** smarcet has joined #openstack-infra17:21
slittle1I hit refresh to see if the assignee would appear.  It didn't, so a re-added it17:22
*** bhavikdbavishi has quit IRC17:22
slittle1so scrolling down now... it is of course gone17:23
fungiit also wants to reload all the story content for me when i add a task, which on that story takes a good 10-20 seconds17:23
openstackgerritMatt McEuen proposed openstack/project-config master: Add airship/election  https://review.opendev.org/66464117:23
fungipresumably because of the associated timeline events17:23
slittle1k, maybe just not waiting long enough17:23
*** jamesmcarthur has quit IRC17:26
*** jamesmcarthur has joined #openstack-infra17:26
*** electrofelix has quit IRC17:27
fungiit doesn't do a good job in some places of indicating when it's still waiting for content to come back from an api call in the webclient17:27
fungicoupled with really long waits for that on some larger stories it can seem like it's completely gone out to lunch17:28
fungiwhich, as i mentioned, we're hoping will be fixed soon17:28
clarkbyoctozepto: and I'm guessing the version of cryptography in lower constraints is too old to build against that?17:29
fungithe api implementation has grown some really inefficient db queries which weren't apparent until we started loading increasing amounts of data in17:29
*** tosky has quit IRC17:30
yoctozeptoclarkb: exactly17:31
yoctozeptofortunately this is some old issue, cryptography is not used in kolla17:32
yoctozeptoprobably remainder after split17:32
fungiyoctozepto: oh! indeed, http://changelogs.ubuntu.com/changelogs/pool/main/o/openssl/openssl_1.1.1-1ubuntu2.1~18.04.1/changelog shows that changelog entry as being from december, which is why i didn't think it had updated recently. i guess it just took half a year to make it in17:32
yoctozeptofungi: maybe they even warned users via some channels :-)17:32
*** jcoufal has quit IRC17:32
fungibut yeah, we should be prepared for some things to suddenly stop being able to compile against openssl on bionic17:33
*** Lucas_Gray has quit IRC17:33
fungiyoctozepto: looking through https://bugs.launchpad.net/ubuntu/+source/openssl/+bug/1797386 it took 6 months to solve compatibility issues in openssl-linked packages in bionic17:35
openstackLaunchpad bug 1797386 in python-tornado (Ubuntu Bionic) "[SRU] OpenSSL 1.1.1 to 18.04 LTS" [Undecided,Fix committed]17:35
fungiand comment #27 covers when it was fixed there with python-cryptography 2.1.4-1ubuntu1.317:37
*** Lucas_Gray has joined #openstack-infra17:40
*** Lucas_Gray has quit IRC17:48
*** jcoufal has joined #openstack-infra17:50
*** ralonsoh has quit IRC17:53
*** priteau has quit IRC17:54
*** ociuhandu has joined #openstack-infra17:54
*** ociuhandu_ has quit IRC17:57
clarkbfungi: is https://review.opendev.org/#/c/631509/1 the sort of thing that requires a gerrit restart?17:58
clarkbmaybe we should get that in and the opendev gitweb links change(s) in then schedule a restart for maybe friday?17:58
clarkbmordred: ^17:58
*** ociuhandu has quit IRC17:59
*** jcoufal has quit IRC18:00
*** weifan has quit IRC18:02
fungiyes, i think commentlinks configuration requires a service restart18:03
*** weifan has joined #openstack-infra18:03
fungifriday seems as good a time as any18:03
yoctozeptointo production on Friday evening :-)18:05
*** eernst has quit IRC18:07
clarkbyoctozepto: problem is gerrit rereplicates all refs on restart so it is quite slow to recover after we restart it so best do it when it should be quiet18:07
fungibasically any change updated or merged after the restart don't replicate to the opendev.org gitea environment until the full replication backlog from the restart clears, which can take hours18:12
*** eernst has joined #openstack-infra18:13
*** weifan has quit IRC18:14
yoctozeptoI see, sad :-(18:14
*** weifan has joined #openstack-infra18:16
*** eernst has quit IRC18:17
*** eernst has joined #openstack-infra18:17
*** eernst has quit IRC18:18
fungiclarkb: should we add something like https://opendev.org/opendev/system-config/src/branch/master/playbooks/roles/gitea-git-repos/tasks/setup-repo.yaml#L27-L30 to https://opendev.org/opendev/system-config/src/branch/master/playbooks/gitea-rename-tasks.yaml or is that step covered somewhere i'm not finding?18:21
clarkbfungi: we probably do want that actually18:21
fungithough i guess the rename_repos playbook doesn't utilize projects.yaml so we may need another way to get at that conditional case18:22
clarkbcould be a flag in the rename.yaml18:23
*** smarcet has quit IRC18:23
openstackgerritDavid Shrewsbury proposed zuul/zuul master: Add caching of autohold requests  https://review.opendev.org/66341218:23
openstackgerritDavid Shrewsbury proposed zuul/zuul master: Add autohold-info CLI command  https://review.opendev.org/66248718:23
openstackgerritDavid Shrewsbury proposed zuul/zuul master: Record held node IDs with autohold request  https://review.opendev.org/66249818:23
openstackgerritDavid Shrewsbury proposed zuul/zuul master: Auto-delete expired autohold requests  https://review.opendev.org/66376218:23
openstackgerritDavid Shrewsbury proposed zuul/zuul master: Mark nodes as USED when deleting autohold  https://review.opendev.org/66406018:23
fungiclarkb: oh, also i guess since there's no actual api for this, we have to pretty much include all of the "Adjust repo settings" task dependencies18:24
*** weifan has quit IRC18:24
fungiwhich depend on a variety of data from projects.yaml18:24
fungimaybe it's easier to just rerun https://opendev.org/opendev/system-config/src/branch/master/playbooks/sync-gitea-projects.yaml instead?18:25
fungias a separate step?18:25
*** ekultails has quit IRC18:25
clarkbya and we can give it a subset of the data too18:26
clarkbso that it doesn't take 4 hours18:26
*** smarcet has joined #openstack-infra18:26
*** weifan has joined #openstack-infra18:29
openstackgerritDavid Shrewsbury proposed zuul/zuul master: Auto-delete expired autohold requests  https://review.opendev.org/66376218:30
openstackgerritDavid Shrewsbury proposed zuul/zuul master: Mark nodes as USED when deleting autohold  https://review.opendev.org/66406018:30
*** ChosSimbaOne has quit IRC18:32
fungianybody know why we would have been getting "E: Failed to fetch http://mirror.dfw.rax.opendev.org/ubuntu/dists/xenial-updates/main/binary-amd64/Packages.gz  Hash Sum mismatch" early today?18:32
clarkbfungi: thats the afs issue again18:33
*** ChosSimbaOne has joined #openstack-infra18:33
fungilooks like there were some for xenial-security as well18:33
clarkbI don't think we know why it is happening18:33
fungioh, right18:33
clarkbbut that removingthose files from the afs cache fixes it18:33
fungiokay, so it's recurrent anyway18:33
fungiianw was looking into custom afsclient packages?18:34
*** smarcet has quit IRC18:34
fungifor newer source?18:34
clarkbI think the next step was deploying a couple more mirrors to see if it happens to them too18:34
clarkbthat was the change I asked people to review earlier /me finds18:34
fungiyep, sorry, it's somewhere in my backlog then18:35
clarkbhttps://review.opendev.org/#/c/663854/ its parents18:35
clarkbthe change itself shouldn't be merged until we have the new mirrors up18:35
fungistack starting at 663854 you said earlier18:35
fungijust found in scrollback18:35
openstackgerritDavid Shrewsbury proposed zuul/zuul master: Auto-delete expired autohold requests  https://review.opendev.org/66376218:36
openstackgerritDavid Shrewsbury proposed zuul/zuul master: Mark nodes as USED when deleting autohold  https://review.opendev.org/66406018:36
*** ricolin_ has quit IRC18:38
openstackgerritTobias Henkel proposed zuul/zuul master: Add spec for enhanced regional executor distribution  https://review.opendev.org/66341318:38
openstackgerritMerged opendev/system-config master: Pin ARA on devel job to stable branch  https://review.opendev.org/66452218:39
*** pcaruana has quit IRC18:39
openstackgerritDavid Shrewsbury proposed zuul/zuul master: Auto-delete expired autohold requests  https://review.opendev.org/66376218:39
openstackgerritDavid Shrewsbury proposed zuul/zuul master: Mark nodes as USED when deleting autohold  https://review.opendev.org/66406018:39
*** jamesmcarthur has quit IRC18:40
*** ekultails has joined #openstack-infra18:42
*** tdasilva has quit IRC18:45
openstackgerritMatt Riedemann proposed opendev/elastic-recheck master: Add query for test_check_doubled_words py27 fail bug 1832392  https://review.opendev.org/66466118:46
openstackbug 1804062 in nova (Ubuntu Eoan) "duplicate for #1832392 test_hacking fails for python 3.6.7 and newer" [High,Triaged] https://launchpad.net/bugs/180406218:46
openstackgerritSean McGinnis proposed zuul/zuul master: Fix minor typos in Nodepool OpenStack instructions  https://review.opendev.org/66466518:49
*** smarcet has joined #openstack-infra18:50
openstackgerritPaul Belanger proposed zuul/zuul master: Bump default_read_timeout for github driver to 300  https://review.opendev.org/66466718:52
openstackgerritPaul Belanger proposed zuul/zuul master: WIP: Bump default_read_timeout for github driver to 300  https://review.opendev.org/66466718:52
*** priteau has joined #openstack-infra18:54
*** priteau has quit IRC18:54
*** markvoelker has joined #openstack-infra18:55
clarkbWe'll have our meeting in ~5 minutes. Join us in #openstack-meeting18:55
*** smarcet has quit IRC18:58
diablo_rojofungi, as a heads up I wont be around for the infra meeting to help with the sb update.18:58
fungino worries, and thanks for the heads up!18:59
*** jamesmcarthur has joined #openstack-infra18:59
openstackgerritMatt McEuen proposed openstack/project-config master: Add airship/election  https://review.opendev.org/66464119:01
*** jamesmcarthur has quit IRC19:01
*** jamesmcarthur has joined #openstack-infra19:03
*** jamesmcarthur has quit IRC19:05
*** jamesmcarthur has joined #openstack-infra19:07
openstackgerritJean-Philippe Evrard proposed zuul/zuul master: Expose date time as facts  https://review.opendev.org/66467419:09
openstackgerritJeremy Stanley proposed openstack/project-config master: Replace the old Stein cycle signing key with Train  https://review.opendev.org/66467519:10
openstackgerritMerged zuul/zuul master: Parallelize github event processing  https://review.opendev.org/66281819:14
*** jamesmcarthur has quit IRC19:15
fungilooks like those xenial package indices are still broken judging from http://logs.openstack.org/70/662870/8/check/zuul-stream-functional-2.5/301f28a/19:17
clarkbfungi: you could try doingthe fs flush on those paths19:18
fungiyeah, i was just extracting them out of root's shell hist19:19
openstackgerritMerged opendev/elastic-recheck master: Add query for test_check_doubled_words py27 fail bug 1832392  https://review.opendev.org/66466119:20
openstackbug 1804062 in nova (Ubuntu Eoan) "duplicate for #1832392 test_hacking fails for python 3.6.7 and newer" [High,Triaged] https://launchpad.net/bugs/180406219:20
*** igordc has quit IRC19:26
*** markvoelker has quit IRC19:29
*** diablo_rojo has quit IRC19:34
openstackgerritMerged opendev/system-config master: Switch mirror Apache logs to ISO8601  https://review.opendev.org/66380619:34
openstackgerritMerged opendev/system-config master: Add RAX IAD/ORD opendev.org mirrors  https://review.opendev.org/66385219:34
*** jamesmcarthur has joined #openstack-infra19:34
fungi#status log flushed /afs/openstack.org/mirror/ubuntu/dists/xenial-{security,updates}/main/binary-amd64/Packages.gz on mirror.dfw.rax19:37
openstackstatusfungi: finished logging19:37
*** alobbs has joined #openstack-infra19:38
jamesmcarthurrpioso: It appears both sets of drivers are still in the yaml: https://opendev.org/x/driverlog/raw/branch/master/etc/default_data.json19:42
jamesmcarthurrpioso: Can you add a patch to remove the old OpenStack Community drivers?19:42
rpiosojamesmcarthur: We want both pairs to exist.19:43
rpiosojamesmcarthur: More than one vendor can support a driver.19:45
*** eernst has joined #openstack-infra19:48
* rpioso will brb19:50
*** woojay has quit IRC19:52
fungior more than one driver can support the same vendor's hardware presumably19:52
*** eernst has quit IRC19:52
fungifor example when there's a basic driver included in the service but the vendor also maintains an out-of-tree driver with additional capabilities not suitable for inclusion upstream19:53
jamesmcarthurrpioso: Understood. That's going to require an update on our side. At the moment it's one vendor per driver.19:54
fungiso users may choose one over the other depending on what features they need out of it19:54
*** eernst has joined #openstack-infra19:54
*** woojay has joined #openstack-infra19:54
jamesmcarthurWe should be able to make it work :)19:55
fungiyeah, vendors<->drivers is a many:many relationship20:00
fungiat least in the real world20:00
*** psachin has quit IRC20:02
*** jamesmcarthur has quit IRC20:03
openstackgerritSean McGinnis proposed zuul/zuul master: Limit jobs runs with standard set of irrelevant files  https://review.opendev.org/66468020:07
*** tdasilva has joined #openstack-infra20:08
openstackgerritEvgeniy L proposed zuul/nodepool master: Add `floating-ip-pool` option to specify custom pool name  https://review.opendev.org/66468120:11
*** efried has joined #openstack-infra20:12
*** alobbs has quit IRC20:12
efriedHey folks. Any way we can get nova-tox-functional a node for 664622 ? This is a gate unblocker. All the other jobs are done and green, but this one's still queued.20:13
openstackgerritClark Boylan proposed opendev/system-config master: Check spamhaus pbl when launching new servers  https://review.opendev.org/66468220:13
clarkblunch is apparently going to be a few minuts so put ^ together20:13
clarkbefried: the way nodepool works, it will have to wait its turn20:13
*** smarcet has joined #openstack-infra20:13
efriedokay, thanks20:13
clarkb(we can't directly assign a node to a job, instead nodepool gives each provider three chances to successfully boot a node for each job before giving up. Chances are it is working through tryingto get a valid node)20:14
openstackgerritSean McGinnis proposed zuul/zuul master: Limit jobs runs with standard set of irrelevant files  https://review.opendev.org/66468020:15
efriedokay20:16
ianwclarkb: i'll get onto the openafs packages and keep track of thingsin https://etherpad.openstack.org/p/opendev-mirror-afs20:16
ianwhave to see about putting them in ppa; i don't want to mess up the existing arm64 packages in there20:17
rpiosojamesmcarthur: Gotcha and cool.20:21
*** jamesmcarthur has joined #openstack-infra20:25
*** markvoelker has joined #openstack-infra20:26
rpiosofungi, jamesmcarthur: Thank you!20:26
openstackgerritJames E. Blair proposed zuul/zuul-jobs master: download-artifacts: correct build selection  https://review.opendev.org/66468520:28
*** jamesmcarthur has quit IRC20:28
*** eharney has quit IRC20:32
* rpioso agrees the vendor to driver relationship is m:n in the wild20:35
*** smarcet has quit IRC20:36
*** jamesmcarthur has joined #openstack-infra20:53
*** markvoelker has quit IRC20:59
*** weifan has quit IRC20:59
*** Lucas_Gray has joined #openstack-infra20:59
*** weifan has joined #openstack-infra21:00
*** jamesmcarthur has quit IRC21:04
*** nicolasbock has quit IRC21:06
openstackgerritGuilherme  Steinmuller Pimentel proposed openstack/project-config master: Retire openstack-ansible unused roles  https://review.opendev.org/65042221:08
*** nicolasbock has joined #openstack-infra21:08
openstackgerritMerged zuul/zuul-jobs master: download-artifacts: correct build selection  https://review.opendev.org/66468521:12
openstackgerritMerged opendev/system-config master: Check spamhaus pbl when launching new servers  https://review.opendev.org/66468221:12
*** eernst has quit IRC21:17
*** mudpuppy has joined #openstack-infra21:18
*** eernst has joined #openstack-infra21:18
*** jtomasek has quit IRC21:22
*** eernst has quit IRC21:23
*** jtomasek has joined #openstack-infra21:24
clarkbI'll boot ne gitea06 as soon as ^ is on bridge21:24
clarkbmordred: I've just realized a flaw with our control plane images idea21:25
clarkbmordred: you cannot delete an imgae used on a boot from volume node until the volume is also deleted21:25
fungiso we have to keep those images ~indefinitely?21:26
clarkbya21:26
clarkbI'm not sure I like that with nodepool21:26
clarkbit will get unhappy21:26
clarkbI need to have a little think about that21:27
ianwhrm, when you "server list" you see recent servers list their image, but then older servers don't, i'd always assumed it was because the base image they booted from disappeared21:28
clarkbianw: ya it is, this particular problem only affects boot from volume servers which we only do in vexxhost21:28
*** ekultails has quit IRC21:28
openstackgerritMerged openstack/project-config master: Replace the old Stein cycle signing key with Train  https://review.opendev.org/66467521:29
*** igordc has joined #openstack-infra21:31
clarkbso I guess the concern is that we may end up with as many images as we have servers21:32
clarkbhowever, these images are much smaller than our test node images21:32
fungii expect more important is figuring out how to keep nodepool from freaking out over continually trying to delete them all21:33
clarkbthey are 3.0GB raw21:33
clarkbfungi: ya and it won't remove them from the builder disk until it deletes the image in the cloud21:34
clarkbI wonder can we "copy" an image21:34
fungiahh, right, so local disk space too21:34
clarkbso our process could be copy whatever image is newest from nodepool then boot off the copy21:35
clarkbthen nodepool can keep managing the image as it already does21:35
* clarkb looks at glance api docs21:35
clarkbno built in functionality for that but we could just upload the image on nb0* with a new name. If we do that we probably don't want nodepool to upload the images (just build them)21:36
*** sreejithp has quit IRC21:37
fungii suppose that's an option. if it doesn't upload them it doesn't have to worry about being able to delete them21:39
fungiand when we're launching a server we upload whatever the latest is21:39
fungiwhich is how i've generally been doing it for my personal servers21:39
*** nicolasbock has quit IRC21:39
fungiwhen it's only a few gb, that doesn't take long21:39
clarkbceph doesn't cow right? so I wonder if there is a way to convince novaand glance we are ok with the non boot from volume behavior21:40
*** diablo_rojo has joined #openstack-infra21:40
clarkbmnaser: ^ do you know if we can tell nova and/or glance it is ok to delete the image from glance when a running server is using that image?21:40
fungii am somewhat curious why our server images would be 3gb though... what extra are we installing? the debian openstack images are half a gb21:41
mnaserclarkb: nope, it doesn't do it because the cow leaves a reference21:41
clarkbfungi: 512MB is ext4 journal21:41
*** nicolasbock has joined #openstack-infra21:41
fungiahh21:41
fungiand that doesn't compress?21:42
clarkbfungi: not with raw images21:42
funginm, i was talking about qcow2 anyway21:42
ianwhave we turned off all caching?21:42
clarkbianw: for afs?21:42
ianwsorry i mean when building the images21:42
clarkbmnaser: right but wouldn't the reference be associated with the running instance(s) and no longer glance?21:43
fungiif it's just the cloud provider where images end up hanging around because we're doing manual uploads, then the raw size is mostly only relevant for the providers which require raw (so roughly just rax)21:43
clarkbfungi: and vexxhost21:43
fungioh, for some reason i thought we uploaded qcow2 there21:43
mnaserclarkb: nope, so glance store a snapshot (i.e. image-foo@snap) and then that snapshot is used to build a volume (disk-uuid)21:43
clarkbfungi: we did until we broke their cloud then switched to raw21:44
fungiahh21:44
mnaseryou cant delete image-foo@snap because disk-uuid uses it21:44
mnaser^ lols21:44
clarkbmnaser: that seems like poorly implemented cow21:44
clarkbbut there is probably some crush algorithm limitation I'm not aware of that leads to that21:44
mnaseryes, its a ceph level thing21:45
mnaserthere is a way to .. uh21:45
mnaser'un-cow-ify'21:45
fungimost cow implementations i'm familiar with have a flatten option where you copy all lower layers into the specified layer and then you can discard them once there are no remaining references to their blocks21:45
mnaseryes! flatten, but that's not really implemented in the openstack layer21:45
fungigot it21:45
clarkbfungi: to store the image in disk on ceph for use by nova you really want to store the raw image (because then you can serve the ceph blocks direclty)21:46
clarkbfungi: so when we were uploading qcow2 images they were convertin to raw behind the scenes21:46
fungiyeah, that makes sense21:46
clarkband that wasn't cheap21:46
*** diablo_rojo has quit IRC21:49
clarkbThinking about other options. If SDK/nodepool can detect that the image isn't being deleted because it is associated with an instance we could tell it to try less forcefully to clean that image up21:52
clarkbmaybe try deleting it once a day or something21:52
clarkbOf course if glance/ceph/nova were to have a bug and let us delete the image that might also nuke the instance(s)21:52
fungiand we could still allow cleaning up the on-disk copy21:53
clarkbmaybe the safest thing is for nodepool to detect that case, remove its local copy and the db record and kick the can down the road (eg manual cleanup of the image later)21:53
fungiif we can detect that the image is associated with an instance we could just avoid deleting it from the provider even if it's "expired"21:53
clarkbya21:54
fungialso possibly remove from np db21:54
fungiso expired images we don't keep the local copy or the db record but we skip the delete from providers if they have references to that image21:55
fungibonus points if we can do that just for bfv providers21:55
*** markvoelker has joined #openstack-infra21:56
clarkbnodepool already knows if it is booting from volume so we should be able to scope any special behavior to those cases21:56
*** armax has joined #openstack-infra22:00
mnaseri think dfw mirors are being iffy again?22:03
mnaserE: Failed to fetch http://mirror.dfw.rax.opendev.org/ubuntu/dists/bionic-updates/universe/binary-amd64/Packages.gz  File has unexpected size (1298819 != 1299303). Mirror sync in progress? [IP: 2001:4800:7819:105:be76:4eff:fe04:9b8a 80]22:03
mnaserhttp://logs.openstack.org/90/664690/1/check/openstack-tox-docs/70590cf/job-output.txt.gz22:04
*** diablo_rojo has joined #openstack-infra22:05
openstackgerritKevin Carter (cloudnull) proposed zuul/zuul-jobs master: Enhance upstream installation mode  https://review.opendev.org/66471122:06
*** kjackal has quit IRC22:06
clarkbmnaser: I'll rerun the flushes22:07
* clarkb makes a script22:07
corvusianw: should we look into kafs for the mirror heads?22:09
*** rcernin has joined #openstack-infra22:10
ianwcorvus: it may be an option, but i think it will be quite bespoke in terms of packages, etc.  i started playing with jobs at one point -> https://review.opendev.org/#/c/623974/22:11
clarkb/home/clarkb/fs_flush.sh on the mirror node for flushing bionic, xenial, trusty repo control files22:11
ianwIIRC the fedora packages were a bit tangled up and some work was going to be done on getting the client bits (python?) working better ... that's about where i stopped22:12
*** rfarr_ has quit IRC22:12
corvusis it not in debuntu?22:12
clarkbcloudnull: I don't think we should fall back with docker installations22:13
clarkbcloudnull: if you want upstream packaging then the job should fail if you don't get it22:13
cloudnullI was kinda trying to implement https://docs.docker.com/install/linux/docker-ce/ubuntu/#uninstall-old-versions as a fall back plan22:14
cloudnullhowever I'd be happy to force the role into one path or the other22:14
ianwcorvus: i do not believe so22:15
corvus:(22:16
cloudnullclarkb - If you think it best I can squash these three commits and make that process change ?22:16
ianwwe do, however, have a pretty great framework for doing a bunch of CI ...22:16
clarkbcloudnull: I think the base change is fine as is, it doens't regress old behavior22:16
* prometheanfire needs to look at zuul docs for what backends it supports22:17
corvusprometheanfire: what meaning of the word "backend" do you have in mind?22:17
cloudnullclarkb ok22:17
prometheanfirewhere can it spin up VMs, openstack sure, but ganeti?22:17
cloudnullclarkb - I'll update that latest change22:17
*** ChosSimbaOne has quit IRC22:18
clarkbcorvus: ianw when not concerned about afs things would be curious about your thoughts on the control plane image thing discussed above. I don't want to boot itea06 if it will make nodepool sad22:18
corvusprometheanfire: ah, that's the drivers section in https://zuul-ci.org/docs/nodepool/22:18
prometheanfirethanks22:19
corvusclarkb: wow, that's very unpleasant.22:21
*** slaweq has joined #openstack-infra22:22
corvusclarkb: i don't think we should subject our production zuul-nodepool to that.  i think it will be very hard to disentangle our control-plane from our data-plane if we do.22:24
clarkbya22:25
corvusclarkb: i think we should find a way to mitigate that (improve sdk/nodepool; separate nodepool instance; improve openstack) in the long term and in the short term maybe manually manage some images or something.22:25
clarkbwe can probably manually upload the latest image that has already been built and that will freeze in time for a bit but that should be fine22:25
corvusyeah, first thing we do is upgrade22:26
openstackgerritKevin Carter (cloudnull) proposed zuul/zuul-jobs master: Isolate installation mode  https://review.opendev.org/66471122:29
*** markvoelker has quit IRC22:30
clarkbmy opensuse kernel seems to have a kafs.ko22:30
cloudnullclarkb ^ that will make it so the role does one or the other, no more distro fall back22:30
*** slaweq has quit IRC22:30
ianwclarkb: sorry if silly question, but how do we delete testing node images then; are we relying on everything moving to the new image before we come back and remove the old one next day?22:31
clarkbianw: ya I think nodepool is doing its best then as all the old nodes go away it cleans them up22:31
ianwclarkb: yeah, it's more the tools ontop that don't seem to get a lot of packaging love22:32
clarkbianw: ah fwiw I don't see kafs in the bionic kernel modules list22:32
ianwhrm, so it sort of works there maybe a bit more by luck than design?22:32
clarkbbut maybe we can install a package22:32
clarkbianw: ya22:32
openstackgerritKendall Nelson proposed opendev/storyboard-webclient master: Enabling suggestions on all user fields  https://review.opendev.org/64852522:32
clarkb/lib/modules/5.1.7-1-default/kernel/fs/afs on my opensuse machine fwiw22:33
openstackgerritKendall Nelson proposed opendev/storyboard-webclient master: Enabling suggestions on all user fields  https://review.opendev.org/64852522:34
ianwclarkb / corvus: so AFAICS https://copr.fedorainfracloud.org/coprs/jsbillings/kafs/packages/ is still pretty much the home of the client utils; which is what was being tested @ https://review.opendev.org/#/c/623974/22:38
corvusianw, clarkb: we shouldn't need any tools for a web head, right?22:39
corvusit's all just anonymous access22:39
clarkbright now we are using the fs flush tools, but otherwise ya I think its just mount and read data (with cache)22:39
fungii have a /lib/modules/4.19.0-5-amd64/kernel/fs/afs/kafs.ko on my (debian unstable/sid) workstation22:40
fungiso seems like the ubuntu kernels would be picking it up too22:40
corvusi also have that on my bionic workstation22:40
corvusi'm not sure i can easily test it atm, because i'm running openafs22:41
openstackgerritKendall Nelson proposed opendev/storyboard-webclient master: Allowing the user to choose what Columns are seen  https://review.opendev.org/65127022:41
fungicorvus: do you maybe have a hwe kernel?22:42
fungi/lib/modules/4.15.0-51-generic/kernel/fs/ on mirror.dfw.rax.opendev.org does not have an afs tree22:42
fungiwhat kernel version are you on?22:42
fungiother possibility is it's coming from a kernel modules package22:43
corvusfungi: no hwe.  dpkg says linux-modules-extra-4.15.0-51-generic:22:43
fungiwacky22:43
fungibut maybe that package22:44
fungiwe only have linux-modules-4.15.0-51-generic installed, no extra22:44
corvusmight be worth a try :)22:45
corvusactually, it may be easy for me to test22:45
ianwyeah, we at least need like the client etc to get the mounts setup ... something i just found https://gitlab.umich.edu/jsbillin/ansible-kafs-workstation/blob/master/roles/kafs/tasks/main.yml22:45
clarkbre the control plane image thing I think I'll hold off on gitea06 for now and we can sync up with mordred when he is around. I think my suggestion will be to take whatever image is current for rax and vexxhost bionic minimal, reupload that under opendev-ubuntu-bionic-$DATE then remove the config from the existing builders22:48
clarkbwant to make sure mordred is on board since he was driving a lot of this22:48
fungiit looks to me like the newer openafs tools rely on libkafs2 to interact with the kafs implementation in kernel22:48
fungiwhich doesn't seem to have made it to ubuntu as far back as bionic22:49
fungior at all ever22:50
fungioh, wait, it's libkopenafs222:50
fungiyeah, bionic has it: https://packages.ubuntu.com/bionic-updates/libkopenafs222:50
fungithe package description there is scant22:51
clarkbkopenafs is a funny name for it22:52
clarkbI wonder if it is shared between implementations?22:52
fungidebian's clarifies "a subset of the interface provided by the Heimdal libkafs library"22:52
fungiso maybe that's not the same thinh22:52
fungior maybe heimdal added kafs support earlier and openafs is reconsuming it for that purpose?22:52
fungirelated presumably to what's packaged in libkafs0-heimdal22:53
fungiwhich says "Heimdal Kerberos - KAFS support library"22:54
fungi"This package contains the library for KAFS support."22:54
*** rkukura_ has joined #openstack-infra22:54
*** auristor has quit IRC22:54
fungiso, yeah, i think the idea must be openafs userspace calling kafs implementation in linux kernel via libkopenafs2 library interface22:54
*** whoami-rajat has quit IRC22:55
ianwright, looking back through my notes and chat logs; at the time, the kafs packages in that copr were getting confused between "/afs" and "/kafs" and changing the mount point was on the "next" branch22:55
*** rkukura has quit IRC22:57
*** rkukura_ is now known as rkukura22:57
fungihuh, this has been around for ages22:58
*** eernst has joined #openstack-infra22:59
corvusthis works: http://paste.openstack.org/show/752781/22:59
*** gfidente has quit IRC22:59
fungiopenafs source package changelog from debian for 1.5.73.3-1 (2010) indicates addition of a "New libkopenafs library providing a kernel PAG manipulation interface (a subset of the API provided by Heimdal's libkafs)."22:59
corvuskafs is supposed to have dynamic root support, but that didn't work so great on my workstation, it's possible that's too new for bionic23:00
*** eernst has quit IRC23:00
*** eernst has joined #openstack-infra23:00
corvusbut if it's more stable, i think we can live with a non-dynamic root23:00
clarkbis that what the stuff after the # is for?23:00
corvusyeah23:00
corvusthat's "mount the root.afs volume in the openstack.org cell (previously defined) as the afs root volume, preferring read-only volumes"23:01
fungii don't see us likely adding other afs roots to our mirror hosts anyway23:01
corvusif the dynamic stuff worked, it'd be "mount -t afs none /afs -o dyn"23:01
corvusdocumented here: https://www.kernel.org/doc/Documentation/filesystems/afs.txt23:01
corvusbut the dyn option got me: [4351958.562340] kAFS: Unknown or invalid mount option: 'dyn'23:02
corvusanyway, those 3 commands are all we should need for a web head23:02
* clarkb tries here23:03
clarkbdo I need to specify the rootcell on the modprobe if attempting the dynamic option?23:03
corvusclarkb: i would try without23:03
*** tkajinam has joined #openstack-infra23:04
clarkbseemed to work23:04
clarkbno errors, do get auto mount like behavior under the first level of /afs though23:05
fungithat's with linux 5.1?23:05
clarkbso have to cd into /afs/openstack.org before it'll list that dir23:05
clarkbfungi: ya23:05
fungivs corvus's 4.1523:05
clarkb5.1.2-1-default23:05
corvusif anyone wants to make a kernel commit -- that documentation refers to 3 kernel modules but only shows installing 2...23:05
*** eernst has quit IRC23:05
corvusclarkb: cool, that's how i would expect it to function23:05
*** auristor has joined #openstack-infra23:06
*** mriedem has quit IRC23:06
fungimaybe i'll have time to try with the 4.19 i'm running23:06
corvusso if the cache manager is any better, then we could live with the manually configured form in bionic and switch to the dynamic one next time23:06
clarkbseems like it would be worth a try at least23:07
clarkbianw: ^ what do you think23:07
corvusi guess to test this, we should spin up another mirror host and swap dns?  either that, or pause rax-dfw in nodepool and manually switch out cache managers on that host?23:08
clarkbcorvus: we can probably use one of the new iad or ord mirros that isn't being used yet23:08
clarkbthen we'll have side by side comparison too23:08
corvusoh, i forgot they were booted but not in prod.  ++23:08
*** Lucas_Gray has quit IRC23:08
openstackgerritIan Wienand proposed opendev/system-config master: [DNM] kafs testing  https://review.opendev.org/62397423:10
fungiremember we don't yet know that the cache problem is related to bionic vs just something going wrong with the cinder volume attached to the new dfw mirror23:10
clarkbfungi: ya we can probably do ord on openafs, iad on kafs (or vice versa) then we'll be testing both things23:11
*** rlandy|ruck has quit IRC23:14
clarkbcd'ing into the ubuntu mirror dir with my kafs mount doesn't seem to be happening quickly23:14
*** goldyfruit has quit IRC23:14
clarkbthat could be my NAT and wifi though23:14
ianwyeah, i'm just looking at what the caching situation is23:14
*** smarcet has joined #openstack-infra23:14
corvusclarkb: yeah, that's normal for even the openafs client23:16
ianwso we want cachefilesd installed23:19
*** smarcet has quit IRC23:20
ianwhow about we put iad in disabled and fiddle on that?  tracking on https://etherpad.openstack.org/p/opendev-mirror-afs23:22
clarkbwfm23:22
*** factor has joined #openstack-infra23:22
*** smarcet has joined #openstack-infra23:23
*** smarcet has quit IRC23:26
*** markvoelker has joined #openstack-infra23:26
openstackgerritIan Wienand proposed opendev/system-config master: Fix syntax for new mirror hosts  https://review.opendev.org/66474523:35
ianwinfra-root: ^ ... :/23:35
ianwthat explains why the new hosts didn't seem to be running ansible against them, anyway23:35
fungii single-core approved it23:37
fungias i suspect it was cutting the inventory in ~half and only getting us ansible runs against the hosts before that23:38
*** goldyfruit has joined #openstack-infra23:40
ianwi feel like we can lint a no-op thing and at least validate the syntax of the inventory23:40
ianwi will look into that as penance for breaking it :)23:41
ianwon the plus side, makes it easy to fiddle on iad rax mirror because nothing's happened just yet23:41
fungiheh. at an old job we used to practice what i liked to call "punitive documentation"23:44
fungiif you broke something, it was your job to document how not to break it that way in the future23:44
*** ianychoi has quit IRC23:48
*** liuyulong has quit IRC23:48
*** ianychoi has joined #openstack-infra23:48
cloudnullany chance I could get - https://review.opendev.org/#/q/topic:docker-multi-distro+(status:open) reviewed? - sorry for the pestering, I'm working on trying to get to the point where i can use the new tox-molecule job w/ `tripleo-ansible`, but I need this docker-install role to work on cent / fedora.23:50
*** aaronsheffield has quit IRC23:50
*** weifan has quit IRC23:51
ianwcloudnull: sure, i had that up, thanks for adding docs.  that symlink had me confused for a bit too :)23:53
*** lseki has quit IRC23:53
*** diablo_rojo has quit IRC23:53
*** smarcet has joined #openstack-infra23:54
*** sthussey has quit IRC23:54
cloudnullyeah, i kinda wish gerrit represented links like that a little better.23:54
*** tdasilva_ has joined #openstack-infra23:54
cloudnullthough maybe I also should have called that out in the comments23:55
*** jcoufal has joined #openstack-infra23:55
*** dchen has joined #openstack-infra23:56
*** tdasilva has quit IRC23:57
*** jcoufal has quit IRC23:57
*** hwoarang has quit IRC23:58
*** nicolasbock has quit IRC23:58
*** jamesmcarthur has joined #openstack-infra23:58

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