Tuesday, 2018-08-07

*** gyee has quit IRC00:06
*** dingyichen has joined #openstack-infra00:10
*** xarses has quit IRC00:19
*** zhurong has joined #openstack-infra00:23
*** harlowja has quit IRC00:26
*** bobh has quit IRC00:28
*** mrsoul has joined #openstack-infra01:01
*** hongbin has joined #openstack-infra01:03
*** mschuppert has quit IRC01:04
*** mriedem has quit IRC01:15
*** pbourke has quit IRC01:17
*** pbourke has joined #openstack-infra01:19
*** zhurong has quit IRC01:26
*** gfidente|afk has quit IRC01:41
*** auristor has quit IRC01:50
*** mrsoul has quit IRC02:02
*** bobh has joined #openstack-infra02:13
*** lbragstad has quit IRC02:18
*** auristor has joined #openstack-infra02:28
*** xarses has joined #openstack-infra02:34
*** bobh has quit IRC02:34
*** psachin has joined #openstack-infra02:40
*** xarses has quit IRC02:42
*** auristor has quit IRC02:57
*** auristor has joined #openstack-infra02:59
*** rosmaita has quit IRC03:13
*** jhesketh_ is now known as jhesketh03:14
*** ramishra has joined #openstack-infra03:16
*** vivsoni_ has quit IRC03:19
*** dave-mccowan has quit IRC03:31
*** janki has joined #openstack-infra03:41
*** xarses has joined #openstack-infra03:43
*** udesale has joined #openstack-infra03:48
*** Bhujay has joined #openstack-infra04:13
*** vivsoni has joined #openstack-infra04:14
*** hongbin has quit IRC04:15
openstackgerritMerged openstack-infra/project-config master: accessbot config: refactor  https://review.openstack.org/58899904:19
openstackgerritMerged openstack-infra/project-config master: Set python35-jobs as voting for monasca-agent  https://review.openstack.org/58735704:19
*** annp has joined #openstack-infra04:19
openstackgerritIan Wienand proposed openstack-infra/zuul-jobs master: [WIP] kerberos & afs client roles  https://review.openstack.org/58933404:21
*** Bhujay has quit IRC04:30
AJaegerconfig-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
AJaegertonyb: thanks for reviewing!04:32
tonybAJaeger: np, sorry it took most of my day to get there.04:34
openstackgerritIan Wienand proposed openstack-infra/openstack-zuul-jobs master: [WIP] openafs client install  https://review.openstack.org/58933504:34
openstackgerritAndreas Jaeger proposed openstack-infra/project-config master: Follow wip-upload-swift rename  https://review.openstack.org/58933604:35
*** ykarel has joined #openstack-infra04:35
openstackgerritIan Wienand proposed openstack-infra/openstack-zuul-jobs master: [WIP] openafs client install  https://review.openstack.org/58933504:36
AJaegertonyb: no worries, I'm just up ;) will you help retiring tripleo stable/newton again?04:36
AJaegerianw: could you also review https://review.openstack.org/#/c/588677/ , please?04:36
tonybAJaeger: 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 it04:37
AJaegerianw: still around and have time to help? Or want to do it tomorrow?04:37
AJaegertonyb: remember that the updated script removes the tag parameter04:38
openstackgerritMerged openstack-infra/project-config master: Import eol_branch.sh  https://review.openstack.org/58923604:38
tonybAJaeger: will do.04:38
ianwAJaeger: i'm around04:38
AJaegerianw: 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.html04:39
tonybAJaeger: I was a little confused why you left EOL_MESSAGE in 237 but it made sense once I really looked at it ;P04:39
openstackgerritMerged openstack-infra/project-config master: Remove eol tagging from eol_branch.sh  https://review.openstack.org/58923704:39
* AJaeger wants to get rid of the error thta shows in http://zuul.openstack.org/api/config-errors04:39
ianwAJaeger tonyb: done04:39
AJaegertonyb: I first removed EOL_MESSAGE but then noticed the other usage...04:40
AJaegerargh, we now have errors in openstack-chef as well ;(04:40
tonybianw: Thanks.04:40
AJaegerfungi: ^04:40
AJaegerscas: please cleanup openstack-chef and openstack-chef-repo zuul configuration, see http://zuul.openstack.org/api/config-errors04:41
AJaegerscas: might be easiest to remove .zuul.yaml from openstack-chef-repo ...04:42
AJaegerthanks, ianw04:44
scasAJaeger: 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 over04:49
openstackgerritIan Wienand proposed openstack-infra/zuul-jobs master: [WIP] kerberos & afs client roles  https://review.openstack.org/58933404:51
openstackgerritIan Wienand proposed openstack-infra/openstack-zuul-jobs master: [WIP] openafs client install  https://review.openstack.org/58933504:52
tonybAJaeger, ianw: those repos don't actuallt seem to have the newton-eol tags http://paste.openstack.org/show/727457/04:54
tonybfor now I can easily use the HEAD^ version of eol_branch.sh04:54
AJaegerscas: thanks - I'm aware of the process04:55
AJaegertonyb: https://review.openstack.org/#/c/583856/ merged - did something fail?04:55
tonybAJaeger: I *think* actually that doesn't tag things on newton04:56
tonybAJaeger: because 'newton' is closed.04:56
AJaegerargh ;(04:56
tonybAJaeger: I think dhellman was supposed to run the tool manually04:56
AJaegerso, you need the old version of the script and tag yourself?04:57
AJaegergo ahead IMHO - ianw, do you agree?04:57
tonybAJaeger: No problem. I'll just use eol_branch.sh from 18aea9c6a04:57
AJaegerthanks04:58
ianwi think it would be right to tag, fwiw ...05:00
*** e0ne has joined #openstack-infra05:00
scasAJaeger: it's late here and i patched over it at the same time05:01
*** xarses has quit IRC05:03
scasAJaeger: it's going to be angry until i patch over the other four open branches05:04
*** stakeda has joined #openstack-infra05:04
AJaegerscas: no worries - do it tomorrow your day and call it a night ;)05:05
AJaegerscas: thanks05:05
scasi'll get a project-config change for my finalizing stuff up during my day05:05
scasprobably less churn than noop'ing the whole repo05:06
tonybAJaeger: done.05:07
AJaegerthanks, tonyb !05:09
tonybAJaeger: You're welcome thanks for the prod05:10
scasnever 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 yaml05:14
AJaegerscas: if you retire the repo, you remove from project-config/zuul/main.yaml - and then it does not show up either05:17
openstackgerritIan Wienand proposed openstack-infra/openstack-zuul-jobs master: [WIP] openafs client install  https://review.openstack.org/58933505:18
*** psachin has quit IRC05:18
*** psachin has joined #openstack-infra05:25
*** jaosorior has quit IRC05:26
scasAJaeger: it's clearing. thanks for the vigilance05:34
openstackgerritIan Wienand proposed openstack-infra/zuul-jobs master: [WIP] kerberos & afs client roles  https://review.openstack.org/58933405:38
*** xarses has joined #openstack-infra05:44
openstackgerritIan Wienand proposed openstack-infra/zuul-jobs master: [WIP] kerberos & afs client roles  https://review.openstack.org/58933405:44
*** nicolasbock has joined #openstack-infra05:46
openstackgerritIan Wienand proposed openstack-infra/openstack-zuul-jobs master: [WIP] openafs client install  https://review.openstack.org/58933505:57
*** hwoarang has joined #openstack-infra05:57
*** e0ne has quit IRC05:58
*** apetrich has joined #openstack-infra06:01
*** e0ne has joined #openstack-infra06:02
*** jaosorior has joined #openstack-infra06:05
AJaegerscas: indeed, it's empty - thanks!06:08
AJaeger#status log http://zuul.openstack.org/api/config-errors shows *no* errors06:08
openstackstatusAJaeger: finished logging06:08
openstackgerritAndreas Jaeger proposed openstack-infra/openstack-zuul-jobs master: Rework translation templates  https://review.openstack.org/58937206:12
AJaegercorvus, 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 fix06: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 IRC06:15
*** xarses has joined #openstack-infra06:15
eumel8AJaeger: I'm not sure if it would helps06:16
*** xarses_ has joined #openstack-infra06:17
*** xarses has quit IRC06:20
*** quiquell has joined #openstack-infra06:21
openstackgerritIan Wienand proposed openstack-infra/infra-specs master: letsencrypt spec  https://review.openstack.org/58728306:28
*** pcaruana has joined #openstack-infra06:36
*** Bhujay has joined #openstack-infra06:42
*** ginopc has joined #openstack-infra07:00
*** e0ne has quit IRC07:02
*** amoralej|off is now known as amoralej07:08
*** mpeterson has quit IRC07:23
*** mpeterson has joined #openstack-infra07:23
*** dpawlik has joined #openstack-infra07:23
*** wznoinsk has quit IRC07:27
*** wznoinsk has joined #openstack-infra07:27
*** e0ne has joined #openstack-infra07:35
*** jpena|off is now known as jpena07:35
*** e0ne has quit IRC07:41
*** tosky has joined #openstack-infra07:42
*** ykarel is now known as ykarel|lunch07:43
*** jpich has joined #openstack-infra07:45
*** jaosorior has quit IRC07:49
*** e0ne has joined #openstack-infra07:52
*** e0ne has quit IRC07:53
*** apetrich has quit IRC07:56
*** yamamoto has joined #openstack-infra08:00
*** jaosorior has joined #openstack-infra08:02
AJaegereumel8: let's see what the problem is ;)08:06
*** yamamoto has quit IRC08:07
*** rcernin has quit IRC08:11
*** dayou has quit IRC08:12
*** boden has joined #openstack-infra08:14
*** Emine has joined #openstack-infra08:16
*** boden has quit IRC08:19
*** owalsh_ is now known as owalsh08:33
*** electrofelix has joined #openstack-infra08:33
*** panda|rover-ish is now known as panda|rover08:33
*** ykarel|lunch is now known as ykarel08:35
*** gfidente has joined #openstack-infra08:35
*** yamamoto has joined #openstack-infra08:46
*** derekh has joined #openstack-infra08:46
*** priteau has joined #openstack-infra08:55
evrardjpAJaeger: 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
evrardjpthank you in advance.08:58
*** jaosorior has quit IRC09:02
cmurphyevrardjp: i'm not a core on bindep nor do i know much about it but left a comment anyways09:02
*** dingyichen has quit IRC09:02
evrardjpas long as we talk about it here, and it's not dying, I am good.09:03
evrardjp:D09:03
evrardjphitting the nail until those merge.09:03
AJaegerevrardjp: I'm not a core there either - but did some patches. Will look later at it.09:04
evrardjpoh that's sad. I got 0% success rate in blind ping.09:05
evrardjplet's try more focused then.09:05
AJaegerevrardjp: you need infra-root for bindep changes09:06
AJaegerevrardjp: best ask again during US morning09:06
evrardjpAJaeger: yeah, or Aussie time with jhesketh :)09:07
evrardjpI will try later.09:07
jheskethevrardjp: I was just pulling up the changes actually :-)09:07
evrardjpoh you're there? Isn't it late for you?09:08
evrardjphaha that's great!09:08
*** yamamoto has quit IRC09:11
jheskethit's just past 7pm and I just returned from dinner, so good timing :-)09:12
*** finucannot is now known as stephenfin09:30
*** zoli is now known as zoli|lunch09:34
*** roman_g has joined #openstack-infra10:01
*** gema has joined #openstack-infra10:14
*** jamesdenton has quit IRC10:18
*** electrofelix has quit IRC10:18
*** olivierbourdon38 has joined #openstack-infra10:26
*** electrofelix has joined #openstack-infra10:26
*** jamesdenton has joined #openstack-infra10:37
*** dayou has joined #openstack-infra10:42
*** jaosorior has joined #openstack-infra10:46
*** dave-mccowan has joined #openstack-infra10:51
*** zeus has quit IRC10:53
*** zoli|lunch is now known as zoli10:53
*** bradm has quit IRC10:56
*** hs366 has quit IRC11:14
*** udesale has quit IRC11:15
*** Bhujay has quit IRC11:18
*** gfidente has quit IRC11:18
*** panda|rover is now known as panda|rover|lunc11:18
*** gfidente has joined #openstack-infra11:20
*** vivsoni has quit IRC11:24
*** stakeda has quit IRC11:26
*** gfidente has quit IRC11:28
*** gfidente has joined #openstack-infra11:30
*** vivsoni has joined #openstack-infra11:31
*** slagle has joined #openstack-infra11:39
*** gbutnaru has joined #openstack-infra11:44
*** apetrich has joined #openstack-infra11:53
*** zul has joined #openstack-infra11:54
*** boden has joined #openstack-infra11:56
*** jaosorior has quit IRC11:59
*** edmondsw has joined #openstack-infra12:04
*** tpsilva has joined #openstack-infra12:07
*** amoralej is now known as amoralej|lunch12:15
*** jpena is now known as jpena|lunch12:15
*** panda|rover|lunc is now known as panda|rover12:16
*** jaosorior has joined #openstack-infra12:18
*** Bhujay has joined #openstack-infra12:24
*** jaosorior has quit IRC12:27
evrardjpmordred: 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
mordredoh no, I'm not invisible12:37
AJaeger;)12:38
*** e0ne has joined #openstack-infra12:40
*** jaosorior has joined #openstack-infra12:42
*** sthussey has joined #openstack-infra12:45
*** sdoran_ has joined #openstack-infra12:49
*** auristor has quit IRC12:50
*** sdoran has quit IRC12:50
*** weshay has quit IRC12:50
*** sdoran_ is now known as sdoran12:50
AJaegerconfig-core, could you review https://review.openstack.org/588920 and https://review.openstack.org/589336 , please?12:54
*** bobh has joined #openstack-infra12:54
openstackgerritsebastian marcet proposed openstack-infra/openstackid-resources master: Presentation POST/PUT Validation  https://review.openstack.org/58945712:55
openstackgerritMerged openstack-infra/openstackid-resources master: Presentation POST/PUT Validation  https://review.openstack.org/58945712:58
*** ccamacho has quit IRC12:58
*** dklyle has quit IRC12:59
openstackgerritMerged openstack-infra/bindep master: Add opensuse-leap support  https://review.openstack.org/58603813:01
openstackgerritMerged openstack-infra/bindep master: Fix bindep path  https://review.openstack.org/58825313:01
AJaegerevrardjp, I guess we need a new bindep release now - correct? Please work with clarkb on this once he's awake13:02
openstackgerritMerged openstack-infra/bindep master: Fix user profile display  https://review.openstack.org/58495513:03
evrardjpmordred: I cannot not think of https://www.youtube.com/watch?v=zKdxd718WXg .13:03
*** janki has quit IRC13:03
evrardjpAJaeger: will do.13:04
*** janki has joined #openstack-infra13:04
*** kgiusti has joined #openstack-infra13:06
*** auristor has joined #openstack-infra13:07
*** jamesdenton has quit IRC13:10
*** jamesdenton has joined #openstack-infra13:11
*** trown|outtypewww is now known as trown13:12
*** mriedem has joined #openstack-infra13:16
*** ykarel is now known as ykarel|away13:21
*** eharney has joined #openstack-infra13:22
*** ccamacho has joined #openstack-infra13:28
*** jpena|lunch is now known as jpena13:30
*** ykarel|away has quit IRC13:31
*** amoralej|lunch is now known as amoralej13:32
scasrandom 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 it13:37
dhellmannAJaeger , tonyb : did you need me to do something with the tripleo newton stuff?13:38
AJaegerdhellmann: all done and fine AFAIK - thanks for checking13:39
dhellmannok, np13:41
AJaegerdhellmann: 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-infra13:42
*** r-mibu has joined #openstack-infra13:42
dhellmannAJaeger : that's the API to fetch the jobs configured for a repo?13:42
dhellmannoh, that gives the job details13:43
AJaegerthe first part - not sure whether you need 52891 as well13:43
dhellmannI don't remember why we needed that13:43
AJaegeror https://review.openstack.org/#/c/55097913:44
dhellmannI think that's the we would need13:44
dhellmann*the one13:44
AJaegerI see13:44
dhellmannI put together some interim code that just looks at project-config and the repo's zuul files for now13:45
*** quiquell is now known as quiquell|off13:47
*** psachin has quit IRC13:51
*** quiquell|off has quit IRC13:51
*** jamesdenton has quit IRC13:52
*** jamesdenton has joined #openstack-infra13:53
openstackgerritMonty Taylor proposed openstack-infra/zuul master: web: add /{tenant}/projects and /{tenant}/project/{project} routes  https://review.openstack.org/55097913:56
openstackgerritMonty Taylor proposed openstack-infra/zuul master: web: add /{tenant}/pipelines route  https://review.openstack.org/54152113:56
openstackgerritMonty Taylor proposed openstack-infra/zuul master: scheduler: add job's parent name to the rpc job_list method  https://review.openstack.org/57347313:56
openstackgerritMonty Taylor proposed openstack-infra/zuul master: web: add /{tenant}/labels route  https://review.openstack.org/55397913:56
openstackgerritMonty Taylor proposed openstack-infra/zuul master: web: add /{tenant}/nodes route  https://review.openstack.org/55399813:56
openstackgerritMerged openstack-infra/project-config master: Check for empty project entries  https://review.openstack.org/58892014:02
*** jamesmcarthur has joined #openstack-infra14:05
*** jrist has quit IRC14:15
*** Bhujay has quit IRC14:18
*** nicolasbock has quit IRC14:18
*** rpioso|afk is now known as rpioso14:26
*** jrist has joined #openstack-infra14:28
openstackgerritsebastian 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/58950214:33
*** dklyle has joined #openstack-infra14:33
*** hwoarang has quit IRC14:37
*** ykarel has joined #openstack-infra14:38
*** hongbin has joined #openstack-infra14:38
*** e0ne has quit IRC14:39
*** rasca is now known as rascasoft14:49
*** priteau has quit IRC14:50
openstackgerritMerged openstack-infra/nodepool master: Switch storyboard url to be by name  https://review.openstack.org/58860014:55
*** bobh has quit IRC14:56
*** ykarel has quit IRC14:57
*** rosmaita has joined #openstack-infra14:57
*** Emine has quit IRC14:58
*** jaosorior has quit IRC15:01
*** panda|rover is now known as panda|backin2h15:01
*** lamt has joined #openstack-infra15:02
*** mriedem is now known as mriedem_afk15:06
*** nicolasbock has joined #openstack-infra15:07
*** dpawlik has quit IRC15:10
*** pcaruana has quit IRC15:11
*** hemna_ has joined #openstack-infra15:12
mordredcorvus: 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
openstackgerritMerged openstack-infra/system-config master: Remove mention of excercises.sh  https://review.openstack.org/58314715:16
*** bobh has joined #openstack-infra15:17
*** bobh has quit IRC15:17
toskyI see something weird, it's like zuul did not report back the results to gerrit for few reviews against stable/pike15:19
toskynamely: https://review.openstack.org/#/c/589395/ https://review.openstack.org/#/c/589396/ and https://review.openstack.org/#/c/588515/15:19
toskythe jobs were in the zuul gate queue, and I don't see unmerged dependencies15:19
*** Bhujay has joined #openstack-infra15:23
*** priteau has joined #openstack-infra15:26
*** hamerins has joined #openstack-infra15:30
*** gyee has joined #openstack-infra15:32
*** ramishra has quit IRC15:34
*** dpawlik has joined #openstack-infra15:38
*** boden has quit IRC15:41
*** dpawlik has quit IRC15:42
*** v1k0d3n has joined #openstack-infra15:44
*** bobh has joined #openstack-infra15:45
*** olivierbourdon38 has quit IRC15:46
evrardjpHow 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 still15:47
evrardjppresent in case of infrastructure failure - disk crash or flush, etc.)15:47
toskyzuul ate my job results :)15:48
clarkbtosky: corvus: possible http reporting fallout? I see it claim to report check job results about half and hour ago in the zuul scheduler logs15:48
clarkbtosky: corvus: but doesn't seem to make it to Gerrit15:48
clarkbcorvus: tosky yup, looks like a new bug15:49
toskyclarkb: but why those jobs specifically? Other jobs reported the results in the meantime15:49
clarkbcorvus: tosky http://paste.openstack.org/show/727545/ change ids are not unique identifiers15:49
clarkbtosky: because they use non unique change ids (they are cherry picks with change id preserved)15:49
mordredclarkb: oh fun!15:49
toskyclarkb: but isn't that what happens normally with cherry-picks?15:50
mordredtosky: yes - but we changed how zuul reports results yesterday15:50
toskyoh15:50
tosky'-_-15:50
clarkbtosky: yes, we just changed zuul to report job results over http and not ssh. This changed behavior and broke this case15:50
mordredtosky: in order to support zuul being able to leave in-line comments15:50
toskyokidoki15:51
clarkbevrardjp: 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 that15:51
corvusand also, as we can all see, the rest api is far superior to the ssh api ;)15:51
evrardjpclarkb: that is interesting.15:52
clarkbcorvus: fwiw it also doesn't look like it retries 3 times15:52
clarkbcorvus: so might be multiple bugs there15:52
toskybut shouldn't at least return something in case of error? Or is it one of those errors that should not return anything?15:52
clarkbtosky: it is erroring in its attempt to report to the change15:52
openstackgerritMerged openstack-infra/openstackid master: Fix on OIDC session status check added more logging info fixed toString missing call method  https://review.openstack.org/58950215:52
evrardjpclarkb: do you have a spec I can read about that somewhere? Or a patch maybe?15:53
openstackgerritAndy Smith proposed openstack-infra/project-config master: devstack-plugin-kafka tests moved to project repo  https://review.openstack.org/58952515:53
clarkbevrardjp: I don't think there is anything that formal at least not yet. Id60b5e292b5750d701ec77b51ff73f12becf3cbd is the change id for the change that dded the pipeline15:54
corvusclarkb: i see 3 attempts15:54
clarkbcorvus: oh maybe my grep -A number wasn't big enough15:54
corvusclarkb: it has a 10 second then 20 second delay15:55
corvushttp://paste.openstack.org/show/727547/15:55
clarkbcorvus: that would explain why I didn't immediately see them15:55
evrardjpclarkb: for my understanding "promote" could be used for publishing artifacts from gate pipeline.15:55
AJaegerdhellmann: I followed your ask to not merge oslo changes and gave a -2 on https://review.openstack.org/#/c/58952515:56
evrardjpthat is indeed what I'd look for.15:56
corvusevrardjp, 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 there15:56
dhellmannAJaeger : yeah, we have a problem with that repo15:57
dhellmannI thought ansmith was going to propose making the job  non-voting temporarily15:57
corvusevrardjp, 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
fungithinking 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
fungijust thinking through the use cases15:57
evrardjpcorvus: yes but at least it would be probably more reliable and easy to find15:57
*** ansmith has joined #openstack-infra15:58
evrardjpthan hoping from a file from a gate log x15:58
evrardjpI see15:58
AJaegerdhellmann, happy to remove my -2 if you +1 it ;)15:58
dhellmannAJaeger : 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 now15:58
dhellmannI 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
corvusfungi: i was imagining the single-artifact use case, like "latest docs" or "latest build"15:58
dhellmannremoving it might be easier. Either way I have to resolve the merge conflict15:58
fungicorvus: oh, got it. so why tie it to change-merged rather than ref-updated?15:59
dhellmannAJaeger : if we work out what to do here, I'll leave a comment on the patch for the record15:59
kgiustidhellmann: for the amqp1 plugin we only define the tox-bashate, but the plugin does have a .zuul.yaml that includes the full set of tests15:59
AJaegerdhellmann: yes, please work it out - I'll take a +1 from you as a sign to remove my -2 ;)15:59
dhellmannkgiusti : ok, we should be good to go for amqp1, then.15:59
dhellmannAJaeger : ok15:59
corvustosky, clarkb: i used the wrong change-id format, i need to use project~branch~changeid16:00
dhellmannAJaeger : well, I was looking for your advice about the best approach :-)16:00
clarkbcorvus: ya project + branch + changeid is unique key in gerrit16:00
*** Bhujay has quit IRC16:00
kgiustidhellmann: I told ansmith to use amqp1 as the correct model /my fault/16:00
dhellmannah16:00
dhellmannI guess since we're just turning the tests off and adding them back there's no reason not to do it this way16:01
*** onovy has quit IRC16:01
AJaegerdhellmann: sorry... Shouldn't multitask ;(16:01
AJaegerdhellmann: 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 IRC16:02
dhellmannAJaeger : ok, I left a +1 to go ahead with this16:02
dhellmannnow we just need ansmith to remove his W-116:02
AJaegerdhellmann: ok16:02
AJaeger;)16:02
ansmithremoved :)16:02
dhellmannansmith : cool, thanks16:02
* AJaeger reviews to know what vote to give ;)16:03
ansmithdhellmann: will add .zuul.yaml to the plugin next16:03
dhellmannperhaps we can get mnaser to approve https://review.openstack.org/#/c/589525/1 then?16:03
dhellmannansmith : my existing patch should do that16:03
dhellmannansmith : https://review.openstack.org/#/c/588685/16:03
AJaegeransmith, dhellmann, why is bashate not moved in-repo? I would just remove the complete section and import everything in-repo.16:04
openstackgerritJames E. Blair proposed openstack-infra/zuul master: Use full gerrit changeid in HTTP report  https://review.openstack.org/58953016:04
AJaegeransmith, dhellmann, left a +1 with that question in there...16:05
dhellmannAJaeger : I guess we could do it that way. My other project-config patch will finish the cleanup16:05
kgiustiAJaeger: I'm +1 with that - I'd prefer the test config be local to the plugin16:05
AJaegerdhellmann: so, leave bashate in, import it in-repo, and then your large project-config change removes it?16:05
*** r-mibu has quit IRC16:06
dhellmannAJaeger : yeah, unless you'd rather ansmith remove it in his patch now16:06
corvustosky: 589530 should fix it, and i'll restart zuul with that as soon as it lands.  sorry, and thanks for the report.16:06
dhellmannwhat happens if a repo has no test jobs attached at all?16:06
AJaegerdhellmann: whatever works for you ;)16:06
dhellmannI prefer moving all of this along quickly :-)16:06
AJaegerdhellmann: you cannot merge anything - you need noop-jobs template in this case to merge first16:06
* kgiusti votes for removing it completely....16:06
toskycorvus: thank you for the quick fix; please let me know when the restart is done and I will recheck the jobs16:06
dhellmannAJaeger : ok, so it sounds like we want to leave bash8 in place16:07
dhellmann*bashate16:07
dhellmannthat way we have *some* voting job16:07
* kgiusti changes vote to whatever makes dhellmann happy :)16:07
AJaegerdhellmann: 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-infra16:08
dhellmannAJaeger : ok, thanks16:08
*** ginopc has quit IRC16:08
*** mriedem_afk has quit IRC16:11
mnaserdhellmann: approved16:14
dhellmannmnaser : thank you!16:14
openstackgerritJames E. Blair proposed openstack-infra/zuul master: Use full gerrit changeid in HTTP report  https://review.openstack.org/58953016:15
*** e0ne has joined #openstack-infra16:18
*** armax has joined #openstack-infra16:19
*** amoralej is now known as amoralej|off16:21
*** jpena is now known as jpena|off16:21
openstackgerritJames E. Blair proposed openstack-infra/zuul master: Use full gerrit changeid in HTTP report  https://review.openstack.org/58953016:24
*** zoli is now known as zoli|gone16:24
*** zoli|gone is now known as zoli16:24
*** jpich has quit IRC16:27
*** electrofelix has quit IRC16:29
*** lbragstad[m] has quit IRC16:32
*** trown is now known as trown|lunch16:33
openstackgerritJames E. Blair proposed openstack-infra/zuul-website master: Add BMW case study  https://review.openstack.org/58954116:37
corvusclarkb, fungi: i'm going to direct-enqueue 58953016:38
clarkbcorvus: ok16:38
*** janki has quit IRC16:47
*** weshay has joined #openstack-infra16:47
weshayzuul ok?16:47
clarkbweshay: 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
clarkboh zuul web is not running16:48
clarkbcorvus: ^16:48
weshay:)16:49
clarkbhrm the process is running so tis is different than yesterday16:50
clarkbthe direct enqueue seems to be running still as well, I think the scheduler must just be busy rebuilding layouts?16:50
mordredclarkb: openstacksdk 0.17.2 has been released, so we can restart nodepool launchers to pick it up16:50
corvusclarkb: it cleared16:51
clarkbcorvus: I wonder if direct enqueues are what create that behavior? I saw similar with a direct enqueue a while back (last week?)16:51
openstackgerritsebastian marcet proposed openstack-infra/openstackid master: Fix for OIDC session checking  https://review.openstack.org/58954916:52
clarkbmordred: 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 restart16:52
mordredclarkb: 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 later16:53
mordredor, if someone beats me to it, it's all good16:53
clarkbmordred: limestone has gone idle again. It is served by nl02. I'll start by looking there16:54
corvusclarkb: https://review.openstack.org/582794 will help a lot with this16:55
clarkbya needs manual pip3 install. I will do that now on nl02 then restart16:55
*** Emine has joined #openstack-infra16:55
*** tosky has quit IRC16:57
clarkbmordred: `sudo pip3 install -U /opt/nodepool` on nl02 fwiw. Restarting launcher now16:58
mordredmordred@nl02:~$ pbr freeze | grep openstacksdk16:59
mordredopenstacksdk==0.17.2  # git sha 3ad9dab16:59
mordredlooks like that worked16:59
*** derekh has quit IRC17:00
clarkblogan-: ^ fyi we think we have addressed the limestone idling problem17:00
clarkbcorvus: good to know17:00
logan-clarkb: awesome! thanks17:07
*** panda|backin2h is now known as panda|rover17:08
*** xarses_ has quit IRC17:10
AJaegercorvus: 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
clarkbmordred: ok I havne't seen nl02 have problems yet. I will do the others now17:15
clarkbmordred: we might consider bumping the min for openstacksdk in nodepool too give that bug17:16
Shrewsclarkb: i wonder if that warrants a new release, too17:18
AJaegerconfig-core, could you review https://review.openstack.org/589336  so that we can merge the dependend change, please?17:18
clarkbShrews: probably a good idea also wb17:18
clarkbI know rdo hit this bug too17:18
*** e0ne has quit IRC17:20
*** e0ne has joined #openstack-infra17:21
clarkb#status log Updated openstacksdk on nodepool-launchers to 0.17.2 to fix provider thread crashes that result in idle providers17:21
openstackstatusclarkb: finished logging17:21
clarkbmordred: ^ its done17:21
*** e0ne has quit IRC17:22
*** gfidente has quit IRC17:23
clarkbAJaeger: 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
clarkbcorvus: ^17:23
AJaegerclarkb: job will fail AFAIK - and this is only a test job, so fine IMHO to merge them after each other.17:25
clarkbAJaeger: got it17:25
AJaegerbut I want them merged quickly nevertheless ;)17:26
clarkbyup both are approved though may have to reapprove the second one if they don't share a gate queue17:26
AJaegerthey don't share one - but will +1 myself if needed...17:27
*** harlowja has joined #openstack-infra17:31
*** trown|lunch is now known as trown17:42
*** harlowja has quit IRC17:43
*** markvoelker_ has quit IRC17:45
fungiscas: 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
scasfungi: i thought i did mention it yesterday morning17:48
scasit was indeed everything i needed17:48
fungioh, awesome i probably missed it. thanks!17:49
fricklercorvus: 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 , too17:53
openstackgerritMerged openstack-infra/zuul master: Use full gerrit changeid in HTTP report  https://review.openstack.org/58953018:01
corvusfrickler: yeah, hopefully it will be installed during the current puppet run, then i'll immediately restart and send a notice18:03
*** e0ne has joined #openstack-infra18:04
corvusAJaeger: i don't know, but we should look into it18:04
AJaegercorvus: 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/rocky18:05
AJaegercorvus: there's no stable/rocky on horizon yet18:05
corvusAJaeger: 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
AJaegercorvus: the job was removed from horizon, removed from msater on senlin-dashboard - and senlin-dashboard branched *before* that removal ;818:07
AJaegercorvus: so, nothing we can do ;818:07
AJaeger;(18:07
corvusAJaeger: solum-dashboard you mean?18:07
AJaegercorvus: yes, indeed18:08
corvusin that case, we should remove it from solum-dashboard stable/rocky18:08
AJaegeryeah, will send a patch now...18:08
AJaegercorvus: 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/58937218:10
corvusAJaeger: 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
corvusso this case may not happen again18:10
corvuslet's look into it if it does18:10
AJaegerok18:11
corvusAJaeger: i haven't had time to look into the traslation issue yet.  i may not be able to today.  :(18:11
openstackgerritMerged openstack-infra/zuul-website master: Add BMW case study  https://review.openstack.org/58954118:11
AJaegercorvus: ok - it was broken for long enough without anybody noticing, I hope that we can wait another day18:12
AJaegerjust wanted to point out 58937218:12
*** Swami has joined #openstack-infra18:12
corvusAJaeger: thanks18:12
mordredAJaeger: we just got this: https://review.openstack.org/#/c/584484/ over in python-openstackclient18:13
mordredAJaeger: I am certain that patch is the wrong way to do that thing .. but I wasn't sure what the correct way to communicate was18:13
AJaegermordred: let me -1 with comment...18:13
mordredAJaeger: thank you!18:14
AJaegermordred: done18:14
clarkbI've got to step out before the meeting. I'll be back before then18:15
openstackgerritDavid Shrewsbury proposed openstack-infra/nodepool master: Bump minimum openstacksdk version to 0.17.2  https://review.openstack.org/58956318:15
mordredAJaeger: 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
mordredAJaeger: did we maybe do something wrong in python-openstackclient?18:17
AJaegercorvus: pushed https://review.openstack.org/#/c/589564/ for solum-dashboard, hope they take it in quickly...18:18
AJaegermordred: since 18th of July the translation sync is broken, see my discussions with eumel8 and corvus since Sunday18:18
mordredAJaeger: ah ! great (I mean not great, but at least there is an ongoing issue already)18:18
AJaegermordred: BUT we only sync if there are new translated strings. so, if nobody translated the project, then nothing gets proposed18:19
corvusAJaeger: 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
AJaegermordred: indeed, http://zuul.openstack.org/builds.html?job_name=propose-translation-update&project=openstack%2Fpython-openstackclient last run 18th of July18:19
corvuszuul is installed.  i will restart the scheduler now.18:20
AJaegersorry, need to step out18:20
corvusactually, 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
openstackgerritMerged openstack-infra/project-config master: Add Tripleo Ansible repo  https://review.openstack.org/58341518:34
openstackgerritMerged openstack-infra/project-config master: devstack-plugin-kafka tests moved to project repo  https://review.openstack.org/58952518:34
openstackgerritMerged openstack-infra/zuul-jobs master: Un-wip upload-logs-swift  https://review.openstack.org/58867718:34
*** apetrich has quit IRC18:35
openstackgerritwes hayutin proposed openstack-infra/project-config master: Add tripleo-collect-logs project  https://review.openstack.org/58956818:38
*** fabo has joined #openstack-infra18:40
*** openstackgerrit has quit IRC18:49
*** diablo_rojo has joined #openstack-infra18:52
*** boden has joined #openstack-infra18:54
*** sdatko has joined #openstack-infra18:57
*** jamesmcarthur has quit IRC18:58
clarkbcorvus: is the http timeout and retry causing some of the slowdown in processing results in zuul?18:58
clarkbAlso meeting time over in #openstack-meeting in 2 minutes18:58
*** jamesmcarthur has joined #openstack-infra18:59
*** bobh has quit IRC19:01
*** diablo_rojo has quit IRC19:02
corvusclarkb: 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-infra19:18
*** openstackgerrit has joined #openstack-infra19:21
openstackgerritMohammed Naser proposed openstack/ansible-role-cloud-launcher master: Add extra_specs to flavor  https://review.openstack.org/58833219:21
openstackgerritMohammed Naser proposed openstack/ansible-role-cloud-launcher master: Add os_project_access  https://review.openstack.org/58833519:21
*** bobh has quit IRC19:23
*** mriedem has joined #openstack-infra19:24
openstackgerritMerged openstack-infra/openstackid master: Fix for OIDC session checking  https://review.openstack.org/58954919:25
openstackgerritMohammed Naser proposed openstack/ansible-role-cloud-launcher master: Add os_project_access  https://review.openstack.org/58833519:25
*** jamesmcarthur has quit IRC19:25
openstackgerritMohammed Naser proposed openstack/ansible-role-cloud-launcher master: Add is_public to flavor  https://review.openstack.org/58957419:28
*** e0ne has quit IRC19:32
corvusclarkb, tobiash: and i think something is wrong with the branch caching -- we're querying gerrit for the branch names for every tenant reconfiguration19:33
*** apetrich has joined #openstack-infra19:33
tobiashcorvus: should there be branch caching for gerrit?19:34
fungithat could certainly slow things down19:34
corvustobiash: a tenant reconfiguration is supposed to use the branch list from the previous config except for projects which have updated19:35
tobiashcorvus: it's calling getprojectbranches19:35
tobiashIn github that was never cached, but I don't know if it was for gerrit19:36
*** jamesmcarthur has joined #openstack-infra19:36
openstackgerritMerged openstack-infra/project-config master: Follow wip-upload-swift rename  https://review.openstack.org/58933619:46
openstackgerritJeremy Stanley proposed openstack-infra/system-config master: Remove references to long-gone odsreg.o.o service  https://review.openstack.org/58958019:54
fungi#status log deleted a/aaaa rrs for the long-gone odsreg.openstack.org19:54
openstackstatusfungi: finished logging19:54
fungicmurphy: ^ should cease to be a problem once that merges19:55
*** jamesmcarthur has quit IRC19:57
*** jamesmcarthur has joined #openstack-infra19:57
*** jamesmcarthur_ has joined #openstack-infra19:58
openstackgerritsebastian marcet proposed openstack-infra/openstackid master: Fixes on session cookies  https://review.openstack.org/58958219:58
*** bobh has joined #openstack-infra19:59
openstackgerritSlawek Kaplonski proposed openstack-infra/project-config master: Add neutron-fullstack-python35 job to Neutron dashboard  https://review.openstack.org/58958420:01
*** jamesmcarthur has quit IRC20:01
*** sdatko has quit IRC20:01
fricklercorvus: 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 implications20:04
* fungi predicts an ara port to shockwave flash20:11
openstackgerritMerged openstack-infra/zuul master: Cache decryption results  https://review.openstack.org/58279420:13
openstackgerritMerged openstack-infra/zuul master: Switch storyboard url to be by name  https://review.openstack.org/58859720:13
openstackgerritColleen Murphy proposed openstack-infra/system-config master: Remove references to stackalytics.o.o  https://review.openstack.org/58520820:13
*** zul has quit IRC20:13
mordredfungi: wow, porting to a very old version of flash ftw20:16
*** ansmith has quit IRC20:17
openstackgerritDoug Hellmann proposed openstack-dev/pbr master: add python 3.6 unit test job  https://review.openstack.org/58958920:20
odyssey4mefolks, 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
fungiodyssey4me: do they share a change-id in common with other changes?20:21
fungiif so, known issue, fix in place shortly20:21
odyssey4mehere's a patch with +w around 30 mins back, and nothing in zuul: https://review.openstack.org/#/c/588508/20:21
odyssey4meanother two: https://review.openstack.org/589064 & https://review.openstack.org/58906520:21
odyssey4mehmm, oh yeah - those are all backports20:21
fungiregression from a very recent switch to start having zuul use the gerrit rest api instead of the ssh api20:22
fungijust came to our attention a few hours back20:22
odyssey4meah ok, I've kinda noticed this happening for a day or two20:23
mordredodyssey4me: https://review.openstack.org/#/c/589305/1/.zuul.yaml is what we get for our trouble though20:23
*** kgiusti has left #openstack-infra20:23
odyssey4memordred: it took me a minute to figure it out - but now I see it... zuul's commenting in line?20:24
fungiyeah, next stop is to find someone who wants to make flake8 leave inline comments in your source code20:24
openstackgerritAdam Coldrick proposed openstack-infra/storyboard-webclient master: Don't display 404 errors when using the "Jump to..." box  https://review.openstack.org/58961920:24
odyssey4methat.would.be.awesome.20:25
mordredodyssey4me: IKR?20:25
odyssey4mezomg that is great though :) it'd be nice to have ansible-lint do much the same20:25
mordredodyssey4me: you can!20:25
fungii want a variant which insults my lack of style20:25
* odyssey4me starts dribbling20:25
odyssey4memordred: how?!20:26
* SotK wishes he had time to experiment with things like that20:26
mordredodyssey4me: if you write a thing to parse the output, you can leave info for zuul and it'll leave the comments inline20:26
mordredodyssey4me: one sec- lemme find you the docs20:26
odyssey4mewow, this is revolutionary - no more log trawling to get the pertinent info... for some things at least20:27
Shrewsfungi: i don't think we've loaded the hawaiian shirt module into zuul yet20:27
fungihah20:27
Shrewsfungi: oh, coding style  ;)20:27
fungispeaking 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 while20:27
openstackgerritIan Wienand proposed openstack-infra/zuul-jobs master: [WIP] kerberos & afs client roles  https://review.openstack.org/58933420:27
openstackgerritIan Wienand proposed openstack-infra/openstack-zuul-jobs master: [WIP] openafs client install  https://review.openstack.org/58933520:28
*** diablo_rojo has joined #openstack-infra20:28
mordredodyssey4me: https://zuul-ci.org/docs/zuul/user/jobs.html#leaving-file-comments20:29
odyssey4mewell now, let me cast gimme-more-time++ and use the extra time to do something about that20:32
odyssey4melooks great though!20:32
mordred\o/20:32
openstackgerritMerged openstack-infra/puppet-log_processor master: Add severity info to logstash and filter out DEBUG lines  https://review.openstack.org/58838320:33
* mordred is excited for flake8 jobs to grow the feature - but hasn't yet grown enough time to actually do something about it20:33
corvusrestarting zuul nowv20:34
corvusstatus 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
corvushow's that look?20:40
mordred++20:40
corvus(re-enqueue is proceeding now)20:40
clarkblgtm20: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
openstackstatuscorvus: sending notice20:41
corvusokay, back to branch caching20: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
ianwcorvus / 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 keys20:44
openstackstatuscorvus: finished sending notice20:44
*** dpawlik has joined #openstack-infra20:45
*** apetrich has quit IRC20:45
*** nicolasbock has quit IRC20:45
openstackgerritMerged openstack-infra/openstackid master: Fixes on session cookies  https://review.openstack.org/58958220:51
openstackgerritMonty Taylor proposed openstack-infra/zuul-jobs master: WIP Extract pep8 messages for inline comments  https://review.openstack.org/58963420:55
mordredShrews, 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
mordredof course, I cannot remember the last time I got something right on the first try20:56
*** slaweq has quit IRC20:58
corvusapparently not this time :)20:59
corvusor so says software factory20:59
Shrewsmordred: you probably should handle the parameter to extract_file_comments()  :)21:00
corvusmordred: before we go crazy on this, we might want to think about https://review.openstack.org/57903321:01
mordredcorvus: oh - yes - I was looking for that earlier and coulnd't find it21:01
corvusmordred: 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
corvusor, i guess, we could decide it's definitely the right approach, and we must find a way to implement it.  :)21:02
corvusthe downside to that is it'd be extra work for the mergers.  but hey.21:03
corvusthey need to feel appreciated21:03
mordredI 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 above21:03
corvusmordred: register?21:04
mordredbecause I can get a handle to the stdout of the command in that playbook, but once we're out of that playbook, it's weird21:04
mordredoh, yeah, I guess I could just include_role21:04
corvusanyway, there's like 2 serious zuul bugs right now, so it'll be a couple days before i get to that21:06
mordredtotally21:06
mordredit 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 confusde21:06
corvusyeah.  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
mordredif you need to disable it, you're more likely to know what you're doing21:07
corvusmordred: 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
corvusanyway, 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
mordredwe could - but I kinda like that there is an api, and that it's well documented and easy to understand21:08
mordred++21:08
mordredif we can't do it - then I totally agree we should just say "use this role" :)21:08
mordredcorvus: we could also do a hybrid internal/external and write a new module similar to zuul_return21:09
corvusmordred: 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
openstackgerritMonty Taylor proposed openstack-infra/zuul-jobs master: WIP Extract pep8 messages for inline comments  https://review.openstack.org/58963421:10
mordredShrews: fixed your comment :)21:11
corvusmordred: sf caught an error too21:12
mordred\o/21:13
mordredbwhahahaha21:14
openstackgerritMonty Taylor proposed openstack-infra/zuul-jobs master: WIP Extract pep8 messages for inline comments  https://review.openstack.org/58963421:15
*** slaweq has joined #openstack-infra21:23
*** boden has quit IRC21:23
*** trown is now known as trown|outtypewww21:27
*** edmondsw has quit IRC21:29
clarkbcorvus: I've restarted the log worker proceses on logstash-worker01 to pick up the new client side filtering21:39
clarkbit should noop for now, but should shake out any major issues21:39
corvusclarkb: thanks!21:39
clarkbre 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 well21:40
*** jamesmcarthur_ has quit IRC21:41
mordred++21:42
* mordred is offering booze21:42
ianwmordred: so this playbook isn't actually run *on* bridge.o.o?21:48
mordredianw: well, it is21:49
mordredianw: but I may have also already run it on bridge21:49
mordredor, more correctly - I ran it on my laptop21:50
mordredpointed at bridge21:50
ianwright ... but then ansible running on bridge upgrades it's own ansible?21:50
mordredyup21:50
mordredmoving forward21:50
ianwand that doesn't eat itself?21:50
mordrednope - should be fine :)21:50
mordredianw: thanks!21:55
*** bdodd has quit IRC21:56
*** slagle has quit IRC21:56
mordredclarkb, ianw: I clicked +a on the stack - will continue iterating on bridge when it lands21:57
mordredif you feel like another two: https://review.openstack.org/#/c/588626/ and https://review.openstack.org/#/c/58862721:57
clarkbmordred: why would we use ansible-role-puppet if they aren't running puppet?22:00
clarkbwouldn't it be easier to just not run that role against those nodes?22:01
clarkboh this is to modify the hierdata on the "master"22:01
clarkbeven then that is just a fact so it should be fine?22:01
ianwit 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 data22:03
clarkbmordred: oh hrm I think I understand now. I thought the point was to keep the data in /etc/puppet/hieradata22:03
clarkbthen point ansible at that?22:03
clarkbbut its the inverse, put hieradata in /etc/ansible/hosts and point puppet at that22:03
*** dpawlik has quit IRC22:09
ianwmy 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-infra22:09
*** priteau has quit IRC22:09
ianwto the remote host22:10
clarkbianw: 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 setup22:10
openstackgerritAdam Coldrick proposed openstack-infra/storyboard-webclient master: Don't display 404 errors when using the "Jump to..." box  https://review.openstack.org/58961922:11
openstackgerritAdam Coldrick proposed openstack-infra/storyboard-webclient master: Add a route to redirect from a Task ID to the relevant Story  https://review.openstack.org/58965222:11
openstackgerritAdam Coldrick proposed openstack-infra/storyboard-webclient master: Support Task ID in the "Jump to..." box  https://review.openstack.org/58965322:11
*** rosmaita has quit IRC22:11
*** owalsh has quit IRC22:12
*** owalsh- has joined #openstack-infra22:12
*** bobh has quit IRC22:12
*** owalsh- is now known as owalsh22:13
*** beagles has quit IRC22:13
*** owalsh_ has quit IRC22:14
mordredclarkb, ianw: right - on bridge there is no puppet, so there is no hiera nor is there hieradata22:16
ianwi think i agree maybe with clarkb that puppet_get_hiera_file_list might be looking in the wrong place22:16
clarkbmordred: well there is hieradata, but not hiera22:16
mordredit's not hieradata22:17
mordredit's just data22:17
mordredbut yes - I agree with you that there is a bug22:17
mordredgood catch :)22:17
*** beagles has joined #openstack-infra22:20
*** rcernin has joined #openstack-infra22:20
*** strigazi has joined #openstack-infra22:20
*** strigazi has quit IRC22:21
*** strigazi has joined #openstack-infra22:21
openstackgerritMonty Taylor proposed openstack-infra/ansible-role-puppet master: Allow explicit override for mgmt_hieradata  https://review.openstack.org/58862622:23
mordredclarkb, ianw: ^^ how does that look?22:23
*** strigazi has quit IRC22:27
*** strigazi has joined #openstack-infra22:28
*** strigazi has quit IRC22:28
*** strigazi has joined #openstack-infra22:29
*** strigazi has quit IRC22:32
openstackgerritKendall Nelson proposed openstack-infra/storyboard master: Make email public field  https://review.openstack.org/58966322:33
*** strigazi has joined #openstack-infra22:33
openstackgerritIan Wienand proposed openstack-infra/zuul-jobs master: [WIP] kerberos & afs client roles  https://review.openstack.org/58933422:38
*** hongbin has quit IRC22:39
openstackgerritMerged openstack-infra/system-config master: Install and configure ansible on bridge  https://review.openstack.org/58798522:40
*** diablo_rojo has quit IRC22:41
*** gothicmindfood has quit IRC22:42
openstackgerritMerged openstack-infra/system-config master: Add an Ansible role to configure exim  https://review.openstack.org/58808922:43
openstackgerritMerged openstack-infra/system-config master: Update install_modules to not need puppet  https://review.openstack.org/58839422:43
*** bradm has joined #openstack-infra22:44
clarkbmordred: +2'd but not approving as I need to run some errands and that will affect existing puppetmaster22:45
*** gothicmindfood has joined #openstack-infra22:45
openstackgerritMerged openstack-infra/system-config master: Rename update_puppet to update-system-config  https://review.openstack.org/58839622:47
openstackgerritMerged openstack-infra/system-config master: Fix some little ansible issues  https://review.openstack.org/58839722:52
openstackgerritMerged openstack-infra/system-config master: Replace with_first_found with lookup first_found  https://review.openstack.org/58854622:52
*** rpioso is now known as rpioso|afk22:54
*** edmondsw has joined #openstack-infra22:54
*** ansmith has joined #openstack-infra22:55
*** dbkinder_ has joined #openstack-infra22:58
*** edmondsw has quit IRC22: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
fungidbkinder_: the wiki isn't really a file server23:10
fungialso 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 spam23:11
openstackgerritJames E. Blair proposed openstack-infra/project-config master: Add openstack-manuals to periodic pipeline  https://review.openstack.org/58966923:12
*** tpsilva has quit IRC23:12
fungidbkinder_: 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
corvusAJaeger, 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
corvusfungi, clarkb, ianw: ^ can we merge that change soon, so we see it run in tonights periodic builds?23:13
fungilooking now23: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-infra23:15
mordredclarkb: ++ to not merging it23:22
openstackgerritIan Wienand proposed openstack-infra/zuul-jobs master: [WIP] kerberos & afs client roles  https://review.openstack.org/58933423:22
mordredclarkb: 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 expected23:22
mordredtomorrow23:22
mordrednot today. today it has become caipirinha-thirty23: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
openstackgerritIan Wienand proposed openstack-infra/zuul-jobs master: [WIP] kerberos & afs client roles  https://review.openstack.org/58933423:36
fungidbkinder_: 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
fungidbkinder_: 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 things23:38
dbkinder_ah. someone at the openstack foundation then.  I'll need to track them down. sigh.23:39
dbkinder_thanks.23:39
fungidbkinder_: we see some of them in here from time to time (jamesmacarthur, jpmaxman, smarcet) but none seem to be in channel at the moment23:39
openstackgerritMerged openstack-infra/project-config master: Add openstack-manuals to periodic pipeline  https://review.openstack.org/58966923:42
fungidbkinder_: 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 IRC23:46
fungidbkinder_: 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-website23:48
openstackgerritIan Wienand proposed openstack-infra/openstack-zuul-jobs master: [WIP] openafs client install  https://review.openstack.org/58933523:49
fungizuul jobs react to merges in that repo and continuously deploy builds of the content23:49
mordreddbkinder_, 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 that23: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
mordredfungi: 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
fungithat would be pretty awesome to have written out23:54
dbkinder_you mean document stuff, how novel :)23:54
fungidbkinder_: i've asked how/who to put you in touch with regarding the website. will let you know23:54
dbkinder_ok.  david.b.kinder@intel.com23:55
mordreddbkinder_: documenting things about documenting things. my favorite :)23:55
fungidbkinder_: thanks!23:55
dbkinder_thank you!23:56
dbkinder_ttfn23:56
*** dbkinder_ has quit IRC23:56

Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!