Skip to content
This repository has been archived by the owner on Jan 8, 2019. It is now read-only.

contributed smartsense-chef recipe #699

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

dbist
Copy link

@dbist dbist commented Oct 25, 2016

No description provided.

"recipe[bcpc_jmxtrans]"
"recipe[bcpc_jmxtrans]",
"recipe[smartsense-chef::default]",
"recipe[smartsense-chef::smartsense_server]"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Does server need to be setup before the agent (in default)?

@cbaenziger
Copy link
Member

cbaenziger commented Oct 25, 2016

I would think we'll need a wrapper recipe here @dbist. I see a number of attributes in the Smartsense cookbook which should be aligned with what we use in Chef-BACH:

As an aside, it looks like your Test Kitchen configs may have polluted your cookbook:

Also you'll want to pull your apt config into attributes from hardcoded in your recipe as some Chef-BACH clusters are run behind proxies or without Internet access.

@cbaenziger
Copy link
Member

For reference, a wrapper cookbook in Chef-BACH today is the kafka-bcpc cookbook which has a recipe to wrap attributes; for Smart Sense I would think simply a recipe in bcpc-hadoop would suffice. (Some theory behind wrapper cookbooks can be found web-searching around too.)

@dbist
Copy link
Author

dbist commented Oct 27, 2016

node[:smartsense-chef][:java_home] should equal node[:bcpc][:hadoop][:java]
node[:smartsense-chef][:cluster_name] should equal node.chef_environment as is our convention.

As an aside, it looks like your Test Kitchen configs may have polluted your cookbook:

Java attributes and an errant include_recipe would be good to pull out into your .kitchen.yml

Also you'll want to pull your apt config into attributes from hardcoded in your recipe as some Chef-BACH clusters are run behind proxies or without Internet access.

@cbaenziger also, not sure how to leverage wrapper recipe over .kitchen.yml entries, it did not take effect, please advise

I updated trunk, most likely need to generate new PR?

@dbist
Copy link
Author

dbist commented Nov 9, 2016

@cbaenziger any updates on this?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants