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

detect postgres 10 on centos properly #3833

Merged
merged 2 commits into from Feb 27, 2019

Conversation

Projects
None yet
4 participants
@chris-rock
Copy link
Collaborator

commented Feb 25, 2019

$ inspec shell -t ssh://vagrant@127.0.0.1:2222 --sudo
Welcome to the interactive InSpec Shell
To find out how to use it, type: help

You are currently running on:

    Name:      centos
    Families:  redhat, linux, unix, os
    Release:   7.4.1708
    Arch:      x86_64

inspec> command("psql --version").stdout.strip
=> "psql (PostgreSQL) 10.7"
inspec> command("ls '/var/lib/pgsql'").stdout
=> "10\n"
inspec> 

This work is related to dev-sec/postgres-baseline#31 for additional postgres 10 support.

Signed-off-by: Christoph Hartmann chris@lollyrock.com

detect postgres 10 on centos properly
Signed-off-by: Christoph Hartmann <chris@lollyrock.com>
@clintoncwolfe

This comment has been minimized.

Copy link
Contributor

commented Feb 25, 2019

Closes #3783

make rubocop happy
Signed-off-by: Christoph Hartmann <chris@lollyrock.com>

@chris-rock chris-rock force-pushed the chris-rock:chris-rock/postgres branch from 22dcc55 to afadc89 Feb 26, 2019

@miah

miah approved these changes Feb 27, 2019

Copy link
Contributor

left a comment

Thanks @chris-rock

@clintoncwolfe
Copy link
Contributor

left a comment

Thanks!

@clintoncwolfe clintoncwolfe merged commit c23d828 into inspec:master Feb 27, 2019

3 checks passed

DCO This commit has a DCO Signed-off-by
Details
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@eloyekunle

This comment has been minimized.

Copy link
Contributor

commented Feb 27, 2019

Thanks @chris-rock

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.