*** jamesmcarthur has quit IRC | 00:04 | |
*** sugaar has quit IRC | 00:10 | |
*** jamesmcarthur has joined #zuul | 00:30 | |
*** jamesmcarthur has quit IRC | 00:43 | |
*** rfolco has joined #zuul | 00:52 | |
*** jamesmcarthur has joined #zuul | 00:59 | |
*** jamesmcarthur has quit IRC | 01:05 | |
*** jamesmcarthur has joined #zuul | 01:05 | |
*** rfolco has quit IRC | 01:45 | |
*** jamesmcarthur has quit IRC | 01:47 | |
*** jamesmcarthur has joined #zuul | 01:49 | |
*** jamesmcarthur has quit IRC | 01:54 | |
*** openstackgerrit has quit IRC | 02:04 | |
*** openstackgerrit has joined #zuul | 02:15 | |
openstackgerrit | Monty Taylor proposed zuul/zuul-jobs master: Add support for installing python with pyenv https://review.opendev.org/704266 | 02:15 |
---|---|---|
*** jamesmcarthur has joined #zuul | 02:18 | |
*** jamesmcarthur has quit IRC | 02:24 | |
*** jamesmcarthur has joined #zuul | 02:52 | |
*** jamesmcarthur has quit IRC | 02:56 | |
*** bhavikdbavishi has joined #zuul | 03:27 | |
*** jamesmcarthur has joined #zuul | 03:31 | |
*** jamesmcarthur has quit IRC | 03:32 | |
*** jamesmcarthur has joined #zuul | 03:32 | |
*** bhavikdbavishi1 has joined #zuul | 03:36 | |
*** bhavikdbavishi has quit IRC | 03:38 | |
*** bhavikdbavishi1 is now known as bhavikdbavishi | 03:38 | |
*** jhesketh has quit IRC | 03:56 | |
*** jamesmcarthur has quit IRC | 04:44 | |
*** raukadah is now known as chkumar|rover | 04:45 | |
*** zxiiro has joined #zuul | 05:51 | |
*** AJaeger has quit IRC | 07:05 | |
*** AJaeger has joined #zuul | 07:10 | |
*** bhavikdbavishi has quit IRC | 07:31 | |
*** tosky has joined #zuul | 07:56 | |
*** bhavikdbavishi has joined #zuul | 08:08 | |
*** avass has joined #zuul | 08:24 | |
*** pcaruana has joined #zuul | 08:25 | |
*** sugaar has joined #zuul | 08:36 | |
*** jpena|off is now known as jpena | 08:51 | |
openstackgerrit | Tobias Henkel proposed zuul/zuul master: Optionally support mitogen for job execution https://review.opendev.org/657024 | 08:55 |
*** avass has quit IRC | 09:12 | |
openstackgerrit | Jan Kubovy proposed zuul/zuul master: Enforce sql connections for scheduler and web https://review.opendev.org/630472 | 09:20 |
*** fbo is now known as fbo|off | 09:39 | |
*** bhavikdbavishi has quit IRC | 09:58 | |
*** saneax has joined #zuul | 09:59 | |
*** sshnaidm is now known as sshnaidm|afk | 10:27 | |
*** sanjayu_ has joined #zuul | 10:32 | |
*** saneax has quit IRC | 10:34 | |
reiterative | I'm trying to run the latest version of Zuul from master, but when I browse to the web interface I get a 404. I installed via 'sudo python3 -m pip install .' from a clone of the repo, having checked out the 3.15.0 label. When I install the same version from PyPI ( sudo python3 -m pip install zuul), everything works just fine. Is there something different i need to do when installing in this way? | 10:37 |
*** adamw has joined #zuul | 10:40 | |
*** avass has joined #zuul | 10:48 | |
*** etp has quit IRC | 10:58 | |
*** etp has joined #zuul | 11:00 | |
*** saneax has joined #zuul | 11:06 | |
*** sanjayu_ has quit IRC | 11:07 | |
*** sshnaidm|afk is now known as sshnaidm | 11:34 | |
*** zxiiro has quit IRC | 11:50 | |
*** rfolco has joined #zuul | 12:00 | |
mordred | reiterative: if you're installing from git, you'll need the javascript toolchain installed. the wheels we publish to pypi have the javascript things pre-built | 12:13 |
mordred | reiterative: https://zuul-ci.org/docs/zuul/reference/developer/javascript.html#yarn-dependency-management has some docs on installing those | 12:14 |
mordred | (really just the install-js-tools.sh script should get you set up - it's yarn you need - yarn will install the other stuff needed) | 12:15 |
openstackgerrit | Tristan Cacqueray proposed zuul/zuul-jobs master: Add a markdownlint job and role https://review.opendev.org/607691 | 12:23 |
reiterative | Thanks mordred! I thought I'd installed everything specified in bindep.txt (which includes yarn and nodejs-npm), but bindep fails to mention that those are missing... | 12:25 |
mordred | reiterative: hrm. | 12:25 |
mordred | reiterative: what platform are you on? I wonder if we've got a bad tag in our bindep file ... | 12:26 |
reiterative | I'm working on Ubuntu 18.04 | 12:26 |
lennyb | Hi, why openstack-python3-ussuri-jobs-neutron suppose runs openstack-tox-py27 #link https://review.opendev.org/#/c/704269/8? According to #link https://docs.openstack.org/infra/openstack-zuul-jobs/project-templates.html#project_template-openstack-python3-ussuri-jobs-neutron it should not to | 12:26 |
mordred | oh - yeah - so - the issue there ... | 12:26 |
mordred | reiterative: is that bindep only lists them for alpine because it's the only ones with them built in. for ubuntu and red hat distros you need to add package repos, and we don't have any way to express that in bindep sadly | 12:27 |
*** jpena is now known as jpena|lunch | 12:28 | |
* mordred keeps trying to brainstorm some way to do it that isn't horrific - but hasn't yet been able to | 12:28 | |
reiterative | mordred Understood! This sort of mult-platform dependency problem is a familiar one :-) | 12:29 |
mordred | :) ... you'd think this would be a solved problem by now ... | 12:30 |
AJaeger | lennyb: where are your jobs defined? | 12:36 |
lennyb | AJaeger: https://review.opendev.org/#/c/704269/8/.zuul.yaml | 12:37 |
AJaeger | lennyb: and here as well: https://opendev.org/openstack/project-config/src/branch/master/zuul.d/projects.yaml#L5886 | 12:38 |
AJaeger | Send a change to remove the project stanza from project-config and you'll be fine.. | 12:38 |
lennyb | AJaeger: thanks a lot. | 12:38 |
lennyb | AJaeger: Will it be enough to define it in my x/ repo only, or I will need to update https://opendev.org/openstack/project-config/src/branch/master/zuul.d/projects.yaml#L5886 each time ? | 12:40 |
*** bhavikdbavishi has joined #zuul | 12:42 | |
*** bhavikdbavishi1 has joined #zuul | 12:45 | |
*** bhavikdbavishi has quit IRC | 12:47 | |
*** bhavikdbavishi1 is now known as bhavikdbavishi | 12:47 | |
*** rlandy has joined #zuul | 12:50 | |
*** Goneri has quit IRC | 12:53 | |
*** jamesmcarthur has joined #zuul | 13:06 | |
*** jamesmcarthur has quit IRC | 13:21 | |
*** bhavikdbavishi has quit IRC | 13:24 | |
*** jamesmcarthur has joined #zuul | 13:27 | |
openstackgerrit | Jan Kubovy proposed zuul/zuul master: Enforce sql connections for scheduler and web https://review.opendev.org/630472 | 13:28 |
*** jpena|lunch is now known as jpena | 13:31 | |
*** jamesmcarthur has quit IRC | 13:37 | |
*** Goneri has joined #zuul | 13:44 | |
*** jamesmcarthur has joined #zuul | 13:49 | |
openstackgerrit | Jan Kubovy proposed zuul/zuul master: Enforce sql connections for scheduler and web https://review.opendev.org/630472 | 13:50 |
*** smcginnis has quit IRC | 13:50 | |
*** smcginnis has joined #zuul | 13:52 | |
*** zxiiro has joined #zuul | 13:52 | |
*** smcginnis is now known as smcginnis|FOSDEM | 13:53 | |
*** avass has quit IRC | 14:04 | |
*** bolg has joined #zuul | 14:12 | |
bolg | corvus: Updated spec for scale out scheduler (https://review.opendev.org/c/621479) and would like to proceed with required sql connections for scheduler and web (https://review.opendev.org/c/630472) how would be best to proceed? | 14:15 |
bolg | Are there any points which should be considered in the spec? | 14:16 |
openstackgerrit | Tobias Henkel proposed zuul/zuul master: Don't reset file list to None in updateChange https://review.opendev.org/704328 | 14:20 |
openstackgerrit | Merged zuul/nodepool master: kubernetes: add release note about service account token issue https://review.opendev.org/703869 | 14:27 |
openstackgerrit | Simon Westphahl proposed zuul/nodepool master: Annotate logs in launcher https://review.opendev.org/703558 | 14:29 |
openstackgerrit | Simon Westphahl proposed zuul/nodepool master: Annotate logs in node request handler https://review.opendev.org/703559 | 14:29 |
openstackgerrit | Simon Westphahl proposed zuul/nodepool master: Include event id in node request listings https://review.opendev.org/703560 | 14:29 |
openstackgerrit | Simon Westphahl proposed zuul/nodepool master: Annotate logs in zk module https://review.opendev.org/703561 | 14:29 |
swest | Shrews: sorry for the delayed response. eliminated the duplicate data in the log messages ^ | 14:32 |
*** hashar has joined #zuul | 14:38 | |
openstackgerrit | Antoine Musso proposed zuul/zuul master: Enhance some logging messages https://review.opendev.org/704331 | 14:39 |
openstackgerrit | Tobias Henkel proposed zuul/zuul master: Evaluate CODEOWNERS settings during canMerge check https://review.opendev.org/644557 | 14:39 |
AJaeger | lennyb: your proposed change is just fine, let's continue discussing there... | 14:41 |
lennyb | AJaeger: I feel that my answers and your proposals are the same :). looks like I am missing small words, that make all the difference when I read :). Sorry for that | 14:52 |
openstackgerrit | Tobias Henkel proposed zuul/zuul master: Handle draft pull requests in canMerge https://review.opendev.org/704150 | 14:55 |
openstackgerrit | Tobias Henkel proposed zuul/zuul master: Evaluate CODEOWNERS settings during canMerge check https://review.opendev.org/644557 | 14:55 |
*** paladox is now known as paladox|UKInEU | 14:56 | |
pabelanger | zbr: a new role could be added for it, we generate html before uploading to swift. Look to htmlify-logs role for example | 14:58 |
AJaeger | lennyb: yeah, all looks fine. What I'm saying is that you need to add to stable/train (and other branches) the python-jobs template. | 14:58 |
zbr | pabelanger: my impression is that this view is generating at runtime by loading the json file. which means that ansi2html must happen in the browser. not a big deal as there are several libraries able to do that. | 14:59 |
pabelanger | zbr: ah, the dashboard might be different, was sharing on straight html file that is uploaded into logs folder | 15:00 |
zbr | my first worry is the dashboard, as it has two problems: the tail is too short, even on linting errors often you do not see the error, and is messed due to lack of ANSI processing. | 15:01 |
zbr | both of them fixable, i would at least double the tail lenght. | 15:02 |
fungi | or add a clickable control to expose another chunk of the log | 15:03 |
fungi | if it's too short some of the time but not most of the time, that would allow us to keep a shorter and more focused log snippet by default | 15:04 |
fungi | if it's too short most of the time, then yeah maybe we should change the number of lines we display by default | 15:04 |
corvus | bolg: i'll review the spec asap | 15:11 |
*** jamesmcarthur has quit IRC | 15:17 | |
*** hashar has quit IRC | 15:30 | |
*** bhavikdbavishi has joined #zuul | 15:38 | |
*** bolg has quit IRC | 16:01 | |
*** saneax has quit IRC | 16:10 | |
pabelanger | a heads up, zbr spend some time last week migrating molecule from travis CI to zuul.ansible.com. I'm happy to share this | 16:32 |
*** jamesmcarthur has joined #zuul | 16:51 | |
*** chkumar|rover is now known as raukadah | 16:54 | |
clarkb | hashar tobiash please see comment on https://review.opendev.org/#/c/704331/1 I think it may be too minor to worry about but wasn't super sure | 17:04 |
clarkb | corvus: re my comment on https://review.opendev.org/#/c/704222/1 you might have a process for editing those screenshots to highlight fields? | 17:04 |
tobiash | clarkb: good catch | 17:08 |
*** paladox|UKInEU is now known as paladox | 17:09 | |
openstackgerrit | Clark Boylan proposed zuul/zuul master: Be more specific with remaining time https://review.opendev.org/704191 | 17:13 |
*** hashar has joined #zuul | 17:18 | |
clarkb | AJaeger: pabelanger ^ thanks for the catch on that. using the constant as specified should fix it | 17:22 |
pabelanger | +2 looks better! | 17:26 |
pabelanger | thanks | 17:26 |
clarkb | tobiash: before I approve https://review.opendev.org/#/c/703651/15 do you know why True gets string converted the way it does in the test added there (comment has more details) | 17:44 |
Shrews | oops, i just +Ad that | 17:44 |
clarkb | Shrews: oh I think its fine | 17:44 |
clarkb | the tests pass and it must be owrking :) | 17:44 |
clarkb | its just not exactly how I would haev expected | 17:44 |
clarkb | also I enjoy that the ec2 api uses key: 'foo', value: 'bar' not realizing those are implicit in the datastructure | 17:45 |
tobiash | clarkb: I think that's magic done by the yaml parser | 17:46 |
tobiash | clarkb: scratch that, I misunderstood your question | 17:47 |
tobiash | clarkb: are you sure that this gets deserialized to a bool? | 17:48 |
clarkb | tobiash: now that you've asked that I'm not compeltely sure it isn't a pyyaml supplied bool type | 17:48 |
clarkb | possibly one that stringifies to match the yaml input | 17:48 |
clarkb | or perhaps zuul does that conversion somewhere internally | 17:49 |
clarkb | er nodepool | 17:49 |
tobiash | clarkb: hrm, local testing in a python console results in 'True' | 17:53 |
*** mattw4 has joined #zuul | 17:53 | |
fungi | clarkb: if python True is being fed to the yaml serializer/dumper then "true" is the default truthy boolean value for pyyaml (yaml as a whole presumably?) | 17:53 |
tobiash | clarkb: I wonder if that assert is actually executed | 17:53 |
clarkb | fungi: this is the other direction though | 17:53 |
tobiash | ah ok, that explains it | 17:53 |
fungi | it interprets "true" in yaml as the True boolean in python | 17:54 |
clarkb | fungi: ya then nodepool is serializing it back out again to str($yaml_input_true) | 17:54 |
clarkb | fungi: I would expect that to be 'True' not 'true' | 17:54 |
fungi | agreed | 17:55 |
tobiash | clarkb: just verified, safe_dump results in 'true' | 17:55 |
fungi | str(True) yields "True" in my python3.8 | 17:55 |
tobiash | which is the yaml preferred standard representation of a bool | 17:55 |
fungi | but if it's going through the pyyaml dumper then that will generate "true" instead | 17:55 |
tobiash | yaml prefers true over True (e.g. verified by yamllint) but accepts both | 17:56 |
fungi | right | 17:56 |
tobiash | actually yaml accepts a variety of values as bool, but has to decide to one when serializing: https://yaml.org/type/bool.html | 17:57 |
fungi | but according to clarkb this is generated by str() of the python value we deserialized? | 17:57 |
*** jpena is now known as jpena|off | 17:57 | |
clarkb | fungi: ya line 134 https://review.opendev.org/#/c/703651/15/nodepool/driver/aws/config.py | 17:58 |
tobiash | there are probably further deserialization/serialization steps in between which are done by boto3 | 17:59 |
fungi | so many layers | 18:00 |
clarkb | like onions and ogres | 18:00 |
fungi | ogrions | 18:01 |
openstackgerrit | Merged zuul/nodepool master: Annotate logs in launcher https://review.opendev.org/703558 | 18:05 |
openstackgerrit | Merged zuul/nodepool master: Annotate logs in node request handler https://review.opendev.org/703559 | 18:14 |
openstackgerrit | David Shrewsbury proposed zuul/nodepool master: Remove duplicate log info from OpenStack driver https://review.opendev.org/704379 | 18:15 |
Shrews | tobiash: swest: you two may be interested in 704379 ^^ | 18:15 |
Shrews | saw that while looking at test result logs | 18:15 |
Shrews | there was also at least 1 missing self.log() parameter bug | 18:16 |
Shrews | line 279 | 18:17 |
openstackgerrit | Antoine Musso proposed zuul/zuul master: Add details to waitUntilSettled() debug log https://review.opendev.org/704380 | 18:17 |
openstackgerrit | Antoine Musso proposed zuul/zuul master: tests: throttle after all builds have reported https://review.opendev.org/704381 | 18:17 |
*** bhavikdbavishi has quit IRC | 18:18 | |
openstackgerrit | Merged zuul/nodepool master: Include event id in node request listings https://review.opendev.org/703560 | 18:25 |
openstackgerrit | James E. Blair proposed zuul/zuul master: Update screenshot for gerrit profile in quick-start https://review.opendev.org/704222 | 18:28 |
corvus | clarkb: ^ thx, see comment | 18:31 |
corvus | tflink: ^ | 18:32 |
openstackgerrit | James E. Blair proposed zuul/zuul master: Fix path in quick-start localtest https://review.opendev.org/704385 | 18:32 |
*** sshnaidm is now known as sshnaidm|afk | 18:35 | |
openstackgerrit | Merged zuul/nodepool master: Annotate logs in zk module https://review.opendev.org/703561 | 18:37 |
*** notnone is now known as at_work | 18:39 | |
openstackgerrit | Jeremy Stanley proposed zuul/zuul master: Flesh out the glossary significantly https://review.opendev.org/704391 | 18:51 |
fungi | still in progress ^ but the entries i've added so far are fare game for critical feedback | 18:51 |
corvus | fungi: there's a special ref for terms in the glossary; after bulking it out, you might want to grep through the source and set up those links | 18:55 |
corvus | fungi: :term: | 18:56 |
corvus | like: tutorials/quick-start.rst::term:`check` and :term:`gate`. In our system, ``check`` will be | 18:56 |
fungi | corvus: awesome. and yeah, step 1 is to compose the prose and identify gaps | 18:59 |
fungi | the other step can hopefully be done semi-programmatically? | 19:00 |
fungi | i'll admit i really hadn't even looked closely at that mechanism yet | 19:00 |
corvus | i'd expect it to be grep + human editing | 19:04 |
corvus | i don't think we'd want to :term: every occurance -- just the first time it shows up in a section | 19:04 |
fungi | if it's a bunch, it may be grep/sed -i followed by human cleanup ;) | 19:04 |
fungi | thankfully it's not hard to tell the stream editor to bail after the first edit | 19:05 |
*** jamesmcarthur has quit IRC | 19:05 | |
*** jamesmcarthur has joined #zuul | 19:28 | |
openstackgerrit | Merged zuul/zuul master: Update Quick-Start Fedora Package Install Command https://review.opendev.org/704221 | 19:38 |
*** mattw4 has quit IRC | 19:49 | |
*** jamesmcarthur has quit IRC | 20:03 | |
tobiash | Shrews: thanks, lgtm | 20:13 |
*** jamesmcarthur has joined #zuul | 20:16 | |
openstackgerrit | Antoine Musso proposed zuul/zuul-jobs master: Skip test-setup.sh in pep8 jobs https://review.opendev.org/704414 | 20:17 |
*** rlandy is now known as rlandy|brb | 20:19 | |
openstackgerrit | Merged zuul/nodepool master: add tags support for aws provider https://review.opendev.org/703651 | 20:21 |
*** SotK has quit IRC | 20:25 | |
*** SotK has joined #zuul | 20:25 | |
tobiash | fungi: awesome, one additional important term would be project-pipeline | 20:28 |
fungi | tobiash: please add a review comment, i don't know how a project-pipeline differs from a normal pipeline | 20:28 |
*** openstackstatus has joined #zuul | 20:28 | |
*** ChanServ sets mode: +v openstackstatus | 20:28 | |
corvus | (it's the pipeline part of a project stanza) | 20:29 |
*** rlandy|brb is now known as rlandy | 20:29 | |
fungi | ahh, got it | 20:31 |
fungi | as opposed to the pipeline definitions in the tenant configuration | 20:31 |
openstackgerrit | Antoine Musso proposed zuul/zuul master: tox: rename pep8 to linters https://review.opendev.org/703635 | 20:34 |
openstackgerrit | Antoine Musso proposed zuul/zuul master: tox: do not install bindep for linters https://review.opendev.org/703636 | 20:36 |
fungi | for full disclosure, i phoned into the first "meeting" of the cd foundation's interoperability sig last week, and mentioned that before it made the least bit of sense to talk about interoperability between ci/cd software we needed a common language to be able to hold those discussions in, so i'm going to push the folks involved in the sig to try and work out relevant terminology for the tools they represent. in | 20:36 |
fungi | the same vein, i figured our glossary could use some work so that i have a way to map our terms to their terms | 20:36 |
fungi | in good news, there are people involved in that sig who are happy users of zuul, so i feel at least somewhat obligated to find out where we fit in there | 20:37 |
fungi | the interim chair also asked me to deliver a brief (~30-minute) presentation during the 2020-02-20 call on what measures zuul takes in support of "interoperability" (setting aside i suppose the fact that the term hasn't been clearly defined yet) | 20:43 |
fungi | i figure i'll talk about how zuul is designed to support integration between deployments through sharing of job definitions and ansible playbooks, though i'm open to ideas of course | 20:43 |
fungi | our decision not to nih a dsl is a big part of our stance on some kinds of interoperability | 20:44 |
fungi | as is the fundamental design around distribution and reuse of content in git repositories and connecting a deployment to repositories in multiple systems | 20:46 |
fungi | i still don't really buy into the underlying premise of the sig, but i'm open to seeing where it goes | 20:47 |
*** jamesmcarthur has quit IRC | 20:59 | |
*** mattw4 has joined #zuul | 21:07 | |
*** mattw4 has quit IRC | 21:13 | |
*** mattw4 has joined #zuul | 21:14 | |
corvus | fungi: that sounds relevant and correct. if they feel like "iteroperability" means "exchange standardized messages over a standardized message bus", then zuul's flexible driver system with user-configurable mqtt, etc drivers may be relevant. | 21:24 |
fungi | ooh, yes i should something something mqtt something in there ;) | 21:25 |
corvus | fungi: also tristanC has an amqp driver ready for review https://review.opendev.org/637458 | 21:26 |
fungi | for reference, it's a continuation of some of the things we talked to fdegir and zxiiro about at the ci/cd opendev event in vancouver a couple years ago | 21:26 |
corvus | ++ | 21:27 |
fungi | these discussions may eventually result in some sort of protocol standards for cross-systems communication, but for now i'm pushing for a commons of terminology and reviewing case studies to understand the problems folks are facing before we assume that's a solution | 21:29 |
*** Goneri has quit IRC | 21:39 | |
*** jamesmcarthur has joined #zuul | 21:41 | |
*** jamesmcarthur has quit IRC | 21:45 | |
*** jamesmcarthur has joined #zuul | 21:48 | |
*** decimuscorvinus has quit IRC | 22:15 | |
*** jamesmcarthur has quit IRC | 22:16 | |
*** decimuscorvinus has joined #zuul | 22:16 | |
*** jamesmcarthur has joined #zuul | 22:17 | |
*** jamesmcarthur has quit IRC | 22:18 | |
*** jamesmcarthur has joined #zuul | 22:25 | |
*** jamesmcarthur has quit IRC | 22:27 | |
*** jamesmcarthur has joined #zuul | 22:29 | |
*** jamesmcarthur has quit IRC | 22:31 | |
*** jamesmcarthur has joined #zuul | 22:34 | |
*** hashar has quit IRC | 22:36 | |
*** jamesmcarthur has quit IRC | 22:38 | |
*** jamesmcarthur has joined #zuul | 22:38 | |
*** armstrongs has joined #zuul | 22:46 | |
*** armstrongs has quit IRC | 22:55 | |
*** jamesmcarthur has quit IRC | 23:13 | |
*** jamesmcarthur has joined #zuul | 23:17 | |
*** jamesmcarthur has quit IRC | 23:21 | |
*** tosky has quit IRC | 23:45 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!