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

Use Icinga 2 API to detect the IDO host/database/user in the setup wizard #2544

Open
icinga-migration opened this issue Sep 5, 2016 · 11 comments
Labels
area/installation Affects the installation enhancement New feature or improvement queue/wishlist Probable candidate for a future release

Comments

@icinga-migration
Copy link

icinga-migration commented Sep 5, 2016

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

Created by elippmann on 2016-09-05 06:13:10 +00:00

@icinga-migration icinga-migration added enhancement New feature or improvement area/installation Affects the installation labels Jan 17, 2017
@icinga-migration icinga-migration added this to the Backlog milestone Jan 17, 2017
@lippserd
Copy link
Member

Hi Alex,

The setup of the monitoring module could be simplified by fetching the Icinga 2 API for all necessary information. Of course the user has to provide API crendentials beforehand. Please have a look how we could implement this.

Cheers,
Eric

@Thomas-Gelf
Copy link
Contributor

This absolutely makes sense and can be helpful especially for smallish setups. Some thoughts:

  • Suggesting the database name (nearly) always makes sense, the field should be pre-filled in case the information is available
  • Suggesting the host name makes sense for smaller setups with no clustering involved, the field could be filled per default
  • I would not pre-fill the username at all, using the same user for web and core should be considered bad practice for various reasons. But we could offer a link or button allowing one to "copy" the user name, combined with a short notice telling him that we would advice against doing so.

Cheers,
Thomas

@Al2Klimov
Copy link
Member

@Thomas-Gelf: You'd provide a button to do bad practice OOTB? C'mon...

@Thomas-Gelf
Copy link
Contributor

Believe me, I wouldn't. Still, there is a lot of lazy people out there. And this is what this feature request asks for. I'm fine with all variants, as long as the username is not going to be populated with the one you can read through the API.

Al2Klimov added a commit that referenced this issue Oct 12, 2017
Al2Klimov added a commit that referenced this issue Oct 12, 2017
Al2Klimov added a commit that referenced this issue Oct 12, 2017
Al2Klimov added a commit that referenced this issue Oct 12, 2017
Al2Klimov added a commit that referenced this issue Oct 12, 2017
Al2Klimov added a commit that referenced this issue Oct 12, 2017
@Al2Klimov
Copy link
Member

Note for the future myself:
TODO:

  • rethink 430f823 (explicit CN?)
  • ReST request w/ server CA

@Al2Klimov
Copy link
Member

blocked by #2541

Al2Klimov added a commit that referenced this issue Oct 13, 2017
@Al2Klimov
Copy link
Member

Note: 430f823 is OK.

@Al2Klimov
Copy link
Member

blocked by #3016, not #2541

@Al2Klimov
Copy link
Member

Note for the future myself at work: While playing around w/ TLS certchains for my home setup (really, I don't work on IW2 at half past 23:00 (especially on Fri 13th ;-) )) I discovered yet another "oh not again" factor: To validate a remote's TLS cert you have to store all CA's in the chain locally. (Yeah, that will be a kinda longer journey...) Luckily no problem in the I2 context.

@Al2Klimov
Copy link
Member

blocked by #3036, not #3016

@Al2Klimov
Copy link
Member

"To validate a remote's TLS cert you have to store all CA's in the chain locally." WRONG! The remote has to provide all intermediate CAs and may provide also the root CA.

@lippserd lippserd removed this from the Backlog milestone Apr 11, 2018
@Al2Klimov Al2Klimov removed their assignment Dec 7, 2018
@lippserd lippserd changed the title [dev.icinga.com #12638] Use Icinga 2 API to detect the IDO host/database/user in the setup wizard Use Icinga 2 API to detect the IDO host/database/user in the setup wizard May 6, 2019
@lippserd lippserd added this to the 2.8.0 milestone May 6, 2019
@nilmerg nilmerg removed this from the 2.8.0 milestone Dec 3, 2019
@nilmerg nilmerg added the queue/wishlist Probable candidate for a future release label Dec 3, 2019
@nilmerg nilmerg added this to Wishlist in v2.9.0 Jun 16, 2020
@nilmerg nilmerg added this to Wishlist in v2.10.0 Apr 13, 2021
@nilmerg nilmerg removed this from Wishlist in v2.9.0 Apr 13, 2021
@nilmerg nilmerg removed this from Wishlist in v2.10.0 Jan 31, 2022
@nilmerg nilmerg added this to To do in 2.11.0 via automation Jan 31, 2022
@nilmerg nilmerg moved this from To do to Wishlist in 2.11.0 Jan 31, 2022
@nilmerg nilmerg removed this from Wishlist in 2.11.0 Jun 14, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/installation Affects the installation enhancement New feature or improvement queue/wishlist Probable candidate for a future release
Projects
None yet
Development

No branches or pull requests

5 participants