Thursday, 2019-10-10

*** saneax has quit IRC00:12
*** jamesmcarthur has joined #zuul00:14
*** jamesmcarthur has quit IRC00:21
*** jamesmcarthur has joined #zuul00:26
*** jamesmcarthur has quit IRC00:53
*** jamesmcarthur has joined #zuul00:54
*** jamesmcarthur has quit IRC01:01
*** jamesmcarthur has joined #zuul01:15
*** jamesmcarthur has quit IRC01:28
*** michael-beaver has quit IRC01:46
*** jamesmcarthur has joined #zuul02:01
*** threestrands has joined #zuul02:13
*** bhavikdbavishi has joined #zuul02:24
*** bhavikdbavishi1 has joined #zuul02:27
*** bhavikdbavishi has quit IRC02:29
*** bhavikdbavishi1 is now known as bhavikdbavishi02:29
*** jamesmcarthur has quit IRC02:30
*** jamesmcarthur_ has joined #zuul02:30
*** jamesmcarthur_ has quit IRC02:45
*** jamesmcarthur has joined #zuul02:53
*** jamesmcarthur has quit IRC04:04
*** bolg has joined #zuul04:18
*** swest has joined #zuul04:57
*** bolg has quit IRC04:58
*** bolg has joined #zuul05:03
*** swest has quit IRC05:03
*** swest has joined #zuul05:04
*** bolg has quit IRC05:06
*** pcaruana has joined #zuul05:07
openstackgerritSimon Westphahl proposed zuul/nodepool master: Sort waiting static nodes by creation time  https://review.opendev.org/68727105:19
openstackgerritMerged zuul/zuul master: Include session expired reason in API fetch error message.  https://review.opendev.org/68697605:24
openstackgerritMerged zuul/zuul master: Ensure tenant web_root url has a trailing slash  https://review.opendev.org/67682605:28
*** bolg has joined #zuul05:28
*** pcaruana has quit IRC05:35
*** mhu has quit IRC05:42
*** saneax has joined #zuul05:50
*** bolg has quit IRC05:54
*** threestrands has quit IRC06:36
*** threestrands has joined #zuul06:36
*** threestrands has quit IRC06:41
*** pcaruana has joined #zuul06:51
openstackgerritSimon Westphahl proposed zuul/zuul master: Spec for allowing circular dependencies  https://review.opendev.org/64330906:54
openstackgerritSimon Westphahl proposed zuul/zuul master: Add optional support for circular dependencies  https://review.opendev.org/68535406:55
*** tosky has joined #zuul07:12
*** jpena|off is now known as jpena07:13
openstackgerritSimon Westphahl proposed zuul/nodepool master: Sort waiting static nodes by creation time  https://review.opendev.org/68727107:19
openstackgerritSimon Westphahl proposed zuul/nodepool master: Don't touch static nodes that are allocated  https://review.opendev.org/68780607:19
*** zbr has joined #zuul07:33
*** zbr has quit IRC07:44
*** zbr has joined #zuul08:02
openstackgerritSimon Westphahl proposed zuul/nodepool master: Don't touch static nodes that are allocated  https://review.opendev.org/68780608:03
openstackgerritSimon Westphahl proposed zuul/nodepool master: Sort waiting static nodes by creation time  https://review.opendev.org/68727108:03
*** jangutter has joined #zuul08:03
*** recheck has joined #zuul08:04
*** themroc has joined #zuul08:06
*** hashar has joined #zuul08:08
*** arxcruz|rover is now known as arxcruz08:11
*** bolg has joined #zuul08:33
*** Miouge has quit IRC08:36
*** Miouge has joined #zuul08:38
*** lennyb has quit IRC08:57
*** lennyb has joined #zuul08:57
*** mhu has joined #zuul09:05
openstackgerritMatthieu Huin proposed zuul/zuul master: Zuul Web: add /api/user/authorizations endpoint  https://review.opendev.org/64109909:23
openstackgerritMatthieu Huin proposed zuul/zuul master: Reduce sleep to avoid race conditions  https://review.opendev.org/68472609:24
*** bolg has quit IRC09:31
*** mhu has quit IRC09:59
*** mhu has joined #zuul10:33
*** bolg has joined #zuul10:41
openstackgerritMerged zuul/nodepool master: Add port-cleanup-interval config option  https://review.opendev.org/68702411:00
*** jpena is now known as jpena|lunch11:04
*** avass has joined #zuul11:20
*** jhesketh has quit IRC11:29
*** sanjayu_ has joined #zuul11:31
*** sanjayu__ has joined #zuul11:33
*** saneax has quit IRC11:34
*** sanjayu_ has quit IRC11:36
*** jhesketh has joined #zuul11:43
*** jpena|lunch is now known as jpena11:58
*** rfolco has joined #zuul12:05
*** rfolco is now known as rfolco|ruck12:07
openstackgerritSimon Westphahl proposed zuul/nodepool master: Don't touch static nodes that are allocated  https://review.opendev.org/68780612:10
openstackgerritSimon Westphahl proposed zuul/nodepool master: Sort waiting static nodes by creation time  https://review.opendev.org/68727112:10
*** AJaeger has quit IRC12:12
*** rlandy has joined #zuul12:15
*** AJaeger has joined #zuul12:17
*** sanjayu_ has joined #zuul12:32
*** sanjayu__ has quit IRC12:35
pabelangerHmm, TIL (I think) that for tox siblings to work, said project needs to be added as a required-project12:41
pabelangerhttps://opendev.org/zuul/zuul-jobs/src/branch/master/roles/tox/tasks/siblings.yaml#L1312:41
pabelangermordred: do you know why that is?^12:41
pabelangerfor example, we'd like to test a depends-on to molecule, but it isn't a direct requirement to our repo12:42
pabelanger99.9% of the time, we'll install via pypi12:42
pabelangerhowever, in some cases, we do want to test molecule PRs12:42
pabelangereg: https://github.com/ansible-security/ids_config/pull/812:43
*** bolg has quit IRC12:45
pabelangerokay, that actually makes sense. Because even if we confirmed the molecule PR is good, we'd still want it to be a required-project, because once merged on their side, it still wouldn't mean a release to pypi is happening12:52
fungiright12:55
fungiyou's merge that change and then your jobs would no longer work12:55
pabelanger++ thanks12:56
fungiwe get to explain this nuance to opendev users almost weekly12:56
pabelangeryah, now that I think though it, it makes total sense12:56
pabelangerbut, I didn't understand at first12:56
fungi"but i just want the job to install a dependency from source if my change depends on a change/pr from it" ... "no, you actually *don't* want that, and here's why"12:57
pabelangeryah, I think in our case, we likely want to add molecule as required-project now, to aid in their development. But also means, we start using their master branch for our test runner12:58
fungithere are some partial alternatives12:58
fungilike run two jobs, one with the dep from source and one from releases12:59
pabelangeryah, have done that before for some other roles12:59
fungithen you can see that your new change is working on the from-source job, but won't be able to merge until the dependent behavior shows up in a release and the from-releases job starts passing12:59
fungior you can edit the job temporarily in your change or in another do-not-merge change with a dependency relationship to use the source version so you can prove it works13:01
mordredpabelanger: yes - all of the words you said above are right! :)13:02
mordredglad I could help13:02
pabelangermordred; solver of all things!13:04
*** bolg has joined #zuul13:06
fungiin this case, he's a dependency solver13:07
* fungi couldn't resist13:07
mordredoh my13:08
*** avass has quit IRC13:16
*** michael-beaver has joined #zuul13:18
openstackgerritMonty Taylor proposed zuul/zuul-registry master: HEAD object after PUT  https://review.opendev.org/68768113:30
*** jangutter_ has joined #zuul13:37
openstackgerritMonty Taylor proposed zuul/zuul-registry master: HEAD object after PUT  https://review.opendev.org/68768113:40
*** jangutter has quit IRC13:40
*** jangutter has joined #zuul13:53
*** tosky has quit IRC13:55
*** jangutter_ has quit IRC13:57
*** spsurya has joined #zuul13:57
*** fdegir has quit IRC14:06
*** fdegir has joined #zuul14:07
*** hashar has quit IRC14:25
*** bolg has quit IRC14:37
corvuswe should add that to the siblings role documentation14:38
*** jpena is now known as jpena|off14:39
*** pcaruana has quit IRC14:39
*** bolg has joined #zuul14:40
*** chandankumar is now known as raukadah14:43
*** jamesmcarthur has joined #zuul14:52
*** pcaruana has joined #zuul15:01
*** bhavikdbavishi has quit IRC15:12
*** bhavikdbavishi has joined #zuul15:13
*** pcaruana has quit IRC15:26
*** zbr has quit IRC15:29
*** zbr has joined #zuul15:31
*** hashar has joined #zuul15:34
*** sanjayu_ has quit IRC15:37
*** bhavikdbavishi has quit IRC15:45
*** bhavikdbavishi1 has joined #zuul15:45
*** bhavikdbavishi1 is now known as bhavikdbavishi15:48
*** themroc has quit IRC15:54
*** hashar is now known as hasharAway16:10
*** mhu has quit IRC16:19
*** jpena|off is now known as jpena16:19
*** igordc has joined #zuul16:20
pabelangerdo we think zuul-registry is usable outside of opendev today? Or should people hold off until 0.0.1 release?16:24
fungii don't think we're sure it's entirely usable within opendev yet. there were still significant bugs being ironed out of it as of yesterday, as well as additional debugging added to investigate some unexpected behavior16:26
openstackgerritJames E. Blair proposed zuul/zuul-registry master: Add docker buildset test  https://review.opendev.org/68795316:34
pabelangerfungi: ack, thanks16:34
corvusi agree with that assesment :)16:34
fungipabelanger: that said, if you want to try it (without expecting it to work) and help us fix more bugs we don't know about, i doubt anyone will mind ;)16:39
*** pcaruana has joined #zuul16:40
corvusspeaking of which ... i registered zuul-registry on pypi16:42
corvusso we now have a 0.0.0 release16:43
*** phildawson has quit IRC16:43
corvushttps://github.com/andrewp-as-is/pypi-register/blob/master/scripts/pypi-register is handy16:43
fungineat16:45
fungitheir readme is missing a disclaimer that the tool is basically a workaround for the pypi maintainers' effort to prevent folks from registering empty projects before they have a package to upload ;)16:47
openstackgerritJames E. Blair proposed zuul/zuul-registry master: Run docker and podman push/pull tests  https://review.opendev.org/68769216:54
openstackgerritJames E. Blair proposed zuul/zuul-registry master: Add docker buildset test  https://review.opendev.org/68795316:55
*** jamesmcarthur has quit IRC17:03
*** jamesmcarthur_ has joined #zuul17:04
*** rlandy is now known as rlandy|brb17:08
*** hasharAway has quit IRC17:13
openstackgerritJames E. Blair proposed zuul/zuul-registry master: Run docker and podman push/pull tests  https://review.opendev.org/68769217:21
openstackgerritJames E. Blair proposed zuul/zuul-registry master: Add docker buildset test  https://review.opendev.org/68795317:21
*** jamesmcarthur_ has quit IRC17:44
*** rlandy|brb is now known as rlandy17:45
*** hashar has joined #zuul17:50
*** jpena is now known as jpena|off17:57
openstackgerritDavid Shrewsbury proposed zuul/nodepool master: WIP: experimenting with using ZK for fake driver  https://review.opendev.org/68715017:59
*** jamesmcarthur has joined #zuul18:00
*** bhavikdbavishi has quit IRC18:08
*** noorul has joined #zuul18:13
*** igordc has quit IRC18:19
noorulofosos: hi18:29
openstackgerritDavid Shrewsbury proposed zuul/nodepool master: Fix builder shutdown race in tests  https://review.opendev.org/68796518:49
Shrewszuul-maint: just discovered ^^^ in a random test failure on another change18:51
*** noorul has quit IRC18:51
Shrewsthe example failure is https://zuul.opendev.org/t/zuul/build/23db1fd7ea0746278733738e536fcf1e/log/job-output.txt#59418:53
Shrewsoops, i need a join() call in that19:07
openstackgerritMerged zuul/nodepool master: Don't touch static nodes that are allocated  https://review.opendev.org/68780619:26
openstackgerritDavid Shrewsbury proposed zuul/nodepool master: Fix builder shutdown race in tests  https://review.opendev.org/68796519:32
openstackgerritMerged zuul/nodepool master: Sort waiting static nodes by creation time  https://review.opendev.org/68727119:38
Shrewscorvus: just happened to notice that https://review.opendev.org/685116 seems to be "stuck" in zuul check queue for over 32 hours19:49
*** igordc has joined #zuul20:03
*** michael-beaver has quit IRC20:08
*** jamesmcarthur has quit IRC20:08
*** jamesmcarthur has joined #zuul20:09
*** jamesmcarthur has quit IRC20:14
*** pcaruana has quit IRC20:15
corvusShrews: i guess that's better than not being in the queue at all?20:23
corvusShrews: there's a paused job, but its dependency has finished (and failed).  i'm guessing something went wrong there.20:24
*** jamesmcarthur has joined #zuul20:25
*** jamesmcarthur has quit IRC20:27
*** spsurya has quit IRC20:28
*** jamesmcarthur has joined #zuul20:30
openstackgerritJames E. Blair proposed zuul/zuul-registry master: Add docker buildset test  https://review.opendev.org/68795320:39
*** jamesmcarthur has quit IRC20:43
*** jamesmcarthur has joined #zuul20:44
*** jamesmcarthur has quit IRC20:50
*** jamesmcarthur has joined #zuul20:50
*** jamesmcarthur has quit IRC20:52
openstackgerritJames E. Blair proposed zuul/zuul-registry master: Add podman buildset test  https://review.opendev.org/68798620:59
corvustristanC: ^ that shows the namespaced registry in action.  do you have a a suggestion for a small test image on quay.io that we can use for validation (ideally something like alpine -- ~5mb)21:00
corvushttps://quay.io/repository/0xff/alpine-sshd?tab=tags looks promising?21:06
*** jamesmcarthur has joined #zuul21:07
openstackgerritJames E. Blair proposed zuul/zuul-registry master: Add podman buildset test  https://review.opendev.org/68798621:08
openstackgerritJames E. Blair proposed zuul/zuul-registry master: Add podman buildset test  https://review.opendev.org/68798621:11
*** rfolco|ruck has quit IRC21:13
openstackgerritJames E. Blair proposed zuul/zuul-registry master: Add podman buildset test  https://review.opendev.org/68798621:14
corvustristanC: ^ there we go!  a buildset registry shadowing both docker.io and quay.io21:14
clarkbcorvus: do you need to ensure your test images have different hashes/blobs/layers?21:14
clarkb(if you pull alpine in both cases it is possible docker client might noop and not cover as much code as we want when pulling from the secnd source?)21:15
corvusclarkb: by the end of the stack at least, i tried to make sure each test segment deletes each image21:16
clarkbah cool21:16
corvusbut do keep an eye out for if i didn't do that right :)21:16
clarkbok I've got to pop out for a short bit but intend to do some reviews when back21:17
corvuscool, i think that stack is ready21:17
fungii'm headed out to dinner but should be back later21:17
*** jamesmcarthur has quit IRC21:46
*** hashar has quit IRC21:57
clarkbcan you check my comment on https://review.opendev.org/#/c/687613/4 I'm not sure I understand how the shadowing is meant to work if we have to explicitly copy content into the registry beforehand22:09
*** rlandy is now known as rlandy|bbl22:23
corvusclarkb: let me know if that reply makes sense22:27
clarkbyup I was missing the bit for falling back on other mirrors22:28
openstackgerritJames E. Blair proposed zuul/zuul-registry master: Remove unused file from functional test  https://review.opendev.org/68799822:28
clarkbcorvus: item in https://review.opendev.org/#/c/687953/422:33
corvusanswered!22:35
*** igordc has quit IRC22:36
clarkbah ya I see it in a later change too. The when conditional is the inverse22:38
clarkbcorvus: comment on https://review.opendev.org/#/c/687986/4 related to the question I had earlier today22:38
clarkband I suggsted a minor code simplification in https://review.opendev.org/#/c/687613/4 but I don't think that needs a new patchset22:39
corvuscool, i should change my habits to match that :)22:40
corvusclarkb: i'll rejigger the deletes when i figure out why there's no /run/user/1000/containers/auth.json (even after having performed a 'podman login' earlier)22:40
corvusi just set an autohold to try to figure that out22:41
clarkbok22:41
corvus(i've run the playbook many times locally with success; this is an unexpected difference)22:41
corvusoh, apparently it's just /run/user/1000/auth.json22:54
corvusthat seems... weird.22:54
corvusit also contradicts the docs22:55
clarkbcould potentially collide with other services adding a auth.json to the systemd user runtime dir22:57
corvusyeah, that's why it seems like such a weird/wrong choice23:08
corvuslooking at the code i can not figure out how it ended up there23:21
openstackgerritJames E. Blair proposed zuul/zuul-registry master: Add podman buildset test  https://review.opendev.org/68798623:25
corvus<shrug>23:25
clarkbcorvus: one small thing in that latest patchset23:41
clarkbotherwise lgtm23:41
openstackgerritJames E. Blair proposed zuul/zuul-registry master: Add podman buildset test  https://review.opendev.org/68798623:49
corvusclarkb: oops thx23:49

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