Wednesday, 2019-01-16

fungiclarkb: i just now approved 63108200:00
clarkbfungi: thanks00:00
*** wolverineav has joined #openstack-infra00:00
clarkbI'm still here watching things00:00
*** yamamoto has joined #openstack-infra00:01
*** tosky has quit IRC00:08
*** wolverineav has quit IRC00:08
*** rascasoft has joined #openstack-infra00:09
*** wolverineav has joined #openstack-infra00:11
*** rascasoft has quit IRC00:14
*** roman_g has quit IRC00:19
*** hamzy_ has joined #openstack-infra00:20
*** yamamoto has quit IRC00:22
ianwfungi:  https://review.openstack.org/630889 Install from bindep.txt in plugins -- is basically where i ended up from our disucssion yesterday00:23
*** harlowja has joined #openstack-infra00:23
fungineat!00:27
fungilooks like a good solutio00:28
fungin00:28
openstackgerritIan Wienand proposed openstack-infra/nodepool master: Use bindep.txt with devstack  https://review.openstack.org/63089000:30
openstackgerritIan Wienand proposed openstack-infra/nodepool master: [wip] Use bionic for functional testing  https://review.openstack.org/63110200:30
ianwfungi: ^ and yeah, so that's the point of the whole thing :)00:30
*** yamamoto has joined #openstack-infra00:31
fungii'm still curious whether it would work to have a dib devstack plugin which the nodepool devstack plugin depended on, which could then provide its own bindep.txt to cover dib runtime deps just for devstack00:31
fungibut maybe that's a lot of hassle for just a package list00:32
*** yamamoto has quit IRC00:33
*** yamamoto has joined #openstack-infra00:34
ianwthat doesn't really solve the problem though, because in one case we want to install from release/pypi, and in the other (-src job) from source00:35
ianwso it just becomes the devstack's plugin problem ... does it use the files/bindep.txt file, or the one from the chekced out dib00:36
fungiwell, meaning that the dib bindep plugin could carry a list of dependencies relevant to exercising dib under devstack specifically, and then not bother using the bindep.txt in the dib repo for devstack00:37
ianwah, sure.  i mean, we could make devstack look for devstack/files/bindep*.txt and do a split up like that too, to keep things a bit more separate00:39
ianwit was kind of annoying to me, when i had "bindep.txt" in the master branch is the binary dependencies required for this package -- so duplicating it, and not being able to test it in situ was annoying.  but i see your point that environments should provide their own bindep.txt as appropriate00:40
*** yamamoto has quit IRC00:40
ianwwhat i do *not* want to do is have any more bespoke "do this on this platform" matching in devstack, when bindep does that perfectly well00:41
*** yamamoto has joined #openstack-infra00:41
fungirelying on bindep.txt inside the sdist of course doesn't help if the packages listed are required to be able to install the sdist (libssl headers, et cetera)00:42
openstackgerritMerged openstack-infra/system-config master: Use correct 2.13.12 war on review-dev  https://review.openstack.org/63108200:42
fungiwhich is why that wasn't really a prime use case for bindep00:43
fungior at least for in-repo bindep lists anyway00:43
clarkbianw: ++ to dropping the files/ stuff00:44
ianwyeah, unless it was published on pypi and pip sourced it prior to install for all packages then deduped ... well we can dream :)00:44
*** jklare has quit IRC00:44
*** calbers has quit IRC00:44
*** yamamoto has quit IRC00:45
fungicould embed it in the long description field ;)00:46
*** yamamoto has joined #openstack-infra00:47
ianwi can't get anyone to respond to my proposal to output the sha of built wheels into the logs, https://github.com/pypa/pip/pull/5908 , i can't see new fields etc being ever possible :/00:49
*** jklare has joined #openstack-infra00:52
*** yamamoto has quit IRC00:55
*** owalsh_ has joined #openstack-infra00:57
*** yamamoto has joined #openstack-infra00:57
*** owalsh has quit IRC00:58
*** eernst has quit IRC01:03
*** rascasoft has joined #openstack-infra01:03
clarkbpuppet run in ~40 minutes should apply the new war01:07
*** whoami-rajat has joined #openstack-infra01:08
*** rascasoft has quit IRC01:09
*** wolverineav has quit IRC01:09
*** owalsh_ has quit IRC01:09
*** owalsh has joined #openstack-infra01:10
*** wolverineav has joined #openstack-infra01:12
*** gregoryo has joined #openstack-infra01:13
*** jamesmcarthur has joined #openstack-infra01:15
*** yamamoto has quit IRC01:18
*** jamesmcarthur has quit IRC01:20
*** yamamoto has joined #openstack-infra01:22
*** yamamoto has quit IRC01:28
*** hwoarang has quit IRC01:30
*** hwoarang has joined #openstack-infra01:32
openstackgerritClint 'SpamapS' Byrum proposed openstack-infra/nodepool master: Amazon EC2 driver  https://review.openstack.org/53555801:35
*** yamamoto has joined #openstack-infra01:37
openstackgerritMerged openstack-infra/storyboard master: Rationalise worklist ordering code  https://review.openstack.org/60047201:38
openstackgerritMerged openstack-infra/storyboard master: Fix the stories relation in StoryTag  https://review.openstack.org/62104501:38
clarkbok review-dev seems mostly happy01:39
clarkbI did forget to do the chown because puppet won't do it unless the exec runs01:39
clarkbso that means doing the upgrades again as they will be done on production is extra important, but other than that it seems to be happy01:40
*** yamamoto has quit IRC01:42
*** yamamoto has joined #openstack-infra01:42
fungiexcellent!01:45
clarkb(the init failed but thats ok between basically the same version of gerrit)01:45
*** bobh has joined #openstack-infra01:48
*** wolverineav has quit IRC01:50
*** wolverineav has joined #openstack-infra01:51
*** wolverineav has quit IRC02:00
*** wolverineav has joined #openstack-infra02:01
*** hongbin has joined #openstack-infra02:03
openstackgerritMerged openstack-infra/nodepool master: Document default values of runtime arguments  https://review.openstack.org/63067902:04
*** hwoarang has quit IRC02:07
*** hwoarang has joined #openstack-infra02:07
*** wolverineav has quit IRC02:08
*** yamamoto has quit IRC02:08
*** rh-jelabarre has quit IRC02:12
*** mriedem has quit IRC02:16
*** ijw has quit IRC02:23
*** ijw has joined #openstack-infra02:25
*** ijw has quit IRC02:29
*** ramishra has quit IRC02:40
*** bgmccollum has joined #openstack-infra02:43
*** ijw has joined #openstack-infra02:47
*** yamamoto has joined #openstack-infra02:47
*** yamamoto has quit IRC02:53
openstackgerritIan Wienand proposed openstack-infra/nodepool master: [wip] Use bionic for functional testing  https://review.openstack.org/63110202:57
*** psachin has joined #openstack-infra02:59
*** ijw has quit IRC03:04
*** d0ugal has quit IRC03:10
*** apetrich has quit IRC03:13
*** ykarel has joined #openstack-infra03:21
openstackgerritSagi Shnaidman proposed openstack-infra/nodepool master: Support userdata for instances in openstack  https://review.openstack.org/63064903:26
*** smarcet has quit IRC03:31
openstackgerritlijunjie proposed openstack-infra/gerrit master: Fix the misspelling of "language"  https://review.openstack.org/63112403:33
openstackgerritlijunjie proposed openstack-infra/groups master: Fix the misspelling of "language"  https://review.openstack.org/63112503:36
openstackgerritIan Wienand proposed openstack-infra/nodepool master: [wip] Use bionic for functional testing  https://review.openstack.org/63110203:41
*** rcernin has quit IRC03:46
*** eernst has joined #openstack-infra03:46
*** rcernin has joined #openstack-infra03:48
*** ykarel has quit IRC03:48
*** diablo_rojo has quit IRC03:48
*** udesale has joined #openstack-infra03:50
*** bobh has quit IRC03:54
*** eernst has quit IRC03:56
*** bobh has joined #openstack-infra04:06
*** yamamoto has joined #openstack-infra04:07
*** ramishra has joined #openstack-infra04:09
openstackgerritSagi Shnaidman proposed openstack-infra/nodepool master: Support userdata for instances in openstack  https://review.openstack.org/63064904:10
*** yamamoto has quit IRC04:13
openstackgerritIan Wienand proposed openstack/diskimage-builder master: [wip] rhel8 beta support  https://review.openstack.org/62313704:17
*** bobh has quit IRC04:33
*** ykarel has joined #openstack-infra04:43
*** yamamoto has joined #openstack-infra04:43
*** dave-mccowan has quit IRC04:44
*** bobh has joined #openstack-infra04:52
*** sshnaidm has quit IRC04:52
*** eernst has joined #openstack-infra04:53
*** sshnaidm has joined #openstack-infra04:55
*** ramishra has quit IRC04:56
*** bobh has quit IRC04:57
*** hwoarang has quit IRC05:00
*** hwoarang has joined #openstack-infra05:01
*** d0ugal has joined #openstack-infra05:04
*** ykarel has quit IRC05:08
*** ramishra has joined #openstack-infra05:11
*** rcernin has quit IRC05:17
*** wolverineav has joined #openstack-infra05:18
*** rcernin has joined #openstack-infra05:19
*** ykarel has joined #openstack-infra05:25
openstackgerritIan Wienand proposed openstack-infra/nodepool master: Switch devstack jobs to bionic  https://review.openstack.org/63110205:35
*** udesale has quit IRC05:35
*** udesale has joined #openstack-infra05:40
*** wolverineav has quit IRC05:48
*** wolverineav has joined #openstack-infra05:48
*** wolverineav has quit IRC05:57
*** rcernin has quit IRC05:58
*** rcernin has joined #openstack-infra06:00
*** bobh has joined #openstack-infra06:04
*** bobh has quit IRC06:09
*** eernst has quit IRC06:10
openstackgerritSagi Shnaidman proposed openstack-infra/nodepool master: Support userdata for instances in openstack  https://review.openstack.org/63064906:13
*** hongbin has quit IRC06:16
openstackgerritOpenStack Proposal Bot proposed openstack-infra/project-config master: Normalize projects.yaml  https://review.openstack.org/63114206:19
*** wolverineav has joined #openstack-infra06:22
*** yamamoto has quit IRC06:26
*** armax has quit IRC06:27
*** kjackal has joined #openstack-infra06:30
*** yboaron has joined #openstack-infra06:34
*** kjackal has quit IRC06:50
*** e0ne has joined #openstack-infra06:50
*** diablo_rojo has joined #openstack-infra06:54
*** yamamoto has joined #openstack-infra07:03
*** rcernin has quit IRC07:04
*** lpetrut has joined #openstack-infra07:06
*** yamamoto has quit IRC07:08
*** wolverineav has quit IRC07:08
*** udesale has quit IRC07:11
*** pcaruana has joined #openstack-infra07:12
*** quiquell|off is now known as quiquell07:13
*** wolverineav has joined #openstack-infra07:15
spsuryacan anyone please +w https://review.openstack.org/#/c/629150/07:17
spsuryafungi: clarkb07:17
*** jtomasek has joined #openstack-infra07:19
*** jtomasek has quit IRC07:20
*** jtomase__ has joined #openstack-infra07:21
*** udesale has joined #openstack-infra07:24
openstackgerritMerged openstack-infra/project-config master: Normalize projects.yaml  https://review.openstack.org/63114207:25
*** pgaxatte has joined #openstack-infra07:28
*** udesale has quit IRC07:29
*** udesale has joined #openstack-infra07:29
*** apetrich has joined #openstack-infra07:31
*** ykarel is now known as ykarel|lunch07:32
*** rascasoft has joined #openstack-infra07:35
*** gfidente has joined #openstack-infra07:40
*** AJaeger has quit IRC07:53
*** quiquell is now known as quiquell|brb07:58
*** ginopc has joined #openstack-infra07:58
*** roman_g has joined #openstack-infra08:00
*** bobh has joined #openstack-infra08:00
*** wolverineav has quit IRC08:00
*** gregoryo has quit IRC08:05
*** bobh has quit IRC08:05
*** AJaeger has joined #openstack-infra08:05
*** tkajinam has joined #openstack-infra08:07
*** kopecmartin|off is now known as kopecmartin08:15
*** rpittau has joined #openstack-infra08:16
*** jtomase__ has quit IRC08:21
*** jtomasek has joined #openstack-infra08:22
*** quiquell|brb is now known as quiquell08:23
*** xek has joined #openstack-infra08:29
*** ykarel|lunch is now known as ykarel08:35
*** verdurin has quit IRC08:36
openstackgerritOpenStack Proposal Bot proposed openstack-infra/storyboard master: Imported Translations from Zanata  https://review.openstack.org/62539308:37
*** tosky has joined #openstack-infra08:37
*** tkajinam has quit IRC08:41
*** verdurin has joined #openstack-infra08:42
*** e0ne has quit IRC08:45
*** ginopc has quit IRC08:49
openstackgerritSagi Shnaidman proposed openstack-infra/nodepool master: Support userdata for instances in openstack  https://review.openstack.org/63064908:51
*** yamamoto has joined #openstack-infra08:52
*** ccamacho has joined #openstack-infra08:52
*** yamamoto has quit IRC08:56
*** jpich has joined #openstack-infra09:00
*** jpena|off is now known as jpena09:01
*** bobh has joined #openstack-infra09:12
*** Bhujay has joined #openstack-infra09:14
*** Bhujay has quit IRC09:15
*** Bhujay has joined #openstack-infra09:15
*** Bhujay has quit IRC09:16
*** Bhujay has joined #openstack-infra09:17
*** bobh has quit IRC09:17
*** Bhujay has quit IRC09:18
*** Bhujay has joined #openstack-infra09:18
*** Bhujay has quit IRC09:19
*** Bhujay has joined #openstack-infra09:20
*** Bhujay has quit IRC09:21
*** Bhujay has joined #openstack-infra09:21
*** Bhujay has quit IRC09:22
*** Bhujay has joined #openstack-infra09:23
*** Bhujay has quit IRC09:24
*** Bhujay has joined #openstack-infra09:24
*** Bhujay has quit IRC09:25
*** Bhujay has joined #openstack-infra09:26
*** Bhujay has quit IRC09:27
*** Bhujay has joined #openstack-infra09:27
*** Bhujay has quit IRC09:28
*** Bhujay has joined #openstack-infra09:29
*** Bhujay has quit IRC09:30
*** Bhujay has joined #openstack-infra09:30
*** Bhujay has quit IRC09:31
*** Bhujay has joined #openstack-infra09:32
*** yboaron_ has joined #openstack-infra09:32
*** yboaron has quit IRC09:33
*** calbers has joined #openstack-infra09:36
*** derekh has joined #openstack-infra09:37
*** Bhujay has quit IRC09:37
*** betherly has joined #openstack-infra09:42
*** betherly has quit IRC09:46
*** yboaron_ has quit IRC09:52
*** yboaron_ has joined #openstack-infra09:57
*** ginopc has joined #openstack-infra09:57
*** diablo_rojo has quit IRC10:05
*** Bhujay has joined #openstack-infra10:06
*** Bhujay has quit IRC10:07
*** Bhujay has joined #openstack-infra10:07
*** Bhujay has quit IRC10:08
*** ginopc has quit IRC10:09
*** Bhujay has joined #openstack-infra10:09
*** electrofelix has joined #openstack-infra10:09
*** Bhujay has quit IRC10:10
*** Bhujay has joined #openstack-infra10:10
*** Bhujay has quit IRC10:11
*** Bhujay has joined #openstack-infra10:12
*** Bhujay has quit IRC10:13
*** Bhujay has joined #openstack-infra10:13
*** Bhujay has quit IRC10:14
*** Bhujay has joined #openstack-infra10:15
*** yboaron_ has quit IRC10:15
*** Bhujay has quit IRC10:16
*** Bhujay has joined #openstack-infra10:16
openstackgerritMatthieu Huin proposed openstack-infra/zuul-jobs master: install-nodejs: add support for RPM-based OSes  https://review.openstack.org/63104910:16
*** Bhujay has quit IRC10:17
*** ginopc has joined #openstack-infra10:18
*** Bhujay has joined #openstack-infra10:18
*** priteau has joined #openstack-infra10:18
*** Bhujay has quit IRC10:19
*** Bhujay has joined #openstack-infra10:19
openstackgerritMatthieu Huin proposed openstack-infra/zuul-jobs master: install-nodejs: add support for RPM-based OSes  https://review.openstack.org/63104910:20
*** Bhujay has quit IRC10:20
*** Bhujay has joined #openstack-infra10:21
*** Bhujay has quit IRC10:22
*** e0ne has joined #openstack-infra10:22
*** Bhujay has joined #openstack-infra10:22
openstackgerritTristan Cacqueray proposed openstack-infra/zuul master: WIP: Separate out executor server from runner  https://review.openstack.org/60707910:23
openstackgerritTristan Cacqueray proposed openstack-infra/zuul master: WIP: runner: implement prep-workspace  https://review.openstack.org/60708210:23
openstackgerritTristan Cacqueray proposed openstack-infra/zuul master: WIP: runner: add execute sub-command  https://review.openstack.org/63094410:23
*** Bhujay has quit IRC10:23
*** Bhujay has joined #openstack-infra10:24
*** Bhujay has quit IRC10:25
*** Bhujay has joined #openstack-infra10:25
*** Bhujay has quit IRC10:26
*** Bhujay has joined #openstack-infra10:27
*** Bhujay has quit IRC10:28
*** sshnaidm is now known as sshnaidm|afk10:28
*** Bhujay has joined #openstack-infra10:28
*** Bhujay has quit IRC10:29
*** Bhujay has joined #openstack-infra10:30
*** Bhujay has quit IRC10:31
*** Bhujay has joined #openstack-infra10:31
*** Bhujay has quit IRC10:32
*** Bhujay has joined #openstack-infra10:33
openstackgerritMatthieu Huin proposed openstack-infra/zuul-jobs master: install-nodejs: add support for RPM-based OSes  https://review.openstack.org/63104910:33
*** Bhujay has quit IRC10:34
*** Bhujay has joined #openstack-infra10:34
*** Bhujay has quit IRC10:35
*** Bhujay has joined #openstack-infra10:36
*** Bhujay has quit IRC10:37
*** Bhujay has joined #openstack-infra10:37
*** Bhujay has quit IRC10:38
*** Bhujay has joined #openstack-infra10:39
*** Bhujay has quit IRC10:40
*** Bhujay has joined #openstack-infra10:40
*** Bhujay has quit IRC10:41
*** Bhujay has joined #openstack-infra10:42
*** Bhujay has quit IRC10:43
*** Bhujay has joined #openstack-infra10:43
*** Bhujay has quit IRC10:44
*** Bhujay has joined #openstack-infra10:45
*** Bhujay has quit IRC10:46
*** Bhujay has joined #openstack-infra10:46
*** Bhujay has quit IRC10:47
*** Bhujay has joined #openstack-infra10:48
*** Bhujay has quit IRC10:49
*** Bhujay has joined #openstack-infra10:49
*** Bhujay has quit IRC10:50
*** Bhujay has joined #openstack-infra10:51
*** ginopc has quit IRC10:52
*** Bhujay has quit IRC10:52
*** Bhujay has joined #openstack-infra10:52
*** Bhujay has quit IRC10:53
*** Bhujay has joined #openstack-infra10:54
*** Bhujay has quit IRC10:55
*** Bhujay has joined #openstack-infra10:55
*** Bhujay has quit IRC10:56
*** Bhujay has joined #openstack-infra10:57
*** Bhujay has quit IRC10:58
*** Bhujay has joined #openstack-infra10:58
*** Bhujay has quit IRC10:59
*** psachin has quit IRC10:59
*** Bhujay has joined #openstack-infra11:00
*** Bhujay has quit IRC11:01
*** Bhujay has joined #openstack-infra11:03
AJaegercmurphy, dirk, could you review 631049, please? ^11:11
*** dtantsur has joined #openstack-infra11:12
*** udesale has quit IRC11:12
*** panda|off is now known as panda11:13
*** ginopc has joined #openstack-infra11:14
openstackgerritTobias Urdin proposed openstack-infra/zuul-jobs master: Fix upload-puppetforge role  https://review.openstack.org/63119411:32
openstackgerritMatthieu Huin proposed openstack-infra/zuul-jobs master: install-nodejs: add support for RPM-based OSes  https://review.openstack.org/63104911:35
*** bobh has joined #openstack-infra11:51
*** yamamoto has joined #openstack-infra11:55
*** ginopc has quit IRC11:55
*** bobh has quit IRC11:56
*** ginopc has joined #openstack-infra11:56
cmurphyAJaeger: done11:56
*** Bhujay has quit IRC12:06
*** Bhujay has joined #openstack-infra12:07
*** rh-jelabarre has joined #openstack-infra12:07
AJaegerthanks, cmurphy12:13
openstackgerritMerged openstack-infra/storyboard master: Add a popularity measurement to tags  https://review.openstack.org/62104612:14
ssbarnea|roverclarkb: e-r https://review.openstack.org/#/c/622352/ and https://review.openstack.org/#/c/629921/ when you can, thanks.12:16
*** e0ne has quit IRC12:17
*** rpittau has quit IRC12:18
*** rpittau has joined #openstack-infra12:19
*** panda is now known as panda|lunch12:31
*** bgmccollum has quit IRC12:33
*** jpena is now known as jpena|lunch12:35
openstackgerritSimon Westphahl proposed openstack-infra/zuul master: Add change message to Zuul vars in inventory  https://review.openstack.org/63120712:36
openstackgerritSimon Westphahl proposed openstack-infra/zuul master: Include PR title in change message  https://review.openstack.org/63120812:36
*** sshnaidm|afk is now known as sshnaidm12:41
*** udesale has joined #openstack-infra12:42
*** rpittau has quit IRC12:50
*** rpittau has joined #openstack-infra12:50
openstackgerritSagi Shnaidman proposed openstack-infra/nodepool master: Support userdata for instances in openstack  https://review.openstack.org/63064912:51
*** smarcet has joined #openstack-infra12:52
*** smarcet has quit IRC12:53
*** ykarel is now known as ykarel|away12:54
*** bgmccollum has joined #openstack-infra12:55
*** electrofelix has quit IRC12:58
*** yamamoto has quit IRC12:59
*** rpittau has quit IRC13:04
openstackgerritSimon Westphahl proposed openstack-infra/zuul master: Don't request node nodes for skipped child jobs  https://review.openstack.org/63121313:04
*** rpittau has joined #openstack-infra13:04
*** e0ne has joined #openstack-infra13:04
openstackgerritSimon Westphahl proposed openstack-infra/zuul master: Don't request nodes for skipped child jobs  https://review.openstack.org/63121313:05
*** bobh has joined #openstack-infra13:06
*** kjackal has joined #openstack-infra13:17
openstackgerritMatthieu Huin proposed openstack-infra/zuul-jobs master: install-nodejs: add support for RPM-based OSes  https://review.openstack.org/63104913:18
*** rpittau has quit IRC13:19
*** rpittau has joined #openstack-infra13:22
openstackgerritIvan Kolodyazhny proposed openstack-infra/project-config master: Stop project gating before retirement  https://review.openstack.org/63121813:23
*** Bhujay has quit IRC13:25
*** bobh has quit IRC13:25
*** Bhujay has joined #openstack-infra13:25
*** trown|outtypewww is now known as trown13:25
*** dave-mccowan has joined #openstack-infra13:26
*** ykarel|away has quit IRC13:26
*** Bhujay has quit IRC13:26
*** Bhujay has joined #openstack-infra13:27
*** Bhujay has quit IRC13:28
*** panda|lunch is now known as panda13:28
*** Bhujay has joined #openstack-infra13:28
*** Bhujay has quit IRC13:29
*** Bhujay has joined #openstack-infra13:30
*** yamamoto has joined #openstack-infra13:30
*** Bhujay has quit IRC13:31
*** Bhujay has joined #openstack-infra13:31
*** Bhujay has quit IRC13:32
*** Bhujay has joined #openstack-infra13:33
*** Bhujay has quit IRC13:34
*** jpena|lunch is now known as jpena13:34
*** Bhujay has joined #openstack-infra13:34
*** rpittau has quit IRC13:34
*** rpittau has joined #openstack-infra13:35
*** Bhujay has quit IRC13:35
*** Bhujay has joined #openstack-infra13:36
*** Bhujay has quit IRC13:37
*** yamamoto has quit IRC13:37
*** Bhujay has joined #openstack-infra13:37
*** Bhujay has quit IRC13:38
*** rlandy has joined #openstack-infra13:39
*** Bhujay has joined #openstack-infra13:39
*** roman_g has quit IRC13:39
*** Bhujay has quit IRC13:40
*** Bhujay has joined #openstack-infra13:40
*** d0ugal has quit IRC13:41
*** Bhujay has quit IRC13:41
*** d0ugal has joined #openstack-infra13:42
*** Bhujay has joined #openstack-infra13:42
*** dtantsur is now known as dtantsur|brb13:42
*** Bhujay has quit IRC13:43
*** Bhujay has joined #openstack-infra13:43
*** Bhujay has quit IRC13:44
*** Bhujay has joined #openstack-infra13:45
*** Bhujay has quit IRC13:46
openstackgerritFelix Schmidt proposed openstack-infra/zuul master: Add action to task result in zuul_json callback  https://review.openstack.org/63062213:46
*** roman_g has joined #openstack-infra13:46
*** Bhujay has joined #openstack-infra13:46
*** ykarel|away has joined #openstack-infra13:47
*** kgiusti has joined #openstack-infra13:48
*** boden has joined #openstack-infra13:48
*** ykarel|away is now known as ykarel13:50
*** armax has joined #openstack-infra14:01
*** kjackal has quit IRC14:03
*** rpittau has quit IRC14:05
*** Goneri has joined #openstack-infra14:08
*** rpittau has joined #openstack-infra14:10
openstackgerritFelix Schmidt proposed openstack-infra/zuul master: Add action to host results in zuul_json callback  https://review.openstack.org/63062214:10
*** kjackal has joined #openstack-infra14:12
*** d0ugal has quit IRC14:18
*** d0ugal has joined #openstack-infra14:18
*** jamesmcarthur has joined #openstack-infra14:21
*** shardy has joined #openstack-infra14:23
*** Goneri has quit IRC14:24
*** mriedem has joined #openstack-infra14:31
openstackgerritSagi Shnaidman proposed openstack-infra/nodepool master: Support userdata for instances in openstack  https://review.openstack.org/63064914:37
*** jcoufal has joined #openstack-infra14:39
*** quiquell is now known as quiquell|off14:39
*** ssbarnea|rover has quit IRC14:40
*** ssbarnea|rover has joined #openstack-infra14:40
*** eharney has joined #openstack-infra14:41
*** ginopc has quit IRC14:48
*** amoralej has joined #openstack-infra14:49
amoralejhi ianw, i'm working to create a new rhel8 element in dib that can be used to create an image from existing cloud image, as done in tripleo world14:50
amoralejfor some reason i'm getting /boot emptied14:50
amoraleji can't find what's doing that14:50
amoralejany idea, what could be removing what is in /boot14:51
amoralej?14:51
fungiamoralej: ianw may already be away from his computer for the day, as he's on sydney time14:53
fungiamoralej: there's also a #openstack-dib channel which might be more on topic for general dib questions14:54
*** gouthamr has quit IRC14:54
*** stevebaker has quit IRC14:55
*** dmellado has quit IRC14:56
pabelangeramoralej: https://review.openstack.org/623137/ might be a starting place, that is the element ianw is working on for rhel814:57
amoralejfungi, thanks14:59
amoralejpabelanger, yeah, i used that as starting point but that's rhel8-minimal while i'm working in a rhel8 one14:59
*** yamamoto has joined #openstack-infra15:04
*** Bhujay has quit IRC15:06
*** yamamoto has quit IRC15:10
*** kjackal has quit IRC15:16
*** rkukura has joined #openstack-infra15:17
*** roman_g has quit IRC15:20
*** kjackal has joined #openstack-infra15:24
*** rkukura has quit IRC15:27
openstackgerritLiam Young proposed openstack-infra/project-config master: Add Cinder Storage charm interface  https://review.openstack.org/63125215:28
*** derekh has quit IRC15:30
openstackgerritTobias Urdin proposed openstack-infra/zuul-jobs master: Fix upload-puppetforge role  https://review.openstack.org/63119415:39
openstackgerritJeremy Stanley proposed openstack-infra/storyboard master: Implement support for oauthlib 3  https://review.openstack.org/63126015:39
*** ramishra has quit IRC15:46
*** dtantsur|brb is now known as dtantsur15:46
*** lpetrut has quit IRC15:46
*** derekh has joined #openstack-infra15:55
*** udesale has quit IRC15:56
openstackgerritMonty Taylor proposed openstack-infra/zuul-jobs master: Fix docker image password docstring  https://review.openstack.org/63126615:57
clarkbspsurya: that isn't an infra project.15:58
clarkband now that stackalytics is using a different code base I'm not sure we should ocntinue to merge changes to that one15:58
*** yamamoto has joined #openstack-infra15:59
clarkbfyi I've put review-dev01.openstack.org in the emergency file so that I can downgrade the gerrit there (and do a full offline reindex ont he old version to properly ensure we test the upgrade)16:04
*** eernst has joined #openstack-infra16:09
mordredclarkb: is that not the stackalytic codebase anymore?16:13
clarkbmordred: as far as we can tell it isn't. If you go to stackalytics.com its got a new shiny UI none of which is refelcted in that code base16:14
clarkbfungi asked them where the code is hosted now and didn't get a reponse16:14
mordredyay fun16:14
fungiasked in http://lists.openstack.org/pipermail/openstack-discuss/2019-January/001526.html16:15
openstackgerritMerged openstack-infra/zuul-jobs master: Add docker image promotion roles  https://review.openstack.org/63107816:17
*** chrisyang_0660 has joined #openstack-infra16:18
spsuryaclarkb:  actually https://review.openstack.org/#/admin/groups/183,members  includes infra core team, so  i thought i can get help in review from here too.16:19
clarkbspsurya: yes, I don't know why we were added to the core group on that repo16:19
clarkbregardless it seems that they (whoever is running stackalytics now) isn't using that code base16:20
spsuryaclarkb: Yeah seems so stackalytics is using different code base16:20
fungiclarkb: spsurya: we had the stackalytics maintainers add the infra-core reviewers back when we were working to take over hosting the service and moving it to stackalytics.openstack.org16:21
fungibut then that stalled due to disengagement on the mirantis side and we eventually dropped the effort entirely and deleted the server16:21
fungii'm in favor of removing the infra-core group from stackalytics-core just didn't want to do it without some discussion16:22
spsuryafungi: thanks for update16:23
*** roman_g has joined #openstack-infra16:24
spsuryafor infra-core group removal you guys can start discussion on ML to avoid confusion in future w.r.t stackalytics changes16:25
*** pcaruana has quit IRC16:25
fungispsurya: yeah, we weren't on there to review contributor configuration changes, just to be able to maintain the software which provides the service16:25
*** kjackal has quit IRC16:26
spsuryafungi: roger that16:31
openstackgerritSagi Shnaidman proposed openstack-infra/nodepool master: Support userdata for instances in openstack  https://review.openstack.org/63064916:32
corvusfungi: perhaps we should retire the repo?16:40
*** gouthamr has joined #openstack-infra16:44
fungicorvus: if they confirm they're really not using it any longer, then yes it should be retired16:45
fungibut it seems like they still take/import configuration from there for the service16:46
fungiand they seem to semi-regularly review and approve configuration changes in it16:46
pabelangerat one point, we talked about moving the data into project-config for stackalytics.o.o16:47
corvuswell, we definitely shouldn't do that now16:47
pabelanger+116:47
fungiagreed16:47
*** e0ne_ has joined #openstack-infra16:50
*** e0ne has quit IRC16:50
corvusfungi: can you review https://review.openstack.org/631089 and all of its parents?  but don't approve; i'll do that one at a time to verify uploads work16:52
*** pgaxatte has quit IRC16:53
*** ccamacho has quit IRC16:56
*** ginopc has joined #openstack-infra16:56
*** rpittau has quit IRC16:56
openstackgerritTobias Urdin proposed openstack-infra/zuul-jobs master: Use PDK to build puppet module  https://review.openstack.org/62753416:57
openstackgerritDean Troyer proposed openstack-infra/project-config master: Remove stx-* Gerrit master notifications from #starlingx  https://review.openstack.org/63127917:00
openstackgerritMerged openstack-infra/zuul-jobs master: Fix docker image password docstring  https://review.openstack.org/63126617:01
fungicorvus: yep, taking a look now17:02
*** ijw has joined #openstack-infra17:05
openstackgerritTobias Urdin proposed openstack-infra/zuul-jobs master: Fix upload-puppetforge role  https://review.openstack.org/63119417:06
*** smarcet has joined #openstack-infra17:07
openstackgerritTobias Urdin proposed openstack-infra/zuul-jobs master: Use PDK to build puppet module  https://review.openstack.org/62753417:08
*** e0ne_ has quit IRC17:18
*** kopecmartin is now known as kopecmartin|off17:21
*** smarcet has quit IRC17:21
*** jpich has quit IRC17:28
fungicorvus: okay, i think i'm all caught up on that series now. looks great!17:30
*** dtantsur is now known as dtantsur|afk17:31
corvusfungi: thanks!  i'll start approving them stepwise17:31
clarkbI'm going to stop gerrit on review-dev now. chown the files I think need chowning. Run gerrit init and an offline reindex to go back to 2.13.9 then assuming that is all happy remove review-dev01 from the emergency file and have puppet reupgrade to 2.13.12 which should confirm (or not) that our upgrade process is clean now17:31
*** smarcet has joined #openstack-infra17:31
clarkbapparently stackalytics does intend to merge back into our hosted repo17:32
clarkb(from twitter after sparkycollier asked them)17:32
clarkbHopefulyl that doesn't require us to force merge anything. Also we should have ac onversation about maintainership (eg I'm not sure infra-root is appropriate there)17:32
corvusclarkb: i would say it will not require us to force merge anything.17:32
corvusthis is not an infrastructure or community for code dumping.17:33
clarkbya maybe better way to phrase that was "I hope they don't ask us to do it"17:33
corvus++17:33
ssbarnea|roverclarkb: interesting subject. i seen their lack of interest on contributing back, email, response to CRs,....17:34
clarkbssbarnea|rover: ya thats why I'd like to push back on us appearing to be maintainers. If they are activiely dev'ing it then that should happen in the canonical repo and they should be able to act as maintainers17:35
clarkbas is its just confusing17:35
ssbarnea|roverclarkb: i agree. i am also curious if we have any guidelines regarding lifecycle of projects because sooner or later we will have more cases like this and somehow infra-root ends-up as collateral damage.17:36
ssbarnea|roverarchival/abandonware/unmaintained/forked/migrated.... all things that can happen but not much guidelines regarding how to approach them.17:37
corvusssbarnea|rover: theres a section of the manual on it: https://docs.openstack.org/infra/manual/drivers.html#retiring-a-project17:38
*** ykarel has quit IRC17:40
clarkbreview-dev01 is doing offline reindex now. I've discovered that the problem is the .bcp*.backup files. If I delete those before running the gerrit init then it works fine17:40
clarkbI'm debating if I want to encode that into puppet or just manually do that before we do the update in prod17:41
clarkbits annoying that gerrit.war creates files that prevent it from running17:41
ssbarnea|rovercorvus: thanks. this helps once we know a project needs to be retired. but the is a lot of gray area before knowing exactly what happens with the project. lots of small projects that are in reality abandoned but nobody cared to declare them. anyway this is something to debate over a beer not here.17:41
corvusi feel like encoding deleting backup files in puppet is dangerous17:41
clarkbcorvus: ++17:42
corvusssbarnea|rover: also, not our problem :)17:42
corvusssbarnea|rover: we're not the project activity police17:42
clarkbI'm going to run through the current upgrade with a manual delete to prove that it works when we update to 2.13.1217:42
clarkband if we decide we want somethign different for prod we can test that again17:42
corvusssbarnea|rover: individual projects, or in the case of openstack, the tc get to decide that, not us17:43
*** smarcet has quit IRC17:43
*** jamesmcarthur has quit IRC17:45
*** jamesmcarthur has joined #openstack-infra17:46
ssbarnea|roverreally happy to see work done about gerrit upgrade :D17:48
openstackgerritClint 'SpamapS' Byrum proposed openstack-infra/nodepool master: Amazon EC2 driver  https://review.openstack.org/53555817:51
openstackgerritSagi Shnaidman proposed openstack-infra/nodepool master: Support userdata for instances in openstack  https://review.openstack.org/63064917:54
*** ginopc has quit IRC17:55
ssbarnea|rovercorvus: clarkb: i was planning to work on git-review to change the way we do start/run gerrit, to use containers instead of current background daemons.... which have the bad habit to remain running and taking a lot of cpu doing nothing. ok with the idea? -- side effect: testing with multiple gerrit versions will be very easy.17:56
clarkbssbarnea|rover: my only concern with that is the git review tests will require root if you do that17:56
clarkbmaybe we can split the test suite into functional and unit tests to make that distinction more clear17:57
ssbarnea|roverclarkb: no way. root is out of question.17:57
clarkbssbarnea|rover: docker implies root17:57
clarkband I think podman does but maybe it doesn't anymore?17:57
clarkb(docker makes it easier to run everywhere though) unsure about lxc17:57
ssbarnea|roverclarkb: afaik not, not to mention that docker can also run on another machine, just setting DOCKER_HOST would work.17:58
clarkbssbarnea|rover: sure but then the test suite has root on that machine.17:58
pabelangeryah, podman is non root container17:58
clarkb(the issue is you either have to run docker as root or be in the docker group, being in the docker group allows you to create containers that will give you root)17:58
clarkbits not hte end of the world, but we have in the past tried to prevent leakages like that with our unit tests17:59
fungiright, the current git-review unit tests at least run gerrit as a normal user, the same user running the tests even17:59
openstackgerritMerged openstack-infra/zuul master: Fix test_load_governor on large machines  https://review.openstack.org/63011818:00
fungigranted, i wasn't a fan of the full integration model for git-review's primary testsuite anyway, that's just what the volunteer who agreed to build them decided to work on18:00
*** roman_g has quit IRC18:00
*** derekh has quit IRC18:00
fungii was much more in favor of something like mocking the shell interaction layer18:01
ssbarnea|roverfungi: func vs integration has no clear winner. both have pros/cons.18:01
fungisure, i didn't say i was against _also_ having integration testing for git-review18:01
clarkbreview-dev.o.o finished reindexing and has been started. web ui shows the correct 2.13.9 version. I'll manually clear the backup jar files now then remove the host from the emergency file. This should cause puppet to do an upgrade to 2.13.1218:01
corvusi think testing against gerrit itself is important for git-review; regardless of what else we do, i think we need to do at least that.18:01
*** jamesmcarthur has quit IRC18:02
*** jamesmcarthur has joined #openstack-infra18:02
ssbarnea|rovermaintain two libraries with are at extreames: python-jira that has almost only integration, and python-jenkins that does only unit. Unit has the bad habbit of not validating compatibility with newer version of the service.18:02
ssbarnea|roveroptimal is to have both, for whom can afford that luxury18:03
*** jpena is now known as jpena|off18:03
clarkbJust waiting for puppet to run against review-dev now for the 2.13.12 upgrade18:03
fungiright, i'd prefer to see both run in ci, but for local rapid development unit tests with mocks are convenient18:03
fungithen again, there's not a ton of development on git-review (nor does there need to be), so maybe just integration testing is fine18:04
clarkbcorvus: ya the tests against real gerrit really helped us improve a few things18:04
clarkblike I said not opposed just need to keep in mind we may be changing the parameters of "unit" testing and should understand that18:04
clarkbpabelanger: looks like it uses user namespaces like bwrap so ya should work as long as you have a newer kernel and don't try to run containers that need privs on stuff they can't have18:06
clarkbit being podman18:06
*** jamesmcarthur has quit IRC18:07
*** jamesmcarthur has joined #openstack-infra18:07
pabelangerclarkb: yah, that seems to be the way rhel8 will work, no docker and only podman18:08
pabelangerfrom everything I hear, it should work the same as docker, but no use root18:08
*** priteau has quit IRC18:08
pabelangerunless you do --priviliged18:08
clarkbwell you'd still need root for ya stuff like that18:08
clarkbso it partly depends on the workload (gerrit shouldn't need that)18:08
pabelangeryup18:09
fungissbarnea|rover: digging into the current git-review testsuite, it spins up/down an arbitrary number of gerrit services to do parallel testing. is that going to be a problem when trying to container it?18:11
ssbarnea|roverfungi: i know about this aspect. i am going to test it. there is nothing preventing us from starting multiple instances/containers but I am wondering if we really need to.18:12
fungior would the new implementation just launch each one in a separate container?18:12
openstackgerritMerged openstack-infra/system-config master: Add support for per-image build-arg settings  https://review.openstack.org/63077718:12
ssbarnea|roverfungi: i didn't make a decision yet, I don't know if the test suite can work well with a single gerrit instance but running tests in parallel.  based on this i can decide which path to take,18:13
clarkbssbarnea|rover: ya there has already been talk of starting a single gerrit instance containers or not iirc18:13
fungissbarnea|rover: the choice to have multiple concurrent gerrit deployments was so that tests didn't develop dependencies or conflicts between one another, and also to ease cleanup from tests so they don't each pollute the others' environemnt18:13
clarkblargely depends on writing tests that are independent of each other. So you can't rely on change numbers iwthin a single project18:13
clarkbone solution is not rely on specific numbers, another is different project for each test18:13
ssbarnea|roverif only one container is enough, i would probably being tempted to make use of tox-docker18:14
fungiis that different from dox?18:14
ssbarnea|roverfungi: very. tox-docker manages a side-load container that you need for testing, but runs the tests on your machine.18:15
*** gfidente is now known as gfidente|afk18:15
ssbarnea|roverdox does something else18:15
clarkbreview-dev is now running 2.13.1218:15
fungissbarnea|rover: neat, thanks18:16
ssbarnea|roverrunning the tests themselves inside the container, which is not really needed for our scope.18:16
ssbarnea|rovergit-review is a cli tool, does not need root, easy to run locally.18:16
*** smarcet has joined #openstack-infra18:17
clarkbthe puppet is chatty but I don't see any errors this time around18:17
ssbarnea|roverif any of you know other approaches it would be a good time to tell me, one condition I have is that any developer should be able to run the tests after cloning without extra steps. (having a docker instance running is acceptable) but requiring root or other complex steps is something i really want to avoid.18:18
ssbarnea|rovereven better, with container approach we no longer need java on the client, as only servers needs java.18:18
*** trown is now known as trown|lunch18:19
clarkbI want to see how the next puppet run cleans up /home/gerrit2/review_site/lib (if at all) before declaring this a success, but I think this has done it18:19
clarkbupgrade process is delete the /home/gerrit2/review_site/lib/.bcp*.backup files, then update the war in puppet18:19
*** jamesmcarthur has quit IRC18:20
ssbarnea|rovertwo git-review i need help with: https://review.openstack.org/#/c/628655/ and https://review.openstack.org/#/c/630661/18:20
*** smarcet has quit IRC18:23
*** jamesmcarthur has joined #openstack-infra18:25
openstackgerritMerged openstack-infra/zuul master: tests: improve test_web to only provision events when needed  https://review.openstack.org/63057518:27
clarkbssbarnea|rover: one small dependency bug in the stestr one, otherwise looks fine to m18:29
*** jamesmcarthur has quit IRC18:30
*** jamesmcarthur has joined #openstack-infra18:39
*** jamesmcarthur has quit IRC18:42
openstackgerritSorin Sbarnea proposed openstack-infra/git-review master: Switch from testrepository to stestr  https://review.openstack.org/62865518:43
*** smarcet has joined #openstack-infra18:44
openstackgerritSagi Shnaidman proposed openstack-infra/nodepool master: Support userdata for instances in openstack  https://review.openstack.org/63064918:48
openstackgerritJeremy Stanley proposed openstack-infra/system-config master: Upgrade to LimeSurvey 3.15.6  https://review.openstack.org/57990218:48
fungirefreshed that ^ change since the self-healing sig has indicated an interest in trying it out18:49
corvusfungi: https://docs.openstack.org/infra/system-config/survey.html#admin-survey-user says log in using openstackid, but when i go to the link i get a form; is that expected?18:52
*** wolverineav has joined #openstack-infra18:53
*** wolverineav has quit IRC18:54
*** wolverineav has joined #openstack-infra18:55
*** wolverineav has quit IRC18:55
*** wolverineav has joined #openstack-infra18:56
openstackgerritPaul Belanger proposed openstack-infra/project-config master: Create windmill-ops project  https://review.openstack.org/63130318:57
*** smarcet has quit IRC18:58
*** diablo_rojo has joined #openstack-infra19:00
fungicorvus: interesting. it used to do openid. i'll see if i can tell what's changed19:04
*** efried has quit IRC19:04
fungicorvus: yeah, urls under admin are supposed to be covered by https://git.openstack.org/cgit/openstack-infra/system-config/tree/modules/openstack_project/templates/survey.vhost.erb#n2719:06
fungiso that seems to have broken. will dig into logs19:06
pabelangerinfra-root: now that https://docs.openstack.org/infra/system-config/sysadmin.html#github-access is updated to use a shared github account, there is no issue for me removing my account (pabelanger) from the infra-core team.19:10
pabelangerI want to log into gerrithub.io, but it ask for permissions on openstack orgs, and don't want to allow that19:10
fungipabelanger: yes, several of us have already removed our personal accounts from those orgs too19:11
clarkbpabelanger: we've been having people remove themselves, feel free to do that19:11
pabelangergreat, thanks19:11
*** jtomasek has quit IRC19:12
*** wolverineav has quit IRC19:13
ssbarnea|roverpabelanger: github has an option to prevent even admins from doing a push but it needs to be enabled per repo per branch, a pita.19:13
*** wolverineav has joined #openstack-infra19:13
*** jtomasek has joined #openstack-infra19:14
ssbarnea|roverbut as safety it would be useful to assure that 2 others accounts have admin on gerrit as having only one account as sole admin seems too risky to get yourself lockedout.19:15
pabelangeryah, don't want to enable branch protection for all our projects, removing myself is much easier19:15
*** wolverineav has quit IRC19:15
pabelanger#status log removed pabelanger admin user from openstack orgs (all) on github.com19:15
*** wolverineav has joined #openstack-infra19:15
openstackstatuspabelanger: finished logging19:15
*** gouthamr_ has joined #openstack-infra19:16
fungicorvus: i suspect this is a puppet 4 issue... the configuration from that conditional block is missing from the vhost on the server now (and it used to be there) so probably something changed with that != nil evaluation in erb or (un)setting of the auth_openid class parameter19:19
fungishould be able to nail it down after the storyboard meeting concludes19:20
*** efried has joined #openstack-infra19:21
*** trown|lunch is now known as trown19:21
*** shardy has quit IRC19:22
*** dmellado has joined #openstack-infra19:22
clarkbfungi: oh sorry about that. I had tried to monitor all of these but I guess in this case apache contineud to work so I didn't catch it19:23
openstackgerritMerged openstack-infra/zuul master: Gracefully close db connections in tests  https://review.openstack.org/63047119:23
fungiclarkb: can't catch 'em all, and this was subtle19:24
*** smarcet has joined #openstack-infra19:24
fungiand it might not be puppet 4 fallout, just seems the most likely cause based on limited investigation so far19:24
openstackgerritJames E. Blair proposed openstack-infra/system-config master: Build a gerrit image  https://review.openstack.org/63046819:28
openstackgerritJames E. Blair proposed openstack-infra/system-config master: Remove all docker image building jobs  https://review.openstack.org/63108419:28
openstackgerritJames E. Blair proposed openstack-infra/system-config master: Add docker image building jobs  https://review.openstack.org/63108519:28
openstackgerritJames E. Blair proposed openstack-infra/system-config master: Add jinja-init image jobs  https://review.openstack.org/63108619:28
openstackgerritJames E. Blair proposed openstack-infra/system-config master: Add gitea-init image jobs  https://review.openstack.org/63108719:28
openstackgerritJames E. Blair proposed openstack-infra/system-config master: Add gitea image jobs  https://review.openstack.org/63108819:28
openstackgerritJames E. Blair proposed openstack-infra/system-config master: Add gerrit image jobs  https://review.openstack.org/63108919:28
corvusthat backports the required-projects fix from 631089 to 63046819:28
corvuswhich did, in fact, fail the gate job :)19:29
corvusclarkb, fungi: can you re-review https://review.openstack.org/630468 ?  that's the only one that changed19:30
fungiyou bet19:32
*** shardy has joined #openstack-infra19:32
fungithat was a fairly minimal adjustment at least19:33
clarkbdone19:34
clarkb(I approved it)19:34
clarkbwe could also just remove that change if it continues to give us truoble and rely on the follow on change to implement it for us?19:34
*** shardy has quit IRC19:38
clarkbok puppet tidied up jars as expected but not the backup jar files19:40
clarkbIf we want to life support puppet-gerrit longer I think one thing we could do is have puppet download the files to the location that gerrit downloads them (though that gets complicated if you want puppet to overwrite gerrits' files)19:40
*** mriedem has quit IRC19:40
clarkbin the mean time I think we are ready to propose an update on prod with manual removal of the backup jars unless someone has suggestions for further testing19:41
clarkbthis chagne does update jgit so maybe something that better exercises git on review-dev? I can view diffs19:41
*** mriedem has joined #openstack-infra19:41
*** jamesmcarthur has joined #openstack-infra19:43
*** eernst has quit IRC19:45
*** e0ne has joined #openstack-infra19:46
*** eharney has quit IRC19:47
*** eernst_ has joined #openstack-infra19:47
*** eernst_ has quit IRC19:47
*** stevebaker has joined #openstack-infra19:48
*** jamesmcarthur has quit IRC19:48
*** eernst has joined #openstack-infra19:48
fungiclarkb: in survey.vhost.erb i suppose switching the comparison from @auth_openid to scope.lookupvar("openstack_project::survey::auth_openid") ought to fix this?19:48
fungiis the problem that it's looking for auth_openid in the vhost resource scope rather than the survey class itself?19:49
clarkbyup, puppet4 is strict about scopes19:49
clarkbso when we do that lookup within the vhost from ::httpd we have to be specific when it isn't in httpd19:49
clarkbpuppet3 and earlier put everythign in global scope too19:49
*** jamesmcarthur has joined #openstack-infra19:51
clarkb#status log Submitted PTG attendace survey for the Infra team. Requested 2 days for ~10 people but mentioned we can be flexible. Hope to see you there.19:52
openstackstatusclarkb: finished logging19:52
fungilooks like it's just the vhost config template which suffers from this, since the application config template is a file resource within the survey class19:52
clarkbcorrect19:52
*** shardy has joined #openstack-infra19:54
*** jaosorior has quit IRC19:57
*** dklyle has quit IRC19:57
*** dklyle has joined #openstack-infra19:57
*** eernst has quit IRC19:57
openstackgerritJeremy Stanley proposed openstack-infra/system-config master: Correct auth_openid scope in survey vhost  https://review.openstack.org/63132019:59
fungicorvus: clarkb: cmurphy: ^ should hopefully fix survey admin auth on survey.o.o20:00
clarkbfungi: I went ahead and approved it. I was single approving some of those fixes earlier since they are pretty repetive/consistent20:02
*** xek has quit IRC20:05
*** xek has joined #openstack-infra20:05
fungithanks! i'll test it once that merges20:05
fungimake sure everything else is working as expected20:06
openstackgerritDouglas Mendizábal proposed openstack-infra/project-config master: Add ansible roles for managing HSM software  https://review.openstack.org/63132620:07
*** yamamoto has quit IRC20:08
*** zul has quit IRC20:09
openstackgerritDouglas Mendizábal proposed openstack-infra/project-config master: Add ansible roles for managing HSM software  https://review.openstack.org/63132620:09
JpMaxManmordred: quick question re: Gerrit API - want to confirm there is no way to receive a directory listing for an individual branch?  I found this and assume that's right, but thought I'd double check here https://stackoverflow.com/questions/48550302/retrieve-dir-structure-in-gerrit-for-a-project-using-rest-api20:14
clarkbJpMaxMan: I think gerrit expects you to get that info via git20:15
fungialso, git doesn't actually have any concept of directories20:15
fungiit knows files by a full path20:15
*** eernst has joined #openstack-infra20:16
fungiit doesn't track directories, which is why you can't rely on git to populate an empty directory in a source tree20:16
*** smarcet has left #openstack-infra20:17
*** wolverineav has quit IRC20:17
*** shardy has quit IRC20:18
*** eernst has quit IRC20:19
*** eernst has joined #openstack-infra20:19
clarkbJpMaxMan: what is the use case you are trying to achieve?20:19
JpMaxManSo we're replicating the GitHub integration for Netlify to use Gerrit instead20:21
JpMaxManthe Github uses this API call https://developer.github.com/v3/repos/contents/#get-contents20:21
JpMaxManso were looking for something equivalent with Gerrit20:21
openstackgerritMerged openstack-infra/system-config master: Build a gerrit image  https://review.openstack.org/63046820:22
JpMaxManwe are able to use a work around with the gitweb link20:22
JpMaxManhttps://git.openstack.org/cgit/openstack-infra/netlify-sandbox/plain/?h=refs/heads/master20:22
JpMaxManas an example20:22
JpMaxManbut wanted to make sure we weren't missing something20:23
clarkbJpMaxMan: https://github.com/isomorphic-git/isomorphic-git something like that may allow you to do it via git directly instead20:23
fungishould be possible to use a git remote query to obtain a file listing at a given ref20:23
fungiyeah, something like that20:23
fungiso basically speaking git protocol instead of gerrit rest api protocol20:23
JpMaxManright could use that approach - per our discussion with mordred we were hoping to make a pure REST API implementation20:24
clarkbthat said the gerrit file editing api may also help, however I think that requires a preexisting change20:24
corvus(jumping in late, but i concur -- i don't think the gerrit rest api hadles that particular case but isomorphic or something like it sounds like a good solution)20:24
JpMaxManit may be to have a proper netlify plugin this is required - I'd have to check20:24
clarkbI don't know if that is documented as a stable public api though20:24
corvusJpMaxMan: i think the isomorphic approach would still be a pure js approach20:25
corvusJpMaxMan: the git protocol works over http, so it shouldn't be much different20:25
JpMaxManyes pure JS for sure20:25
JpMaxManok cool - as long as its not hiding somewhere - we have a work around using the gitweb for now - and the isomorphic approach is a good one as well - just trying to keep it as light weight as possible20:26
corvus++20:27
corvus(we might be able to get something like that added to gerrit, but we shouldn't count on that -- let's file that away as a potential future improvement)20:27
*** kjackal has joined #openstack-infra20:28
JpMaxMancorvus: yes that was our thought as well, would be cool to possibly extend the gerrit API but not something we want to count on ;)20:28
*** jtomasek_ has joined #openstack-infra20:33
*** jtomasek has quit IRC20:36
*** sthussey has quit IRC20:36
*** whoami-rajat has quit IRC20:37
*** yamamoto has joined #openstack-infra20:38
anteayafungi: where is limesurvey 3.15.6 coming from? I don't see it listed here: https://github.com/LimeSurvey/LimeSurvey/tags this is regarding https://review.openstack.org/#/c/57990220:39
anteayaI believe you I'd just like to verify the tag20:40
fungithat's apparently a laggy mirror, per the commit message20:41
anteayaah sorry, /me reads20:41
fungithe limesurvey download site has its own tarballs which seem to lead the copy on github by a week or more20:42
anteayanot helpful as they release every 8 or 10 days20:42
anteayayay for releasing often20:43
*** lpetrut has joined #openstack-infra20:43
*** e0ne has quit IRC20:44
*** e0ne has joined #openstack-infra20:45
*** e0ne has quit IRC20:48
*** wolverineav has joined #openstack-infra20:54
*** eharney has joined #openstack-infra20:55
*** lpetrut has quit IRC21:05
*** kgiusti has quit IRC21:06
*** jamesmcarthur has quit IRC21:06
*** jamesmcarthur has joined #openstack-infra21:08
openstackgerritTrevor McCasland proposed openstack-infra/subunit2sql master: Fix 'Content' object is not iterable  https://review.openstack.org/62740921:12
*** xek has quit IRC21:12
*** kjackal has quit IRC21:15
*** priteau has joined #openstack-infra21:18
*** wolverineav has quit IRC21:24
*** priteau has quit IRC21:25
*** yamamoto has quit IRC21:27
*** yamamoto has joined #openstack-infra21:27
*** smarcet has joined #openstack-infra21:27
*** smarcet has left #openstack-infra21:27
*** wolverineav has joined #openstack-infra21:29
*** jamesmcarthur has quit IRC21:30
*** jamesmcarthur has joined #openstack-infra21:32
*** jamesmcarthur has quit IRC21:32
*** wolverineav has quit IRC21:35
*** wolverineav has joined #openstack-infra21:35
corvusyay: https://hub.docker.com/r/opendevorg/gerrit/tags has 2.1521:40
corvusso the "old" version of the gerrit build job (which just landed) works :)21:40
fungiyep, cleanup seems to be working then?21:44
fungioh, or was it https://hub.docker.com/r/opendevorg/gitea/tags which we were watching for cleanup21:45
openstackgerritSagi Shnaidman proposed openstack-infra/nodepool master: Support userdata for instances in openstack  https://review.openstack.org/63064921:45
corvusfungi: yeah it was gitea, but also since there are no change_ tags in gerrit.... it's still working :)21:49
openstackgerritJeremy Stanley proposed openstack-infra/system-config master: Correct auth_openid scope in survey vhost  https://review.openstack.org/63132021:49
corvusi've approved the next batch of delete/add changes, so next up is to verify the new jobs work the same way.21:49
fungiclarkb: oops, i didn't meant to parent that to the upgrade change, so have rebased it separate and will re-add the approval21:50
fungiwas wondering why it hadn't merged yet21:51
*** gouthamr has quit IRC21:56
*** gouthamr_ is now known as gouthamr21:56
*** jamesmcarthur has joined #openstack-infra21:58
*** priteau has joined #openstack-infra21:59
*** priteau has quit IRC22:01
*** jamesmcarthur has quit IRC22:02
*** trown is now known as trown|outtypewww22:05
openstackgerritMerged openstack-infra/system-config master: Remove all docker image building jobs  https://review.openstack.org/63108422:07
openstackgerritMerged openstack-infra/system-config master: Add docker image building jobs  https://review.openstack.org/63108522:07
*** imacdonn has quit IRC22:07
*** imacdonn has joined #openstack-infra22:07
clarkbcorvus: update on voip.ms, scott says at least one of their payment methods works for foundation side of things so when he is here he and I will set up an account together and we can shift to that22:10
clarkbcorvus: I think that means we can probably leave the account in its current state and we'll swtich numbers/accounts in the near future22:10
*** jcoufal has quit IRC22:12
clarkbfungi: sorry I had completely missed that22:12
corvusclarkb: great, thanks!22:13
clarkbwe'll be in the same physical location next week22:13
clarkbI guess Ileft that bit out22:13
fungiokay, headed out to find dinner, should be back in an hour or so22:14
*** yamamoto has quit IRC22:14
openstackgerritClark Boylan proposed openstack-infra/system-config master: Upgrade review.o.o to gerrit 2.13.12  https://review.openstack.org/63134622:15
clarkbI'm going to WIP ^ but I think its ready to go as soon as we are happy with what review-dev looks like22:15
openstackgerritMerged openstack-infra/system-config master: Add jinja-init image jobs  https://review.openstack.org/63108622:19
openstackgerritDouglas Mendizábal proposed openstack-infra/project-config master: Add ansible roles for managing HSM software  https://review.openstack.org/63132622:22
*** ijw has quit IRC22:24
openstackgerritDouglas Mendizábal proposed openstack-infra/project-config master: Add ansible roles for managing HSM software  https://review.openstack.org/63132622:25
corvusclarkb, fungi: looks like the new jinja-init jobs worked: https://review.openstack.org/631086  and  https://hub.docker.com/r/opendevorg/jinja-init/tags22:28
openstackgerritDouglas Mendizábal proposed openstack-infra/project-config master: Add ansible roles for managing HSM software  https://review.openstack.org/63132622:28
corvusi've approved the remaining changes22:28
*** boden has quit IRC22:28
*** wolverineav has quit IRC22:28
*** yamamoto has joined #openstack-infra22:29
*** wolverineav has joined #openstack-infra22:30
*** jtomasek_ has quit IRC22:32
mordredcorvus: that's so exciting!22:32
*** yamamoto has quit IRC22:33
*** jtomasek has joined #openstack-infra22:33
*** ijw has joined #openstack-infra22:35
*** eernst has quit IRC22:36
*** eernst has joined #openstack-infra22:38
*** ijw has quit IRC22:40
*** jtomasek has quit IRC22:42
*** rcernin has joined #openstack-infra22:45
openstackgerritMerged openstack-infra/system-config master: Correct auth_openid scope in survey vhost  https://review.openstack.org/63132022:49
*** threestrands has joined #openstack-infra22:51
*** ijw has joined #openstack-infra22:52
openstackgerritSagi Shnaidman proposed openstack-infra/nodepool master: Support userdata for instances in openstack  https://review.openstack.org/63064922:54
openstackgerritMerged openstack-infra/system-config master: Add gitea-init image jobs  https://review.openstack.org/63108722:57
*** slaweq has quit IRC23:03
*** rascasoft has quit IRC23:13
openstackgerritMerged openstack-infra/system-config master: Add gitea image jobs  https://review.openstack.org/63108823:13
openstackgerritMerged openstack-infra/system-config master: Add gerrit image jobs  https://review.openstack.org/63108923:13
*** rlandy is now known as rlandy|bbl23:26
*** slaweq has joined #openstack-infra23:29
*** efried has quit IRC23:30
corvusclarkb, fungi, mordred: https://hub.docker.com/r/opendevorg/gerrit/tags still looks good!  and all the promote jobs succeeded23:33
*** gfidente|afk has quit IRC23:33
corvusi'll start drafting emails now23:33
clarkbwoot23:34
*** slaweq has quit IRC23:34
*** ijw has quit IRC23:35
corvusis there a list of recommended subject tags for openstack-discuss?23:35
clarkbcorvus: project names and all iirc, I don't think we have something that would capture docker things well23:36
clarkbhogepodge: ^ might know though23:36
mordredmaybe [containers] ? that's the name of the IRC channel ...23:36
corvuswell, it's not about the openstack container project(s)23:36
mordredthat's a good point23:36
fungithere is a published list of subject tags, yes23:36
corvushttps://docs.openstack.org/project-team-guide/open-community.html#mailing-lists23:37
corvusit's right where i was looking for it.  in the mailing list description.23:37
fungihttps://docs.openstack.org/project-team-guide/open-community.html#mailing-lists23:37
corvusit's just that it wasn't a hyperlink so i didn't see it.23:37
hogepodgeI typically use K8s for anything related to kubernetes, the project name if it’s a container specific project (so loci for example)23:37
fungid'oh, i'm, slow23:37
fungijust returned from the bar, sorry!23:38
corvusi don't suppose we can "a href" that ?23:38
hogepodgeI’ve never had cause to call out “containers” as a generic23:38
corvusit's for anyone who might want to build a container image.23:38
corvusso... anyone.23:38
hogepodgemordred: containers is misleading because if I’m correct it’s the magnum channel23:38
fungicorvus: i'm happy to experiment with values in the list description field, sure23:38
hogepodgeFor irc23:39
corvusi'm thinking '[infra][dev]'23:39
fungiwfm23:39
mordredcorvus: wfm23:39
hogepodgeWhat’s the context for the mail?23:39
fungiexciting containery infrastructure things23:39
corvushogepodge: new zuul feature for building/publishing containers23:39
corvusi'll etherpad it and link here in a bit23:40
corvus(i'd like input on my write-up before i send it)23:40
fungii'll post-dinner catch up on all the discussion23:40
hogepodgeI would just use Zuul as the tag and containers in the subject23:40
fungitechnically [zuul] doesn't fit any of the recommended subject tags. practically however, it's been sort of a free-for-all so far23:41
fungiso... why not23:42
corvushogepodge: zuul isn't an openstack project so that's not appropriate.  the feature already exists in zuul, the point of this is to announce to the openstack development community (users of zuul) that there is a new thing they can use.23:42
fungiyeah, [dev][infra] wfm23:42
*** efried has joined #openstack-infra23:43
hogepodgeThen zuul and containers in the subject line23:43
*** sshnaidm is now known as sshnaidm|afk23:43
corvusi think i can swing that23:44
fungikeywords which are not subject tags ftw23:44
*** rascasoft has joined #openstack-infra23:46
*** ijw has joined #openstack-infra23:50
*** rascasoft has quit IRC23:51
*** ijw has quit IRC23:55
pabelangerI've kinda missed the promote pipeline stuff, looking forward to the ML post23:56
mordredpabelanger: it's super bomb23:56
mordredpabelanger: you end up with the image in dockerhub being the actual image you built in gate - rather than a rebuild in post23:57
pabelangernice23:59

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