Skip to content
This repository has been archived by the owner on Nov 30, 2021. It is now read-only.

376 move slugbuilder hook #410

Merged
merged 3 commits into from
Jan 5, 2014
Merged

Conversation

tombh
Copy link
Contributor

@tombh tombh commented Dec 18, 2013

Relates to #376

@mboersma
Copy link
Member

Some flake8 errors, apparently.

@coveralls
Copy link

Coverage Status

Coverage remained the same when pulling 0896b9b on tombh:376-move-slugbuilder-hook into b15d949 on opdemand:master.

@mboersma
Copy link
Member

I think I have everything correct--I purged all my cookbooks and uploaded those from @tombh's branch, then provisioned a new controller, then updated its codebase to this PR, reloaded, chef-client, etc. This is all I can get:

$ git push deis master
Counting objects: 17, done.
Delta compression using up to 4 threads.
Compressing objects: 100% (14/14), done.
Writing objects: 100% (17/17), 8.02 KiB | 0 bytes/s, done.
Total 17 (delta 5), reused 0 (delta 0)
Traceback (most recent call last):
  File "/opt/deis/controller/bin/slugbuilder-hook.py", line 12, in <module>
    CONTROLLER_DIR = os.environ['CONTROLLER_DIR']
  File "/opt/deis/controller/venv/lib/python2.7/UserDict.py", line 23, in __getitem__
    raise KeyError(key)
KeyError: 'CONTROLLER_DIR'
To git@deis-controller.local:easier-vineyard.git
 * [new branch]      master -> master
error: failed to push some refs to 'git@deis-controller.local:easier-vineyard.git'

Indeed, my /etc/environment looks like this:

PATH="/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/usr/games"
SLUG_DIR=/opt/deis/build/slugs

I can't tell why CONTROLLER_DIR isn't getting set.

@tombh
Copy link
Contributor Author

tombh commented Dec 22, 2013

I've had a quick look and can reproduce this, but can't figure it out either. Most odd. I'll have to have a go again after Christmas now.

mboersma added a commit that referenced this pull request Jan 5, 2014
Tested, works great. Thanks very much, Tom!
@mboersma mboersma merged commit 8cdba92 into deis:master Jan 5, 2014
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants