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

$BLT_DIR/scripts/travis/deploy_branch running on TravisCI results in error #1308

Closed
arknoll opened this issue Apr 4, 2017 · 2 comments
Closed
Labels
Bug Something isn't working

Comments

@arknoll
Copy link
Contributor

arknoll commented Apr 4, 2017

My system information:

  • Operating system type: OSX
  • Operating system version: Sierra
  • BLT version: 8.7.0-beta3

When I merge into a branch that has a deploy step:

   - provider: script
     script: "$BLT_DIR/scripts/travis/deploy_branch"
     skip_cleanup: true
     on:
       branch: experimental
       php: 5.6

I get the following output:
"There are uncommitted changes, commit or stash these changes before deploying."

And I expected this to happen:
deploy phing task to execute normally

Additional info:
It appears there are changes to the project.yml file in the working dir. This command was run right before the deploy step.

travis@testing-gce-12a7d536-80ef-411a-811e-dfc2aa90172e:~/build/acquia-pso/canary$ git status
HEAD detached at ae36707
Changes not staged for commit:
  (use "git add <file>..." to update what will be committed)
  (use "git checkout -- <file>..." to discard changes in working directory)

        modified:   blt/project.yml

no changes added to commit (use "git add" and/or "git commit -a")
travis@testing-gce-12a7d536-80ef-411a-811e-dfc2aa90172e:~/build/acquia-pso/canary$ git diff
diff --git a/blt/project.yml b/blt/project.yml
index 964a8e2..3ab6cc2 100644
--- a/blt/project.yml
+++ b/blt/project.yml
@@ -6,7 +6,7 @@ project:
     name: canary
   local:
     protocol: http
-    hostname: 'local.${project.machine_name}.com'
+    hostname: '127.0.0.1:8888'
 git:
   default_branch: master
   remotes:
@@ -60,4 +60,3 @@ behat:
   launch-phantomjs: true
 cm:
   strategy: config-split
-
@arknoll
Copy link
Contributor Author

arknoll commented Apr 4, 2017

This looks to be due to this new addition: https://github.com/acquia/blt/blob/8.x/scripts/travis/run_tests#L6

@grasmash
Copy link
Contributor

grasmash commented Apr 4, 2017

This was resolved by this revert commit: d1cef2f

@grasmash grasmash closed this as completed Apr 4, 2017
@grasmash grasmash added the Bug Something isn't working label Apr 4, 2017
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

2 participants