-
Notifications
You must be signed in to change notification settings - Fork 11
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
reactor: transition to Publish-PasswordHealth
#111
Conversation
- Updates `Publish-PasswordHealth` to return the results from the SoS Health Summary JSON data. #90 - Updates `Invoke-VcfHealthReport` to use the `Publish-PasswordHealth` cmdlet. #90 - Updates `Publish-PasswordHealth` to include an "Expires In (Days)" column. #40 - Removes `Publish-LocalUserExpiry` in favor of the `Publish-PasswordHealth` cmdlet. #90 - Bumps the module version to v2.0.0.1004. - Updates `CHANGELOG.md`. Signed-off-by: Ryan Johnson <johnsonryan@vmware.com>
3a97750
to
e65b293
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🚀 Ship it!
- Removes `Request-SddcManagerUserExpiry` which supported the `Publish-LocalUserExpiry` cmdlet. - Removes `Request-vCenterUserExpiry` which supported the `Publish-LocalUserExpiry` cmdlet. - Removes `Request-NsxtManagerUserExpiry` which supported the `Publish-LocalUserExpiry` cmdlet. - Removes `Request-NsxtEdgeUserExpiry` which supported the `Publish-LocalUserExpiry` cmdlet. - Removes `Request-Request-vRslcmUserExpiry` which supported the `Publish-LocalUserExpiry` cmdlet. - Updates `CHANGELOG.md`. Ref: #111 Signed-off-by: Ryan Johnson <johnsonryan@vmware.com>
- Removes `Request-SddcManagerUserExpiry` which supported the `Publish-LocalUserExpiry` cmdlet. - Removes `Request-vCenterUserExpiry` which supported the `Publish-LocalUserExpiry` cmdlet. - Removes `Request-NsxtManagerUserExpiry` which supported the `Publish-LocalUserExpiry` cmdlet. - Removes `Request-NsxtEdgeUserExpiry` which supported the `Publish-LocalUserExpiry` cmdlet. - Removes `Request-Request-vRslcmUserExpiry` which supported the `Publish-LocalUserExpiry` cmdlet. - Updates `CHANGELOG.md`. Ref: #111 Signed-off-by: Ryan Johnson <johnsonryan@vmware.com>
- Removes `Request-SddcManagerUserExpiry` which supported the `Publish-LocalUserExpiry` cmdlet. - Removes `Request-vCenterUserExpiry` which supported the `Publish-LocalUserExpiry` cmdlet. - Removes `Request-NsxtManagerUserExpiry` which supported the `Publish-LocalUserExpiry` cmdlet. - Removes `Request-NsxtEdgeUserExpiry` which supported the `Publish-LocalUserExpiry` cmdlet. - Removes `Request-Request-vRslcmUserExpiry` which supported the `Publish-LocalUserExpiry` cmdlet. - Updates `CHANGELOG.md`. Ref: #111 Signed-off-by: Ryan Johnson <johnsonryan@vmware.com>
- Removes `Request-SddcManagerUserExpiry` which supported the `Publish-LocalUserExpiry` cmdlet. - Removes `Request-vCenterUserExpiry` which supported the `Publish-LocalUserExpiry` cmdlet. - Removes `Request-NsxtManagerUserExpiry` which supported the `Publish-LocalUserExpiry` cmdlet. - Removes `Request-NsxtEdgeUserExpiry` which supported the `Publish-LocalUserExpiry` cmdlet. - Removes `Request-Request-vRslcmUserExpiry` which supported the `Publish-LocalUserExpiry` cmdlet. - Updates `CHANGELOG.md`. Ref: #111 Signed-off-by: Ryan Johnson <johnsonryan@vmware.com>
- Removed `Request-LocalUserExpiry` which supported the `Publish-LocalUserExpiry` cmdlet. - Removes `Request-SddcManagerUserExpiry` which supported the `Publish-LocalUserExpiry` cmdlet. - Removes `Request-vCenterUserExpiry` which supported the `Publish-LocalUserExpiry` cmdlet. - Removes `Request-NsxtManagerUserExpiry` which supported the `Publish-LocalUserExpiry` cmdlet. - Removes `Request-NsxtEdgeUserExpiry` which supported the `Publish-LocalUserExpiry` cmdlet. - Removes `Request-Request-vRslcmUserExpiry` which supported the `Publish-LocalUserExpiry` cmdlet. - Updates `CHANGELOG.md`. Ref: #111 Signed-off-by: Ryan Johnson <johnsonryan@vmware.com>
- Removed `Request-LocalUserExpiry` which supported the `Publish-LocalUserExpiry` cmdlet. - Removes `Request-SddcManagerUserExpiry` which supported the `Publish-LocalUserExpiry` cmdlet. - Removes `Request-vCenterUserExpiry` which supported the `Publish-LocalUserExpiry` cmdlet. - Removes `Request-NsxtManagerUserExpiry` which supported the `Publish-LocalUserExpiry` cmdlet. - Removes `Request-NsxtEdgeUserExpiry` which supported the `Publish-LocalUserExpiry` cmdlet. - Removes `Request-Request-vRslcmUserExpiry` which supported the `Publish-LocalUserExpiry` cmdlet. - Updates incorrect parameter names in examples and descriptions. - Updates `CHANGELOG.md`. Ref: #111 Signed-off-by: Ryan Johnson <johnsonryan@vmware.com>
- Removed `Request-LocalUserExpiry` which supported the `Publish-LocalUserExpiry` cmdlet. - Removes `Request-SddcManagerUserExpiry` which supported the `Publish-LocalUserExpiry` cmdlet. - Removes `Request-vCenterUserExpiry` which supported the `Publish-LocalUserExpiry` cmdlet. - Removes `Request-NsxtManagerUserExpiry` which supported the `Publish-LocalUserExpiry` cmdlet. - Removes `Request-NsxtEdgeUserExpiry` which supported the `Publish-LocalUserExpiry` cmdlet. - Removes `Request-Request-vRslcmUserExpiry` which supported the `Publish-LocalUserExpiry` cmdlet. - Updates incorrect parameter names in examples and descriptions. - Updates `CHANGELOG.md`. Ref: #111 Signed-off-by: Ryan Johnson <johnsonryan@vmware.com>
- Removed `Request-LocalUserExpiry` which supported the `Publish-LocalUserExpiry` cmdlet. - Removes `Request-SddcManagerUserExpiry` which supported the `Publish-LocalUserExpiry` cmdlet. - Removes `Request-vCenterUserExpiry` which supported the `Publish-LocalUserExpiry` cmdlet. - Removes `Request-NsxtManagerUserExpiry` which supported the `Publish-LocalUserExpiry` cmdlet. - Removes `Request-NsxtEdgeUserExpiry` which supported the `Publish-LocalUserExpiry` cmdlet. - Removes `Request-Request-vRslcmUserExpiry` which supported the `Publish-LocalUserExpiry` cmdlet. - Updates incorrect parameter names in examples and descriptions. - Updates `CHANGELOG.md`. Ref: #111 Signed-off-by: Ryan Johnson <johnsonryan@vmware.com>
- Removed `Request-LocalUserExpiry` which supported the `Publish-LocalUserExpiry` cmdlet. - Removes `Request-SddcManagerUserExpiry` which supported the `Publish-LocalUserExpiry` cmdlet. - Removes `Request-vCenterUserExpiry` which supported the `Publish-LocalUserExpiry` cmdlet. - Removes `Request-NsxtManagerUserExpiry` which supported the `Publish-LocalUserExpiry` cmdlet. - Removes `Request-NsxtEdgeUserExpiry` which supported the `Publish-LocalUserExpiry` cmdlet. - Removes `Request-Request-vRslcmUserExpiry` which supported the `Publish-LocalUserExpiry` cmdlet. - Updates incorrect parameter names in examples and descriptions. - Updates `CHANGELOG.md`. Ref: #111 Signed-off-by: Ryan Johnson <johnsonryan@vmware.com>
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. |
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
Publish-PasswordHealth
to return the results from the SoS Health Summary JSON data. Transition toPublish-PasswordHealth
fromPublish-LocalUserExpiry
#90Invoke-VcfHealthReport
to use thePublish-PasswordHealth
cmdlet. Transition toPublish-PasswordHealth
fromPublish-LocalUserExpiry
#90Publish-PasswordHealth
to include an "Expires In (Days)" column. Add support to display "Expires In (Days)" in the Password section of the Health Report #40Publish-LocalUserExpiry
in favor of thePublish-PasswordHealth
cmdlet. Transition toPublish-PasswordHealth
fromPublish-LocalUserExpiry
#90CHANGELOG.md
.Type of Pull Request
Please describe:
Related to Existing Issues
Closes #90
Closes #40
Closes #39
Test and Documentation Coverage
Breaking Changes?