*** zbr9 has joined #softwarefactory | 03:04 | |
*** zbr has quit IRC | 03:04 | |
*** dpawlik2 has quit IRC | 03:04 | |
*** zbr9 is now known as zbr | 03:04 | |
*** irclogbot_3 has quit IRC | 03:05 | |
*** irclogbot_0 has joined #softwarefactory | 03:08 | |
*** danpawlik has joined #softwarefactory | 06:37 | |
*** hashar has joined #softwarefactory | 06:49 | |
*** danpawlik has quit IRC | 06:54 | |
*** danpawlik has joined #softwarefactory | 06:55 | |
*** hashar has quit IRC | 07:05 | |
sfbender | Merged software-factory/dhall-software-factory master: Compose obj.Name to simplify the renderManagesf function https://softwarefactory-project.io/r/19250 | 07:36 |
---|---|---|
sfbender | Merged software-factory/dhall-software-factory master: Remove unused SourceRepositoryOptions package https://softwarefactory-project.io/r/19251 | 07:38 |
sfbender | Merged software-factory/dhall-zuul master: Add tenant configuration https://softwarefactory-project.io/r/19248 | 07:38 |
*** hashar has joined #softwarefactory | 07:51 | |
*** jpena|off is now known as jpena | 07:54 | |
sfbender | Fabien Boucher created software-factory/dhall-software-factory master: renderManagesf: Add missing connections rendering https://softwarefactory-project.io/r/19254 | 08:03 |
*** apevec has joined #softwarefactory | 10:51 | |
*** hashar has quit IRC | 10:53 | |
*** hashar has joined #softwarefactory | 11:12 | |
*** hashar has quit IRC | 11:12 | |
*** jpena is now known as jpena|lunch | 11:33 | |
*** hashar has joined #softwarefactory | 12:36 | |
*** jpena|lunch is now known as jpena | 12:39 | |
sfbender | Tristan de Cacqueray created software-factory/sf-config master: nodepool: fix issues with exceptions logs http location https://softwarefactory-project.io/r/19267 | 14:35 |
zbr | tristanC: can you please help me with https://review.opendev.org/#/c/739482/ | 14:49 |
tristanC | zbr: i need to finish sf-3.5 release and ensure zk tls is working, i can have a look before my pto | 14:50 |
zbr | tristanC: it would really be useful as I kinda find hard to convince it to pass (many are on pto) | 14:51 |
sfbender | Tristan de Cacqueray created www.softwarefactory-project.io master: Add sf-3.5 release note update https://softwarefactory-project.io/r/19268 | 15:00 |
sfbender | Merged software-factory/dhall-software-factory master: renderManagesf: Add missing connections rendering https://softwarefactory-project.io/r/19254 | 15:07 |
sfbender | Merged software-factory/dhall-containerfile master: Update RUN statement to single Text and refactor render functions https://softwarefactory-project.io/r/19219 | 15:12 |
sfbender | Merged software-factory/dhall-containerfile master: Add RUN exec form https://softwarefactory-project.io/r/19220 | 15:12 |
sfbender | Merged software-factory/dhall-containerfile master: Add CMD statement https://softwarefactory-project.io/r/19221 | 15:12 |
sfbender | Merged software-factory/dhall-containerfile master: Add LABEL statement https://softwarefactory-project.io/r/19222 | 15:12 |
sfbender | Merged software-factory/dhall-containerfile master: Add EXPOSE statement https://softwarefactory-project.io/r/19223 | 15:14 |
sfbender | Merged software-factory/dhall-containerfile master: Add ADD statement https://softwarefactory-project.io/r/19224 | 15:14 |
sfbender | Merged software-factory/dhall-containerfile master: Add COPY statement https://softwarefactory-project.io/r/19225 | 15:14 |
sfbender | Merged software-factory/dhall-containerfile master: Add VOLUME statement https://softwarefactory-project.io/r/19226 | 15:14 |
sfbender | Merged software-factory/dhall-containerfile master: Add USER statement https://softwarefactory-project.io/r/19227 | 15:14 |
sfbender | Merged software-factory/dhall-containerfile master: Add ARG statement https://softwarefactory-project.io/r/19228 | 15:25 |
sfbender | Merged software-factory/dhall-containerfile master: Fix Statement order https://softwarefactory-project.io/r/19229 | 15:25 |
*** jpena is now known as jpena|off | 16:13 | |
dmsimard | <3 software factory zuul in pagure | 16:25 |
tristanC | dmsimard: thanks, we appreciate the feedback! | 16:29 |
sfbender | Merged www.softwarefactory-project.io master: Add sf-3.5 release note update https://softwarefactory-project.io/r/19268 | 16:29 |
sfbender | Merged software-factory/sf-config master: nodepool: fix issues with exceptions logs http location https://softwarefactory-project.io/r/19267 | 16:32 |
*** hashar has quit IRC | 17:10 | |
*** adamw has joined #softwarefactory | 18:04 | |
adamw | ahoy folks | 18:04 |
adamw | a CI job I have running in pagure.io that uses tox is suddenly not working properly, and we didn't change anything on our end | 18:05 |
adamw | anyone know what's going no? | 18:05 |
adamw | https://fedora.softwarefactory-project.io/zuul/build/211ab6902e4b4e8b9da532eda802abd8 | 18:05 |
adamw | https://pagure.io/fedora-qa/os-autoinst-distri-fedora/blob/master/f/ci/tox.yaml is the config, it includes the ensure-tox role | 18:06 |
tristanC | adamw: that is because we updated the zuul-jobs using the latest opendev.org/zuul/zuul-jobs | 18:22 |
adamw | tristanC: hum. is there somewhere this sort of change and the changes required to our CI config are announced? | 18:23 |
tristanC | adamw: that was unexpected, iirc upstream changed how tox gets installed in their image, resulting in modification of zuul-jobs | 18:24 |
tristanC | adamw: let me propose a fix | 18:24 |
adamw | thanks | 18:24 |
tristanC | adamw: hopefully https://pagure.io/fedora-qa/os-autoinst-distri-fedora/pull-request/183 should work | 18:31 |
tristanC | adamw: btw, the upgrade was announced https://lists.fedorahosted.org/archives/list/ci@lists.fedoraproject.org/thread/UGBPOUXKIHY246L6UJ6OE7ADBXKCNVYP/ | 18:31 |
adamw | tristanC: noted, though that doesn't give any indication of what's changing or that jobs will stop working :) | 18:35 |
adamw | so they "added an option" to do something but made the new default the opposite of what it did before? gotta love that. | 18:36 |
tristanC | adamw: such change are usually announced in http://lists.zuul-ci.org/pipermail/zuul-announce/ | 18:38 |
tristanC | adamw: but note that we don't run opendev.org/zuul/zuul-jobs master, we update a local copy when doing upgrade | 18:41 |
adamw | i'm not seeing anything there :/ | 18:41 |
tristanC | adamw: yeah we are discussing this issue in #zuul, but that's probably an old change that introduced the new behavior | 18:42 |
adamw | tristanC: is there a magic comment or something i can use to re-test an existing pr? | 18:47 |
tristanC | adamw: `recheck` should do the trick | 18:47 |
adamw | hmm, on a random side note, sure would be nice if i didn't have to manually bump the list of python versions to install... | 18:48 |
adamw | thanks | 18:48 |
tristanC | adamw: pipeline triggers are defined here: https://pagure.io/fedora-project-config/blob/master/f/zuul.d/_pipelines.yaml | 18:48 |
tristanC | adamw: what do you mean bump the list of python versions? | 18:48 |
adamw | - name: Install all Python versions to test | 18:48 |
adamw | package: | 18:48 |
adamw | name: ['python37', 'python38', 'python39'] | 18:48 |
adamw | state: present | 18:48 |
adamw | i mean that :D | 18:48 |
tristanC | adamw: perhaps we should use the `python-tox` fedora package instead? | 18:51 |
adamw | that would get you whatever the system python currently is, only | 18:51 |
adamw | whereas i'm installing multiple versions so tox will test them all | 18:51 |
adamw | probably a bit too specialized to abstract out somewhere, oh well | 18:51 |
adamw | using the package for tox instead of ensure-tox would...work too, i guess? i think you included ensure-tox in the initial config you gave me or recommended it and i just used it without wondering what it did | 18:52 |
adamw | i guess i was assuming it installed the package | 18:52 |
tristanC | adamw: running `dnf install python-tox` does try to install all the available python runtime | 18:54 |
tristanC | adamw: there is also an `ensure-python` role, but i think what you have here is ideal as you pull only what you need | 18:55 |
adamw | aha, i see, it's because python3-tox recommends all the various runtimes | 18:55 |
adamw | oh well, i'll leave it for now i guess, it works :) thanks for the fix | 18:56 |
tristanC | adamw: about the breaking change, ftr here is the result of the upstream discussion: | 18:56 |
tristanC | < corvus> tristanC: i suspect that the reviewers only considered the case of other roles in zuul-jobs relying on the behavior, not end-user playbooks, and so probably didn't realize it would be a breaking change in that case | 18:56 |
tristanC | < corvus> tristanC: (iow, we probably looked at it and said: "ensure-tox role + tox role" works, so we're good) | 18:56 |
adamw | should i be using the tox role? | 18:57 |
tristanC | adamw: it's worth a try, but i think what you have now is correct | 18:58 |
tristanC | adamw: next time we'll explicit mention possible change in the provided zuul-jobs roles | 19:03 |
adamw | thanks | 19:04 |
*** hashar has joined #softwarefactory | 19:31 | |
*** hashar has quit IRC | 20:07 | |
*** hashar has joined #softwarefactory | 20:09 | |
*** hashar has quit IRC | 21:00 | |
*** logan- has quit IRC | 23:05 | |
*** apevec has quit IRC | 23:33 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!