-
Notifications
You must be signed in to change notification settings - Fork 683
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
RegExp in processes resource can't match long-run process #1332
Comments
Wing924
pushed a commit
to Wing924/inspec
that referenced
this issue
Nov 29, 2016
Wing924
pushed a commit
to Wing924/inspec
that referenced
this issue
Nov 29, 2016
…spec#1332 Signed-off-by: Wei, He <weihe924stephen@gmail.com>
Wing924
pushed a commit
to Wing924/inspec
that referenced
this issue
Nov 29, 2016
…spec#1332 Signed-off-by: Wei, He <weihe924stephen@gmail.com>
CI test failed on Oracle-linux installing openssh-server phase :( |
Wing924
pushed a commit
to Wing924/inspec
that referenced
this issue
Nov 30, 2016
Signed-off-by: Wei, He <weihe924stephen@gmail.com>
Wing924
pushed a commit
to Wing924/inspec
that referenced
this issue
Nov 30, 2016
Signed-off-by: Wei, He <weihe924stephen@gmail.com>
Wing924
pushed a commit
to Wing924/inspec
that referenced
this issue
Nov 30, 2016
Signed-off-by: Wei, He <weihe924stephen@gmail.com>
Wing924
pushed a commit
to Wing924/inspec
that referenced
this issue
Nov 30, 2016
Signed-off-by: Wei, He <weihe924stephen@gmail.com>
chris-rock
added a commit
that referenced
this issue
Nov 30, 2016
fix bug: RegExp in processes resource can't match long-run process #1332
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Description
It use
([^ ]+)
to match each ps columns in file: lib/resources/processes.rb.But as man page of ps says, start format may be mmm dd or HH:MM:SS.
Because
mmm dd
comtains space, it can't be match with the RegExp.man of ps
InSpec and Platform Version
Tested against 1.5 and 1.6
Replication Case
It will output
Possible Solutions
Change 9th
([^ ]+)
to(\w{3} \d{2}|\d{2}:\d{2}:\d{2})
The text was updated successfully, but these errors were encountered: