Jay Faulkner

Results 40 comments of Jay Faulkner
trafficstars

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