Closed Bug 826896 Opened 11 years ago Closed 11 years ago

Update gaiatest mozharness script to set DM_TRANS and TEST_DEVICE env variables

Categories

(Release Engineering :: Applications: MozharnessCore, defect, P1)

All
Gonk (Firefox OS)
defect

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: jgriffin, Assigned: armenzg)

References

Details

Attachments

(1 file)

There's a pull request pending for gaia-ui-tests that allows them to populate media files using devicemanager prior to test, see: https://github.com/mozilla/gaia-ui-tests/pull/236/files

In order for this to work, we'll need to set DM_TRANS='sut' and TEST_DEVICE=(ip of panda) before invoking gaiatest.
Blocks: 802317
Assignee: nobody → armenzg
Priority: -- → P1
Aki, is there a way to show on the logs that these env variables have been set?

The current logs do not show any env changes.

Perhaps this should be configuration value rather than modified in the code?
Do we dump env variables on mozharness? I can't seem to find that.
Attachment #698768 - Flags: review?(aki)
Comment on attachment 698768 [details] [diff] [review]
adjust env for gaia ui tests

+        env = self.query_env()
+        env["DM_TRANS"] = "sut"
+        env["TEST_DEVICE"] = self.mozpool_device

This is equivalent to

env = self.query_env(partial_env={'DM_TRANS': 'sut', 'TEST_DEVICE': self.mozpool_device})
Attachment #698768 - Flags: review?(aki) → review+
(In reply to Armen Zambrano G. [:armenzg] from comment #1)
> Aki, is there a way to show on the logs that these env variables have been
> set?
> 
> The current logs do not show any env changes.
> 
> Perhaps this should be configuration value rather than modified in the code?
> Do we dump env variables on mozharness? I can't seem to find that.

There isn't, currently.

I've been meaning to echo env settings to the log, but have been putting it off since android signing can use env vars for passphrases.  I thought about making the env logging configurable, but haven't gotten to it.

If this would be helpful here, I can revisit (or point you to where/how to make the changes).
Comment on attachment 698768 [details] [diff] [review]
adjust env for gaia ui tests

http://hg.mozilla.org/build/mozharness/rev/740e75fc5af5
Attachment #698768 - Flags: checked-in+
Feel free to throw mozharness work at me and see where I can help.
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Product: mozilla.org → Release Engineering
Component: General Automation → Mozharness
You need to log in before you can comment on or make changes to this bug.