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 #12778] Advice against creating zones/endpoints #463

Closed
icinga-migration opened this issue Sep 23, 2016 · 4 comments
Closed
Assignees
Milestone

Comments

@icinga-migration
Copy link

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

Created by tgelf on 2016-09-23 15:28:20 +00:00

Assignee: (none)
Status: New
Target Version: (none)
Last Update: 2016-11-30 08:06:09 +00:00 (in Redmine)


It rarely makes any sense to manually create zones/endpoints. Beginners coming from plain Icinga 2 config often want to start to build their infrastructure with Director exactly like they did before. That way they likely manage it to shoot themselves in the foot while doing completely needless work.

The related forms (Zones, endpoints) should eventually show a big note suggesting to re-read the docs to be absolutely sure that this is really what they need and want.

Changesets

2016-09-29 19:46:21 +00:00 by (unknown) 2a83225

Render the Endpoint to it's Zone parent zone

So you are actually able to define a satellite zone/endpoint from inside the director.

Lower-Knowledge will be deployed via director, higher (or same level) zones and endpoints have to be configured in static local config.

refs #12778

Relations:

@icinga-migration
Copy link
Author

Updated by mfrosch on 2016-09-29 19:49:41 +00:00

Yes we should discuss this.

I've added a commit to branch bugfix/render-endpoint-to-parent-zone which would basically allow at least the satellites to be configured via Director.

@icinga-migration
Copy link
Author

Updated by mfrosch on 2016-10-26 14:56:34 +00:00

  • Relates set to 12988

@icinga-migration
Copy link
Author

Updated by copis on 2016-11-30 08:06:09 +00:00

There are some reasons about why this branch are not merged in lastest released director 1.2.0?

I tested the new director_1.2.0 with modified IcingaEndpoing.php and works

Thomas-Gelf added a commit that referenced this issue Mar 29, 2017
@Thomas-Gelf Thomas-Gelf self-assigned this Mar 29, 2017
@Thomas-Gelf Thomas-Gelf added this to the 1.3.2 milestone Mar 29, 2017
@Thomas-Gelf
Copy link
Contributor

Advice has been added long time ago, rendering has been fixed, slightly different than proposed.

tobiasvdk pushed a commit to tobiasvdk/icingaweb2-module-director that referenced this issue May 22, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants