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

Update Windows wizard from enhanced CSR signing (optional ticket) #5681

Closed
4 tasks done
dnsmichi opened this issue Oct 16, 2017 · 4 comments · Fixed by #5719
Closed
4 tasks done

Update Windows wizard from enhanced CSR signing (optional ticket) #5681

dnsmichi opened this issue Oct 16, 2017 · 4 comments · Fixed by #5719
Assignees
Labels
area/cli Command line helpers area/distributed Distributed monitoring (master, satellites, clients) area/windows Windows agent and plugins enhancement New feature or request
Milestone

Comments

@dnsmichi
Copy link
Contributor

dnsmichi commented Oct 16, 2017

Blocked by #5545

Tasks

  • Terminology on "Master" nodes - "master/satellite"
  • Optional ticket - verify node setup CLI command
  • Asynchronous CSR signing with no immediate answer
  • Dokumentation, Screenshots
@dnsmichi dnsmichi added area/cli Command line helpers area/distributed Distributed monitoring (master, satellites, clients) enhancement New feature or request area/windows Windows agent and plugins labels Oct 16, 2017
@dnsmichi dnsmichi self-assigned this Oct 16, 2017
@dnsmichi dnsmichi added this to the 2.8.0 milestone Oct 16, 2017
@dnsmichi
Copy link
Contributor Author

Wizard is currently here:

Running command 'icinga2.exe node setup --master_host mbmif.int.netways.de,5665 --endpoint mbmif.int.netways.de,mbmif.int.netways.de,5665 --accept-config --accept-commands --trustedcert "C:\Users\michi\AppData\Local\Temp\tmp1D62.tmp" --cn "DESKTOP-IHRPO96" --zone "DESKTOP-IHRPO96"' produced the following output:

critical

Needs a proper error handling for pending "no ticket" answers.

@dnsmichi
Copy link
Contributor Author

dnsmichi commented Nov 2, 2017

[2017-11-02 16:17:27 +0100] information/ApiListener: New client connection for identity 'winmif' from [::ffff:10.0.10.57]:51429 (certificate validation failed: code 18: self signed certificate)
[2017-11-02 16:17:27 +0100] notice/ApiListener: New JSON-RPC client
[2017-11-02 16:17:27 +0100] notice/WorkQueue: Spawning WorkQueue threads for 'JsonRpcConnection, #1'
[2017-11-02 16:17:27 +0100] notice/JsonRpcConnection: Received 'icinga::Hello' message from 'winmif'
[2017-11-02 16:17:27 +0100] notice/JsonRpcConnection: Received 'pki::RequestCertificate' message from 'winmif'
[2017-11-02 16:17:27 +0100] information/JsonRpcConnection: Received certificate request for CN 'winmif' not signed by our CA.
[2017-11-02 16:17:27 +0100] information/JsonRpcConnection: Certificate request for CN 'winmif' is pending. Waiting for approval.

I still need to add something to the client that the request is pending, e.g. after installing the service.

@dnsmichi
Copy link
Contributor Author

dnsmichi commented Nov 2, 2017

screen shot 2017-11-02 at 16 24 39

@dnsmichi
Copy link
Contributor Author

dnsmichi commented Nov 3, 2017

While at it, the inputs via form do not trim strings as we do with the node wizard. I'm fixing this as part of this update.

screen shot 2017-11-03 at 13 44 03

dnsmichi pushed a commit that referenced this issue Nov 3, 2017
We always have to tick them anyways, as otherwise the setup wizard
won't work with CSR signing.

refs #5681
dnsmichi pushed a commit that referenced this issue Nov 3, 2017
dnsmichi pushed a commit that referenced this issue Nov 3, 2017
This avoids white space problems with tickets and host names.

refs #5681
refs #5705
dnsmichi pushed a commit that referenced this issue Nov 3, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/cli Command line helpers area/distributed Distributed monitoring (master, satellites, clients) area/windows Windows agent and plugins enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant