Skip to content

Conversation

Al2Klimov
Copy link
Member

@Al2Klimov Al2Klimov commented Jun 1, 2023

@cla-bot cla-bot bot added the cla/signed label Jun 1, 2023
@Al2Klimov
Copy link
Member Author

Walked through Icinga/icinga2#9062 (comment) again. Of course I chose the latest Icinga 2 Windows MSI mentioned in that PR. After the latter I patched the local file like here. Now, after Write-IcingaFrameworkCodeCache, Get-IcingaCheckCommandConfig output changed exactly like the PR diff. So far, so good.

@Al2Klimov
Copy link
Member Author

Successfully finished the walkthrough. I.e. everything works, w/ and w/o ifw-api.

@LordHepipud
Copy link
Collaborator

Thanks for the PR. Can you please add the following to the changelog.md for v1.11.0 under the Enhancements section and sqush it to one commit:

### Enhancements

* [#629](https://github.com/Icinga/icinga-powershell-framework/pull/629) Adds support for Icinga 2.14 direct API call to Icinga for Windows

@LordHepipud
Copy link
Collaborator

One side note: You will also have to edit the plain Icinga configuration setting at line 635 and beyond. Can you please update the PR accordingly?

@Al2Klimov
Copy link
Member Author

I must be blind and/or stupid not to find the changelog:

➜  icinga-powershell-framework git:(ifw-api) ls
LICENSE                          cache                            icinga-powershell-framework.psd1
PSScriptAnalyzerSettings.psd1    certificate                      icinga-powershell-framework.psm1
README.md                        custom                           lib
RoleCapabilities                 doc                              templates
➜  icinga-powershell-framework git:(ifw-api) find . -iname changelog.md
➜  icinga-powershell-framework git:(ifw-api)

@Al2Klimov
Copy link
Member Author

You can commit it as second commit and thereby get the credits for it.

@Al2Klimov
Copy link
Member Author

⚠️ Please don't merge yet! ⚠️

We've discovered more problems this thing solely can't cope with!

Maybe this will even become unnecessary.

@LordHepipud

@Al2Klimov Al2Klimov changed the title Don't pass -NoProfile, -NoLogo, -ExecutionPolicy and -C if $ifw_api_port$ is set [⚠️ Please don't merge yet! ⚠️] Don't pass -NoProfile, -NoLogo, -ExecutionPolicy and -C if $ifw_api_port$ is set Jun 6, 2023
@Al2Klimov Al2Klimov marked this pull request as draft June 6, 2023 11:36
@Al2Klimov
Copy link
Member Author

This has never actually worked, see discussion since Icinga/icinga2#9062 (comment)

@Al2Klimov Al2Klimov closed this Jun 7, 2023
@Al2Klimov Al2Klimov deleted the ifw-api branch June 7, 2023 14:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants