Page MenuHomePhabricator

Clean up gui from the wdqs deploy repo and puppet
Closed, ResolvedPublic

Description

Since T266702: Move WDQS UI to microsites is done, there's no need for them anymore. While I'd wait for a couple of days to make sure we don't need to revert.

Related Objects

StatusSubtypeAssignedTask
Resolved Ladsgroup
OpenNone
OpenNone
DuplicateNone
OpenEBernhardson
OpenNone
OpenGehel
Opendcausse
OpenNone
ResolvedGehel
ResolvedRobH
ResolvedPapaul
OpenMPhamWMF
OpenRKemper
OpenNone
OpenNone
OpenNone
OpenNone
OpenEBernhardson
OpenNone
OpenEBernhardson
ResolvedZbyszko
OpenNone
OpenNone

Event Timeline

Restricted Application added a subscriber: Aklapper. · View Herald Transcript

Change 655739 had a related patch set uploaded (by Ladsgroup; owner: Ladsgroup):
[wikidata/query/deploy@master] Remove gui from deploy repo

https://gerrit.wikimedia.org/r/655739

Change 655739 merged by DCausse:
[wikidata/query/deploy@master] Remove gui from deploy repo

https://gerrit.wikimedia.org/r/655739

Change 655955 had a related patch set uploaded (by Ladsgroup; owner: Ladsgroup):
[operations/puppet@production] Remove gui stuff from wdqs

https://gerrit.wikimedia.org/r/655955

Change 655955 merged by Ryan Kemper:
[operations/puppet@production] query_service: Remove gui files from wdqs

https://gerrit.wikimedia.org/r/655955

@RKemper I'm so sorry it was buggy, the cleanest way forward here is to deploy wcqs using microsites and we can easily drop everything from puppet then. Does that make sense?

@RKemper I'm so sorry it was buggy, the cleanest way forward here is to deploy wcqs using microsites and we can easily drop everything from puppet then. Does that make sense?

Makes sense! I'm working on getting the wcqs microsite working and then we can revert my earlier revert and nuke the remaining gui file at the same time. I'll tag you on the patch(es) when ready.

Change 657765 had a related patch set uploaded (by Ryan Kemper; owner: Ryan Kemper):
[operations/puppet@production] wcqs: create wcqs microsite && move gui

https://gerrit.wikimedia.org/r/657765

Change 657773 had a related patch set uploaded (by Gehel; owner: Gehel):
[operations/puppet@production] query_service: remove monitoring check for UI.

https://gerrit.wikimedia.org/r/657773

Change 657773 merged by Gehel:
[operations/puppet@production] query_service: remove monitoring check for UI.

https://gerrit.wikimedia.org/r/657773

Seems like everything have been done for this ticket.

Seems like everything have been done for this ticket.

puppet says there is wdqs::gui still and it's needed for wcqs, once wcqs is moved to microsites, this can be cleaned properly.

Change 657765 abandoned by Ryan Kemper:

[operations/puppet@production] wcqs: create wcqs microsite && move gui

Reason:

https://gerrit.wikimedia.org/r/657765

This is, with the refactors, is done.