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

[dev.icinga.com #11439] Update the CentOS installation documentation #4057

Closed
icinga-migration opened this issue Mar 23, 2016 · 11 comments
Milestone

Comments

@icinga-migration
Copy link
Member

@icinga-migration icinga-migration commented Mar 23, 2016

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

Created by Assaf on 2016-03-23 11:02:19 +00:00

Assignee: Assaf
Status: Resolved (closed on 2016-03-24 13:51:29 +00:00)
Target Version: 2.4.5
Last Update: 2016-04-20 08:16:02 +00:00 (in Redmine)

Icinga Version: 2.4.1
Backport?: Already backported
Include in Changelog: 1

The documentation for Centos installation is out-of-date and pointing to wrong repositories and actions.

Attachments

Changesets

2016-03-24 13:50:44 +00:00 by (unknown) a6c4a9b

Update Centos/Redhat/Fedora installation instructions

Old Docs gave wrong details on how to install Icinga 2.

fixes #11439

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

2016-03-24 13:51:03 +00:00 by (unknown) 572afeb

Update AUTHORS

refs #11439

2016-03-24 14:07:56 +00:00 by (unknown) 4cac1c8

Update .mailmap for Assaf Flatto

refs #11439

2016-04-20 08:07:23 +00:00 by (unknown) 23d4455

Update Centos/Redhat/Fedora installation instructions

Old Docs gave wrong details on how to install Icinga 2.

fixes #11439

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

2016-04-20 08:07:23 +00:00 by (unknown) 9c6b070

Update AUTHORS

refs #11439

2016-04-20 08:07:23 +00:00 by (unknown) eb53677

Update .mailmap for Assaf Flatto

refs #11439

Relations:

@icinga-migration

This comment has been minimized.

Copy link
Member Author

@icinga-migration icinga-migration commented Mar 23, 2016

Updated by Assaf on 2016-03-23 11:03:00 +00:00

  • File added 0001-Updating-Docs.patch
@icinga-migration

This comment has been minimized.

Copy link
Member Author

@icinga-migration icinga-migration commented Mar 23, 2016

Updated by gbeutner on 2016-03-23 12:33:01 +00:00

Doesn't the repository RPM already install the appropriate repository signing key?

@icinga-migration

This comment has been minimized.

Copy link
Member Author

@icinga-migration icinga-migration commented Mar 23, 2016

Updated by Assaf on 2016-03-23 12:37:48 +00:00

gunnarbeutner wrote:

Doesn't the repository RPM already install the appropriate repository signing key?

It points to , but even in the new document it states to manually import the key.

@icinga-migration

This comment has been minimized.

Copy link
Member Author

@icinga-migration icinga-migration commented Mar 24, 2016

Updated by gbeutner on 2016-03-24 07:21:41 +00:00

Well, what I'm wondering is whether that's necessary. Does the repository RPM not install that key?

@icinga-migration

This comment has been minimized.

Copy link
Member Author

@icinga-migration icinga-migration commented Mar 24, 2016

Updated by mfriedrich on 2016-03-24 10:41:01 +00:00

  • Status changed from New to Assigned

The package already installs the icinga.key file. Please update your patch accordingly.

Thanks,
Michael

@icinga-migration

This comment has been minimized.

Copy link
Member Author

@icinga-migration icinga-migration commented Mar 24, 2016

Updated by gbeutner on 2016-03-24 13:20:13 +00:00

The patch does not apply cleanly:

$ git am ../Downloads/0001-Updating-Docs.patch
Applying: Updating Docs
.git/rebase-apply/patch:25: trailing whitespace.

fatal: corrupt patch at line 48
Patch failed at 0001 Updating Docs
The copy of the patch that failed is found in: .git/rebase-apply/patch
When you have resolved this problem, run "git am --continue".
If you prefer to skip this patch, run "git am --skip" instead.
To restore the original branch and stop patching, run "git am --abort".
@icinga-migration

This comment has been minimized.

Copy link
Member Author

@icinga-migration icinga-migration commented Mar 24, 2016

Updated by Assaf on 2016-03-24 13:34:59 +00:00

gunnarbeutner wrote:

The patch does not apply cleanly:

[...]

I've uploaded the patch again , i don't know why it gave that error

@icinga-migration

This comment has been minimized.

Copy link
Member Author

@icinga-migration icinga-migration commented Mar 24, 2016

Updated by Anonymous on 2016-03-24 13:51:29 +00:00

  • Status changed from Assigned to Resolved

Applied in changeset a6c4a9b.

@icinga-migration

This comment has been minimized.

Copy link
Member Author

@icinga-migration icinga-migration commented Mar 24, 2016

Updated by gbeutner on 2016-03-24 13:53:34 +00:00

  • Relates set to 11258
@icinga-migration

This comment has been minimized.

Copy link
Member Author

@icinga-migration icinga-migration commented Apr 20, 2016

Updated by gbeutner on 2016-04-20 06:30:37 +00:00

  • Subject changed from Icinga 2 CentOs installation Docs to Update the CentOS installation documentation
@icinga-migration

This comment has been minimized.

Copy link
Member Author

@icinga-migration icinga-migration commented Apr 20, 2016

Updated by gbeutner on 2016-04-20 08:16:02 +00:00

  • Backport? changed from Not yet backported to Already backported
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
1 participant
You can’t perform that action at this time.