Skip to content

Conversation

kalpaitch
Copy link
Contributor

@kalpaitch kalpaitch commented Mar 29, 2017

The main reason for this proposal is to allow console commands to be run on all environments hence:

  1. Replacement of hardcoded root dir with a configurable placeholder, this does provide an extra interactive dialogue but is perhaps an improvement on the current implementation. And allows us to further improve this process later.
  2. More granular division of commands with an emphasis on splitting out those commands we would not want to run on a live production environment.

In summary the rebuild is now split into two commands, site:rebuild:fresh for dev environments and new build production sites which:

# Compile the packages (composer, npm, grunt).
# Generate the settings (db, memcache, cdn etc).
# Imports db or installs a site.
# Build the site based on existing setup (drush commands).

and site:rebuild:prod for existing prod deployments which:

# Compile the packages (composer, npm, grunt).
# Build the site based on existing setup (drush commands).

An alternative options seems to be to do away with exec bash commands and run everything as a new console command that is able to pick up the 'root' from the site.yml file, this might be better but might also take away a bit of the flexibility in quickly adding new steps to the chain...

@kalpaitch
Copy link
Contributor Author

Seem to be getting this error at the moment so not sure I can really test this - hechoendrupal/drupal-console#3100

@davidgrayston davidgrayston changed the title Proposal for console command changes, removal of hardcoded values. 27689 Proposal for console command changes, removal of hardcoded values. Mar 30, 2017
@guardiola86 guardiola86 merged commit e12739e into master Apr 19, 2017
@marcelovani marcelovani deleted the proposal_console_chains branch July 6, 2017 22:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants