[dev.icinga.com #9666] checkcommand disk does not check free inode - check_disk #3161

Closed
icinga-migration opened this Issue Jul 17, 2015 · 9 comments

Comments

Projects
None yet
1 participant
@icinga-migration
Member

icinga-migration commented Jul 17, 2015

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

Created by jeromed on 2015-07-17 17:34:09 +00:00

Assignee: gbeutner
Status: Resolved (closed on 2015-07-20 05:50:44 +00:00)
Target Version: 2.3.8
Last Update: 2016-08-29 16:09:10 +00:00 (in Redmine)

Icinga Version: 2.3.7 - all
Backport?: Already backported
Include in Changelog: 1

Hi,

since no values are defined for disk_inode_wfree and disk_inode_cfree , by default the disk CheckCommand does not check free inode count.

please include the attached patch to enable free inode count by default on disk checks

regards,

Attachments

Changesets

2015-07-20 05:49:14 +00:00 by jeromed 95914ea

Enable inode check on disk by default

fixes #9666

Signed-off-by: Gunnar Beutner <gunnar@beutner.name>

2015-07-20 06:50:23 +00:00 by jeromed 2038937

Enable inode check on disk by default

fixes #9666

Signed-off-by: Gunnar Beutner <gunnar@beutner.name>

Relations:

@icinga-migration

This comment has been minimized.

Show comment
Hide comment
@icinga-migration

icinga-migration Jul 17, 2015

Member

Updated by jeromed on 2015-07-17 17:38:53 +00:00

  • File added 0001-Enable-inode-check-on-disk-by-default.patch

See attached patch

Member

icinga-migration commented Jul 17, 2015

Updated by jeromed on 2015-07-17 17:38:53 +00:00

  • File added 0001-Enable-inode-check-on-disk-by-default.patch

See attached patch

@icinga-migration

This comment has been minimized.

Show comment
Hide comment
@icinga-migration

icinga-migration Jul 20, 2015

Member

Updated by gbeutner on 2015-07-20 05:50:33 +00:00

  • Status changed from New to Assigned
  • Assigned to set to gbeutner
  • Target Version set to 2.4.0
Member

icinga-migration commented Jul 20, 2015

Updated by gbeutner on 2015-07-20 05:50:33 +00:00

  • Status changed from New to Assigned
  • Assigned to set to gbeutner
  • Target Version set to 2.4.0
@icinga-migration

This comment has been minimized.

Show comment
Hide comment
@icinga-migration

icinga-migration Jul 20, 2015

Member

Updated by jeromed on 2015-07-20 05:50:44 +00:00

  • Status changed from Assigned to Resolved
  • Done % changed from 0 to 100

Applied in changeset 95914ea.

Member

icinga-migration commented Jul 20, 2015

Updated by jeromed on 2015-07-20 05:50:44 +00:00

  • Status changed from Assigned to Resolved
  • Done % changed from 0 to 100

Applied in changeset 95914ea.

@icinga-migration

This comment has been minimized.

Show comment
Hide comment
@icinga-migration

icinga-migration Jul 20, 2015

Member

Updated by gbeutner on 2015-07-20 05:51:05 +00:00

Merged, thanks.

Member

icinga-migration commented Jul 20, 2015

Updated by gbeutner on 2015-07-20 05:51:05 +00:00

Merged, thanks.

@icinga-migration

This comment has been minimized.

Show comment
Hide comment
@icinga-migration

icinga-migration Jul 20, 2015

Member

Updated by gbeutner on 2015-07-20 06:50:48 +00:00

  • Target Version changed from 2.4.0 to 2.3.8
  • Backport? changed from TBD to Yes
Member

icinga-migration commented Jul 20, 2015

Updated by gbeutner on 2015-07-20 06:50:48 +00:00

  • Target Version changed from 2.4.0 to 2.3.8
  • Backport? changed from TBD to Yes
@icinga-migration

This comment has been minimized.

Show comment
Hide comment
@icinga-migration

icinga-migration Aug 15, 2015

Member

Updated by gbeutner on 2015-08-15 08:21:24 +00:00

This patch causes the default "disk" check on Windows to fail because check_disk.exe doesn't support -K.

Member

icinga-migration commented Aug 15, 2015

Updated by gbeutner on 2015-08-15 08:21:24 +00:00

This patch causes the default "disk" check on Windows to fail because check_disk.exe doesn't support -K.

@icinga-migration

This comment has been minimized.

Show comment
Hide comment
@icinga-migration

icinga-migration Aug 15, 2015

Member

Updated by gbeutner on 2015-08-15 08:23:09 +00:00

  • Relates set to 9913
Member

icinga-migration commented Aug 15, 2015

Updated by gbeutner on 2015-08-15 08:23:09 +00:00

  • Relates set to 9913
@icinga-migration

This comment has been minimized.

Show comment
Hide comment
@icinga-migration

icinga-migration Aug 29, 2016

Member

Updated by gbeutner on 2016-08-29 16:08:37 +00:00

  • Relates set to 12588
Member

icinga-migration commented Aug 29, 2016

Updated by gbeutner on 2016-08-29 16:08:37 +00:00

  • Relates set to 12588
@icinga-migration

This comment has been minimized.

Show comment
Hide comment
@icinga-migration

icinga-migration Aug 29, 2016

Member

Updated by gbeutner on 2016-08-29 16:09:10 +00:00

Apparently this commit breaks disk checks for XFS-based file systems.

Member

icinga-migration commented Aug 29, 2016

Updated by gbeutner on 2016-08-29 16:09:10 +00:00

Apparently this commit breaks disk checks for XFS-based file systems.

@icinga-migration icinga-migration added this to the 2.3.8 milestone Jan 17, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment