-
Notifications
You must be signed in to change notification settings - Fork 24
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
Pantheon aliases disappear when paliases fails #187
Comments
That's unexpected. If it's not able to fetch the aliases nothing should be written: https://github.com/pantheon-systems/terminus/blob/master/terminus.drush.inc#L3628 However, we might be able to guard against that further by not initializing any file handler until aliases are successfully fetched. This would require verifying alias writability differently here: https://github.com/pantheon-systems/terminus/blob/master/terminus.drush.inc#L3593 |
paliases has been failing more often than not lately, and it still truncates the file every time. |
I can confirm this is happening as well: vagrant@kala:/etc/drush$ drush ta
Unable to fetch the aliases. Try again? [error]
vagrant@kala:/etc/drush$ ls -la
total 40
drwxrwxrwx 2 root root 4096 Nov 20 16:38 .
drwxr-xr-x 90 root root 4096 Nov 20 16:31 ..
-rwxrwxrwx 1 root vagrant 15621 Oct 12 23:41 drushrc.php
-rw-rw-r-- 1 vagrant vagrant 9968 Nov 20 11:58 kalastack.aliases.drushrc.php
-rw-r----- 1 vagrant vagrant 0 Nov 20 16:37 pantheon.aliases.drushrc.php
-rw-rw-r-- 1 vagrant vagrant 47 Nov 20 16:38 terminatur.aliases.drushrc.php |
Hey guys, We can try escalating this to guard against the file being wiped by a failed request. Sorry for the trouble. |
I am experiencing the same issue. I also had trouble getting aliases in the past and the only thing that helps is waiting. Just now I had to try for almost an hour. |
I think the answer here is probably to write to a temporary resource first and then mv to the destination after the fclose. |
Sometimes when running drush paliases, I get this error:
When that happens, the pantheon.aliases.drushrc.php is truncated:
So all of the aliases disappear.
The text was updated successfully, but these errors were encountered: