[dev.icinga.com #9205] port empty when using icinga2 node wizard

[icinga-migration]

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

Created by bsheqa on 2015-04-30 10:16:10 +00:00

Assignee: jflach
Status: Resolved (closed on 2015-04-30 11:31:00 +00:00)
Target Version: 2.3.6
Last Update: 2015-07-03 07:42:05 +00:00 (in Redmine)

Icinga Version: v2.3.0-186-ge2247d3
Backport?: Already backported
Include in Changelog: 1

---

When using the 'icinga2 node wizard' to add a new icinga2 agent, the port in zones.conf is being set to "" by default.

root@debian:/etc/apt/sources.list.d# icinga2 node wizard
Welcome to the Icinga 2 Setup Wizard!

We'll guide you through all required configuration details.



Please specify if this is a satellite setup ('n' installs a master setup) [Y/n]: 
Starting the Node setup routine...
Please specifiy the common name (CN) [debian]: 
Please specifiy the local zone name [debian]: 
Please specify the master endpoint(s) this node should connect to:
Master Common Name (CN from your master setup): icinga1
Do you want to establish a connection to the master from this node? [Y/n]: 
Please fill out the master connection information:
Master endpoint host (Your master's IP address or FQDN): icinga1
Master endpoint port [5665]: 
Add more master endpoints? [y/N]: 
Please specify the master connection for CSR auto-signing (defaults to master endpoint host):
Host [icinga1]: 
Port [5665]: 
information/base: Writing private key to '/etc/icinga2/pki/debian.key'.
information/base: Writing X509 certificate to '/etc/icinga2/pki/debian.crt'.
information/cli: Generating self-signed certifiate:
information/cli: Fetching public certificate from master (icinga1, 5665):

information/cli: Writing trusted certificate to file '/etc/icinga2/pki/trusted-master.crt'.
information/cli: Stored trusted master certificate in '/etc/icinga2/pki/trusted-master.crt'.

Please specify the request ticket generated on your Icinga 2 master.
 (Hint: # icinga2 pki ticket --cn 'debian'): af1ba25be8a9f1b2673a82cc78e8d2fe89992650
information/cli: Processing self-signed certificate request. Ticket 'af1ba25be8a9f1b2673a82cc78e8d2fe89992650'.

information/cli: Created backup file '/etc/icinga2/pki/debian.crt.orig'.
information/cli: Writing signed certificate to file '/etc/icinga2/pki/debian.crt'.
information/cli: Writing CA certificate to file '/etc/icinga2/pki/ca.crt'.
Please specify the API bind host/port (optional):
Bind Host []: 
Bind Port []: 
Accept config from master? [y/N]: y
Accept commands from master? [y/N]: y
information/cli: Disabling the Notification feature.
Disabling feature notification. Make sure to restart Icinga 2 for these changes to take effect.
information/cli: Enabling the Apilistener feature.
Enabling feature api. Make sure to restart Icinga 2 for these changes to take effect.
information/cli: Created backup file '/etc/icinga2/features-available/api.conf.orig'.
information/cli: Generating local zones.conf.
information/cli: Dumping config items to file '/etc/icinga2/zones.conf'.
information/cli: Created backup file '/etc/icinga2/zones.conf.orig'.
information/cli: Updating constants.conf.
information/cli: Created backup file '/etc/icinga2/constants.conf.orig'.
information/cli: Updating constants file '/etc/icinga2/constants.conf'.
information/cli: Updating constants file '/etc/icinga2/constants.conf'.
Done.

Now restart your Icinga 2 daemon to finish the installation!

The generated zones.conf looks like this:

/*
 * Generated by Icinga 2 node setup commands
 * on 2015-04-30 11:35:07 +0200
 */

object Endpoint "icinga1" {
    host = "icinga1"
    port = ""
}

object Zone "master" {
    endpoints = [ "icinga1" ]
}

object Endpoint "debian" {
}

object Zone "debian" {
    //this is the local node = "debian"
    endpoints = [ "debian" ]
    parent = "master"
}

Icinga2 Version

root@debian:/etc/icinga2# icinga2 --version
icinga2 - The Icinga 2 network monitoring daemon (version: v2.3.0-186-ge2247d3)

Copyright (c) 2012-2015 Icinga Development Team (https://www.icinga.org)
License GPLv2+: GNU GPL version 2 or later 
This is free software: you are free to change and redistribute it.
There is NO WARRANTY, to the extent permitted by law.

Application information:
  Installation root: /usr
  Sysconf directory: /etc
  Run directory: /run
  Local state directory: /var
  Package data directory: /usr/share/icinga2
  State path: /var/lib/icinga2/icinga2.state
  Objects path: /var/cache/icinga2/icinga2.debug
  Vars path: /var/cache/icinga2/icinga2.vars
  PID path: /run/icinga2/icinga2.pid

System information:
  Operating system: Linux
  Operating system version: 3.2.0-4-amd64
  Architecture: x86_64
  Distribution: Debian GNU/Linux 7.8 (wheezy)

Changesets

2015-04-30 10:52:35 +00:00 by jflach f907a7f

Fix Port wrong default port in Node Wizard

fixes #9205

2015-07-03 07:42:27 +00:00 by jflach 9d3020c

Fix Port wrong default port in Node Wizard

fixes #9205

2015-07-03 10:58:54 +00:00 by (unknown) f4aeb19

Cli: Don't add empty host/port endpoint attributes in node wizard

refs #9205
refs #9536

2015-07-03 11:03:07 +00:00 by (unknown) d3cb1ad

Cli: Don't add empty host/port endpoint attributes in node wizard

refs #9205
refs #9536

---

Relations:

• relates #9205
• relates #9205

[icinga-migration]

Updated by jflach on 2015-04-30 10:55:03 +00:00

• Status changed from New to Resolved
• Done % changed from 0 to 100

Applied in changeset f907a7f.

[icinga-migration]

Updated by mfriedrich on 2015-04-30 11:01:33 +00:00

• Status changed from Resolved to Assigned
• Assigned to set to jflach

Please set time estimates, spent time, target versions before adding git commit fixes messages.

[icinga-migration]

Updated by jflach on 2015-04-30 11:31:00 +00:00

• Status changed from Assigned to Resolved
• Target Version set to 2.4.0
• Estimated Hours set to 0.05

[icinga-migration]

Updated by gbeutner on 2015-05-13 08:12:27 +00:00

• Category set to CLI

[icinga-migration]

Updated by gbeutner on 2015-05-13 08:18:35 +00:00

• Target Version changed from 2.4.0 to 2.3.5

[icinga-migration]

Updated by mfriedrich on 2015-07-03 07:41:03 +00:00

• Relates set to 9536

[icinga-migration]

Updated by mfriedrich on 2015-07-03 07:42:05 +00:00

• Target Version changed from 2.3.5 to 2.3.6
• Backport? changed from TBD to Yes

This wasn't backported to 2.3.5, I'm applying this now for 2.3.6

[icinga-migration]

Updated by mfriedrich on 2015-07-15 07:51:25 +00:00

• Relates set to 9623