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

Config split ignored during config-import on Acquia cloud #1891

Closed
danepowell opened this issue Aug 7, 2017 · 4 comments
Closed

Config split ignored during config-import on Acquia cloud #1891

danepowell opened this issue Aug 7, 2017 · 4 comments
Labels
Bug Something isn't working

Comments

@danepowell
Copy link
Contributor

My system information:

  • BLT version: 8.9.0

When I deploy code for BLT 8.9.0 sites to Acquia Cloud, it imports configuration as part of Cloud post-deploy/post-update hooks. Normally this config import applies config splits as well, but as of 8.9.0 this is no longer the case.

BLT is essentially running this command:
/var/www/html/foo.dev/vendor/bin/drush config-import vcs

When I run that command manually, I can verify that it does not properly include config splits.

If I simply change that to use the sync key instead of vcs, it does take into account config splits:
/var/www/html/foo.dev/vendor/bin/drush config-import sync

This makes sense, since Config Split (or Config Filter really) only supports using the default sync directory: https://www.drupal.org/node/2865912#comment-12020955

The only thing that doesn't quite make sense to me is why this only changed recently. I'm assuming BLT versions prior to 8.9 used the sync directory during deployments.

@grasmash
Copy link
Contributor

grasmash commented Aug 7, 2017

@danepowell A bug was recently fixed in BLT, it formerly caused the sync key to be used everywhere. Ironically, that bug appears to have been responsible for making config split work.

@grasmash grasmash added the Bug Something isn't working label Aug 7, 2017
@pixlkat
Copy link
Contributor

pixlkat commented Aug 7, 2017

This was the problem I had when I updated to 8.9.0 -- I worked around it by adding these lines in the cm section of my /blt/project.yml file

cm:
  strategy: config-split
  core:
    deploy-key: sync

grasmash added a commit to grasmash/bolt that referenced this issue Aug 8, 2017
@danepowell
Copy link
Contributor Author

That commit worked, thanks!

@alex-moreno
Copy link
Contributor

alex-moreno commented Aug 16, 2017

I was having the same issue in Acquia Cloud. deploy-key: sync trick fixed the issue (although right approach would be to update the branch)

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

No branches or pull requests

4 participants