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

file.managed makes minion hang if drive doesn't exist on Windows #9880

Closed
otrempe opened this issue Jan 22, 2014 · 2 comments · Fixed by #11738
Closed

file.managed makes minion hang if drive doesn't exist on Windows #9880

otrempe opened this issue Jan 22, 2014 · 2 comments · Fixed by #11738
Assignees
Labels
Bug broken, incorrect, or confusing behavior Platform Relates to OS, containers, platform-based utilities like FS, system based apps severity-low 4th level, cosemtic problems, work around exists Windows

Comments

@otrempe
Copy link
Contributor

otrempe commented Jan 22, 2014

Executing the state below on a Windows minion where the D: drive doesn't exist makes the minion process hang. The memory consumed by the process rises indefinitely and the process never returns. It should instead return an error saying this drive doesn't exist. Observed on 0.17.2.

my_file:
    file.managed:
        - name: D:/MyFolder/my_file.txt
@basepi
Copy link
Contributor

basepi commented Jan 22, 2014

Very strange. This may be a Python problem, as opposed to a Salt problem, as I would expect an IOError quite promptly in that case. We'll look into this.

@ghost ghost assigned UtahDave Jan 23, 2014
@UtahDave
Copy link
Contributor

UtahDave commented Apr 1, 2014

I can confirm that this behavior is still happening.

@jfindlay jfindlay added Platform Relates to OS, containers, platform-based utilities like FS, system based apps and removed Platform Relates to OS, containers, platform-based utilities like FS, system based apps labels May 26, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug broken, incorrect, or confusing behavior Platform Relates to OS, containers, platform-based utilities like FS, system based apps severity-low 4th level, cosemtic problems, work around exists Windows
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants