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.

Sign up for GitHub

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

Jump to bottom

Can't create resources and command transports which require a port #2998

Closed
mwaldmueller opened this issue Oct 6, 2017 · 3 comments · Fixed by #3010
Closed

Can't create resources and command transports which require a port #2998

mwaldmueller opened this issue Oct 6, 2017 · 3 comments · Fixed by #3010
Assignees
@nilmerg
Labels
bug Something isn't working queue/important Blocks a release or needs immediate attention
Milestone
2.5.0

Comments

@mwaldmueller
Copy link

It's not possible to create a new Command Transport for the monitoring module using the Icinga 2 API via GUI:

bildschirmfoto 2017-10-06 um 15 39 16

bildschirmfoto 2017-10-06 um 11 27 02

Environment

  • Icinga Web 2 version and modules (System - About): 2.4.2
  • Version used (icinga2 --version): r2.7.1-1
  • Operating System and version: CentOS Linux release 7.4.1708 (Core)
@nbuchwitz
Copy link
Contributor

Same error (port) here while adding a new ressource.

Environment

Icinga Web 2 version and modules (System - About): 2.4.2
Version used (icinga2 --version): r2.7.1-1
Operating System and version: CentOS Linux release 7.4.1708 (Core)

@lippserd lippserd changed the title New Command Transport doesn't recognize port setting Can't create resources and command transports which require a port Oct 9, 2017
@lippserd
Copy link
Member

lippserd commented Oct 9, 2017

Hi,

I included eb34300 in v2.4.2 and didn't recognize this issue in our final tests. Will be fixed asap.

@nilmerg Please take care of this issue.

Cheers,
Eric

@lippserd lippserd mentioned this issue Oct 9, 2017
Closed
@lippserd lippserd added bug Something isn't working queue/important Blocks a release or needs immediate attention labels Oct 9, 2017
@lippserd lippserd added this to the 2.5.0 milestone Oct 9, 2017
nilmerg added a commit that referenced this issue Oct 10, 2017
@nilmerg
Ensure that we still provide all submitted values when validating forms
554b79e 
fixes #2998
refs #2509
nilmerg added a commit that referenced this issue Oct 10, 2017
@nilmerg
Ensure that forms properly mimic all side-effects of isValidPartial()..
dbd45db 
..if overidden

refs #2998
nilmerg added a commit that referenced this issue Oct 10, 2017
@nilmerg
LdapBackendForm: Don't handle discovery as part of isValidPartial()
921c339 
We can't do such things at the moment if it's a subform.

refs #2998
nilmerg added a commit that referenced this issue Oct 10, 2017
@nilmerg
Set a submit label in method init() and not just when creating a form
d118b5c 
refs #2998
nilmerg added a commit that referenced this issue Oct 10, 2017
@nilmerg
Ensure that we still provide all submitted values when validating forms
33d2231 
fixes #2998
refs #2509
@nilmerg nilmerg mentioned this issue Oct 10, 2017
Merged
@dnsmichi
Copy link
Contributor

https://monitoring-portal.org/index.php?thread/42126-ldap-ad-binding/

nilmerg added a commit that referenced this issue Oct 23, 2017
@nilmerg
Revert "Form: Don't prevent disabled elements from getting overwritte…
d348d95 
…n.."

This reverts commit eb34300.

refs #2998
nilmerg added a commit that referenced this issue Oct 23, 2017
@nilmerg
Form: Create the form as part of method `setDefaults'
2e1bac8 
We don't care anymore about the BC in Zend version 1.12.7.
Hence we'll happily override setDefaults now. Yay!

refs #2998
@dnsmichi dnsmichi mentioned this issue Oct 30, 2017
Closed
nilmerg added a commit that referenced this issue Nov 6, 2017
@nilmerg
RepositoryForm: Utilize setDefaults instead of populate now
4cf577b 
refs #3010
refs #2998
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@nilmerg nilmerg

Labels
bug Something isn't working queue/important Blocks a release or needs immediate attention
Projects
None yet
Milestone
2.5.0
Development

Successfully merging a pull request may close this issue.

Bugfix/cant create resources and command transports 2998 Icinga/icingaweb2
5 participants
@dnsmichi @lippserd @mwaldmueller @nbuchwitz @nilmerg