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

Investigate windows Permissions Error on Jenkins Windows Unit tests #21449

Closed
damccorm opened this issue Jun 4, 2022 · 2 comments · Fixed by #23044
Closed

Investigate windows Permissions Error on Jenkins Windows Unit tests #21449

damccorm opened this issue Jun 4, 2022 · 2 comments · Fixed by #23044
Labels
core done & done Issue has been reviewed after it was closed for verification, followups, etc. P2 python run-inference sub-task

Comments

@damccorm
Copy link
Contributor

damccorm commented Jun 4, 2022

Skip tests that require temporary file open on windows. For some reason Jenkins on windows gives a permission error with tmp files.
 
apache_beam\ml\inference\sklearn_loader.py:68: in load_model
150 file = FileSystems.open(self._model_uri, 'rb')
151apache_beam\io\filesystems.py:244: in open
152 return filesystem.open(path, mime_type, compression_type)
153apache_beam\io\localfilesystem.py:181: in open
154 return self.path_open(path, 'rb', mime_type, compression_type)
155
_ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _
156
157 def _path_open(
158 self,
159 path,
160 mode,
161 mime_type='application/octet-stream',
162 compression_type=CompressionTypes.AUTO):
163 """Helper functions to open a file in the provided mode.
164 """
165 compression_type = FileSystem._get_compression_type(path, compression_type)
166> raw_file = io.open(path, mode)
167E PermissionError: [Errno 13] Permission denied: 'D:\a\beam\beam\sdks\python\target\.tox\py38-win\tmp\tmp6zl6x438'
168

Imported from Jira BEAM-14359. Original Jira may contain additional context.
Reported by: Ryan.Thompson.
Subtask of issue #21435

@damccorm
Copy link
Contributor Author

damccorm commented Sep 2, 2022

@ryanthompson591 is there still work to be done on this one? If so, which test suite is it occurring in?

@ryanthompson591
Copy link
Contributor

I haven't investigated this deeply.

The disabled tests are in this file
https://github.com/apache/beam/blob/master/sdks/python/apache_beam/ml/inference/sklearn_inference_test.py

For example

'https://github.com/apache/beam/issues/21449')

@damccorm damccorm added the done & done Issue has been reviewed after it was closed for verification, followups, etc. label Sep 8, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
core done & done Issue has been reviewed after it was closed for verification, followups, etc. P2 python run-inference sub-task
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants