| *** marios is now known as marios|ruck | 05:15 | |
| *** rpittau|afk is now known as rpittau | 06:15 | |
| opendevreview | Felix Edel proposed zuul/zuul master: web UI: Show autohold and re-enqueue butttons next to title result pages https://review.opendev.org/c/zuul/zuul/+/804043 | 07:21 |
|---|---|---|
| opendevreview | Felix Edel proposed zuul/zuul master: web UI: Alternative styling for autohold and re-enqueue buttons https://review.opendev.org/c/zuul/zuul/+/804043 | 07:22 |
| *** jpena|off is now known as jpena | 07:37 | |
| opendevreview | Matthieu Huin proposed zuul/zuul master: web UI: user login with OpenID Connect https://review.opendev.org/c/zuul/zuul/+/734082 | 09:01 |
| opendevreview | Matthieu Huin proposed zuul/zuul master: Add authentication-realm attribute to tenants https://review.opendev.org/c/zuul/zuul/+/735586 | 09:10 |
| opendevreview | Matthieu Huin proposed zuul/zuul master: web UI: allow a privileged user to dequeue a change https://review.opendev.org/c/zuul/zuul/+/734850 | 09:11 |
| opendevreview | Dong Zhang proposed zuul/zuul master: Disable aliases in inventory.yaml for better readibility https://review.opendev.org/c/zuul/zuul/+/802674 | 09:15 |
| opendevreview | Dong Zhang proposed zuul/zuul master: WIP: Disable aliases in inventory.yaml for better readibility https://review.opendev.org/c/zuul/zuul/+/802674 | 09:21 |
| opendevreview | Dong Zhang proposed zuul/zuul master: WIP: Disable aliases in inventory.yaml for better readibility https://review.opendev.org/c/zuul/zuul/+/802674 | 09:27 |
| opendevreview | Matthieu Huin proposed zuul/zuul master: web UI: allow a privileged user to re-enqueue a change https://review.opendev.org/c/zuul/zuul/+/736772 | 09:28 |
| opendevreview | Matthieu Huin proposed zuul/zuul master: Web UI: allow a privileged user to request autohold https://review.opendev.org/c/zuul/zuul/+/768115 | 09:35 |
| opendevreview | Matthieu Huin proposed zuul/zuul master: Web UI: add Autoholds, Autohold page https://review.opendev.org/c/zuul/zuul/+/768199 | 09:39 |
| YanivLevy[m] | Hi, I need help with the zuul 'quickstart' example. someone available to answer? | 10:55 |
| *** jpena is now known as jpena|lunch | 11:25 | |
| *** dviroel|out is now known as dviroel | 11:26 | |
| opendevreview | Dong Zhang proposed zuul/zuul master: WIP: Disable aliases in inventory.yaml for better readibility https://review.opendev.org/c/zuul/zuul/+/802674 | 11:39 |
| opendevreview | Matthieu Huin proposed zuul/zuul master: Web UI: add Autoholds, Autohold page https://review.opendev.org/c/zuul/zuul/+/768199 | 11:40 |
| RaviSalamani[m] | Hi, I know Zuul CI runs nodes with x86 architectures, Does it also support more architectures? | 12:18 |
| RaviSalamani[m] | * Hi, I know Zuul CI can run nodes with x86 architectures, Does it also support more architectures? | 12:18 |
| *** jpena|lunch is now known as jpena | 12:27 | |
| opendevreview | Matthieu Huin proposed zuul/zuul master: Web UI: add Autoholds, Autohold page https://review.opendev.org/c/zuul/zuul/+/768199 | 12:51 |
| opendevreview | Matthieu Huin proposed zuul/zuul master: REST API, Web UI: add pipelines' manager, triggers data in status https://review.opendev.org/c/zuul/zuul/+/736968 | 12:53 |
| opendevreview | Matthieu Huin proposed zuul/zuul master: web UI: allow a privileged user to promote a change https://review.opendev.org/c/zuul/zuul/+/781858 | 13:01 |
| corvus | Ravi Salamani: yes; for example, opendev runs arm test nodes. | 13:24 |
| avass[m] | I don't think there's anything stopping any kind of target as long as it supports ssh and python | 13:24 |
| corvus | avass: yep; if you use nodepool to build images there are some additional considerations about what platform to run the nodepool builder on. | 13:28 |
| fungi | yes, in opendev we run an aarch64/arm64 native nodepool builder, though in theory you could probably do it under emulation or with a cross-build environment, since we have arm resources available we just use them directly | 13:35 |
| opendevreview | Matthieu Huin proposed zuul/zuul master: web UI: allow a privileged user to promote a change https://review.opendev.org/c/zuul/zuul/+/781858 | 13:40 |
| opendevreview | Matthieu Huin proposed zuul/zuul master: Web UI: Add "Create Autohold Request" form, improve API error messages https://review.opendev.org/c/zuul/zuul/+/802559 | 13:59 |
| opendevreview | Matthieu Huin proposed zuul/zuul master: Example Docker compose: keycloak integration https://review.opendev.org/c/zuul/zuul/+/769943 | 14:07 |
| RaviSalamani[m] | <corvus> "Ravi Salamani: yes; for example," <- Any idea about ppc64le, s390 etc? | 14:42 |
| clarkb | RaviSalamani[m]: as avass[m] mentioned if you can run ssh and python it should be doable. You might discover some issues that need to be addressed (like perhaps cryptography doesn't compile on some arch though they seem to have good coverage) | 14:45 |
| fungi | RaviSalamani[m]: i would expect any of those to work if you have images nodepool can boot. if you want to use nodepool's builder service to create images with diskimage-builder, then it may need support (but it may also be able to run fine natively on those architectures) | 14:45 |
| clarkb | someone from IBM asked about a month ago if opendev had s390x test instances. My response was no (because we don't) but then basically said we'd be happy to give it a go as I don't expect that would be a problem for nodepool or zuul | 14:47 |
| clarkb | but may require a bit of effort to get going | 14:47 |
| opendevreview | Dong Zhang proposed zuul/zuul master: WIP: Disable aliases in inventory.yaml for better readibility https://review.opendev.org/c/zuul/zuul/+/802674 | 14:52 |
| opendevreview | Dong Zhang proposed zuul/zuul master: Disable aliases in inventory.yaml for better readibility https://review.opendev.org/c/zuul/zuul/+/802674 | 14:54 |
| opendevreview | Clark Boylan proposed zuul/zuul master: Fix race in test_lost_merge_requests https://review.opendev.org/c/zuul/zuul/+/803850 | 14:54 |
| clarkb | corvus: ^ that hit a linting error, I decided it was a good opporunity to try out gerrit's modern edit in change functionality | 14:55 |
| corvus | clarkb: thanks! | 14:56 |
| clarkb | we should probably try and land that as it seems a number of changes are hitting the issue that chagne fixes? | 14:56 |
| opendevreview | James E. Blair proposed zuul/zuul master: Join the scheduler thread before stopping the zk client https://review.opendev.org/c/zuul/zuul/+/803851 | 14:56 |
| corvus | clarkb: yep | 14:56 |
| corvus | tobiash: if you're around, can you re-review https://review.opendev.org/803850 please? | 14:56 |
| corvus | that's also going to be an actual-bug-in-production issue too (but very unlikely) | 14:58 |
| dmsimard | o/ 3.11 relnotes (https://zuul-ci.org/docs/zuul/reference/releasenotes.html#relnotes-3-11-0) mentions "Autohold requests are now stored in ZooKeeper, rather than in memory. As a result of this change, a new zuul CLI command, autohold-delete has been added to remove existing requests." <-- My zuul is rusty but does that mean autohold requests were | 15:25 |
| dmsimard | deleted automatically before and it's no longer the case ? | 15:25 |
| corvus | dmsimard: they were ephemeral and would be lost when zuul-scheduler stopped | 15:26 |
| mhuin | dmsimard, they weren't, until you restarted | 15:26 |
| dmsimard | ah, looking at autohold-list, it seems like we've accumulated quite the backlog and I guess we should delete them :p | 15:27 |
| fungi | yeah, we try to clean ours up in opendev occasionally | 15:28 |
| dmsimard | thanks ! | 15:28 |
| fungi | dmsimard: another thing which changed a while back is that held nodes will be automatically cleaned up when the autohold is deleted, whereas if you delete the nodes directly the autohold hangs around indefinitely | 15:29 |
| fungi | so if you get in the habit of deleting autoholds when you're done with the nodes, they don't tend to pile up quite as badly (at least not for us in opendev anyway) | 15:30 |
| fungi | and it's less work all around | 15:30 |
| dmsimard | looks like we did not have default_hold_expiration set in the scheduler's config, that will surely help keep things tidy | 15:36 |
| *** jpena is now known as jpena|off | 15:37 | |
| clarkb | just keep in mind that will delete the instances | 15:39 |
| dmsimard | clarkb: yeah, including the autohold request, right ? | 15:40 |
| clarkb | yes | 15:40 |
| dmsimard | fair enough, it'll be a safety net to avoid incurring costs for people that forget :) | 15:41 |
| *** marios|ruck is now known as marios|out | 16:01 | |
| opendevreview | Merged zuul/zuul master: Fix race in test_lost_merge_requests https://review.opendev.org/c/zuul/zuul/+/803850 | 16:21 |
| *** rpittau is now known as rpittau|afk | 16:31 | |
| opendevreview | Merged zuul/zuul master: Suppress spurious semaphore lock error logs https://review.opendev.org/c/zuul/zuul/+/803948 | 16:32 |
| opendevreview | Merged zuul/zuul master: Join the scheduler thread before stopping the zk client https://review.opendev.org/c/zuul/zuul/+/803851 | 16:36 |
| Shrews | I don't suppose anyone here knows of the existence of a zuul job that works with uploading to codecov.io, do they? | 17:13 |
| clarkb | I'm not aware of one, but aiui you just run their bash script with some credentials? | 17:18 |
| clarkb | shouldn't be too bad to vender that script in a zuul role (I would vendor it since that script has been the source of an exploit before...) and have the job require secrets? | 17:18 |
| Shrews | i figure it would do something like that. parent to tox-cover and a post to run that script. was just curious if it had already been done. asking for someone else. | 17:20 |
| mordred | shrews: for a "friend"? | 17:20 |
| Shrews | silly mordred | 17:21 |
| Shrews | fungi: dmsimard : that autohold change to auto-delete the nodes when deleted is called the "mordred-factor" (at least that's what i named it in my head when i wrote it) | 17:24 |
| fungi | so a subclass of the usual "blame-mordred" rule | 17:25 |
| Shrews | eggzactly | 17:25 |
| mordred | That's always a good rule | 17:51 |
| *** dviroel is now known as dviroel|brb | 18:41 | |
| *** dviroel|brb is now known as dviroel | 19:00 | |
| *** dviroel is now known as dviroel|out | 20:52 | |
| opendevreview | James E. Blair proposed zuul/zuul master: Don't load project keys at startup https://review.opendev.org/c/zuul/zuul/+/804171 | 22:29 |
| corvus | clarkb, tobiash, avass, tristanC: if you can think of any operational reasons not to merge a change like that ^ please let me know. | 22:30 |
| corvus | i think that might cut startup times by another 50%; so opendev would be looking at ~4m (down from 12m originally and then 8m after the recent cache change). | 22:32 |
| avass[m] | corvus: i can't think of a reason. But it's 00:32 here so I'll try again tomorrow when my brain is awake :) | 22:33 |
| clarkb | ya that seems safe. I'll give it a proper review in a bit | 22:33 |
| opendevreview | Ronelle Landy proposed zuul/zuul-jobs master: DNM - Testing rpm update from compose - iptables https://review.opendev.org/c/zuul/zuul-jobs/+/800629 | 23:56 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!