-
Notifications
You must be signed in to change notification settings - Fork 1
REL-1158490- EW document review comments #37
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
Conversation
… install 2. Created a file for upgrade 3. Removed Next step in monitoring-agents.md under post install 4. Removed Port related redundant information from files under troubleshooting folder and created a seperate file for the same. 5. Updated resolution image for alerts resolution link
Added table for Port references
…ting files to seperate file
…troubleshooting.md Updated Upgrade document Included apm server and kibana server service status check in monitoring-agent-and-otel-collector.md
…ing-agents.md file
Removed Table of contents from secret store configuration
…de port , secret store and certificate troubleshooting steps.. Updated curl commands for verifying Elastic kibana and APM agent troubleshooting guide
…ibes this agent model
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Approving the PR as all the review comments have been updated as per discussion.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Reviewed Inbound and outbound checkbox incorporated, and local host mentioned in the purpose.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I reviewed this PR with Karuna, identified several small issues, but agreed to merge these changes and identify the remaining open items that can be addressed afterwards.
Once this PR is merged, do not close this story; rather, create another branch using the same REL-* prefix and issue another PR.
Test Evidence:
README.md:
README.md.docx
monitoring-agents.md:
monitoring-agents.md.docx
environment_watch_troubleshooting.md
environment_watch_troubleshooting.md.docx
4.docs/environment_watch_upgrade.md
upgrade.md.docx
pre-requisite-troubleshooting.md
pre-requisites troubleshooting.docx
monitoring-agent-and-otel-collector.md
monitoring-agent-and-otel-collector.docx
docs/datagrid_audit_setup.md
DataGrid Audit.docx