Skip to content

Use right job type when making decisions for variant task.#1078

Merged
inferno-chromium merged 5 commits into
masterfrom
a10
Oct 4, 2019
Merged

Use right job type when making decisions for variant task.#1078
inferno-chromium merged 5 commits into
masterfrom
a10

Conversation

@inferno-chromium

Copy link
Copy Markdown
Collaborator

Variant task does not use testcase.job_type, but overridden
env variable JOB_NAME. Fix tests to account for it.

Variant task does not use testcase.job_type, but overridden
env variable JOB_NAME. Fix tests to account for it.
@googlebot googlebot added the cla: yes CLA signed. label Oct 4, 2019
@inferno-chromium

Copy link
Copy Markdown
Collaborator Author

/gcbrun

@inferno-chromium

Copy link
Copy Markdown
Collaborator Author

another way is to pass job_type as a setup_testcase argument.

@inferno-chromium

Copy link
Copy Markdown
Collaborator Author

fixing

@inferno-chromium

Copy link
Copy Markdown
Collaborator Author

/gcbrun

@inferno-chromium

Copy link
Copy Markdown
Collaborator Author

/gcbrun

@inferno-chromium

Copy link
Copy Markdown
Collaborator Author

/gcbrun

@inferno-chromium

Copy link
Copy Markdown
Collaborator Author

/gcbrun

@inferno-chromium inferno-chromium merged commit 6a9d97b into master Oct 4, 2019
@oliverchang oliverchang deleted the a10 branch October 9, 2019 03:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla: yes CLA signed.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants