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

Status Monitoring - Auto Updating #77

Closed
wants to merge 4 commits into from

Conversation

NOYB
Copy link
Contributor

@NOYB NOYB commented Mar 10, 2016

Auto update graph 4 times per the selected resolution period.
Auto update graph on settings change.

Auto update graph 4 times per the selected resolution period.
Auto update graph on settings change.
@jdillard
Copy link
Contributor

I like the idea of the auto update, but I think it should be a setting that is off by default and has the options of updating every 1, 5, or 10 minutes. There is no point in updating less than once a minute since the RRD data is polled once a minute.

@NOYB
Copy link
Contributor Author

NOYB commented Mar 10, 2016

Agree with it being user enabled option.

Rational for updating 4 times per selected resolution is that the graph updating and RRD updating are not synchronized. Thus the graph updating is kept from lagging the RRD updates by no more than 1/4th the selected resolution.

@jdillard
Copy link
Contributor

I wouldn't over engineer it, RRD isn't really meant for real time stats. I am tempted to change the name back to RRD Graphs from Monitoring as it might be a little misleading on functionality.

Make auto update a user selectable option.
@NOYB
Copy link
Contributor Author

NOYB commented Mar 11, 2016

If a decision is made not to use “Monitoring” how about one of these?
Status / System Charts
Status / System Graphs
Status / System Health
With or without “System”.
I like Graphs or System Graphs best. Never did like “RRD Graphs”. RRD is techie term only known by those who know what it is.

Edit: or could go medical: EKG ;)

@jdillard
Copy link
Contributor

I think just Graphs might be best. RRD does sounds technical. System is one of the categories of graphs, so System Graphs doesn't sound all encompassing, and Status > Status Graphs sounds repetitive.

Ever Knowing (but not understanding) Graphs :)

I'll run it by the other devs and see what they think.

@NOYB NOYB closed this Mar 12, 2016
@NOYB NOYB deleted the Status_Monitoring_-_Auto_Updating branch March 12, 2016 10:30
@NOYB
Copy link
Contributor Author

NOYB commented Mar 12, 2016

Replaced by PR #83

netgate-git-updates pushed a commit that referenced this pull request Dec 2, 2016
graphics/giflib: Update to 5.1.2

* Code hardening using reallocarray() from OpenBSD.

* Sanity check in giffilter catches files with malformed extension records
  Fixes SourceForge bug #63: malformed gif causes segfault in giffilter.

* Inexpensive sanity check in DGifSlurp() catches malformed files with
  no image descriptor. Fixes SourceForge bug #64: malformed gif causes
  crash in giftool.

* Fix SourceForge bug #66: GifDrawBoxedText8x8() modifying constant input
  parameter.

* Bail out of GIF read on invalid pixel width. Addresses Savannah bug
  #67: invalid shift in dgif_lib.c

* Fix SourceForge bug #69: #69 Malformed: Gif file with no extension
  block after a GRAPHICS_EXT_FUNC_CODE extension causes segfault (in
  giftext).

* Fix SourceForge bug #71: Buffer overwrite when giffixing a malformed gif.

* Fix SourceForge bug #73: Null pointer deference in gifclrmap (only
  reachable with malformed GIF).

* Fix SourceForge bug #74: Double free in gifsponge under 5.1,1,
  for any valid gif image.

* Fix SourceForge bug #75: GAGetArgs overflows due to uncounted use of va_arg.

* Sanity check in giffix catches some malformed files. Addresses
  SourceForge bug #77: dgif_lib.c: extension processing error

Security:	CVE-2015-7555
Approved by:	ports-secteam (with hat)
netgate-git-updates pushed a commit that referenced this pull request Jul 11, 2022
Changelog:
 * Fix build with gcc 11.3.0 (resolves #57) by @danvratil in #58
 * Fix ODR violation when LTO is enabled by @danvratil in #59
 * Rework Github CI - add compilers and platforms by @danvratil in #60
 * CI: consolidate the CI builds into a single matrix by @danvratil in #61
 * Export QCoro wrapper classes by @danvratil in #65
 * CI: disable shared libs on Windows by @danvratil in #67
 * Enable "modern Qt" compiler definitions by @danvratil in #66
 * Introduce Generator and AsyncGenerator coroutine return types by @danvratil in #69
 * Fix installing export header by @danvratil in #77
 * Deprecate task.h in favor of qcorotask.h (#70) by @danvratil in #76
 * QtWebSockets support by @danvratil in #75
 * Document -DQCORO_WITH_QTWEBSOCKETS by @dangelog in #81
 * Fix the build by #include'ing by @dangelog in #82
 * QCoroTask: fixup a template type parameter name by @dangelog in #83
 * Add support for clang-cl (fixes #84) by @danvratil in #86
 * Fix possible race in TestWsServer by @danvratil in #88
 * WebSocket bugfixes & documentation by @danvratil in #89
 * Explicitly enable exception support for clang-cl (fixes #90) by @danvratil in #91
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