spotz | Who knew it was harder to say nothing:( | 00:00 |
---|---|---|
*** frenzyfriday has joined #zuul | 00:14 | |
*** frenzyfriday has quit IRC | 00:19 | |
openstackgerrit | Merged zuul/zuul master: GitHub Reporter: Fix User Email in Merge Commit Message https://review.opendev.org/741157 | 01:13 |
*** rlandy has quit IRC | 01:57 | |
*** sanjayu_ has joined #zuul | 02:38 | |
openstackgerrit | Amy Marrich (spotz) proposed zuul/zuul master: Add content to User Tutorials page https://review.opendev.org/748555 | 03:15 |
*** bhavikdbavishi has joined #zuul | 03:44 | |
*** bhavikdbavishi1 has joined #zuul | 03:54 | |
*** bhavikdbavishi has quit IRC | 03:56 | |
*** bhavikdbavishi1 is now known as bhavikdbavishi | 03:56 | |
*** bhavikdbavishi1 has joined #zuul | 04:13 | |
*** bhavikdbavishi has quit IRC | 04:15 | |
*** bhavikdbavishi1 is now known as bhavikdbavishi | 04:15 | |
*** bhavikdbavishi has quit IRC | 04:29 | |
*** bhavikdbavishi has joined #zuul | 04:30 | |
*** evrardjp has quit IRC | 04:33 | |
*** evrardjp has joined #zuul | 04:33 | |
*** frenzyfriday has joined #zuul | 04:34 | |
*** frenzyfriday has quit IRC | 04:38 | |
*** vishalmanchanda has joined #zuul | 05:01 | |
*** frenzyfriday has joined #zuul | 05:24 | |
*** frenzyfriday has quit IRC | 05:33 | |
*** sanjayu__ has joined #zuul | 05:50 | |
*** sanjayu_ has quit IRC | 05:53 | |
*** sanjayu_ has joined #zuul | 06:00 | |
*** sanjayu__ has quit IRC | 06:02 | |
*** frenzyfriday has joined #zuul | 06:23 | |
*** bhavikdbavishi1 has joined #zuul | 06:25 | |
*** bhavikdbavishi has quit IRC | 06:27 | |
*** bhavikdbavishi1 is now known as bhavikdbavishi | 06:27 | |
*** frenzyfriday has quit IRC | 06:27 | |
*** frenzyfriday has joined #zuul | 06:30 | |
*** bhavikdbavishi has quit IRC | 06:46 | |
openstackgerrit | Merged zuul/zuul master: web: restore scrollbars and scroll behaviour https://review.opendev.org/742759 | 07:05 |
*** zbr has quit IRC | 07:23 | |
*** zbr has joined #zuul | 07:24 | |
*** tosky has joined #zuul | 07:35 | |
*** bhavikdbavishi has joined #zuul | 07:38 | |
*** hashar has joined #zuul | 08:03 | |
*** vishalmanchanda has quit IRC | 09:27 | |
*** rlandy has joined #zuul | 11:18 | |
openstackgerrit | Sorin Sbarnea (zbr) proposed zuul/zuul-jobs master: Partial address ansible-lint E208 https://review.opendev.org/748480 | 11:49 |
openstackgerrit | Sorin Sbarnea (zbr) proposed zuul/zuul-jobs master: WIP: E208 work https://review.opendev.org/748606 | 11:50 |
openstackgerrit | Sorin Sbarnea (zbr) proposed zuul/zuul-jobs master: Partial address ansible-lint E208 https://review.opendev.org/748480 | 12:46 |
*** Goneri has joined #zuul | 12:49 | |
openstackgerrit | Sorin Sbarnea (zbr) proposed zuul/zuul-jobs master: Partial address ansible-lint E208 https://review.opendev.org/748480 | 13:12 |
*** mach1na has joined #zuul | 13:14 | |
*** mach1na has quit IRC | 13:17 | |
openstackgerrit | Sorin Sbarnea (zbr) proposed zuul/zuul-jobs master: WIP: E208 work https://review.opendev.org/748606 | 13:26 |
*** bhavikdbavishi has quit IRC | 13:26 | |
*** benoit_lbc has joined #zuul | 13:27 | |
*** benoit_lbc has quit IRC | 13:37 | |
*** vishalmanchanda has joined #zuul | 13:38 | |
zbr | apparently tox_install_sibling_packages is broken on py27 due to missing pkg_resources .... | 13:39 |
zbr | https://ede78553fdedfba4a90b-bf4df36ee9e72417a89c120068385812.ssl.cf5.rackcdn.com/748606/2/check/zuul-jobs-test-fetch-sphinx-tarball-debian-stretch/5b7ed9c/job-output.txt | 13:39 |
zbr | what are the plans on fixing this because several testing jobs in zuul-jobs are broken due to it | 13:41 |
zbr | another case of stuff that gets broken and nobody knows because we have no periodic jobs | 13:42 |
zbr | https://zuul.opendev.org/t/zuul/builds?job_name=zuul-jobs-test-fetch-sphinx-tarball-debian-stretch | 13:42 |
*** _erlon_ has joined #zuul | 14:42 | |
openstackgerrit | Denis proposed zuul/zuul-jobs master: [terraform role] Add option to set the color arg on terraform commands https://review.opendev.org/748679 | 14:46 |
openstackgerrit | Sorin Sbarnea (zbr) proposed zuul/zuul-jobs master: Add managed jobs to periodic https://review.opendev.org/748682 | 14:47 |
corvus | zbr: are you going to volunteer to monitor and fix these jobs when they break? | 14:48 |
corvus | zbr: or do you know of someone who will? | 14:48 |
zbr | corvus: I was planning to offer to help on that. | 14:49 |
corvus | zbr: okay, sounds great, thanks | 14:49 |
corvus | zbr: i just wanted to make sure we had both ends covered :) | 14:49 |
zbr | corvus: is there a easy way to add email for failures on these? | 14:50 |
zbr | or we need a new pipeline special for this? | 14:50 |
corvus | zbr: we can set up a mailing list for that and update the periodic pipeline for the zuul tenant to send to it | 14:50 |
corvus | zbr: note that the periodic pipeline is daily, not weekly | 14:51 |
corvus | zbr: you can make the new list with a change to opendev/system-config; and update the pipeline config with a change to zuul/project-config | 14:51 |
zbr | corvus: maybe weekly is too much load? | 14:51 |
* zbr doing it.... | 14:52 | |
zbr | does "zuul-job-admin" sounds ok or we need other name? we could use it for other purposes than spam. | 14:54 |
corvus | zbr: i'd go with something just for this purpose, like zuul-job-reports or something | 14:55 |
corvus | zuul-jobs-reports | 14:55 |
zbr | i will use zuul-jobs-failures to match exiting pattern | 14:56 |
zbr | so i am going to be the first... failure. | 14:56 |
*** hashar has quit IRC | 15:01 | |
zbr | corvus: should I create a new pipeline periodic-zuul-jobs? the existing on periodic-weekly runs lots of tripleo jobs and would polute the mailing list. | 15:25 |
corvus | zbr: i don't think there are any tripleo jobs in the zuul tenant? | 15:26 |
zbr | not on zuul tenant | 15:26 |
corvus | then there's no problem | 15:27 |
*** frenzyfriday has quit IRC | 15:32 | |
*** rlandy is now known as rlandy|brb | 15:34 | |
*** frenzyfriday has joined #zuul | 15:38 | |
*** frenzyfriday has quit IRC | 15:42 | |
*** rlandy|brb is now known as rlandy | 15:50 | |
openstackgerrit | Sorin Sbarnea (zbr) proposed zuul/project-config master: Assure periodic-weekly emails to zuul-jobs-failures https://review.opendev.org/748706 | 15:51 |
*** bhavikdbavishi has joined #zuul | 15:56 | |
openstackgerrit | Sorin Sbarnea (zbr) proposed zuul/zuul-jobs master: More E208 mode fixes https://review.opendev.org/748498 | 16:12 |
*** frenzyfriday has joined #zuul | 16:15 | |
*** frenzyfriday has quit IRC | 16:19 | |
*** armstrongs has joined #zuul | 16:26 | |
zbr | corvus: clarkb: how should we deal with https://opendev.org/opendev/base-jobs/src/branch/master/test-requirements.txt#L14-L16 which is clear conflict | 16:30 |
zbr | the linter requires minimal 2.8, i am surprised it did not choke on that | 16:30 |
zbr | we also need to add a "pip check" as first command to avoid introducing conflicts | 16:30 |
zbr | the linter is recommended to run with newest ansible as this can help it uncover more problems | 16:31 |
openstackgerrit | Timo Lotterbach proposed zuul/zuul master: Fix typo in documentation https://review.opendev.org/748716 | 16:33 |
clarkb | zbr: I think the idea is to catch problems that older ansible can't support | 16:33 |
*** armstrongs has quit IRC | 16:35 | |
clarkb | bceause each ansible release tends to add new features/flags/options and if we use those older ansible will break | 16:36 |
zbr | in that case you may want to write your own linter, as I doubt you would find many interested into writing code at the end of 2020, that tests bugs from versions of ansible that are no longer supported: https://docs.ansible.com/ansible/devel/reference_appendices/release_and_maintenance.html | 16:36 |
clarkb | I do think we can bump that up to ansible 2.6 now though as zuul's lower bound has gone up tothat | 16:37 |
clarkb | its less a question of what ansible is suported and more what ansible will people use to run these roles and playbooks | 16:37 |
zbr | even 2.7 will be unsupported un less than a month, if it got the dates correctly | 16:37 |
clarkb | and for zuul currently I believe the answer is 2.6 - 2.9 | 16:38 |
* clarkb double checks zuul's ansible list | 16:38 | |
zbr | the reality is that the linter does not give you much indication if ansible will choke or not with the code, only testing will tell you this | 16:38 |
clarkb | no 2.6 just 2.7 - 2.9 | 16:39 |
clarkb | so we can bump that all the way up to 2.7 | 16:39 |
zbr | and unless you test with minimal and maximal, you do not get any useful info. | 16:39 |
clarkb | why is maximal also required to get useful info? | 16:39 |
zbr | because it tells you if your code is already broken or not | 16:39 |
clarkb | if you test with minimal only it tells you if you work on the old stuff, ansible has gotten pretty good at not constantly breaking old stuff but it does add a lot of new stuff constantly | 16:40 |
clarkb | that means the bigger risk is in using new features that break older ansible | 16:40 |
clarkb | not old ansible breaking under newer releaess | 16:40 |
zbr | clarkb: hehe, wait for 2.10 release,.... | 16:40 |
clarkb | risk exists in both directions BUT you get information for each depending on whether you test min or max | 16:41 |
zbr | we seen what the CVE security fix did :D | 16:41 |
clarkb | you don't need to test both to get useful info | 16:41 |
zbr | clarkb: in fact now that we introduce periodic, we may have a way to assert risk on that and run both if we want. | 16:42 |
zbr | while avoiding to test this on normal patches. | 16:42 |
zbr | i am not saying we should do it now, but it would be interesting to try it once and see the difference | 16:42 |
clarkb | corvus: would now be a good time to restart zuul-web on zuul.opendev.org? Its been running since late july so will pick up ianw's scrolling fix and whatever else was in the delta | 16:43 |
clarkb | I'm happy to do that and watch it, but just not sure if there are any other concerns in that delta if we do that | 16:43 |
* zbr feeling quite exited to see the zuul UI changes up | 16:47 | |
*** SotK has quit IRC | 17:03 | |
*** tosky has quit IRC | 17:03 | |
*** SotK has joined #zuul | 17:03 | |
*** tosky has joined #zuul | 17:04 | |
zbr | does anyone really wants the parseable format from the ansible-lint? asking because the default format recently become more useful, also including recomandations as the end, but these are not visible in parseable mode. | 17:11 |
clarkb | I think the parseable format is used so that zuul can parse them and leave inline comments in gerrit? | 17:13 |
zbr | clarkb: regarding this, few weeks ago I made the linter detect when is run under github-actions and produce annotation output automatically, described at https://ansible-lint.readthedocs.io/en/latest/usage.html#ci-cd | 17:16 |
zbr | maybe we can do something similar for zuul too, to make the integration easier | 17:16 |
zbr | i was considering writing a filter that recognize the annotation format, which is superior to the old pep8 format, and very easy to produce. | 17:17 |
zbr | that nice part about this is that it does not change the output, it only adds extra output when the CI is recognized. | 17:18 |
zbr | it has the advantage that you can keep friendly console output, while still producing annotations when run on ci. | 17:19 |
zbr | still, afaik, gerrit cannot produce annotations for files outside the current change, something that is not uncommon. | 17:20 |
*** hamalq has joined #zuul | 17:21 | |
clarkb | I've not tested that so not sure. I know it can outside of the current diff contenxt within a file though | 17:24 |
*** frenzyfriday has joined #zuul | 17:34 | |
*** frenzyfriday has quit IRC | 17:39 | |
*** tosky has quit IRC | 17:44 | |
clarkb | corvus: if you're arround any objections to me restarting zuul-web on opendev's zuul nowish? that will pick up about a months worth of js updates | 17:48 |
corvus | clarkb: ++ | 17:51 |
clarkb | ok starting that momentarily | 17:52 |
clarkb | done new processes should be running | 17:53 |
clarkb | status dashboard is working and my spacebar works there | 17:54 |
clarkb | the build pages are quite different | 17:54 |
clarkb | my spacebar works in job log pages now too \o/ | 17:55 |
corvus | \o/ | 17:55 |
clarkb | one suggestion is that the build tabs are all navigable without losing the build "header" until you open a log file | 17:55 |
clarkb | might be nice to keep the log file browsing under that "header" too | 17:56 |
clarkb | the jump to log line number problem persists | 17:56 |
clarkb | overall seems fine (no regressions that I see and keyboard nav works) | 17:57 |
zbr | Pipeline descriptions looks bad at https://zuul.opendev.org/t/openstack/status | 17:59 |
clarkb | zbr: they were made bold, i'm not sure I would say they look bad | 17:59 |
zbr | not a deal breaker, but I would prefer to have them as tooltips | 17:59 |
clarkb | I think people overlook tooltips | 18:00 |
zbr | bad = taking precious screen real estate. | 18:00 |
clarkb | In this case I think those descriptions are important (they are user configurable, not zuul built ins) | 18:01 |
zbr | maybe we could add an (i) icon near the title that displays the description? | 18:01 |
clarkb | without them a lot of people wouldn't understand the idfference between the different pipelines | 18:01 |
corvus | i think maybe that was a mistake and we should have bolded the pipeline names | 18:02 |
zbr | I could easily "improve" the description of the check or gate pipeline and render this webpage unusable ;) | 18:02 |
clarkb | corvus: ya I could go either way on the bolding. But don't want to take the info off the page | 18:02 |
clarkb | zbr: you'd need to get it through review first | 18:02 |
zbr | maybe we need a "title" that is expandable, something in between pipeline "id" and the full description. | 18:03 |
zbr | i totally agree that we should popularize pipeline descriptions, but better to do it nicely. | 18:04 |
zbr | i would personally make the pipline color blue, to invite people to hover over it, and display the description as a tooltip, native or react-base (if we want links, maybe formatting) | 18:05 |
zbr | Obviously that I could also propose adding a custom css field into preferences where hackers like me could tune the UI the way they like ;) | 18:07 |
zbr | the real issue that see is that now, 50% of the pipelines from the 3rd column, are outside the visible screen, only due to descriptions | 18:08 |
zbr | everything after periodic-weekly is outside the first screen, requiring scroll | 18:09 |
clarkb | it depends on the number of changes too | 18:09 |
corvus | avoiding scrolling is not a goal for the status page | 18:09 |
clarkb | if things are merging then post pushes everything down | 18:09 |
corvus | the pipelines are intentionally ordered, so later ones are deemed less important. it's okay to scroll to them. | 18:11 |
zbr | can we create the mailing list and the new periodic pipeline? | 18:14 |
zbr | https://review.opendev.org/#/q/topic:zuul-jobs-failures+(status:open+OR+status:merged) | 18:14 |
zbr | it would be quite useful to catch results from this weekend | 18:14 |
*** frenzyfriday has joined #zuul | 18:15 | |
corvus | i don't think there's a rush; you can trigger those jobs with a dnm change to get results now | 18:15 |
zbr | i know for sure that at least some of them are broken | 18:16 |
zbr | take a look at https://review.opendev.org/#/c/748606/ -- failures are unrelated to the change | 18:17 |
*** bhavikdbavishi has quit IRC | 18:18 | |
*** frenzyfriday has quit IRC | 18:19 | |
clarkb | https://zuul.opendev.org/t/zuul/build/bbb51e76b3074486b5da2283838b57e7 is continued fallout from the local code restrictions on the executor | 18:20 |
clarkb | I'm not sure why pkg_resources is a problem on centos 7. Maybe we need to add ensure-pip to the job (that pulls in pip and friends) | 18:21 |
zbr | if i remember well, pkg_resources was not always distributed with setuptools, so older versions.... | 18:24 |
openstackgerrit | James E. Blair proposed zuul/zuul master: Bold pipeline names not descriptions https://review.opendev.org/748733 | 18:25 |
corvus | yeah, https://review.opendev.org/747152 simply did not do what the commit message said it did | 18:25 |
corvus | PS1 worked as advertised, PS2 did not | 18:25 |
*** sanjayu_ has quit IRC | 18:28 | |
openstackgerrit | Sorin Sbarnea (zbr) proposed zuul/project-config master: Assure periodic-weekly emails to zuul-jobs-failures https://review.opendev.org/748706 | 18:34 |
zbr | https://stackoverflow.com/questions/7446187/no-module-named-pkg-resources | 18:37 |
zbr | that explain why it fails and I am not sure who is going to fix it, as upgrading setuptools is not possible | 18:38 |
clarkb | zbr: I think ensure-pip fixes it | 18:41 |
zbr | it would break it if it does upgrade it as it would install an unsupported version | 18:41 |
zbr | but this issue reminds me of nodejs issues, the only import seems to be to_filename | 18:42 |
zbr | https://github.com/pypa/setuptools/blob/213baae85f0b9c2640fdf7bf270f6b210f119c54/setuptools/_distutils/command/install_egg_info.py#L72 | 18:42 |
clarkb | the problem is it isnt installed at all iirc | 18:45 |
clarkb | and ensure-pip installs it? something like that | 18:45 |
openstackgerrit | Sorin Sbarnea (zbr) proposed zuul/zuul-jobs master: Remove dependency on pkg_resources https://review.opendev.org/748737 | 18:45 |
zbr | do we really need an import to do .replace('-','_') ?! | 18:45 |
clarkb | the rules for normalization are more complicated than that | 18:48 |
clarkb | but also I think that us ansible doing that? | 18:49 |
clarkb | I dunno need to pop out niw | 18:49 |
zbr | they are but that is the method used | 18:57 |
zbr | the fact that his module change does not trigger right jobs another issue | 18:58 |
openstackgerrit | Sorin Sbarnea (zbr) proposed zuul/zuul-jobs master: Remove dependency on pkg_resources https://review.opendev.org/748737 | 19:00 |
* zbr need to go too, is already 8pm here. Have a nice weekend! | 19:01 | |
*** vishalmanchanda has quit IRC | 21:25 | |
*** rlandy has quit IRC | 21:30 | |
*** tosky has joined #zuul | 21:33 | |
*** frenzyfriday has joined #zuul | 21:58 | |
*** frenzyfriday has quit IRC | 22:02 | |
-openstackstatus- NOTICE: A zuul server ended up with read only filesystems which caused many jobs to hit retry_limit. The server has been rebooted and appears happy. Jobs can be rechecked. | 22:15 | |
*** tosky has quit IRC | 23:13 | |
*** hamalq has quit IRC | 23:28 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!