[Wikidata-bugs] [Maniphest] [Commented On] T220402: Introduce wikidata termbox SSR to kubernetes

gerritbot added a comment. Change 515069 **merged** by Alexandros Kosiaris: [operations/dns@master] Enable discovery for termbox https://gerrit.wikimedia.org/r/515069 TASK DETAIL https://phabricator.wikimedia.org/T220402 EMAIL PREFERENCES

[Wikidata-bugs] [Maniphest] [Commented On] T220402: Introduce wikidata termbox SSR to kubernetes

gerritbot added a comment. Change 517628 **merged** by Alexandros Kosiaris: [operations/puppet@production] termbox: Use HTTP for WMF service checks https://gerrit.wikimedia.org/r/517628 TASK DETAIL https://phabricator.wikimedia.org/T220402 EMAIL PREFERENCES

[Wikidata-bugs] [Maniphest] [Commented On] T220402: Introduce wikidata termbox SSR to kubernetes

gerritbot added a comment. Change 517628 had a related patch set uploaded (by Alexandros Kosiaris; owner: Alexandros Kosiaris): [operations/puppet@production] termbox: Use HTTP for WMF service checks https://gerrit.wikimedia.org/r/517628 TASK DETAIL

[Wikidata-bugs] [Maniphest] [Commented On] T220402: Introduce wikidata termbox SSR to kubernetes

gerritbot added a comment. Change 515074 **merged** by Alexandros Kosiaris: [operations/puppet@production] Introduce termbox LVS configuration https://gerrit.wikimedia.org/r/515074 TASK DETAIL https://phabricator.wikimedia.org/T220402 EMAIL PREFERENCES

[Wikidata-bugs] [Maniphest] [Commented On] T220402: Introduce wikidata termbox SSR to kubernetes

gerritbot added a comment. Change 515067 **merged** by Alexandros Kosiaris: [operations/dns@master] Assign termbox.svc.{eqiad,codfw}.wmnet LVS IPs https://gerrit.wikimedia.org/r/515067 TASK DETAIL https://phabricator.wikimedia.org/T220402 EMAIL PREFERENCES

[Wikidata-bugs] [Maniphest] [Commented On] T220402: Introduce wikidata termbox SSR to kubernetes

Stashbot added a comment. Mentioned in SAL (#wikimedia-operations) [2019-06-18T08:52:35Z] deploy termbox T220402 TASK DETAIL https://phabricator.wikimedia.org/T220402 EMAIL PREFERENCES

[Wikidata-bugs] [Maniphest] [Commented On] T220402: Introduce wikidata termbox SSR to kubernetes

gerritbot added a comment. Change 515177 **merged** by Alexandros Kosiaris: [operations/puppet@production] termbox: Add kubernetes stanzas https://gerrit.wikimedia.org/r/515177 TASK DETAIL https://phabricator.wikimedia.org/T220402 EMAIL PREFERENCES

[Wikidata-bugs] [Maniphest] [Commented On] T220402: Introduce wikidata termbox SSR to kubernetes

Tarrow added a comment. @akosiaris Great! Thanks for that. We look forward to seeing how it all goes forward post-offsite :) TASK DETAIL https://phabricator.wikimedia.org/T220402 EMAIL PREFERENCES https://phabricator.wikimedia.org/settings/panel/emailpreferences/ To: Tarrow Cc:

[Wikidata-bugs] [Maniphest] [Commented On] T220402: Introduce wikidata termbox SSR to kubernetes

akosiaris added a comment. Just as an FYI, everything looks ok on this end, but there's a train freeze this week, so we have to wait before deploying this. Patches are up and waiting to be merged on Monday the 17th TASK DETAIL https://phabricator.wikimedia.org/T220402 EMAIL PREFERENCES

[Wikidata-bugs] [Maniphest] [Commented On] T220402: Introduce wikidata termbox SSR to kubernetes

gerritbot added a comment. Change 515177 had a related patch set uploaded (by Alexandros Kosiaris; owner: Alexandros Kosiaris): [operations/puppet@production] termbox: Add kubernetes stanzas https://gerrit.wikimedia.org/r/515177 TASK DETAIL https://phabricator.wikimedia.org/T220402

[Wikidata-bugs] [Maniphest] [Commented On] T220402: Introduce wikidata termbox SSR to kubernetes

gerritbot added a comment. Change 515028 **merged** by Alexandros Kosiaris: [operations/deployment-charts@master] termbox: Use newer ENV variables https://gerrit.wikimedia.org/r/515028 TASK DETAIL https://phabricator.wikimedia.org/T220402 EMAIL PREFERENCES

[Wikidata-bugs] [Maniphest] [Commented On] T220402: Introduce wikidata termbox SSR to kubernetes

