Skip to content
This repository has been archived by the owner on Jan 15, 2019. It is now read-only.

[dev.icinga.com #4825] ido2db returning incorrect codes #1357

Closed
icinga-migration opened this issue Oct 9, 2013 · 3 comments
Closed

[dev.icinga.com #4825] ido2db returning incorrect codes #1357

icinga-migration opened this issue Oct 9, 2013 · 3 comments

Comments

@icinga-migration
Copy link

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

Created by adrianlzt on 2013-10-09 13:16:43 +00:00

Assignee: (none)
Status: Resolved (closed on 2014-01-23 15:16:42 +00:00)
Target Version: 1.10.3
Last Update: 2014-12-08 14:36:02 +00:00 (in Redmine)

Icinga Version: 1.10.0
OS Version: any

ido2db init script return code 0 even if it is not running.

  1. /etc/init.d/ido2db status; echo $?
    ido2db is not running
    0
  2. ps aux | grep ido2db
    root 21034 0.0 0.0 105300 912 pts/2 S+ 12:13 0:00 grep ido2db

In my opinion, cheking if there is a process called ido2db as well as cheking the pid file will be the most correct.

Thanks!

Changesets

2014-01-23 14:30:12 +00:00 by (unknown) 6b48de7

Fix ido2db initscript.

- removed mixed indentation
- modified status check to be LSB conform
- changed structure of status check

Signed-off-by: Michael Friedrich <michael.friedrich@netways.de>

Refs #4825

2014-01-23 14:44:41 +00:00 by (unknown) a086cf8

Merge branch 'fix/ido2db-init-4825' into support/1.10

Fixes #4825

2014-01-23 14:45:33 +00:00 by (unknown) ea237fe

Add Marco Hoyer to THANKS.

Refs #4825

2014-01-23 15:15:33 +00:00 by (unknown) 144a0b7

Update Changelog.

Refs #4968
Refs #5434
Refs #4427
Refs #4825
Refs #5263
Refs #5545
@icinga-migration
Copy link
Author

Updated by mfriedrich on 2014-01-23 14:50:23 +00:00

  • Target Version set to 1.10.3

@icinga-migration
Copy link
Author

Updated by Anonymous on 2014-01-23 15:16:42 +00:00

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

Applied in changeset icinga-core:a086cf8223120a4df53b9529275d67820078ec73.

@icinga-migration
Copy link
Author

Updated by mfriedrich on 2014-12-08 14:36:02 +00:00

  • Project changed from 18 to Core, Classic UI, IDOUtils
  • Category changed from 79 to IDOUtils
  • Icinga Version changed from 1 to 1
  • OS Version set to any

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

1 participant