openstackgerrit | Ian Wienand proposed zuul/zuul-jobs master: Add ensure-rust role https://review.opendev.org/746423 | 00:06 |
---|---|---|
*** rfolco has quit IRC | 00:15 | |
openstackgerrit | Ian Wienand proposed zuul/zuul-jobs master: Add ensure-rust role https://review.opendev.org/746423 | 00:19 |
openstackgerrit | Ian Wienand proposed zuul/zuul-jobs master: Add ensure-rust role https://review.opendev.org/746423 | 00:30 |
*** yolanda has quit IRC | 01:26 | |
*** yolanda has joined #zuul | 01:26 | |
*** yolanda has quit IRC | 01:27 | |
*** yolanda has joined #zuul | 01:27 | |
ianw | corvus/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 |
clarkb | I think it should go both directions | 03:16 |
ianw | https://opendev.org/zuul/zuul/src/branch/master/zuul/driver/github/githubconnection.py#L1177 makes me think not | 03:17 |
ianw | https://zuul.opendev.org/t/pyca/build/f9978ceba64a454184f4ca1e9871fdcd is an example of the github change depends-on: zuul-jobs | 03:18 |
ianw | zuul-jobs isn't listed in the projects | 03:18 |
ianw | https://zuul.opendev.org/t/pyca/build/f9978ceba64a454184f4ca1e9871fdcd/log/zuul-info/inventory.yaml#53 | 03:18 |
*** bhavikdbavishi has joined #zuul | 03:36 | |
openstackgerrit | Ian Wienand proposed zuul/zuul master: github connection: remove unused depends_on_re https://review.opendev.org/746437 | 04:19 |
*** raukadah is now known as chkumar|rover | 04:20 | |
ianw | corvus/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 tomorrow | 04:20 |
*** evrardjp has quit IRC | 04:33 | |
*** evrardjp has joined #zuul | 04:33 | |
*** sgw1 has joined #zuul | 04:54 | |
*** saneax has joined #zuul | 05:04 | |
tobiash | ianw: in github the depends-on must be in the pr description, not in a commit message | 05:06 |
swest | clarkb: thanks for 746349. we started seeing exceptions for the build case aswell. | 05:07 |
openstackgerrit | Ian Wienand proposed zuul/zuul master: github connection: remove unused depends_on_re https://review.opendev.org/746437 | 05:44 |
*** mach1na has joined #zuul | 06:08 | |
*** mach1na has quit IRC | 06:08 | |
*** mach1na has joined #zuul | 06:08 | |
*** mach1na has quit IRC | 06:36 | |
*** bhavikdbavishi has quit IRC | 06:39 | |
*** vishalmanchanda has joined #zuul | 06:49 | |
openstackgerrit | Simon Westphahl proposed zuul/zuul master: Add optional support for circular dependencies https://review.opendev.org/685354 | 07:03 |
openstackgerrit | Simon Westphahl proposed zuul/zuul master: Check cycle items are mergeable before reporting https://review.opendev.org/743450 | 07:03 |
*** bhavikdbavishi has joined #zuul | 07:08 | |
*** mach1na has joined #zuul | 07:15 | |
*** mach1na has quit IRC | 07:19 | |
*** mach1na has joined #zuul | 07:19 | |
openstackgerrit | Albin Vass proposed zuul/zuul master: Add commit id and owner to Change for mqtt reporter https://review.opendev.org/722478 | 07:37 |
openstackgerrit | Tobias Henkel proposed zuul/zuul master: WIP: Switch to using zookeeper instead of gearman for jobs (keep gearman for mergers) https://review.opendev.org/744416 | 07:39 |
*** hashar has joined #zuul | 07:49 | |
*** tosky has joined #zuul | 07:56 | |
openstackgerrit | Simon Westphahl proposed zuul/zuul master: Add optional support for circular dependencies https://review.opendev.org/685354 | 07:57 |
openstackgerrit | Simon Westphahl proposed zuul/zuul master: Check cycle items are mergeable before reporting https://review.opendev.org/743450 | 07:57 |
*** jpena|off is now known as jpena | 07:58 | |
*** hashar has quit IRC | 08:10 | |
openstackgerrit | Felix Edel proposed zuul/zuul master: Return number of total results from builds/ and buildsets/ API endpoint https://review.opendev.org/746328 | 08:10 |
*** bhavikdbavishi1 has joined #zuul | 08:22 | |
*** bhavikdbavishi has quit IRC | 08:24 | |
*** bhavikdbavishi1 is now known as bhavikdbavishi | 08:24 | |
*** nils has joined #zuul | 08:40 | |
*** hashar has joined #zuul | 08:49 | |
openstackgerrit | Felix Edel proposed zuul/zuul master: web: Return number of total results from builds/ and buildsets/ API https://review.opendev.org/746328 | 09:01 |
openstackgerrit | Felix Edel proposed zuul/zuul master: WIP PF4: Add pagination to builds and buildsets pages https://review.opendev.org/746463 | 09:07 |
*** bhavikdbavishi has quit IRC | 09:09 | |
*** evgenyl has quit IRC | 09:10 | |
*** bhavikdbavishi has joined #zuul | 09:10 | |
*** tdasilva has quit IRC | 09:11 | |
*** evgenyl has joined #zuul | 09:18 | |
*** tdasilva has joined #zuul | 09:19 | |
ianw | tobiash: 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 |
tobiash | ianw: yes exactly | 09:48 |
tobiash | you can just edit that and then recheck | 09:49 |
ianw | tobiash: ahh, it's not my change but ok i can point that out. thanks, this had me in a muddle! | 09:56 |
tobiash | yeah, zuul uses the change message which fits better to the pr description in the github workflow | 09:59 |
mach1na | Hi 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 |
tobiash | mach1na: looking at the reviews it was supposed to be announced and merged after two weeks: https://review.opendev.org/#/c/742971/ | 10:12 |
tobiash | mach1na: this has been announced on the zuul-announce mailing list: http://lists.zuul-ci.org/pipermail/zuul-announce/2020-July/000080.html | 10:14 |
tobiash | mach1na: if you don't yet, you might wan't to subscribe to this list | 10:14 |
mach1na | tobiash: I am doing that right now. Thanks greatly for the clarification. | 10:16 |
tobiash | mach1na: this is the description of the deprecation policy in zuul-jobs: https://zuul-ci.org/docs/zuul-jobs/policy.html#deprecation-policy | 10:16 |
tobiash | mach1na: could you work around this or is this still a problem for you? | 10:16 |
mach1na | tobiash: 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 |
tobiash | mach1na: I was not involved in this change but it looks like that, yes | 10:23 |
mach1na | tobiash: cheers, thank you so much for the help. | 10:24 |
tobiash | no problem | 10:24 |
openstackgerrit | Matthieu Huin proposed zuul/zuul master: How-to: using the REST API with cURL https://review.opendev.org/727785 | 10:27 |
mhu | Hey 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 IRC | 10:41 | |
*** bhavikdbavishi has joined #zuul | 11:00 | |
*** mach1na has quit IRC | 11:13 | |
*** jpena is now known as jpena|lunch | 11:33 | |
*** weshay_ is now known as weshay|ruck | 11:36 | |
*** bhavikdbavishi has quit IRC | 11:40 | |
*** rfolco has joined #zuul | 11:43 | |
*** wuchunyang has joined #zuul | 11:49 | |
*** wuchunyang has quit IRC | 11:52 | |
*** mach1na has joined #zuul | 11:52 | |
*** wuchunyang has joined #zuul | 11:52 | |
*** wuchunyang has quit IRC | 11:54 | |
*** wuchunyang has joined #zuul | 11:54 | |
*** wuchunyang has quit IRC | 11:56 | |
openstackgerrit | Felix Edel proposed zuul/zuul master: PF4: Add icons and empty state to artifacts section on build result page https://review.opendev.org/739973 | 11:56 |
*** wuchunyang has joined #zuul | 11:57 | |
*** wuchunyang has quit IRC | 12:00 | |
*** mach1na has quit IRC | 12:02 | |
*** mach1na has joined #zuul | 12:03 | |
*** rlandy has joined #zuul | 12:05 | |
*** bhavikdbavishi has joined #zuul | 12:12 | |
*** jpena|lunch is now known as jpena | 12:38 | |
*** bhavikdbavishi has quit IRC | 12:44 | |
zbr | Any chance to get the +W on the preferences dialog change? https://review.opendev.org/#/c/739482/ | 13:05 |
*** Goneri has joined #zuul | 13:13 | |
mhu | zbr: 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 |
mhu | or would it be better to switch later | 13:18 |
zbr | mhu: we can do this later if needed. likely once the list grows | 13:19 |
zbr | but until we have >10 settings there, I do not see a need. | 13:19 |
zbr | i plan to add ~5 so it may not be very far in the future | 13:19 |
*** mach1na has quit IRC | 13:33 | |
*** mach1na has joined #zuul | 13:33 | |
*** mach1na has quit IRC | 13:34 | |
*** mach1na has joined #zuul | 13:35 | |
*** hashar has quit IRC | 13:51 | |
*** hashar has joined #zuul | 13:54 | |
*** LLIU82 has joined #zuul | 13:54 | |
openstackgerrit | Benjamin Schanzel proposed zuul/zuul-jobs master: Multi Node/Context support for mirroring Git workspaces via kubectl https://review.opendev.org/746508 | 13:56 |
LLIU82 | I 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-dateutil | 14:04 |
corvus | LLIU82: 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 |
openstackgerrit | Benjamin Schanzel proposed zuul/zuul master: Fix Multi Node Builds with k8s Build Nodes https://review.opendev.org/746515 | 14:16 |
*** sshnaidm is now known as sshnaidm|MYG | 14:27 | |
*** sshnaidm|MYG is now known as sshnaidm|mtg | 14:27 | |
avass | corvus, 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 raukadah | 14:45 | |
tobiash | corvus: 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 again | 14:49 |
clarkb | LLIU82: 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 too | 14:50 |
clarkb | my hunch is that the issue is related to that | 14:51 |
fungi | we 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 feature | 14:51 |
openstackgerrit | Jonas Sticha proposed zuul/zuul master: WIP: Make repo state buildset global https://review.opendev.org/738603 | 14:58 |
openstackgerrit | Lida Liu proposed zuul/zuul master: Add commit id to Change for mqtt reporter https://review.opendev.org/722478 | 15:27 |
*** LLIU82 has quit IRC | 15:28 | |
fungi | er, i meant older pip | 15:28 |
*** mach1na has quit IRC | 15:42 | |
*** mach1na has joined #zuul | 15:43 | |
*** mach1na has quit IRC | 15:47 | |
*** mach1na has joined #zuul | 15:47 | |
*** mach1na has quit IRC | 15:59 | |
*** sshnaidm|mtg is now known as sshnaidm | 16:08 | |
*** jpena is now known as jpena|off | 16:14 | |
*** bhavikdbavishi has joined #zuul | 16:38 | |
*** hashar is now known as hasharAway | 17:10 | |
*** vishalmanchanda has quit IRC | 17:37 | |
*** wuchunyang has joined #zuul | 18:04 | |
*** wuchunyang has quit IRC | 18:05 | |
*** bhavikdbavishi has quit IRC | 18:50 | |
*** nils has quit IRC | 18:59 | |
*** saneax has quit IRC | 19:38 | |
*** hasharAway is now known as hashar | 19:58 | |
*** hashar has quit IRC | 20:10 | |
*** sgw1 has quit IRC | 20:51 | |
*** rfolco has quit IRC | 20:51 | |
corvus | tobiash, zbr: prefs dialog lgtm -- https://review.opendev.org/739482 i left a long comment (cc tristanC) | 21:05 |
corvus | zbr: can you reply to felixedel's comments on that? | 21:05 |
corvus | clarkb: can you re-review that ^? | 21:05 |
clarkb | ya | 21:06 |
*** rlandy has quit IRC | 21:08 | |
*** sgw1 has joined #zuul | 21:28 | |
*** wuchunyang has joined #zuul | 22:06 | |
*** wuchunyang has quit IRC | 22:07 | |
*** mgoddard has quit IRC | 22:10 | |
ianw | tobiash/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 #zuul | 22:18 | |
*** rfolco has joined #zuul | 22:34 | |
*** tosky has quit IRC | 22:57 | |
*** saneax has joined #zuul | 23:14 | |
*** rfolco has quit IRC | 23:29 | |
*** saneax has quit IRC | 23:50 | |
*** Goneri has quit IRC | 23:54 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!