| openstackgerrit | Andrii Ostapenko proposed zuul/zuul-jobs master: Add ability to use (upload|promote)-docker-image roles in periodic jobs https://review.opendev.org/740560 | 00:13 |
|---|---|---|
| openstackgerrit | Andrii Ostapenko proposed zuul/zuul-jobs master: Add ability to use (upload|promote)-docker-image roles in periodic jobs https://review.opendev.org/740560 | 00:14 |
| *** rfolco has quit IRC | 00:32 | |
| *** jamesmcarthur has joined #zuul | 00:50 | |
| *** Goneri has quit IRC | 01:49 | |
| *** Goneri has joined #zuul | 01:50 | |
| *** Goneri has quit IRC | 01:50 | |
| *** jamesmcarthur has quit IRC | 01:51 | |
| *** jamesmcarthur has joined #zuul | 01:55 | |
| *** Goneri has joined #zuul | 01:59 | |
| *** Goneri has quit IRC | 01:59 | |
| *** Goneri has joined #zuul | 02:02 | |
| *** Goneri has joined #zuul | 02:04 | |
| *** jamesmcarthur has quit IRC | 02:15 | |
| *** Goneri has quit IRC | 02:20 | |
| *** Goneri has joined #zuul | 02:23 | |
| *** jamesmcarthur has joined #zuul | 02:27 | |
| *** saneax has joined #zuul | 02:27 | |
| *** Goneri has quit IRC | 02:31 | |
| *** jamesmcarthur has quit IRC | 02:33 | |
| *** saneax has quit IRC | 02:33 | |
| *** jamesmcarthur has joined #zuul | 02:36 | |
| *** jamesmcarthur has quit IRC | 02:37 | |
| *** saneax has joined #zuul | 02:38 | |
| *** jamesmcarthur has joined #zuul | 02:39 | |
| *** sgw1 has quit IRC | 02:40 | |
| *** jamesmcarthur has quit IRC | 02:53 | |
| *** sgw1 has joined #zuul | 02:53 | |
| *** sgw1 has quit IRC | 02:55 | |
| *** saneax has quit IRC | 02:57 | |
| *** saneax has joined #zuul | 03:10 | |
| *** sgw1 has joined #zuul | 03:11 | |
| *** jamesmcarthur has joined #zuul | 03:35 | |
| *** bhavikdbavishi has joined #zuul | 03:39 | |
| *** chandankumar has joined #zuul | 03:44 | |
| *** jamesmcarthur has quit IRC | 03:45 | |
| *** jamesmcarthur has joined #zuul | 03:45 | |
| *** fdegir2 has joined #zuul | 03:49 | |
| *** fdegir has quit IRC | 03:49 | |
| *** jamesmcarthur has quit IRC | 03:52 | |
| *** sgw1 has quit IRC | 04:24 | |
| *** Goneri has joined #zuul | 04:52 | |
| *** Goneri has quit IRC | 05:03 | |
| *** Goneri has joined #zuul | 05:16 | |
| *** Goneri has quit IRC | 05:22 | |
| *** Goneri has joined #zuul | 05:23 | |
| *** Goneri has quit IRC | 05:30 | |
| *** Goneri has joined #zuul | 05:32 | |
| *** Goneri has quit IRC | 05:34 | |
| *** Goneri has joined #zuul | 05:37 | |
| *** Goneri has quit IRC | 05:43 | |
| *** Goneri has joined #zuul | 05:46 | |
| *** bhavikdbavishi has quit IRC | 06:01 | |
| *** bhavikdbavishi has joined #zuul | 06:01 | |
| *** marios has joined #zuul | 06:03 | |
| *** bhavikdbavishi1 has joined #zuul | 06:17 | |
| *** bhavikdbavishi has quit IRC | 06:18 | |
| *** bhavikdbavishi1 is now known as bhavikdbavishi | 06:18 | |
| *** bhavikdbavishi has quit IRC | 06:24 | |
| *** Goneri has quit IRC | 06:26 | |
| *** tosky has joined #zuul | 06:42 | |
| *** Goneri has joined #zuul | 06:59 | |
| *** jcapitao has joined #zuul | 07:10 | |
| *** bhavikdbavishi has joined #zuul | 07:10 | |
| *** jpena|off is now known as jpena | 07:21 | |
| *** dpawlik2 has joined #zuul | 07:23 | |
| *** holser has joined #zuul | 07:23 | |
| *** hashar has joined #zuul | 07:24 | |
| openstackgerrit | Felix Edel proposed zuul/zuul master: PF4: Update to newest patternfly 4 release (2020.09) https://review.opendev.org/743157 | 07:33 |
| *** Goneri has quit IRC | 07:35 | |
| *** holser_ has joined #zuul | 07:39 | |
| *** holser__ has joined #zuul | 07:40 | |
| *** holser has quit IRC | 07:41 | |
| *** holser has joined #zuul | 07:44 | |
| *** holser_ has quit IRC | 07:44 | |
| *** holser__ has quit IRC | 07:45 | |
| openstackgerrit | Guillaume Chauvel proposed zuul/zuul master: Tenant reconfiguration: Add tests https://review.opendev.org/742747 | 07:47 |
| openstackgerrit | Guillaume Chauvel proposed zuul/zuul master: Tenant reconfiguration: Reorg scheduler event process loop https://review.opendev.org/742748 | 07:47 |
| openstackgerrit | Guillaume Chauvel proposed zuul/zuul master: Tenant reconfiguration: Scheduler ref-updated create/delete https://review.opendev.org/739198 | 07:47 |
| openstackgerrit | Guillaume Chauvel proposed zuul/zuul master: Tenant reconfiguration: Allow ref-updated newrev+oldrev reconfiguration https://review.opendev.org/742749 | 07:47 |
| openstackgerrit | Guillaume Chauvel proposed zuul/zuul master: Tenant reconfiguration: Get ref-updated modified files https://review.opendev.org/739078 | 07:47 |
| *** harrymichal has joined #zuul | 07:59 | |
| *** tosky has quit IRC | 08:03 | |
| *** hrw has joined #zuul | 08:16 | |
| hrw | morning | 08:17 |
| *** bhavikdbavishi has quit IRC | 08:19 | |
| *** nils has joined #zuul | 08:23 | |
| *** harrymichal has quit IRC | 08:28 | |
| *** bhavikdbavishi has joined #zuul | 08:28 | |
| *** vishalmanchanda has joined #zuul | 08:29 | |
| *** holser_ has joined #zuul | 08:39 | |
| *** rpittau has joined #zuul | 08:39 | |
| *** holser has quit IRC | 08:40 | |
| *** holser_ has quit IRC | 08:40 | |
| *** fdegir2 is now known as fdegir | 08:40 | |
| *** holser has joined #zuul | 08:42 | |
| *** holser_ has joined #zuul | 08:48 | |
| *** holser has quit IRC | 08:50 | |
| *** holser_ has quit IRC | 08:52 | |
| *** holser has joined #zuul | 08:57 | |
| *** holser has quit IRC | 09:01 | |
| *** holser has joined #zuul | 09:02 | |
| *** holser has quit IRC | 09:09 | |
| *** holser has joined #zuul | 09:09 | |
| openstackgerrit | Guillaume Chauvel proposed zuul/zuul master: Tenant reconfiguration: Add tests https://review.opendev.org/742747 | 09:23 |
| openstackgerrit | Guillaume Chauvel proposed zuul/zuul master: Tenant reconfiguration: Reorg scheduler event process loop https://review.opendev.org/742748 | 09:23 |
| openstackgerrit | Guillaume Chauvel proposed zuul/zuul master: Tenant reconfiguration: Scheduler ref-updated create/delete https://review.opendev.org/739198 | 09:23 |
| openstackgerrit | Guillaume Chauvel proposed zuul/zuul master: Tenant reconfiguration: Allow ref-updated newrev+oldrev reconfiguration https://review.opendev.org/742749 | 09:23 |
| openstackgerrit | Guillaume Chauvel proposed zuul/zuul master: Tenant reconfiguration: Get ref-updated modified files https://review.opendev.org/739078 | 09:23 |
| *** hrw has left #zuul | 09:27 | |
| *** tosky has joined #zuul | 09:33 | |
| openstackgerrit | Benjamin Schanzel proposed zuul/zuul master: Disable Log Streaming from k8s Pod Nodes https://review.opendev.org/743182 | 09:53 |
| *** bhavikdbavishi has quit IRC | 10:03 | |
| *** bhavikdbavishi has joined #zuul | 10:03 | |
| *** harrymichal has joined #zuul | 10:48 | |
| *** bhavikdbavishi has quit IRC | 10:53 | |
| tobiash | zuul-maint: fyi, we discovered that kubectl port-forward doesn't work reliably for log streaming ^ | 10:55 |
| *** avass has joined #zuul | 10:58 | |
| *** harrymichal has quit IRC | 10:59 | |
| *** harrymichal has joined #zuul | 10:59 | |
| *** iurygregory has quit IRC | 10:59 | |
| *** bhavikdbavishi has joined #zuul | 11:00 | |
| *** iurygregory has joined #zuul | 11:01 | |
| *** jcapitao is now known as jcapitao_lunch | 11:02 | |
| openstackgerrit | Tobias Henkel proposed zuul/zuul master: Evaluate CODEOWNERS settings during canMerge check https://review.opendev.org/644557 | 11:15 |
| openstackgerrit | Tobias Henkel proposed zuul/zuul master: GitHub Reporter: Fix User Email in Merge Commit Message https://review.opendev.org/741157 | 11:25 |
| *** jpena is now known as jpena|lunch | 11:38 | |
| *** sanjayu_ has joined #zuul | 11:53 | |
| *** saneax has quit IRC | 11:56 | |
| *** rfolco has joined #zuul | 12:00 | |
| *** bhavikdbavishi has quit IRC | 12:04 | |
| *** rlandy has joined #zuul | 12:04 | |
| openstackgerrit | Benjamin Schanzel proposed zuul/zuul master: GitHub Reporter: Fix User Email in Merge Commit Message https://review.opendev.org/741157 | 12:07 |
| *** bhavikdbavishi has joined #zuul | 12:17 | |
| *** mordred has joined #zuul | 12:20 | |
| openstackgerrit | Merged zuul/zuul master: Revert "Revert "Tune automatic garbage collection of git repos"" https://review.opendev.org/723800 | 12:28 |
| openstackgerrit | Merged zuul/zuul master: Ensure refs for recent branches are not GCed https://review.opendev.org/738454 | 12:30 |
| *** evrardjp has quit IRC | 12:32 | |
| *** evrardjp has joined #zuul | 12:32 | |
| *** evrardjp has quit IRC | 12:32 | |
| *** evrardjp has joined #zuul | 12:33 | |
| *** jcapitao_lunch is now known as jcapitao | 12:33 | |
| *** jpena|lunch is now known as jpena | 12:34 | |
| openstackgerrit | Tobias Henkel proposed zuul/zuul master: GitHub Reporter: Fix User Email in Merge Commit Message https://review.opendev.org/741157 | 12:57 |
| openstackgerrit | Tobias Henkel proposed zuul/zuul master: Evaluate CODEOWNERS settings during canMerge check https://review.opendev.org/644557 | 13:25 |
| *** bhavikdbavishi has quit IRC | 13:47 | |
| *** sgw1 has joined #zuul | 13:50 | |
| openstackgerrit | Felix Edel proposed zuul/zuul master: PF4: Fix scrolling behaviour when using keyboard https://review.opendev.org/743239 | 14:11 |
| felixedel | zuul-maint: This https://review.opendev.org/#/c/743239/ should fix the scrolling behaviour when using space, pageup/down and so on. I'm still not entirely sure what's exactly the root cause for this, but this seems to do the trick. Took me quite some time :D | 14:13 |
| *** harrymichal has quit IRC | 14:16 | |
| *** harrymichal has joined #zuul | 14:16 | |
| *** hashar has quit IRC | 14:16 | |
| *** harrymichal has quit IRC | 14:22 | |
| openstackgerrit | Benjamin Schanzel proposed zuul/zuul master: GitHub Reporter: Fix User Email in Merge Commit Message https://review.opendev.org/741157 | 14:23 |
| *** bhagyashris is now known as bhagyashris|away | 14:24 | |
| *** harrymichal has joined #zuul | 14:41 | |
| *** harrymichal has quit IRC | 14:59 | |
| *** harrymichal has joined #zuul | 15:02 | |
| openstackgerrit | Sorin Sbarnea (zbr) proposed zuul/zuul master: WIP: Generalize localStorage state persistency https://review.opendev.org/743252 | 15:08 |
| zbr | felixedel: how does your change relate to https://review.opendev.org/#/c/742759/ | 15:13 |
| zbr | apparently very different approach to address same issue | 15:13 |
| zbr | only thing in common is that nobody knows the root cause | 15:13 |
| *** bhavikdbavishi has joined #zuul | 15:15 | |
| *** jamesmcarthur has joined #zuul | 15:42 | |
| tobiash | zbr: at least what I noticed is that 742759 changes the page to scroll completely compared to the current implementation that doesn't scroll the header | 15:45 |
| zbr | not scrolling the header is a big improvement IMHO, ideally change number and title should also be part of the header. | 15:48 |
| zbr | so you do not loose context | 15:48 |
| zbr | when i seen the css change, i was almost sure something was fishy about it | 15:49 |
| *** sanjayu__ has joined #zuul | 15:49 | |
| *** sanjayu_ has quit IRC | 15:52 | |
| openstackgerrit | Ghanshyam Mann proposed zuul/zuul master: Add ubuntu-focal for python3.7 platforms https://review.opendev.org/743267 | 15:53 |
| openstackgerrit | Tobias Henkel proposed zuul/zuul master: Evaluate CODEOWNERS settings during canMerge check https://review.opendev.org/644557 | 15:54 |
| *** marios has quit IRC | 16:02 | |
| *** jcapitao has quit IRC | 16:15 | |
| *** hashar has joined #zuul | 16:29 | |
| *** Goneri has joined #zuul | 16:38 | |
| *** jpena is now known as jpena|off | 16:41 | |
| *** nils has quit IRC | 16:43 | |
| *** sanjayu__ has quit IRC | 16:48 | |
| *** hamalq has joined #zuul | 16:55 | |
| *** nils has joined #zuul | 17:04 | |
| *** bhavikdbavishi has quit IRC | 17:21 | |
| *** bhavikdbavishi has joined #zuul | 17:24 | |
| *** jamesmcarthur has quit IRC | 17:26 | |
| *** jamesmcarthur has joined #zuul | 17:27 | |
| *** vishalmanchanda has quit IRC | 18:18 | |
| *** evrardjp has quit IRC | 18:21 | |
| *** bhavikdbavishi has quit IRC | 18:23 | |
| *** evrardjp has joined #zuul | 18:24 | |
| gmann | clarkb: fungi corvus can you please check this - https://review.opendev.org/#/c/743267/1 | 18:30 |
| clarkb | gmann: why is openstack installing zuul? | 18:31 |
| clarkb | gmann: also https://packages.ubuntu.com/search?suite=focal&arch=any&searchon=names&keywords=python3.7 that package doesn't exist on focal | 18:32 |
| *** tosky has quit IRC | 18:33 | |
| fungi | yeah, focal has 3.8, you need bionic for older pythons | 18:35 |
| fungi | or you need to custom hack in your own python builds | 18:35 |
| gmann | clarkb: humm, i think I saw it from here with python3.7-dev - https://launchpad.net/ubuntu/focal/+source/python3.7 | 18:36 |
| gmann | yeah, i think running 3.7 on bionic node. i initially thought focal has. | 18:36 |
| clarkb | gmann: but also its not clear to me why zuul's bindep file matters, that is for zuul th service and has nothing to do with zuul jobs | 18:36 |
| gmann | that's mistake. i thought i did in zuul jobs. sorry for that. | 18:37 |
| clarkb | mordred: btw https://review.opendev.org/#/c/741973/ is where we ended up with making nodepool arm64 builds work | 19:09 |
| clarkb | mordred: that got it working but it still takes close to the hour timeout and requires us to work in lockstep with openstack. Another idea thrown out there was to create an intermediate layer for nodepool that has its deps in it, then we can build that periodically and the on demand builds for nodepool changes happen much more quickly (in theory) | 19:09 |
| clarkb | doing that what we don't know is how long it will take to build wheels for cryptography and bcrypt and pynacl on that intermediate layer | 19:10 |
| *** jamesmcarthur has quit IRC | 19:43 | |
| *** SpamapS has quit IRC | 19:46 | |
| *** SpamapS has joined #zuul | 19:50 | |
| *** jamesmcarthur has joined #zuul | 19:51 | |
| *** jamesmcarthur has quit IRC | 19:55 | |
| *** jamesmcarthur has joined #zuul | 19:56 | |
| *** hashar has quit IRC | 20:02 | |
| *** jamesmcarthur has quit IRC | 20:14 | |
| *** jamesmcarthur has joined #zuul | 20:17 | |
| *** y2kenny has joined #zuul | 20:26 | |
| y2kenny | is max-servers driver specific/openstack only? | 20:27 |
| clarkb | y2kenny: it looks like aws and azure support it too? | 20:28 |
| clarkb | its possible that the k8s driver doesn't though | 20:28 |
| y2kenny | ok... thanks for confirming...(I just found out... :)) | 20:29 |
| y2kenny | probably launched 100 jobs together on 10 machines... | 20:29 |
| y2kenny | clarkb: are there other ways to limit launches? I looked at label but looks like it only has min | 20:31 |
| clarkb | y2kenny: the only other thing I know about is the quota based management which I think only openstack has added. Its probably not too difficult to add max-servers support to k8s if that is whta you need | 20:32 |
| clarkb | y2kenny: the config loading in the base class for drivers laready handles that aspect of it, it should just be a matter of pausing and waiting for resources to be freed before continuning within the k8s driver | 20:33 |
| clarkb | tristanC: ^ do you know why this wasn't added initially? | 20:33 |
| tristanC | clarkb: i don't remember, oODpenshift does have a `max-projects` quota system | 20:59 |
| clarkb | well we don't need to rely on the remote to tell us, we can track it locally | 20:59 |
| clarkb | we deal with openstack quotas because they may not be aligned with the nodepool set limits and we want to avoid errors by respecting them, but that is another level of limit handling | 21:00 |
| tristanC | i'm not sure if and how vanilla kubernetes handle quota, but for openshit the handler code check for `quota exceeded` exception message and raise a QuotaException, which should delay and auto retry creation attempt | 21:04 |
| *** jamesmcarthur has quit IRC | 21:07 | |
| *** jamesmcarthur has joined #zuul | 21:08 | |
| clarkb | Looking at the k8s driver we listNodes() and provide the total number of namespaces | 21:09 |
| clarkb | I think where this gets weird is that a namespace does't have a set size | 21:10 |
| clarkb | what we could do fairly easily is map max-servers to a max number of namespaces | 21:10 |
| y2kenny | I don't think k8s namespace is some kind of countable resource. But with the way nodepool uses it's kind of an implied resource. | 21:11 |
| clarkb | y2kenny: correct | 21:11 |
| y2kenny | I think max-servers mapping to max number of nameapces is a good limit | 21:11 |
| y2kenny | at least there's a limit :) | 21:11 |
| clarkb | we could have createNamespace block if it is already at the limit and wait for namespace count to drop | 21:12 |
| clarkb | which is basically what the openstack driver does (and I assume gce, aws, etc) | 21:12 |
| y2kenny | that make sense | 21:12 |
| *** jamesmcarthur has quit IRC | 21:22 | |
| *** tosky has joined #zuul | 21:25 | |
| *** jamesmcarthur has joined #zuul | 21:27 | |
| *** jamesmcarthur has quit IRC | 21:32 | |
| *** jamesmcarthur has joined #zuul | 21:34 | |
| openstackgerrit | Clark Boylan proposed zuul/nodepool master: Add basic max-servers handling to the k8s driver https://review.opendev.org/743424 | 21:43 |
| clarkb | y2kenny: tristanC ^ I don't have a quick and easy way to test that locally but I believe that the k8s jobs that will run for the chagne should give us enough of a "does this regress" check | 21:44 |
| clarkb | if that doesn't completely explode we can try and set max-servers in those jobs and actually exercise the limit maybe | 21:45 |
| *** SpamapS has quit IRC | 21:52 | |
| corvus | clarkb: the last time i needed to locally test a nodepool/zuul/k8s change, i used KIND and it worked well | 21:55 |
| *** SpamapS has joined #zuul | 21:57 | |
| avass | yeah KIND has been great so far | 22:00 |
| fungi | that's hard to perform a web search for... https://kind.sigs.k8s.io/ i guess? | 22:01 |
| fungi | ooh, i should have said "kind of hard to..." | 22:02 |
| y2kenny | It should stand for Kubernetes in Docker | 22:02 |
| y2kenny | clarkb: do you want me to try a local nodepool build and deploy it and see how that goes? Or do you mean there will be a nodepool build from opendev that I can try? | 22:02 |
| y2kenny | or "Kubernetes IN Docker" | 22:03 |
| avass | fungi: yeah | 22:03 |
| fungi | cool, thanks | 22:03 |
| clarkb | y2kenny: ya Zuul's CI system (opendev) should do some checks for us. If you're able to test that without impacting your production use then I'd say go for it | 22:04 |
| avass | also: https://review.opendev.org/#/c/740935/ :) | 22:04 |
| clarkb | y2kenny: also feel free to push updates if you find problems | 22:04 |
| clarkb | basically what the change does is assumes an infinite k8s quota and a 1 instance == 1 namespace usage rate. Then applies some existing lib code to those assumptions which should get us working max-servers | 22:05 |
| clarkb | its also stubbed out so that we could more accurately check things if we want but that would be good for followons I think | 22:05 |
| y2kenny | clarkb: ok, I will see what I can do this week. I should be able to run multiple nodepool instances. One more experimental nodepool shouldn't hurt. | 22:06 |
| clarkb | y2kenny: fwiw if the testing shows it doens't work I'll do my best to fix it from that too. And maybe tomorrow I'll get a KIND running | 22:07 |
| openstackgerrit | Clark Boylan proposed zuul/nodepool master: Add basic max-servers handling to the k8s driver https://review.opendev.org/743424 | 22:14 |
| *** jamesmcarthur has quit IRC | 22:31 | |
| *** jamesmcarthur has joined #zuul | 22:31 | |
| *** nils has quit IRC | 22:32 | |
| webknjaz | PSA: Cheroot v8.1.0 refactoring introduced a performance drop. Upgrade to v8.4.2 for the hotfix! | 22:53 |
| *** harrymichal has quit IRC | 22:59 | |
| *** rlandy is now known as rlandy|bbl | 23:03 | |
| *** sanjayu__ has joined #zuul | 23:10 | |
| *** sanjayu__ has quit IRC | 23:10 | |
| openstackgerrit | James E. Blair proposed zuul/zuul master: Add user preferences dialog https://review.opendev.org/739482 | 23:11 |
| openstackgerrit | James E. Blair proposed zuul/zuul master: Use redux for handling autoReload preference https://review.opendev.org/743434 | 23:11 |
| clarkb | running nodepool unittests locally haslocked up my laptop | 23:13 |
| openstackgerrit | Clark Boylan proposed zuul/nodepool master: Add basic max-servers handling to the k8s driver https://review.opendev.org/743424 | 23:17 |
| clarkb | apologies for the spam but I've tried to run it locally and my laptop was not having it | 23:19 |
| clarkb | I should probably look at some newer hardware | 23:19 |
| *** tosky has quit IRC | 23:25 | |
| *** jamesmcarthur has quit IRC | 23:28 | |
| *** jamesmcarthur has joined #zuul | 23:29 | |
| corvus | zbr: take a look at https://review.opendev.org/743434 i left a lot of comments about redux there. let me know if you have questions. | 23:29 |
| corvus | felixedel, tristanC, avass: ^ fyi you may be interested | 23:29 |
| *** holser has quit IRC | 23:30 | |
| *** jamesmcarthur has quit IRC | 23:33 | |
| *** holser has joined #zuul | 23:34 | |
| openstackgerrit | Clark Boylan proposed zuul/nodepool master: Add basic max-servers handling to the k8s driver https://review.opendev.org/743424 | 23:34 |
| openstackgerrit | Clark Boylan proposed zuul/nodepool master: Add basic max-servers handling to the k8s driver https://review.opendev.org/743424 | 23:47 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!