Skip to content

Update site-utils to start specific site containers #1163

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.

Sign up for GitHub

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

Merged
mbtamuli merged 2 commits into from
Aug 20, 2018
Merged

Update site-utils to start specific site containers #1163

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
3c2c3c0
Update site-utils to start specific site conatainers
mrrobot47 Aug 9, 2018
fcfa391
Merge branch 'develop-v4' into update/site-utils-for-specific-contain…
mrrobot47 Aug 20, 2018
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
7 changes: 4 additions & 3 deletions php/site-utils.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -275,17 +275,18 @@ function get_curl_info( $url, $port = 80, $port_info = false ) {
/**
* Function to pull the latest images and bring up the site containers.
*
* @param string $site_root Root directory of the site.
* @param string $site_root Root directory of the site.
* @param array $containers The minimum required conatainers to start the site. Default null, leads to starting of all containers.
*
* @throws \Exception when docker-compose up fails.
*/
function start_site_containers( $site_root ) {
function start_site_containers( $site_root, $containers = [] ) {

EE::log( 'Pulling latest images. This may take some time.' );
chdir( $site_root );
EE::exec( 'docker-compose pull' );
EE::log( 'Starting site\'s services.' );
if ( ! EE::docker()::docker_compose_up( $site_root ) ) {
if ( ! EE::docker()::docker_compose_up( $site_root, $containers ) ) {
throw new \Exception( 'There was some error in docker-compose up.' );
}
}
Expand Down