<p dir="ltr">On 12 Apr 2016 12:19 AM, "Sean Dague" <<a href="mailto:sean@dague.net">sean@dague.net</a>> wrote:<br>
><br>
> On 04/11/2016 10:08 AM, Ed Leafe wrote:<br>
> > On 04/11/2016 08:38 AM, Julien Danjou wrote:<br>
> ><br>
> >> There's a lot of assumption in oslo.log about Nova, such as talking<br>
> >> about "instance" and "context" in a lot of the code by default. There's<br>
> >> even a dependency on oslo.context. >.<<br>
> >><br>
> >> That's being an issue for projects not being Nova, where we end up<br>
> >> having configuration options talking about "instances" and with default<br>
> >> values referring to that.<br>
> >> I'm at least taking that as being a serious UX issue for telemetry<br>
> >> projects.<br>
> >><br>
> >> I'd love to sanitize that library a bit. So, is this an option, or would<br>
> >> I be better off starting something new?<br>
> ><br>
> > The nova team spent a lot of time in Mitaka starting to clean up the<br>
> > config options that were scattered all over the codebase, and improve<br>
> > the help text for each of them so that you didn't need to grep the<br>
> > source code to find out what they did.<br>
> ><br>
> > I could see a similar effort for oslo.log (and maybe other oslo<br>
> > projects), and I would be happy to help out.<br>
><br>
> This isn't so much about scattered options, oslo.log options are all in<br>
> one place already, it's about the application specific ones that are<br>
> embedded.<br>
><br>
> I agree that "instance" being embedded all the way back to oslo.log is<br>
> weird. Ideally we'd have something like "resource" that if specified<br>
> would be the primary resource the request was acting on. Or could even<br>
> just build some custom loggers Nova side to inject the instance when we<br>
> have it.</p>
<p dir="ltr">Instance was put in there a long time ago, before Oslo existed. It was then just forklifted out blindly. I like the idea of changing to something like "resource" or even better "resources", but we do need to present this information in a structured way. We need to remember this was added because of specific concerns around operational usability of or logging.<br></p>
<p dir="ltr">> I'm not sure why oslo.context is an issue though. That's mostly about<br>
> putting in the common information about the identity of the requester<br>
> into the stream.</p>
<p dir="ltr">Michael<br>
</p>