| *** jamesmcarthur has joined #zuul | 00:02 | |
| *** sdake has joined #zuul | 00:03 | |
| *** jamesmcarthur has quit IRC | 00:17 | |
| *** jamesmcarthur has joined #zuul | 00:18 | |
| *** jamesmcarthur has quit IRC | 00:22 | |
| openstackgerrit | Clark Boylan proposed openstack-infra/zuul-website master: Add a promotional message banner and events list https://review.openstack.org/639871 | 00:35 |
|---|---|---|
| clarkb | fungi: ^ based that on your original change. | 00:35 |
| clarkb | corvus: ^ we've been asked if we can add that type of banner back for the denver summit | 00:35 |
| fungi | SpamapS: i agree with you on premise ;) | 00:36 |
| SpamapS | fungi: excellent | 00:58 |
| *** sdake has quit IRC | 01:08 | |
| openstackgerrit | Tristan Cacqueray proposed openstack-infra/zuul master: Add API endpoint to get frozen jobs https://review.openstack.org/607077 | 01:59 |
| openstackgerrit | Tristan Cacqueray proposed openstack-infra/zuul master: Get executor job params https://review.openstack.org/607078 | 01:59 |
| *** sdake has joined #zuul | 02:10 | |
| *** bhavikdbavishi has joined #zuul | 02:29 | |
| *** bhavikdbavishi has quit IRC | 02:31 | |
| *** rlandy|bbl is now known as rlandy | 02:45 | |
| *** sdake has quit IRC | 02:46 | |
| *** sdake has joined #zuul | 03:22 | |
| *** sdake has quit IRC | 03:23 | |
| *** bhavikdbavishi has joined #zuul | 03:24 | |
| *** takamatsu has quit IRC | 03:25 | |
| *** sdake has joined #zuul | 03:28 | |
| *** sdake has quit IRC | 03:40 | |
| *** nilashishc has joined #zuul | 03:44 | |
| *** nilashishc has quit IRC | 03:48 | |
| openstackgerrit | Tristan Cacqueray proposed openstack-infra/zuul master: WIP: add job filter to trigger event https://review.openstack.org/639905 | 03:49 |
| *** nilashishc has joined #zuul | 03:52 | |
| *** spsurya has joined #zuul | 04:14 | |
| openstackgerrit | Joshua Hesketh proposed openstack-infra/zuul master: Get executor job params https://review.openstack.org/607078 | 04:16 |
| *** nilashishc has quit IRC | 04:46 | |
| *** rlandy has quit IRC | 04:50 | |
| *** saneax has joined #zuul | 04:55 | |
| openstackgerrit | Tristan Cacqueray proposed openstack-infra/zuul master: webtrigger: add initial driver and event https://review.openstack.org/555153 | 05:00 |
| *** raukadah has quit IRC | 05:03 | |
| *** chandankumar has joined #zuul | 05:04 | |
| *** takamatsu has joined #zuul | 05:05 | |
| *** bjackman has joined #zuul | 05:09 | |
| bjackman | AJaeger, tobiash, ping for https://review.openstack.org/#/c/636870/ (let me know if I've picked the wrong people to pester!) | 05:10 |
| *** takamatsu has quit IRC | 05:10 | |
| openstackgerrit | Tristan Cacqueray proposed openstack-infra/zuul master: webtrigger: add web route and rpclistener https://review.openstack.org/554839 | 05:15 |
| tristanC | corvus: could you please revisit your -2 on https://review.openstack.org/555153 | 05:17 |
| *** saneax has quit IRC | 05:27 | |
| *** saneax has joined #zuul | 05:28 | |
| *** sdake has joined #zuul | 05:30 | |
| *** nilashishc has joined #zuul | 05:45 | |
| *** nilashishc has quit IRC | 05:51 | |
| *** sdake has quit IRC | 06:02 | |
| *** bhavikdbavishi has quit IRC | 06:19 | |
| *** bhavikdbavishi has joined #zuul | 06:20 | |
| *** sdake has joined #zuul | 06:22 | |
| *** sdake has quit IRC | 06:26 | |
| *** nilashishc has joined #zuul | 06:30 | |
| *** nilashishc has left #zuul | 06:36 | |
| *** sdake has joined #zuul | 06:43 | |
| openstackgerrit | Tristan Cacqueray proposed openstack-infra/zuul master: web: add build button to trigger job https://review.openstack.org/635716 | 07:04 |
| openstackgerrit | Merged openstack-infra/zuul-jobs master: Explanatory comment for Editable Git install https://review.openstack.org/639649 | 07:09 |
| *** sdake has quit IRC | 07:15 | |
| *** sshnaidm|afk is now known as sshnaidm | 07:32 | |
| openstackgerrit | Tristan Cacqueray proposed openstack-infra/zuul master: web: switch jobs list to a tree view https://review.openstack.org/633437 | 07:41 |
| *** quiquell is now known as quiquell|brb | 07:48 | |
| openstackgerrit | Tristan Cacqueray proposed openstack-infra/zuul master: web: add jobs list filter https://review.openstack.org/633652 | 07:48 |
| openstackgerrit | Tristan Cacqueray proposed openstack-infra/zuul master: scheduler: add job's tags to the rpc job_list method https://review.openstack.org/633653 | 07:48 |
| openstackgerrit | Tristan Cacqueray proposed openstack-infra/zuul master: web: add tags to jobs list https://review.openstack.org/633654 | 07:48 |
| *** quiquell|brb is now known as quiquell | 08:14 | |
| *** pcaruana has joined #zuul | 08:32 | |
| *** bhavikdbavishi has quit IRC | 08:38 | |
| *** bhavikdbavishi has joined #zuul | 08:38 | |
| *** jpena|off is now known as jpena | 08:54 | |
| *** electrofelix has joined #zuul | 09:07 | |
| *** electrofelix has left #zuul | 09:08 | |
| *** electrofelix has joined #zuul | 09:11 | |
| *** sdake has joined #zuul | 09:14 | |
| *** gtema has joined #zuul | 09:16 | |
| jkt | corvus: that's a nice one :), thanks! | 09:34 |
| *** sdake has quit IRC | 09:46 | |
| *** gtema has quit IRC | 09:48 | |
| *** gtema has joined #zuul | 09:49 | |
| *** panda|ruck|off is now known as panda|ruck | 09:51 | |
| *** sdake has joined #zuul | 10:36 | |
| *** hashar has joined #zuul | 10:53 | |
| *** saneax has quit IRC | 10:56 | |
| openstackgerrit | Merged openstack-infra/nodepool master: Support requests for specific providers https://review.openstack.org/639418 | 11:02 |
| *** bhavikdbavishi has quit IRC | 11:03 | |
| *** mhu has joined #zuul | 11:09 | |
| openstackgerrit | Merged openstack-infra/zuul master: Request child nodes of paused jobs at higher priority https://review.openstack.org/638796 | 11:10 |
| openstackgerrit | Matthieu Huin proposed openstack-infra/zuul master: [WIP] Add AuthorizationRule Configuration Object, claim validation https://review.openstack.org/639855 | 11:11 |
| *** sdake has quit IRC | 11:18 | |
| *** sdake has joined #zuul | 11:20 | |
| *** hashar has quit IRC | 11:33 | |
| *** sdake has quit IRC | 11:55 | |
| *** bjackman has quit IRC | 12:07 | |
| *** bhavikdbavishi has joined #zuul | 12:12 | |
| *** bhavikdbavishi has quit IRC | 12:14 | |
| *** bhavikdbavishi has joined #zuul | 12:14 | |
| *** bhavikdbavishi1 has joined #zuul | 12:19 | |
| *** bhavikdbavishi has quit IRC | 12:19 | |
| *** bhavikdbavishi1 is now known as bhavikdbavishi | 12:19 | |
| *** jpena is now known as jpena|lunch | 12:41 | |
| *** saneax has joined #zuul | 12:43 | |
| *** panda|ruck is now known as panda|ruck|lunch | 12:44 | |
| *** hashar has joined #zuul | 13:01 | |
| openstackgerrit | Matthieu Huin proposed openstack-infra/zuul master: [WIP] Add AuthorizationRule Configuration Object, claim validation https://review.openstack.org/639855 | 13:14 |
| *** jamesmcarthur has joined #zuul | 13:16 | |
| *** gtema has quit IRC | 13:21 | |
| *** jamesmcarthur has quit IRC | 13:23 | |
| *** jamesmcarthur has joined #zuul | 13:26 | |
| *** panda|ruck|lunch is now known as panda|ruck | 13:30 | |
| *** jamesmcarthur has quit IRC | 13:31 | |
| *** rlandy has joined #zuul | 13:35 | |
| *** quiquell is now known as quiquell|lunch | 13:38 | |
| *** Shrews_ is now known as Shrews | 13:43 | |
| *** jpena|lunch is now known as jpena | 13:43 | |
| *** jamesmcarthur has joined #zuul | 13:46 | |
| *** quiquell|lunch is now known as quiquell | 13:52 | |
| *** rfolco|rover is now known as rfolco|afk | 13:54 | |
| *** jamesmcarthur has quit IRC | 14:04 | |
| *** jamesmcarthur has joined #zuul | 14:05 | |
| *** gtema has joined #zuul | 14:09 | |
| *** jamesmcarthur has quit IRC | 14:10 | |
| *** ianychoi has joined #zuul | 14:20 | |
| *** gtema has quit IRC | 14:25 | |
| *** jamesmcarthur has joined #zuul | 14:38 | |
| *** bhavikdbavishi has quit IRC | 14:43 | |
| *** bhavikdbavishi has joined #zuul | 14:44 | |
| *** jamesmcarthur has quit IRC | 14:44 | |
| pabelanger | mordred: corvus: tobiash: fungi: could we determine what the next step for https://review.openstack.org/638801/ is, and I'm happy to work on it today. Just seen another user hit this bug, would love to get a zuul release to fix | 14:55 |
| pabelanger | specifically, the backwards compat work | 14:55 |
| *** jamesmcarthur has joined #zuul | 15:18 | |
| *** jamesmcarthur has quit IRC | 15:22 | |
| tobiash | pabelanger: the next step is a release note but I was too busy this week so far to do it | 15:27 |
| *** gtema has joined #zuul | 15:28 | |
| *** saneax has quit IRC | 15:30 | |
| *** jamesmcarthur has joined #zuul | 15:42 | |
| *** hashar has quit IRC | 15:45 | |
| *** jamesmcarthur has quit IRC | 15:46 | |
| *** jamesmcarthur has joined #zuul | 15:47 | |
| tobiash | pabelanger, corvus: I'll provide a release note later this evening, sorry for the delay on this | 16:00 |
| *** rfolco|afk is now known as rfolco|rover | 16:10 | |
| corvus | tobiash: np | 16:15 |
| corvus | it'll be good to get 637472 in too | 16:16 |
| *** SpamapS has quit IRC | 16:18 | |
| *** bhavikdbavishi has quit IRC | 16:18 | |
| *** bhavikdbavishi has joined #zuul | 16:19 | |
| *** SpamapS has joined #zuul | 16:19 | |
| *** quiquell is now known as quiquell|off | 16:20 | |
| *** quiquell|off is now known as quique | 16:23 | |
| *** quique is now known as quiquell|off | 16:23 | |
| mordred | corvus: 637472 is a monasca-events-api change? | 16:28 |
| corvus | mordred: absolutely critical for us. almost as important as 639472. | 16:29 |
| mordred | corvus: good, because it got landed already :) | 16:29 |
| *** gtema has quit IRC | 16:29 | |
| *** chandankumar is now known as raukadah | 16:34 | |
| *** dkehn has joined #zuul | 16:38 | |
| *** bhavikdbavishi1 has joined #zuul | 16:45 | |
| *** bhavikdbavishi has quit IRC | 16:46 | |
| *** bhavikdbavishi1 is now known as bhavikdbavishi | 16:46 | |
| openstackgerrit | Merged openstack-infra/zuul master: Request dependent job nodes from paused parent provider https://review.openstack.org/639467 | 16:56 |
| openstackgerrit | Jean-Philippe Evrard proposed openstack-infra/zuul-jobs master: Add tag in upload image name https://review.openstack.org/640096 | 16:56 |
| *** electrofelix has quit IRC | 16:59 | |
| openstackgerrit | Merged openstack-infra/zuul master: Fix multiple jobs with provides/requires https://review.openstack.org/639472 | 17:09 |
| openstackgerrit | Tobias Henkel proposed openstack-infra/zuul master: Encode zuul.message with base64 https://review.openstack.org/638801 | 17:09 |
| tobiash | corvus, pabelanger: now with release note ^ | 17:09 |
| corvus | tobiash: thanks! | 17:10 |
| tobiash | pabelanger: speaking of release I think these are important too and need a second review: 639118, 639023 | 17:10 |
| tobiash | (semaphore fixes) | 17:14 |
| *** jamesmcarthur has quit IRC | 17:27 | |
| mordred | tobiash: crap. I reviewed those the other day but didn't actually, you know, vote | 17:33 |
| tobiash | mordred: thanks :) | 17:35 |
| *** jamesmcarthur has joined #zuul | 17:37 | |
| pabelanger | tobiash: thanks! had to step away for a bit | 17:40 |
| *** panda|ruck is now known as panda|ruck|off | 17:42 | |
| *** jamesmcarthur has quit IRC | 17:45 | |
| *** jamesmcarthur has joined #zuul | 17:48 | |
| *** bhavikdbavishi has quit IRC | 17:50 | |
| *** bhavikdbavishi has joined #zuul | 17:52 | |
| *** jamesmcarthur has quit IRC | 17:52 | |
| *** jamesmcarthur has joined #zuul | 17:53 | |
| openstackgerrit | Merged openstack-infra/zuul master: Fix missing semaphore release around dequeue https://review.openstack.org/639023 | 17:54 |
| openstackgerrit | Merged openstack-infra/zuul master: Fix rare semaphore leak during reconfiguration https://review.openstack.org/639118 | 17:59 |
| jkt | hey, why am I getting this from Zuul? This change or one of its cross-repo dependencies was unable to be automatically merged with the current state of its repository. Please rebase the change and upload a new patchset. | 18:00 |
| jkt | I have default settings for merging, this change's parent is just one commit behind master, it is changing README.md in repo's root | 18:00 |
| jkt | and the change which is in master changed one file called ci/build.sh and a submodule called submodules/dependencies | 18:01 |
| jkt | I looked into merger's and executor's logs, but I didn't find any reason listed in there | 18:02 |
| openstackgerrit | Merged openstack-infra/zuul master: Encode zuul.message with base64 https://review.openstack.org/638801 | 18:06 |
| *** bhavikdbavishi has quit IRC | 18:09 | |
| *** jpena is now known as jpena|off | 18:12 | |
| *** bhavikdbavishi has joined #zuul | 18:22 | |
| *** bhavikdbavishi has quit IRC | 18:32 | |
| *** jamesmcarthur has quit IRC | 18:37 | |
| josefwells | I have my zuul install cleaned up and working with enterprise github instead of gerrit | 18:42 |
| josefwells | I'm going to get some time this weekend to submit some changes back to you guys, as there was quite a bit of unraveling to do | 18:43 |
| josefwells | So now I am at the interesting point of the project, pipeline and job development | 18:43 |
| josefwells | what is the best resource for learning about tying together repositories with dependent pipelines, etc | 18:44 |
| josefwells | Figured just looking at openstack's configs would be a good start | 18:44 |
| josefwells | but maybe too much? | 18:44 |
| *** sdake has joined #zuul | 18:44 | |
| *** pcaruana has quit IRC | 19:14 | |
| josefwells | Is there a way to override zuul-jobs defaults (zuul_workspace_root) for instance | 19:47 |
| clarkb | josefwells: yes, where you use the prepare-workspace role you can set a different value for that var | 19:48 |
| clarkb | (I think you can actually set it asa a var on any job but in the case of that role you'll likely set it on your base-job and forget it | 19:48 |
| josefwells | and it seems like maybe there is a better thing to do, like override a root var, and have zuul_workspace_root get created from that <root>/<pr>/<sha>/workspace | 19:49 |
| josefwells | or something like that | 19:49 |
| josefwells | right | 19:49 |
| josefwells | or os zuul_workspace_root already that variable | 19:50 |
| josefwells | os = is | 19:50 |
| clarkb | that is the variable to set | 19:50 |
| clarkb | it has a default value whihc you can override | 19:50 |
| *** hashar has joined #zuul | 19:51 | |
| *** jamesmcarthur has joined #zuul | 19:56 | |
| *** sdake has quit IRC | 20:07 | |
| josefwells | so I see that workspace honored by prepare-workspace, but that is not where my job starts, how do I get into my playbooks to actually run there, do I have a step that does a chdir: to {some_variable_constructed_from_root}? | 20:18 |
| clarkb | because zuul uses ansible, the default behavior is to end up in the homedir of the user you ssh into as iirc | 20:24 |
| josefwells | yeah, that seems to be the case | 20:25 |
| josefwells | so is ansible umm.. stateless, like a makefile | 20:25 |
| clarkb | there might be an ansible configuration setting to change the dir you end up in but I'm not sure about that | 20:25 |
| josefwells | each order is independent of the others, no, 'cd foo' in one command, then 'do thing that needs to be done in foo' in another command? | 20:25 |
| josefwells | seems like I would want this in my pre. setup, like chdir: {zuul.project.src_dir}, but I don't know how sticky that is | 20:26 |
| josefwells | yeah, seems like you can run a command in a directory, but ansible don't have that kind of state, which makes sense | 20:28 |
| josefwells | to me anyway | 20:28 |
| pabelanger | most task in ansible have the ability to setup args: chdir: foo | 20:29 |
| pabelanger | otherwise, you could use full path to think you want to run | 20:29 |
| pabelanger | {{ ansible_user_dir }}/{{ zuul.project.src_dir }}/foo | 20:30 |
| josefwells | ah, that is the sticker, beauty, thanks pabelanger and clarkb! | 20:36 |
| josefwells | this is going to get interesting when we have interdependencies and want to try proposed changes with other proposed changes | 20:37 |
| SpamapS | josefwells: Ansible is just SSH in a for loop in python... with some really nice python bits included as modules. :) | 20:52 |
| *** sdake has joined #zuul | 21:04 | |
| jkt | josefwells: my impression was that a lot of existing roles and playbooks depend on a particular directory layout, though | 21:14 |
| jkt | so stuff like changing jobs to run from somewhere else than $HOME would be a bit impractical | 21:14 |
| jkt | but anyway, ymmv. | 21:14 |
| *** sdake has quit IRC | 21:23 | |
| *** sdake has joined #zuul | 21:23 | |
| pabelanger | yah, I'd definitely say the pattern so far is based of $HOME | 21:38 |
| *** sdake has quit IRC | 22:13 | |
| *** hashar has quit IRC | 22:44 | |
| mrhillsman | any idea why zuul would complain about a job not being defined that clearly is? | 23:05 |
| clarkb | defined on another branch maybe? | 23:05 |
| mrhillsman | i have a job defined in my config-project on branch master | 23:06 |
| mrhillsman | i forked an untrusted project and submitted a pr to merge to master of that untrusted project | 23:07 |
| clarkb | are they in the same tenant? | 23:07 |
| mrhillsman | yes sir | 23:07 |
| mrhillsman | the job is not defined in the untrusted project but rather in the config-project | 23:08 |
| mrhillsman | in the untrusted i am just adding pipeline stanza | 23:08 |
| mrhillsman | https://www.irccloud.com/pastebin/NzYV95Sr/ | 23:09 |
| mrhillsman | the only change i am making to branches is in a periodic pipeline | 23:09 |
| clarkb | I would triple check the job names are the same (maybe even make sure there is no unicode - or similar causing confusion) | 23:10 |
| mrhillsman | so do i have to modify the branch? | 23:10 |
| mrhillsman | i quadrupled * 2 checked the job name and it is just not making sense to me | 23:13 |
| *** threestrands has joined #zuul | 23:13 | |
| mrhillsman | probably just my ignorance of how it works | 23:13 |
| clarkb | on the branch side of things if you haven't set branches in the trusted project or in your pipeline on the untrusted project that makes it easier to debug | 23:14 |
| clarkb | (once you set branches then you have to think about when it will and won't apply) | 23:14 |
| mrhillsman | exclude-unprotected-branches: true for this particular untrusted-project | 23:15 |
| mrhillsman | jobs in the config-project - https://github.com/theopenlab/openlab-zuul-jobs/blob/master/zuul.d/jobs.yaml | 23:15 |
| mrhillsman | in particular - https://github.com/theopenlab/openlab-zuul-jobs/blob/master/zuul.d/jobs.yaml#L343-L349 | 23:15 |
| mrhillsman | untrusted project PR - https://github.com/kubernetes/cloud-provider-openstack/pull/512 | 23:16 |
| clarkb | mrhillsman: the reason is your use of secrets | 23:16 |
| mrhillsman | oh? | 23:16 |
| clarkb | actually maybe not | 23:16 |
| clarkb | since the secrets carrying job is in a trusted repo right? | 23:17 |
| mrhillsman | yes sir | 23:17 |
| clarkb | ok then probably not that | 23:17 |
| mrhillsman | that untrusted is only supposed to reference the trusted | 23:17 |
| mrhillsman | does not have anything but .zuul.yaml living there | 23:17 |
| mrhillsman | defining pipelines and jobs | 23:17 |
| mrhillsman | so i forked the untrusted, make changes, submitted the pr, and it says the job is not defined | 23:18 |
| mrhillsman | is it possible the pr is using an older version of the config-project that does not have the jobs defined? | 23:19 |
| mrhillsman | on each fresh push does zuul not check if the config-project is outdated and update it? | 23:19 |
| clarkb | that depends on your connection type. If using the raw git connection it checks on a timer, but the github and gerrit connections should trigger a refresh on merge | 23:20 |
| mrhillsman | if the pr went in before the config-project was updated...that is the only thing i can think that makes sense | 23:20 |
| mrhillsman | since a job never ran do you know which log file i would check to see the hash/commit it is trying to use of the config-project for that pr? | 23:21 |
| mrhillsman | that is the only thing that makes sense to me really | 23:22 |
| clarkb | the scheduler | 23:22 |
| mrhillsman | ok, thx | 23:22 |
| clarkb | you should have logs around the cat gearman jobs | 23:22 |
| clarkb | those literally cat the contents of repos to find the zuul config files and their contents | 23:23 |
| mrhillsman | ++ | 23:23 |
| corvus | mrhillsman: do you have a pipeline for every job? | 23:23 |
| mrhillsman | for that project yes sir | 23:23 |
| clarkb | another thing to check is if you have any config errors | 23:23 |
| corvus | mrhillsman: why is that? | 23:23 |
| clarkb | there is a little bell in the top right of your status web ui that will show them to you | 23:23 |
| clarkb | (it could be you haven't loaded up to date config because there is an error in that config) | 23:24 |
| mrhillsman | that is what was requested but honestly i have no idea why it could not be cleaner | 23:24 |
| mrhillsman | no config errors clarkb | 23:24 |
| corvus | mrhillsman: i'd love to understand the reasoning for that, because that's certainly not how we expect it to be used, and it's... a little hard to follow. | 23:25 |
| corvus | mrhillsman: aside from that -- on http://status.openlabtesting.org/jobs i don't see the 1.13 job | 23:26 |
| mrhillsman | hrmm...maybe i need to restart zuul | 23:26 |
| mrhillsman | the scheduler that is | 23:27 |
| mrhillsman | i'll get an explanation re so many pipelines | 23:28 |
| mrhillsman | i too had questions but other things pulling my attention and was not primary contact on setting up this particular repo | 23:28 |
| corvus | mrhillsman: is openlab-zuul-jobs gated by zuul? | 23:29 |
| corvus | it looks like individual humans are merging pull requests to that, not zuul | 23:29 |
| mrhillsman | zuul is not merging automatically no | 23:30 |
| mrhillsman | i believe there are syntax checks however | 23:30 |
| corvus | i recommend having zuul gate that repo. there shouldn't be any reason not too, since it's dedicated to zuul. doing so will help protect from system-wide problems. | 23:31 |
| mrhillsman | ++ | 23:31 |
| corvus | mrhillsman: it's possible that zuul was offline or missed the pull request merge event for the change which added the 1.13 job | 23:31 |
| mrhillsman | ok, i see them now | 23:32 |
| mrhillsman | that may have been it :( | 23:32 |
| mrhillsman | gating will help with that? | 23:32 |
| corvus | mrhillsman: you should be able to correct by running "zuul-scheduler full-reconfigure"; you don't necessarily need to restart the scheduler | 23:32 |
| mrhillsman | yeah, sorry, i ran that | 23:32 |
| mrhillsman | it is showing up now | 23:32 |
| mrhillsman | i did not know the pipeline would show up before actual job run | 23:32 |
| mrhillsman | so now i know, i just could not for the life of me figure it out without having run into this before | 23:33 |
| mrhillsman | especially when i knew the config-project repo had already merged the commit but i see what you mean now | 23:33 |
| mrhillsman | we may have pushed it causing zuul to miss it somehow | 23:33 |
| mrhillsman | but let zuul do the merging and it should not happen | 23:34 |
| mrhillsman | if that is the lesson to learn you mean | 23:34 |
| corvus | mrhillsman: yeah, there are still cases where this could happen even when gating, but i think they are less likely. but more importantly, it prevents other classes of errors, like merging incompatible changes. | 23:35 |
| mrhillsman | awesome, thx clarkb and corvus | 23:35 |
| *** jamesmcarthur has quit IRC | 23:46 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!