Sumo: Set Python path to search Kitsune git repo before installed packages

RESOLVED FIXED

Status

Infrastructure & Operations
WebOps: Other
RESOLVED FIXED
2 years ago
2 years ago

People

(Reporter: mythmon, Assigned: cyliang)

Tracking

(Blocks: 1 bug)

Details

(Whiteboard: [kanban:https://webops.kanbanize.com/ctrl_board/2/2596] )

(Reporter)

Description

2 years ago
Cyliang did this on -dev. We'll need it on stage and prod for some code changes that are coming down soon (In particular, for bug 1164035).

Updated

2 years ago
Whiteboard: [kanban:https://webops.kanbanize.com/ctrl_board/2/2596]
(Assignee)

Updated

2 years ago
Assignee: server-ops-webops → cliang
(Assignee)

Comment 1

2 years ago
@mythmon: I've applied the changes to -stage.  Can you please verify that things are still working as expected?  Once you've given the thumbs up, I'll make the changes to -prod.
Flags: needinfo?(mcooper)
(Reporter)

Comment 2

2 years ago
-stage looks good. bd to continue to -prod.
Flags: needinfo?(mcooper)
(Reporter)

Updated

2 years ago
Blocks: 1164035
(Reporter)

Updated

2 years ago
No longer blocks: 1164035
(Reporter)

Updated

2 years ago
Blocks: 1164035
(Assignee)

Comment 3

2 years ago
Change pushed to -prod (r 115240).  Holding this bug open for a few days to make sure there no unintended consequences. =)

Comment 4

2 years ago
Seems that there are. :(

Since this change, we've been getting sporadic alerts from Zeus about queuing time. New Relic shows something amiss too:

https://rpm.newrelic.com/accounts/263620/applications/2779374?tw%5Bend%5D=1456527916&tw%5Bstart%5D=1456517116#tab_hosts=breakout


What's the underlying change here? The old python-path directives would have been harmless I think (wrong python version, so they shouldn't get loaded... right?). What's in the new one that wasn't already getting pulled in some other way?
(Reporter)

Comment 5

2 years ago
This change was to accommodate the Python package "authority" being moved into the git repo. Authority is unmaintained and not compatible with Django 1.8 (bug 1164035). We moved it into kitsune's git repo to make it compatible.
(Reporter)

Comment 6

2 years ago
Alternatively, I think we can revert this change, and uninstall authority from the virtualenv, as a part of bug 1251744.
(Assignee)

Comment 7

2 years ago
The queue time alerts largely appear to be firing in reaction to client behavior (i.e. excessive # of requests and/or excessive traffic being generated in response to requests).  Closing this bug.
Status: NEW → RESOLVED
Last Resolved: 2 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.