-
Notifications
You must be signed in to change notification settings - Fork 396
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
BLT post-code-deploy hook failing on ACSF #2673
Comments
There are two issues:
BLT provides post-code-update hooks (as opposed to post-code-deploy) for default ACSF environments 01dev, 01test but not custom environments like |
I'm not sure I understand what the proposed resolution is. We're just using the same default hooks that were created for the previous version of the BLT project. Are they still needed in 9.x (and must be moved) or should they be removed entirely? Also, this custom environment is a D8 stack https://docs.acquia.com/site-factory/tiers/stacks so I feel it is reasonable to expect ACSF and BLT to fully support it. |
@malcolmp I created #2676 to track your feature request. I should have clarified in my response that by "custom environment" I meant environments that are not the default 01dev, 01test, or 01live on ACSF which in your case would include Stacks. If you would like to use post-code-update hooks immediately to resolve your error message you can follow the recommendations above which I have pasted below to clarify that it is an action item:
|
So I've commented out the code from the post-code-deploy hook, however the deployment is still failing.
It seems like it's failing on the existing 8.x-1.x-dev branch while I'm deploying the feature/8.4.x-dev branch so removing the deploy_updates function isn't working. Can you think of any reason that would be an issue? |
@malcolmp I put in #2682 which should resolve your issue and also adds some additional ACSF backports from BLT 9.x. You can patch your version of BLT with the PR or update to the latest version once it is merged and then run |
@lcatlett I merged in the PR and ran both commands but am still getting the below error attempting to deploy our new branch.
|
With the patch from #2685 I am able to apply to do a fresh deployment to our 02test environment but still getting the below error on 02dev.
|
@lcatlett is following up on this issue in an Acquia support ticket. It seems to be a project-specific issue. |
BLT version: 8.9.14
We've been referred to report a BLT issue we're encountering on D8 an Acquia Sitefactory deployment. As a result of this post-deploy hook the ACSF deployment is failing.
This is the portion of the stacktrace that is failing in BLT:
Full error log output:
The text was updated successfully, but these errors were encountered: