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

NMS-13810: Phase 4 flows documentation: Nephron for aggregation #47

Merged
merged 6 commits into from
Feb 7, 2022

Conversation

Bonrob2
Copy link
Contributor

@Bonrob2 Bonrob2 commented Jan 31, 2022

First draft. I am not sure where this content from the Core guide (https://docs.opennms.com/meridian/2021/operation/flows/aggregation.html) should go. Also would appreciate suggestions for Nephron troubleshooting topics/content.

NMS-13810: Phase 4 flows documentation: Nephron for aggregation - added nephron documentation to nephron docs repository - about, configuration, persistence, etc.
NMS-13810: Phase 4 flows documentation: Nephron for aggregation: updates to get TOC working.
NMS-13810: Phase 4 flows documentation: Nephron for aggregation - restructuring documentation for Nephron and updating content.
NMS-13810: Phase 4 flows documentation: Nephron for aggregation - updated and restructured Nephron documentation. Still need to write "how it works" section.
NMS-13810: Phase 4 flows documentation: Nephron for aggregation - added documentation for Nephron - compiled from confluence "Nephron Field Guide", Tech talk presentations, Nephron repo/readme, etc.
indigo423
indigo423 previously approved these changes Feb 2, 2022
Copy link
Member

@indigo423 indigo423 left a comment

Choose a reason for hiding this comment

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

Besides the compatibility SNAPSHOT version, looks good to me.

docs/antora.yml Outdated Show resolved Hide resolved
NMS-13810: Phase 4 flows documentation: Nephron for aggregation - updated Horizon version as per reviewer feedback.
@Bonrob2
Copy link
Contributor Author

Bonrob2 commented Feb 4, 2022

@indigo423 , updated but requires an approval.

Copy link
Member

@indigo423 indigo423 left a comment

Choose a reason for hiding this comment

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

LGTM!

@Bonrob2 Bonrob2 merged commit 7c3b13d into master Feb 7, 2022
@indigo423 indigo423 deleted the jira/NMS-13810 branch February 7, 2022 16:35
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.

2 participants