Closed Bug 901095 Opened 12 years ago Closed 11 years ago

Desktop UnitTests should not rely on PyPi as a repo

Categories

(Release Engineering :: General, defect)

x86
macOS
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: mtabara, Unassigned)

Details

(Whiteboard: [mozharness] [unittest] [puppet])

Attachments

(1 file)

While testing a new feature in Desktop_unittest mozharness script I came to an issue. I needed to add a python package script as a dependency, which currently lays in our puppet repos. I've changed the config to ignore the base index and take into consideration our repos. Execution broke before it reached my package. That leads me to the question: sould we rely on pypi for our packages or should we start, gradually, copy them into our own repos so that we can manage them better?
Whiteboard: [mozharness] [unittest] [puppet]
Aki, I know that we want to get rid of using "repos" at some point and this patch may break local tests. Do you have any ideas how we should proceed here?
Flags: needinfo?(aki)
Found in triage. Not exactly sure if this is correct component, given the new scope of new bugzilla components, so feel free to move it on again if I guessed wrong!
Component: Release Engineering → Release Engineering: Machine Management
QA Contact: armenzg
That's bug 885780. I think unittests are already non-standalone friendly, sadly. We shouldn't have anything relying on pypi in production, so this should only be a staging/standalone patch. Possibly our path to victory is a standalone config that overrides production config items, or vice versa.
Component: Release Engineering: Machine Management → Release Engineering: Automation (General)
Flags: needinfo?(aki)
QA Contact: armenzg → catlee
Product: mozilla.org → Release Engineering
I think this should be fixed with recent VirtualenvMixin changes... is this still an issue?
Flags: needinfo?(tabara.mihai)
AFAIK, it should no longer be an issue. However, I'm not very well updated with the changes so I'll double check on Rail on this.
Flags: needinfo?(tabara.mihai) → needinfo?(rail)
(In reply to Aki Sasaki [:aki] from comment #4) > I think this should be fixed with recent VirtualenvMixin changes... is this > still an issue? Whatever aki says. :)
Status: NEW → RESOLVED
Closed: 11 years ago
Flags: needinfo?(rail)
Resolution: --- → FIXED
Component: General Automation → General
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: