Better logging for Kube pod failures

[thedanvail]

• Handles the error more gracefully if we receive from Kube's API when trying to spin up a pod
• Updated CHANGELOG with new changes

[thedanvail]

Hopefully this is the functionality that was requested from #110 - if not, can still make changes. Thought about using Result and/or adding onto error.rs, but neither seemed to quite fit the bill for this in my opinion.

[aviramha]

Thanks for your contribution! See comments inline.

[thedanvail]

All right, I've switched over to using anyhow for attaching context to these errors. I'm signing off for the night, but should any additional changes be needed, please do not hesitate to let me know and I'll get on them tomorrow evening or possibly Wednesday.

[thedanvail]

I see some checks failed, I'll look at those in the morning

[aviramha]

LGTM, @meowjesty can you review this please?

[meowjesty]

Hey Dan, thanks for the help in improving this! The lack of context kinda tripped me over a few times.

My main suggestion comes in the form that we probably don't need to have a Result here, as we just crash when this fails anyway. Just change some of the unwraps into expects (also no need to change every single one).

Edit: Outdated, please ignore this (my bad)!

[aviramha]

Last change and we can merge it :)

[aviramha]

Great, can you merge main into your branch so tests will run before merging?
Thanks!

[thedanvail]

Working on fixing the tests

[eyalb181]

Tests are passing, the e2e are just a bit flaky at the moment (need to look into it). Merging to main, thanks again @thedanvail !