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

ProcessList native resource #167

Merged
merged 11 commits into from
Aug 24, 2023
Merged

ProcessList native resource #167

merged 11 commits into from
Aug 24, 2023

Conversation

anmenaga
Copy link
Contributor

PR Summary

Cross-platform native DSCv3 resource that returns running processes for the current user.
This resource will be used in testing Export functionality, so it is pretty limited at this point.

PR Context

Example on Windows:

C:\DSCv3\bin\debug\process.exe list
{"pid":29016,"name":"Microsoft.SharePoint.exe","cmdline":"\"C:\\Program Files\\Microsoft OneDrive\\23.174.0820.0001\\Microsoft.SharePoint.exe\""}
{"pid":7332,"name":"smartscreen.exe","cmdline":"C:\\Windows\\System32\\smartscreen.exe -Embedding"}
{"pid":27660,"name":"Notepad.exe","cmdline":"\"C:\\Program Files\\WindowsApps\\Microsoft.WindowsNotepad_11.2306.15.0_x64__8wekyb3d8bbwe\\Notepad\\Notepad.exe\" "}
{"pid":10192,"name":"process.exe","cmdline":"\"C:\\DSCv3\\bin\\debug\\process.exe\" list"}
...

Example on Linux:

PS /home/andrew/DSC-3> /home/andrew/DSC-3/bin/debug/process list
{"pid":55140,"name":"pulseaudio","cmdline":"/usr/bin/pulseaudio --daemonize=no --log-target=journal"}
{"pid":55192,"name":"gvfsd-fuse","cmdline":"/usr/libexec/gvfsd-fuse /run/user/1000/gvfs -f -o big_writes"}
{"pid":55320,"name":"gvfsd-metadata","cmdline":"/usr/libexec/gvfsd-metadata"}
{"pid":55341,"name":"pwsh","cmdline":"./PS734/pwsh"}
{"pid":69967,"name":"process","cmdline":"/home/andrew/DSC-3/bin/debug/process list"}
...

@anmenaga anmenaga requested a review from SteveL-MSFT August 24, 2023 01:14
Copy link
Member

@SteveL-MSFT SteveL-MSFT left a comment

Choose a reason for hiding this comment

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

make it pedantic clean and fix manifest

@SteveL-MSFT
Copy link
Member

@anmenaga in the future, you'll need to submit PRs from a fork rather than a branch from this repo

@SteveL-MSFT SteveL-MSFT merged commit 5a1f8b5 into main Aug 24, 2023
@SteveL-MSFT SteveL-MSFT deleted the export_scenario branch August 24, 2023 18:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants