| *** rlandy is now known as rlandy|bbl | 01:06 | |
| *** threestrands has joined #zuul | 01:46 | |
| *** caphrim007 has quit IRC | 01:57 | |
| *** rlandy|bbl is now known as rlandy | 04:23 | |
| *** rlandy has quit IRC | 04:23 | |
| *** threestrands has quit IRC | 04:59 | |
| *** threestrands has joined #zuul | 05:02 | |
| *** bjackman has quit IRC | 05:08 | |
| *** chandankumar has joined #zuul | 05:09 | |
| *** chandankumar is now known as chkumar|ruck | 05:10 | |
| *** bjackman has joined #zuul | 05:13 | |
| *** quiquell|off is now known as quiquell | 06:50 | |
| bjackman | If a job is aborted, seems like the post-run playbooks aren't run, is that right? | 07:08 |
|---|---|---|
| bjackman | By "aborted" I mean when the gerrit change that triggered the job is superseded (the log shows "RUN END RESULT_ABORTED") | 07:11 |
| bjackman | (project.dequeue-on-new-patchset = true) | 07:22 |
| clarkb | bjackman: correct | 07:36 |
| clarkb | there has been talk of adding a cleanup phase that would always run | 07:37 |
| *** threestrands has quit IRC | 08:04 | |
| bjackman | clarkb: OK. I wonder if there's also some way to abort it in such a way that an "always: " block in the playbook would get run too | 08:17 |
| bjackman | So it kinda looks like the ansible task that was underway at the time of the abort just had an error | 08:17 |
| *** hashar has joined #zuul | 08:26 | |
| *** quiquell has quit IRC | 08:32 | |
| *** jpena|off is now known as jpena | 08:50 | |
| *** pcaruana has joined #zuul | 09:56 | |
| *** quiquell has joined #zuul | 09:58 | |
| quiquell | Hello someone know where is the Dockerfile of "zuul/nodepool-builder" ? | 09:58 |
| quiquell | clarkb: ^ ? | 09:58 |
| quiquell | AJaeger: | 09:58 |
| quiquell | ? | 09:58 |
| corvus | quiquell: there isn't one -- it's built with pbrx | 10:00 |
| quiquell | corvus: whats pbrx ? | 10:00 |
| quiquell | corvus: I see the image at dockerhub | 10:00 |
| corvus | quiquell: http://git.openstack.org/cgit/openstack/pbrx/tree/README.rst | 10:02 |
| corvus | i can't seem to find the docs published anywhere | 10:03 |
| corvus | quiquell: http://git.openstack.org/cgit/openstack/pbrx/tree/doc/source/user/images.rst | 10:03 |
| quiquell | corvus: thanks | 10:04 |
| *** electrofelix has joined #zuul | 10:11 | |
| fungi | corvus: claire is asking about the possibility of getting a digital copy of the zuul one-pager served from the zuul-ci.org web site. how do you feel about having an 84kb pdf committed to the zuul-website repo? | 10:39 |
| AJaeger | fungi: perhaps adding it to zuul-website-media instead? | 10:55 |
| clarkb | oh ya with thr video ++ | 10:57 |
| *** bjackman has quit IRC | 11:02 | |
| *** electrofelix has quit IRC | 11:03 | |
| corvus | yeah, what AJaeger said :) | 11:03 |
| openstackgerrit | Clark Boylan proposed openstack-infra/zuul-website-media master: Add Zuul poster/page pdf https://review.openstack.org/617581 | 11:07 |
| clarkb | there is the content, will also need a change to point to it from the website | 11:07 |
| corvus | clarkb, fungi: where are you planning to lunch? | 11:10 |
| clarkb | corvus:in the community contributor award room | 11:10 |
| clarkb | corvus: which is hall 7? I'm in a session now with diablo_rojo who has to go there to host so Iwill just follow her | 11:11 |
| corvus | that is not tagged lunch :( | 11:11 |
| clarkb | corvus: ya I too had missed it. let me see if that can be updated (at this point I'm unsure of how much that will help) | 11:12 |
| *** electrofelix has joined #zuul | 11:13 | |
| *** panda|rover|lch is now known as panda|rover | 11:13 | |
| AJaeger | so, looks like a hidden gem - let's go to Hall 7 - Level 1 - 7.1a / NY2 at 12:30 for lunch ;) | 11:14 |
| *** chkumar|ruck has quit IRC | 11:14 | |
| *** chandankumar has joined #zuul | 11:15 | |
| *** chandankumar is now known as chkumar|ruck | 11:15 | |
| corvus | clarkb: 581 seems like the sort of thing i can single-core +3 so i did | 11:15 |
| *** dkehn has quit IRC | 11:17 | |
| openstackgerrit | Merged openstack-infra/zuul-website-media master: Add Zuul poster/page pdf https://review.openstack.org/617581 | 11:18 |
| corvus | clarkb, fungi: ^ | 11:18 |
| clarkb | cool the other piece is to make it availabe on zuul-ci.org in some way. Maybe as a Documentation item on the top level docs page? | 11:19 |
| clarkb | schedule is being updated to try and make the CCA thing more obviously a lunch option | 11:22 |
| clarkb | (of course its late, but if you have the app maybe it will hel) | 11:23 |
| clarkb | *help | 11:23 |
| *** chkumar has joined #zuul | 11:36 | |
| *** chkumar|ruck has quit IRC | 11:37 | |
| *** chkumar is now known as chkumar|ruck | 11:37 | |
| *** bjackman has joined #zuul | 11:53 | |
| fungi | AJaeger: oh! you're genius, yes zwm is where it should go of course | 11:55 |
| fungi | looks like it published to https://zuul-ci.org/media/zuul_solution_brief.pdf fine. where on the site should we add a link for it do you think? | 11:59 |
| fungi | clarkb suggests linking it somehow at https://zuul-ci.org/docs/ | 12:00 |
| fungi | anyone have better ideas? | 12:01 |
| fungi | or on the main page between the video and the get started banner? | 12:02 |
| fungi | i'll push up something for that | 12:08 |
| corvus | hrm, i think i must have failed to find the awards room | 12:24 |
| clarkb | corvus: it is/was a bit out of the way | 12:33 |
| clarkb | fungi and I have relocated to the level 3 forum area for the openapi openstack session | 12:33 |
| openstackgerrit | Jeremy Stanley proposed openstack-infra/zuul-website master: Link the overview brochure from the splash page https://review.openstack.org/617623 | 12:38 |
| *** jpena is now known as jpena|lunch | 12:40 | |
| openstackgerrit | James E. Blair proposed openstack-infra/zuul-website master: Alter brochure link https://review.openstack.org/617631 | 12:56 |
| openstackgerrit | James E. Blair proposed openstack-infra/zuul-website master: Switch brochure link to h4 https://review.openstack.org/617632 | 12:57 |
| corvus | fungi, AJaeger: ^ two alternatives for visibility | 12:57 |
| corvus | i like the first option | 12:58 |
| AJaeger | corvus, clarkb, both of you gave +2 on 623 - will either of you +A? | 12:58 |
| clarkb | corvus: I +2'd fungi's and the first of the alterations | 12:59 |
| AJaeger | corvus: thanks - waiting for build result | 12:59 |
| fungi | i'll take a look, thanks! | 12:59 |
| corvus | how about i +a after we decide on any of the followups | 12:59 |
| AJaeger | ok | 13:00 |
| AJaeger | btw. should we add a "." to end of the "Read the ... Brochure *.*" - like it's done for other actions? | 13:01 |
| AJaeger | ... on the zuul-ci page ? | 13:01 |
| * AJaeger is for option 1 - will comment | 13:01 | |
| fungi | probably yes, i just wasn't paying close enough attention | 13:01 |
| *** nilashishc has joined #zuul | 13:02 | |
| corvus | AJaeger: hrm, i only see a '.' at the end of the video caption | 13:02 |
| AJaeger | corvus: see the three items project gating, CI/CD with ansible, Cross-project dependencies - all have a "." | 13:03 |
| AJaeger | corvus: I see we're not consistent - let me propose a nit on top of yours... | 13:03 |
| AJaeger | I'm for 617631 | 13:04 |
| corvus | cool, let's wait for fungi to weigh in, then approve some things | 13:05 |
| openstackgerrit | Andreas Jaeger proposed openstack-infra/zuul-website master: Add some "." https://review.openstack.org/617634 | 13:06 |
| * AJaeger hands out some periods ^ | 13:07 | |
| fungi | distracted by forum session, sorry! looks like draft is posted now so at least i don | 13:12 |
| fungi | t need to pull those locally | 13:12 |
| fungi | blue link on a light background is a little harder to read, so the h4 may be superior in that regard? | 13:14 |
| fungi | also, is there maybe too much vertical negative space between the video and the brochure link maybe? | 13:14 |
| AJaeger | fungi: for me it's not clear that the h4 gives you a link - since it's text color, not link color | 13:15 |
| fungi | that i also agree with | 13:16 |
| fungi | i guess i'm cool with the stack then (not including the h4 alternatve) | 13:17 |
| fungi | i'll push up another on top of 617634 to get rid of half of the dead padding | 13:18 |
| corvus | fungi: yeah, i think the dual color thing is the best of both worlds. also, maybe we want to change the link color. :) | 13:18 |
| openstackgerrit | Jeremy Stanley proposed openstack-infra/zuul-website master: Reduce vertical spacing between video and brochure https://review.openstack.org/617639 | 13:21 |
| fungi | deepen/darken the link hue so it doesn't fade into the white background quite so much? | 13:21 |
| openstackgerrit | Merged openstack-infra/zuul-website master: Link the overview brochure from the splash page https://review.openstack.org/617623 | 13:23 |
| openstackgerrit | Merged openstack-infra/zuul-website master: Alter brochure link https://review.openstack.org/617631 | 13:23 |
| openstackgerrit | Merged openstack-infra/zuul-website master: Add some "." https://review.openstack.org/617634 | 13:23 |
| *** rlandy has joined #zuul | 13:27 | |
| *** dkehn has joined #zuul | 13:36 | |
| AJaeger | fungi, 617639 is a noop on my firefox - unless I'm blind... | 13:39 |
| *** jpena|lunch is now known as jpena | 13:40 | |
| *** bjackman has quit IRC | 13:52 | |
| *** spsurya has joined #zuul | 14:03 | |
| corvus | fungi, clarkb: the pdf is live and linked (refresh needed) | 14:04 |
| fungi | AJaeger: i agree | 14:12 |
| *** chkumar|ruck has quit IRC | 14:17 | |
| *** bjackman has joined #zuul | 14:19 | |
| clarkb | corvus: thanks | 14:20 |
| openstackgerrit | Jeremy Stanley proposed openstack-infra/zuul-website master: Reduce vertical spacing between video and brochure https://review.openstack.org/617639 | 14:20 |
| fungi | maybe that'll do it | 14:21 |
| fungi | nope :/ | 14:26 |
| dmsimard | fungi: the firefox live inspector has typically been helpful for me when I wanted to test things like that | 14:41 |
| dmsimard | chrome has a similar capability | 14:41 |
| dmsimard | right click around the general idea where you want to edit something, inspect, and you can live edit html/css | 14:42 |
| dmsimard | fungi: looks like a big part contributing to the whitespace is the "wrapper" class' padding | 14:47 |
| dmsimard | current padding: https://screenshots.firefox.com/KFs79bvqzgn3LlL3/zuul-ci.org | 14:51 |
| dmsimard | reduced padding: https://screenshots.firefox.com/R1aosMl6conDNrDV/zuul-ci.org | 14:51 |
| openstackgerrit | David Moreau Simard proposed openstack-infra/zuul-website master: Reduce padding to remove extraneous whitespace https://review.openstack.org/617680 | 14:58 |
| fungi | dmsimard: thanks! | 15:01 |
| dmsimard | fungi: I don't think it solves what you set out to do exactly but it helps a bit? | 15:02 |
| dmsimard | ¯\_(ツ)_/¯ | 15:02 |
| dmsimard | oh, I found it | 15:03 |
| openstackgerrit | David Moreau Simard proposed openstack-infra/zuul-website master: Reduce padding to remove extraneous whitespace https://review.openstack.org/617680 | 15:10 |
| dmsimard | slightly better now | 15:10 |
| dmsimard | The remainder is mostly because the brochure is in it's own container/wrapper | 15:10 |
| *** nilashishc has quit IRC | 15:11 | |
| openstackgerrit | Tobias Henkel proposed openstack-infra/zuul master: Convert zuul_return into action plugin https://review.openstack.org/591168 | 15:14 |
| openstackgerrit | Tobias Henkel proposed openstack-infra/zuul master: Convert zuul_return into action plugin https://review.openstack.org/591168 | 15:20 |
| *** hashar has quit IRC | 15:28 | |
| *** quiquell is now known as quiquell|off | 15:38 | |
| openstackgerrit | James E. Blair proposed openstack-infra/nodepool master: Move k8s install to pre playbook https://review.openstack.org/617699 | 15:39 |
| *** quiquell|off is now known as quiquell | 16:00 | |
| *** hashar has joined #zuul | 16:00 | |
| *** quiquell is now known as quiquell|off | 16:04 | |
| *** spsurya has quit IRC | 16:12 | |
| dmsimard | corvus: hrm, compiling the scss with "sass www/assets/sass/main.scss www/assets/css/main.css" straight from the master branch isn't no-op for me, changes a bunch of things.. "sass --version" prints "1.14.3 compiled with dart2js 2.0.0" | 16:13 |
| dmsimard | it's not just changes that could've been forgotten, it's stuff like changing single quotes to double quotes and so on | 16:13 |
| dmsimard | the diff: http://paste.openstack.org/show/734770/ | 16:14 |
| openstackgerrit | Tristan Cacqueray proposed openstack-infra/zuul master: Add allowed-triggers and allowed-reporters tenant settings https://review.openstack.org/554082 | 16:16 |
| openstackgerrit | Tristan Cacqueray proposed openstack-infra/zuul master: Add allowed-triggers and allowed-reporters tenant settings https://review.openstack.org/554082 | 16:34 |
| corvus | dmsimard: it's possible i last ran it with Sass 3.5.5 (Bleeding Edge) | 16:41 |
| *** irclogbot_3 has joined #zuul | 16:42 | |
| *** irclogbot_3 has quit IRC | 16:43 | |
| corvus | dmsimard: which i think matches what came from the upstream template | 16:46 |
| openstackgerrit | Tristan Cacqueray proposed openstack-infra/zuul master: Add allowed-labels tenant setting https://review.openstack.org/617740 | 16:49 |
| tristanC | pabelanger: tobiash: here is a draft for the allowed-label thing we talked about ^ | 16:49 |
| corvus | tristanC: was there an older version of that change? | 16:52 |
| *** bjackman has quit IRC | 17:30 | |
| *** hashar has quit IRC | 17:56 | |
| *** ianychoi has quit IRC | 18:00 | |
| *** ianychoi has joined #zuul | 18:01 | |
| *** jpena is now known as jpena|off | 18:07 | |
| *** panda|rover is now known as panda|off | 18:11 | |
| *** electrofelix has quit IRC | 19:01 | |
| *** irclogbot_3 has joined #zuul | 19:12 | |
| *** hashar has joined #zuul | 19:36 | |
| dmsimard | corvus: neither sass or node-sass comes available in a 3.5.5 version.. should I just send a patch which recompiles the scss with the latest version of sass ? | 19:36 |
| dmsimard | sass: https://www.npmjs.com/package/sass?activeTab=versions and node-sass: https://www.npmjs.com/package/node-sass?activeTab=versions | 19:37 |
| tristanC | dmsimard: are we using sass at all? | 19:46 |
| *** hashar has quit IRC | 19:46 | |
| tristanC | corvus: allowed-labels is similar to the allowed-triggers code, but it's a new thing that was discussed in the hallway | 19:48 |
| dmsimard | tristanC: for zuul-ci.org | 19:48 |
| openstackgerrit | Tristan Cacqueray proposed openstack-infra/zuul master: Add allowed-labels tenant setting https://review.openstack.org/617740 | 19:49 |
| *** irclogbot_3 has quit IRC | 20:10 | |
| *** irclogbot_3 has joined #zuul | 20:14 | |
| tobiash | tristanC: added a couple of comments to ^ | 20:14 |
| *** jimi|ansible has quit IRC | 20:36 | |
| *** ssbarnea has quit IRC | 20:42 | |
| *** ssbarnea|bkp2 has joined #zuul | 20:42 | |
| *** pcaruana has quit IRC | 21:56 | |
| *** threestrands has joined #zuul | 22:00 | |
| *** panda|off has quit IRC | 22:09 | |
| *** panda has joined #zuul | 22:10 | |
| *** rlandy is now known as rlandy|brb | 22:16 | |
| *** sshnaidm is now known as sshnaidm|pto | 22:34 | |
| *** hashar has joined #zuul | 22:47 | |
| *** rlandy|brb is now known as rlandy | 22:47 | |
| *** hashar has quit IRC | 22:53 | |
| *** hashar has joined #zuul | 22:53 | |
| *** hashar has quit IRC | 23:11 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!