Closed Bug 1002039 Opened 10 years ago Closed 10 years ago

Cannot find virtualenv.py to setup marionette environment

Categories

(Testing :: Marionette Client and Harness, defect)

ARM
Gonk (Firefox OS)
defect
Not set
critical

Tracking

(blocking-b2g:1.3+, firefox30 wontfix, firefox31 wontfix, firefox32 fixed, b2g-v1.3 fixed, b2g-v1.3T fixed, b2g-v1.4 fixed, b2g-v2.0 fixed)

RESOLVED FIXED
mozilla32
blocking-b2g 1.3+
Tracking Status
firefox30 --- wontfix
firefox31 --- wontfix
firefox32 --- fixed
b2g-v1.3 --- fixed
b2g-v1.3T --- fixed
b2g-v1.4 --- fixed
b2g-v2.0 --- fixed

People

(Reporter: m1, Assigned: m1)

Details

(Keywords: pi-marionette-client, pi-marionette-runner)

Attachments

(1 file)

No, the URL is correct but curl should follow the redirect. So adding -L before the PIPE will fix it.
Ah, yeah that's nicer :)
out of curiosity, are you using venv_automation.sh?
and what is it being used for?
Yeah, we are.  Aren't you? :)

We use it to setup the virtual env that all our Marionette-based tests run out of.
(In reply to Michael Vines [:m1] [:evilmachines] from comment #5)
> Yeah, we are.  Aren't you? :)

Not for a while, we use mozharness to set up our environments in tbpl, or jenkins scripts that invoke virtualenv directly in other automation.
> 
> We use it to setup the virtual env that all our Marionette-based tests run
> out of.

Interesting! I just wanted to know its current usecases, thanks.
Attached patch -L x 3Splinter Review
I think I got 'em all.
Attachment #8414200 - Flags: review?(mdas)
Comment on attachment 8414200 [details] [diff] [review]
-L x 3

Review of attachment 8414200 [details] [diff] [review]:
-----------------------------------------------------------------

lgtm, thanks!
Attachment #8414200 - Flags: review?(mdas) → review+
Does this need to be pushed to any other branches?
Comment on attachment 8414200 [details] [diff] [review]
-L x 3

NOTE: This flag is now for security issues only. Please see https://wiki.mozilla.org/Release_Management/B2G_Landing to better understand the B2G approval process and landings.

[Approval Request Comment]
Bug caused by (feature/regressing bug #): github.com adding a HTTP redirect to their raw URLs
User impact if declined:  NPOTB.  Our B2G tests will continue to be blocked (without a locally patched Gecko).
Testing completed:  Yes
Risk to taking this patch (and alternatives if risky):  Our B2G tests will become unblocked and we may find new issues or regressions, which could have a negative impact on the v1.3+ bug burndown rate.
String or UUID changes made by this patch: n/a
Attachment #8414200 - Flags: approval-mozilla-b2g28?
https://hg.mozilla.org/mozilla-central/rev/66eeca26b328
Assignee: nobody → mvines
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla32
Comment on attachment 8414200 [details] [diff] [review]
-L x 3

This is test-only, so it doesn't need extra approval.
Attachment #8414200 - Flags: approval-mozilla-b2g28?
Product: Testing → Remote Protocol

Moving bugs for Marionette client due to component changes.

Component: Marionette → Marionette Client and Harness
Product: Remote Protocol → Testing
You need to log in before you can comment on or make changes to this bug.