akosiaris added a comment. In T220402#5243209 , @Tarrow wrote: > This should now be fixed. Sadly this was due to a mismatch between the code in wikibase master and that deployed on Wikidata.org And yes, we are finally in greener

[Wikidata-bugs] [Maniphest] [Commented On] T220402: Introduce wikidata termbox SSR to kubernetes

Tarrow added a comment. In T220402#5242902 , @akosiaris wrote: > Indeed this was fixed. However another regression has crept up it's head This should now be fixed. Sadly this was due to a mismatch between the code in wikibase

[Wikidata-bugs] [Maniphest] [Commented On] T220402: Introduce wikidata termbox SSR to kubernetes

gerritbot added a comment. Change 515074 had a related patch set uploaded (by Alexandros Kosiaris; owner: Alexandros Kosiaris): [operations/puppet@production] Introduce termbox LVS configuration https://gerrit.wikimedia.org/r/515074 TASK DETAIL

[Wikidata-bugs] [Maniphest] [Commented On] T220402: Introduce wikidata termbox SSR to kubernetes

gerritbot added a comment. Change 515069 had a related patch set uploaded (by Alexandros Kosiaris; owner: Alexandros Kosiaris): [operations/dns@master] Enable discovery for termbox https://gerrit.wikimedia.org/r/515069 TASK DETAIL https://phabricator.wikimedia.org/T220402 EMAIL

[Wikidata-bugs] [Maniphest] [Commented On] T220402: Introduce wikidata termbox SSR to kubernetes

gerritbot added a comment. Change 515067 had a related patch set uploaded (by Alexandros Kosiaris; owner: Alexandros Kosiaris): [operations/dns@master] Assign termbox.svc.{eqiad,codfw}.wmnet LVS IPs https://gerrit.wikimedia.org/r/515067 TASK DETAIL

[Wikidata-bugs] [Maniphest] [Commented On] T220402: Introduce wikidata termbox SSR to kubernetes

Tarrow added a comment. @akosiaris Thanks! Now https://gerrit.wikimedia.org/r/c/wikibase/termbox/+/515040 is merged the Healthcheck query should be correct :) TASK DETAIL https://phabricator.wikimedia.org/T220402 EMAIL PREFERENCES

[Wikidata-bugs] [Maniphest] [Commented On] T220402: Introduce wikidata termbox SSR to kubernetes

akosiaris added a comment. @tarrow, @WMDE-leszek Hi, sorry for taking so long to answer to this, it's been really busy. In T220402#5214471 , @Tarrow wrote: > @mobrovac Thanks! I think we've now taken most of this onboard and

[Wikidata-bugs] [Maniphest] [Commented On] T220402: Introduce wikidata termbox SSR to kubernetes

gerritbot added a comment. Change 515028 had a related patch set uploaded (by Alexandros Kosiaris; owner: Alexandros Kosiaris): [operations/deployment-charts@master] termbox: Use newer ENV variables https://gerrit.wikimedia.org/r/515028 TASK DETAIL

[Wikidata-bugs] [Maniphest] [Commented On] T220402: Introduce wikidata termbox SSR to kubernetes

gerritbot added a comment. Change 510470 abandoned by Alexandros Kosiaris: openapi.json: Fix x-amples syntax Reason: Already added in a better way in https://gerrit.wikimedia.org/r/#/c/wikibase/termbox/ /511892/ https://gerrit.wikimedia.org/r/510470 TASK DETAIL

[Wikidata-bugs] [Maniphest] [Commented On] T220402: Introduce wikidata termbox SSR to kubernetes

WMDE-leszek added a comment. Hello @akosiaris, this is a friendly ping from your favourite WMDE customers :) Any news on the deployment front? Is there anything you would like us to do next, or we could be of help with in any way? TASK DETAIL https://phabricator.wikimedia.org/T220402

[Wikidata-bugs] [Maniphest] [Commented On] T220402: Introduce wikidata termbox SSR to kubernetes

Stashbot added a comment. Mentioned in SAL (#wikimedia-operations) [2019-05-27T15:40:07Z] initialize termbox namespace on eqiad/codfw/staging kubernetes clusters T220402 TASK DETAIL https://phabricator.wikimedia.org/T220402 EMAIL PREFERENCES

[Wikidata-bugs] [Maniphest] [Commented On] T220402: Introduce wikidata termbox SSR to kubernetes

Tarrow added a comment. @mobrovac Thanks! I think we've now taken most of this onboard and merged it. @akosiaris could you take a look at out new x-amples block and Host header? Hopefully this meets our needs :). You'll see there are now two new ENV variables: HEALTHCHECK_QUERY and

[Wikidata-bugs] [Maniphest] [Commented On] T220402: Introduce wikidata termbox SSR to kubernetes

mobrovac added a comment. In T220402#5199730 , @Tarrow wrote: > Should we be using http rather than https internally? Yes, indeed, sorry that was an omission on my part. Since setting up TLS connections takes time, though, you

[Wikidata-bugs] [Maniphest] [Commented On] T220402: Introduce wikidata termbox SSR to kubernetes

Tarrow added a comment. @mobrovac and @akosiaris Sorry to drop in late to the conversation but I've been on holiday and then at the hackathon. I wanted to check a couple of things: In T220402#5186782 , @mobrovac wrote: >

[Wikidata-bugs] [Maniphest] [Commented On] T220402: Introduce wikidata termbox SSR to kubernetes

mobrovac added a comment. @Pablo-WMDE all you have to do is have the request be of this form: URI: http://appservers.discovery.wmnet/{+your_path_here} headers: Host: www.wikidata.org The curl you are trying does not work because you are executing it outside the

[Wikidata-bugs] [Maniphest] [Commented On] T220402: Introduce wikidata termbox SSR to kubernetes

Pablo-WMDE added a comment. Hi @akosiaris, could you please provide a (curl) example of how the request is supposed to look like? Seems we will need this either way - as specification for what we are supposed to implement - because of lack of understanding on our part. What I

[Wikidata-bugs] [Maniphest] [Commented On] T220402: Introduce wikidata termbox SSR to kubernetes

gerritbot added a comment. Change 505788 **merged** by Alexandros Kosiaris: [operations/deployment-charts@master] Introduce the wikibase-termbox chart https://gerrit.wikimedia.org/r/505788 TASK DETAIL https://phabricator.wikimedia.org/T220402 EMAIL PREFERENCES

[Wikidata-bugs] [Maniphest] [Commented On] T220402: Introduce wikidata termbox SSR to kubernetes

gerritbot added a comment. Change 510470 had a related patch set uploaded (by Alexandros Kosiaris; owner: Alexandros Kosiaris): [wikibase/termbox@master] openapi.json: Fix x-amples syntax https://gerrit.wikimedia.org/r/510470 TASK DETAIL https://phabricator.wikimedia.org/T220402

[Wikidata-bugs] [Maniphest] [Commented On] T220402: Introduce wikidata termbox SSR to kubernetes

WMDE-leszek added a comment. In T220402#5183351 , @Krenair wrote: > May 14 08:24:34 <_joe_> and thanks for helping with that. In an ideal world, the teams developing new services would install them on beta > May 14 08:24:56

[Wikidata-bugs] [Maniphest] [Commented On] T220402: Introduce wikidata termbox SSR to kubernetes

Krenair added a comment. May 14 08:24:34 <_joe_> and thanks for helping with that. In an ideal world, the teams developing new services would install them on beta May 14 08:24:56 <_joe_> it might work out most of the times, but then we have people with no beta access, like wmde

[Wikidata-bugs] [Maniphest] [Commented On] T220402: Introduce wikidata termbox SSR to kubernetes

akosiaris added a comment. In T220402#5180031 , @Pablo-WMDE wrote: > Hi @akosiaris, > > thanks for taking the time to explain the way the `Host` header is intended to be used. > If I understand correctly the goal is to ensure

[Wikidata-bugs] [Maniphest] [Commented On] T220402: Introduce wikidata termbox SSR to kubernetes

Pablo-WMDE added a comment. Hi @akosiaris, thanks for taking the time to explain the way the `Host` header is intended to be used. If I understand correctly the goal is to ensure that requests originating from our service bear a header `Host: (www.)wikidata.org` and reach which ever

[Wikidata-bugs] [Maniphest] [Commented On] T220402: Introduce wikidata termbox SSR to kubernetes

gerritbot added a comment. Change 509391 **merged** by jenkins-bot: [wikibase/termbox@master] Add x-amples to termbox openapi.json https://gerrit.wikimedia.org/r/509391 TASK DETAIL https://phabricator.wikimedia.org/T220402 EMAIL PREFERENCES

[Wikidata-bugs] [Maniphest] [Commented On] T220402: Introduce wikidata termbox SSR to kubernetes

akosiaris added a comment. In T220402#5177862 , @Pablo-WMDE wrote: > Hi @akosiaris - thanks for getting back to us. > > > sending a Host: HTTP for the identification of the exact project. Would it be possible to add that

[Wikidata-bugs] [Maniphest] [Commented On] T220402: Introduce wikidata termbox SSR to kubernetes

akosiaris added a comment. > With respect to the end point checks it would be great to hear what we are trying to achieve with them. Our service depends on the availability of another service. If the examples are to act as smoke tests then their reliability depends on the upstream service;

[Wikidata-bugs] [Maniphest] [Commented On] T220402: Introduce wikidata termbox SSR to kubernetes

Pablo-WMDE added a comment. Hi @akosiaris - thanks for getting back to us. > sending a Host: HTTP for the identification of the exact project. Would it be possible to add that functionality? It certainly //is// possible and depending on operational needs we certainly can make this

[Wikidata-bugs] [Maniphest] [Commented On] T220402: Introduce wikidata termbox SSR to kubernetes

akosiaris added a comment. @Tarrow , @WMDE-leszek I 've noticed 3 things while working on the above - The service seems to be configurable to reach directly out to the wikidata endpoint. e.g. `WIKIBASE_REPO: '{env(WIKIBASE_REPO,https://www.wikidata.org/w)}'`. Since, in the general case,

[Wikidata-bugs] [Maniphest] [Commented On] T220402: Introduce wikidata termbox SSR to kubernetes

gerritbot added a comment. Change 509391 had a related patch set uploaded (by Alexandros Kosiaris; owner: Alexandros Kosiaris): [wikibase/termbox@master] Add x-amples to termbox openapi.json https://gerrit.wikimedia.org/r/509391 TASK DETAIL https://phabricator.wikimedia.org/T220402

[Wikidata-bugs] [Maniphest] [Commented On] T220402: Introduce wikidata termbox SSR to kubernetes

Tarrow added a comment. @akosiaris and @WMDE-leszek I think the needed changes to proceed with the helm chart are now done. Feel free to poke me if there is something we missed though: we're keen not to be a blocker for getting this deployed :) TASK DETAIL

[Wikidata-bugs] [Maniphest] [Commented On] T220402: Introduce wikidata termbox SSR to kubernetes

Pablo-WMDE added a comment. @mobrovac Thanks for the feedback. If it is possible at all we would really appreciate if you could link us to the check list you seem to be following while reviewing this, so we can try for a more holistic approach in bringing this to completion. This would

[Wikidata-bugs] [Maniphest] [Commented On] T220402: Introduce wikidata termbox SSR to kubernetes

mobrovac added a comment. In T220402#5146064 , @Pablo-WMDE wrote: > @mobrovac During T221755 & T221754 we tended to `/?spec`

[Wikidata-bugs] [Maniphest] [Commented On] T220402: Introduce wikidata termbox SSR to kubernetes

mobrovac added a comment. In T220402#5136129 , @Tarrow wrote: > We are indeed using service-runner; I don't think this provides /_info or /?spec though? Or are we missing something? Ah, indeed, sorry, it does not :) I meant

[Wikidata-bugs] [Maniphest] [Commented On] T220402: Introduce wikidata termbox SSR to kubernetes

Tarrow added a comment. We are indeed using service-runner; I don't think this provides /_info or /?spec though? Or are we missing something? TASK DETAIL https://phabricator.wikimedia.org/T220402 EMAIL PREFERENCES https://phabricator.wikimedia.org/settings/panel/emailpreferences/ To:

[Wikidata-bugs] [Maniphest] [Commented On] T220402: Introduce wikidata termbox SSR to kubernetes

mobrovac added a comment. In T220402#5133994 , @akosiaris wrote: > @Tarrow, @WMDE-leszek. I 've been working on the termbox helm chart and while the service seems to be up and running, I see no `/_info` endpoint nor a

[Wikidata-bugs] [Maniphest] [Commented On] T220402: Introduce wikidata termbox SSR to kubernetes

akosiaris added a comment. @Tarrow, @WMDE-leszek. I 've been working on the termbox helm chart and while the service seems to be up and running, I see no `/_info` endpoint nor a swagger/openapi[1] spec published under `/?spec`. Both are crucial for deploying, as the former is used as a

[Wikidata-bugs] [Maniphest] [Commented On] T220402: Introduce wikidata termbox SSR to kubernetes

gerritbot added a comment. Change 505788 had a related patch set uploaded (by Alexandros Kosiaris; owner: Alexandros Kosiaris): [operations/deployment-charts@master] First draft of a wikibase-termbox chart https://gerrit.wikimedia.org/r/505788 TASK DETAIL

[Wikidata-bugs] [Maniphest] [Commented On] T220402: Introduce wikidata termbox SSR to kubernetes

gerritbot added a comment. Change 505755 **merged** by jenkins-bot: [integration/config@master] Create/Publish OCI images for termbox https://gerrit.wikimedia.org/r/505755 TASK DETAIL https://phabricator.wikimedia.org/T220402 EMAIL PREFERENCES

[Wikidata-bugs] [Maniphest] [Commented On] T220402: Introduce wikidata termbox SSR to kubernetes

gerritbot added a comment. Change 505755 had a related patch set uploaded (by Alexandros Kosiaris; owner: Alexandros Kosiaris): [integration/config@master] Create/Publish OCI images for termbox https://gerrit.wikimedia.org/r/505755 TASK DETAIL