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

Made cloud hooks executable. #648

Merged
merged 1 commit into from
Nov 10, 2016
Merged

Conversation

danepowell
Copy link
Contributor

@danepowell danepowell commented Nov 10, 2016

Sorry I missed this. Cloud hooks won't run unless they are executable.

@grasmash grasmash added in progress Bug Something isn't working labels Nov 10, 2016
@grasmash grasmash merged commit 0be4f52 into acquia:8.x Nov 10, 2016
dreamfony pushed a commit to dreamfony/blt that referenced this pull request Nov 11, 2016
@seanhamlin
Copy link

seanhamlin commented Nov 14, 2016

That explains this error:

22: exec: /var/www/html/sitename.01test/hooks/common/post-code-deploy/post-code-deploy.sh: Permission denied[2016-11-14 08:10:07] 

Also, these fire for ACSF, is this expected? Seeing as the only thing it does is try to revert configuration using VCS, and this is not really supported at the moment, I guess this is another issue.

@grasmash
Copy link
Contributor

I suspect that we do not want this on ACSF.

@danepowell
Copy link
Contributor Author

To my knowledge, cloud hooks are not supported on ACSF.

@seanhamlin
Copy link

@danepowell well the hooks do fire on ACSF, it is just that the environments are named differently (e.g. 01live, 01test), the common hooks fire. I would imagine it would make sense to verify you are running in ACE prior to invoking that blt script.

@kevinquillen
Copy link

Small note, I think this is the issue I just ran into. I generated a vanilla project, built it, deployed to Acquia Cloud. When switching to the build branch, it errored out because the hooks could not be executed. Support replied with:

chmod a+x hooks/common/post-code-deploy/post-code-deploy.sh
chmod a+x hooks/common/post-code-update/post-code-update.sh
git add hooks/common/post-code-deploy/post-code-deploy.sh
git add hooks/common/post-code-update/post-code-update.sh
git commit -m "Make the post-code hooks executable."
git push

@danepowell danepowell deleted the feature/hook-exec branch December 6, 2017 00:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants