Skip to content

report list of detail files so devicelab can save them#64009

Merged
flar merged 2 commits intoflutter:masterfrom
flar:devicelab-report-trace-files
Aug 20, 2020
Merged

report list of detail files so devicelab can save them#64009
flar merged 2 commits intoflutter:masterfrom
flar:devicelab-report-trace-files

Conversation

@flar
Copy link
Contributor

@flar flar commented Aug 17, 2020

This change simply adds a new (optional) entry to the TaskResults reported from a devicelab test which is a list of filenames to save for future diagnosis. There is no code here to save them (though they will appear in the output for a developer running a devicelab task locally), the code to save them is in the Cocoon agents.

@flar flar requested review from keyonghan and liyuqian August 17, 2020 23:53
@flutter-dashboard flutter-dashboard bot added the c: contributor-productivity Team-specific productivity, code health, technical debt. label Aug 17, 2020
@flutter-dashboard
Copy link

It looks like this pull request may not have tests. Please make sure to add tests before merging. If you need an exemption to this rule, contact Hixie on the #hackers channel in Chat.

Reviewers: Read the Tree Hygiene page and make sure this patch meets those guidelines before LGTMing.

@flar flar force-pushed the devicelab-report-trace-files branch from 674cede to fac9e72 Compare August 18, 2020 00:03
Copy link
Contributor

@keyonghan keyonghan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Contributor

@liyuqian liyuqian left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@flar flar force-pushed the devicelab-report-trace-files branch from 8faf59c to 89a99f8 Compare August 19, 2020 20:50
@flar flar merged commit 9e86e9f into flutter:master Aug 20, 2020
smadey pushed a commit to smadey/flutter that referenced this pull request Aug 27, 2020
mingwandroid pushed a commit to mingwandroid/flutter that referenced this pull request Sep 6, 2020
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Aug 20, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

c: contributor-productivity Team-specific productivity, code health, technical debt.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants