Page MenuHomePhabricator

Deploy phabricator to aphlict2001.codfw.wmnet
Closed, ResolvedPublic

Description

tl;dr: Can we add aphlict1001.eqiad.wmnet and aphlict2001.codfw.wmnet to the phabricator scap targets and deploy?

We added a new aphlict host in codfw in T322369, however puppet is currently throwing an error:

Feb 16 22:42:51 aphlict2001 puppet-agent[313667]: (/Stage[main]/Profile::Phabricator::Aphlict/Scap::Target[phabricator/deployment]/Package[phabricator/deployment]/ensure) change from 'absent' to 'present' failed: Execution of '/usr/bin/scap deploy-local --repo phabricator/deployment -D log_json:False' returned 70: #007

This line in the puppet config depends on phabricator/deployment being deployed with scap on the machine, and we can see in aphlict1001.eqiad that directory is there. I think this error, however, is because the host isn't included in the scap targets.

This hasn't been a problem on the existing host because it looks like the host was manually deployed to somehow (the dates on /srv/deployment/phabricator/deployment are from June 2020).

So I think to get us to the correct state, we need to add both hosts to be deploy targets of phabricator in scap, and initiate a deployment.

Details

Related Changes in GitLab:
TitleReferenceAuthorSource BranchDest Branch
scap: add aphlict2001 to phabricator-targetsrepos/phabricator/deployment!3brennenwork/aphlict2001-targetwmf/stable
Customize query in GitLab

Event Timeline

19:41 < mutante> scap has defaults to use gerrit as origin: modules/scap/lib/puppet/provider/scap_source/default.rb: "https://gerrit.wikimedia.org/r/#{repo_name}.git"
19:42 < mutante> so you cant use gitlab repos as origin for something deployed with scap
19:42 < mutante> this currently is a problem for phabricator repos which got moved from phabricator itself to gitlab

brennen renamed this task from Deploy phabricator to aphlict2002.codfw.wmnet to Deploy phabricator to aphlict2001.codfw.wmnet.Feb 23 2023, 9:22 PM

Change 891841 had a related patch set uploaded (by Brennen Bearnes; author: EoghanGaffney):

[operations/puppet@production] Split out a new class for phabricator::config

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

eoghan claimed this task.

scap deployments on the aphlict host now works, some updates are here: T322369#8690007