fix: update Test-VcfReportingPrereq#82
Merged
Merged
Conversation
Signed-off-by: bhumitra nagar <bnagar@vmware.com>
Test-VcfReportingPrereq
Contributor
|
Update: I will review it soon now that v1.0.6 is now released. |
tenthirtyam
approved these changes
Jan 31, 2023
Contributor
tenthirtyam
left a comment
There was a problem hiding this comment.
Photon OS
PS /home/vmware/code/VMware.CloudFoundation.Reporting> $PSVersionTable
Name Value
---- -----
PSVersion 7.2.7
PSEdition Core
GitCommitId 7.2.7
OS Linux 5.10.109-2.ph4-esx #1-photon SMP Sat May 14 02:59:09 UTC 2022
Platform Unix
PSCompatibleVersions {1.0, 2.0, 3.0, 4.0…}
PSRemotingProtocolVersion 2.3
SerializationVersion 1.1.0.1
WSManStackVersion 3.0
PS /home/vmware/code> cd ./VMware.CloudFoundation.Reporting/
PS /home/vmware/code/VMware.CloudFoundation.Reporting> Import-Module ./VMware.CloudFoundation.Reporting.psd1
PS /home/vmware/code/VMware.CloudFoundation.Reporting> Test-VcfReportingPrereq
WARNING: PowerShell Module: VMware.PowerCLI Version: 12.7.0 Not Installed, Please update before proceeding.
PS /home/vmware/code/VMware.CloudFoundation.Reporting> Update-Module -Name VMware.PowerCLI
Untrusted repository
You are installing the modules from an untrusted repository. If you trust this repository, change its InstallationPolicy value by running the
Set-PSRepository cmdlet. Are you sure you want to install the modules from 'PSGallery'?
[Y] Yes [A] Yes to All [N] No [L] No to All [S] Suspend [?] Help (default is "N"): A
PS /home/vmware/code/VMware.CloudFoundation.Reporting> Get-InstalledModule VMware.PowerCLI
Version Name Repository Description
------- ---- ---------- -----------
13.0.0.20829139 VMware.PowerCLI PSGallery This Windows PowerShell module contains VMware.PowerCLI
PS /home/vmware/code/VMware.CloudFoundation.Reporting> Test-VcfReportingPrereq
PowerShell Module: VMware.PowerCLI Version: 13.0.0.20829139 Found, Supports the minimum required version.
PowerShell Module: VMware.vSphere.SsoAdmin Version: 1.3.8 Found, Supports the minimum required version.
PowerShell Module: PowerVCF Version: 2.2.0 Found, Supports the minimum required version.
PowerShell Module: PowerValidatedSolutions Version: 2.0.0.1010 Found, Supports the minimum required version.
PS /home/vmware/code/VMware.CloudFoundation.Reporting>Windows Server 2022 with Windows PowerShell 5.1
PS F:\VMware.CloudFoundation.Reporting> $PSVersionTable
Name Value
---- -----
PSVersion 5.1.14393.5127
PSEdition Desktop
PSCompatibleVersions {1.0, 2.0, 3.0, 4.0...}
BuildVersion 10.0.14393.5127
CLRVersion 4.0.30319.42000
WSManStackVersion 3.0
PSRemotingProtocolVersion 2.3
SerializationVersion 1.1.0.1
PS F:\VMware.CloudFoundation.Reporting> Import-Module .\VMware.CloudFoundation.Reporting.psd1
PS F:\VMware.CloudFoundation.Reporting> Test-VcfReportingPrereq
PowerShell Module: VMware.PowerCLI Version: 12.7.0.20091289 Found, Supports the minimum required version.
PowerShell Module: VMware.vSphere.SsoAdmin Version: 1.3.8 Found, Supports the minimum required version.
PowerShell Module: PowerVCF Version: 2.2.0 Found, Supports the minimum required version.
WARNING: PowerShell Module: PowerValidatedSolutions Version: 2.0.0 Not Installed, Please update before proceeding.
PS F:\VMware.CloudFoundation.Reporting> Update-Module -Name PowerValidatedSolutions
PS F:\VMware.CloudFoundation.Reporting> Test-VcfReportingPrereq
PowerShell Module: VMware.PowerCLI Version: 12.7.0.20091289 Found, Supports the minimum required version.
PowerShell Module: VMware.vSphere.SsoAdmin Version: 1.3.8 Found, Supports the minimum required version.
PowerShell Module: PowerVCF Version: 2.2.0 Found, Supports the minimum required version.
PowerShell Module: PowerValidatedSolutions Version: 2.0.0.1010 Found, Supports the minimum required version.
PS F:\VMware.CloudFoundation.Reporting>|
I'm going to lock this pull request because it has been closed for 30 days. This helps our maintainers find and focus on the active issues. If you have found a problem that seems related to this change, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Signed-off-by: bhumitra nagar bnagar@vmware.com
In order to have a good experience with our community, we recommend that you read the contributing guidelines for making a pull request.
Summary of Pull Request
Test-VcfReportingPrereqdoesnt return any output on Photon OSThere were following issues with the code logic -
Get-Moduleshould be changed toGet-InstalledModule.if (!$module -eq "VMware.PowerCLI")condition for checking PowerCli module is not correct.Here is the output after the fix
on Windows -
On Photon OS
Type of Pull Request
Please describe:
Related to Existing Issues
Closes #81
Test and Documentation Coverage
Breaking Changes?