Monday, 2020-12-21

« Sunday, 2020-12-20 Index Tuesday, 2020-12-22 »
*** tosky has quit IRC00:12
*** sshnaidm is now known as sshnaidm|afk00:22
*** holser_ has quit IRC00:49
*** ianychoi_ has quit IRC00:57
*** gundalow has quit IRC00:57
*** nhicher has quit IRC00:57
*** mhu has quit IRC00:57
*** irclogbot_1 has quit IRC00:59
*** openstackgerrit has quit IRC01:00
*** irclogbot_2 has joined #zuul01:01
*** ianychoi_ has joined #zuul01:17
*** gundalow has joined #zuul01:17
*** nhicher has joined #zuul01:17
*** mhu has joined #zuul01:17
*** holser has joined #zuul01:21
*** holser has quit IRC02:15
*** ikhan has joined #zuul02:17
*** ikhan has quit IRC02:22
*** ikhan has joined #zuul02:32
*** ikhan has quit IRC02:37
*** ikhan has joined #zuul03:03
*** bhavikdbavishi has joined #zuul03:21
*** ikhan has quit IRC03:26
*** ikhan has joined #zuul03:26
*** ianw is now known as ianw_pto04:22
*** bhagyashris is now known as bhagyashri|ruck05:27
*** bhagyashri|ruck is now known as bhagyashris|ruck05:28
*** evrardjp has quit IRC05:33
*** evrardjp has joined #zuul05:33
*** vishalmanchanda has joined #zuul05:56
*** zenkuro has joined #zuul06:25
*** saneax has joined #zuul06:39
*** bhavikdbavishi1 has joined #zuul06:42
*** bhavikdbavishi has quit IRC06:43
*** bhavikdbavishi1 is now known as bhavikdbavishi06:43
*** hamalq has joined #zuul06:50
*** bhavikdbavishi has quit IRC06:51
*** zenkuro has quit IRC06:51
*** zenkuro has joined #zuul06:52
*** bhavikdbavishi has joined #zuul06:53
*** hamalq has quit IRC06:55
*** bhavikdbavishi has quit IRC06:59
*** bhavikdbavishi has joined #zuul07:50
*** bhavikdbavishi1 has joined #zuul07:53
*** bhavikdbavishi has quit IRC07:54
*** bhavikdbavishi1 is now known as bhavikdbavishi07:54
*** rpittau|afk is now known as rpittau08:07
*** bhavikdbavishi has quit IRC08:29
*** bhavikdbavishi has joined #zuul08:29
*** bhavikdbavishi1 has joined #zuul08:38
*** bhavikdbavishi has quit IRC08:39
*** bhavikdbavishi1 is now known as bhavikdbavishi08:39
*** cloudnull has quit IRC08:53
*** cloudnull6 has joined #zuul08:53
*** jpena|off is now known as jpena08:55
*** jfoufas1 has joined #zuul09:04
*** bhavikdbavishi has quit IRC09:05
*** bhavikdbavishi has joined #zuul09:07
*** bhavikdbavishi has quit IRC09:17
*** nils has joined #zuul09:17
*** hashar has joined #zuul09:18
*** hamalq has joined #zuul09:28
*** hamalq_ has joined #zuul09:29
*** hamalq has quit IRC09:33
*** hamalq_ has quit IRC09:34
*** ykarel has joined #zuul10:02
*** tosky has joined #zuul10:03
*** bhavikdbavishi has joined #zuul10:38
*** ykarel_ has joined #zuul10:55
*** ykarel has quit IRC10:58
*** bhavikdbavishi1 has joined #zuul11:01
*** bhavikdbavishi has quit IRC11:03
*** bhavikdbavishi1 is now known as bhavikdbavishi11:03
*** sshnaidm|afk is now known as sshnaidm11:17
*** bhavikdbavishi has quit IRC11:26
*** hamalq has joined #zuul11:30
*** hamalq has quit IRC11:35
*** ykarel_ is now known as ykarel11:42
*** rfolco has joined #zuul11:57
*** hamalq has joined #zuul11:58
*** msuszko has quit IRC11:58
*** msuszko has joined #zuul12:00
*** hamalq has quit IRC12:03
*** bhavikdbavishi has joined #zuul12:03
*** bhavikdbavishi has quit IRC12:18
*** bhavikdbavishi has joined #zuul12:21
*** rlandy has joined #zuul12:27
*** jpena is now known as jpena|lunch12:33
*** bhavikdbavishi has quit IRC12:38
*** jfoufas1 has quit IRC12:40
*** bhavikdbavishi has joined #zuul12:49
*** sduthil has joined #zuul13:04
*** iurygregory has joined #zuul13:16
*** bhavikdbavishi has quit IRC13:18
*** ikhan has quit IRC13:20
*** jpena|lunch is now known as jpena13:33
*** ikhan has joined #zuul13:49
*** ikhan has joined #zuul13:49
*** ykarel_ has joined #zuul13:52
*** ykarel has quit IRC13:54
*** ykarel_ is now known as ykarel13:55
*** cloudnull6 is now known as kecarter13:58
*** hamalq has joined #zuul13:59
*** kecarter is now known as cloudnull14:03
*** hamalq has quit IRC14:04
*** sduthil has quit IRC14:04
*** sduthil has joined #zuul14:04
*** ykarel has quit IRC15:36
*** vishalmanchanda has quit IRC15:53
*** hamalq has joined #zuul16:00
*** hamalq has quit IRC16:04
avasscorvus: maybe I should have put a comment about this but it seems like defaulting to private logs is a better idea than defaulting to public logs since someone could easily forget about making the logs private16:22
avassre https://review.opendev.org/c/zuul/zuul-jobs/+/76448316:22
avassalso buckets are private by default as well so users could expect the logs to be private by default16:23
corvusavass: well, the other roles are public by default and i think it makes sense to be consistent16:24
avassI agree with that but it would make more sense to make others private by default16:24
avassbut that would require a bit more work :)16:25
corvusi'm not so sure.  private buckets are inaccessible by anyone; it takes a lot of thought to design a system with private buckets that's accessible, so i don't think people are going to forget about it16:25
avassI still think it's a better idea to not expose data unless explicitly told to do so16:27
corvusgenerally, i'd agree, but it's pretty standard for a ci/cd system to expose logs publically, especially one that has no user authentication of its own.  plus, i think this makes for a better experience for new users.16:29
corvusavass: maybe we could leave public as the default and make folks more aware by highlighting that in the docs?16:30
corvusavass: or, if you wanted to be very security concious, we could remove the default altogether and require that a user explicitly choose one or the other.16:30
corvusavass: bottom line though: i think all the roles should behave the same way16:31
*** bhavikdbavishi has joined #zuul16:33
*** hamalq has joined #zuul17:00
*** ikhan has quit IRC17:09
*** openstackgerrit has joined #zuul17:12
openstackgerritAlbin Vass proposed zuul/zuul-jobs master: Add configuration to make logs public  https://review.opendev.org/c/zuul/zuul-jobs/+/76448317:12
*** rpittau is now known as rpittau|afk17:13
*** hamalq has quit IRC17:14
*** hamalq has joined #zuul17:14
avasscorvus: ^ that should default the logs to public then. I think we'll be using a private endpoint with public read anyway.17:15
corvusavass: cool; and yeah, that's the other thing i was thinking about -- it will be nice to learn about other log config scenarious (like you describe -- or if folks are using private read, how they are doing it)17:16
*** hamalq has quit IRC17:17
*** hamalq has joined #zuul17:17
*** bhavikdbavishi1 has joined #zuul17:23
*** holser_ has joined #zuul17:25
*** fdegir has quit IRC17:25
*** bhavikdbavishi has quit IRC17:25
*** bhavikdbavishi1 is now known as bhavikdbavishi17:25
*** fdegir has joined #zuul17:25
*** ikhan has joined #zuul17:25
avasscorvus: yeah not entirely sure and I have a feeling that putting  zuul_return for the log_url into the role might make that a bit harder. but adding a toggle for zuul_return when someone needs a different solution should be easy enough.17:27
*** ikhan has quit IRC17:30
openstackgerritAlbin Vass proposed zuul/zuul-jobs master: upload-artifactory: no_log upload task  https://review.opendev.org/c/zuul/zuul-jobs/+/76811117:33
avassalso that ^ was fun :)17:33
openstackgerritAlbin Vass proposed zuul/zuul-jobs master: upload-artifactory: no_log upload task  https://review.opendev.org/c/zuul/zuul-jobs/+/76811117:33
*** ikhan has joined #zuul17:36
openstackgerritJimmy McArthur proposed zuul/zuul-website master: Adding Get Started page  https://review.opendev.org/c/zuul/zuul-website/+/76652017:40
*** ikhan has quit IRC17:41
openstackgerritClark Boylan proposed zuul/zuul-website master: Update the volvo user story links  https://review.opendev.org/c/zuul/zuul-website/+/76811317:42
*** reiterative has quit IRC17:44
*** tristanC has quit IRC17:44
*** jpena has quit IRC17:45
*** jkt has quit IRC17:45
*** jkt has joined #zuul17:45
*** reiterative has joined #zuul17:45
*** tristanC has joined #zuul17:46
*** jpena has joined #zuul17:48
openstackgerritMatthieu Huin proposed zuul/zuul master: Web UI: allow a privileged user to request autohold  https://review.opendev.org/c/zuul/zuul/+/76811517:52
*** reiterative has quit IRC17:53
*** reiterative has joined #zuul17:54
*** hashar has quit IRC18:00
openstackgerritMatthieu Huin proposed zuul/zuul master: web UI: allow a privileged user to re-enqueue a change  https://review.opendev.org/c/zuul/zuul/+/73677218:00
*** jpena is now known as jpena|off18:01
*** nils has quit IRC18:05
*** ikhan has joined #zuul18:09
*** bhavikdbavishi has quit IRC18:21
mhupabelanger: what's your email on github? we can try to set up a small demo for the admin actions in zuul's web ui18:36
clarkbhello zuulians. The foundation has asked us to start to put together a zuul project update for the foundation's annual report. I've put 2019's update in an etherpad here https://etherpad.opendev.org/p/zuul-2020-annual-report to give context/example and below that have started to brainstorm ideas in an outline18:43
clarkbif you'd like to help everything from adding ideas to that etherpad to drafting the actual content would be great. I'll be chipping away at that over the next little bit too18:43
corvusavass: want to +1 https://review.opendev.org/768113 ?18:49
fungii'll also be using the script at https://review.opendev.org/729293 to gather some basic usage stats from opendev's gerrit and mailman services after the end of the year, so we'll already have info like number of changes merged, reviews done, patchsets uploaded, distinct committers and reviewers for zuul/ namespace repos, lists.zuul.org ml posts, number of people posting to the lists, et cetera to draw on in the18:54
fungireport as well18:54
avasscorvus: sur18:56
avasse18:56
avassI'm fine with putting that there18:56
corvusavass, clarkb: thanks!  +w18:56
openstackgerritMerged zuul/zuul-website master: Update the volvo user story links  https://review.opendev.org/c/zuul/zuul-website/+/76811319:00
openstackgerritJimmy McArthur proposed zuul/zuul-website master: Adding Get Started page  https://review.opendev.org/c/zuul/zuul-website/+/76652019:15
*** ikhan has quit IRC19:30
*** ikhan has joined #zuul19:30
openstackgerritMatthieu Huin proposed zuul/zuul master: Web UI: allow a privileged user to request autohold  https://review.opendev.org/c/zuul/zuul/+/76811519:49
openstackgerritMatthieu Huin proposed zuul/zuul master: Web UI: allow a privileged user to request autohold  https://review.opendev.org/c/zuul/zuul/+/76811519:51
*** yoctozepto has quit IRC19:55
*** yoctozepto has joined #zuul19:56
openstackgerritJimmy McArthur proposed zuul/zuul-website master: Adding Get Started page  https://review.opendev.org/c/zuul/zuul-website/+/76652020:03
*** ikhan has quit IRC20:06
*** ikhan has joined #zuul21:20
*** PrinzElvis has quit IRC21:33
*** hamalq has quit IRC21:34
*** PrinzElvis has joined #zuul21:34
*** hamalq has joined #zuul21:34
corvusavass: i left some q's on the cache change (i like it :)21:49
*** zenkuro has quit IRC21:58
corvusmhu: can you look into my comment on https://review.opendev.org/767691 ?22:01
*** ikhan has quit IRC22:28
*** ikhan has joined #zuul22:46
*** ikhan has joined #zuul22:49
avasscorvus: cool :). ah yeah I can try to make the docs a bit clearer. but zuul_cache_artifacts is a list of aritfacts that should be prepared in the pre-run while zuul_cache_name specifies which one of those artifacts the user wants to pull from the prepared storage22:56
avassor when pushing an artifact the name it should be stored under22:56
corvusavass: oh that makes sense, then yeah i agree it's probably just a doc change22:57
avassI also specifically didn't want to force all cache backend to use a prepared cache on the node in case they can give access to them directly somehow22:57
avasssay a using a temporary read-only key to the storage or something like that.22:58
corvusyeah, that's possible with swift at least22:59
avasscorvus: also: https://review.opendev.org/c/zuul/zuul/+/766982 if that's fine. But I don't think there's any hurry so we could go with zuul.pipeline.post_review as well23:03
avasswe could probably filter on pipeline name for now23:03
openstackgerritMerged zuul/zuul master: doc: update example setup playbook  https://review.opendev.org/c/zuul/zuul/+/76589923:03
avassIf we want to deprecate the current zuul.pipeline that is23:03
corvusavass: i think 766982 is fine (i think it's unambiguous -- a job where job.post_review=True will only ever run in a pipeline where pipeline.post_review is true).  but i think that needs a doc update23:05
corvusavass: here https://zuul-ci.org/docs/zuul/reference/jobs.html#zuul-variables23:05
avassah yeah that might be good23:07
corvusavass: i think the ideal sequence would be to update and merge 766982, then structure the cache role to rely on it23:07
avassyep23:07
openstackgerritAlbin Vass proposed zuul/zuul master: Add post_review pipeline variable to jobs  https://review.opendev.org/c/zuul/zuul/+/76698223:14
avasscorvus: ^23:14
corvuslgtm23:15
avassIn that case I'll allow the cache role to upload artifacts in pre-review but not be able to update the project-global cache when it does23:18
*** rpittau|afk has quit IRC23:21
*** johnsom has quit IRC23:21
*** ericsysmin has quit IRC23:22
*** maxamillion has quit IRC23:23
*** ikhan has joined #zuul23:24
avassI found out about pre-signed urls for s3 objects earlier today so the s3 implementation could probably be optimized too :)23:25
*** ikhan has quit IRC23:32
*** tosky has quit IRC23:41
« Sunday, 2020-12-20 Index Tuesday, 2020-12-22 »

Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!