Jay Faulkner
Jay Faulkner
Is my proof of concept for supporting this if you want. It's tested working on my machine. I'm not PRing it because: 1) I don't know if you expect this...
I'll note I found a bug in that code, I have to also check for NaN and return "Unknown" in that case (or else on startup you get a couple...
I have a similar bug that sounds like it could be related. When moving from a single monitor (laptop) to two external monitors (docked), I will have applications get "stuck"...
Python upstream is saying that https://github.com/python/cpython/issues/130327 is the issue impacting us; not 135552. I don't really know for sure, but I'm going to link this issue here as well.
This seems OK to me. Package looks to have an MIT license.
@niedbalski hello, what is necessary to get a review on this and get it merged? We'd like to continue enhancing this project, but it's going to be difficult without timely...
Adam and I did some testing this afternoon and determined that in cases where a field is omitted for being unavailable in a given microversion that we export a default...
You may want to squash this? I'm not sure this is 11 commits worth of change, and it doesn't look like squash-and-merge github feature is usual practice here.
Thanks for all the good work keeping openstack-exporter running. I believe our (G-Research OSS) corporate github organization should be sponsoring you now. I recommend any other companies or individuals relying...
:+1: LGTM, this matches our testing with an older ironic microversion This should be good to merge @niedbalski