*** threestrands has joined #zuul | 00:04 | |
*** cdearborn has quit IRC | 00:06 | |
*** rlandy has quit IRC | 00:15 | |
*** y2kenny has quit IRC | 00:43 | |
openstackgerrit | Ian Wienand proposed zuul/nodepool master: Dockerfile: incorporate workaround deboostrap https://review.opendev.org/721394 | 00:45 |
---|---|---|
openstackgerrit | Ian Wienand proposed zuul/nodepool master: Siblings container build: set work dir to nodepool https://review.opendev.org/721514 | 00:45 |
openstackgerrit | Ian Wienand proposed zuul/nodepool master: Functional container tests: update to CentOS 8 https://review.opendev.org/721509 | 00:45 |
openstackgerrit | Ian Wienand proposed zuul/nodepool master: func-tests: drop debuntu specific env vars https://review.opendev.org/721754 | 00:45 |
openstackgerrit | Tristan Cacqueray proposed zuul/zuul-jobs master: cabal-test: add initial haskell job https://review.opendev.org/721735 | 00:55 |
openstackgerrit | Tristan Cacqueray proposed zuul/zuul-jobs master: cabal-test: add initial haskell job https://review.opendev.org/721735 | 01:10 |
*** swest has quit IRC | 01:10 | |
*** Goneri has quit IRC | 01:12 | |
*** swest has joined #zuul | 01:25 | |
*** bhavikdbavishi has joined #zuul | 03:27 | |
*** bhavikdbavishi1 has joined #zuul | 03:30 | |
*** bhavikdbavishi has quit IRC | 03:31 | |
*** bhavikdbavishi1 is now known as bhavikdbavishi | 03:31 | |
*** bhavikdbavishi has quit IRC | 04:12 | |
*** bhavikdbavishi has joined #zuul | 04:12 | |
*** bhavikdbavishi has quit IRC | 04:15 | |
*** bhavikdbavishi has joined #zuul | 04:15 | |
*** evrardjp has quit IRC | 04:35 | |
*** evrardjp has joined #zuul | 04:35 | |
*** rfolco has quit IRC | 05:09 | |
*** ysandeep|away is now known as ysandeep | 05:34 | |
*** bhavikdbavishi has quit IRC | 05:37 | |
*** bhavikdbavishi has joined #zuul | 06:02 | |
*** saneax has joined #zuul | 06:04 | |
openstackgerrit | Merged zuul/zuul-jobs master: helm-template: enable using values file https://review.opendev.org/721365 | 06:09 |
*** dpawlik has joined #zuul | 06:24 | |
*** bhavikdbavishi has quit IRC | 06:29 | |
*** bhavikdbavishi has joined #zuul | 06:30 | |
*** bhavikdbavishi1 has joined #zuul | 06:48 | |
*** bhavikdbavishi has quit IRC | 06:50 | |
*** bhavikdbavishi1 is now known as bhavikdbavishi | 06:50 | |
*** threestrands has quit IRC | 06:53 | |
*** threestrands has joined #zuul | 06:54 | |
*** threestrands has quit IRC | 06:55 | |
*** threestrands has joined #zuul | 06:55 | |
*** threestrands has quit IRC | 06:56 | |
*** threestrands has joined #zuul | 06:56 | |
*** threestrands has quit IRC | 06:58 | |
*** threestrands has joined #zuul | 06:58 | |
*** threestrands has quit IRC | 06:59 | |
*** threestrands has joined #zuul | 06:59 | |
*** threestrands has quit IRC | 07:01 | |
*** threestrands has joined #zuul | 07:01 | |
*** threestrands has quit IRC | 07:02 | |
*** threestrands has joined #zuul | 07:02 | |
*** threestrands has quit IRC | 07:04 | |
*** threestrands has joined #zuul | 07:04 | |
*** threestrands has quit IRC | 07:05 | |
*** threestrands has joined #zuul | 07:06 | |
*** threestrands has quit IRC | 07:07 | |
*** threestrands has joined #zuul | 07:07 | |
*** threestrands has quit IRC | 07:08 | |
*** threestrands has joined #zuul | 07:08 | |
*** jcapitao has joined #zuul | 07:09 | |
*** threestrands has quit IRC | 07:10 | |
*** threestrands has joined #zuul | 07:10 | |
*** threestrands has quit IRC | 07:11 | |
*** threestrands has joined #zuul | 07:12 | |
*** threestrands has quit IRC | 07:13 | |
*** threestrands has joined #zuul | 07:13 | |
*** threestrands has quit IRC | 07:14 | |
*** threestrands has joined #zuul | 07:14 | |
*** threestrands has quit IRC | 07:16 | |
*** threestrands has joined #zuul | 07:16 | |
*** threestrands has quit IRC | 07:17 | |
*** threestrands has joined #zuul | 07:18 | |
*** threestrands has quit IRC | 07:19 | |
*** threestrands has joined #zuul | 07:19 | |
*** threestrands has quit IRC | 07:20 | |
*** threestrands has joined #zuul | 07:21 | |
*** threestrands has quit IRC | 07:22 | |
*** threestrands has joined #zuul | 07:22 | |
*** threestrands has quit IRC | 07:23 | |
*** threestrands has joined #zuul | 07:24 | |
*** rpittau|afk is now known as rpittau | 07:24 | |
*** threestrands has quit IRC | 07:25 | |
*** threestrands has joined #zuul | 07:25 | |
*** sugaar has quit IRC | 07:25 | |
*** threestrands has quit IRC | 07:26 | |
*** threestrands has joined #zuul | 07:27 | |
*** threestrands has quit IRC | 07:28 | |
*** threestrands has joined #zuul | 07:28 | |
*** threestrands has quit IRC | 07:29 | |
*** threestrands has joined #zuul | 07:30 | |
*** yolanda has joined #zuul | 07:30 | |
*** threestrands has quit IRC | 07:31 | |
*** threestrands has joined #zuul | 07:31 | |
*** threestrands has quit IRC | 07:32 | |
*** threestrands has joined #zuul | 07:33 | |
*** threestrands has quit IRC | 07:34 | |
*** threestrands has joined #zuul | 07:34 | |
*** threestrands has quit IRC | 07:35 | |
*** panda|ruck has quit IRC | 07:35 | |
*** threestrands has joined #zuul | 07:36 | |
*** bhavikdbavishi has quit IRC | 07:36 | |
*** threestrands has quit IRC | 07:37 | |
*** panda has joined #zuul | 07:37 | |
*** threestrands has joined #zuul | 07:37 | |
*** threestrands has quit IRC | 07:38 | |
*** sugaar has joined #zuul | 07:38 | |
*** threestrands has joined #zuul | 07:39 | |
*** threestrands has quit IRC | 07:40 | |
*** threestrands has joined #zuul | 07:40 | |
*** threestrands has quit IRC | 07:41 | |
*** threestrands has joined #zuul | 07:41 | |
*** threestrands has quit IRC | 07:43 | |
*** threestrands has joined #zuul | 07:43 | |
*** threestrands has quit IRC | 07:44 | |
*** threestrands has joined #zuul | 07:45 | |
*** tosky has joined #zuul | 07:45 | |
*** threestrands has quit IRC | 07:46 | |
*** threestrands has joined #zuul | 07:46 | |
*** threestrands has quit IRC | 07:47 | |
*** threestrands has joined #zuul | 07:47 | |
*** threestrands has quit IRC | 07:49 | |
*** threestrands has joined #zuul | 07:49 | |
*** threestrands has quit IRC | 07:50 | |
*** threestrands has joined #zuul | 07:51 | |
*** threestrands has quit IRC | 07:52 | |
*** threestrands has joined #zuul | 07:52 | |
*** threestrands has quit IRC | 07:53 | |
*** threestrands has joined #zuul | 07:54 | |
*** threestrands has quit IRC | 07:55 | |
*** threestrands has joined #zuul | 07:55 | |
*** jpena|off is now known as jpena | 07:55 | |
*** threestrands has quit IRC | 07:56 | |
*** threestrands has joined #zuul | 07:57 | |
*** threestrands has quit IRC | 07:58 | |
*** threestrands has joined #zuul | 07:58 | |
*** threestrands has quit IRC | 07:59 | |
*** threestrands has joined #zuul | 08:00 | |
*** threestrands has quit IRC | 08:01 | |
*** threestrands has joined #zuul | 08:01 | |
*** threestrands has quit IRC | 08:02 | |
*** threestrands has joined #zuul | 08:02 | |
*** threestrands has quit IRC | 08:04 | |
*** threestrands has joined #zuul | 08:04 | |
*** threestrands has quit IRC | 08:05 | |
*** threestrands has joined #zuul | 08:06 | |
*** threestrands has quit IRC | 08:07 | |
*** threestrands has joined #zuul | 08:07 | |
openstackgerrit | Jan Kubovy proposed zuul/zuul master: Mandatory Zookeeper connection for ZuulWeb in tests https://review.opendev.org/721254 | 08:08 |
openstackgerrit | Jan Kubovy proposed zuul/zuul master: WIP: Driver event ingestion https://review.opendev.org/717299 | 08:08 |
*** threestrands has quit IRC | 08:08 | |
*** bhavikdbavishi has joined #zuul | 08:10 | |
openstackgerrit | Albin Vass proposed zuul/zuul-jobs master: tox: run default envlist if tox_envlist is empty https://review.opendev.org/721790 | 08:22 |
*** bhavikdbavishi has quit IRC | 08:31 | |
openstackgerrit | Albin Vass proposed zuul/zuul-jobs master: tox: Run all environments in tox_envlist https://review.opendev.org/721790 | 08:36 |
openstackgerrit | Simon Westphahl proposed zuul/zuul master: Add optional support for circular dependencies https://review.opendev.org/685354 | 08:36 |
*** ysandeep is now known as ysandeep|lunch | 08:46 | |
*** panda is now known as panda|ruck | 08:48 | |
openstackgerrit | Albin Vass proposed zuul/zuul-jobs master: tox: Run all environments in tox_envlist https://review.opendev.org/721790 | 09:00 |
*** ysandeep|lunch is now known as ysandeep | 09:06 | |
openstackgerrit | Jan Kubovy proposed zuul/zuul master: Driver event ingestion https://review.opendev.org/717299 | 09:08 |
openstackgerrit | Albin Vass proposed zuul/zuul-jobs master: tox: update tox_envlist documentation https://review.opendev.org/721796 | 09:19 |
*** bhavikdbavishi has joined #zuul | 09:24 | |
*** evrardjp has quit IRC | 09:51 | |
*** evrardjp has joined #zuul | 09:51 | |
*** rpittau is now known as rpittau|bbl | 10:42 | |
*** jcapitao is now known as jcapitao_lunch | 10:45 | |
tobiash | zuul-maint: fyi, we're currently trying out some settings regarding git auto gc to prevent job startup times from getting worse over time (http://paste.openstack.org/show/792528/) | 10:55 |
zbr | tobiash: i guess this has to to with zuul issues at this moment. i was checking my email to see if i missed something. | 10:56 |
tobiash | zbr: which zuul issues? | 10:58 |
zbr | i seend lots of config errors on openstack tenant https://zuul.opendev.org/t/openstack/status | 10:58 |
tobiash | zbr: that's unrelated | 10:59 |
zbr | ahh, ok. | 10:59 |
tobiash | zuul-maint: I see two possibilities to continue on auto gc if that works out. 1. document, 2. configure automatically when cloning/updating a repo. I guess I'd vote for 2 | 11:04 |
jkt | where do I find that code which takes the "last part" of a job's failing task log and shows that in zuul-web's per-build page? | 11:21 |
jkt | I'd like to plug ansi2html there | 11:21 |
zbr | jkt: not sure if a problem here but ansi2html is GPL! | 11:22 |
zbr | i was planning to rewrite it as MIT but never got time | 11:22 |
jkt | zbr: no idea about acceptable licenses for opendev | 11:23 |
jkt | looks like it's web/src/containers/build/Console.jsx -- too bad I don't know React | 11:23 |
zbr | jkt: there is a JS based approach to render ANSI, i even have a change but i depend on react-app to be upgraded. | 11:23 |
zbr | mainly, don't work on this, I already started :D | 11:24 |
jkt | awesome | 11:24 |
jkt | I think I'll also take a look at htmlify-logs, that sounds relevant even if my jobs currently don't use that | 11:25 |
zbr | jkt: but watch https://review.opendev.org/#/c/716251/ - is abandoned but I can reopne. | 11:25 |
jkt | zbr: no hurries. I got here when I tried to de-uglify pytest' diffing of looong strings | 11:25 |
zbr | mainly I hope to see someone with react skills fix https://review.opendev.org/#/c/716305/ | 11:25 |
zbr | once done, i can go back to enable ANSI | 11:26 |
jkt | we have a physics sim library, and we recently added "end-to-end" testing which takes a look at the output of our demo utilities | 11:26 |
AJaeger | zbr: those are a half-done project retirement of openstack/openstack-ansible-pip_install, openstack/openstack-ansible-repo_build - mnaser, could continue to look into that, please? | 11:26 |
AJaeger | mnaser: https://zuul.opendev.org/t/openstack/config-errors | 11:26 |
jkt | I found pytest-clarity, but its output is really hard to read in a plaintext log, especially when our internal code uses this coloring unconditionally | 11:27 |
*** jpena is now known as jpena|lunch | 11:29 | |
zbr | jkt: what i use with molecule is pytest-html, which provides really easy to read html reports. let me give an example | 11:39 |
zbr | look at https://zuul.opendev.org/t/openstack/build/085e989873ec4cf0808f0d53ede427d9 - click "Molecule report" | 11:40 |
zbr | that is nothing else than "pytest-html" output. | 11:41 |
zbr | you need to also install ansi2html to benefit from ANSI feature (is not enabled as direct dependency due to GPL) | 11:42 |
zbr | but as this is test code, is ok to mix GPL, as it never ends up being shipped. | 11:42 |
*** ysandeep is now known as ysandeep|brb | 11:46 | |
tristanC | jkt: i think you are looking for https://opendev.org/zuul/zuul/src/branch/master/web/src/containers/build/BuildOutput.jsx#L56 , L75 is doing the "last part" split | 11:47 |
tristanC | tobiash: i would also vote for self-healing strategy. is there already a change to review for that? | 11:48 |
*** rpittau|bbl is now known as rpittau | 11:50 | |
tristanC | jkt: react, at least its usage in zuul, shouldn't be too difficult to learn after python. I'd be happy to help if you have questions. | 11:51 |
tristanC | jkt: i think the main source of confusion is that react is written in ES6 (the js bits) and inlined JSX (the html bits). ES6 can also be inlined inside JSX. Thus understanding the different part independently is valuable for correctly reading react code. | 11:54 |
tobiash | tristanC: not yet | 12:02 |
tristanC | zuul-maint : it seems like zuul images could use a fixed openssl package (it needs >= 1.1.1d-0+deb10u3). | 12:06 |
*** rlandy has joined #zuul | 12:08 | |
*** jcapitao_lunch is now known as jcapitao | 12:15 | |
jkt | tristanC: thanks -- it's just a matter of priorities for me | 12:19 |
jkt | zbr: I'll check this out, thanks | 12:19 |
*** rfolco has joined #zuul | 12:22 | |
*** jpena|lunch is now known as jpena | 12:35 | |
*** Goneri has joined #zuul | 12:38 | |
*** ysandeep|brb is now known as ysandeep | 12:41 | |
corvus | jkt: i think the new js build log display has largely replaced htmlify-logs; there's a lot more we can do there, we can do it dynamically (so it applies to already stored logs), and it keeps log processing in jobs shorter and simpler. so if there's something you want to do in that area, i'd recommend doing it in the web js rather than htmlify-logs | 13:07 |
corvus | jkt: the js for that is pretty straightforword. there are 2 bits of code that are relevant. 1) when the log is fetched, we do some pre-processing on it here: https://opendev.org/zuul/zuul/src/branch/master/web/src/actions/logfile.js#L45 | 13:08 |
corvus | jkt: 2) then we display that data structure here: https://opendev.org/zuul/zuul/src/branch/master/web/src/containers/logfile/LogFile.jsx#L69-L87 | 13:09 |
corvus | tobiash: yes, i don't think we want zuul admins knowing too much about zuul's git repos, zuul should be in complete control of those | 13:11 |
tobiash | corvus: yeah, I'll pepare a change for that then | 13:11 |
corvus | tobiash: i'm looking forward to learning what you've learned :) | 13:12 |
tobiash | first thing I've learned today is that we need to disable autoDetach :) | 13:12 |
mordred | tristanC: thanks - I just approved a patch from tobiash so we should get new images when it lands | 13:13 |
mordred | tobiash, corvus: the patch from tobiash I just approved (not trying to stream from winrm) made me think - maybe we should add a flag variable that can be set on a host to disable attempting to stream logs - when we're doing production deploys in opendev the zuul log is full of "waiting for streamer" - because we don't run zuul-console on the static host | 13:14 |
tobiash | mordred: that's an interesting idea | 13:15 |
mordred | and we _know_ that about the host - so telling zuul to not try streaming stdout/stderr seems maybe appropriate? | 13:15 |
openstackgerrit | Monty Taylor proposed zuul/zuul master: Add flag for disabling log streaming on a host https://review.opendev.org/721953 | 13:21 |
mordred | tobiash, corvus : ^^ | 13:21 |
*** bhavikdbavishi has quit IRC | 13:22 | |
jkt | zbr: your ANSI coloring looks wonderful :). Now compare it with this mess, https://object-storage-ca-ymq-1.vexxhost.net/v1/bfd521072e894ebb99e66f72619daa8a/vexxhost-ci_0bf/490704/1/check/tox-py37-cover/0bfd50e/tox/reports.html | 13:42 |
jkt | corvus: makes sense, thanks | 13:43 |
corvus | mordred: yeah, i think i could see that. i think we can test that with the 'remote' tox env | 13:43 |
zbr | jkt: you need to get bit more creative with how you do these asserts, i agree that is not readable | 13:44 |
jkt | zbr: here's how a similar error looks with pytest-clarity, https://zuul.vexxhost.dev/t/TelecomInfraProject/build/1a65b9019cce4f4e900a7da75d575f04 when ignoring ANSI | 13:44 |
jkt | yeah, that one is a full-program-output comparison; I think that there *is* some value in checking that as well | 13:44 |
zbr | i remember I used a strip_ansi function before doing the asserts | 13:44 |
zbr | this will help, also you need a better printer for multilines, to avoid the b'' parts and escaping. | 13:45 |
zbr | i do not rememeber what i did about this | 13:45 |
* jkt tries that | 13:45 | |
jkt | hey, a familiar name :), https://github.com/pytest-dev/pytest/issues/6586 | 13:46 |
zbr | my second recommandation is to relax testing, never test for exact log output, check of "expected_line(s) in result", allow for extra content. | 13:47 |
zbr | i seen this kind of test failures in many projects, where the tests where too strict about the outcome, they failed in various schenarios. | 13:47 |
*** sgw has quit IRC | 13:47 | |
*** zxiiro has joined #zuul | 13:52 | |
openstackgerrit | Merged zuul/nodepool master: Dockerfile: incorporate workaround deboostrap https://review.opendev.org/721394 | 13:57 |
*** sgw has joined #zuul | 14:06 | |
avass | mordred, tobiash: We have started work to get a log streamer for windows, but it still needs some fixing :) | 14:11 |
mordred | avass: neat! | 14:13 |
tobiash | avass: woot, wanna have :) | 14:13 |
avass | it could take some time until it's ready becaue the guy who wrote it quit, so I'll have to pick it up some time in the future | 14:13 |
openstackgerrit | Merged zuul/zuul master: Don't try to stream from winrm connections https://review.opendev.org/717973 | 14:18 |
mordred | avass: might be worth pushing it up half-finished - you might nerd-snipe tobiash into finishing it ;) | 15:06 |
mnaser | AJaeger: let me delegate that for now.. | 15:11 |
AJaeger | mnaser: sure ;) | 15:12 |
avass | there's no generic http reporter for zuul riht? | 15:13 |
fungi | nope, not sure what a generic http reporter would be anyway | 15:14 |
fungi | i guess something which fires off arbitrarily configurable http requests? | 15:15 |
avass | yup | 15:15 |
fungi | doesn't seem like it would be hard to incorporate, but sounds like it might be a challenge to design something flexibly general | 15:15 |
avass | We want to send some json to another service with pipeline name, ref, success/failure in a json format | 15:15 |
fungi | we have reporters which report via http requests, but they're designed with the expectations of the apis to which they report in mind | 15:16 |
fungi | so basically designing an http+json reporting api, as an analog of the python interface for reporter connection drivers? | 15:19 |
avass | something like that | 15:19 |
fungi | seems like the api design is going to be the complicated part, not the implementation | 15:19 |
fungi | (deciding on stability and backward compatibility policies, versioning mechanism, et cetera) | 15:21 |
avass | mordred: I'll see what I can do ;) | 15:23 |
avass | sometimes I wish I could run custom ansible scripts as a reporter | 15:24 |
avass | I guess what I want is to be able to run the uri module but on start/success/failure of the pipeline: https://docs.ansible.com/ansible/latest/modules/uri_module.html | 15:48 |
fungi | that would also probably be challenging. at the moment zuul only runs ansible in the context of a specific build, there's nothing which executes ansible in the context of an entire buildset, though maybe you could emulate that with a job depending on the other jobs run for a project in that pipeline | 16:01 |
avass | yeah that was the initial idea, but if any job fails, that job doesn't run | 16:01 |
fungi | a nodeless executor-only job calling the uri module | 16:01 |
fungi | hrm, i thought we had a way to run a dependent job even if its dependency fails, but maybe not | 16:02 |
avass | I'll check | 16:02 |
avass | doesn't look like it | 16:04 |
avass | fungi: I guess you're thinking of dependencies https://zuul-ci.org/docs/zuul/reference/job_def.html#attr-job.dependencies | 16:05 |
avass | or does the soft dependency mean, do not run and do not cause an error or, do not cause an error but still run? I don't remember | 16:06 |
fungi | avass: yep, and soft is only about it being okay that a dependency did not get run at all. failures still look like they cause the dependent job not to run | 16:08 |
*** igordc has joined #zuul | 16:12 | |
openstackgerrit | Albin Vass proposed zuul/zuul master: Make the soft job dependency doc clearer https://review.opendev.org/722058 | 16:13 |
avass | fungi: how about we make that a bit clearer | 16:14 |
avass | since 'A soft dependency will simply be ignored' sounds like the job will wait for the dependency if it exists and otherwise act as if the dependency wasn't configured | 16:15 |
fungi | i'm not sure that's any clearer. what it was trying to say is that if a job defines a soft dependency, then it's okay for that dependency to not actually be selected to run, and that soft dependencies which aren't run would be treated as if they succeeded | 16:16 |
openstackgerrit | James E. Blair proposed zuul/zuul-registry master: Support listing tags https://review.opendev.org/722062 | 16:16 |
corvus | mordred: ^ piece of cake | 16:16 |
fungi | avass: well, i suppose your wording there works in the context of the sentences leading up to it | 16:17 |
corvus | avass: there's an mqtt reportor; maybe you can subscribe your external service to an mqtt bus? | 16:18 |
avass | corvus: it's not our service unfortunately | 16:19 |
corvus | avass: do you control the input schema? | 16:19 |
avass | coruvs: unfortunately not | 16:19 |
fungi | is it generally available software, or some custom internal system? | 16:20 |
corvus | avass: so if you did a zuul http reporter, you'd probably need string formatting for the schema to address the schema issue that fungi raised, very similar to the uri module. | 16:20 |
fungi | just wondering if it would make sense to add a specific reporter connection driver for it | 16:20 |
corvus | or that | 16:21 |
avass | custom internal system, that's why we don't want to write a specific reporter for it | 16:21 |
avass | since I have a lot of bad experience trusting other groups to not break their api | 16:21 |
corvus | avass: do you need to report the entire buildset, or individual jobs? | 16:22 |
fungi | sounds like they could benefit from a project gating system ;) | 16:22 |
corvus | avass: (i'm wondering if calling the uri module from a cleanup playbook in individual jobs would work) | 16:23 |
avass | fungi: we are trying to push that for everyone :) | 16:23 |
corvus | avass: (in opendev, we do something similar where we submit logstash processing queue entries for each job on completion) | 16:24 |
avass | coruvs: that was an idea as well, but we just want to log the result of each pipeline | 16:24 |
avass | corvus: since it's another vizualisation tool we don't want to use, but sort of have to use. so we want to maintain it as little as possible | 16:25 |
corvus | avass: then yeah, i think either writing a new generic http reporter might be an option, or you could write a small python script to transfer mqtt messages to the http endpoint. | 16:25 |
avass | corvus: yeah that was my next idea | 16:26 |
avass | or at least something like it | 16:27 |
corvus | we probably don't want to try to do that in a job, because there's no way we can guarantee a job really knows the pipeline result | 16:28 |
*** rpittau is now known as rpittau|afk | 16:29 | |
avass | well, unless we can have a job that runs at the end no matter what the result of the other jobs are :) | 16:33 |
corvus | avass: yeah, but we want to keep pipeline result processing fast, and that could slow it down a lot | 16:33 |
*** evrardjp has quit IRC | 16:35 | |
*** evrardjp has joined #zuul | 16:35 | |
fungi | also if you do decide to go the mqtt route and there's data it's not exposing, adding more there is probably easy as long as it's part of or easily derived from the buildset result | 16:37 |
avass | I think having an mqtt service could solve a couple of other problems for us anyway | 16:37 |
avass | fungi: cool | 16:38 |
fungi | avass: we have some other tooling around mqtt too if you decide that's something you're interested in | 16:38 |
fungi | like lightweight daemons to translate the gerrit ssh event stream into mqtt, ingest stuff via smtp and turn out mqtt events, et cetera | 16:39 |
fungi | we==opendev i mean | 16:39 |
*** dpawlik has quit IRC | 16:40 | |
fungi | some of us in opendev experimented with running a central mqtt broker for a variety of our services, though nobody ever found much use for it | 16:40 |
clarkb | it is still running though | 16:40 |
*** dpawlik has joined #zuul | 16:40 | |
fungi | https://docs.opendev.org/opendev/system-config/latest/firehose.html | 16:41 |
fungi | well, at least the server still exists, i haven't tried to sub to it with an mqtt client in a while so no idea if the service is actually working or has fallen over | 16:41 |
avass | fungi: cool, but probably not what we need :) | 16:42 |
avass | at least at the moment | 16:42 |
fungi | yeah, turns out we didn't need it either, apparently ;) | 16:42 |
*** dpawlik has quit IRC | 16:45 | |
*** yolanda has quit IRC | 16:54 | |
*** jcapitao has quit IRC | 17:02 | |
*** jpena is now known as jpena|off | 17:12 | |
openstackgerrit | Merged zuul/zuul master: Improve display of stdout/stderr task errors https://review.opendev.org/681532 | 17:42 |
mordred | zbr, tristanC: fwiw - I have *no*idea* what's going on with the "update react" patch - I'm kind of out of ideas at the moment. mostly letting y'all know that I'm not actively trying to fix it currently. I think it woudl be good to get it fixed and landed, because the longer we wait the more likely it is to get harder to fix, I'd guess | 17:50 |
mordred | I'm sure there's just something silly somewhere - but for the life of me I can't figure it out | 17:50 |
*** sugaar has quit IRC | 17:52 | |
*** cdearborn has joined #zuul | 17:55 | |
*** sshnaidm is now known as sshnaidm|afk | 18:04 | |
avass | mordred: could it be something similar to this? https://github.com/reduxjs/react-redux/issues/1279 | 18:12 |
*** sugaar has joined #zuul | 18:16 | |
avass | (can't say I know much about react though) | 18:17 |
openstackgerrit | Monty Taylor proposed zuul/nodepool master: Actually install extras from nodepool_base https://review.opendev.org/722135 | 20:06 |
corvus | mordred: remember that because we're crossing tenants, depends-on won't be honored | 20:07 |
corvus | (so that's not a fully tested series) | 20:08 |
corvus | tristanC: do we expect podman to work on centos7? https://zuul.opendev.org/t/openstack/build/118d513c8a764ade8f1307f8483f0a3c/log/logs/containerfile_bionic-build-succeeds.FAIL.log#133-136 | 20:11 |
clarkb | mordred: corvus I've approved https://review.opendev.org/#/c/722125/1 so maybe just recheck 722135 once that lands? | 20:13 |
corvus | ++ | 20:13 |
*** rlandy is now known as rlandy|brb | 21:23 | |
*** tobiash has quit IRC | 21:45 | |
*** tobiash has joined #zuul | 21:47 | |
*** panda|ruck has quit IRC | 21:47 | |
*** panda has joined #zuul | 21:50 | |
clarkb | https://review.opendev.org/722135 has been rechecked fwiw (it failed py-38 anyway) | 21:54 |
*** armstrongs has joined #zuul | 22:14 | |
avass | mordred: I think I found it | 22:15 |
avass | Provider was update to a functional component in 7.1.1: https://github.com/reduxjs/react-redux/releases/tag/v7.1.1 | 22:15 |
avass | mordred: which causes TestUtils.renderIntoDocument return null: https://github.com/facebook/react/issues/15370 | 22:16 |
avass | mordred: it works with react-redux 7.1.0 | 22:23 |
openstackgerrit | Albin Vass proposed zuul/zuul master: Update to create-react-app 3.4.1 https://review.opendev.org/716305 | 22:23 |
*** armstrongs has quit IRC | 22:24 | |
avass | gotta go, I hope that works because it fixed in on my local machine at least :) | 22:24 |
mordred | avass: awesome -thanks! | 23:26 |
mordred | avass: still broken, sadly. if you look at the job results from zuul-build-dashboard and then click site preview - you'll see if fails to load. this is because we have config errors in the openstack tenant, which that is pointed at. _why_ those are broken is beyond me | 23:28 |
mordred | it's still on my list one day - but if you continue to be interested ... that's the next issue :) | 23:29 |
*** tosky has quit IRC | 23:51 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!