-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
Upgrade to 1.2 failed to load config file, /credentials/cloud #2191
Comments
@Stolr I've never seen that one before! What provider are you using? Is it possible to provide the credentials file with any sensitive information stripped out? I wonder if it's some kind of weird whitespace error. |
@skriss : No issue with same config file using the v1.1.
With 1.1 , using :
Working. When upgrading : Not working ( see below ) When installing 1.2 from scratch
Same error
I also tried to put a fake credentials to see and I got a proper error
|
hmm - it looks pretty normal to me. Does anything change if you remove the whitespace around the two |
@skriss : Same error : credentials-velero
Velero logs
|
@Stolr I see a couple references to similar issues in the AWS SDK repo (e.g. aws/aws-sdk-go#2239) - anything like that seem relevant? Does this credentials file work fine with the It certainly seems to be related to the AWS SDK's parsing of the file. Perhaps you could generate a new access key and see if that fixes things? |
Hey @Stolr, has your issue been resolved? |
Hey , Sorry for the late answer. I can't really re generat because it is already used. I'll test with aws cli and let you know. Still having the issue |
Hey , Works fine with aws cli
|
@Stolr OK, thanks for the info. We could try updating the version of the AWS SDK that we use to the latest, to see if that resolves it. If I publish a test image with that change for you, would you be able to try it out? |
Sure , I'd love to |
OK, I pushed If you already have velero installed somewhere, you can do the following:
Or you can Or, if you don't have velero up and running, you can use the new image as the value of the If you get an error again, please include the full log + error message! Thanks for helping us dig into this. |
Ok so , Install velero from fresh instance
==> ERROR ( same as above ) Then I edited the deployment to change the init container to your image => ERROR
What king of log do you need ? I'll try to create a whole new instance of minio and changing access key tomorrow and test again I've got some caracters |
Argh. You should probably file an issue in the https://github.com/aws/aws-sdk-go repo, as it seems pretty likely that there's a bug in there. I would include the fact that the file works OK with the |
Hey @skriss thanks for your feedback. Are you able to reproduce on your side with my credentials ?
I just secured my instance so not a problem to give you theses keys If you can give me a feedback if you are able to reproduce that would be awesome. Thanks |
@Stolr yep, I was able to reproduce. However, I did find a workaround - putting double quotes around the secret access key value allows it to parse, i.e.:
Sure looks like that leading |
Awesome ! I should have think of this workaround ! Maybe the documentation can be updated to put that doubles quotes to make sure this will work for Everyone. @skriss I'll close the issue. Thanks a lot for your help. |
I was also facing the same issue and opened an issue on the AWS SDK project: aws/aws-sdk-go#3958 |
@Stolr the issue aws/aws-sdk-go#3958 is now fixed maybe can you upgrade to the its latest version. |
What steps did you take and what happened:
I upgraded from 1.1 to 1.2 ( Followed https://velero.io/docs/v1.2.0/upgrade-to-1.2/ )
The velero pod crashed with error :
I double checked and seems that the secret still exists. Seems more like a parsing issue.
What did you expect to happen:
No regression
Environment:
velero version
):velero client config get features
):kubectl version
):The text was updated successfully, but these errors were encountered: