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.

Sign up for GitHub

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

Jump to bottom

Update the file.jl tests to allow for both EPERM and EINVAL in the non-root CHOWN tests #41682

Merged
merged 1 commit into from
Jul 23, 2021
Merged

Update the file.jl tests to allow for both EPERM and EINVAL in the non-root CHOWN tests #41682

merged 1 commit into from
Jul 23, 2021

Conversation

DilumAluthge
Copy link
Member

@DilumAluthge DilumAluthge commented Jul 22, 2021 •
edited
Loading

See #41541 (comment) and #41541 (comment) for context.

@DilumAluthge @staticfloat
Update the file.jl tests to allow for both EPERM and EINVAL in …
5249635 
…the non-root CHOWN tests

Co-authored-by: Elliot Saba <staticfloat@gmail.com>
@DilumAluthge DilumAluthge added test This change adds or pertains to unit tests backport 1.6 Change should be backported to release-1.6 backport 1.7 labels Jul 22, 2021
@DilumAluthge
Copy link
Member Author

#41541 is marked for backport to 1.6 and 1.7, and this PR is necessary for #41541. Therefore, I am marking this PR for backport to 1.6 and 1.7.

@DilumAluthge DilumAluthge mentioned this pull request Jul 22, 2021
Merged
4 tasks
@DilumAluthge DilumAluthge added the status:merge me PR is reviewed. Merge when all tests are passing label Jul 22, 2021
@staticfloat staticfloat merged commit 114ee17 into master Jul 23, 2021
@staticfloat staticfloat deleted the dpa/chown_uid_test_einval_eperm branch July 23, 2021 00:21
@DilumAluthge DilumAluthge removed the status:merge me PR is reviewed. Merge when all tests are passing label Jul 23, 2021
KristofferC pushed a commit that referenced this pull request Jul 26, 2021
@DilumAluthge @KristofferC
Update the file.jl tests to allow for both EPERM and EINVAL in …
e5267fa 
…the non-root CHOWN tests (#41682)

Co-authored-by: Elliot Saba <staticfloat@gmail.com>

Co-authored-by: Elliot Saba <staticfloat@gmail.com>
(cherry picked from commit 114ee17)
@KristofferC KristofferC mentioned this pull request Jul 26, 2021
Merged
82 tasks
KristofferC pushed a commit that referenced this pull request Jul 26, 2021
@DilumAluthge @KristofferC
Update the file.jl tests to allow for both EPERM and EINVAL in …
45afb6b 
…the non-root CHOWN tests (#41682)

Co-authored-by: Elliot Saba <staticfloat@gmail.com>

Co-authored-by: Elliot Saba <staticfloat@gmail.com>
(cherry picked from commit 114ee17)
@KristofferC KristofferC mentioned this pull request Jul 26, 2021
Merged
75 tasks
@KristofferC KristofferC mentioned this pull request Aug 25, 2021
Merged
63 tasks
KristofferC pushed a commit that referenced this pull request Aug 31, 2021
@DilumAluthge @KristofferC
Update the file.jl tests to allow for both EPERM and EINVAL in …
cd97c4d 
…the non-root CHOWN tests (#41682)

Co-authored-by: Elliot Saba <staticfloat@gmail.com>

Co-authored-by: Elliot Saba <staticfloat@gmail.com>
(cherry picked from commit 114ee17)
KristofferC pushed a commit that referenced this pull request Sep 3, 2021
@DilumAluthge @KristofferC
Update the file.jl tests to allow for both EPERM and EINVAL in …
3e8eb9e 
…the non-root CHOWN tests (#41682)

Co-authored-by: Elliot Saba <staticfloat@gmail.com>

Co-authored-by: Elliot Saba <staticfloat@gmail.com>
(cherry picked from commit 114ee17)
@KristofferC KristofferC removed the backport 1.6 Change should be backported to release-1.6 label Sep 7, 2021
@DilumAluthge DilumAluthge mentioned this pull request Oct 6, 2021
Merged
staticfloat pushed a commit that referenced this pull request Dec 23, 2022
@DilumAluthge @staticfloat
Update the file.jl tests to allow for both EPERM and EINVAL in …
f64e12c 
…the non-root CHOWN tests (#41682)

Co-authored-by: Elliot Saba <staticfloat@gmail.com>

Co-authored-by: Elliot Saba <staticfloat@gmail.com>
(cherry picked from commit 114ee17)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@staticfloat staticfloat staticfloat approved these changes

@Keno Keno Awaiting requested review from Keno

Assignees
No one assigned
Labels
test This change adds or pertains to unit tests
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@DilumAluthge @staticfloat @KristofferC