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

Replace CIM/WMI calls for wrapper function for proper error/permission handling #34

Merged
merged 1 commit into from
Aug 21, 2020

Conversation

LordHepipud
Copy link
Collaborator

@LordHepipud LordHepipud commented Jul 27, 2020

Author: @LordHepipud

This issue is a direct reference to #80 of the PowerShell Framework

Tasks: @LordHepipud

  • Code review
  • Testing
  • Update developer guidelines for new handling

Tasks: @Crited

  • Testing

LordHepipud added a commit that referenced this pull request Jul 27, 2020
@LordHepipud LordHepipud self-assigned this Jul 27, 2020
@LordHepipud LordHepipud added this to the v.1.2.0 milestone Jul 27, 2020
@LordHepipud LordHepipud added the enhancement New feature or request label Jul 27, 2020
@LordHepipud LordHepipud requested a review from Crited July 27, 2020 10:01
LordHepipud added a commit that referenced this pull request Jul 27, 2020
Implements #34
LordHepipud added a commit that referenced this pull request Aug 3, 2020
LordHepipud added a commit that referenced this pull request Aug 21, 2020
LordHepipud added a commit that referenced this pull request Aug 21, 2020
Implements #34
LordHepipud added a commit that referenced this pull request Aug 21, 2020
@LordHepipud LordHepipud force-pushed the feature/cim_wmi_wrapper branch from c0454ce to 3752657 Compare August 21, 2020 21:17
@LordHepipud LordHepipud force-pushed the feature/cim_wmi_wrapper branch from 3752657 to 377bb4b Compare August 21, 2020 21:18
@LordHepipud LordHepipud merged commit beb2b00 into master Aug 21, 2020
@pakyg
Copy link

pakyg commented Aug 28, 2020

Hi, updating to the latest version of the Plugins, I was presented with this error output:

Get-IcingaWindowsInformation : The term 'Get-IcingaWindowsInformation' is not
recognized as the name of a cmdlet, function, script file, or operable
program.

How can I fix it?
Thanks so much

@yhabteab
Copy link
Member

Hiii @patrickgidiuli, When you open a new powershell instance you will either need to run Use-Icinga to load all Icinga related modules or you can run the commands as follows. icinga { Your Commands here; }

Hope this will help you!

Best,
YH

@pakyg
Copy link

pakyg commented Aug 28, 2020

Hi @Yonas-net ,thanks for the reply. I can see the error output from IcingaWeb .. From the Powershell window everything works.
IcingaWeb

@yhabteab
Copy link
Member

If you use icinga-powershell-plugins release tags it will not work because these new features will be released on Monday. But if you have the branch master it should work. Is the user who runs the checks in Icinga Web 2 identical to the user who tested the checks locally?

@pakyg
Copy link

pakyg commented Aug 28, 2020

@Yonas-net Yes the user is the same, even with the Master version, the same error output occurs .. Do you recommend waiting for Monday for the final release? this is the first time that I have this problem, with the previous version everything worked great..

Thank you very much!
PG

@yhabteab
Copy link
Member

yhabteab commented Aug 28, 2020

Waiting for the final release on Monday would be an option but you also have to check out the master in your icinga-powershell-framework repo. Because Get-IcingaWindowsInformation is in the framework and unless you check out the master it will not work. And if none of this helps you, please contact us again.

Best,
YH

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants