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

[dev.icinga.com #1666] update icinga.spec from rhel bug #664

Closed
icinga-migration opened this Issue Jun 20, 2011 · 7 comments

Comments

Projects
None yet
1 participant
@icinga-migration
Copy link
Member

icinga-migration commented Jun 20, 2011

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

Created by mfriedrich on 2011-06-20 15:53:11 +00:00

Assignee: mfriedrich
Status: Resolved (closed on 2012-04-26 14:15:27 +00:00)
Target Version: 1.7
Last Update: 2012-04-26 14:15:27 +00:00 (in Redmine)


ofc this should be kept compatible, but there are some nice ideas behind that to provide clean upstream packages, split by their sections.

https://bugzilla.redhat.com/show\_bug.cgi?id=693608

also the newly introduced cgi logging feature should be moved somewhere (maybe add a configure option for that?).

furthermore some permissions are still wrong, and should be fixed.

Attachments

Changesets

2011-06-29 11:55:20 +00:00 by mfriedrich dab63b9

icinga.spec: move logging into single location, fix permissions and cfgs

partly taken from https://bugzilla.redhat.com/show_bug.cgi?id=693608

refs #1666
refs #1685

2012-04-19 19:52:20 +00:00 by mfriedrich 222bd9a

update icinga.spec from rhel bug #1666

a bit more work on that, breaking things.

- use the _sbindir macro instead of hardcoded /usr/sbin
- move pid file to _localstatedir/run/icinga.pid
- use name macro instead of hardcoded "icinga" everywhere
- introduce plugindir macro for global usage
- install icinga.logrotate example
- move ext cmd file location to _localstatedir/spool/icinga/cmd/icinga.cmd
- set icinga user's home to _localstatedir/spool/icinga
- move checkresults to _localstatedir/spool/icinga/checkresults
- use --with-http-auth-file from #2533
- add default /etc/icinga/passwd with icingaadmin:icingaadmin default login

refs #1666

2012-04-19 19:53:35 +00:00 by mfriedrich 59a3ba1

update README.RHEL from icinga.spec changes in #1666 and #2361

refs #1666
refs #2361

2012-04-19 20:10:44 +00:00 by mfriedrich 8f4a4ce

fix forgotten leftovers from #1666 in icinga.spec

refs #1666

Relations:

@icinga-migration

This comment has been minimized.

Copy link
Member Author

icinga-migration commented Jul 29, 2011

Updated by mfriedrich on 2011-07-29 15:07:22 +00:00

  • Status changed from New to Feedback
  • Target Version deleted 1.5
  • Done % changed from 0 to 80

let's see how it will change, feedback needed.

@icinga-migration

This comment has been minimized.

Copy link
Member Author

icinga-migration commented Apr 19, 2012

Updated by mfriedrich on 2012-04-19 18:15:40 +00:00

  • Category set to Packaging
  • Status changed from Feedback to Assigned
  • Assigned to set to mfriedrich
  • Target Version set to 1.7
  1. Please use %{_sbindir}/usermod instead of /usr/sbin/usermod .

  2. --with-lockfile="%{_localstatedir}/icinga/icinga.pid" should be
    %{_localstatedir}/run/%{name}.pid like nagios. If it really needs its own
    sub-folder, %{_localstatedir}/run/%{name}/%{name}.pid.

and for reference,
http://pkgs.fedoraproject.org/gitweb/?p=nagios.git;a=blob;f=nagios.spec;hb=HEAD

@icinga-migration

This comment has been minimized.

Copy link
Member Author

icinga-migration commented Apr 19, 2012

Updated by mfriedrich on 2012-04-19 21:01:34 +00:00

- use the _sbindir macro instead of hardcoded /usr/sbin
- move pid file to _localstatedir/run/icinga.pid
- use name macro instead of hardcoded "icinga" everywhere
- introduce plugindir macro for global usage
- install icinga.logrotate example
- move ext cmd file location to _localstatedir/spool/icinga/cmd/icinga.cmd
- set icinga user's home to _localstatedir/spool/icinga
- move checkresults to _localstatedir/spool/icinga/checkresults
- use --with-http-auth-file from #2533
- add default /etc/icinga/passwd with icingaadmin:icingaadmin default login
@icinga-migration

This comment has been minimized.

Copy link
Member Author

icinga-migration commented Apr 19, 2012

Updated by mfriedrich on 2012-04-19 21:02:07 +00:00

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

there still might be more, but for now these packages look good for further tests.

@icinga-migration

This comment has been minimized.

Copy link
Member Author

icinga-migration commented Apr 19, 2012

Updated by mfriedrich on 2012-04-19 21:03:06 +00:00

well, and the wiki entries must be updated.

@icinga-migration

This comment has been minimized.

Copy link
Member Author

icinga-migration commented Apr 19, 2012

Updated by mfriedrich on 2012-04-19 21:12:33 +00:00

https://wiki.icinga.org/display/howtos/Setting+up+Icinga+with+IDOUtils+on+RHEL

@icinga-migration

This comment has been minimized.

Copy link
Member Author

icinga-migration commented Apr 26, 2012

Updated by mfriedrich on 2012-04-26 14:15:27 +00:00

  • Tracker changed from Support to Feature
  • Status changed from Feedback to Resolved

i think this is good for now.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
You can’t perform that action at this time.