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

Executable scripts no longer have executable permissions after upload #745

Open
maxwang7 opened this issue May 3, 2017 · 5 comments
Open

Executable scripts no longer have executable permissions after upload #745

maxwang7 opened this issue May 3, 2017 · 5 comments
Labels
backend bug p3 Do it some day.

Comments

@maxwang7
Copy link
Contributor

maxwang7 commented May 3, 2017 •
edited
Loading

No description provided.

@maxwang7 maxwang7 self-assigned this May 3, 2017
@fabeschan fabeschan mentioned this issue May 8, 2017
Closed
@maxwang7
Copy link
Contributor Author

maxwang7 commented Jun 3, 2017

Even though I have a script with executable permissions, I'm still not getting the appropriate permissions.

https://worksheets.codalab.org/worksheets/0xe4b0468ed9d94509b6e0ac5bef2abd9a/

@maxwang7
Copy link
Contributor Author

I've spoken with Stephen about these changes and know the approach to take

@maxwang7
Copy link
Contributor Author

Use tar everything (flatten even single files)

@maxwang7
Copy link
Contributor Author

maxwang7 commented Oct 9, 2017

This PR is able to tar, upload, and extract single files, but I'm still unable to run the executable. More detailed discussion in here: #795

@maxwang7
Copy link
Contributor Author

I've tried changing the docker code to stop making Docker volumes read only, but the behavior is still the same. Next hypothesis: something about how we're placing dependencies on docker volumes is breaking

@percyliang percyliang assigned fabeschan and unassigned maxwang7 Feb 2, 2018
@percyliang percyliang assigned bkgoksel and unassigned fabeschan Feb 16, 2018
@percyliang percyliang mentioned this issue Apr 5, 2020
Closed
OrthoDex added a commit that referenced this issue Apr 26, 2020
@OrthoDex
test: add regression
d9f4074 
   - This PR fixes #745
@OrthoDex OrthoDex mentioned this issue Apr 26, 2020
Closed
@percyliang percyliang added the p2 Do it this quarter. label Aug 10, 2020
@epicfaace epicfaace added p3 Do it some day. and removed p2 Do it this quarter. labels Jun 18, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
backend bug p3 Do it some day.
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix file exec permissions changing on upload codalab/codalab-worksheets
7 participants
@percyliang @epicfaace @fabeschan @maxwang7 @bkgoksel @OrthoDex @teetone