Tuesday, 2022-06-21

@blaisep-sureify:matrix.orgThis is great, now I understand how software factory does the github integration. Thank you02:34
-@gerrit:opendev.org- David Ostrovsky proposed: [zuul/zuul-jobs] 846909: ensure-java role: Bump default java_version to 11 https://review.opendev.org/c/zuul/zuul-jobs/+/84690906:34
@chgans:matrix.orgI've reported this issue: https://storyboard.openstack.org/#!/story/201009809:53
@chgans:matrix.orgThis is related to my attempt at demo'ing zuul+gerrit+repo (AOSP)09:53
@chgans:matrix.orgI found it strange not being abble to attached files (demo source, screenshot, ...).09:54
@chgans:matrix.orgPlease let me know if there's anything i can do to improve the issue report09:54
@fungicide:matrix.org> <@chgans:matrix.org> I found it strange not being abble to attached files (demo source, screenshot, ...).13:59
yes, latest storyboard has a story attachments feature, but we're stuck on an older version in opendev until someone finds time to redo the deployment mechanism (we need to move to a newer platform in order to have new enough python for it, and our old configuration management software is incompatible with the newer platform we'd need, so the plan is to switch to deploying containers of storyboard services)
@fzzfh:matrix.orghello folks. I use softwarefactory zuul with remote logserver as CI. but zuul don't return logserver address on opendev comment.  I have add zuul/web-root: http://47.111.113.211:8081/zuul/t/local. now it return http://47.111.113.211:8081/logs/19/194c4c8994a6a63782408f9c078dd9e3fb9ea89c/post/config-update/3a89a7b/, but my actual log address is http://47.111.113.211:8081/logs/19/194c4c8994a6a63782408f9c078dd9e3fb9ea89c/post/config-update/3a89a7b/.  which param shoud I modify, any help would be appreciated.15:44
@fzzfh:matrix.org * hello folks. I use softwarefactory zuul with remote logserver as CI. but zuul don't return logserver address on opendev comment.  I have add zuul/web-root: http://47.111.113.211:8081/zuul/t/local. now it return http://47.111.113.211:8081/zuul/t/local/build/19e74214767543b298c4e6c94c7788ed , but my actual log address is http://47.111.113.211:8081/logs/19/194c4c8994a6a63782408f9c078dd9e3fb9ea89c/post/config-update/3a89a7b/.  which param shoud I modify, any help would be appreciated.15:46
@fzzfh:matrix.orgI had use uploads-logs role with var: zuul_log_url. this include zuul_reutrn: log_url15:47
@fzzfh:matrix.orglog can upload to remote logserver. but zuul don't return actual address.15:48
@fungicide:matrix.orgfzzf: opendev's zuul doesn't comment with a link to the job logs, it comments with a link to the build result on the zuul dashboard15:48
@fungicide:matrix.orgwhat you've observed is the intended behavior15:50
@fzzfh:matrix.orgin https://review.opendev.org/c/openstack/manila/+/845586, like https://elab-os-logsrv.delllabs.net/86/845586/5/check/DellEMC_Unity_Manila_dhss_true/dfedc34/DellEMC_Unity_Manila_dhss_true/55515:51
@fzzfh:matrix.orgwhat should I modify, for return log..15:51
@clarkb:matrix.orgIn opendev the base job runs the upload-logs-swift role which registers upload_results.url with zuul as a zuul return with this path: zuul.log_url15:51
@fungicide:matrix.orgfzzf: there isn't an option to do what you're describing. the intended behavior is that zuul comments with the build result page for each build15:52
@fungicide:matrix.orgcan you describe why that isn't suitabel for your use case?15:53
@fungicide:matrix.org * can you describe why that isn't suitable for your use case?15:53
@clarkb:matrix.orgOh I see its the link to the build record in zuul that is the problem. Ignore me fungi has got it handle15:53
@clarkb:matrix.org * Oh I see its the link to the build record in zuul that is the problem. Ignore me fungi has got it handled15:53
@fzzfh:matrix.org I see other CI have return log on comment. what should I do for this?15:55
@fzzfh:matrix.org> <@clarkb:matrix.org> In opendev the base job runs the upload-logs-swift role which registers upload_results.url with zuul as a zuul return with this path: zuul.log_url15:57
do you mean use upload-logs-swift role? I found SF base job have use add-fileserver and upload-logs role.
@fungicide:matrix.orgfzzf: those ci systems are using different software. i don't understand your question15:57
@fzzfh:matrix.orgI want let zuul return log on opendev comment as CI..15:57
@fungicide:matrix.orgfzzf: zuul doesn't do that15:58
@fzzfh:matrix.org * I want let zuul return log on opendev comment for finish CI work15:58
@fungicide:matrix.orgit's not how zuul is designed to work15:58
@fzzfh:matrix.orgI see zuul have return like http://47.111.113.211:8081/zuul/t/local/build/19e74214767543b298c4e6c94c7788ed on comment..16:00
@fzzfh:matrix.orgwhen I use **success: gerrit:{}**. zuul would return result on comment16:02
@tristanc_:matrix.orgfzzf: I guess your Zuul service is not publicly available, in that case, you'll have to setup the zuul-web service on the public host.16:10
@clarkb:matrix.orgI guess the use case here is not wanting to make the zuul public, but being ok with the logs thesmelves being public?16:11
@tristanc_:matrix.orgit's a common issue with third-party CI where the zuul control plane is not public16:11
@clarkb:matrix.orgThe zuul interface for job results really is much nicer than the raw logs. I'm not going to speak for openstack and the requirements they want to maintain but it seems like a good idea to make that a requirement16:12
@fzzfh:matrix.orgI want let CI return logs, what should be done. is there any other way16:27
@clarkb:matrix.orgfzzf:  currently I think theo nly way is to report the build link.16:38
@clarkb:matrix.org * fzzf:  currently I think the only way is to report the build link.16:38
-@gerrit:opendev.org- Zuul merged on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com: [zuul/nodepool] 846625: Suppress component registry logging in command https://review.opendev.org/c/zuul/nodepool/+/84662517:10
@fungicide:matrix.orgfzzf: zuul is designed so you can run the zuul-web service on a completely separate machine and just give it access to the database and zookeeper. that way your build results pages and the rest of the zuul dashboard can be put in a "dmz" or otherwise outside whatever packet filtering you have in place for the rest of the deployment. alternatively, you can set up a reverse proxy or do some sort of port redirection to make it accessible from the internet17:44
@clarkb:matrix.orgcorvus: can you check the comments on https://review.opendev.org/c/zuul/zuul-jobs/+/846909 as far as sneding notice for that change goes18:16
@jim:acmegating.comClark: fungi i *think* that one makes sense to announce... i feel like it's highly likely to be used outside of opendev, so it's high on the list of roles i wouldn't consider codesearch to be exhaustive for.  i think the main argument against it would be "it's so ancient of course no one uses that default" but then i think about enterprise use.  at least, a 2 week notice seems prudent?  or am i missing something?18:30
@clarkb:matrix.orgI'll defer to your judgement on that. I'm not sure how many others are using zuul with java projects. I don't mind either as we've already set it to 11 where we need 1118:36
@clarkb:matrix.orgFrom a design perspective I almost wonder if we shouldn't drop defautls like that entirely and force users to be explicit about what they want always18:36
@jim:acmegating.comdefinitely yes for non-opendev java users with zuul.  i can't say how many are using that role from upstream though.18:37
@jim:acmegating.comi'll send the email18:39
@jim:acmegating.comClark: i think i agree a discussion about defaults sounds good.  maybe the expectation could be that projects set project vars for things like that.18:41
@jim:acmegating.comClark: maybe worth a zuul-discuss thread?18:42
@clarkb:matrix.orgSure. Do you want me to start one?18:42
@jim:acmegating.comi just sent the announce email18:43
@jim:acmegating.comzuul-maint: how is this proposed tag for nodepool (third time's the charm?):18:44
commit a63f128d7392bf0e60f6150637a5564da536b0e3 (HEAD -> master, tag: 6.1.0, origin/master, refs/changes/25/846625/1)
@jim:acmegating.comzuul-maint: and also, how does this look for zuul (the rare synchronized zuul/nodepool release!):18:46
commit f3e4d399b0edcd98c97176fdfe38251e0ea86bd4 (HEAD -> master, tag: 6.1.0, origin/master)
@mhuin:matrix.orgwanna make it a hat trick and add a release for zuul-client? 🙂18:47
@jim:acmegating.comzuul-maint: and this for zuul-client (sure thing!):18:48
commit a6ce77acffd852219fd43a6dc61cbe637aa85bf2 (HEAD -> master, tag: 0.1.0, origin/master, origin/HEAD, refs/changes/22/834522/1)
@mhuin:matrix.orgdo you think it warrants a minor version bump? Last release was 0.0.418:50
@clarkb:matrix.org> <@jim:acmegating.com> zuul-maint: and also, how does this look for zuul (the rare synchronized zuul/nodepool release!):18:50
> commit f3e4d399b0edcd98c97176fdfe38251e0ea86bd4 (HEAD -> master, tag: 6.1.0, origin/master)
This is a change ahead of opendev but it is only a docs update. LGTM
@jim:acmegating.commhu: yeah, i think the new commands + config file change warrant it. (looking back, 0.0.3 probably should have been 0.1.0 but oh well) -- https://zuul-ci.org/docs/zuul-client/releasenotes.html18:58
@mhuin:matrix.orgok LGTM either way18:58
@iwienand:matrix.orgin nodepool cmd line, we have "image-pause" and "image-unpause", but do any of the commands show the current paused status of images?19:46
@jim:acmegating.comianw: believe it or not, that's literally on my todo list for today, so maybe wait a few hours :)20:00
@iwienand:matrix.orgheh, i was going to write something, but your approach sounds even better :)20:03
@jim:acmegating.comianw: ah, apparently "dib-image-list" will have `paused` in the `State` column20:12
@jim:acmegating.comthat will show output for the pause commands, but not show what's in the config file20:18
@jim:acmegating.comianw: were you looking for the "zk pause" or the "config file pause" ?20:24
@iwienand:matrix.orgcorvus: the ZK pause -- basically one admin had paused a build a while ago via cmdline, and i was unsure if it was currently unpaused20:26
@iwienand:matrix.orghave to admit i didn't look at the image-list, i guess it makes some sense that the current image is "paused" and so no new images are built20:26
@iwienand:matrix.orgalthough i was thinking of it more "we've paused building new images", so the current image is active, if that makes sense20:27
@jim:acmegating.comianw: "dib-image-list" but yeah... the data model isn't quite right.... :/20:28
@jim:acmegating.combut it's pretty close20:28
@jim:acmegating.com(image-list, which shows the uploads, won't show that it's paused, so they're still active; it's really the building, and therefore dib-image-list which shows the builds that will output `paused` as the state)20:29
@iwienand:matrix.orgyeah, sorry, dib-image-list.  gentoo is an example from http://nl01.opendev.org/dib-image-list20:30
@iwienand:matrix.orgboth those being paused implies that new builds are paused.  an explicit "image-status" type command that listed the image names and paused status might be slightly clearer 20:33
@jim:acmegating.comianw: i agree; i think ultimately we need to rename a bunch of commands too.  that's what "image-list" should be, and then we should have image-build-list and image-upload-list.  this may be several changes. :)20:34
-@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed on behalf of Simon Westphahl: [zuul/nodepool] 725810: Expose image build requests in web UI and cli https://review.opendev.org/c/zuul/nodepool/+/72581020:45
@jim:acmegating.comthat was just a rebase ^20:46
@jim:acmegating.comi think that the dib-request-list command there should also be combined with what we were just talking about...20:46
-@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/nodepool] 847092: Convert dib-request-list to image-status command https://review.opendev.org/c/zuul/nodepool/+/84709221:13
@jim:acmegating.comianw: swest tobiash ^ the manual build request and manual image pause systems seemed similar enough that it makes sense to have an "image-status" command that output information about both at the same time.  so i morphed swest's "dib-request-list" command into "image-status".  how's that look?21:14
@jim:acmegating.com```21:15
+-------------+--------+---------------+-------------------+
| Image | Paused | Build Request | Build Request Age |
+-------------+--------+---------------+-------------------+
| fake-image | True | pending | 00:00:12:29 |
| fake-image2 | False | | |
+-------------+--------+---------------+-------------------+
```
@jim:acmegating.comthat's what that looks like21:15
@iwienand:matrix.org++ the regular naming pattern of image-* makes sense to 21:22
@iwienand:matrix.orgto me21:22
-@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/nodepool] 847106: Add nodepool.image_build_requests metric https://review.opendev.org/c/zuul/nodepool/+/84710621:53
@jim:acmegating.compushed zuul, nodepool, zuul-client tags22:49
@jim:acmegating.comand emails sent!23:59

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