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

PE-26276 Update regex for TLS check #138

Merged
merged 1 commit into from
Apr 8, 2019
Merged

PE-26276 Update regex for TLS check #138

merged 1 commit into from
Apr 8, 2019

Conversation

shaigy
Copy link
Contributor

@shaigy shaigy commented Apr 8, 2019

Update regex for TLS to inlcude solaris-10, solaris-11 and sles11

@shaigy
PE-26276 Update regex for TLS check
d030ecc 
Update regex for TLS to inlcude solaris-10, solaris-11 and sles11
@shaigy
Copy link
Contributor Author

shaigy commented Apr 8, 2019

Validated 2019.1.x install on solaris-10, solaris-11 and sles11
Output from solaris10 install:

setup/install.rb passed in 534.45 seconds
      Test Suite: pre_suite @ 2019-04-08 11:53:32 -0700

      - Host Configuration Summary -


              - Test Case Summary for suite 'pre_suite' -
       Total Suite Time: 534.43 seconds
      Average Test Time: 534.43 seconds
              Attempted: 1
                 Passed: 1
                 Failed: 0
                Errored: 0
                Skipped: 0
                Pending: 0
                  Total: 1

      - Specific Test Case Status -

actowery
actowery approved these changes Apr 8, 2019
View reviewed changes
Copy link
Contributor

@actowery actowery left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

updated regex and new spec looks good to me 👍

@actowery actowery merged commit fd1876a into puppetlabs:master Apr 8, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@actowery actowery actowery approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@shaigy @actowery