Closed
Bug 1292235
Opened 10 years ago
Closed 10 years ago
socorrolib: remove requirement for pyhs2 and sasl
Categories
(Socorro :: General, task)
Socorro
General
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: willkg, Assigned: willkg)
Details
socorrolib doesn't do anything with pyhs2 or sasl, so it shouldn't require either of them.
We require both of these libraries in socorro for the socorro/cron/jobs/fetch_adi_from_hive.py cron job. Both of these libraries are already in the socorro requirements file, so removing them from the socorrolib requirements file shouldn't affect anything.
This issue covers removing pyhs2 and sasl requirements from socorrolib, doing a release and then updating socorro to use the new version.
| Assignee | ||
Comment 1•10 years ago
|
||
Grabbing this one because I just did it.
In a PR: https://github.com/mozilla/socorrolib/pull/21
Assignee: nobody → willkg
Status: NEW → ASSIGNED
| Assignee | ||
Comment 2•10 years ago
|
||
PR #21 (https://github.com/mozilla/socorrolib/pull/21) has landed in the socorrolib repository.
PR for cutting version 0.2.3 is here: https://github.com/mozilla/socorrolib/pull/22
| Assignee | ||
Comment 3•10 years ago
|
||
I landed that PR and pushed out version 0.2.3.
Next step is to update the socorro repository to use that version.
| Assignee | ||
Comment 4•10 years ago
|
||
Updating socorro in PR https://github.com/mozilla/socorro/pull/3425
Comment 5•10 years ago
|
||
Commit pushed to master at https://github.com/mozilla/socorro
https://github.com/mozilla/socorro/commit/fcbac60f8d32a96e0173f0e62500b9e1ba1a3cef
Bug 1292235 update socorrolib to 0.2.3 (#3425)
I reduced the requirements.txt file for socorrolib to just the
requirements that it needs. The requirements.txt file is sourced as
install dependencies by setup.py. Because of this, it'll be tricky to
update dependencies that socorrolib doesn't actually use in this
repository.
This updates to the latest version of socorrolib.
| Assignee | ||
Comment 6•10 years ago
|
||
This should be all set now. Closing it out.
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•