Add label user to process collector

[aburtasov]

Added label "user" to process collector. Using WIN32_LoggedOnUser and Win32_SessionProcess

Example metric with new label:

HELP windows_process_cpu_time_total Returns elapsed time that all of the threads of this process used the processor to execute instructions by mode (privileged, user).
TYPE windows_process_cpu_time_total counter
windows_process_cpu_time_total{creating_process_id="12244",mode="privileged",process="powershell",process_id="17512",user="MYDOMAIN\aburtasov"} 0.296875
windows_process_cpu_time_total{creating_process_id="12244",mode="privileged",process="powershell",process_id="19688",user="MYDOMAIN\aburtasov"} 14.296875
windows_process_cpu_time_total{creating_process_id="12244",mode="privileged",process="powershell",process_id="25612",user="MYDOMAIN\aburtasov"} 0.484375

[jkroepke]

I tested this PR on my local system and it works.

However, I see that the collector is 10 times slower compared to master branch.

master:
windows_exporter_collector_duration_seconds{collector="process"} 0.0121184

PR:
windows_exporter_collector_duration_seconds{collector="process"} 0.1610989

Thats to much. I have provide some suggestions, I hope they help. After apply the suggestion, verify locally, if everything still works as expected.

[jkroepke]

I tested this PR on my local system and it works.

However, I see that the collector is 10 times slower compared to master branch.

master:
windows_exporter_collector_duration_seconds{collector="process"} 0.0121184

PR:
windows_exporter_collector_duration_seconds{collector="process"} 0.1610989

Thats to much. I have provide some suggestions, I hope they help. After apply the suggestion, verify locally, if everything still works as expected.

[aburtasov]

@jkroepke After the changes, it still works, but I'm not sure it improved the speed

[jkroepke]

Hey @aburtasov

I looked into it and in term of performance, it overhead compared to the first version is low now.

However, I would also looked into it, if I can avoid the WMI query. Thats something, what I will looked into it.

[jkroepke]

Please take a look, if it still works for you.

[aburtasov]

@jkroepke Hi!

I tried the new version with your latest commit. There is no "user" label. Do I understand correctly, you want to optimize the process of adding the "user" label?

[jkroepke]

Hey @aburtasov

the feature is now optional, behind a toggle --collector.process.report-owner

[aburtasov]

@jkroepke Good job. Everything works correctly.