Parse OSX process details correctly #64
Open
Conversation
This file contains 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
|
@neekey have you had time to look at this yet? |
|
Hi @milankinen thanks for your PR. Your way of parsing the output is quite the same as the original version of this module, which works in this case but will fail in other cases as well, so I won't accept this as a solution, though it works in your case. you can take a look at this discussion neekey/table-parser#3 (comment), there are more cases like the discussion mentioned. Actually table-parser has done its best already (since it tries to catch as much edge cases as it can), but it can't work well if the input's format is very hard to predict, there is a boundary needs to be set for table-parser's ability. So instead of improving table-parser itself, I am thinking if there's any way to regulate the output format of |
|
yes, I think I can definitely narrow down the metrics to only |
I coudn't find those cases (not even from the table parser thread you posted). Could you provide a link to them?
Yes that's definitely the right way, although it'd introduce non-backwards compatible API changes. The purpose of this PR is to fix the parsing without breaking the backwards compatibility so that's why I thought it could be the first step. |
|
Hello, when will this commit be merged into the release branch? I am dealing with an error and this is the solution needed.. I am currently using this commit in order to work now. would appreciate it if we could get in though.. |
|
I believe you could run the |
Empact
added a commit
to Empact/zap-desktop
that referenced
this pull request
Dec 24, 2017
On mac, the ps output can be misaligned, e.g. when some VSZ field is
longer than expected.
The default ps-node lookup is via ps lx, which includes:
-l Display information associated with the following keywords:
uid, pid, ppid, flags, cpu, pri, nice, vsz=SZ, rss, wchan,
state=S, paddr=ADDR, tty, time, and command=CMD.
Many of these fields are unnecessary because we're looking up by name only,
so we can lookup using only the default fields of ps.
neekey/table-parser#11
neekey/ps#64
This was referenced Dec 24, 2017
Closed
Empact
added a commit
to Empact/zap-desktop
that referenced
this pull request
Dec 25, 2017
`ps x -o pid,command` rather than `ps lx`
The default ps-node lookup is via ps lx, which includes:
-l Display information associated with the following keywords:
uid, pid, ppid, flags, cpu, pri, nice, vsz=SZ, rss, wchan,
state=S, paddr=ADDR, tty, time, and command=CMD.
On mac, the `ps l` output can be misaligned, e.g. when some VSZ field is
longer than expected.
Many of these fields are unnecessary because we're looking up by name only,
so we can lookup using only the command fields of ps.
https://gist.github.com/ivankovacevic/9918272
More on the issue with ps-node:
neekey/table-parser#11
neekey/ps#64
Empact
added a commit
to Empact/zap-desktop
that referenced
this pull request
Dec 25, 2017
`ps x -o pid,command` rather than `ps lx`
The default ps-node lookup is via ps lx, which includes:
-l Display information associated with the following keywords:
uid, pid, ppid, flags, cpu, pri, nice, vsz=SZ, rss, wchan,
state=S, paddr=ADDR, tty, time, and command=CMD.
On mac, the `ps l` output can be misaligned, e.g. when some VSZ field is
longer than expected.
Many of these fields are unnecessary because we're looking up by name only,
so we can lookup using only the command fields of ps.
https://gist.github.com/ivankovacevic/9918272
More on the issue with ps-node:
neekey/table-parser#11
neekey/ps#64
Empact
added a commit
to Empact/zap-desktop
that referenced
this pull request
Dec 26, 2017
`ps x -o pid,command` rather than `ps lx`
The default ps-node lookup is via ps lx, which includes:
-l Display information associated with the following keywords:
uid, pid, ppid, flags, cpu, pri, nice, vsz=SZ, rss, wchan,
state=S, paddr=ADDR, tty, time, and command=CMD.
On mac, the `ps l` output can be misaligned, e.g. when some VSZ field is
longer than expected.
Many of these fields are unnecessary because we're looking up by name only,
so we can lookup using only the command fields of ps.
https://gist.github.com/ivankovacevic/9918272
More on the issue with ps-node:
neekey/table-parser#11
neekey/ps#64
Empact
added a commit
to Empact/zap-desktop
that referenced
this pull request
Dec 29, 2017
`ps x -o pid,command` rather than `ps lx`
The default ps-node lookup is via ps lx, which includes:
-l Display information associated with the following keywords:
uid, pid, ppid, flags, cpu, pri, nice, vsz=SZ, rss, wchan,
state=S, paddr=ADDR, tty, time, and command=CMD.
On mac, the `ps l` output can be misaligned, e.g. when some VSZ field is
longer than expected.
Many of these fields are unnecessary because we're looking up by name only,
so we can lookup using only the command fields of ps.
https://gist.github.com/ivankovacevic/9918272
More on the issue with ps-node:
neekey/table-parser#11
neekey/ps#64
Empact
added a commit
to Empact/zap-desktop
that referenced
this pull request
Jan 2, 2018
`ps x -o pid,command` rather than `ps lx`
The default ps-node lookup is via ps lx, which includes:
-l Display information associated with the following keywords:
uid, pid, ppid, flags, cpu, pri, nice, vsz=SZ, rss, wchan,
state=S, paddr=ADDR, tty, time, and command=CMD.
On mac, the `ps l` output can be misaligned, e.g. when some VSZ field is
longer than expected.
Many of these fields are unnecessary because we're looking up by name only,
so we can lookup using only the command fields of ps.
https://gist.github.com/ivankovacevic/9918272
More on the issue with ps-node:
neekey/table-parser#11
neekey/ps#64
Empact
added a commit
to Empact/zap-desktop
that referenced
this pull request
Jan 5, 2018
`ps x -o pid,command` rather than `ps lx`
The default ps-node lookup is via ps lx, which includes:
-l Display information associated with the following keywords:
uid, pid, ppid, flags, cpu, pri, nice, vsz=SZ, rss, wchan,
state=S, paddr=ADDR, tty, time, and command=CMD.
On mac, the `ps l` output can be misaligned, e.g. when some VSZ field is
longer than expected.
Many of these fields are unnecessary because we're looking up by name only,
so we can lookup using only the command fields of ps.
https://gist.github.com/ivankovacevic/9918272
More on the issue with ps-node:
neekey/table-parser#11
neekey/ps#64
Empact
added a commit
to Empact/zap-desktop
that referenced
this pull request
Jan 10, 2018
`ps x -o pid,command` rather than `ps lx`
The default ps-node lookup is via ps lx, which includes:
-l Display information associated with the following keywords:
uid, pid, ppid, flags, cpu, pri, nice, vsz=SZ, rss, wchan,
state=S, paddr=ADDR, tty, time, and command=CMD.
On mac, the `ps l` output can be misaligned, e.g. when some VSZ field is
longer than expected.
Many of these fields are unnecessary because we're looking up by name only,
so we can lookup using only the command fields of ps.
https://gist.github.com/ivankovacevic/9918272
More on the issue with ps-node:
neekey/table-parser#11
neekey/ps#64
Empact
added a commit
to Empact/zap-desktop
that referenced
this pull request
Jan 20, 2018
`ps x -o pid,command` rather than `ps lx`
The default ps-node lookup is via ps lx, which includes:
-l Display information associated with the following keywords:
uid, pid, ppid, flags, cpu, pri, nice, vsz=SZ, rss, wchan,
state=S, paddr=ADDR, tty, time, and command=CMD.
On mac, the `ps l` output can be misaligned, e.g. when some VSZ field is
longer than expected.
Many of these fields are unnecessary because we're looking up by name only,
so we can lookup using only the command fields of ps.
https://gist.github.com/ivankovacevic/9918272
More on the issue with ps-node:
neekey/table-parser#11
neekey/ps#64
Empact
added a commit
to Empact/zap-desktop
that referenced
this pull request
Jan 22, 2018
`ps x -o pid,command` rather than `ps lx`
The default ps-node lookup is via ps lx, which includes:
-l Display information associated with the following keywords:
uid, pid, ppid, flags, cpu, pri, nice, vsz=SZ, rss, wchan,
state=S, paddr=ADDR, tty, time, and command=CMD.
On mac, the `ps l` output can be misaligned, e.g. when some VSZ field is
longer than expected.
Many of these fields are unnecessary because we're looking up by name only,
so we can lookup using only the command fields of ps.
https://gist.github.com/ivankovacevic/9918272
More on the issue with ps-node:
neekey/table-parser#11
neekey/ps#64
Empact
added a commit
to Empact/zap-desktop
that referenced
this pull request
Mar 17, 2018
`ps x -o pid,command` rather than `ps lx`
The default ps-node lookup is via ps lx, which includes:
-l Display information associated with the following keywords:
uid, pid, ppid, flags, cpu, pri, nice, vsz=SZ, rss, wchan,
state=S, paddr=ADDR, tty, time, and command=CMD.
On mac, the `ps l` output can be misaligned, e.g. when some VSZ field is
longer than expected.
Many of these fields are unnecessary because we're looking up by name only,
so we can lookup using only the command fields of ps.
https://gist.github.com/ivankovacevic/9918272
More on the issue with ps-node:
neekey/table-parser#11
neekey/ps#64
Empact
added a commit
to Empact/zap-desktop
that referenced
this pull request
Jun 6, 2018
`ps x -o pid,command` rather than `ps lx`
The default ps-node lookup is via ps lx, which includes:
-l Display information associated with the following keywords:
uid, pid, ppid, flags, cpu, pri, nice, vsz=SZ, rss, wchan,
state=S, paddr=ADDR, tty, time, and command=CMD.
On mac, the `ps l` output can be misaligned, e.g. when some VSZ field is
longer than expected.
Many of these fields are unnecessary because we're looking up by name only,
so we can lookup using only the command fields of ps.
https://gist.github.com/ivankovacevic/9918272
More on the issue with ps-node:
neekey/table-parser#11
neekey/ps#64
Empact
added a commit
to Empact/zap-desktop
that referenced
this pull request
Jun 15, 2018
`ps x -o pid,command` rather than `ps lx`
The default ps-node lookup is via ps lx, which includes:
-l Display information associated with the following keywords:
uid, pid, ppid, flags, cpu, pri, nice, vsz=SZ, rss, wchan,
state=S, paddr=ADDR, tty, time, and command=CMD.
On mac, the `ps l` output can be misaligned, e.g. when some VSZ field is
longer than expected.
Many of these fields are unnecessary because we're looking up by name only,
so we can lookup using only the command fields of ps.
https://gist.github.com/ivankovacevic/9918272
More on the issue with ps-node:
neekey/table-parser#11
neekey/ps#64
Empact
added a commit
to Empact/zap-desktop
that referenced
this pull request
Aug 17, 2018
`ps x -o pid,command` rather than `ps lx`
The default ps-node lookup is via ps lx, which includes:
-l Display information associated with the following keywords:
uid, pid, ppid, flags, cpu, pri, nice, vsz=SZ, rss, wchan,
state=S, paddr=ADDR, tty, time, and command=CMD.
On mac, the `ps l` output can be misaligned, e.g. when some VSZ field is
longer than expected.
Many of these fields are unnecessary because we're looking up by name only,
so we can lookup using only the command fields of ps.
https://gist.github.com/ivankovacevic/9918272
More on the issue with ps-node:
neekey/table-parser#11
neekey/ps#64
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Hello!
Here is a fix for #62.
TableParserwas a bit too generic so I tailored the parsing logic forpsin order to improve robustness. I added a test case based on the given example output from #62.I hope we can get this fix released asap. Our testing pipeline needs this fix because
testcafe(which depends on this package) can't parse browser PID correctly, hence leaving the browser process open after every test run. 😕