Skip to content

[Linux] Ignore provider-generated events#1042

Merged
chrisd8088 merged 2 commits intomicrosoft:features/linuxprototypefrom
github:linux-projfs-pid-check
May 1, 2019
Merged

[Linux] Ignore provider-generated events#1042
chrisd8088 merged 2 commits intomicrosoft:features/linuxprototypefrom
github:linux-projfs-pid-check

Conversation

@chrisd8088
Copy link
Contributor

@chrisd8088 chrisd8088 commented Apr 16, 2019

We want to ignore events triggered by the provider process itself, and we also should handle the case where we can't map a pid to a process command line because the process has already exited.

/cc @kivikakk

Copy link
Contributor

@kivikakk kivikakk left a comment

Choose a reason for hiding this comment

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

LGTM!

@chrisd8088 chrisd8088 force-pushed the features/linuxprototype branch from 3e25110 to 6a487f5 Compare April 18, 2019 17:49
chrisd8088 and others added 2 commits April 18, 2019 10:56
Also handle missing pid files when looking for a process's command
line string; in this case, we can only report an empty string.
Co-Authored-By: chrisd8088 <chrisd@apache.org>
@chrisd8088 chrisd8088 force-pushed the linux-projfs-pid-check branch from bf40d52 to 91959e6 Compare April 18, 2019 18:01
@chrisd8088
Copy link
Contributor Author

Updated to pick up latest CI checks.

@kivikakk
Copy link
Contributor

Looks good for merge.

@chrisd8088
Copy link
Contributor Author

@jrbriggs or @wilbaker -- would you have some time to take a look at this PR for the Linux branch?

@chrisd8088
Copy link
Contributor Author

Thanks, @wilbaker!

@chrisd8088 chrisd8088 merged commit ecf3a35 into microsoft:features/linuxprototype May 1, 2019
@chrisd8088 chrisd8088 deleted the linux-projfs-pid-check branch May 1, 2019 16:43
chrisd8088 added a commit that referenced this pull request May 2, 2019
[Linux] Ignore provider-generated events
chrisd8088 added a commit that referenced this pull request May 7, 2019
[Linux] Ignore provider-generated events
chrisd8088 added a commit that referenced this pull request May 8, 2019
[Linux] Ignore provider-generated events
chrisd8088 added a commit that referenced this pull request May 8, 2019
[Linux] Ignore provider-generated events
chrisd8088 added a commit that referenced this pull request May 9, 2019
[Linux] Ignore provider-generated events
chrisd8088 added a commit that referenced this pull request May 11, 2019
[Linux] Ignore provider-generated events
chrisd8088 added a commit that referenced this pull request May 21, 2019
[Linux] Ignore provider-generated events
chrisd8088 added a commit that referenced this pull request May 21, 2019
[Linux] Ignore provider-generated events
kivikakk pushed a commit to github/VFSForGit that referenced this pull request May 24, 2019
chrisd8088 added a commit that referenced this pull request May 24, 2019
[Linux] Ignore provider-generated events
chrisd8088 added a commit that referenced this pull request Jul 20, 2019
[Linux] Ignore provider-generated events
chrisd8088 added a commit that referenced this pull request Jul 26, 2019
[Linux] Ignore provider-generated events
chrisd8088 added a commit that referenced this pull request Jul 28, 2019
[Linux] Ignore provider-generated events
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants