Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[test/plugin] Print systeminfo on failure #3874

Merged
merged 19 commits into from
Nov 19, 2019
Merged

Conversation

arugal
Copy link
Member

@arugal arugal commented Nov 17, 2019

Please answer these questions before submitting pull request

  • Why submit this pull request?

  • Bug fix

  • New feature provided

  • Improve performance

  • Related issues


Bug fix

  • Bug description.

  • How to fix?


New feature or improvement

  • Describe the details and related test reports.

@arugal arugal added agent Language agent related. plugin Plugin for agent or collector. Be used to extend the capabilities of default implementor. test Test requirements about performance, feature or before release. labels Nov 17, 2019
@arugal arugal added this to the 6.6.0 milestone Nov 17, 2019
dmsolr
dmsolr previously approved these changes Nov 18, 2019
@arugal
Copy link
Member Author

arugal commented Nov 19, 2019

/run agent-plugin-test

@arugal arugal closed this Nov 19, 2019
@arugal arugal reopened this Nov 19, 2019
Copy link
Member

@wu-sheng wu-sheng left a comment

Choose a reason for hiding this comment

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

LGTM. @dmsolr Is this good to merge?

@wu-sheng wu-sheng merged commit 6d3b0e2 into apache:master Nov 19, 2019
@arugal arugal deleted the systeminfo branch November 21, 2019 04:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
agent Language agent related. plugin Plugin for agent or collector. Be used to extend the capabilities of default implementor. test Test requirements about performance, feature or before release.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants