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

Remove legacy buitin Arms (#1983) #2097

Merged
merged 1 commit into from
Feb 25, 2016
Merged

Conversation

kz0817
Copy link
Member

@kz0817 kz0817 commented Feb 25, 2016

Now HAPI2 is a standard interface for collecting data from
monitoring systems. So this patch removes regacy builtin arms.

This change involves two major fixes internally.

  1. Some tests using legacy arms has been changed to use HAPI2 based arms.
  2. Some API such as registering a monitoring server requires an ArmPluginInfo
    instance mandatorily, which was used to be an option.

Now HAPI2 is a standard interface for collecting data from
monitoring systems. So this patch removes regacy builtin arms.

This change involves two major fixes internally.
1. Some tests using legacy arms has been changed to use HAPI2 based arms.
2. Some API such as registering a monitoring server requires an ArmPluginInfo
   instance mandatorily, which was used to be an option.
@cosmo0920
Copy link
Contributor

👍

ashie added a commit that referenced this pull request Feb 25, 2016
@ashie ashie merged commit 18de079 into master Feb 25, 2016
@ashie ashie deleted the remove-builtin-arms-#1983-a3 branch February 25, 2016 06:49
@kz0817
Copy link
Member Author

kz0817 commented Feb 25, 2016

Thank you.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants