1412209 - Determine why production flipped to using BMO staging

Status: RESOLVED FIXED

(Conduit :: Phabricator, defect, P1)

Description

[Piotr Zalewa [:zalun]]

Changed just the description of the D157.

On [Save changes] Phabricator displayed an error:
Bugzilla Bug ID: Bugzilla did not provide an expected response (28).

Comment 1

[Piotr Zalewa [:zalun]]

Same error on `arc diff`:

[2017-10-27 10:31:26] EXCEPTION: (ArcanistDifferentialCommitMessageParserException) Invalid or missing field "Bug #": Bugzilla Bug ID: Bugzilla did not provide an expected response (28). at [<arcanist>/src/differential/ArcanistDifferentialCommitMessage.php:54]
arcanist(), phutil()

I can't replicate it locally.

Comment 2

[Piotr Zalewa [:zalun]]

Moving the bug to the Bugzilla API component.

I believe the error is coming from `/rest/phabbugz/check_bug/` endpoint.

Comment 3

[Piotr Zalewa [:zalun]]

Phabricator was canfigured to use dev Bugzilla servers.

Comment 4

[Mark Côté [:mcote]]

Staging, technically.

Comment 5

[David Lawrence [:dkl]]

Fixed now.

Comment 6

[Mark Côté [:mcote]]

We're not sure what happened, though.

Comment 7

[David Lawrence [:dkl]]

Some extra info to help with debugging:

Phabricator code was changed while back to default to https://bugzilla.allizom.org for the bugzilla.url config value. So if Phab is installed and puppet or an ENV var used by entrypoint.sh is not used to change the config value, that is what will be used. So in the recent deployment of production Phab, somehow the setting was not updated properly and the default remained.

Comment 8

[Chris Kolosiwsky [:ckolos] (ckolos has left the building)]

Which ENV var should be passed to the container to configure this setting?

Comment 9

[David Lawrence [:dkl]]

(In reply to Chris Kolosiwsky [:ckolos] from comment #8)
> Which ENV var should be passed to the container to configure this setting?

Fom mozphabs entrypoint.sh:

       test -n "${BUGZILLA_URL}" \
         && ./bin/config set bugzilla.url "${BUGZILLA_URL}"

Comment 10

[Chris Kolosiwsky [:ckolos] (ckolos has left the building)]

Updated in our puppet code for deployment.