of this single call a solum dev can make and then behind the scenes we can send something to oslo messaging or we can log it or we can write it to a DB, whatever
asalkeld
it should grow
it shouldn't grow
kgriffs
contextual info could be attached to messages before they get put onto oslo messaging
asalkeld
kgriffs, I mean oslo rpc uses the context for auth
kgriffs
that being said, why are we sending events through oslo messaging? I guess it's the "event" word we've been using
asalkeld
and sends that to the other end
kgriffs
asalkeld: oic
well, that is another use for sure
paulmo
kgriffs: The storage, method of getting it to the user, whether it is a log or notification is all details I think… my concern is how do we tag the information in the control plane to enable filtering on the back end?
kgriffs
that may be the disconnect - i was purely thinking of the context in terms of metadata that is along for the ride
asalkeld
it seems we are mixing two things here logging context and auth
not convinced they are the same
paulmo
Again, just ignore my code for a moment, what is the proper way to solve this?
then we have a better idea on how to solve the problem
now it's pie in the sky stuff
paulmo
Does everyone feel that way? I'll happily drop this topic and wash my hands. :)
roshanagr1
what is the proposal
I did not exactly follow
asalkeld
paulmo I hope everyone doesn't, it's much more fun when there is disagreement
:)
paulmo
I think Angus is suggesting that we push off any concern about logging to users until later when it becomes more of a pain point (is that the right paraphrase)?
(control plane data that is)
roshanagr1
asalkeld: is that is what you meant, I would have to disagree
gokrokve_
I don't see a reason to blindly copy RequestContext from Nova. We don't use oslo.messging now so Serialization is not a case.
asalkeld
no, I mean add code infrastructure when you need it
paulmo
Perhaps you could articulate your stance a bit more?
asalkeld
sorry fingers are a bit painful and slow
roshanagr1
thats why I prefer voice calls :-)
asalkeld
I am just saying add generic code when you need it
paulczar is now known as zz_paulczar
if there are a bunch of you guys that want to try something else and want to burn time on it, I won't stand in the way any more
for me, it would slow development down
paulmo
I think we are all trying to do the right thing… unfortunately you are the main source of the "OpenStack way" for me so please don't take this as ganging up or anything… just trying to understand.
asalkeld
and makes moving between projects a bit of a pain
kgriffs
asalkeld: well, it comes down to what features we want, wen
s/wen/when
gokrokve_
asalkeld: I am not sure that it will slow doen the development. Better design now faster development later.
jasonb365 has quit
roshanagr1
logging is basic to get right in the beginning
kgriffs
if we want the feature to give app devs a view into some of these checkpoints/events/reports/whatever then we need to do this
if we want to wait, then we don't need to do this now, but that is a roadmap think IMHO
roshanagr1
otherwise we will be promosing technical debt
gokrokve_
There are patches that require some kind of context
kgriffs
roshanagr1: could be; it would be a pain to have to go back and redo everywhere we log, but not intractible
asalkeld
openstack is moving to this feature via ceilometer
paulmo
My goal is to make sure we make this decision with open eyes (whichever way it goes) and with an understanding of technical debt if any we may incur.
gokrokve_
Some of them related to data model
claytonc_ has quit
kgriffs
asalkeld: ceilometer is going to archive a stream of events?
that are generic?
asalkeld
kgriffs, yes
kgriffs thinks ceilometer has a serious case of scope creep
kgriffs
ah, ok
asalkeld
I think you guys (esp. rackers) need to talk to sandy walsh
paulmo
gokrokve_: In Solum?
kgriffs
well, we can leverage that.
roshanagr1
kgriffs: agree
claytonc_ joined the channel
paulmo
I did speak with Sandy before
kgriffs
but we still need to decide up front on our interface
bye , have a good evening (and good morning to Angus)
kgriffs
asalkeld: just starting skimming it - interesting stuff
asalkeld
see you roshanagr1
roshanagr1 has left the channel
kgriffs
marconi is useful for "i want to subscribe and be notified when things go bloop"
but you also want to log in a day later and look for a sequence of things that happened after the fact
anyway, getting a bit off onto a tangent here...
paulmo
Apologies if I'm being dense but I'm not sure why we keep circling back to the method of exposing logs/notifications or how we store them… isn't the real issue how to tag them in the first place?
(the rest is "easy" right?)
asalkeld
also this *could* be like the request_id and all services send to the users queue
kgriffs
paulmo: +1
I don't care if the body of that function is "pass" for the first iteration
paulmo
Marconi, ceilometer, carrier pigeon… I don't care. :)
kgriffs
just so contributors can start getting in the habit of using it
paulmo
Yeah, tagging the information while developing so we don't forget what is what…
kgriffs
that's my $0.02, anyway
think composition, not extension
we have this reporter thingy
paulmo
Now we can go low tech and require specially crafted comments to indicate the confidentiality of data in the code… that works too.
asalkeld
well I'd suggest the secrurity context without the public part
kgriffs
and it composes oslo logging, oslo messaging, anything you like underneath
paulmo
Don't even worry about security context, start fresh, how should we do this.
paulmo nods at kgriffs
kgriffs
context is just a possible means of supporting this other stuff
asalkeld
well you could use the users role to match up with the class of info
paulmo nods again at kgriffs
kgriffs
I think the real question is do we want this reporting interface thingy
asalkeld: that is a good point - it gives you more metadata to draw on
jwforres has quit
asalkeld
if we had a log formatter for operater and user
we could just use the log api
so based on the role the formatter adds the info or not