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

Add config file for ambari url properties #1

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Add config file for ambari url properties #1

wants to merge 3 commits into from

Conversation

spolex
Copy link

@spolex spolex commented Feb 27, 2017

I've been testing this script in azure cloud and it works. I've added some variables in order to improve security and simplicity.

Thanks a lot for this resource!

I've been testing this script in azure cloud and it works. I've added some variables in order to improve security and simplicity
@spolex spolex closed this Jun 21, 2017
@CesarCapillas
Copy link
Member

Hi @spolex, I missed this request. Sorry, I'm not really sure about this. The path may be something like /etc/ambari-shell-script.properties or something similar for using it in a service script or from any path (instead of pwd). Maybe it's a better aprox for nagios plugins too, to prevent some repeated stuff. Thanks for the testing, and let me think a little bit about it.

Regards.
--C.

@CesarCapillas CesarCapillas reopened this Jun 24, 2017
@CesarCapillas CesarCapillas changed the title Improves Add config file for ambari url properties Jun 24, 2017
@spolex
Copy link
Author

spolex commented Jul 6, 2017

Hi @CesarCapillas,
It''s up to you. Depends on the main goal. One different solution could be parametrize in config file the path.

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

Successfully merging this pull request may close these issues.

2 participants