*** gyee has quit IRC | 00:06 | |
*** dingyichen has joined #openstack-infra | 00:10 | |
*** xarses has quit IRC | 00:19 | |
*** zhurong has joined #openstack-infra | 00:23 | |
*** harlowja has quit IRC | 00:26 | |
*** bobh has quit IRC | 00:28 | |
*** mrsoul has joined #openstack-infra | 01:01 | |
*** hongbin has joined #openstack-infra | 01:03 | |
*** mschuppert has quit IRC | 01:04 | |
*** mriedem has quit IRC | 01:15 | |
*** pbourke has quit IRC | 01:17 | |
*** pbourke has joined #openstack-infra | 01:19 | |
*** zhurong has quit IRC | 01:26 | |
*** gfidente|afk has quit IRC | 01:41 | |
*** auristor has quit IRC | 01:50 | |
*** mrsoul has quit IRC | 02:02 | |
*** bobh has joined #openstack-infra | 02:13 | |
*** lbragstad has quit IRC | 02:18 | |
*** auristor has joined #openstack-infra | 02:28 | |
*** xarses has joined #openstack-infra | 02:34 | |
*** bobh has quit IRC | 02:34 | |
*** psachin has joined #openstack-infra | 02:40 | |
*** xarses has quit IRC | 02:42 | |
*** auristor has quit IRC | 02:57 | |
*** auristor has joined #openstack-infra | 02:59 | |
*** rosmaita has quit IRC | 03:13 | |
*** jhesketh_ is now known as jhesketh | 03:14 | |
*** ramishra has joined #openstack-infra | 03:16 | |
*** vivsoni_ has quit IRC | 03:19 | |
*** dave-mccowan has quit IRC | 03:31 | |
*** janki has joined #openstack-infra | 03:41 | |
*** xarses has joined #openstack-infra | 03:43 | |
*** udesale has joined #openstack-infra | 03:48 | |
*** Bhujay has joined #openstack-infra | 04:13 | |
*** vivsoni has joined #openstack-infra | 04:14 | |
*** hongbin has quit IRC | 04:15 | |
openstackgerrit | Merged openstack-infra/project-config master: accessbot config: refactor https://review.openstack.org/588999 | 04:19 |
---|---|---|
openstackgerrit | Merged openstack-infra/project-config master: Set python35-jobs as voting for monasca-agent https://review.openstack.org/587357 | 04:19 |
*** annp has joined #openstack-infra | 04:19 | |
openstackgerrit | Ian Wienand proposed openstack-infra/zuul-jobs master: [WIP] kerberos & afs client roles https://review.openstack.org/589334 | 04:21 |
*** Bhujay has quit IRC | 04:30 | |
AJaeger | config-core, could you review https://review.openstack.org/589237 and https://review.openstack.org/589236, please? That way we have the eol_branch.sh script imported again... | 04:32 |
AJaeger | tonyb: thanks for reviewing! | 04:32 |
tonyb | AJaeger: np, sorry it took most of my day to get there. | 04:34 |
openstackgerrit | Ian Wienand proposed openstack-infra/openstack-zuul-jobs master: [WIP] openafs client install https://review.openstack.org/589335 | 04:34 |
openstackgerrit | Andreas Jaeger proposed openstack-infra/project-config master: Follow wip-upload-swift rename https://review.openstack.org/589336 | 04:35 |
*** ykarel has joined #openstack-infra | 04:35 | |
openstackgerrit | Ian Wienand proposed openstack-infra/openstack-zuul-jobs master: [WIP] openafs client install https://review.openstack.org/589335 | 04:36 |
AJaeger | tonyb: no worries, I'm just up ;) will you help retiring tripleo stable/newton again? | 04:36 |
AJaeger | ianw: could you also review https://review.openstack.org/#/c/588677/ , please? | 04:36 |
tonyb | AJaeger: If someone here adds me to project bootstrappers I'm happy to do it, but I'm also happy for anyone else with permissions to do it | 04:37 |
AJaeger | ianw: still around and have time to help? Or want to do it tomorrow? | 04:37 |
AJaeger | tonyb: remember that the updated script removes the tag parameter | 04:38 |
openstackgerrit | Merged openstack-infra/project-config master: Import eol_branch.sh https://review.openstack.org/589236 | 04:38 |
tonyb | AJaeger: will do. | 04:38 |
ianw | AJaeger: i'm around | 04:38 |
AJaeger | ianw: can you add tonyb to project-bootstrappers so that he can retire tripleo stable/newton, please? See http://lists.openstack.org/pipermail/openstack-dev/2018-August/132940.html | 04:39 |
tonyb | AJaeger: I was a little confused why you left EOL_MESSAGE in 237 but it made sense once I really looked at it ;P | 04:39 |
openstackgerrit | Merged openstack-infra/project-config master: Remove eol tagging from eol_branch.sh https://review.openstack.org/589237 | 04:39 |
* AJaeger wants to get rid of the error thta shows in http://zuul.openstack.org/api/config-errors | 04:39 | |
ianw | AJaeger tonyb: done | 04:39 |
AJaeger | tonyb: I first removed EOL_MESSAGE but then noticed the other usage... | 04:40 |
AJaeger | argh, we now have errors in openstack-chef as well ;( | 04:40 |
tonyb | ianw: Thanks. | 04:40 |
AJaeger | fungi: ^ | 04:40 |
AJaeger | scas: please cleanup openstack-chef and openstack-chef-repo zuul configuration, see http://zuul.openstack.org/api/config-errors | 04:41 |
AJaeger | scas: might be easiest to remove .zuul.yaml from openstack-chef-repo ... | 04:42 |
AJaeger | thanks, ianw | 04:44 |
scas | AJaeger: it's in the process of going away. i'll wipe out the .zuul.yaml now since i'm moving the rest of the repos over | 04:49 |
openstackgerrit | Ian Wienand proposed openstack-infra/zuul-jobs master: [WIP] kerberos & afs client roles https://review.openstack.org/589334 | 04:51 |
openstackgerrit | Ian Wienand proposed openstack-infra/openstack-zuul-jobs master: [WIP] openafs client install https://review.openstack.org/589335 | 04:52 |
tonyb | AJaeger, ianw: those repos don't actuallt seem to have the newton-eol tags http://paste.openstack.org/show/727457/ | 04:54 |
tonyb | for now I can easily use the HEAD^ version of eol_branch.sh | 04:54 |
AJaeger | scas: thanks - I'm aware of the process | 04:55 |
AJaeger | tonyb: https://review.openstack.org/#/c/583856/ merged - did something fail? | 04:55 |
tonyb | AJaeger: I *think* actually that doesn't tag things on newton | 04:56 |
tonyb | AJaeger: because 'newton' is closed. | 04:56 |
AJaeger | argh ;( | 04:56 |
tonyb | AJaeger: I think dhellman was supposed to run the tool manually | 04:56 |
AJaeger | so, you need the old version of the script and tag yourself? | 04:57 |
AJaeger | go ahead IMHO - ianw, do you agree? | 04:57 |
tonyb | AJaeger: No problem. I'll just use eol_branch.sh from 18aea9c6a | 04:57 |
AJaeger | thanks | 04:58 |
ianw | i think it would be right to tag, fwiw ... | 05:00 |
*** e0ne has joined #openstack-infra | 05:00 | |
scas | AJaeger: it's late here and i patched over it at the same time | 05:01 |
*** xarses has quit IRC | 05:03 | |
scas | AJaeger: it's going to be angry until i patch over the other four open branches | 05:04 |
*** stakeda has joined #openstack-infra | 05:04 | |
AJaeger | scas: no worries - do it tomorrow your day and call it a night ;) | 05:05 |
AJaeger | scas: thanks | 05:05 |
scas | i'll get a project-config change for my finalizing stuff up during my day | 05:05 |
scas | probably less churn than noop'ing the whole repo | 05:06 |
tonyb | AJaeger: done. | 05:07 |
AJaeger | thanks, tonyb ! | 05:09 |
tonyb | AJaeger: You're welcome thanks for the prod | 05:10 |
scas | never mind about project-config. that got removed long ago. i'll keep an eye on the config-errors page to see when it clears up as i'm patching over the old jobs yaml | 05:14 |
AJaeger | scas: if you retire the repo, you remove from project-config/zuul/main.yaml - and then it does not show up either | 05:17 |
openstackgerrit | Ian Wienand proposed openstack-infra/openstack-zuul-jobs master: [WIP] openafs client install https://review.openstack.org/589335 | 05:18 |
*** psachin has quit IRC | 05:18 | |
*** psachin has joined #openstack-infra | 05:25 | |
*** jaosorior has quit IRC | 05:26 | |
scas | AJaeger: it's clearing. thanks for the vigilance | 05:34 |
openstackgerrit | Ian Wienand proposed openstack-infra/zuul-jobs master: [WIP] kerberos & afs client roles https://review.openstack.org/589334 | 05:38 |
*** xarses has joined #openstack-infra | 05:44 | |
openstackgerrit | Ian Wienand proposed openstack-infra/zuul-jobs master: [WIP] kerberos & afs client roles https://review.openstack.org/589334 | 05:44 |
*** nicolasbock has joined #openstack-infra | 05:46 | |
openstackgerrit | Ian Wienand proposed openstack-infra/openstack-zuul-jobs master: [WIP] openafs client install https://review.openstack.org/589335 | 05:57 |
*** hwoarang has joined #openstack-infra | 05:57 | |
*** e0ne has quit IRC | 05:58 | |
*** apetrich has joined #openstack-infra | 06:01 | |
*** e0ne has joined #openstack-infra | 06:02 | |
*** jaosorior has joined #openstack-infra | 06:05 | |
AJaeger | scas: indeed, it's empty - thanks! | 06:08 |
AJaeger | #status log http://zuul.openstack.org/api/config-errors shows *no* errors | 06:08 |
openstackstatus | AJaeger: finished logging | 06:08 |
openstackgerrit | Andreas Jaeger proposed openstack-infra/openstack-zuul-jobs master: Rework translation templates https://review.openstack.org/589372 | 06:12 |
AJaeger | corvus, I'm curious about the enhanced logging on translation jobs. One guess: The rework you did on job inheritance, could that have changed it? In that case the change above ^ might be one possible fix | 06:13 |
AJaeger | (note that one is part 1 of the change - just wanted to bring it up for discussion, I can part 2 if this is the right way) | 06:14 |
*** xarses has quit IRC | 06:15 | |
*** xarses has joined #openstack-infra | 06:15 | |
eumel8 | AJaeger: I'm not sure if it would helps | 06:16 |
*** xarses_ has joined #openstack-infra | 06:17 | |
*** xarses has quit IRC | 06:20 | |
*** quiquell has joined #openstack-infra | 06:21 | |
openstackgerrit | Ian Wienand proposed openstack-infra/infra-specs master: letsencrypt spec https://review.openstack.org/587283 | 06:28 |
*** pcaruana has joined #openstack-infra | 06:36 | |
*** Bhujay has joined #openstack-infra | 06:42 | |
*** ginopc has joined #openstack-infra | 07:00 | |
*** e0ne has quit IRC | 07:02 | |
*** amoralej|off is now known as amoralej | 07:08 | |
*** mpeterson has quit IRC | 07:23 | |
*** mpeterson has joined #openstack-infra | 07:23 | |
*** dpawlik has joined #openstack-infra | 07:23 | |
*** wznoinsk has quit IRC | 07:27 | |
*** wznoinsk has joined #openstack-infra | 07:27 | |
*** e0ne has joined #openstack-infra | 07:35 | |
*** jpena|off is now known as jpena | 07:35 | |
*** e0ne has quit IRC | 07:41 | |
*** tosky has joined #openstack-infra | 07:42 | |
*** ykarel is now known as ykarel|lunch | 07:43 | |
*** jpich has joined #openstack-infra | 07:45 | |
*** jaosorior has quit IRC | 07:49 | |
*** e0ne has joined #openstack-infra | 07:52 | |
*** e0ne has quit IRC | 07:53 | |
*** apetrich has quit IRC | 07:56 | |
*** yamamoto has joined #openstack-infra | 08:00 | |
*** jaosorior has joined #openstack-infra | 08:02 | |
AJaeger | eumel8: let's see what the problem is ;) | 08:06 |
*** yamamoto has quit IRC | 08:07 | |
*** rcernin has quit IRC | 08:11 | |
*** dayou has quit IRC | 08:12 | |
*** boden has joined #openstack-infra | 08:14 | |
*** Emine has joined #openstack-infra | 08:16 | |
*** boden has quit IRC | 08:19 | |
*** owalsh_ is now known as owalsh | 08:33 | |
*** electrofelix has joined #openstack-infra | 08:33 | |
*** panda|rover-ish is now known as panda|rover | 08:33 | |
*** ykarel|lunch is now known as ykarel | 08:35 | |
*** gfidente has joined #openstack-infra | 08:35 | |
*** yamamoto has joined #openstack-infra | 08:46 | |
*** derekh has joined #openstack-infra | 08:46 | |
*** priteau has joined #openstack-infra | 08:55 | |
evrardjp | AJaeger: cmurphy could you help me and review https://review.openstack.org/#/q/status:open+project:openstack-infra/bindep+branch:master+topic:loci-suseleap15 ? | 08:58 |
evrardjp | thank you in advance. | 08:58 |
*** jaosorior has quit IRC | 09:02 | |
cmurphy | evrardjp: i'm not a core on bindep nor do i know much about it but left a comment anyways | 09:02 |
*** dingyichen has quit IRC | 09:02 | |
evrardjp | as long as we talk about it here, and it's not dying, I am good. | 09:03 |
evrardjp | :D | 09:03 |
evrardjp | hitting the nail until those merge. | 09:03 |
AJaeger | evrardjp: I'm not a core there either - but did some patches. Will look later at it. | 09:04 |
evrardjp | oh that's sad. I got 0% success rate in blind ping. | 09:05 |
evrardjp | let's try more focused then. | 09:05 |
AJaeger | evrardjp: you need infra-root for bindep changes | 09:06 |
AJaeger | evrardjp: best ask again during US morning | 09:06 |
evrardjp | AJaeger: yeah, or Aussie time with jhesketh :) | 09:07 |
evrardjp | I will try later. | 09:07 |
jhesketh | evrardjp: I was just pulling up the changes actually :-) | 09:07 |
evrardjp | oh you're there? Isn't it late for you? | 09:08 |
evrardjp | haha that's great! | 09:08 |
*** yamamoto has quit IRC | 09:11 | |
jhesketh | it's just past 7pm and I just returned from dinner, so good timing :-) | 09:12 |
*** finucannot is now known as stephenfin | 09:30 | |
*** zoli is now known as zoli|lunch | 09:34 | |
*** roman_g has joined #openstack-infra | 10:01 | |
*** gema has joined #openstack-infra | 10:14 | |
*** jamesdenton has quit IRC | 10:18 | |
*** electrofelix has quit IRC | 10:18 | |
*** olivierbourdon38 has joined #openstack-infra | 10:26 | |
*** electrofelix has joined #openstack-infra | 10:26 | |
*** jamesdenton has joined #openstack-infra | 10:37 | |
*** dayou has joined #openstack-infra | 10:42 | |
*** jaosorior has joined #openstack-infra | 10:46 | |
*** dave-mccowan has joined #openstack-infra | 10:51 | |
*** zeus has quit IRC | 10:53 | |
*** zoli|lunch is now known as zoli | 10:53 | |
*** bradm has quit IRC | 10:56 | |
*** hs366 has quit IRC | 11:14 | |
*** udesale has quit IRC | 11:15 | |
*** Bhujay has quit IRC | 11:18 | |
*** gfidente has quit IRC | 11:18 | |
*** panda|rover is now known as panda|rover|lunc | 11:18 | |
*** gfidente has joined #openstack-infra | 11:20 | |
*** vivsoni has quit IRC | 11:24 | |
*** stakeda has quit IRC | 11:26 | |
*** gfidente has quit IRC | 11:28 | |
*** gfidente has joined #openstack-infra | 11:30 | |
*** vivsoni has joined #openstack-infra | 11:31 | |
*** slagle has joined #openstack-infra | 11:39 | |
*** gbutnaru has joined #openstack-infra | 11:44 | |
*** apetrich has joined #openstack-infra | 11:53 | |
*** zul has joined #openstack-infra | 11:54 | |
*** boden has joined #openstack-infra | 11:56 | |
*** jaosorior has quit IRC | 11:59 | |
*** edmondsw has joined #openstack-infra | 12:04 | |
*** tpsilva has joined #openstack-infra | 12:07 | |
*** amoralej is now known as amoralej|lunch | 12:15 | |
*** jpena is now known as jpena|lunch | 12:15 | |
*** panda|rover|lunc is now known as panda|rover | 12:16 | |
*** jaosorior has joined #openstack-infra | 12:18 | |
*** Bhujay has joined #openstack-infra | 12:24 | |
*** jaosorior has quit IRC | 12:27 | |
evrardjp | mordred: I will try to get your vote on https://review.openstack.org/#/q/status:open+project:openstack-infra/bindep+branch:master+topic:loci-suseleap15 as I saw you were there! | 12:34 |
mordred | oh no, I'm not invisible | 12:37 |
AJaeger | ;) | 12:38 |
*** e0ne has joined #openstack-infra | 12:40 | |
*** jaosorior has joined #openstack-infra | 12:42 | |
*** sthussey has joined #openstack-infra | 12:45 | |
*** sdoran_ has joined #openstack-infra | 12:49 | |
*** auristor has quit IRC | 12:50 | |
*** sdoran has quit IRC | 12:50 | |
*** weshay has quit IRC | 12:50 | |
*** sdoran_ is now known as sdoran | 12:50 | |
AJaeger | config-core, could you review https://review.openstack.org/588920 and https://review.openstack.org/589336 , please? | 12:54 |
*** bobh has joined #openstack-infra | 12:54 | |
openstackgerrit | sebastian marcet proposed openstack-infra/openstackid-resources master: Presentation POST/PUT Validation https://review.openstack.org/589457 | 12:55 |
openstackgerrit | Merged openstack-infra/openstackid-resources master: Presentation POST/PUT Validation https://review.openstack.org/589457 | 12:58 |
*** ccamacho has quit IRC | 12:58 | |
*** dklyle has quit IRC | 12:59 | |
openstackgerrit | Merged openstack-infra/bindep master: Add opensuse-leap support https://review.openstack.org/586038 | 13:01 |
openstackgerrit | Merged openstack-infra/bindep master: Fix bindep path https://review.openstack.org/588253 | 13:01 |
AJaeger | evrardjp, I guess we need a new bindep release now - correct? Please work with clarkb on this once he's awake | 13:02 |
openstackgerrit | Merged openstack-infra/bindep master: Fix user profile display https://review.openstack.org/584955 | 13:03 |
evrardjp | mordred: I cannot not think of https://www.youtube.com/watch?v=zKdxd718WXg . | 13:03 |
*** janki has quit IRC | 13:03 | |
evrardjp | AJaeger: will do. | 13:04 |
*** janki has joined #openstack-infra | 13:04 | |
*** kgiusti has joined #openstack-infra | 13:06 | |
*** auristor has joined #openstack-infra | 13:07 | |
*** jamesdenton has quit IRC | 13:10 | |
*** jamesdenton has joined #openstack-infra | 13:11 | |
*** trown|outtypewww is now known as trown | 13:12 | |
*** mriedem has joined #openstack-infra | 13:16 | |
*** ykarel is now known as ykarel|away | 13:21 | |
*** eharney has joined #openstack-infra | 13:22 | |
*** ccamacho has joined #openstack-infra | 13:28 | |
*** jpena|lunch is now known as jpena | 13:30 | |
*** ykarel|away has quit IRC | 13:31 | |
*** amoralej|lunch is now known as amoralej | 13:32 | |
scas | random question: how can one get github traffic graphs from the mirror? consider it morbid curiosity. i know i don't have push access, so that means i personally can't view it | 13:37 |
dhellmann | AJaeger , tonyb : did you need me to do something with the tripleo newton stuff? | 13:38 |
AJaeger | dhellmann: all done and fine AFAIK - thanks for checking | 13:39 |
dhellmann | ok, np | 13:41 |
AJaeger | dhellmann: did you see that https://review.openstack.org/550978 merged and is live? Is that enough for you to check for release job in-repo? | 13:42 |
*** jcoufal has joined #openstack-infra | 13:42 | |
*** r-mibu has joined #openstack-infra | 13:42 | |
dhellmann | AJaeger : that's the API to fetch the jobs configured for a repo? | 13:42 |
dhellmann | oh, that gives the job details | 13:43 |
AJaeger | the first part - not sure whether you need 52891 as well | 13:43 |
dhellmann | I don't remember why we needed that | 13:43 |
AJaeger | or https://review.openstack.org/#/c/550979 | 13:44 |
dhellmann | I think that's the we would need | 13:44 |
dhellmann | *the one | 13:44 |
AJaeger | I see | 13:44 |
dhellmann | I put together some interim code that just looks at project-config and the repo's zuul files for now | 13:45 |
*** quiquell is now known as quiquell|off | 13:47 | |
*** psachin has quit IRC | 13:51 | |
*** quiquell|off has quit IRC | 13:51 | |
*** jamesdenton has quit IRC | 13:52 | |
*** jamesdenton has joined #openstack-infra | 13:53 | |
openstackgerrit | Monty Taylor proposed openstack-infra/zuul master: web: add /{tenant}/projects and /{tenant}/project/{project} routes https://review.openstack.org/550979 | 13:56 |
openstackgerrit | Monty Taylor proposed openstack-infra/zuul master: web: add /{tenant}/pipelines route https://review.openstack.org/541521 | 13:56 |
openstackgerrit | Monty Taylor proposed openstack-infra/zuul master: scheduler: add job's parent name to the rpc job_list method https://review.openstack.org/573473 | 13:56 |
openstackgerrit | Monty Taylor proposed openstack-infra/zuul master: web: add /{tenant}/labels route https://review.openstack.org/553979 | 13:56 |
openstackgerrit | Monty Taylor proposed openstack-infra/zuul master: web: add /{tenant}/nodes route https://review.openstack.org/553998 | 13:56 |
openstackgerrit | Merged openstack-infra/project-config master: Check for empty project entries https://review.openstack.org/588920 | 14:02 |
*** jamesmcarthur has joined #openstack-infra | 14:05 | |
*** jrist has quit IRC | 14:15 | |
*** Bhujay has quit IRC | 14:18 | |
*** nicolasbock has quit IRC | 14:18 | |
*** rpioso|afk is now known as rpioso | 14:26 | |
*** jrist has joined #openstack-infra | 14:28 | |
openstackgerrit | sebastian marcet proposed openstack-infra/openstackid master: Fix on OIDC session status check added more logging info fixed toString missing call method https://review.openstack.org/589502 | 14:33 |
*** dklyle has joined #openstack-infra | 14:33 | |
*** hwoarang has quit IRC | 14:37 | |
*** ykarel has joined #openstack-infra | 14:38 | |
*** hongbin has joined #openstack-infra | 14:38 | |
*** e0ne has quit IRC | 14:39 | |
*** rasca is now known as rascasoft | 14:49 | |
*** priteau has quit IRC | 14:50 | |
openstackgerrit | Merged openstack-infra/nodepool master: Switch storyboard url to be by name https://review.openstack.org/588600 | 14:55 |
*** bobh has quit IRC | 14:56 | |
*** ykarel has quit IRC | 14:57 | |
*** rosmaita has joined #openstack-infra | 14:57 | |
*** Emine has quit IRC | 14:58 | |
*** jaosorior has quit IRC | 15:01 | |
*** panda|rover is now known as panda|backin2h | 15:01 | |
*** lamt has joined #openstack-infra | 15:02 | |
*** mriedem is now known as mriedem_afk | 15:06 | |
*** nicolasbock has joined #openstack-infra | 15:07 | |
*** dpawlik has quit IRC | 15:10 | |
*** pcaruana has quit IRC | 15:11 | |
*** hemna_ has joined #openstack-infra | 15:12 | |
mordred | corvus: I thnk you were looking at https://review.openstack.org/#/c/587985 on friday and had some concerns - is that a thing we should circle back on? | 15:13 |
openstackgerrit | Merged openstack-infra/system-config master: Remove mention of excercises.sh https://review.openstack.org/583147 | 15:16 |
*** bobh has joined #openstack-infra | 15:17 | |
*** bobh has quit IRC | 15:17 | |
tosky | I see something weird, it's like zuul did not report back the results to gerrit for few reviews against stable/pike | 15:19 |
tosky | namely: https://review.openstack.org/#/c/589395/ https://review.openstack.org/#/c/589396/ and https://review.openstack.org/#/c/588515/ | 15:19 |
tosky | the jobs were in the zuul gate queue, and I don't see unmerged dependencies | 15:19 |
*** Bhujay has joined #openstack-infra | 15:23 | |
*** priteau has joined #openstack-infra | 15:26 | |
*** hamerins has joined #openstack-infra | 15:30 | |
*** gyee has joined #openstack-infra | 15:32 | |
*** ramishra has quit IRC | 15:34 | |
*** dpawlik has joined #openstack-infra | 15:38 | |
*** boden has quit IRC | 15:41 | |
*** dpawlik has quit IRC | 15:42 | |
*** v1k0d3n has joined #openstack-infra | 15:44 | |
*** bobh has joined #openstack-infra | 15:45 | |
*** olivierbourdon38 has quit IRC | 15:46 | |
evrardjp | How hard would it be to publish an artifact of a zuul periodic job (a manifest text file), that could be used as a deliverable file for a release? Or the other way around: how hard would it be in the releasing process to fetch artifacts from gate logs (assuming releasing would happen max 30 days after latest gate log for said branch) (in that last case, the hardest is to ensure the file is still | 15:47 |
evrardjp | present in case of infrastructure failure - disk crash or flush, etc.) | 15:47 |
tosky | zuul ate my job results :) | 15:48 |
clarkb | tosky: corvus: possible http reporting fallout? I see it claim to report check job results about half and hour ago in the zuul scheduler logs | 15:48 |
clarkb | tosky: corvus: but doesn't seem to make it to Gerrit | 15:48 |
clarkb | corvus: tosky yup, looks like a new bug | 15:49 |
tosky | clarkb: but why those jobs specifically? Other jobs reported the results in the meantime | 15:49 |
clarkb | corvus: tosky http://paste.openstack.org/show/727545/ change ids are not unique identifiers | 15:49 |
clarkb | tosky: because they use non unique change ids (they are cherry picks with change id preserved) | 15:49 |
mordred | clarkb: oh fun! | 15:49 |
tosky | clarkb: but isn't that what happens normally with cherry-picks? | 15:50 |
mordred | tosky: yes - but we changed how zuul reports results yesterday | 15:50 |
tosky | oh | 15:50 |
tosky | '-_- | 15:50 |
clarkb | tosky: yes, we just changed zuul to report job results over http and not ssh. This changed behavior and broke this case | 15:50 |
mordred | tosky: in order to support zuul being able to leave in-line comments | 15:50 |
tosky | okidoki | 15:51 |
clarkb | evrardjp: there is in progress work to add a "promote" pipeline that runs on change-merged triggers which will allow it to map back to specific changes. The result is we should be able to do something like that | 15:51 |
corvus | and also, as we can all see, the rest api is far superior to the ssh api ;) | 15:51 |
evrardjp | clarkb: that is interesting. | 15:52 |
clarkb | corvus: fwiw it also doesn't look like it retries 3 times | 15:52 |
clarkb | corvus: so might be multiple bugs there | 15:52 |
tosky | but shouldn't at least return something in case of error? Or is it one of those errors that should not return anything? | 15:52 |
clarkb | tosky: it is erroring in its attempt to report to the change | 15:52 |
openstackgerrit | Merged openstack-infra/openstackid master: Fix on OIDC session status check added more logging info fixed toString missing call method https://review.openstack.org/589502 | 15:52 |
evrardjp | clarkb: do you have a spec I can read about that somewhere? Or a patch maybe? | 15:53 |
openstackgerrit | Andy Smith proposed openstack-infra/project-config master: devstack-plugin-kafka tests moved to project repo https://review.openstack.org/589525 | 15:53 |
clarkb | evrardjp: I don't think there is anything that formal at least not yet. Id60b5e292b5750d701ec77b51ff73f12becf3cbd is the change id for the change that dded the pipeline | 15:54 |
corvus | clarkb: i see 3 attempts | 15:54 |
clarkb | corvus: oh maybe my grep -A number wasn't big enough | 15:54 |
corvus | clarkb: it has a 10 second then 20 second delay | 15:55 |
corvus | http://paste.openstack.org/show/727547/ | 15:55 |
clarkb | corvus: that would explain why I didn't immediately see them | 15:55 |
evrardjp | clarkb: for my understanding "promote" could be used for publishing artifacts from gate pipeline. | 15:55 |
AJaeger | dhellmann: I followed your ask to not merge oslo changes and gave a -2 on https://review.openstack.org/#/c/589525 | 15:56 |
evrardjp | that is indeed what I'd look for. | 15:56 |
corvus | evrardjp, clarkb: yeah, the idea is if you publish the artifact to logs.openstack.org/changeid/patchset/gate/ then a promote job can pick it up from there | 15:56 |
dhellmann | AJaeger : yeah, we have a problem with that repo | 15:57 |
dhellmann | I thought ansmith was going to propose making the job non-voting temporarily | 15:57 |
corvus | evrardjp, clarkb: (basically, leave off the buildset uuid from that, and just count on the fact that the last version of the gate job will have overwritten any previous versions) | 15:57 |
fungi | thinking more about the promote pipeline, since it's tied to merged changes/prs rather than merged commits, is supercedent really the correct pipeline manager for that or would independent make more sense there? | 15:57 |
fungi | just thinking through the use cases | 15:57 |
evrardjp | corvus: yes but at least it would be probably more reliable and easy to find | 15:57 |
*** ansmith has joined #openstack-infra | 15:58 | |
evrardjp | than hoping from a file from a gate log x | 15:58 |
evrardjp | I see | 15:58 |
AJaeger | dhellmann, happy to remove my -2 if you +1 it ;) | 15:58 |
dhellmann | AJaeger : apparently the tests on that repo are stale (or never worked) and so we can't land the patch to import the zuul settings the way things are configured now | 15:58 |
dhellmann | I don't know what's better, removing the job and then landing the patch to add it back in-tree, or marking it non-voting? | 15:58 |
corvus | fungi: i was imagining the single-artifact use case, like "latest docs" or "latest build" | 15:58 |
dhellmann | removing it might be easier. Either way I have to resolve the merge conflict | 15:58 |
fungi | corvus: oh, got it. so why tie it to change-merged rather than ref-updated? | 15:59 |
dhellmann | AJaeger : if we work out what to do here, I'll leave a comment on the patch for the record | 15:59 |
kgiusti | dhellmann: for the amqp1 plugin we only define the tox-bashate, but the plugin does have a .zuul.yaml that includes the full set of tests | 15:59 |
AJaeger | dhellmann: yes, please work it out - I'll take a +1 from you as a sign to remove my -2 ;) | 15:59 |
dhellmann | kgiusti : ok, we should be good to go for amqp1, then. | 15:59 |
dhellmann | AJaeger : ok | 15:59 |
corvus | tosky, clarkb: i used the wrong change-id format, i need to use project~branch~changeid | 16:00 |
dhellmann | AJaeger : well, I was looking for your advice about the best approach :-) | 16:00 |
clarkb | corvus: ya project + branch + changeid is unique key in gerrit | 16:00 |
*** Bhujay has quit IRC | 16:00 | |
kgiusti | dhellmann: I told ansmith to use amqp1 as the correct model /my fault/ | 16:00 |
dhellmann | ah | 16:00 |
dhellmann | I guess since we're just turning the tests off and adding them back there's no reason not to do it this way | 16:01 |
*** onovy has quit IRC | 16:01 | |
AJaeger | dhellmann: sorry... Shouldn't multitask ;( | 16:01 |
AJaeger | dhellmann: I'm fine removing it first or making it non-voting - and then basing your change on top of his. | 16:01 |
*** jcoufal has quit IRC | 16:02 | |
dhellmann | AJaeger : ok, I left a +1 to go ahead with this | 16:02 |
dhellmann | now we just need ansmith to remove his W-1 | 16:02 |
AJaeger | dhellmann: ok | 16:02 |
AJaeger | ;) | 16:02 |
ansmith | removed :) | 16:02 |
dhellmann | ansmith : cool, thanks | 16:02 |
* AJaeger reviews to know what vote to give ;) | 16:03 | |
ansmith | dhellmann: will add .zuul.yaml to the plugin next | 16:03 |
dhellmann | perhaps we can get mnaser to approve https://review.openstack.org/#/c/589525/1 then? | 16:03 |
dhellmann | ansmith : my existing patch should do that | 16:03 |
dhellmann | ansmith : https://review.openstack.org/#/c/588685/ | 16:03 |
AJaeger | ansmith, dhellmann, why is bashate not moved in-repo? I would just remove the complete section and import everything in-repo. | 16:04 |
openstackgerrit | James E. Blair proposed openstack-infra/zuul master: Use full gerrit changeid in HTTP report https://review.openstack.org/589530 | 16:04 |
AJaeger | ansmith, dhellmann, left a +1 with that question in there... | 16:05 |
dhellmann | AJaeger : I guess we could do it that way. My other project-config patch will finish the cleanup | 16:05 |
kgiusti | AJaeger: I'm +1 with that - I'd prefer the test config be local to the plugin | 16:05 |
AJaeger | dhellmann: so, leave bashate in, import it in-repo, and then your large project-config change removes it? | 16:05 |
*** r-mibu has quit IRC | 16:06 | |
dhellmann | AJaeger : yeah, unless you'd rather ansmith remove it in his patch now | 16:06 |
corvus | tosky: 589530 should fix it, and i'll restart zuul with that as soon as it lands. sorry, and thanks for the report. | 16:06 |
dhellmann | what happens if a repo has no test jobs attached at all? | 16:06 |
AJaeger | dhellmann: whatever works for you ;) | 16:06 |
dhellmann | I prefer moving all of this along quickly :-) | 16:06 |
AJaeger | dhellmann: you cannot merge anything - you need noop-jobs template in this case to merge first | 16:06 |
* kgiusti votes for removing it completely.... | 16:06 | |
tosky | corvus: thank you for the quick fix; please let me know when the restart is done and I will recheck the jobs | 16:06 |
dhellmann | AJaeger : ok, so it sounds like we want to leave bash8 in place | 16:07 |
dhellmann | *bashate | 16:07 |
dhellmann | that way we have *some* voting job | 16:07 |
* kgiusti changes vote to whatever makes dhellmann happy :) | 16:07 | |
AJaeger | dhellmann: I gave +2 - since I'll be offline for a bit now. if you remove everything, I'm happy to recast my vote ;) | 16:08 |
*** onovy has joined #openstack-infra | 16:08 | |
dhellmann | AJaeger : ok, thanks | 16:08 |
*** ginopc has quit IRC | 16:08 | |
*** mriedem_afk has quit IRC | 16:11 | |
mnaser | dhellmann: approved | 16:14 |
dhellmann | mnaser : thank you! | 16:14 |
openstackgerrit | James E. Blair proposed openstack-infra/zuul master: Use full gerrit changeid in HTTP report https://review.openstack.org/589530 | 16:15 |
*** e0ne has joined #openstack-infra | 16:18 | |
*** armax has joined #openstack-infra | 16:19 | |
*** amoralej is now known as amoralej|off | 16:21 | |
*** jpena is now known as jpena|off | 16:21 | |
openstackgerrit | James E. Blair proposed openstack-infra/zuul master: Use full gerrit changeid in HTTP report https://review.openstack.org/589530 | 16:24 |
*** zoli is now known as zoli|gone | 16:24 | |
*** zoli|gone is now known as zoli | 16:24 | |
*** jpich has quit IRC | 16:27 | |
*** electrofelix has quit IRC | 16:29 | |
*** lbragstad[m] has quit IRC | 16:32 | |
*** trown is now known as trown|lunch | 16:33 | |
openstackgerrit | James E. Blair proposed openstack-infra/zuul-website master: Add BMW case study https://review.openstack.org/589541 | 16:37 |
corvus | clarkb, fungi: i'm going to direct-enqueue 589530 | 16:38 |
clarkb | corvus: ok | 16:38 |
*** janki has quit IRC | 16:47 | |
*** weshay has joined #openstack-infra | 16:47 | |
weshay | zuul ok? | 16:47 |
clarkb | weshay: there is a bug in reporting that affects a subset of changes (those with non unique change ids) that we are fixing. Otherwise it seems fine. | 16:48 |
clarkb | oh zuul web is not running | 16:48 |
clarkb | corvus: ^ | 16:48 |
weshay | :) | 16:49 |
clarkb | hrm the process is running so tis is different than yesterday | 16:50 |
clarkb | the direct enqueue seems to be running still as well, I think the scheduler must just be busy rebuilding layouts? | 16:50 |
mordred | clarkb: openstacksdk 0.17.2 has been released, so we can restart nodepool launchers to pick it up | 16:50 |
corvus | clarkb: it cleared | 16:51 |
clarkb | corvus: I wonder if direct enqueues are what create that behavior? I saw similar with a direct enqueue a while back (last week?) | 16:51 |
openstackgerrit | sebastian marcet proposed openstack-infra/openstackid master: Fix for OIDC session checking https://review.openstack.org/589549 | 16:52 |
clarkb | mordred: ok, I think we may need to merge a change to nodepool to have it auto update deps. Otherwise we can manually trigger the dep update then restart | 16:52 |
mordred | clarkb: kk. I've got a phone call in 7 minutes, so I can't do that right now - but I can do it a little later | 16:53 |
mordred | or, if someone beats me to it, it's all good | 16:53 |
clarkb | mordred: limestone has gone idle again. It is served by nl02. I'll start by looking there | 16:54 |
corvus | clarkb: https://review.openstack.org/582794 will help a lot with this | 16:55 |
clarkb | ya needs manual pip3 install. I will do that now on nl02 then restart | 16:55 |
*** Emine has joined #openstack-infra | 16:55 | |
*** tosky has quit IRC | 16:57 | |
clarkb | mordred: `sudo pip3 install -U /opt/nodepool` on nl02 fwiw. Restarting launcher now | 16:58 |
mordred | mordred@nl02:~$ pbr freeze | grep openstacksdk | 16:59 |
mordred | openstacksdk==0.17.2 # git sha 3ad9dab | 16:59 |
mordred | looks like that worked | 16:59 |
*** derekh has quit IRC | 17:00 | |
clarkb | logan-: ^ fyi we think we have addressed the limestone idling problem | 17:00 |
clarkb | corvus: good to know | 17:00 |
logan- | clarkb: awesome! thanks | 17:07 |
*** panda|backin2h is now known as panda|rover | 17:08 | |
*** xarses_ has quit IRC | 17:10 | |
AJaeger | corvus: we had *no* zuul errors this morning - and now have one, see http://zuul.openstack.org/api/config-errors . How could one sneak in suddenly? | 17:12 |
clarkb | mordred: ok I havne't seen nl02 have problems yet. I will do the others now | 17:15 |
clarkb | mordred: we might consider bumping the min for openstacksdk in nodepool too give that bug | 17:16 |
Shrews | clarkb: i wonder if that warrants a new release, too | 17:18 |
AJaeger | config-core, could you review https://review.openstack.org/589336 so that we can merge the dependend change, please? | 17:18 |
clarkb | Shrews: probably a good idea also wb | 17:18 |
clarkb | I know rdo hit this bug too | 17:18 |
*** e0ne has quit IRC | 17:20 | |
*** e0ne has joined #openstack-infra | 17:21 | |
clarkb | #status log Updated openstacksdk on nodepool-launchers to 0.17.2 to fix provider thread crashes that result in idle providers | 17:21 |
openstackstatus | clarkb: finished logging | 17:21 |
clarkb | mordred: ^ its done | 17:21 |
*** e0ne has quit IRC | 17:22 | |
*** gfidente has quit IRC | 17:23 | |
clarkb | AJaeger: https://review.openstack.org/#/c/588677/ needs to go in first. What are the affects of referring to a role that doesn't exist? job will just fail until we update? | 17:23 |
clarkb | corvus: ^ | 17:23 |
AJaeger | clarkb: job will fail AFAIK - and this is only a test job, so fine IMHO to merge them after each other. | 17:25 |
clarkb | AJaeger: got it | 17:25 |
AJaeger | but I want them merged quickly nevertheless ;) | 17:26 |
clarkb | yup both are approved though may have to reapprove the second one if they don't share a gate queue | 17:26 |
AJaeger | they don't share one - but will +1 myself if needed... | 17:27 |
*** harlowja has joined #openstack-infra | 17:31 | |
*** trown|lunch is now known as trown | 17:42 | |
*** harlowja has quit IRC | 17:43 | |
*** markvoelker_ has quit IRC | 17:45 | |
fungi | scas: i see in scrollback you addressed some job configuration issues with the imported repo content (thanks AJaeger for spotting those!). i never did hear back but i guess everything i pushed in was what you were looking for (various branches with the correct content, all tags...) | 17:48 |
scas | fungi: i thought i did mention it yesterday morning | 17:48 |
scas | it was indeed everything i needed | 17:48 |
fungi | oh, awesome i probably missed it. thanks! | 17:49 |
frickler | corvus: can you do a status notice for the cherry-pick issue? maybe once it is fixed, assuming that will happen soon. saw the issue for https://review.openstack.org/589399 , too | 17:53 |
openstackgerrit | Merged openstack-infra/zuul master: Use full gerrit changeid in HTTP report https://review.openstack.org/589530 | 18:01 |
corvus | frickler: yeah, hopefully it will be installed during the current puppet run, then i'll immediately restart and send a notice | 18:03 |
*** e0ne has joined #openstack-infra | 18:04 | |
corvus | AJaeger: i don't know, but we should look into it | 18:04 |
AJaeger | corvus: one idea: We just branched stable/rocky on the repo (my assumption, will check now) and the job does not exist in horizon on stable/rocky | 18:05 |
AJaeger | corvus: there's no stable/rocky on horizon yet | 18:05 |
corvus | AJaeger: ok. i think that's more-or-less the use case this idea of allowing configuration errors was designed for. so there may not be much for us to do, other than branch horizon. | 18:06 |
AJaeger | corvus: the job was removed from horizon, removed from msater on senlin-dashboard - and senlin-dashboard branched *before* that removal ;8 | 18:07 |
AJaeger | corvus: so, nothing we can do ;8 | 18:07 |
AJaeger | ;( | 18:07 |
corvus | AJaeger: solum-dashboard you mean? | 18:07 |
AJaeger | corvus: yes, indeed | 18:08 |
corvus | in that case, we should remove it from solum-dashboard stable/rocky | 18:08 |
AJaeger | yeah, will send a patch now... | 18:08 |
AJaeger | corvus: any ideas already about translation updates? I was wondering whether we changed the zuul behaviour and we need to use different templates like https://review.openstack.org/589372 | 18:10 |
corvus | AJaeger: in the future, we shouldn't be able to remove it from horizon as long as it's used by solum-dashboard. we should get errors reported on that now. | 18:10 |
corvus | so this case may not happen again | 18:10 |
corvus | let's look into it if it does | 18:10 |
AJaeger | ok | 18:11 |
corvus | AJaeger: i haven't had time to look into the traslation issue yet. i may not be able to today. :( | 18:11 |
openstackgerrit | Merged openstack-infra/zuul-website master: Add BMW case study https://review.openstack.org/589541 | 18:11 |
AJaeger | corvus: ok - it was broken for long enough without anybody noticing, I hope that we can wait another day | 18:12 |
AJaeger | just wanted to point out 589372 | 18:12 |
*** Swami has joined #openstack-infra | 18:12 | |
corvus | AJaeger: thanks | 18:12 |
mordred | AJaeger: we just got this: https://review.openstack.org/#/c/584484/ over in python-openstackclient | 18:13 |
mordred | AJaeger: I am certain that patch is the wrong way to do that thing .. but I wasn't sure what the correct way to communicate was | 18:13 |
AJaeger | mordred: let me -1 with comment... | 18:13 |
mordred | AJaeger: thank you! | 18:14 |
AJaeger | mordred: done | 18:14 |
clarkb | I've got to step out before the meeting. I'll be back before then | 18:15 |
openstackgerrit | David Shrewsbury proposed openstack-infra/nodepool master: Bump minimum openstacksdk version to 0.17.2 https://review.openstack.org/589563 | 18:15 |
mordred | AJaeger: hrm. there don't seem to have been any patches submitted to python-openstackclient in 5 months: https://review.openstack.org/#/c/548850/ | 18:17 |
mordred | AJaeger: did we maybe do something wrong in python-openstackclient? | 18:17 |
AJaeger | corvus: pushed https://review.openstack.org/#/c/589564/ for solum-dashboard, hope they take it in quickly... | 18:18 |
AJaeger | mordred: since 18th of July the translation sync is broken, see my discussions with eumel8 and corvus since Sunday | 18:18 |
mordred | AJaeger: ah ! great (I mean not great, but at least there is an ongoing issue already) | 18:18 |
AJaeger | mordred: BUT we only sync if there are new translated strings. so, if nobody translated the project, then nothing gets proposed | 18:19 |
corvus | AJaeger: i don't see any timer events for openstack-manuals i suspect this is the cause of the periodic issue: https://review.openstack.org/581553 (cc: tristanC) | 18:19 |
AJaeger | mordred: indeed, http://zuul.openstack.org/builds.html?job_name=propose-translation-update&project=openstack%2Fpython-openstackclient last run 18th of July | 18:19 |
corvus | zuul is installed. i will restart the scheduler now. | 18:20 |
AJaeger | sorry, need to step out | 18:20 |
corvus | actually, i won't; it has too high of an event backlog. i'm going to go grab lunch before the meeting. if someone else wants to restart the scheduler, feel free. otherwise i'll do so when i get back. | 18:21 |
openstackgerrit | Merged openstack-infra/project-config master: Add Tripleo Ansible repo https://review.openstack.org/583415 | 18:34 |
openstackgerrit | Merged openstack-infra/project-config master: devstack-plugin-kafka tests moved to project repo https://review.openstack.org/589525 | 18:34 |
openstackgerrit | Merged openstack-infra/zuul-jobs master: Un-wip upload-logs-swift https://review.openstack.org/588677 | 18:34 |
*** apetrich has quit IRC | 18:35 | |
openstackgerrit | wes hayutin proposed openstack-infra/project-config master: Add tripleo-collect-logs project https://review.openstack.org/589568 | 18:38 |
*** fabo has joined #openstack-infra | 18:40 | |
*** openstackgerrit has quit IRC | 18:49 | |
*** diablo_rojo has joined #openstack-infra | 18:52 | |
*** boden has joined #openstack-infra | 18:54 | |
*** sdatko has joined #openstack-infra | 18:57 | |
*** jamesmcarthur has quit IRC | 18:58 | |
clarkb | corvus: is the http timeout and retry causing some of the slowdown in processing results in zuul? | 18:58 |
clarkb | Also meeting time over in #openstack-meeting in 2 minutes | 18:58 |
*** jamesmcarthur has joined #openstack-infra | 18:59 | |
*** bobh has quit IRC | 19:01 | |
*** diablo_rojo has quit IRC | 19:02 | |
corvus | clarkb: possibly, though i suspect a lot of it is the number of dynamic config changes currently in flight. 582794 will help. | 19:14 |
*** bobh has joined #openstack-infra | 19:18 | |
*** openstackgerrit has joined #openstack-infra | 19:21 | |
openstackgerrit | Mohammed Naser proposed openstack/ansible-role-cloud-launcher master: Add extra_specs to flavor https://review.openstack.org/588332 | 19:21 |
openstackgerrit | Mohammed Naser proposed openstack/ansible-role-cloud-launcher master: Add os_project_access https://review.openstack.org/588335 | 19:21 |
*** bobh has quit IRC | 19:23 | |
*** mriedem has joined #openstack-infra | 19:24 | |
openstackgerrit | Merged openstack-infra/openstackid master: Fix for OIDC session checking https://review.openstack.org/589549 | 19:25 |
openstackgerrit | Mohammed Naser proposed openstack/ansible-role-cloud-launcher master: Add os_project_access https://review.openstack.org/588335 | 19:25 |
*** jamesmcarthur has quit IRC | 19:25 | |
openstackgerrit | Mohammed Naser proposed openstack/ansible-role-cloud-launcher master: Add is_public to flavor https://review.openstack.org/589574 | 19:28 |
*** e0ne has quit IRC | 19:32 | |
corvus | clarkb, tobiash: and i think something is wrong with the branch caching -- we're querying gerrit for the branch names for every tenant reconfiguration | 19:33 |
*** apetrich has joined #openstack-infra | 19:33 | |
tobiash | corvus: should there be branch caching for gerrit? | 19:34 |
fungi | that could certainly slow things down | 19:34 |
corvus | tobiash: a tenant reconfiguration is supposed to use the branch list from the previous config except for projects which have updated | 19:35 |
tobiash | corvus: it's calling getprojectbranches | 19:35 |
tobiash | In github that was never cached, but I don't know if it was for gerrit | 19:36 |
*** jamesmcarthur has joined #openstack-infra | 19:36 | |
openstackgerrit | Merged openstack-infra/project-config master: Follow wip-upload-swift rename https://review.openstack.org/589336 | 19:46 |
openstackgerrit | Jeremy Stanley proposed openstack-infra/system-config master: Remove references to long-gone odsreg.o.o service https://review.openstack.org/589580 | 19:54 |
fungi | #status log deleted a/aaaa rrs for the long-gone odsreg.openstack.org | 19:54 |
openstackstatus | fungi: finished logging | 19:54 |
fungi | cmurphy: ^ should cease to be a problem once that merges | 19:55 |
*** jamesmcarthur has quit IRC | 19:57 | |
*** jamesmcarthur has joined #openstack-infra | 19:57 | |
*** jamesmcarthur_ has joined #openstack-infra | 19:58 | |
openstackgerrit | sebastian marcet proposed openstack-infra/openstackid master: Fixes on session cookies https://review.openstack.org/589582 | 19:58 |
*** bobh has joined #openstack-infra | 19:59 | |
openstackgerrit | Slawek Kaplonski proposed openstack-infra/project-config master: Add neutron-fullstack-python35 job to Neutron dashboard https://review.openstack.org/589584 | 20:01 |
*** jamesmcarthur has quit IRC | 20:01 | |
*** sdatko has quit IRC | 20:01 | |
frickler | corvus: one issue with the swift logs and ara-report: I'm using noscript and while I'm happy with whitelisting openstack.org, doing the same for vexxhost, and possibly a varying number of other providers, might have unwanted implications | 20:04 |
* fungi predicts an ara port to shockwave flash | 20:11 | |
openstackgerrit | Merged openstack-infra/zuul master: Cache decryption results https://review.openstack.org/582794 | 20:13 |
openstackgerrit | Merged openstack-infra/zuul master: Switch storyboard url to be by name https://review.openstack.org/588597 | 20:13 |
openstackgerrit | Colleen Murphy proposed openstack-infra/system-config master: Remove references to stackalytics.o.o https://review.openstack.org/585208 | 20:13 |
*** zul has quit IRC | 20:13 | |
mordred | fungi: wow, porting to a very old version of flash ftw | 20:16 |
*** ansmith has quit IRC | 20:17 | |
openstackgerrit | Doug Hellmann proposed openstack-dev/pbr master: add python 3.6 unit test job https://review.openstack.org/589589 | 20:20 |
odyssey4me | folks, we've definitely been losing events lately - patches are submitted, but never tested - and patches approved, but never hit the gate - is this a known issue? | 20:20 |
fungi | odyssey4me: do they share a change-id in common with other changes? | 20:21 |
fungi | if so, known issue, fix in place shortly | 20:21 |
odyssey4me | here's a patch with +w around 30 mins back, and nothing in zuul: https://review.openstack.org/#/c/588508/ | 20:21 |
odyssey4me | another two: https://review.openstack.org/589064 & https://review.openstack.org/589065 | 20:21 |
odyssey4me | hmm, oh yeah - those are all backports | 20:21 |
fungi | regression from a very recent switch to start having zuul use the gerrit rest api instead of the ssh api | 20:22 |
fungi | just came to our attention a few hours back | 20:22 |
odyssey4me | ah ok, I've kinda noticed this happening for a day or two | 20:23 |
mordred | odyssey4me: https://review.openstack.org/#/c/589305/1/.zuul.yaml is what we get for our trouble though | 20:23 |
*** kgiusti has left #openstack-infra | 20:23 | |
odyssey4me | mordred: it took me a minute to figure it out - but now I see it... zuul's commenting in line? | 20:24 |
fungi | yeah, next stop is to find someone who wants to make flake8 leave inline comments in your source code | 20:24 |
openstackgerrit | Adam Coldrick proposed openstack-infra/storyboard-webclient master: Don't display 404 errors when using the "Jump to..." box https://review.openstack.org/589619 | 20:24 |
odyssey4me | that.would.be.awesome. | 20:25 |
mordred | odyssey4me: IKR? | 20:25 |
odyssey4me | zomg that is great though :) it'd be nice to have ansible-lint do much the same | 20:25 |
mordred | odyssey4me: you can! | 20:25 |
fungi | i want a variant which insults my lack of style | 20:25 |
* odyssey4me starts dribbling | 20:25 | |
odyssey4me | mordred: how?! | 20:26 |
* SotK wishes he had time to experiment with things like that | 20:26 | |
mordred | odyssey4me: if you write a thing to parse the output, you can leave info for zuul and it'll leave the comments inline | 20:26 |
mordred | odyssey4me: one sec- lemme find you the docs | 20:26 |
odyssey4me | wow, this is revolutionary - no more log trawling to get the pertinent info... for some things at least | 20:27 |
Shrews | fungi: i don't think we've loaded the hawaiian shirt module into zuul yet | 20:27 |
fungi | hah | 20:27 |
Shrews | fungi: oh, coding style ;) | 20:27 |
fungi | speaking of lack of time, i have to go try to get a seat at the sushi counter before the tourists bum-rush the joint. back in a while | 20:27 |
openstackgerrit | Ian Wienand proposed openstack-infra/zuul-jobs master: [WIP] kerberos & afs client roles https://review.openstack.org/589334 | 20:27 |
openstackgerrit | Ian Wienand proposed openstack-infra/openstack-zuul-jobs master: [WIP] openafs client install https://review.openstack.org/589335 | 20:28 |
*** diablo_rojo has joined #openstack-infra | 20:28 | |
mordred | odyssey4me: https://zuul-ci.org/docs/zuul/user/jobs.html#leaving-file-comments | 20:29 |
odyssey4me | well now, let me cast gimme-more-time++ and use the extra time to do something about that | 20:32 |
odyssey4me | looks great though! | 20:32 |
mordred | \o/ | 20:32 |
openstackgerrit | Merged openstack-infra/puppet-log_processor master: Add severity info to logstash and filter out DEBUG lines https://review.openstack.org/588383 | 20:33 |
* mordred is excited for flake8 jobs to grow the feature - but hasn't yet grown enough time to actually do something about it | 20:33 | |
corvus | restarting zuul nowv | 20:34 |
corvus | status notice Due to a bug, Zuul has been unable to report on cherry-picked changes over the last 24 hours. This has now been fixed; if you encounter a cherry-picked change missing its results (or was unable to merge), please recheck now. | 20:40 |
corvus | how's that look? | 20:40 |
mordred | ++ | 20:40 |
corvus | (re-enqueue is proceeding now) | 20:40 |
clarkb | lgtm | 20:41 |
corvus | #status notice Due to a bug, Zuul has been unable to report on cherry-picked changes over the last 24 hours. This has now been fixed; if you encounter a cherry-picked change missing its results (or was unable to merge), please recheck now. | 20:41 |
openstackstatus | corvus: sending notice | 20:41 |
corvus | okay, back to branch caching | 20:41 |
-openstackstatus- NOTICE: Due to a bug, Zuul has been unable to report on cherry-picked changes over the last 24 hours. This has now been fixed; if you encounter a cherry-picked change missing its results (or was unable to merge), please recheck now. | 20:42 | |
ianw | corvus / clarkb: one accessbot to look at when you have time -- https://review.openstack.org/589001 -- after looking at it with fresh eyes, i realised what the "global" structure was doing and that we don't need separate keys | 20:44 |
openstackstatus | corvus: finished sending notice | 20:44 |
*** dpawlik has joined #openstack-infra | 20:45 | |
*** apetrich has quit IRC | 20:45 | |
*** nicolasbock has quit IRC | 20:45 | |
openstackgerrit | Merged openstack-infra/openstackid master: Fixes on session cookies https://review.openstack.org/589582 | 20:51 |
openstackgerrit | Monty Taylor proposed openstack-infra/zuul-jobs master: WIP Extract pep8 messages for inline comments https://review.openstack.org/589634 | 20:55 |
mordred | Shrews, fungi, odyssey4me: ok. if I get lucky, https://review.openstack.org/#/c/589635/ will have file comments for a pep8 job - depending on that ^^ | 20:56 |
mordred | of course, I cannot remember the last time I got something right on the first try | 20:56 |
*** slaweq has quit IRC | 20:58 | |
corvus | apparently not this time :) | 20:59 |
corvus | or so says software factory | 20:59 |
Shrews | mordred: you probably should handle the parameter to extract_file_comments() :) | 21:00 |
corvus | mordred: before we go crazy on this, we might want to think about https://review.openstack.org/579033 | 21:01 |
mordred | corvus: oh - yes - I was looking for that earlier and coulnd't find it | 21:01 |
corvus | mordred: my current thought is that jhesketh is right, and that is the wrong approach. but it might be worth making sure we can implement jhesketh's idea first :) | 21:01 |
corvus | or, i guess, we could decide it's definitely the right approach, and we must find a way to implement it. :) | 21:02 |
corvus | the downside to that is it'd be extra work for the mergers. but hey. | 21:03 |
corvus | they need to feel appreciated | 21:03 |
mordred | I think I'm leaning towards jhesketh being right - like, I'm not sure how I'd make use of that role in the patch I made above | 21:03 |
corvus | mordred: register? | 21:04 |
mordred | because I can get a handle to the stdout of the command in that playbook, but once we're out of that playbook, it's weird | 21:04 |
mordred | oh, yeah, I guess I could just include_role | 21:04 |
corvus | anyway, there's like 2 serious zuul bugs right now, so it'll be a couple days before i get to that | 21:06 |
mordred | totally | 21:06 |
mordred | it does feel like something that should be handled on the user's behalf as part of the zuul_return interface - writing to the zuul_return interface is easy/nice - remembering to pass the results through the mapping seems like a thing that people will forget to do and then be confusde | 21:06 |
corvus | yeah. it would still need to be bypassable though, so we'd probably have to add a flag to disable the mapping. but i think map-by-default is the right way to go if we do that. | 21:07 |
mordred | ++ | 21:07 |
mordred | if you need to disable it, you're more likely to know what you're doing | 21:07 |
corvus | mordred: however, if we were to go with the mapping role, we could change the docs to say "use this role" and bury the actual api. | 21:07 |
corvus | anyway, i think we're all on the same page now that internal is better; it's just a bit too hand-wavey for me to say for sure we'll be able to do it. :) | 21:08 |
mordred | we could - but I kinda like that there is an api, and that it's well documented and easy to understand | 21:08 |
mordred | ++ | 21:08 |
mordred | if we can't do it - then I totally agree we should just say "use this role" :) | 21:08 |
mordred | corvus: we could also do a hybrid internal/external and write a new module similar to zuul_return | 21:09 |
corvus | mordred: also, i sort of imagined the role could take yaml/json file inputs, etc, so folks could do your pep8 translating but maybe in a script instead of a module. modules are cool though. :) | 21:10 |
openstackgerrit | Monty Taylor proposed openstack-infra/zuul-jobs master: WIP Extract pep8 messages for inline comments https://review.openstack.org/589634 | 21:10 |
mordred | Shrews: fixed your comment :) | 21:11 |
corvus | mordred: sf caught an error too | 21:12 |
mordred | \o/ | 21:13 |
mordred | bwhahahaha | 21:14 |
openstackgerrit | Monty Taylor proposed openstack-infra/zuul-jobs master: WIP Extract pep8 messages for inline comments https://review.openstack.org/589634 | 21:15 |
*** slaweq has joined #openstack-infra | 21:23 | |
*** boden has quit IRC | 21:23 | |
*** trown is now known as trown|outtypewww | 21:27 | |
*** edmondsw has quit IRC | 21:29 | |
clarkb | corvus: I've restarted the log worker proceses on logstash-worker01 to pick up the new client side filtering | 21:39 |
clarkb | it should noop for now, but should shake out any major issues | 21:39 |
corvus | clarkb: thanks! | 21:39 |
clarkb | re monty asking for reviews on the ansible base server config changes, https://review.openstack.org/#/c/587985/7 is the bottleneck currently. If we get that approved a bunch of the stack behind it can be approved as well | 21:40 |
*** jamesmcarthur_ has quit IRC | 21:41 | |
mordred | ++ | 21:42 |
* mordred is offering booze | 21:42 | |
ianw | mordred: so this playbook isn't actually run *on* bridge.o.o? | 21:48 |
mordred | ianw: well, it is | 21:49 |
mordred | ianw: but I may have also already run it on bridge | 21:49 |
mordred | or, more correctly - I ran it on my laptop | 21:50 |
mordred | pointed at bridge | 21:50 |
ianw | right ... but then ansible running on bridge upgrades it's own ansible? | 21:50 |
mordred | yup | 21:50 |
mordred | moving forward | 21:50 |
ianw | and that doesn't eat itself? | 21:50 |
mordred | nope - should be fine :) | 21:50 |
mordred | ianw: thanks! | 21:55 |
*** bdodd has quit IRC | 21:56 | |
*** slagle has quit IRC | 21:56 | |
mordred | clarkb, ianw: I clicked +a on the stack - will continue iterating on bridge when it lands | 21:57 |
mordred | if you feel like another two: https://review.openstack.org/#/c/588626/ and https://review.openstack.org/#/c/588627 | 21:57 |
clarkb | mordred: why would we use ansible-role-puppet if they aren't running puppet? | 22:00 |
clarkb | wouldn't it be easier to just not run that role against those nodes? | 22:01 |
clarkb | oh this is to modify the hierdata on the "master" | 22:01 |
clarkb | even then that is just a fact so it should be fine? | 22:01 |
ianw | it made sense to me as ansible-role-puppet being run on bridge.o.o that has no puppet, but still wanting to pretend the ansible vars are hiera data | 22:03 |
clarkb | mordred: oh hrm I think I understand now. I thought the point was to keep the data in /etc/puppet/hieradata | 22:03 |
clarkb | then point ansible at that? | 22:03 |
clarkb | but its the inverse, put hieradata in /etc/ansible/hosts and point puppet at that | 22:03 |
*** dpawlik has quit IRC | 22:09 | |
ianw | my understanding, may be wrong -- ansible on bridge is copying the hiera files from /etc/puppet/hieradata/+$(something from puppet --version) | 22:09 |
*** owalsh_ has joined #openstack-infra | 22:09 | |
*** priteau has quit IRC | 22:09 | |
ianw | to the remote host | 22:10 |
clarkb | ianw: that is the default but then mordred is overriding it to be /etc/ansible/hosts at https://review.openstack.org/#/c/588627/1/playbooks/host_vars/bridge.openstack.org.yaml I had to -1 the parent change as there is a bug in that setup | 22:10 |
openstackgerrit | Adam Coldrick proposed openstack-infra/storyboard-webclient master: Don't display 404 errors when using the "Jump to..." box https://review.openstack.org/589619 | 22:11 |
openstackgerrit | Adam Coldrick proposed openstack-infra/storyboard-webclient master: Add a route to redirect from a Task ID to the relevant Story https://review.openstack.org/589652 | 22:11 |
openstackgerrit | Adam Coldrick proposed openstack-infra/storyboard-webclient master: Support Task ID in the "Jump to..." box https://review.openstack.org/589653 | 22:11 |
*** rosmaita has quit IRC | 22:11 | |
*** owalsh has quit IRC | 22:12 | |
*** owalsh- has joined #openstack-infra | 22:12 | |
*** bobh has quit IRC | 22:12 | |
*** owalsh- is now known as owalsh | 22:13 | |
*** beagles has quit IRC | 22:13 | |
*** owalsh_ has quit IRC | 22:14 | |
mordred | clarkb, ianw: right - on bridge there is no puppet, so there is no hiera nor is there hieradata | 22:16 |
ianw | i think i agree maybe with clarkb that puppet_get_hiera_file_list might be looking in the wrong place | 22:16 |
clarkb | mordred: well there is hieradata, but not hiera | 22:16 |
mordred | it's not hieradata | 22:17 |
mordred | it's just data | 22:17 |
mordred | but yes - I agree with you that there is a bug | 22:17 |
mordred | good catch :) | 22:17 |
*** beagles has joined #openstack-infra | 22:20 | |
*** rcernin has joined #openstack-infra | 22:20 | |
*** strigazi has joined #openstack-infra | 22:20 | |
*** strigazi has quit IRC | 22:21 | |
*** strigazi has joined #openstack-infra | 22:21 | |
openstackgerrit | Monty Taylor proposed openstack-infra/ansible-role-puppet master: Allow explicit override for mgmt_hieradata https://review.openstack.org/588626 | 22:23 |
mordred | clarkb, ianw: ^^ how does that look? | 22:23 |
*** strigazi has quit IRC | 22:27 | |
*** strigazi has joined #openstack-infra | 22:28 | |
*** strigazi has quit IRC | 22:28 | |
*** strigazi has joined #openstack-infra | 22:29 | |
*** strigazi has quit IRC | 22:32 | |
openstackgerrit | Kendall Nelson proposed openstack-infra/storyboard master: Make email public field https://review.openstack.org/589663 | 22:33 |
*** strigazi has joined #openstack-infra | 22:33 | |
openstackgerrit | Ian Wienand proposed openstack-infra/zuul-jobs master: [WIP] kerberos & afs client roles https://review.openstack.org/589334 | 22:38 |
*** hongbin has quit IRC | 22:39 | |
openstackgerrit | Merged openstack-infra/system-config master: Install and configure ansible on bridge https://review.openstack.org/587985 | 22:40 |
*** diablo_rojo has quit IRC | 22:41 | |
*** gothicmindfood has quit IRC | 22:42 | |
openstackgerrit | Merged openstack-infra/system-config master: Add an Ansible role to configure exim https://review.openstack.org/588089 | 22:43 |
openstackgerrit | Merged openstack-infra/system-config master: Update install_modules to not need puppet https://review.openstack.org/588394 | 22:43 |
*** bradm has joined #openstack-infra | 22:44 | |
clarkb | mordred: +2'd but not approving as I need to run some errands and that will affect existing puppetmaster | 22:45 |
*** gothicmindfood has joined #openstack-infra | 22:45 | |
openstackgerrit | Merged openstack-infra/system-config master: Rename update_puppet to update-system-config https://review.openstack.org/588396 | 22:47 |
openstackgerrit | Merged openstack-infra/system-config master: Fix some little ansible issues https://review.openstack.org/588397 | 22:52 |
openstackgerrit | Merged openstack-infra/system-config master: Replace with_first_found with lookup first_found https://review.openstack.org/588546 | 22:52 |
*** rpioso is now known as rpioso|afk | 22:54 | |
*** edmondsw has joined #openstack-infra | 22:54 | |
*** ansmith has joined #openstack-infra | 22:55 | |
*** dbkinder_ has joined #openstack-infra | 22:58 | |
*** edmondsw has quit IRC | 22:59 | |
dbkinder_ | Howdy. I'm working with the StarlingX folks and we'd like to upload a 3MB PDF files to the wiki (wiki.openstack.org/wiki/StarlingX but PDF files aren't on the allowed list and there's a limit of 2MB files. Help? | 22:59 |
dbkinder_ | (and is this the right place to ask for support?) | 23:00 |
fungi | dbkinder_: the wiki isn't really a file server | 23:10 |
fungi | also we disabled pdf uploads way back because the fact that they can't be content-checked but search engines still index them made them a ripe target for spammers, and so we were getting flooded with pdf spam | 23:11 |
openstackgerrit | James E. Blair proposed openstack-infra/project-config master: Add openstack-manuals to periodic pipeline https://review.openstack.org/589669 | 23:12 |
*** tpsilva has quit IRC | 23:12 | |
fungi | dbkinder_: any reason you don't want to just serve pdfs from a website? what's the benefit of putting a pdf on a wiki server? | 23:13 |
corvus | AJaeger, eumel8, tristanC: without having spent much time looking into it, https://review.openstack.org/589669 is my first guess as to what the periodic bug may be. that's an easy thing for us to try real quick. if that's it, it should be an easy bugfix to zuul and then we can remove it. | 23:13 |
corvus | fungi, clarkb, ianw: ^ can we merge that change soon, so we see it run in tonights periodic builds? | 23:13 |
fungi | looking now | 23:14 |
dbkinder_ | @fungi hmmm. OK I guess. We'll find another place to put the PDF and link to it from the wiki. | 23:14 |
dbkinder_ | thanks for the explanation. | 23:14 |
*** priteau has joined #openstack-infra | 23:15 | |
mordred | clarkb: ++ to not merging it | 23:22 |
openstackgerrit | Ian Wienand proposed openstack-infra/zuul-jobs master: [WIP] kerberos & afs client roles https://review.openstack.org/589334 | 23:22 |
mordred | clarkb: in fact, now that the install_modules change has landed, I'm in a position to try some things directly on bridge - so I can also confirm it works there as expected | 23:22 |
mordred | tomorrow | 23:22 |
mordred | not today. today it has become caipirinha-thirty | 23:23 |
dbkinder_ | @fungi We're currently using the wiki as the "website" while the doc infrastructure get understood and resolved. Are you behind the starlingx.io site? | 23:34 |
openstackgerrit | Ian Wienand proposed openstack-infra/zuul-jobs master: [WIP] kerberos & afs client roles https://review.openstack.org/589334 | 23:36 |
fungi | dbkinder_: probably not. at least not that i'm aware (unless you're keeping your site content in git/gerrit and writing ci jobs to build and deploy it) | 23:37 |
dbkinder_ | not yet... (I do that for other open source projects such as docs.zephyrproject.org). Do you know who's behind www.starlingx.io ? | 23:38 |
fungi | dbkinder_: we handle the collaboratively-operated/community-maintained infrastructure in here. the openstack foundation has their own web dev team who also does websites and mobile apps for some things | 23:38 |
dbkinder_ | ah. someone at the openstack foundation then. I'll need to track them down. sigh. | 23:39 |
dbkinder_ | thanks. | 23:39 |
fungi | dbkinder_: we see some of them in here from time to time (jamesmacarthur, jpmaxman, smarcet) but none seem to be in channel at the moment | 23:39 |
openstackgerrit | Merged openstack-infra/project-config master: Add openstack-manuals to periodic pipeline https://review.openstack.org/589669 | 23:42 |
fungi | dbkinder_: i have some alternative channels i can reach them through, if you like. i can find out what the best way is for you to reach out to them... are you mostly just looking to get that pdf served from somewhere on the site? | 23:43 |
dbkinder_ | that was the short term problem, but long term, we're trying to learn the "openstack way" to produce and publish documentation (I'm familiar with reST/Sphinx and such from other projects). | 23:44 |
*** Swami has quit IRC | 23:46 | |
fungi | dbkinder_: yeah, i expect the current starlingx.io site is mostly a placeholder while they work with you on something more collaborative. for example we publish https://zuul-ci.org/ from content in https://git.zuul-ci.org/cgit/zuul-website | 23:48 |
openstackgerrit | Ian Wienand proposed openstack-infra/openstack-zuul-jobs master: [WIP] openafs client install https://review.openstack.org/589335 | 23:49 |
fungi | zuul jobs react to merges in that repo and continuously deploy builds of the content | 23:49 |
mordred | dbkinder_, fungi: we've talked about some new tools in that area to allow for the business-y types and the coder-y types to be better able to directly collaborate on content that might have aspects of both. so far I haven't had time to dig in on my part of that | 23:51 |
mordred | (it would still be built and published by zuul from sources in git - the main theoretical new thing would be the addition of a cms-like frontend to the process that could be used by those for whom that process feels more natural) | 23:52 |
mordred | fungi: I should probably write something down about those thoughts and the chats that were had in vancouver with humans shouldn't I? | 23:53 |
dbkinder_ | mordred, fungi: OK, thanks for the info. Gotta head out now. | 23:53 |
fungi | that would be pretty awesome to have written out | 23:54 |
dbkinder_ | you mean document stuff, how novel :) | 23:54 |
fungi | dbkinder_: i've asked how/who to put you in touch with regarding the website. will let you know | 23:54 |
dbkinder_ | ok. david.b.kinder@intel.com | 23:55 |
mordred | dbkinder_: documenting things about documenting things. my favorite :) | 23:55 |
fungi | dbkinder_: thanks! | 23:55 |
dbkinder_ | thank you! | 23:56 |
dbkinder_ | ttfn | 23:56 |
*** dbkinder_ has quit IRC | 23:56 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!