Monday, 2020-08-17

openstackgerritIan Wienand proposed zuul/zuul-jobs master: Add ensure-rust role  https://review.opendev.org/74642300:06
*** rfolco has quit IRC00:15
openstackgerritIan Wienand proposed zuul/zuul-jobs master: Add ensure-rust role  https://review.opendev.org/74642300:19
openstackgerritIan Wienand proposed zuul/zuul-jobs master: Add ensure-rust role  https://review.opendev.org/74642300:30
*** yolanda has quit IRC01:26
*** yolanda has joined #zuul01:26
*** yolanda has quit IRC01:27
*** yolanda has joined #zuul01:27
ianwcorvus/clarkb: i'm going to need some help understanding Depends-On: between github and gerrit.  Can github not depends-on a zuul-jobs change?  But gerrit jobs can depends-on github changes?03:15
clarkbI think it should go both directions03:16
ianwhttps://opendev.org/zuul/zuul/src/branch/master/zuul/driver/github/githubconnection.py#L1177 makes me think not03:17
ianwhttps://zuul.opendev.org/t/pyca/build/f9978ceba64a454184f4ca1e9871fdcd is an example of the github change depends-on: zuul-jobs03:18
ianwzuul-jobs isn't listed in the projects03:18
ianwhttps://zuul.opendev.org/t/pyca/build/f9978ceba64a454184f4ca1e9871fdcd/log/zuul-info/inventory.yaml#5303:18
*** bhavikdbavishi has joined #zuul03:36
openstackgerritIan Wienand proposed zuul/zuul master: github connection: remove unused depends_on_re  https://review.opendev.org/74643704:19
*** raukadah is now known as chkumar|rover04:20
ianwcorvus/clarkb: ^ i think that's a red-herring.  i agree it looks like it should probably work, but it doesn't seem to.  will try to sync up on it tomorrow04:20
*** evrardjp has quit IRC04:33
*** evrardjp has joined #zuul04:33
*** sgw1 has joined #zuul04:54
*** saneax has joined #zuul05:04
tobiashianw: in github the depends-on must be in the pr description, not in a commit message05:06
swestclarkb: thanks for 746349. we started seeing exceptions for the build case aswell.05:07
openstackgerritIan Wienand proposed zuul/zuul master: github connection: remove unused depends_on_re  https://review.opendev.org/74643705:44
*** mach1na has joined #zuul06:08
*** mach1na has quit IRC06:08
*** mach1na has joined #zuul06:08
*** mach1na has quit IRC06:36
*** bhavikdbavishi has quit IRC06:39
*** vishalmanchanda has joined #zuul06:49
openstackgerritSimon Westphahl proposed zuul/zuul master: Add optional support for circular dependencies  https://review.opendev.org/68535407:03
openstackgerritSimon Westphahl proposed zuul/zuul master: Check cycle items are mergeable before reporting  https://review.opendev.org/74345007:03
*** bhavikdbavishi has joined #zuul07:08
*** mach1na has joined #zuul07:15
*** mach1na has quit IRC07:19
*** mach1na has joined #zuul07:19
openstackgerritAlbin Vass proposed zuul/zuul master: Add commit id and owner to Change for mqtt reporter  https://review.opendev.org/72247807:37
openstackgerritTobias Henkel proposed zuul/zuul master: WIP: Switch to using zookeeper instead of gearman for jobs (keep gearman for mergers)  https://review.opendev.org/74441607:39
*** hashar has joined #zuul07:49
*** tosky has joined #zuul07:56
openstackgerritSimon Westphahl proposed zuul/zuul master: Add optional support for circular dependencies  https://review.opendev.org/68535407:57
openstackgerritSimon Westphahl proposed zuul/zuul master: Check cycle items are mergeable before reporting  https://review.opendev.org/74345007:57
*** jpena|off is now known as jpena07:58
*** hashar has quit IRC08:10
openstackgerritFelix Edel proposed zuul/zuul master: Return number of total results from builds/ and buildsets/ API endpoint  https://review.opendev.org/74632808:10
*** bhavikdbavishi1 has joined #zuul08:22
*** bhavikdbavishi has quit IRC08:24
*** bhavikdbavishi1 is now known as bhavikdbavishi08:24
*** nils has joined #zuul08:40
*** hashar has joined #zuul08:49
openstackgerritFelix Edel proposed zuul/zuul master: web: Return number of total results from builds/ and buildsets/ API  https://review.opendev.org/74632809:01
openstackgerritFelix Edel proposed zuul/zuul master: WIP PF4: Add pagination to builds and buildsets pages  https://review.opendev.org/74646309:07
*** bhavikdbavishi has quit IRC09:09
*** evgenyl has quit IRC09:10
*** bhavikdbavishi has joined #zuul09:10
*** tdasilva has quit IRC09:11
*** evgenyl has joined #zuul09:18
*** tdasilva has joined #zuul09:19
ianwtobiash: oh, ok.  so what's the pr description?  like the top-level comment in https://github.com/pyca/cryptography/pull/5410#issue-468498719 ?09:42
tobiashianw: yes exactly09:48
tobiashyou can just edit that and then recheck09:49
ianwtobiash: ahh, it's not my change but ok i can point that out.  thanks, this had me in a muddle!09:56
tobiashyeah, zuul uses the change message which fits better to the pr description in the github workflow09:59
mach1naHi Folks. Asked this a couple of time before and have had no reply. I'm wondering why a known non-backwards compatible change was merged? https://opendev.org/zuul/zuul-jobs/commit/c24c09e82c9bc5028b2f0fd371060b7b8d448c39 This broke our reporting in our prod Zuul system over night on the 11th August. Just after guidance :) This is my first request here so if I'm missing providing any details let me know.10:03
tobiashmach1na: looking at the reviews it was supposed to be announced and merged after two weeks: https://review.opendev.org/#/c/742971/10:12
tobiashmach1na: this has been announced on the zuul-announce mailing list: http://lists.zuul-ci.org/pipermail/zuul-announce/2020-July/000080.html10:14
tobiashmach1na: if you don't yet, you might wan't to subscribe to this list10:14
mach1natobiash: I am doing that right now. Thanks greatly for the clarification.10:16
tobiashmach1na: this is the description of the deprecation policy in zuul-jobs: https://zuul-ci.org/docs/zuul-jobs/policy.html#deprecation-policy10:16
tobiashmach1na: could you work around this or is this still a problem for you?10:16
mach1natobiash: We're using a local, reverted version of the role for now, so we're mitigated. It looks like we just need to change our ara_report_path var to '{{ zuul.executor.log_root }}/custom_location' correct?10:21
tobiashmach1na: I was not involved in this change but it looks like that, yes10:23
mach1natobiash: cheers, thank you so much for the help.10:24
tobiashno problem10:24
openstackgerritMatthieu Huin proposed zuul/zuul master: How-to: using the REST API with cURL  https://review.opendev.org/72778510:27
mhuHey there, change ^ is a simple doc change with already one +2 that has been open and stale for a while, can I get the last +3 on it? Thanks!10:29
*** bhavikdbavishi has quit IRC10:41
*** bhavikdbavishi has joined #zuul11:00
*** mach1na has quit IRC11:13
*** jpena is now known as jpena|lunch11:33
*** weshay_ is now known as weshay|ruck11:36
*** bhavikdbavishi has quit IRC11:40
*** rfolco has joined #zuul11:43
*** wuchunyang has joined #zuul11:49
*** wuchunyang has quit IRC11:52
*** mach1na has joined #zuul11:52
*** wuchunyang has joined #zuul11:52
*** wuchunyang has quit IRC11:54
*** wuchunyang has joined #zuul11:54
*** wuchunyang has quit IRC11:56
openstackgerritFelix Edel proposed zuul/zuul master: PF4: Add icons and empty state to artifacts section on build result page  https://review.opendev.org/73997311:56
*** wuchunyang has joined #zuul11:57
*** wuchunyang has quit IRC12:00
*** mach1na has quit IRC12:02
*** mach1na has joined #zuul12:03
*** rlandy has joined #zuul12:05
*** bhavikdbavishi has joined #zuul12:12
*** jpena|lunch is now known as jpena12:38
*** bhavikdbavishi has quit IRC12:44
zbrAny chance to get the +W on the preferences dialog change? https://review.opendev.org/#/c/739482/13:05
*** Goneri has joined #zuul13:13
mhuzbr: it's a nit but how about making it a user prefs page rather than a dialog box? I'm thinking the customizable settings are bound to increase in number (tz, language, other things we haven't thought of yet)13:18
mhuor would it be better to switch later13:18
zbrmhu: we can do this later if needed. likely once the list grows13:19
zbrbut until we have >10 settings there, I do not see a need.13:19
zbri plan to add ~5 so it may not be very far in the future13:19
*** mach1na has quit IRC13:33
*** mach1na has joined #zuul13:33
*** mach1na has quit IRC13:34
*** mach1na has joined #zuul13:35
*** hashar has quit IRC13:51
*** hashar has joined #zuul13:54
*** LLIU82 has joined #zuul13:54
openstackgerritBenjamin Schanzel proposed zuul/zuul-jobs master: Multi Node/Context support for mirroring Git workspaces via kubectl  https://review.opendev.org/74650813:56
LLIU82I met an error here: https://review.opendev.org/#/c/722478/. According to my understanding, the error comes from the python version is too low. How can I solve it? the python version in the requirements.txt is python-dateutil14:04
corvusLLIU82: i read that as saying that stevedore doesn't work with python 3.5, but i don't see a recent release of stevedore, so i'm a little confused (and other recent changes don't have that problem).  i've rechecked the change to see if it persists.14:14
openstackgerritBenjamin Schanzel proposed zuul/zuul master: Fix Multi Node Builds with k8s Build Nodes  https://review.opendev.org/74651514:16
*** sshnaidm is now known as sshnaidm|MYG14:27
*** sshnaidm|MYG is now known as sshnaidm|mtg14:27
avasscorvus, LLIU82: yeah that's strange, all versions after 1.32.0 requires python>=3.6 according to pypi, but that should have been detected earlier.14:44
*** chkumar|rover is now known as raukadah14:45
tobiashcorvus: do you want to have a second look on the preferences dialog? It has +2 from tristanC and me but you had a -1 in the past so it's not approved yet to give you a chance to look again14:49
clarkbLLIU82: corvus avass installing a version that works with python3.5 relies on pypi's python version metadata in the package indexes. We've seen those go away when pypi CDN nodes serve out of date data and we've seem them serviging out of date data recently too14:50
clarkbmy hunch is that the issue is related to that14:51
fungiwe can also adjust the requirements.txt to assert "stevedore<1.32;python_version<'3.6'" if we think it's warranted, to support older python which doesn't use that pypi api feature14:51
openstackgerritJonas Sticha proposed zuul/zuul master: WIP: Make repo state buildset global  https://review.opendev.org/73860314:58
openstackgerritLida Liu proposed zuul/zuul master: Add commit id to Change for mqtt reporter  https://review.opendev.org/72247815:27
*** LLIU82 has quit IRC15:28
fungier, i meant older pip15:28
*** mach1na has quit IRC15:42
*** mach1na has joined #zuul15:43
*** mach1na has quit IRC15:47
*** mach1na has joined #zuul15:47
*** mach1na has quit IRC15:59
*** sshnaidm|mtg is now known as sshnaidm16:08
*** jpena is now known as jpena|off16:14
*** bhavikdbavishi has joined #zuul16:38
*** hashar is now known as hasharAway17:10
*** vishalmanchanda has quit IRC17:37
*** wuchunyang has joined #zuul18:04
*** wuchunyang has quit IRC18:05
*** bhavikdbavishi has quit IRC18:50
*** nils has quit IRC18:59
*** saneax has quit IRC19:38
*** hasharAway is now known as hashar19:58
*** hashar has quit IRC20:10
*** sgw1 has quit IRC20:51
*** rfolco has quit IRC20:51
corvustobiash, zbr: prefs dialog lgtm -- https://review.opendev.org/739482   i left a long comment (cc tristanC)21:05
corvuszbr: can you reply to felixedel's comments on that?21:05
corvusclarkb: can you re-review that ^?21:05
clarkbya21:06
*** rlandy has quit IRC21:08
*** sgw1 has joined #zuul21:28
*** wuchunyang has joined #zuul22:06
*** wuchunyang has quit IRC22:07
*** mgoddard has quit IRC22:10
ianwtobiash/clarkb: yeah, putting the depends on the PR comment made it work :)  i will add something in docs on that :)22:13
*** mgoddard has joined #zuul22:18
*** rfolco has joined #zuul22:34
*** tosky has quit IRC22:57
*** saneax has joined #zuul23:14
*** rfolco has quit IRC23:29
*** saneax has quit IRC23:50
*** Goneri has quit IRC23:54

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