*** alexpilotti has quit IRC | 00:21 | |
*** michchap has joined #openstack-infra | 00:24 | |
*** changbl has joined #openstack-infra | 00:25 | |
*** sdake_ has joined #openstack-infra | 00:28 | |
clarkb | I am going to start puppet on git.o.o now | 00:35 |
---|---|---|
jeblair | clarkb: ++ | 00:35 |
jeblair | clarkb: i got close but not quite there on nodepool; i'll finish that early in the week. | 00:36 |
clarkb | ok | 00:36 |
clarkb | I plan on working towards a new git.o.o early in the week | 00:36 |
clarkb | then hopefully back to backups afterwards | 00:38 |
clarkb | haproxy did restart | 00:38 |
clarkb | all done. git.o.o seems happy | 00:39 |
jeblair | good time to do it then. :) | 00:39 |
jeblair | the ramp-up seems to just be starting. | 00:39 |
clarkb | yup | 00:40 |
clarkb | I thought I had more tiem but realized the australians were working | 00:40 |
*** michchap has quit IRC | 00:41 | |
*** michchap has joined #openstack-infra | 00:41 | |
jeblair | clarkb: my plan for zuul this week is to get some merger improvents in place, along with a fix to the graceful restart bug. | 00:42 |
clarkb | g-g-p takes under a minute now on the py26 jobs I have checked. so much better | 00:42 |
clarkb | jeblair: merger improvements to support zuul merging code instead of gerrit? | 00:43 |
jeblair | clarkb: not that yet -- | 00:43 |
jeblair | clarkb: we lost the git merge option that we used to have in ggp that causes cgit to use approximately the same strategy that gerrit uses | 00:43 |
jeblair | clarkb: so remember how markmc was complaining about how every few patches in his series, one would refuse to be merged by gerrit? | 00:44 |
clarkb | yeah | 00:44 |
clarkb | going to try and catch that early if possible? | 00:44 |
jeblair | clarkb: apparently when i moved the merge testing from ggp to zuul, i missed that option | 00:44 |
jeblair | clarkb: so zuul has been doing plain 'git merge'. | 00:44 |
clarkb | jeblair: https://review.openstack.org/#/c/42785/ if that could get reviewed early in the week that would be awesome | 00:44 |
jeblair | clarkb: so this should make zuul's merging closer to gerrit and kick things out earlier (resulting in fewer resets) | 00:45 |
jeblair | clarkb: secondarily, i have been looking into making the merger more efficient, but atm, a nova change takes a full minute to be enqueued. | 00:46 |
jeblair | clarkb: both of those should help with our load, but they are also not super-urgent, so that's why i want to try to get them in some time this week, but it's not a rush. | 00:46 |
clarkb | I see | 00:46 |
clarkb | making merges more efficient is a fun problem | 00:47 |
jeblair | i think we somehow ended up doing 'git remote update' twice for every change. | 00:47 |
jeblair | lots of opportunity for improvement here. :) | 00:47 |
*** sdake_ has quit IRC | 00:48 | |
*** dina_belova has joined #openstack-infra | 00:49 | |
jeblair | clarkb: i reviewed that change. i suggested to mrmartin that we go ahead and use a cloud db from the start for the user groups server | 00:50 |
jeblair | clarkb: basically because it looked like we were about ready to start doing backups of trove dbs (at least, we should be ready by the time we need it) | 00:51 |
clarkb | yes, I think so | 00:51 |
clarkb | we will just need to feed in credentials instead of using the defaults file | 00:51 |
openstackgerrit | A change was merged to openstack-infra/config: Make mysql backup crons quiet. https://review.openstack.org/42785 | 00:52 |
*** dims has joined #openstack-infra | 00:53 | |
*** dina_belova has quit IRC | 00:54 | |
openstackgerrit | A change was merged to openstack-infra/gear: fix project dependencies https://review.openstack.org/42194 | 00:59 |
jhesketh | Howdy. I'm having trouble with git-review. Primarily I'm trying to submit a change that depends on an already existing change but git-review insists on submitting them both and fails because the dependency has no new changes | 01:05 |
jhesketh | any ideas? | 01:05 |
clarkb | are you rebasing so that dependency gets a new sha? | 01:06 |
jhesketh | no | 01:07 |
jhesketh | although I could try that | 01:08 |
clarkb | if you don't have a new sha it shouldn't try pushing the dependency | 01:08 |
openstackgerrit | A change was merged to openstack-infra/devstack-gate: Conditionally run the slow heat tests https://review.openstack.org/43500 | 01:08 |
jeblair | you may have ended up with a new sha but no actual change | 01:08 |
jeblair | you might try rebasing on the original sha | 01:09 |
jeblair | (sorry have to run now) | 01:09 |
jhesketh | my sha matches gerrit's | 01:09 |
clarkb | jhesketh: can you paste the output of git log -2 and git review? | 01:11 |
*** michchap_ has joined #openstack-infra | 01:11 | |
jhesketh | clarkb: would you like that from my branch containing the problematic patchset or the one that depends on it? | 01:11 |
clarkb | assuming you are talking to review.o.o | 01:11 |
jhesketh | clarkb: yes | 01:11 |
clarkb | jhesketh: I would like it from your branch | 01:12 |
clarkb | jhesketh: sorry | 01:12 |
clarkb | jhesketh: I want it from whichever commit you are trying to push | 01:12 |
clarkb | the -2 should give me your commit and the dependency | 01:12 |
*** jamiem has quit IRC | 01:12 | |
jhesketh | clarkb: http://paste.openstack.org/show/45120/ | 01:13 |
jhesketh | I'm tempted to work around it by making a cosmetic change to generate a new sha1 | 01:13 |
*** jamiem has joined #openstack-infra | 01:14 | |
*** jamiem has quit IRC | 01:14 | |
*** michchap has quit IRC | 01:14 | |
clarkb | weird | 01:17 |
jhesketh | indeed | 01:18 |
jhesketh | clarkb: do you know how git-review tracks sha's with gerrit? | 01:19 |
jhesketh | is it creating refs somewhere? | 01:19 |
clarkb | it does a remote update I think | 01:19 |
clarkb | jhesketh: it will do a rebase for you it is possible it is doing that and it is causing problems (it will only do that if you conflict with the target branch) | 01:20 |
clarkb | jhesketh: give git review the -v argument and see if it does anything funny? | 01:20 |
jhesketh | perhaps but I rebased myself and it's uptodate | 01:20 |
jhesketh | I've also tried submitting with -R | 01:20 |
jhesketh | verbose doesn't show anything unusual | 01:21 |
jhesketh | I think I might just work around it for now | 01:22 |
jhesketh | that seemed to work | 01:24 |
jhesketh | I think I got into the mess somehow by trying to submit a new patchset for an abandoned change (without restoring it first) which had dependencies | 01:27 |
clarkb | pushing to an abandoned change won't work. that may have been the trouble | 01:28 |
jhesketh | yeah so the abandoned one failed but the dependency went up broken somehow | 01:29 |
*** Prince_Africanus has joined #openstack-infra | 01:33 | |
*** yaguang has joined #openstack-infra | 01:36 | |
openstackgerrit | A change was merged to openstack-infra/devstack-gate: Skip devstack/exercises by default https://review.openstack.org/42082 | 01:40 |
*** Prince_Africanus has quit IRC | 01:41 | |
*** sjing has joined #openstack-infra | 01:43 | |
*** dkehn has quit IRC | 01:46 | |
*** dina_belova has joined #openstack-infra | 01:50 | |
*** dina_belova has quit IRC | 01:55 | |
*** dkehn has joined #openstack-infra | 01:57 | |
*** sdake_ has joined #openstack-infra | 02:19 | |
*** sdake_ has joined #openstack-infra | 02:19 | |
*** erfanian has quit IRC | 02:26 | |
*** rcleere has joined #openstack-infra | 02:40 | |
*** dina_belova has joined #openstack-infra | 02:51 | |
*** dina_belova has quit IRC | 02:55 | |
*** xchu has joined #openstack-infra | 03:04 | |
*** dims has quit IRC | 03:05 | |
*** UtahDave has joined #openstack-infra | 03:07 | |
*** SergeyLukjanov has joined #openstack-infra | 03:19 | |
*** markmcclain has quit IRC | 03:43 | |
*** dina_belova has joined #openstack-infra | 03:51 | |
*** dina_belova has quit IRC | 03:55 | |
*** vogxn has joined #openstack-infra | 03:58 | |
*** afazekas has joined #openstack-infra | 04:10 | |
Alex_Gaynor | So, am I correct in thinking from here through September 4th, the pace of commits is going to continue to rise? | 04:18 |
*** dguitarbite has joined #openstack-infra | 04:18 | |
clarkb | Alex_Gaynor: definitely for the gate queue | 04:19 |
Alex_Gaynor | clarkb: has check queue peaked then? | 04:19 |
Alex_Gaynor | (I assume it's a plateau) | 04:19 |
clarkb | I think the major rush happened as people had to propse their changes. Now they will refine them through review so it will probably still be busy | 04:20 |
clarkb | Alex_Gaynor: http://tinyurl.com/kkgtr6z | 04:28 |
clarkb | I don't expect that same kind of week over week growth with next week | 04:28 |
clarkb | but will still be busy | 04:28 |
Alex_Gaynor | jesus, 11k jobs at hour, that's insane | 04:30 |
clarkb | Alex_Gaynor: thats per day | 04:30 |
clarkb | still crazy | 04:30 |
clarkb | did I not change the title? whoops | 04:30 |
Alex_Gaynor | it's considerably less insane :) down to "only" 500 jobs an hour on average | 04:31 |
*** dina_belova has joined #openstack-infra | 04:52 | |
*** dina_belova has quit IRC | 04:57 | |
*** sjing has quit IRC | 05:03 | |
*** sjing has joined #openstack-infra | 05:04 | |
*** Dr01d has joined #openstack-infra | 05:05 | |
*** nicedice_ has quit IRC | 05:09 | |
*** SergeyLukjanov has quit IRC | 05:13 | |
*** afazekas_ has joined #openstack-infra | 05:34 | |
*** afazekas has quit IRC | 05:47 | |
*** sdake_ has quit IRC | 05:51 | |
*** dina_belova has joined #openstack-infra | 05:52 | |
*** afazekas_ has quit IRC | 05:55 | |
*** dina_belova has quit IRC | 05:57 | |
*** xchu has quit IRC | 06:02 | |
jhesketh | jeblair: ping | 06:10 |
*** mikal has quit IRC | 06:11 | |
*** mikal has joined #openstack-infra | 06:13 | |
*** jfriedly has quit IRC | 06:13 | |
*** xchu has joined #openstack-infra | 06:18 | |
*** dmakogon__ has joined #openstack-infra | 06:35 | |
*** nayward has joined #openstack-infra | 06:36 | |
*** jpich has joined #openstack-infra | 06:39 | |
*** yolanda has joined #openstack-infra | 06:49 | |
*** dina_belova has joined #openstack-infra | 06:53 | |
*** dina_belova has quit IRC | 06:56 | |
*** UtahDave has quit IRC | 07:00 | |
*** jhesketh has quit IRC | 07:06 | |
*** jhesketh has joined #openstack-infra | 07:06 | |
*** hashar has joined #openstack-infra | 07:32 | |
*** vogxn has quit IRC | 07:34 | |
*** nayward has quit IRC | 07:38 | |
*** dina_belova has joined #openstack-infra | 07:53 | |
*** dina_belova has quit IRC | 07:59 | |
*** fbo_away is now known as fbo | 08:01 | |
*** shardy is now known as shardy_afk | 08:02 | |
*** afazekas has joined #openstack-infra | 08:04 | |
*** thomasbiege has joined #openstack-infra | 08:05 | |
*** thomasbiege has quit IRC | 08:10 | |
*** dguitarbite has quit IRC | 08:13 | |
*** sjing has quit IRC | 08:13 | |
*** sjing has joined #openstack-infra | 08:13 | |
*** derekh has joined #openstack-infra | 08:25 | |
*** thomasbiege has joined #openstack-infra | 08:30 | |
*** yolanda has quit IRC | 08:32 | |
*** vogxn has joined #openstack-infra | 08:32 | |
*** dguitarbite has joined #openstack-infra | 08:42 | |
*** xchu has quit IRC | 08:52 | |
*** dina_belova has joined #openstack-infra | 08:54 | |
*** dina_belova has quit IRC | 08:58 | |
*** xchu has joined #openstack-infra | 09:04 | |
*** DennyZhang has joined #openstack-infra | 09:24 | |
*** alexpilotti has joined #openstack-infra | 09:25 | |
*** ruhe has joined #openstack-infra | 09:29 | |
*** mikal has quit IRC | 09:32 | |
*** DennyZha` has joined #openstack-infra | 09:33 | |
*** sjing has quit IRC | 09:34 | |
*** DennyZhang has quit IRC | 09:35 | |
*** DennyZha` has quit IRC | 09:36 | |
*** jhesketh has quit IRC | 09:41 | |
*** thomasbiege has quit IRC | 09:41 | |
*** DennyZhang has joined #openstack-infra | 09:42 | |
*** vogxn has quit IRC | 09:45 | |
*** yolanda has joined #openstack-infra | 09:47 | |
*** thomasbiege has joined #openstack-infra | 09:47 | |
*** hashar has quit IRC | 09:48 | |
*** DennyZhang has quit IRC | 09:50 | |
*** DennyZhang has joined #openstack-infra | 09:50 | |
*** thomasbiege has quit IRC | 09:51 | |
*** dina_belova has joined #openstack-infra | 09:55 | |
*** dguitarbite has quit IRC | 09:55 | |
*** dina_belova has quit IRC | 09:59 | |
*** afazekas has quit IRC | 10:03 | |
*** vogxn has joined #openstack-infra | 10:03 | |
*** afazekas has joined #openstack-infra | 10:04 | |
*** fbo is now known as fbo_away | 10:05 | |
*** xchu has quit IRC | 10:07 | |
*** ruhe has quit IRC | 10:08 | |
*** ruhe has joined #openstack-infra | 10:10 | |
*** pcm_ has joined #openstack-infra | 10:11 | |
*** pcm_ has quit IRC | 10:11 | |
*** pcm_ has joined #openstack-infra | 10:11 | |
*** ruhe has quit IRC | 10:12 | |
*** yaguang has quit IRC | 10:22 | |
*** mikal has joined #openstack-infra | 10:22 | |
*** morganfainberg is now known as morganfainberg|a | 10:25 | |
*** afazekas_ has joined #openstack-infra | 10:27 | |
*** DennyZhang has quit IRC | 10:28 | |
*** afazekas has quit IRC | 10:28 | |
*** jhesketh has joined #openstack-infra | 10:34 | |
*** vogxn has quit IRC | 10:35 | |
*** dims has joined #openstack-infra | 10:40 | |
*** chuckieb has joined #openstack-infra | 10:45 | |
*** ruhe has joined #openstack-infra | 10:48 | |
*** dina_belova has joined #openstack-infra | 10:55 | |
*** dina_belova has quit IRC | 10:59 | |
*** sgviking has quit IRC | 11:01 | |
*** sgviking has joined #openstack-infra | 11:02 | |
*** weshay has joined #openstack-infra | 11:07 | |
*** weshay has quit IRC | 11:08 | |
*** weshay has joined #openstack-infra | 11:08 | |
*** dkranz has quit IRC | 11:10 | |
*** hashar has joined #openstack-infra | 11:14 | |
*** fbo_away is now known as fbo | 11:16 | |
*** lcestari has joined #openstack-infra | 11:20 | |
*** adalbas has quit IRC | 11:21 | |
*** weshay has quit IRC | 11:35 | |
*** rfolco has joined #openstack-infra | 11:39 | |
*** adalbas has joined #openstack-infra | 11:44 | |
*** lcestari has quit IRC | 11:46 | |
*** thomasbiege has joined #openstack-infra | 11:47 | |
*** mugsie has joined #openstack-infra | 11:49 | |
*** lcestari has joined #openstack-infra | 11:50 | |
mugsie | hi, just a quick question about the mailing list | 11:50 |
mugsie | how do we register as a topic for the openstack-dev mailing list? | 11:50 |
mugsie | (We were discussing it in #openstack-dns | 11:50 |
mugsie | )* | 11:50 |
*** apcruz has joined #openstack-infra | 11:54 | |
*** dina_belova has joined #openstack-infra | 11:56 | |
*** dina_belova has quit IRC | 12:00 | |
*** dprince has joined #openstack-infra | 12:01 | |
*** dims has quit IRC | 12:03 | |
*** dkranz has joined #openstack-infra | 12:13 | |
*** dolphm has joined #openstack-infra | 12:13 | |
*** jjmb has joined #openstack-infra | 12:20 | |
*** boris-42 has joined #openstack-infra | 12:20 | |
*** vogxn has joined #openstack-infra | 12:27 | |
*** thomasbiege has quit IRC | 12:29 | |
openstackgerrit | Dolph Mathews proposed a change to openstack/requirements: pin keyring to < 2.0 https://review.openstack.org/43690 | 12:31 |
openstackgerrit | Dolph Mathews proposed a change to openstack/requirements: pin keyring to < 2.0 https://review.openstack.org/43690 | 12:32 |
*** mikal has quit IRC | 12:37 | |
*** saper_ is now known as saper | 12:38 | |
*** mikal has joined #openstack-infra | 12:38 | |
*** mikal has quit IRC | 12:44 | |
*** mikal has joined #openstack-infra | 12:45 | |
*** woodspa has joined #openstack-infra | 12:46 | |
*** jamiem has joined #openstack-infra | 12:47 | |
*** sandywalsh has joined #openstack-infra | 12:52 | |
*** jamiem has quit IRC | 12:54 | |
*** dina_belova has joined #openstack-infra | 12:56 | |
*** beagles has quit IRC | 12:57 | |
*** weshay has joined #openstack-infra | 12:58 | |
*** beagles has joined #openstack-infra | 12:58 | |
*** anteaya has joined #openstack-infra | 12:59 | |
*** dina_belova has quit IRC | 12:59 | |
*** jergerber has joined #openstack-infra | 13:06 | |
*** jergerber has quit IRC | 13:06 | |
*** CaptTofu has joined #openstack-infra | 13:07 | |
*** jamiem has joined #openstack-infra | 13:07 | |
*** jergerber has joined #openstack-infra | 13:09 | |
*** odyssey4me has quit IRC | 13:10 | |
*** mriedem has joined #openstack-infra | 13:12 | |
anteaya | hi mugsie, to the best of my understanding, noone is registered as a topic in the openstack-dev mailing list. When the first post in a new thread is composed if no tags are in the subject [openstack-dev] is placed as a prefix in the subject and that is it. All other tags in a topic subject line are typed in by the originator of the first post. You will see here that both [glance] and [Glance] have been used: http://lists.op | 13:16 |
anteaya | enstack.org/pipermail/openstack-dev/2013-August/subject.html | 13:16 |
anteaya | link again: http://lists.openstack.org/pipermail/openstack-dev/2013-August/subject.html | 13:16 |
*** CliMz has joined #openstack-infra | 13:19 | |
*** CliMz has quit IRC | 13:22 | |
*** Bada has joined #openstack-infra | 13:22 | |
*** cppcabrera has joined #openstack-infra | 13:25 | |
*** cppcabrera has left #openstack-infra | 13:25 | |
*** jog0-away is now known as jog0 | 13:30 | |
*** dims has joined #openstack-infra | 13:31 | |
*** zul has quit IRC | 13:32 | |
*** yolanda has quit IRC | 13:32 | |
anteaya | we had very heavy thunderstorms lasting a surprising long time twice this weekend, forecast says chances of more today - if I disappear quickly, that would be me saving my electronics | 13:35 |
*** arezadr has joined #openstack-infra | 13:35 | |
*** blamar has joined #openstack-infra | 13:35 | |
*** CaptTofu has quit IRC | 13:35 | |
*** thomasbiege has joined #openstack-infra | 13:36 | |
*** CaptTofu has joined #openstack-infra | 13:36 | |
*** thomasbiege has quit IRC | 13:36 | |
*** thomasbiege has joined #openstack-infra | 13:37 | |
*** dina_belova has joined #openstack-infra | 13:37 | |
mugsie | anteaya: cheers! | 13:38 |
*** SergeyLukjanov has joined #openstack-infra | 13:38 | |
anteaya | mugsie: :D | 13:38 |
*** yolanda has joined #openstack-infra | 13:39 | |
*** CaptTofu has quit IRC | 13:40 | |
*** ruhe has quit IRC | 13:40 | |
*** jhesketh has quit IRC | 13:41 | |
*** hashar has quit IRC | 13:42 | |
*** hashar has joined #openstack-infra | 13:42 | |
*** burt has joined #openstack-infra | 13:42 | |
*** thomasbiege has quit IRC | 13:42 | |
*** dkehn is now known as dkehn_away | 13:43 | |
*** datsun180b has joined #openstack-infra | 13:44 | |
*** yolanda has quit IRC | 13:46 | |
*** hashar has quit IRC | 13:51 | |
openstackgerrit | David Kranz proposed a change to openstack-infra/config: Add slow heat job https://review.openstack.org/43511 | 13:51 |
*** HenryG_ has quit IRC | 13:52 | |
*** HenryG has joined #openstack-infra | 13:52 | |
jog0 | Random question: what do people think of moving all the git checks in hacking into a post-commit git hook? | 13:52 |
*** ruhe has joined #openstack-infra | 13:53 | |
jog0 | https://bugs.launchpad.net/hacking/+bug/1216925 | 13:55 |
uvirtbot | Launchpad bug 1216925 in hacking "Pull out git checks to work as post-commit git hook" [Undecided,New] | 13:55 |
*** dina_belova has quit IRC | 13:56 | |
*** rustlebee is now known as russellb | 13:56 | |
*** dkliban has joined #openstack-infra | 13:57 | |
anteaya | morning jog0 | 13:57 |
anteaya | I have no opinion | 13:57 |
*** michchap_ has quit IRC | 13:57 | |
*** michchap has joined #openstack-infra | 13:58 | |
jog0 | anteaya: morning, from the east coast | 13:58 |
anteaya | me too | 13:58 |
anteaya | :D | 13:58 |
dkranz | mtreinish: Never mind. I did not realize the zuul change was also in the config project. Submitted new patch. | 13:59 |
*** dims has quit IRC | 14:00 | |
*** rcleere has quit IRC | 14:02 | |
mtreinish | dkranz: did you want to add the heat slow job to heat also? | 14:04 |
*** yolanda has joined #openstack-infra | 14:04 | |
dkranz | mtreinish: Yes, once it is working. | 14:10 |
dkranz | mtreinish: But that gives us an asymetrical gate. | 14:10 |
dkranz | mtreinish: What if some change in nova breaks this job? | 14:10 |
dkranz | mtreinish: Perhaps it needs to go on all projects that are related in any way, or just all projects? | 14:11 |
mtreinish | dkranz: we should put it on all the projects one we switch it voting | 14:11 |
dkranz | mtreinish: We had a long discussion about this at the last summit and came down on the "gate everything" side I think. | 14:12 |
dkranz | mtreinish: OK, yes I agree. | 14:12 |
dkranz | mtreinish: I wanted to see it work on one project before putting on others. | 14:12 |
mtreinish | dkranz: I'd just put it nonvoting everywhere up front, then it makes the switch to voting patch a one liner | 14:13 |
mtreinish | also, why don't you just test it locally? | 14:14 |
dkranz | mtreinish: By "working" I mean flowing through the whole gate process. I don't know how to test that locally. | 14:15 |
openstackgerrit | Mark McLoughlin proposed a change to openstack/requirements: Allow use of oslo.messaging 1.2.0a11 https://review.openstack.org/43712 | 14:18 |
*** yolanda has quit IRC | 14:19 | |
dkranz | mkoderer: ping | 14:21 |
mkoderer | dkranz: pong | 14:21 |
dkranz | mkoderer: What's with line 36 of https://review.openstack.org/#/c/43444/3/tempest/stress/run_stress.py,unified | 14:22 |
mkoderer | hhhh | 14:22 |
dkranz | mkerrin: leftover from testing? | 14:23 |
mkoderer | that was for debugging | 14:23 |
mkoderer | yes :( | 14:23 |
dkranz | mkerrin: OK | 14:23 |
mkoderer | dkranz: give me a -1 I will fix it | 14:23 |
dkranz | mkerrin: Sorry, meant mkoderer | 14:23 |
*** hashar has joined #openstack-infra | 14:24 | |
*** rnirmal has joined #openstack-infra | 14:25 | |
mkoderer | dkranz: that's why reviews are always a good idea :) | 14:25 |
*** zul has joined #openstack-infra | 14:28 | |
*** michchap has quit IRC | 14:32 | |
anteaya | look at that, the gate is empty - wasn't expecting to see that today | 14:32 |
*** jpeeler has quit IRC | 14:33 | |
*** jpeeler has joined #openstack-infra | 14:33 | |
*** rcleere has joined #openstack-infra | 14:33 | |
*** ryanpetrello has joined #openstack-infra | 14:34 | |
openstackgerrit | A change was merged to openstack-dev/hacking: Update requirements https://review.openstack.org/43457 | 14:37 |
*** jpich has quit IRC | 14:39 | |
*** jpeeler has quit IRC | 14:40 | |
*** jpeeler has joined #openstack-infra | 14:41 | |
*** jpich has joined #openstack-infra | 14:41 | |
*** prad has joined #openstack-infra | 14:47 | |
*** dmakogon__ has quit IRC | 14:47 | |
*** prad has quit IRC | 14:48 | |
*** prad has joined #openstack-infra | 14:48 | |
*** dkehn_away is now known as dkehn | 14:51 | |
*** jpich has quit IRC | 14:53 | |
*** jpich has joined #openstack-infra | 14:55 | |
*** krtaylor has joined #openstack-infra | 14:56 | |
*** dina_belova has joined #openstack-infra | 14:57 | |
*** boris-42 has quit IRC | 14:59 | |
*** dguitarbite has joined #openstack-infra | 15:00 | |
*** CaptTofu has joined #openstack-infra | 15:00 | |
*** dina_belova has quit IRC | 15:01 | |
*** michchap has joined #openstack-infra | 15:02 | |
*** dina_belova has joined #openstack-infra | 15:07 | |
*** michchap has quit IRC | 15:10 | |
*** dina_belova has quit IRC | 15:11 | |
*** nati_ueno has joined #openstack-infra | 15:12 | |
*** nati_ueno has quit IRC | 15:14 | |
*** nati_ueno has joined #openstack-infra | 15:15 | |
*** Dr01d has quit IRC | 15:21 | |
*** Dr01d has joined #openstack-infra | 15:22 | |
*** afazekas_ has quit IRC | 15:22 | |
*** markmcclain has joined #openstack-infra | 15:24 | |
*** CaptTofu has quit IRC | 15:28 | |
*** CaptTofu has joined #openstack-infra | 15:29 | |
*** Bada has quit IRC | 15:30 | |
*** CaptTofu_ has joined #openstack-infra | 15:31 | |
*** CaptTofu has quit IRC | 15:31 | |
*** marun has joined #openstack-infra | 15:32 | |
*** jpich has quit IRC | 15:34 | |
*** _TheDodd_ has joined #openstack-infra | 15:36 | |
*** yaguang has joined #openstack-infra | 15:38 | |
*** rnirmal has quit IRC | 15:39 | |
*** rnirmal has joined #openstack-infra | 15:41 | |
jog0 | I am unable to create branches for hacking | 15:42 |
jog0 | any superadmins around that can help | 15:42 |
anteaya | pleia2: when you are around - does the current config with cgit use an .htaccess file? I can't seem to find one, but it is possible it exists and I have overlooked it. | 15:44 |
*** UtahDave has joined #openstack-infra | 15:46 | |
*** branen_ has joined #openstack-infra | 15:46 | |
*** jswarren has quit IRC | 15:48 | |
jog0 | jeblair: ^ | 15:49 |
*** zul has quit IRC | 15:50 | |
jeblair | jog0: a post commit hook is a fine idea, but local installation of commit hooks is voluntary. of course if you also run it in the style-check gate job, it's not a big deal if they don't run it locally. | 15:52 |
jeblair | jog0: what kind of branch do you need? | 15:52 |
jeblair | mugsie: at the bottom of http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev | 15:54 |
jeblair | mugsie: you will see "OpenStack-dev list run by thierry at openstack.org, stefano at openstack.org" | 15:54 |
openstackgerrit | Jason Meridth proposed a change to openstack-dev/hacking: Adds ability to ignore hacking validations with noqa https://review.openstack.org/41713 | 15:54 |
*** yaguang has quit IRC | 15:55 | |
jeblair | mugsie: so you can email them directly or email openstack-dev-owner@lists.openstack.org with your question about topics | 15:55 |
jog0 | jeblair: a jackign 0.7.x branch | 15:55 |
branen_ | Hi guys, I'm trying to get this global requirement approved. https://review.openstack.org/#/c/42991/ Any help would be greatly appreciated. | 15:56 |
jog0 | jeblair: hacking* | 15:56 |
jog0 | jeblair: starting from the 0.7.0 tag (sha1 6e1404cb221190645c7ef66e8d77e3402cba01eb ) | 15:56 |
jeblair | jog0: why does hacking need branches? | 15:56 |
jog0 | jeblair: and as for the post commit, cool | 15:56 |
jog0 | jeblair: because we cannot cut new 0.7.x releases from master as there is a new check already | 15:57 |
jog0 | jeblair: and to make sure we can bump dependencies, do bug fixes without forcing a new check on people | 15:57 |
jog0 | too often | 15:57 |
jeblair | jog0: ok, i think i understand the branch model | 15:59 |
jog0 | jeblair: cool, thanks | 15:59 |
jog0 | last time I tried to create a branch (0.5.x) I hit similar issues about not being able to create one | 15:59 |
jeblair | jog0: is hacking part of oslo? | 16:02 |
*** nati_ueno_2 has joined #openstack-infra | 16:03 | |
jog0 | jeblair: yes | 16:03 |
jog0 | jeblair: but I am not | 16:03 |
jog0 | but I am part of hacking | 16:03 |
*** thomasbiege has joined #openstack-infra | 16:03 | |
openstackgerrit | James E. Blair proposed a change to openstack-infra/config: Grant branch-creation to hacking-ptl https://review.openstack.org/43731 | 16:04 |
jeblair | jog0: ^ | 16:04 |
jog0 | that group doesn't exist | 16:06 |
*** nati_ueno has quit IRC | 16:06 | |
jeblair | jog0: nope, but it will when the change merges. we can add you and oslo-ptl then. | 16:06 |
jog0 | jeblair: ahh cool | 16:06 |
jog0 | so I am partof hacking-core so why couldn't i create a branch before? | 16:07 |
jeblair | jog0: the top line adds create branch perms; the other part of the change is just cleanup | 16:07 |
*** thomasbiege has quit IRC | 16:07 | |
jeblair | jog0: we like to keep tag permissions limited to a small group of people because it's easy to mess up pushing tags | 16:07 |
jeblair | jog0: and even if core==ptl today, that sort of thing should limit who you decide to give code review permission to | 16:08 |
*** dina_belova has joined #openstack-infra | 16:08 | |
jeblair | "shouldn't limit"! | 16:08 |
jog0 | jeblair: ahh makes sense to me | 16:08 |
jog0 | thanks | 16:08 |
openstackgerrit | A change was merged to openstack-infra/nodepool: Fix typo in node check method https://review.openstack.org/43595 | 16:12 |
openstackgerrit | A change was merged to openstack-infra/nodepool: Fix image delete logic https://review.openstack.org/43623 | 16:12 |
*** hashar has quit IRC | 16:12 | |
*** dina_belova has quit IRC | 16:13 | |
openstackgerrit | A change was merged to openstack-infra/config: Fix join/leave sounds in the conference https://review.openstack.org/42396 | 16:13 |
openstackgerrit | A change was merged to openstack-infra/config: Make oslo.messaging gating on devstack/tempest https://review.openstack.org/42613 | 16:15 |
anteaya | pleia2: nevermind I found the httpd.conf.erb file: https://git.openstack.org/cgit/openstack-infra/config/tree/modules/cgit/templates/httpd.conf.erb | 16:16 |
*** mikal has quit IRC | 16:17 | |
*** zul has joined #openstack-infra | 16:18 | |
*** mikal has joined #openstack-infra | 16:18 | |
*** nicedice_ has joined #openstack-infra | 16:19 | |
*** ruhe has quit IRC | 16:20 | |
*** cppcabrera has joined #openstack-infra | 16:24 | |
annegentle | clarkb: or jeblair or others who know how translations work? Around? | 16:25 |
annegentle | This patch definitely fails to do what we wanted: https://review.openstack.org/#/c/39846/ | 16:26 |
*** ruhe has joined #openstack-infra | 16:26 | |
annegentle | I have a hunch it's because the Post job is against the wrong repo | 16:26 |
annegentle | but now that I'm working on a patch, I'm not sure what pom.xml to point to. | 16:26 |
annegentle | I'll try a patch and see what you all think. | 16:26 |
*** ruhe has quit IRC | 16:26 | |
cppcabrera | Hey, guys. I was looking to get eavesdrop support in for the openstack-marconi channel. I have a patch pending that would address that: https://review.openstack.org/#/c/42956/ | 16:27 |
cppcabrera | Can I get approval/eyes on that? :) | 16:27 |
*** reed has joined #openstack-infra | 16:27 | |
jeblair | annegentle: if it's not urgent, yeah, why don't you propose a patch and lets look at it when clarkb is around | 16:27 |
annegentle | jeblair: sure sounds good. | 16:27 |
*** nati_ueno_2 has quit IRC | 16:29 | |
openstackgerrit | Anne Gentle proposed a change to openstack-infra/config: Update Japanese Ops Guide build job. https://review.openstack.org/43734 | 16:29 |
jeblair | cppcabrera: we'll get to that soon, we're working on a backlog with very few infra-core people around right now. but one pre-req for that is to give me founder access to the channel, at least until https://bugs.launchpad.net/openstack-ci/+bug/1190296 is fixed | 16:29 |
uvirtbot | Launchpad bug 1190296 in openstack-ci "IRC bot to manage official channel settings" [Medium,In progress] | 16:29 |
*** nati_ueno has joined #openstack-infra | 16:29 | |
annegentle | ^^ that's my first attempt. For sure the zuul layout has to change. But what I'm unsure of is whether there's a "generated" pom.xml file. | 16:29 |
uvirtbot | annegentle: Error: "^" is not a valid command. | 16:29 |
jeblair | cppcabrera: and you may want to add it to the channel list in the wiki if it isn't already | 16:29 |
jeblair | cppcabrera: /msg chanserv access #openstack-marconi add jeblair +AFRfiorstv | 16:30 |
cppcabrera | thanks, jeblair. Sounds like things are really busy atm! | 16:30 |
*** dolphm has quit IRC | 16:30 | |
jeblair | cppcabrera: yep | 16:30 |
pleia2 | anteaya: anything specific you were looking for that I can help with? | 16:32 |
*** moted has quit IRC | 16:32 | |
*** moted has joined #openstack-infra | 16:33 | |
anteaya | pleia2: well the blog post talked about using an .htaccess file to redirect requests for .rst files to .html output - I am trying to read how one does redirects and can pipe .html to output using the httpd.conf | 16:33 |
pleia2 | anteaya: gotcha, an .htaccess file is just a local mini version of the httpd.conf, designed so that web people without control over httpd.conf could still make *some* changes to apache via the .htaccess, so yeah it's all doable in httpd.conf and the syntax will be similar | 16:35 |
* pleia2 has a look at the blog post again | 16:36 | |
jeblair | jd__: i left a 0-score comment on https://review.openstack.org/#/c/42642/ let me know if you have questions | 16:37 |
anteaya | pleia2: grand, yes that was what I was understanding - thanks for your confirmation about .htaccess and httpd.conf | 16:37 |
cppcabrera | jeblair: Just made sure - openstack-marconi is on the channel list on the OS IRC wiki. I'll check with kgriffs later regarding getting you founder access on the channel. Thanks again! | 16:38 |
anteaya | I am just going over the httpd.conf file now - trying to find the right section to introduce a setting | 16:38 |
jd__ | jeblair: :-( | 16:38 |
pleia2 | anteaya: so you can do something like <Directory /var/lib/git> and put the rewrite rules within that </Directory> | 16:38 |
jd__ | jeblair: I'll build patches tomorrow for caching then! | 16:39 |
anteaya | pleia2: what section - global? virtual? I'm guessing virtual | 16:39 |
*** markmcclain has left #openstack-infra | 16:39 | |
jeblair | jd__: cool, it shouldn't be too hard. it's just a little scary because we don't have a good way of testing proposed patches to those. but i just had an idea of how we could do that (but probably won't be able to implement it for a while) | 16:39 |
* pleia2 has another look at the httpd.conf too :) | 16:40 | |
*** boris-42 has joined #openstack-infra | 16:41 | |
pleia2 | anteaya: actually, we probably want to put it in http://git.openstack.org/cgit/openstack-infra/config/tree/modules/cgit/templates/git.vhost.erb | 16:41 |
jd__ | jeblair: don't worry I'll as usual when working on infra: http://i.imgur.com/GNmZpEH.gif | 16:41 |
jeblair | jd__: :) | 16:42 |
jd__ | :-) | 16:42 |
*** dina_belova has joined #openstack-infra | 16:42 | |
*** pcrews has quit IRC | 16:42 | |
*** SergeyLukjanov has quit IRC | 16:43 | |
anteaya | pleia2: great, now that I have a proposed home file I can cobble together a rudimentary beginning and we can flesh it out | 16:43 |
*** dina_belova has quit IRC | 16:43 | |
*** ^d has joined #openstack-infra | 16:43 | |
*** ^d has joined #openstack-infra | 16:43 | |
pleia2 | anteaya: sounds good | 16:44 |
anteaya | pleia2: let me know how that .html file looks on cgit so we can figure out if using rst2html even gives us what we want: https://review.openstack.org/#/c/42961/ | 16:45 |
*** SergeyLukjanov has joined #openstack-infra | 16:45 | |
pleia2 | anteaya: ah yes, I'll toss this up in a repo in my test instance | 16:46 |
anteaya | thanks | 16:47 |
*** fbo is now known as fbo_away | 16:47 | |
cppcabrera | jeblair: kgriffs just let me know you've got founder access on openstack-marconi. :) | 16:49 |
clarkb | morning | 16:51 |
jeblair | clarkb: good morning | 16:51 |
*** SergeyLukjanov has quit IRC | 16:52 | |
anteaya | morning clarkb | 16:54 |
* anteaya makes some lunch | 16:54 | |
*** derekh has quit IRC | 16:57 | |
pleia2 | anteaya: just tossed it in a test repo here http://15.185.127.146/cgit/testing/project/tree/ | 17:01 |
anteaya | pleia2: that doesn't render .html for me | 17:04 |
pleia2 | anteaya: nope, it does if you click on "plain" though | 17:05 |
anteaya | that renders an .html file in plain text when I view it | 17:05 |
anteaya | ah | 17:05 |
* anteaya clicks | 17:05 | |
anteaya | can we have the plain setting as default for .html files? | 17:05 |
anteaya | how many people are going to know to click on plain? | 17:05 |
*** sdake has joined #openstack-infra | 17:07 | |
pleia2 | great question, I don't know :) | 17:07 |
anteaya | having to spend time telling people to click on plain - where on earth could we put that as a direction that is discoverable? - kind of defeats the purpose of the exercise | 17:07 |
pleia2 | yep | 17:07 |
anteaya | I wonder if time is better spend working on getting sphnix generated docs up and linking wiki articles to sphnix docs | 17:07 |
pleia2 | I haven't looked into the magic they use to make the About pages show up | 17:07 |
anteaya | seems longer, but the result is more effective | 17:08 |
*** Ryan_Lane has joined #openstack-infra | 17:08 | |
anteaya | there is a setting re: about pages I have found | 17:08 |
anteaya | let's see if I can find it again | 17:08 |
anteaya | https://git.openstack.org/cgit/openstack-infra/config/tree/modules/openstack_project/files/git/cgitrc#n44 | 17:09 |
anteaya | looks like it might be as simple as uncommenting out line #45 and provided said file | 17:09 |
*** sarob_ has joined #openstack-infra | 17:09 | |
*** leifmadsen has quit IRC | 17:11 | |
pleia2 | anteaya: yeah, so that enables it for the about page on each repo, I'm just not sure how that works internally and how/if we could make it work for other .html files that are floating around | 17:11 |
pleia2 | (and eventually, .rst) | 17:11 |
anteaya | yeah | 17:11 |
anteaya | would be awesome if we could | 17:11 |
*** dina_belova has joined #openstack-infra | 17:12 | |
openstackgerrit | A change was merged to openstack-infra/config: Grant branch-creation to hacking-ptl https://review.openstack.org/43731 | 17:12 |
anteaya | but since sphnix docs are part of the long game anyway, I will redirect to sphnix docs | 17:12 |
openstackgerrit | James E. Blair proposed a change to openstack-infra/config: Use "sort -V" for tags when building docs https://review.openstack.org/43739 | 17:12 |
anteaya | I have confidence I should get an acceptable result with the direction | 17:12 |
*** cppcabrera has left #openstack-infra | 17:12 | |
anteaya | but yeah, if we could figure out how cgit renders the about.html files, perhaps we can get it to render the other types too - across the repos | 17:13 |
anteaya | pleia2: any objection to me abandoning 42961 now? | 17:13 |
anteaya | I don't think we need it anymore | 17:13 |
anteaya | do you? | 17:13 |
*** mriedem has quit IRC | 17:14 | |
pleia2 | anteaya: go for it | 17:14 |
anteaya | thanks | 17:14 |
* anteaya abandons | 17:14 | |
*** Dr01d has quit IRC | 17:15 | |
clarkb | pleia2: jeblair and I would like to replace git.o.o with an 8GB node (down from 30GB) that only runs haproxy and doesn't do the git + cgit stuff | 17:16 |
*** sandywalsh has quit IRC | 17:16 | |
clarkb | pleia2: I can whip that up but was wondering if you had an opinion on whether or not we should keep haproxy stuff in the cgit module or just lift it into the openstack_projects manifest for git.openstack.org | 17:16 |
anteaya | okay, now lunch | 17:17 |
openstackgerrit | A change was merged to openstack-infra/config: Use "sort -V" for tags when building docs https://review.openstack.org/43739 | 17:18 |
pleia2 | clarkb: I'm thinking we put cgit back to being basic module for cgit and split out haproxy to openstack_projects | 17:20 |
jeblair | annegentle: can you weigh in on https://review.openstack.org/#/c/42988/2 | 17:20 |
jeblair | pleia2, clarkb: sgtm | 17:20 |
*** SergeyLukjanov has joined #openstack-infra | 17:22 | |
*** mriedem has joined #openstack-infra | 17:22 | |
*** dolphm has joined #openstack-infra | 17:28 | |
openstackgerrit | A change was merged to openstack-infra/jenkins-job-builder: Adding AnsiColor Support https://review.openstack.org/43121 | 17:30 |
*** sarob_ has quit IRC | 17:30 | |
*** sarob has joined #openstack-infra | 17:31 | |
*** vogxn has quit IRC | 17:31 | |
*** jfriedly has joined #openstack-infra | 17:31 | |
*** ruhe has joined #openstack-infra | 17:32 | |
*** sandywalsh has joined #openstack-infra | 17:32 | |
*** sarob_ has joined #openstack-infra | 17:34 | |
*** rfolco has quit IRC | 17:34 | |
*** pcrews has joined #openstack-infra | 17:34 | |
*** sarob has quit IRC | 17:35 | |
openstackgerrit | A change was merged to openstack-infra/config: Add the ability to filter Zuul Status on multiple (comma-delimited) terms. https://review.openstack.org/42382 | 17:37 |
*** sandywalsh has quit IRC | 17:38 | |
jeblair | clarkb: i don't understand your comment in https://review.openstack.org/#/c/43596/ | 17:41 |
*** dolphm has left #openstack-infra | 17:41 | |
jeblair | clarkb: (i've been using the new graph for a couple of days locally) | 17:41 |
openstackgerrit | Clark Boylan proposed a change to openstack-infra/config: Split git.o.o haproxy out of cgit module. https://review.openstack.org/43743 | 17:42 |
dkranz | jeblair: Thanks for the review of https://review.openstack.org/#/c/43511/ but you said there was an (unspecifiede) race. | 17:42 |
clarkb | jeblair: pleia2 ^ something like that? it needs testing, thankfully we need a new node node anyways so that fits right in | 17:42 |
dkranz | jeblair: Can you say what needs to be fixed? This is intended to go voting as soon as it works. | 17:42 |
jeblair | dkranz: i was responding to the comment about including both jjb and zuul in the same change | 17:42 |
clarkb | jeblair: I took the new string there and pasted it into the old jobs launched graph and got an image that said no data | 17:42 |
clarkb | jeblair: let me try again as I may have typoed something | 17:43 |
jeblair | dkranz: nothing needs to be fixed | 17:43 |
openstackgerrit | A change was merged to openstack-infra/config: Disable salt master and minions globally https://review.openstack.org/43030 | 17:43 |
dkranz | jeblair: Ah. So it would have been better to do it as two patches? | 17:43 |
jeblair | dkranz: zuul might get the new config before jjb, for a few minutes, in which case the job will be marked as LOST, but it won't affect anything because it is non-voting. | 17:43 |
dkranz | jeblair: OK, thanks. | 17:43 |
jeblair | dkranz: it's fine as one patch because it's non-voting; if it were voting, 2 patches would be better | 17:43 |
jeblair | clarkb: also compare to http://tinyurl.com/kg93m7r | 17:44 |
*** dprince has quit IRC | 17:45 | |
clarkb | jeblair: yup it works and matches your tinyurl | 17:45 |
*** vipul is now known as vipul-away | 17:46 | |
jeblair | clarkb: whew | 17:46 |
*** burt has quit IRC | 17:46 | |
*** sarob_ has quit IRC | 17:47 | |
*** sarob has joined #openstack-infra | 17:47 | |
*** sdake_ has joined #openstack-infra | 17:47 | |
*** sdake_ has joined #openstack-infra | 17:47 | |
pleia2 | clarkb: right now, what is the difference between git.o.o server and the others? (trying to figure out exactly what we ended up with in our current load balancing scheme) | 17:50 |
*** sandywalsh has joined #openstack-infra | 17:51 | |
clarkb | pleia2: right now git.o.o runs haproxy, apache, git-daemon and is synced to by gerrit. So the only difference is it layers haproxy over the top of everything | 17:52 |
pleia2 | clarkb: so are the other servers running haproxy at all? | 17:53 |
clarkb | pleia2: they are not | 17:54 |
pleia2 | or just straight up git/http(s) on the default ports | 17:54 |
clarkb | they run on git/http(s) on the non default ports | 17:55 |
pleia2 | ah ok, got it | 17:55 |
pleia2 | thanks | 17:55 |
*** dims has joined #openstack-infra | 17:57 | |
*** weshay has quit IRC | 18:01 | |
openstackgerrit | James E. Blair proposed a change to openstack-infra/config: Use stacked area test node graph https://review.openstack.org/43565 | 18:01 |
openstackgerrit | James E. Blair proposed a change to openstack-infra/config: Add sparklines to pipeline headers https://review.openstack.org/43567 | 18:01 |
openstackgerrit | James E. Blair proposed a change to openstack-infra/config: Use more efficient jobs/hour graph https://review.openstack.org/43596 | 18:01 |
*** sarob has quit IRC | 18:02 | |
*** sarob has joined #openstack-infra | 18:03 | |
*** morganfainberg|a has quit IRC | 18:04 | |
*** vipul-away is now known as vipul | 18:04 | |
clarkb | pleia2: I debated switching back to the default ports, but I think that should happen in a subsequent change if we do that | 18:05 |
clarkb | pleia2: fewer moving parts to juggle that way | 18:05 |
*** morganfainberg|a has joined #openstack-infra | 18:05 | |
*** morganfainberg|a is now known as morganfainberg | 18:06 | |
*** sarob has quit IRC | 18:07 | |
pleia2 | clarkb: given how flexible we are in the config with default/non-default ports (nice job!), I think it's ok to keep them non-default, but I agree | 18:07 |
*** ruhe has quit IRC | 18:10 | |
*** sdake_ has quit IRC | 18:10 | |
*** melwitt has joined #openstack-infra | 18:13 | |
reed | fungi, are you back on dry land? | 18:14 |
clarkb | reed: I think he is on the boat until Friday | 18:15 |
reed | I need help with solving this issue: https://bugs.launchpad.net/bugs/1172944 | 18:15 |
uvirtbot | Launchpad bug 1172944 in openstack-community "Misattributed commit" [High,New] | 18:15 |
reed | clarkb, lucky man... | 18:15 |
openstackgerrit | Tomas Sedovic proposed a change to openstack-infra/config: Add Jenkins jobs for Tuskar https://review.openstack.org/43461 | 18:16 |
reed | that issue is about matching identities across different systems | 18:17 |
reed | launchpad to gerrit, especially | 18:17 |
clarkb | yeah, I think there are two ways around this. The first is what fungi describes which requires elevated gerrit sql privs. Or just use the info in the git commit | 18:18 |
reed | clarkb, explain the git commit thing please, how would that work? | 18:19 |
clarkb | reed: each git commit has an author with a name and email address | 18:19 |
clarkb | reed: you could use that name and email address in an attempt to lookup people in launchpad | 18:19 |
clarkb | reed: the most reliable thing would be getting that user's openid from gerrit though | 18:19 |
clarkb | jeblair: ^ | 18:19 |
reed | what if launchpad user has no email address? | 18:20 |
clarkb | reed: fall back on name search? | 18:20 |
clarkb | the second option I describe isn't perfect | 18:20 |
reed | maybe we should just drop the connection to launchpad ... it's not that useful, we only use it for bugs and blueprints anyway | 18:21 |
*** dkehn has quit IRC | 18:21 | |
reed | i mean, if we find a match with the email then fine, if not... too bad | 18:21 |
jeblair | reed: yeah, we've tried really hard to make that connection as loose as possible because things were always breaking when it was tighter. | 18:22 |
jeblair | reed: as you'd probably expect, admin access isn't really an option. we could consider a periodic data dump (that's a bit of work and kind of fragile. also, note that it would likely take many hours, if not days, to actually run) | 18:23 |
reed | ok, thanks so far, I'll suggest JP to loosen up that connection and see how that goes | 18:23 |
jeblair | (those lp openid lookups are really slow) | 18:23 |
reed | jeblair, I knew it :) | 18:23 |
clarkb | pleia2: thank you for the review. I missed those things | 18:24 |
reed | and we're not going to rely on LP for OpenID for much longer hopefully anyway :) | 18:24 |
*** dkehn has joined #openstack-infra | 18:24 | |
clarkb | jeblair: did you catch that a couple zuul changes failed in the gate with python26 failures? | 18:29 |
*** zul has quit IRC | 18:31 | |
annegentle | jeblair: yep I'm good with the OpenSUSE addition for install. | 18:34 |
jeblair | clarkb: yes, i suspect it's because we're now actually running 26 tests and there may be race conditions in the tests | 18:35 |
openstackgerrit | Marton Kiss proposed a change to openstack-infra/config: Add groups-dev.openstack.org node https://review.openstack.org/42608 | 18:36 |
jeblair | clarkb: it's about #4 on my list of zuul things to look at | 18:36 |
jeblair | clarkb: see -dev | 18:39 |
*** dprince has joined #openstack-infra | 18:39 | |
clarkb | jeblair: mordred did that once. It should prevent the release jobs from running correctly but nothing will go sideways | 18:48 |
jeblair | clarkb: i believe it was released successfully | 18:49 |
jeblair | clarkb: it's very wrong though. git describe (and likely pbr) will not identify the correct version | 18:49 |
clarkb | hmm I wonder if that commit is available to gerrit somewhere then | 18:50 |
clarkb | jeblair: when mordred did it I believe the fix was to push an update to the tag | 18:50 |
jeblair | clarkb: it's in the repo; it got pushed with the tag | 18:51 |
jeblair | that's why pushing tags is so dangerous | 18:51 |
clarkb | jeblair: ya ok. When mordred did it the local commit was local only | 18:51 |
jeblair | and we only let people who, ideally, know what they are doing do it. | 18:51 |
jeblair | clarkb: i'm not sure how mordred could have pushed a tag without a commit | 18:51 |
jeblair | clarkb: it does not make sense for a repo to have a tag that points to a commit that is not in the repo | 18:51 |
jeblair | clarkb: maybe you're remembering something else? perhaps pushing an unsigned tag or something? or maybe the job failed for some other reason, since 'corrected'? | 18:52 |
clarkb | jeblair: I think pushing the tag pushes the commit | 18:52 |
jeblair | clarkb: i agree | 18:52 |
jeblair | clarkb: which is why markwash's error 'worked'. he pushed a commit that isn't in the repo. i imagine he could have pushed an entire tree if he wanted. | 18:53 |
clarkb | jeblair: so mordred's local commit ended up in gerrit with the tag. But g-g-p in the jobs couldn't checkout the tag because by default we don't fetch all tags | 18:53 |
jeblair | (that wasn't in the repo) | 18:53 |
clarkb | jeblair: we only fetch things reachalbe by the branches | 18:53 |
clarkb | which mordred's change was not | 18:53 |
jeblair | clarkb: that sounds likely; i agree. i wonder why it could check this out. | 18:54 |
jeblair | clarkb: perhaps because it was cloned | 18:54 |
jeblair | clarkb: that was the case for the docs job | 18:54 |
clarkb | That may do it | 18:54 |
jeblair | we really need code review for tags | 18:55 |
jeblair | people are just not able to follow directions | 18:55 |
*** mrmartin has joined #openstack-infra | 18:55 | |
*** xBsd has joined #openstack-infra | 18:55 | |
*** rfolco has joined #openstack-infra | 18:56 | |
*** burt has joined #openstack-infra | 18:58 | |
clarkb | ++ | 19:00 |
*** vipul is now known as vipul-away | 19:02 | |
*** pentameter has joined #openstack-infra | 19:04 | |
*** hashar has joined #openstack-infra | 19:05 | |
openstackgerrit | Clark Boylan proposed a change to openstack-infra/config: Split git.o.o haproxy out of cgit module. https://review.openstack.org/43743 | 19:06 |
clarkb | pleia2: ^ that addresses your comments. Tahnks | 19:06 |
jog0 | how can I create a new devstack test, where the only change is the config? | 19:07 |
jog0 | the nova.conf file taht is | 19:07 |
clarkb | jog0: nova.conf is updated by devstack. If the update you want to make is already supported by devstack then all you need is an update to devstack-gate to toggle that change and a new job that toggles devstack-gate | 19:09 |
clarkb | jog0: if devstack does not already support that toggle I think you need to update devstack as well | 19:09 |
*** nati_ueno has quit IRC | 19:09 | |
jog0 | clarkb: right, how do I add a new job? | 19:09 |
*** nati_ueno has joined #openstack-infra | 19:09 | |
clarkb | jog0: edit openstack-infra/config/modules/openstack_project/files/jenkins_job_builder/config/devstack.yaml | 19:10 |
clarkb | jog0: http://ci.openstack.org/jjb.html describes what goes into those files and has links into more in depth info | 19:10 |
clarkb | jog0: then update openstack-infra/config/modules/openstack_project/files/zuul/layout.yaml to tell zuul to run the job. http://ci.openstack.org/zuul.html has info on that and links into more info | 19:11 |
*** sarob has joined #openstack-infra | 19:11 | |
jog0 | clarkb: awesome thanks | 19:11 |
jog0 | I have two new checks i want to run for nova | 19:11 |
jog0 | one is should already be green (spin up 25 vms at once) | 19:12 |
jog0 | the other will fail at first | 19:12 |
clarkb | do they need to be new jenkins jobs or can they just run as part of existing tests? | 19:12 |
clarkb | I suppose something that isn't passing can't run as part of the gate but the passings bits could | 19:12 |
*** sarob has quit IRC | 19:12 | |
*** sarob has joined #openstack-infra | 19:13 | |
*** vipul-away is now known as vipul | 19:13 | |
jog0 | clarkb: they both need seperate nova.conf values so AFAIK we need different jobs | 19:13 |
jog0 | anyway will do this right after lunch | 19:14 |
*** nati_ueno has quit IRC | 19:14 | |
clarkb | jog0: you can't restart nova after tempest runs? | 19:14 |
clarkb | with the new config? | 19:14 |
clarkb | you might find the post test hook in devstack-gate useful as well | 19:15 |
jog0 | clarkb: that could work, be back in 30 | 19:15 |
*** sarob has quit IRC | 19:17 | |
*** nati_ueno has joined #openstack-infra | 19:19 | |
openstackgerrit | Anita Kuno proposed a change to openstack-infra/config: Creating/adding the openstack/governance repository https://review.openstack.org/43002 | 19:19 |
*** sdake_ has joined #openstack-infra | 19:22 | |
*** sdake_ has quit IRC | 19:22 | |
*** sdake_ has joined #openstack-infra | 19:22 | |
*** nati_ueno has quit IRC | 19:23 | |
*** nati_ueno has joined #openstack-infra | 19:24 | |
*** fbo_away is now known as fbo | 19:25 | |
*** nati_ueno_2 has joined #openstack-infra | 19:26 | |
*** nati_ueno has quit IRC | 19:27 | |
*** sarob has joined #openstack-infra | 19:27 | |
*** mrodden has quit IRC | 19:28 | |
*** mrodden has joined #openstack-infra | 19:28 | |
*** nati_ueno_2 has quit IRC | 19:31 | |
*** nati_ueno has joined #openstack-infra | 19:31 | |
*** nati_ueno has quit IRC | 19:35 | |
*** morganfainberg has left #openstack-infra | 19:38 | |
*** SergeyLukjanov has quit IRC | 19:38 | |
*** morganfainberg has joined #openstack-infra | 19:38 | |
*** afazekas has joined #openstack-infra | 19:39 | |
*** Dr01d has joined #openstack-infra | 19:39 | |
jog0 | clarkb: I think at least for now keeping the test seperate will make things easier | 19:45 |
*** nati_ueno has joined #openstack-infra | 19:46 | |
*** nati_ueno has quit IRC | 19:47 | |
*** nati_ueno has joined #openstack-infra | 19:48 | |
*** redbeard2 has joined #openstack-infra | 19:51 | |
anteaya | I let the salt guys know about: https://review.openstack.org/#/c/43030/ which disables salt masters and minions - they know we are busy and are trying to see if they can get a better sense of the issue so they can offer some support | 19:51 |
anteaya | jeblair or clarkb if either of you could run `salt --versions-report` I think they would be grateful to access the output - if we can still run it if they are disabled | 19:52 |
*** nati_ueno has quit IRC | 19:52 | |
jeblair | anteaya: ok; we're not really frustrated or anything, it's mostly that we don't have a lot of time to be good users (if i were to ask for help or report a problem, i'd at least want to be able to articulate the problem better and have thought about it a little myself) | 19:53 |
jeblair | anteaya: on the master? | 19:53 |
anteaya | redbeard2: on the master? | 19:54 |
anteaya | jeblair: yes that was my sense of it and what I tried to convey | 19:54 |
jeblair | http://paste.openstack.org/show/45161/ | 19:54 |
anteaya | thanks | 19:54 |
redbeard2 | anteaya and jeblair: both master and minion is ideal | 19:55 |
redbeard2 | oh, 0.15.3, that's a little old | 19:55 |
anteaya | redbeard2: what is current? | 19:55 |
redbeard2 | i think we're on 0.16.3 or 0.16.4 right now | 19:55 |
redbeard2 | UtahDave ^ | 19:56 |
UtahDave | jeblair: Yeah, 0.16.3 is the latest. and we're getting ready to cut 0.16.4 | 19:57 |
openstackgerrit | Antoine Musso proposed a change to openstack-infra/jenkins-job-builder: Revert "Adding AnsiColor Support" https://review.openstack.org/43771 | 19:58 |
*** svarnau has joined #openstack-infra | 19:59 | |
UtahDave | jeblair: and zmq 2.2.0 is basically just broken | 19:59 |
*** harlowja has quit IRC | 19:59 | |
*** harlowja has joined #openstack-infra | 20:00 | |
*** xBsd has quit IRC | 20:04 | |
*** wenlock has joined #openstack-infra | 20:06 | |
*** vipul is now known as vipul-away | 20:06 | |
clarkb | jog0: the reason I ask is we do have an upper bound on the number of slaves available at any one time and he seem to hit that periodically. So two new tests per nova patchset takes us from 7 d-g slaves to 9 | 20:09 |
jog0 | clarkb: ahh | 20:11 |
jog0 | can we just run these periodically on trunk for now? | 20:11 |
clarkb | we can do that | 20:11 |
jog0 | well one of them is tempest.scenario.test_large_ops.TestLargeOpsScenario.test_large_ops_scenario | 20:11 |
jog0 | in tempest | 20:11 |
jog0 | and that can be periodic | 20:11 |
jog0 | at least for now | 20:11 |
openstackgerrit | Stefano Maffulli proposed a change to openstack-infra/config: Added list for Track Chairs Committee for Summit Nov 2013 https://review.openstack.org/43773 | 20:11 |
jog0 | and the other I want to add is a compute rpcapi backwards compat test | 20:12 |
jeblair | clarkb: when would we make them gating? | 20:14 |
clarkb | jeblair: if one is a tempest test it can be made gating in the same way other tempest tests are added to the mix | 20:15 |
clarkb | inless that is the one that requires a different nova config | 20:15 |
jeblair | clarkb: you may have misunderstood my question; by what criteria would one determine when it should be gating as opposed to periodic? | 20:16 |
jeblair | clarkb: and yes, that is the one that requires a different nova config | 20:16 |
clarkb | jeblair: when it passes regularly | 20:16 |
clarkb | jeblair: but I think we need to get away from the practice of non voting check tests for random things that don't pass | 20:17 |
jeblair | clarkb: then silent or the experimental pipeline would be good choices? | 20:17 |
clarkb | jeblair: I like mordred's experimental queue idea | 20:17 |
jeblair | what happened to that change? | 20:17 |
clarkb | jeblair: good question. I know it needed work, maybe it got auto abandoned? | 20:17 |
jeblair | clarkb: i think i mostly agree with you; non-voting should be the last thing that happens before it becomes voting; check shouldn't end up being a test graveyard -- but neither should periodic | 20:18 |
clarkb | jeblair: I agree | 20:18 |
clarkb | let me find the experimental queue change and resurrect it if necessary | 20:19 |
jeblair | clarkb: here it is https://review.openstack.org/#/c/41954/ | 20:19 |
clarkb | jeblair: I restored it | 20:20 |
jeblair | clarkb: so let's say expiremental is for "i'm trying to get this thing working", silent is for "i think it works but it needs lots of testing to get data", check/gate non-voting is for "this is about to gate" | 20:21 |
clarkb | ++ | 20:21 |
anteaya | UtahDave tells me zmq 2.2.0 had memory leak issues | 20:25 |
jog0 | clarkb: one more question, i don't want to run all of tempest just 'tox -eall tempest.scenario.test_large_ops' | 20:26 |
jog0 | and change a the tempest config | 20:26 |
anteaya | after feature freeze is over, he says there will be more salt support in channel | 20:26 |
anteaya | I suggested after feature freeze | 20:26 |
anteaya | s/had/has | 20:27 |
*** mrmartin has quit IRC | 20:27 | |
anteaya | he said once zmq is on 3.2.x most of those issues go away | 20:27 |
clarkb | jog0: you may need to write your own test running hook for d-g then. As by default it will do the regular tempest stuff | 20:27 |
clarkb | jog0: in devstack.yaml the swift functional and pbr install tests do this | 20:27 |
openstackgerrit | James E. Blair proposed a change to openstack-infra/zuul: Fetch specific refs on ref-updated events https://review.openstack.org/43775 | 20:28 |
jog0 | clarkb: thanks | 20:28 |
jeblair | jog0: unless there's a tox env in tempest for it? | 20:28 |
jeblair | (i'd lean toward putting that in devstack-gate so it's self | 20:29 |
jeblair | -gating) | 20:29 |
jog0 | 'tox -eall tempest.scenario.test_large_ops' | 20:30 |
* anteaya goes for a walk | 20:30 | |
jeblair | and so it fits in with the rest of the devstack-gate tempest tests | 20:30 |
*** nati_ueno has joined #openstack-infra | 20:30 | |
jog0 | jeblair: it needs a seperate nova.conf (fake virt driver) | 20:30 |
*** weshay has joined #openstack-infra | 20:31 | |
jeblair | jog0: i know, i'm just saying that if this is going to run alongside the other 6 or so devstack tempest tests, it would be nice if it didn't have a completely different way of being run | 20:31 |
jeblair | jog0: so you can add the 'tox -eall tempest.scenario.test_large_ops' to devstack-gate | 20:31 |
hashar | I am on vacations for 3 weeks and you guys managed to colorize the success,failure and lost messages reported in Gerrit… crazy :) | 20:31 |
jog0 | jeblair: it can be done with a 'unstack; change localrc; stack; tempest tox | 20:31 |
clarkb | hashar: jeblair has been really busy. I don't know how he does it | 20:31 |
jeblair | jog0: that's not what i'm trying to say | 20:32 |
jeblair | jog0: i'm saying that clarkb suggested that you launch the test in a way that is completely different than the way the other devstack-gate tempest tests are launched, and i'm not certain it's appropriate | 20:32 |
jeblair | jog0: the heat slow tests are probably a good pattern | 20:33 |
dkranz | clarkb: Any chance you could approve https://review.openstack.org/#/c/43511/ to get the heat jobs going? | 20:33 |
jeblair | speaking of :) | 20:33 |
jog0 | I was looking at cells code as a reference but will checkou t the heat code | 20:34 |
jog0 | jeblair: thanks | 20:34 |
clarkb | dkranz: jeblair: 43511 reopens the experimental vs silent vs non voting discussion | 20:34 |
jeblair | jog0: cells works too; they both do the same thing | 20:34 |
clarkb | I don't think that test belongs there unless we expect to make it voting in the next couple weeks | 20:35 |
hashar | clarkb: I suspect some javascript voodoo :D | 20:35 |
*** nati_ueno has quit IRC | 20:35 | |
jeblair | clarkb: if we had an alternative in place, i might agree with you. | 20:35 |
clarkb | jeblair: I can rebase and cleanup mordred's change if you want to try and get experimental pipeline in soon | 20:36 |
jeblair | clarkb: keep in mind, heat is just being added to tempest | 20:36 |
jeblair | hashar: actually, just a gerrit commentlink and some css | 20:36 |
jeblair | hashar: (and a teeny change to zuul) | 20:36 |
hashar | I thought Gerrit was escaping html | 20:37 |
jog0 | dumb question | 20:37 |
jeblair | clarkb: i'd be happy to merge that if you get it in shape. | 20:37 |
jog0 | for https://review.openstack.org/#/c/43511/2/modules/openstack_project/files/jenkins_job_builder/config/devstack-gate.yaml | 20:37 |
*** vipul-away is now known as vipul | 20:37 | |
clarkb | jeblair: great. Let me see what I can do here really quick | 20:37 |
dkranz | clarkb: The intent is to turn it voting as soon as it is demonstrated to be working. | 20:37 |
jeblair | hashar: it does! and the commentlink actually matches the generated html. it's crazy. :) | 20:37 |
jog0 | how does that run heat slow tests? | 20:38 |
clarkb | dkranz: I understand, but that is the intent with the other non passing jobs as well | 20:38 |
jeblair | jog0: corresponding change to devstack-gate repo | 20:38 |
jog0 | jeblair: ohh | 20:38 |
dkranz | jog0: And see the heat section in https://github.com/openstack/tempest/blob/master/tox.ini | 20:38 |
hashar | jeblair: smart trick :-] | 20:39 |
dkranz | clarkb: The other non-passing jobs are there because no one has yet fixed them | 20:39 |
dkranz | clarkb: I think we need to push these new heat tests through the infrastructure before letting them block merges. No? | 20:40 |
dkranz | clarkb: that's what we did with parallel testr as well. | 20:41 |
clarkb | dkranz: we do, but we shouldn't be running them on every patchset by default | 20:41 |
dkranz | clarkb: I'm not sure what you mean | 20:41 |
clarkb | dkranz: mordred has proposed an experimental queue that would make it easy to push tests through on specific patchsets with manual nudging. So that you could run the tests on patchsets that address test issues | 20:41 |
clarkb | dkranz: then when the problems are sorted out we can promote you to non voting then gating | 20:42 |
clarkb | dkranz: the trouble with non voting by default is we have in the past run a ton of tests that do nothing for us | 20:42 |
dkranz | clarkb: So there are a bunch of heat tests waiting on this patch to run. What do you suggest we do? | 20:43 |
clarkb | dkranz: I am fixing mordred's change to add an experimental queue so that you can run your tests there | 20:43 |
dkranz | clarkb: OK. Is this something general that you will announce to the dev list? | 20:44 |
dkranz | mtreinish: ^^^^ | 20:45 |
*** weshay has quit IRC | 20:45 | |
*** Dr01d has quit IRC | 20:45 | |
*** emagana has joined #openstack-infra | 20:48 | |
marun | lifeless: Hoping you can take a look at this review that adds postmortem support to neutron's tests: https://review.openstack.org/#/c/43776/ | 20:49 |
marun | lifeless: Please let me know if you think this is the way to go or whether the functionality should be added to testtools directly. | 20:50 |
*** amotoki has joined #openstack-infra | 20:55 | |
*** nicedice__ has joined #openstack-infra | 20:56 | |
*** nati_ueno has joined #openstack-infra | 20:57 | |
openstackgerrit | Clark Boylan proposed a change to openstack-infra/config: Move non-voting jobs to an experimental queue https://review.openstack.org/41954 | 20:58 |
clarkb | dkranz: jeblair ^ | 20:58 |
clarkb | dkranz: I don't know that it needs an announcement to -dev but it can definitely be documented somewhere | 20:58 |
*** nicedice_ has quit IRC | 21:00 | |
openstackgerrit | Clark Boylan proposed a change to openstack-infra/config: Move non-voting jobs to an experimental queue https://review.openstack.org/41954 | 21:01 |
dkranz | clarkb: So how do you invoke a job in this experimental queue? | 21:02 |
*** rfolco has quit IRC | 21:02 | |
clarkb | dkranz: you leave a comment with "check experimental" as the only content in the review | 21:02 |
clarkb | dkranz: that string can be modified and is on line 83 of the layout.yaml file in my change | 21:03 |
dkranz | clarkb: OK, works for me. | 21:03 |
*** boris-42 has quit IRC | 21:03 | |
jeblair | clarkb: can you add the instructions to the pipeline description? | 21:04 |
clarkb | jeblair: yes | 21:04 |
*** dprince has quit IRC | 21:04 | |
*** apcruz has quit IRC | 21:04 | |
openstackgerrit | Clark Boylan proposed a change to openstack-infra/config: Move non-voting jobs to an experimental queue https://review.openstack.org/41954 | 21:05 |
dkranz | jeblair, clarkb : I will resubmit the heat patch with experimental as soon as 41954 goes in. | 21:05 |
openstackgerrit | Joe Gordon proposed a change to openstack-infra/devstack-gate: Add support for large_ops tempest test https://review.openstack.org/43779 | 21:05 |
jog0 | first of my patches for a new tests: https://review.openstack.org/#/c/43778/ | 21:06 |
*** pcrews has quit IRC | 21:06 | |
dkranz | jeblair: BTW, I could not figure out from the docs how to decide whether some job should go in check, gate, or both. Is this explained somewhere? | 21:07 |
dkranz | jeblair: I couldn't tell from the examples in the yaml file either. | 21:07 |
jeblair | dkranz: they should be 1:1, especially now with experimental | 21:07 |
dkranz | jeblair: Well, that's easy :) | 21:08 |
openstackgerrit | A change was merged to openstack-infra/config: Move non-voting jobs to an experimental queue https://review.openstack.org/41954 | 21:08 |
jeblair | dkranz: pretty much at this point the only jobs that should be in check but not gate are informative ones, like we have a job that tells us if a change alters the jenkins configuration | 21:08 |
jeblair | dkranz: but that's not something we gate on (sometimes it is okay to alter the jenkins config!) | 21:08 |
openstackgerrit | Khai Do proposed a change to openstack-infra/gear: Update gear docs to include gearman server daemon https://review.openstack.org/43780 | 21:09 |
jeblair | dkranz: there should never be a gate job that isn't also in check | 21:09 |
openstackgerrit | Joe Gordon proposed a change to openstack-infra/config: Add gate-tempest-devstack-vm-large-ops-full job https://review.openstack.org/43781 | 21:09 |
*** pcrews has joined #openstack-infra | 21:09 | |
jog0 | jeblair: here are the patches https://review.openstack.org/#/q/status:open++branch:master+topic:bug/1199433,n,z | 21:10 |
jog0 | clarkb: ^ | 21:10 |
reed | jeblair, pleia2: modified https://review.openstack.org/#/c/42998/2, when you have time please give me feedback | 21:12 |
clarkb | jog0: d-g change lgtm. The openstack-infra/config change is mostly there (see comments). I can give you a +1 on the devstack change but do not have core there | 21:15 |
jog0 | clarkb: thanks | 21:15 |
clarkb | jeblair: I am going to noop apply https://review.openstack.org/#/c/43743/ to the existing git nodes to make sure that I didn't miss anything too horrible and if that is mostly clean next step is spinning up a new git.o.o with the development env | 21:17 |
jeblair | clarkb: roger; i'll try to review it soon | 21:20 |
*** dina_belova has quit IRC | 21:20 | |
clarkb | jeblair: https://bugs.launchpad.net/openstack-ci/+bug/1217097 | 21:21 |
uvirtbot | Launchpad bug 1217097 in openstack-ci "LaunchPad Core developer lists are out of date" [Undecided,Confirmed] | 21:21 |
*** dina_belova has joined #openstack-infra | 21:21 | |
ttx | clarkb: that's a dupe | 21:21 |
ttx | for 1160277 | 21:22 |
clarkb | ttx: thanks. I will update | 21:22 |
ttx | bug 1160277 | 21:22 |
uvirtbot | Launchpad bug 1160277 in openstack-ci "Groups have similar names in LP and gerrit but are no longer synced" [Medium,In progress] https://launchpad.net/bugs/1160277 | 21:22 |
clarkb | done | 21:22 |
ttx | I should just jump and remove them, but I keep using them as subscriptoin groups for private bugs | 21:23 |
ttx | we should actually replace them with a ~nova-dev-interested-in-security group | 21:23 |
clarkb | ttx: Maybe update the group descriptions? | 21:23 |
clarkb | or that | 21:24 |
clarkb | I think that the only bug here is one of documentation | 21:24 |
ttx | because subscribing PROJ-core is not really efficient. Lost in the noise | 21:24 |
openstackgerrit | Joe Gordon proposed a change to openstack-infra/config: Add gate-tempest-devstack-vm-large-ops-full job https://review.openstack.org/43781 | 21:24 |
ttx | clarkb: last time I looke we could not remove because quantum-core had a ML and LP is picky about removing groups with MLs | 21:25 |
ttx | but now the ML should be removed and fogotten (takes a few days/weeks to remove) | 21:25 |
ttx | so will fix it when I come back | 21:25 |
*** dina_belova has quit IRC | 21:25 | |
jog0 | clarkb: ^ I have a hunch I still borked something there | 21:26 |
mtreinish | dkranz: one sec reading backlog | 21:26 |
*** nati_ueno has quit IRC | 21:28 | |
*** nati_ueno has joined #openstack-infra | 21:28 | |
*** dkranz has quit IRC | 21:28 | |
clarkb | jog0: that is really close. Just one little thing noted inline | 21:29 |
openstackgerrit | Joe Gordon proposed a change to openstack-infra/config: Add gate-tempest-devstack-vm-large-ops-full job https://review.openstack.org/43781 | 21:30 |
jog0 | clarkb: thanks | 21:30 |
clarkb | jeblair: the git change is relatively clean with --noop. No errors and only expected changes happen | 21:32 |
mtreinish | clarkb: now that we've got an experimental queue how about we push this in: https://review.openstack.org/#/c/41797/ and drop the testr-full job too :) | 21:32 |
*** mriedem has quit IRC | 21:32 | |
clarkb | jeblair: is it possible to nova boot a second git.openstack.org in the same AZ? | 21:33 |
clarkb | mtreinish: wfm | 21:34 |
clarkb | ttx: ^ do you have an opinion? | 21:34 |
ttx | clarkb: missing a bit of context | 21:35 |
ttx | $ | 21:35 |
clarkb | ttx: mtreinish is proposing we run most tempest tests with testr an run them in parallel | 21:35 |
mtreinish | all but neutron | 21:36 |
ttx | clarkb: right, was wondering about the "experimental queue" | 21:36 |
clarkb | ttx: I think that is just noise for this particular discussion | 21:36 |
anteaya | ttx if you want to discuss the goverance repo patch in tc meeting tomorrow: https://review.openstack.org/#/c/43002/4 | 21:36 |
clarkb | ttx: the experimental queue gives us a much nicer place to develop new tests with low impact against jenkins and zuul | 21:37 |
ttx | clarkb: it's the last window to do it. I don't think we'll have the freedom to get creative afterwards (untli FF is done) | 21:37 |
clarkb | ttx: tempest testr is beyond that point and is ready to go into gating | 21:37 |
clarkb | jeblair: any opinion on merging tempest testr? | 21:38 |
ttx | clarkb: so if infra feels comfortable with it, I'd say go for it | 21:38 |
clarkb | I am comfortable with it. Let us see what jeblair has to say | 21:38 |
clarkb | jog0: one more comment. I missed it earlier sorry | 21:39 |
*** reed has quit IRC | 21:41 | |
openstackgerrit | Joe Gordon proposed a change to openstack-infra/config: Add gate-tempest-devstack-vm-large-ops job https://review.openstack.org/43781 | 21:42 |
*** thomasbiege has joined #openstack-infra | 21:42 | |
mtreinish | jog0: on https://review.openstack.org/#/c/43779 why don't we just add a large ops tox job to tempest | 21:43 |
jog0 | mtreinish: we can | 21:44 |
*** nati_uen_ has joined #openstack-infra | 21:44 | |
mtreinish | jog0: ok, lets do it that way then. I like having a separate tox job for each jenkins job. | 21:45 |
jeblair | clarkb, mtreinish: i defer to the tempest folks on that; i have been preoccupied and haven't kept up with the stats; and i am less likely to be called upon to help devs if it's flaky | 21:45 |
*** nati_ueno has quit IRC | 21:46 | |
mtreinish | although it does add a third patch to that series of getting it into zuul | 21:46 |
jog0 | mtreinish: thats why I didn't do it | 21:46 |
clarkb | jeblair: that is how I feel. From the infrastructures perspective I don't think there is any reason not to now that we are mostly stable | 21:46 |
jog0 | but happt to do it if you want | 21:46 |
clarkb | mtreinish: ^ | 21:46 |
mtreinish | jeblair, clarkb: I think it's as good as it's going to get right now with just me looking at things | 21:46 |
mtreinish | there are still a couple of flaky fails I've seen but there're not that common | 21:47 |
mtreinish | and there is a requirments pinning that should fix the most common one | 21:47 |
mtreinish | but the flaky rate will temporarily increase a bit | 21:47 |
jeblair | mtreinish: ok. you know the drill, flaky will get us into trouble around the milestone, but if you think you can wrangle people to help fix the remaining issues, it's probably worth it. | 21:48 |
*** nati_ueno has joined #openstack-infra | 21:48 | |
jeblair | mtreinish: i suppose this can always be fairly easily backed out too. :) | 21:48 |
*** nati_uen_ has quit IRC | 21:48 | |
clarkb | jeblair: will the launch script allow me to boot a second git.openstack.org? | 21:49 |
clarkb | jeblair: or do I need to use some other name then change it over later? | 21:49 |
mtreinish | jeblair: yeah it'll just be a revert. :) As soon as I push it through I'll send another note out to the list announcing the change and asking people to watch the recheck list and help with the debugging. | 21:49 |
jeblair | clarkb: it will; don't run the cert gen step and use the existing cert | 21:49 |
*** redbeard2 has quit IRC | 21:49 | |
clarkb | jeblair: ok, doing that now | 21:49 |
*** pcm_ has quit IRC | 21:49 | |
jeblair | mtreinish: cool. need anything from us, or is it just that tempest change at this point? | 21:49 |
mtreinish | jeblair: it's just a tempest change to switch the tox jobs over. I guess there aren't any other tempest cores around now though | 21:50 |
jeblair | clarkb: the experimental pipeline messed up the status page layout i was used to. :( maybe we should call it 'testing' :) | 21:50 |
mtreinish | I'll just self approve :) | 21:50 |
*** nati_uen_ has joined #openstack-infra | 21:50 | |
clarkb | jeblair: I am fine with a name change | 21:50 |
anteaya | what is the priority for the experimental pipeline? The priority that I know about is gate, post, check - where does experimental fit? | 21:50 |
clarkb | jeblair: the behavior is what is important to me :) | 21:51 |
anteaya | soon to be called something else | 21:51 |
*** nati_uen_ has quit IRC | 21:51 | |
clarkb | anteaya: it is low | 21:51 |
clarkb | anteaya: so shares resources with check | 21:51 |
anteaya | after check then? | 21:51 |
jeblair | clarkb: or we could encode the order into the status page: (check, gate, everything else in alpha order) | 21:51 |
anteaya | ah the same as check | 21:51 |
lifeless | marun: thanks for the ping; commented in the review | 21:51 |
clarkb | jeblair: either way. I defer to you as I don't know how difficult the status change would be | 21:51 |
jeblair | clarkb: it's just javascript; how hard could it be? ;) | 21:52 |
clarkb | jeblair: 8GB for git.o.o? | 21:52 |
jeblair | clarkb: t | 21:52 |
*** nati_uen_ has joined #openstack-infra | 21:52 | |
clarkb | server is being launched | 21:52 |
*** nati_ueno has quit IRC | 21:53 | |
*** thomasbiege has quit IRC | 21:53 | |
mtreinish | jeblair: actually if you guys could look at https://review.openstack.org/#/c/43690/ that will probably fix one of the flaky fails on the keystone cli tests | 21:53 |
clarkb | lookig | 21:54 |
*** lcestari has quit IRC | 21:55 | |
*** nati_uen_ has quit IRC | 21:55 | |
*** nati_ueno has joined #openstack-infra | 21:56 | |
openstackgerrit | James E. Blair proposed a change to openstack-infra/zuul: Don't reset the local repo https://review.openstack.org/43792 | 21:58 |
notmyname | jeblair: clarkb: in gerrit, is the "needed by" flag not working for the base of a series of patches? | 21:58 |
jeblair | notmyname: it hides it by default in the ui; click 'dependencies' or hit 'd' to see it | 21:58 |
jeblair | notmyname: isn't that helpful? :) | 21:59 |
openstackgerrit | Joe Gordon proposed a change to openstack-infra/devstack-gate: Add support for large_ops tempest test https://review.openstack.org/43779 | 21:59 |
notmyname | jeblair: ya. thanks. so if it's a child patch, it's shown by default, else it isn't | 21:59 |
jeblair | notmyname: yep. no idea why they decided that was a good idea. | 21:59 |
notmyname | jeblair: and also apparently I'm blind ;-) | 21:59 |
notmyname | jeblair: this is actually the second thing (in about as many minutes) that | 22:00 |
notmyname | jeblair: this is actually the second thing (in about as many minutes) that's been right in front of me and I haven't seen ;-) | 22:00 |
notmyname | sorry for the trouble | 22:00 |
clarkb | still stuck at BUILD 0 .... | 22:00 |
jeblair | notmyname: i dunno about you, but i'm a little frazzled atm. :) | 22:00 |
clarkb | jeblair: 43690 is a simple change | 22:01 |
*** ^d has quit IRC | 22:02 | |
jeblair | clarkb: should there be an open bug about that so we don't lose track of it? | 22:02 |
jeblair | i'm just more than usually worried about pinning something called 'keyring' and missing security updates. | 22:03 |
clarkb | jeblair: probably not a bad idea as the bunch referenced is for ubuntu | 22:03 |
jeblair | clarkb: i'll open one and then leave a +2 on that | 22:04 |
clarkb | jeblair: at the very least we have a link to that bug, but having a distinct bug for how it affects us will help us track it | 22:04 |
*** thomasbiege has joined #openstack-infra | 22:05 | |
clarkb | jeblair: new git.o.o errored. trying again | 22:07 |
*** woodspa has quit IRC | 22:08 | |
*** hashar has quit IRC | 22:09 | |
*** thomasbiege has quit IRC | 22:09 | |
*** nati_uen_ has joined #openstack-infra | 22:11 | |
*** nati_uen_ has quit IRC | 22:11 | |
*** portante is now known as portante|afk | 22:11 | |
*** nati_uen_ has joined #openstack-infra | 22:12 | |
openstackgerrit | James E. Blair proposed a change to openstack-infra/zuul: Don't reset the local repo https://review.openstack.org/43792 | 22:13 |
openstackgerrit | James E. Blair proposed a change to openstack-infra/zuul: Fetch specific refs on ref-updated events https://review.openstack.org/43775 | 22:13 |
marun | lifeless: thank you for the review feedback. It would be great if if pm debugging support could be added directly to testtools, but given that testtools doesn't necessarily run tests (and therefore can't be relied upon to configure debugging), would support be limited to defining the appropriate exception handler so it can be reused? | 22:13 |
jeblair | clarkb: i'm hopeful that those 2 changes will do a great deal to reduce zuul's gerrit trigger backlog. | 22:14 |
*** nati_ueno has quit IRC | 22:14 | |
*** burt has quit IRC | 22:14 | |
*** datsun180b has quit IRC | 22:15 | |
clarkb | hmm pulling the salt stuff broke the launch script. I will make that a flag | 22:15 |
jeblair | clarkb, jog0, mtreinish: i ran 'check experimental' on https://review.openstack.org/43791 to see it work. | 22:16 |
jeblair | clarkb: oh how exciting. | 22:17 |
jeblair | clarkb: we're about to test (in production) the result of having only non-voting jobs in an independent pipeline. | 22:18 |
clarkb | cool | 22:18 |
jeblair | clarkb: if you say so. | 22:18 |
jeblair | clarkb: it's entirely possible it could wedge zuul. | 22:18 |
jeblair | clarkb: i sort of expected jobs in the experimental pipeline to be voting. | 22:19 |
clarkb | jeblair: oh that issue | 22:19 |
clarkb | jeblair: hmm | 22:19 |
jeblair | clarkb: not issue, just unknown. | 22:19 |
clarkb | jeblair: I guess they can be voting as long as they don't live in two pipelines | 22:20 |
jeblair | clarkb: if i had to guess, it would be: that it will work, but report overall failure regardless of the outcome. | 22:21 |
*** ryanpetrello has quit IRC | 22:21 | |
openstackgerrit | A change was merged to openstack-infra/config: Use stacked area test node graph https://review.openstack.org/43565 | 22:24 |
openstackgerrit | A change was merged to openstack-infra/config: Add sparklines to pipeline headers https://review.openstack.org/43567 | 22:24 |
openstackgerrit | A change was merged to openstack-infra/config: Use more efficient jobs/hour graph https://review.openstack.org/43596 | 22:24 |
openstackgerrit | Khai Do proposed a change to openstack-infra/gear: Update gear docs to include gearman server daemon https://review.openstack.org/43780 | 22:25 |
openstackgerrit | Clark Boylan proposed a change to openstack-infra/config: Make salt key management on node launch optional. https://review.openstack.org/43794 | 22:26 |
clarkb | jeblair: ^ what do you think of that as a workaround for the salt stuff? | 22:26 |
openstackgerrit | A change was merged to openstack-infra/config: Fix releastestatus dependency https://review.openstack.org/43516 | 22:27 |
*** nati_ueno has joined #openstack-infra | 22:28 | |
*** _TheDodd_ has quit IRC | 22:28 | |
openstackgerrit | A change was merged to openstack-infra/config: Make salt key management on node launch optional. https://review.openstack.org/43794 | 22:31 |
*** dina_belova has joined #openstack-infra | 22:31 | |
*** nati_uen_ has quit IRC | 22:32 | |
clarkb | launchig new git.o.o now | 22:32 |
*** ^demon has joined #openstack-infra | 22:33 | |
*** ^demon is now known as ^d | 22:33 | |
*** jhesketh has joined #openstack-infra | 22:34 | |
*** nati_ueno has quit IRC | 22:34 | |
clarkb | jeblair: https://review.openstack.org/43791 experimental tests were removed form the pipeline but I don't see a comment | 22:34 |
clarkb | jeblair: to have force-message work do we need to provide a non default message or is this prossible related to the non voting? | 22:34 |
*** nati_ueno has joined #openstack-infra | 22:34 | |
*** vipul has quit IRC | 22:35 | |
*** vipul has joined #openstack-infra | 22:35 | |
*** dina_belova has quit IRC | 22:36 | |
mtreinish | clarkb: do you think I should pull the testr-full job or should I wait until after the milestone? (in case we have to revert back to serial?) | 22:37 |
clarkb | mtreinish: I think you can go ahead and pull it | 22:37 |
clarkb | mtreinish: updating zuul configs is really easy these days | 22:37 |
mtreinish | clarkb: ok, will do | 22:38 |
mtreinish | I'll leave the devstack-gate and tempest bits lying around though | 22:38 |
jeblair | 2013-08-26 22:33:15,152 DEBUG zuul.IndependentPipelineManager: Change <Change 0x15fb5d0 43791,2> is a new version of <Change 0x7f123c57a610 43791,1>, removing <Change 0x7f123c57a610 43791,1> | 22:38 |
jeblair | clarkb: ^ :) | 22:38 |
anteaya | nice logging | 22:40 |
jeblair | clarkb: we could accept that behavior, or set that pipeline _not_ to remove changes. | 22:40 |
jeblair | anteaya: thanks; "what was zuul thinking?" is a common and important enough question that it records pretty much every decision it makes. | 22:41 |
jeblair | biab | 22:41 |
anteaya | k | 22:41 |
clarkb | jeblair: I am fine with that behaviour | 22:41 |
anteaya | very friendly to read | 22:41 |
clarkb | jeblair: as gerrit will not let you merge an old patchset. It is possible the old data may be useful, but typically you will want info for new patchsets | 22:42 |
clarkb | mtreinish: yeah those bits are harder to remove or add back in | 22:42 |
anteaya | is the zuul DEBUG info publicly consumable? if so, I haven't recognized it if I have seen it | 22:44 |
clarkb | anteaya: it isn't | 22:44 |
anteaya | that would explain why I haven't seen it | 22:44 |
anteaya | so it appears that "check experimental" puts the patch in question into the experimental pipeline _instead of_ the check pipeline, not _as well as_ | 22:46 |
clarkb | anteaya: I think it can do both. The behavior we saw is related to pushing a new patchset | 22:46 |
clarkb | anteaya: typically new patchset == no need for old tests | 22:47 |
*** fbo is now known as fbo_away | 22:47 | |
clarkb | jeblair: 192.237.223.224 is the new git.o.o haproxy host | 22:47 |
anteaya | clarkb: oh okay, I will continue to observe | 22:48 |
clarkb | jeblair: it seems to be happy with the proposed puppet change. If you like it I think we should go ahead and lower the DNS ttl, swap dns, merge the puppet change, then restart gerrit to pick up new replication config | 22:48 |
*** redbeard2 has joined #openstack-infra | 22:49 | |
anteaya | clarkb: umm, but jeblair ran "check experimental" on patchset 1 of 42791, so none of the check tests has run on it yet | 22:49 |
anteaya | patchset 2 is currently in the check pipeline | 22:50 |
clarkb | anteaya: no, patchset 2 kicked out any tests for patchset1 | 22:50 |
anteaya | ah ha | 22:50 |
openstackgerrit | Matthew Treinish proposed a change to openstack-infra/config: Remove testr-full job https://review.openstack.org/43797 | 22:50 |
anteaya | makes sense now | 22:50 |
clarkb | anteaya: we do that because gerrit doesn't let you revert to an old patchset. So there is little value in tests belonging to an older patchset when you push a new one | 22:51 |
clarkb | pleia2: you may be intrested in 192.237.223.224 as well | 22:51 |
anteaya | mtreinish: can we run "check experimental" on 43797? | 22:51 |
anteaya | clarkb: makes sense | 22:51 |
clarkb | anteaya: there are no experimental jobs for that change | 22:51 |
clarkb | s/change/project/ | 22:51 |
anteaya | clarkb: not much point in running them then | 22:51 |
*** pcrews has quit IRC | 22:52 | |
*** nati_ueno has quit IRC | 22:52 | |
*** nati_ueno has joined #openstack-infra | 22:53 | |
clarkb | mtreinish: good announcement | 22:55 |
mtreinish | clarkb: looks like the 4th time we ran tempest we hit a flaky fail: https://jenkins01.openstack.org/job/gate-tempest-devstack-vm-postgres-full/4875/console . Not the most promising start... | 22:55 |
mtreinish | clarkb: thanks. Hopefully people will look at things a bit more closely now. | 22:56 |
mtreinish | I know I'm guilty of just rechecking and moving on all the time | 22:56 |
clarkb | mtreinish: that looks like a cli keystone test. Perhaps the requirements change is behind it? | 22:56 |
clarkb | hmm no I don't see the requirements change in the list | 22:56 |
mtreinish | clarkb: yeah I'm hoping the requirments change will fix it | 22:56 |
mtreinish | jeblair: only +2'd no +a... | 22:57 |
clarkb | oh | 22:57 |
* clarkb approves | 22:57 | |
mtreinish | clarkb: well I'm gonna head home for the day now (it's getting kind of late here) I'll be back tomorrow morning. | 22:58 |
clarkb | ok, I have approved the keyring pinning. Should have a better idea of what that fixes tomorrow | 22:58 |
mtreinish | ok cool | 22:59 |
*** dims has quit IRC | 23:00 | |
*** wenlock has quit IRC | 23:02 | |
anteaya | look at the pretty zuul sparklines | 23:03 |
*** dims has joined #openstack-infra | 23:04 | |
clarkb | jeblair: I went ahead and lowered the TTL on git.openstack.org A/AAAA records to 300 seconds | 23:04 |
openstackgerrit | Khai Do proposed a change to openstack-infra/gear: Update gear docs to include gearman server daemon https://review.openstack.org/43780 | 23:05 |
clarkb | jeblair: also rax will have potential network outages for up to 10 minutes on wednesday according to their maintenance page | 23:06 |
openstackgerrit | Khai Do proposed a change to openstack-infra/gear: Update gear docs to include gearman server daemon https://review.openstack.org/43780 | 23:08 |
*** dims has quit IRC | 23:08 | |
*** dolphm has joined #openstack-infra | 23:10 | |
openstackgerrit | Khai Do proposed a change to openstack-infra/gear: Update gear docs to include gearman server daemon https://review.openstack.org/43780 | 23:11 |
*** nicedice__ has quit IRC | 23:12 | |
*** nicedice_ has joined #openstack-infra | 23:12 | |
*** dolphm has quit IRC | 23:14 | |
*** nati_ueno has quit IRC | 23:15 | |
*** pentameter has quit IRC | 23:17 | |
clarkb | jeblair: ping me when you get back. I think we can do the git.o.o switch fairly quickly this evening | 23:19 |
*** dims has joined #openstack-infra | 23:22 | |
*** sarob has quit IRC | 23:30 | |
*** sarob has joined #openstack-infra | 23:31 | |
*** rnirmal has quit IRC | 23:31 | |
*** dina_belova has joined #openstack-infra | 23:32 | |
*** sarob has quit IRC | 23:35 | |
*** pcrews has joined #openstack-infra | 23:36 | |
*** sarob has joined #openstack-infra | 23:37 | |
*** dina_belova has quit IRC | 23:37 | |
clarkb | jeblair: I am going to head home now. I am still on board with switching git.o.o if you are. | 23:40 |
jeblair | clarkb: ++ | 23:41 |
jeblair | heh, it looks like if a change is kicked out of a pipeline, the graphite gauge for the pipeline isn't updated. | 23:43 |
*** zul has joined #openstack-infra | 23:47 | |
openstackgerrit | Joe Gordon proposed a change to openstack-infra/devstack-gate: Add support for large_ops tempest test https://review.openstack.org/43779 | 23:49 |
*** dims has quit IRC | 23:53 | |
*** dims has joined #openstack-infra | 23:55 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!