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

Allow either file-based or command-line AWS credentials #145

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

Conversation

nyetsche
Copy link

This fixes #49 and it's being used without issue in our environment.

Lesko, Matthew (NIH/NLM/NCBI) [C] added 4 commits February 4, 2015 15:31
…ration file ($ICE_HOME/ice.properties). In our environment, we want those credentials in the ice.properties files, plus it lets you stick everything in a .war for serving up by tomcat
…ICE_HOME definitions for the AWS credentials, first preference is given to those on the command line.
@cloudbees-pull-request-builder

ice-pull-requests #64 SUCCESS
This pull request looks good

…been added which causes the startup to fail with a NullPointerException. in our environment we do not need the reservation info
@cloudbees-pull-request-builder

ice-pull-requests #70 SUCCESS
This pull request looks good

ansoni and others added 2 commits February 26, 2015 13:19
… we get more than an ugly NullPointer exception
Revert "comment out reservationService.init() - a new reservaration type has been added which causes the startup to fail with a NullPointerException. in our environment we do not need the reservation info"

This reverts commit 11824b46941c47a77a8b45fcaba631f4a65f4f26.
@cloudbees-pull-request-builder

ice-pull-requests #71 SUCCESS
This pull request looks good

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

Successfully merging this pull request may close these issues.

Ice ignoring credentials in ice.properties on EC2 instance
3 participants