-
Notifications
You must be signed in to change notification settings - Fork 39
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
no implicit conversion of nil into String on knife google server create #61
Comments
I've noticed that |
|
Thanks for digging into this. I'll make some time on Monday to take a look On Fri, Mar 27, 2015, 15:27 ChadScott [email protected] wrote:
|
Alright, I'm at a loss. I've dug on this for a little bit and I don't know enough about the GCE API nor the internals of Knife to really be able to get to the bottom of it without going into a cave for awhile. Any luck on your end, @paulrossman? |
If it helps, here are the versions of the Google related gems:
|
Sorry for the late reply. I've not run in to this issue at all and have been building lots of instances recently. My suggestion is to update to knife-google 1.4.x. There were a bunch of gemspec changes in #54. I currently have these gem versions installed: chef (12.2.1) |
After upgrading to knife-google 1.4.3 (which isn't in the gem repo, by the way... I had to pull it down from Github and manually install it), I get yet another error:
I thought at first it must be something in my SSL stack, but it works fine for ec2:
...etc |
Okay, I finally figured this out. For those that find this via googling or whatever, this is a problem with chefdk on MacOS. I'm unsure exactly what the issue is, but this solves it for me: Install brew (brew.sh) Voila! |
I am still getting this error as well. I'm not really interested in building my own gem. I'll see if I can resolve for knife-google 1.3.1 |
So, I figured out the issue with a small code change. Chad, you were absolutely right. The code line @start_time = Time.parse( data["startTime"] ) was the culprit. The reason why when you build the 1.4.3 gem it works is in that version there is a null check on the variable. I think that the new gem version should be built and distributed. I simply changed the following lines in my lib/google/compute/zone_operation.rb file @insert_time = Time.parse( data["insertTime"] ) if data.key?("insertTime") |
I have tried that in both global_operation.rb and zone_operation.rb and it doesn't seem to work. |
Is there another gem that does that? |
ohadpartuck, I've used -VV on the end of the string. Can you output what you see when you run the command. This plugin is really important especially if you are negotiating with Amazon about your monthly AWS bill. knige google zone list -VV |
@jeremiahsnapp ok , that helped , After 15 mins, I tracked the issue to weirdly, I ran it in the console and it works:
Now that I see that, it's not the same error as stated in this log. any ideas? |
@sqrrrl maybe you can take a look? |
Argh. Yes, there was a stupid bug introduced in 0.7. Just pushed 0.7.1 which should fix the issue. @ohadpartuck - Please update and let me know if it addresses the issue you're seeing. |
@sqrrrl I don't see 0.7.1 neither here |
nvm , found it here |
I've whacked at this issue for a couple hours now and I can't make sense of it. I'm guessing Google is returning a nil and we're trying to print it.
The text was updated successfully, but these errors were encountered: