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

awscrededentials.conf in git, but it's singular on the filesystem #30

Closed
pcn opened this issue Jun 23, 2012 · 3 comments
Closed

awscrededentials.conf in git, but it's singular on the filesystem #30

pcn opened this issue Jun 23, 2012 · 3 comments

Comments

@pcn
Copy link

pcn commented Jun 23, 2012

In git, this file exists:

~/dvcs/Priam $ git blame src/main/resources/conf/awscredentials.properties 
52d95031 (Praveen Sadhu 2012-01-27 17:20:57 -0800 1) AWSACCESSID=""
52d95031 (Praveen Sadhu 2012-01-27 17:20:57 -0800 2) AWSKEY=""

However on the filesystem, using that filename gives you an error:

ERROR 17:05:48,759 Exception with credential file 
java.io.FileNotFoundException: /etc/awscredential.properties (No such file or directory)
        at java.io.FileInputStream.open(Native Method)
        at java.io.FileInputStream.<init>(FileInputStream.java:137)

Is there any reason that the file in git has a different name from the file that should reside on the filesystem?

@Vijay2win
Copy link
Contributor

The problem is when ever you update Priam you will have to update this conf file, it is better to be in /etc...
I understand we need to documented well. Any suggestion to making it explicit is welcomed.

@pcn
Copy link
Author

pcn commented Jun 25, 2012

Maybe we're talking about different aspects of this file. I'm not concerned about the fact that this is in /etc, I'm only talking about the fact that this file, when it's in /etc/, is awscredential.conf (singular).

When it's in git, it's called awscredentials.conf (plural, awscredential_S_).

So, if the installation instructions are naively followed and awscredential_s_.conf is copied to /etc, a java.io.FileNotFoundException is raised.

@jasobrown
Copy link
Contributor

renamed the file in the distro to be awscrendential.properties. This was preferred to changing the name of the file in the code, as there's many deployments that would break.

Now the documentation should match the simple deploy scenario. Thanks

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

No branches or pull requests

3 participants