Skip to content
This repository has been archived by the owner on Mar 19, 2022. It is now read-only.

Fix don't copy https_proxy_*** to solo.rb. #421

Merged
merged 1 commit into from
Jan 28, 2015

Conversation

masakura
Copy link
Contributor

Proxy settings to solo.rb is copied from .chef/knife.rb with knife solo cook host. But, don't copy https_proxy_user and https_proxy_pass.

client:$ cat .chef/knife.rb
cookbook_path    ["cookbooks", "site-cookbooks"]
node_path        "nodes"
role_path        "roles"
environment_path "environments"
data_bag_path    "data_bags"
#encrypted_data_bag_secret "data_bag_key"

knife[:berkshelf_path] = "cookbooks"

http_proxy "http://proxy.server.com:3128/"
http_proxy_user "http_user"
http_proxy_pass "http_pass"
https_proxy "http://https.proxy.server.com:3128/"
https_proxy_user "https_user"
https_proxy_pass "https_pass"

After run knife solo cook host.

host:$ cat chef-solo/solo.rb
node_name "nodejs"

base = File.expand_path('..', __FILE__)

nodes_path                File.join(base, 'nodes')
role_path                 File.join(base, 'roles')
data_bag_path             File.join(base, 'data_bags')
encrypted_data_bag_secret File.join(base, 'data_bag_key')
environment_path          File.join(base, 'environments')
environment               "_default"
ssl_verify_mode           :verify_none

cookbook_path []
cookbook_path << File.join(base, 'cookbooks-1') # /var/lib/gems/1.9.1/gems/knife-solo-0.4.2/lib/knife-solo/resources/patch_cookbooks
cookbook_path << File.join(base, 'cookbooks-2') # /home/directory/documents/tmp/chefrepo/cookbooks
cookbook_path << File.join(base, 'cookbooks-3') # /home/directory/documents/tmp/chefrepo/site-cookbooks

http_proxy     "http://proxy.server.com:3128/"
https_proxy     "http://https.proxy.server.com:3128/"
http_proxy_user     "http_user"
http_proxy_pass     "http_pass"

@matschaffer
Copy link
Owner

Good catch. Thanks!

matschaffer added a commit that referenced this pull request Jan 28, 2015
Fix don't copy https_proxy_*** to solo.rb.
@matschaffer matschaffer merged commit 0bc80a1 into matschaffer:master Jan 28, 2015
@masakura
Copy link
Contributor Author

Thanks!

I am waiting eagerly to be released!

@masakura masakura deleted the fix-no-https-proxy-user branch January 29, 2015 07:52
@matschaffer
Copy link
Owner

We move to Japan next week so could be a little while. In the mean time you
should be able to run rake install from a clone to get a local copy
installed from master. Or bundle with binstubs and use the knife that
gets generated from that.

-Mat

about.me/matschaffer

On Wed, Jan 28, 2015 at 11:52 PM, Tomo Masakura notifications@github.com
wrote:

Thanks!

I am waiting eagerly to be released!


Reply to this email directly or view it on GitHub
#421 (comment)
.

@masakura
Copy link
Contributor Author

I am not familiar with ruby. I modify "/var/lib/gems/..../solo-cook.rb". It's no problem.

For my study, I will try the method.

Thanks!

We move to Japan next week so could be a little while.

I am Japanese. Japan is a good place. Have a nice trip!

@matschaffer
Copy link
Owner

That works too. Thanks!

-Mat

about.me/matschaffer

On Thu, Jan 29, 2015 at 3:32 PM, Tomo Masakura notifications@github.com
wrote:

I am not familiar with ruby. I modify "/var/lib/gems/..../solo-cook.rb".
It's no problem.

For my study, I will try the method.

Thanks!

We move to Japan next week so could be a little while.

I am Japanese. Japan is a good place. Have a nice trip!


Reply to this email directly or view it on GitHub
#421 (comment)
.

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.

2 participants