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

Updated Example configuration for Netdata #9463

Closed
wants to merge 3 commits into from
Closed

Updated Example configuration for Netdata #9463

wants to merge 3 commits into from

Conversation

noodlemctwoodle
Copy link
Contributor

Updated example in documentation as per this Github post

Description:

Pull request in home-assistant (if applicable): home-assistant/home-assistant#

Checklist:

  • Branch: next is for changes and new documentation that will go public with the next home-assistant release. Fixes, changes and adjustments for the current release should be created against current.
  • The documentation follows the standards.

Updated example in documentation as per this [Github](home-assistant/core#16656 (comment)) post
@balloob balloob added the to-do label May 15, 2019
@klaasnicolaas
Copy link
Member

Please rebase this to the current branch

@klaasnicolaas klaasnicolaas added needs-rebase The PR has been branched of the wrong base branch or targets an incorrect target branch and removed to-do labels May 16, 2019
source/_components/netdata.markdown Outdated Show resolved Hide resolved
source/_components/netdata.markdown Outdated Show resolved Hide resolved
source/_components/netdata.markdown Outdated Show resolved Hide resolved
Copy link
Member

@fabaff fabaff left a comment

Choose a reason for hiding this comment

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

source/_components/netdata.markdown Show resolved Hide resolved
 - Added another way to read the API data in the web browser.
 - Added example and description
 - Reverted `configuration` example back to required
```

Once the `data_group` and the `element` have been identified from the JSON it can be configured as per the example below.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Added another example of how to collect data from the API for those unable to collect it using curl.

Copy link
Contributor Author

@noodlemctwoodle noodlemctwoodle left a comment

Choose a reason for hiding this comment

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

Added addition example.

@noodlemctwoodle
Copy link
Contributor Author

updated component

@klaasnicolaas
Copy link
Member

Can you rebase this PR to the current branch? Your changes are not related to something that will come in a next release.

@jjanderson
Copy link
Contributor

HI, is this stalled? I am trying to configure but really getting lost... an example would be great... especially if it were say based on Ubuntu 18.04? :-)

@noodlemctwoodle
Copy link
Contributor Author

I can assist with instructions for 18.04. I'll get the rebase done this evening.

@noodlemctwoodle
Copy link
Contributor Author

created new pull request here as i broke my repo :|

Updated Netdata documentation #9606

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs-rebase The PR has been branched of the wrong base branch or targets an incorrect target branch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants