Skip to content
This repository has been archived by the owner. It is now read-only.

[dev.icinga.com #3476] Add configuration declaration to make expiring acknowledgments the default #1177

Closed
icinga-migration opened this issue Dec 2, 2012 · 8 comments
Milestone

Comments

@icinga-migration
Copy link
Member

@icinga-migration icinga-migration commented Dec 2, 2012

This issue has been migrated from Redmine: https://dev.icinga.com/issues/3476

Created by crfriend on 2012-12-02 23:14:09 +00:00

Assignee: crfriend
Status: Resolved (closed on 2013-01-16 21:35:04 +00:00)
Target Version: 1.9
Last Update: 2014-12-08 09:27:53 +00:00 (in Redmine)


This proposition adds a configuration declaration to cgi.cfg, "use_expiring_acknowledgements", that will automatically tick the "Use Expire Time" box when acknowledging problems.

The intent is to modify observed admin behaviour in large organisations where chronic problems get acknowledged and never resolved leading, eventually, to system failure. It also completes the configuration notion begun with "default_expiring_acknowledgement_duration".

There is a documentation change involved as well as code. I have added the English section that applies to the change; however, as I neither speak nor write German, a translation will need to be applied to the /de/ documentation hierarchy.

I believe there are no backwards-compatibility problems.

I've pushed this to my "cgi" branch, but will attach the patch here just in case it doesn't work. Note: Do not apply my changes to "Makefile.in" at this time; that is a work in progress to make Solaris zones work as intended.

Attachments

Changesets

2012-12-11 22:44:14 +00:00 by crfriend 6139cf5

classic ui: Implement #3476, "set_expire_ack_by_default"

refs: #3476

This commit implements proposed feature #3476, with input from
Ricardo (and with thanks), in cgi.cfg which automatically ticks the
"Use Expire Time:" check-box in cmd.cgi.  The default behaviour is
to *NOT* tick the checkbox thereby preserving well-established bevaviour.

The intent of the change is in response to observed administrator behaviour
in large organisations where problems get acknowledged and then never fixed
leading to later system failure.  Expiring acknowledgments will assist in
reminding staff in disparate departments and locations that a problem has
not been fixed in a reasonable amount of time specified by the cgi.cfg
setting "default_expiring_acknowledgement_duration" or a different value
given in the "acknowledge problem" dialog.

2012-12-15 15:29:24 +00:00 by crfriend b0a51c6

Fix changelog omission from #3476

refs: #3476

This corrects an omission in Changelog for feature #3476. See
commit 6139cf55be7defce633739efcfd8cb579adba756 for details of
the change.

Relations:

@icinga-migration

This comment has been minimized.

Copy link
Member Author

@icinga-migration icinga-migration commented Dec 3, 2012

Updated by crfriend on 2012-12-03 22:10:00 +00:00

  • File added 0001-Add-the-new-declaration-use_expiring_acknowledgement.patch

Add changed cgi.cfg.in file to document declaration addition.

@icinga-migration

This comment has been minimized.

Copy link
Member Author

@icinga-migration icinga-migration commented Dec 10, 2012

Updated by ricardo on 2012-12-10 09:53:39 +00:00

  • Assigned to set to crfriend
@icinga-migration

This comment has been minimized.

Copy link
Member Author

@icinga-migration icinga-migration commented Dec 11, 2012

Updated by crfriend on 2012-12-11 14:37:41 +00:00

  • Target Version set to 1.9
  • Done % changed from 0 to 80

I've coded this, committed it as 119b6775fa93d772cffbe7b82df708c49f0da122 in cfriend/cgi, and pushed to remote. If it passes the "eyeball test", please move it to test for formal testing (the final 20%).

@icinga-migration

This comment has been minimized.

Copy link
Member Author

@icinga-migration icinga-migration commented Dec 11, 2012

Updated by crfriend on 2012-12-11 23:48:53 +00:00

  • Status changed from New to Assigned

Commit ID has changed to 6139cf5 due to a blunder on my part which required backing the original commit out. Please test. And note that I managed to ignore the Changelist add for the item.

@icinga-migration

This comment has been minimized.

Copy link
Member Author

@icinga-migration icinga-migration commented Dec 14, 2012

Updated by crfriend on 2012-12-14 12:17:45 +00:00

  • Status changed from Assigned to Feedback
  • Done % changed from 80 to 100

Coded, committed, and ready for testing. Change to docs on approval.

@icinga-migration

This comment has been minimized.

Copy link
Member Author

@icinga-migration icinga-migration commented Dec 19, 2012

Updated by ricardo on 2012-12-19 12:47:58 +00:00

  • Status changed from Feedback to 7

in current 'dev/cgis'

Thanks Carl

please test

@icinga-migration

This comment has been minimized.

Copy link
Member Author

@icinga-migration icinga-migration commented Jan 16, 2013

Updated by mfriedrich on 2013-01-16 21:35:04 +00:00

  • Status changed from 7 to Resolved
@icinga-migration

This comment has been minimized.

Copy link
Member Author

@icinga-migration icinga-migration commented Dec 8, 2014

Updated by mfriedrich on 2014-12-08 09:27:53 +00:00

  • Project changed from 19 to Core, Classic UI, IDOUtils
  • Category changed from 43 to Classic UI
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
1 participant
You can’t perform that action at this time.