Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Unified Diff: README.md

Issue 5712550654115840: #765 - Update "Monitoring" section in README.md (Closed)
Patch Set: #765 - Update "Monitoring" section in README.md (incl. markup fix) Created July 10, 2014, 10:36 a.m.
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: README.md
===================================================================
--- a/README.md
+++ b/README.md
@@ -165,12 +165,13 @@
Monitoring
----------
-Monitoring is fully functional in the development environment:
-[https://10.8.0.98/](https://10.8.0.98/)
+Monitoring is fully functional in any environment, including development.
+Here, after bootstrapping the `server4` box, one can access the Nagios GUI
+from the host machine via <https://nagiosadmin:nagiosadmin@10.8.0.99/> --
Wladimir Palant 2014/07/10 11:00:31 Looks good but why -- rather than period?
mathias 2014/07/10 13:38:10 Just because some UI guy once told me that it's no
Wladimir Palant 2014/07/10 13:51:05 That shouldn't be a concern for a linkified URL. Y
-User name and password are both _nagiosadmin_.
+The monitoring service of our production environment, however, is accessible
+via <https://monitoring.adblockplus.org/>.
+Add yourself to _files/nagios-htpasswd_ in the _private_ module used on the
+server, or have someone add you if you don't have access.
-The monitoring service of our production environment runs on
-_monitoring.adblockplus.org_. Add yourself to _files/nagios-htpasswd_
-in the _private_ module used on the server, or have someone add you if
-you don't have access.
+
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld