These are chat archives for exceptionless/Discuss

25th
Feb 2016
Eric J. Smith
@ejsmith
Feb 25 2016 03:55 UTC
aspnet/Logging#370
Really want them to separate their logging extensions into a separate project.
Blake Niemyjski
@niemyjski
Feb 25 2016 04:18 UTC
Seems they don't care
benmaina
@benmaina
Feb 25 2016 05:26 UTC
@niemyjski needed help on the errors being logged on my dashboard for my client. Currently my dashboard aint loging any exceptions yet i have already set up the configuration file with my self hosted instance.Kindly assist.
Jamie Clarke
@jamie94bc
Feb 25 2016 08:32 UTC
woke up to this
blob
Jamie Clarke
@jamie94bc
Feb 25 2016 08:50 UTC
what do you guys use for exceptionless? I'll probably move to Sendgrid - have to deal with IP warmup though.
Blake Niemyjski
@niemyjski
Feb 25 2016 12:38 UTC
We use mandrill
We'll be moving to something
Eric sent me this https://sendy.co/ we might also look at ses
@benmaina let me know when your free and I'll take a look with you
I'd enable client tracing and take a look
Jamie Clarke
@jamie94bc
Feb 25 2016 12:55 UTC
@niemyjski yeah i've used https://emailoctopus.com/ which also uses SES
strange Azure don't handle this yet
Blake Niemyjski
@niemyjski
Feb 25 2016 13:06 UTC
Yup
Wonder if there are any azure templates / dicker images for this
Eric J. Smith
@ejsmith
Feb 25 2016 13:08 UTC
Yeah, I think we will switch to SES.
That email octopus looks interesting.
I got all the tests to pass last night.
The logging was super helpful to see what was going on. :-)
Bill Henning
@BillHenning
Feb 25 2016 13:10 UTC
oh crap, we use mandrill too
Blake Niemyjski
@niemyjski
Feb 25 2016 13:12 UTC
Was just thinking what are we going to do for logging and the exceptionless client
Eric J. Smith
@ejsmith
Feb 25 2016 13:15 UTC
What do you mean @niemyjski ?
Blake Niemyjski
@niemyjski
Feb 25 2016 13:26 UTC
Would we want to inject the loggers like we are doing for foundatio
Guess you'd have to use our existing di
Eric J. Smith
@ejsmith
Feb 25 2016 13:29 UTC
The way we do logging for Exceptionless is fine.
I wish there was an easier way for people to see what is going on though.
Jamie Clarke
@jamie94bc
Feb 25 2016 13:30 UTC
@ejsmith what was the cause? - sorry i didn't get a chance to look into it! :(
Eric J. Smith
@ejsmith
Feb 25 2016 13:30 UTC
I wonder if we should use the in memory log to have a property on your client instances that contains the last X Log messages.
Blake Niemyjski
@niemyjski
Feb 25 2016 13:30 UTC
Yeah
Eric J. Smith
@ejsmith
Feb 25 2016 13:31 UTC
Just make that 1st class and it would be a lot easier to diagnose issues.
@jamie94bc it was a timing issue where things were executing in a different order on the build server. I just put Task.Delay in there to make sure they executed in the order I was expecting for the test.
Jamie Clarke
@jamie94bc
Feb 25 2016 13:34 UTC
that's strange!
Eric J. Smith
@ejsmith
Feb 25 2016 13:35 UTC
Yeah, our tests should probably be setup in a different way. Async testing is hard.
Blake Niemyjski
@niemyjski
Feb 25 2016 13:54 UTC
We should test the azure service bus too
Those are Bing skipped
Eric J. Smith
@ejsmith
Feb 25 2016 13:54 UTC
How?
I'm not paying for a servicebus account to test against.
Blake Niemyjski
@niemyjski
Feb 25 2016 13:56 UTC
We already are paying for a test one
Isn't it usage based
Eric J. Smith
@ejsmith
Feb 25 2016 13:58 UTC
Just not interested in testing something that is external to the build server. That's why servicebus sucks.
Eric J. Smith
@ejsmith
Feb 25 2016 14:03 UTC
Did you see that MS released an open source client for Kafka?
Blake Niemyjski
@niemyjski
Feb 25 2016 14:06 UTC
Cool
Blake Niemyjski
@niemyjski
Feb 25 2016 15:06 UTC
@all, how is the sessions feature working out for you? We’ve been tweaking it the past few weeks
Eric J. Smith
@ejsmith
Feb 25 2016 15:17 UTC
@niemyjski we should do a blog post covering the usage chart you added
Blake Niemyjski
@niemyjski
Feb 25 2016 15:17 UTC
yeah
Blake Niemyjski
@niemyjski
Feb 25 2016 16:43 UTC
I don’t even think this is used anymore or you can even pin apps in edge
can you?
guess it hurts nothing by leaving it
but we could shrink the html as well as get rid of the icon files
Eric J. Smith
@ejsmith
Feb 25 2016 16:44 UTC
its for creating a tile
that is still a thing.
Blake Niemyjski
@niemyjski
Feb 25 2016 16:44 UTC
ok
Bill Henning
@BillHenning
Feb 25 2016 16:44 UTC
or you guys could just make a windows store wrapper web-based app for exceptionless :)
Blake Niemyjski
@niemyjski
Feb 25 2016 16:45 UTC
yeah
that would be a great pull request @BillHenning
Eric J. Smith
@ejsmith
Feb 25 2016 16:48 UTC
just tried it and it didn’t seem to respect any of those
Blake Niemyjski
@niemyjski
Feb 25 2016 16:48 UTC
maybe someone could try it on windows mobile :)
@renekarberg have you tried pinning our website (as an app) on windows mobile
if so can you let us know what it looks like
pushing it out now
@ejsmith the serialization changes have been pushed as of an hour ago as well
I submitted events and ran through a bunch of pages and nothing looked out of place.
had to fix a few more bugs (found during testing)
I also forced refreshed any connected clients
@ejsmith I’m going to update our repositories to use the new logging any suggestions
or should I hold off on updating to the latest version of foundatio until you tell me how much of a pain it is.. I need to look into a batch size error with elastic too
Eric J. Smith
@ejsmith
Feb 25 2016 17:13 UTC
paul is about to convert our app
I guarantee it will be a pain.
but it will be worth it.
Blake Niemyjski
@niemyjski
Feb 25 2016 17:30 UTC
yeah
Eric J. Smith
@ejsmith
Feb 25 2016 17:31 UTC
Adding a nuget package for the xunit logging implementations. Also going to add another one for NLog.
Blake Niemyjski
@niemyjski
Feb 25 2016 17:32 UTC
ok
Eric J. Smith
@ejsmith
Feb 25 2016 17:41 UTC
SyntaxEditor on Xbox!
Bill Henning
@BillHenning
Feb 25 2016 17:41 UTC
Code Writer on xbox!
Eric J. Smith
@ejsmith
Feb 25 2016 17:41 UTC
hehe
Blake Niemyjski
@niemyjski
Feb 25 2016 18:02 UTC
@ejsmith you got a big problem with the latest foundatio build
Eric J. Smith
@ejsmith
Feb 25 2016 18:02 UTC
what is that?
Blake Niemyjski
@niemyjski
Feb 25 2016 18:02 UTC
you started targeting 4.5.1 I think?
Eric J. Smith
@ejsmith
Feb 25 2016 18:02 UTC
yeah
why?
Blake Niemyjski
@niemyjski
Feb 25 2016 18:02 UTC
lib/net45 contains a single pdb and then /net451 contains a dll
Eric J. Smith
@ejsmith
Feb 25 2016 18:03 UTC
ahh
Blake Niemyjski
@niemyjski
Feb 25 2016 18:03 UTC
so when you upgrade a 4.5.0 package it actually gets uninstallled
Eric J. Smith
@ejsmith
Feb 25 2016 18:03 UTC
so we need to change the folder names?
we can actually go back to just 4.5
Blake Niemyjski
@niemyjski
Feb 25 2016 18:03 UTC
btw, please migrate the targets to 4.5.2 it’s the onlything supported by microsoft
yeah
Eric J. Smith
@ejsmith
Feb 25 2016 18:03 UTC
since I’m not using their package
Blake Niemyjski
@niemyjski
Feb 25 2016 18:03 UTC
ok
Eric J. Smith
@ejsmith
Feb 25 2016 18:03 UTC
I guess we can change to 4.5.2
Frank Ebersoll
@frankebersoll
Feb 25 2016 19:29 UTC
Hey there :-)
Blake Niemyjski
@niemyjski
Feb 25 2016 19:29 UTC
hey
Frank Ebersoll
@frankebersoll
Feb 25 2016 19:29 UTC
How is it going
Are there any open issues in the current javascript main?
I don’t know how your testing turned out
Blake Niemyjski
@niemyjski
Feb 25 2016 19:31 UTC
very tired, slept on a couch last night
little neice is running around with so much energy, wish there was a way to extract it
actually yes, can you knock out something for me quick?
Frank Ebersoll
@frankebersoll
Feb 25 2016 19:31 UTC
I have accepted that being tired all the time
i can try
we changed SetManualStackingKey and also added SetManualStackingInfo
changed the data key name and moved things into a model
Eric J. Smith
@ejsmith
Feb 25 2016 19:33 UTC
hey @frankebersoll
Frank Ebersoll
@frankebersoll
Feb 25 2016 19:34 UTC
hey eric
@niemyjski i’m looking at it
Blake Niemyjski
@niemyjski
Feb 25 2016 19:35 UTC
Thanks, how about you?
Frank Ebersoll
@frankebersoll
Feb 25 2016 19:35 UTC
how about me?
Blake Niemyjski
@niemyjski
Feb 25 2016 19:37 UTC
yeah
hows your day going
Frank Ebersoll
@frankebersoll
Feb 25 2016 19:38 UTC
ah! better than expected.
Eric J. Smith
@ejsmith
Feb 25 2016 19:41 UTC
/all how do you guys feel about having a slack channel instead of using gitter?
Frank Ebersoll
@frankebersoll
Feb 25 2016 19:41 UTC
never used slack
Eric J. Smith
@ejsmith
Feb 25 2016 19:42 UTC
the nice thing about gitter is that it uses github as auth so people can jump right in.
Blake Niemyjski
@niemyjski
Feb 25 2016 19:42 UTC
yeah I like that too
and you can embed it into sites
Eric J. Smith
@ejsmith
Feb 25 2016 19:42 UTC
it is driving me nuts having to swap between 3 different chat apps.
Blake Niemyjski
@niemyjski
Feb 25 2016 19:42 UTC
and it’s picking up steam
lol
trillian :D
Eric J. Smith
@ejsmith
Feb 25 2016 19:43 UTC
2 different slack accounts, gitter and imessages
Bill Henning
@BillHenning
Feb 25 2016 19:43 UTC
yeah, i just joined a slack channel for a third party company and had to wait to be approved, etc. gitter is nicer with that aspect, but slack's desktop client is way better
Eric J. Smith
@ejsmith
Feb 25 2016 19:43 UTC
really really wish slack had public channels
Bill Henning
@BillHenning
Feb 25 2016 19:44 UTC
yup
if slack would allow you to tie your GitHub ID to their user accounts, you slack could easily provide more GitHub-focused public profile info
Eric J. Smith
@ejsmith
Feb 25 2016 19:48 UTC
yeah, they all suck
all require invite system
Blake Niemyjski
@niemyjski
Feb 25 2016 20:54 UTC
dang dang dang
got some of the dashboards updated to use our new stuff
wanted to do the main timeline but it uses term stats
sigh
can I catch a break
Frank Ebersoll
@frankebersoll
Feb 25 2016 21:17 UTC
SetManualStackingKey(title, key) and ..(key) is a little difficult to overload in javascript
is it okay if I’d make it (key, title?) ?
Eric J. Smith
@ejsmith
Feb 25 2016 21:38 UTC
@frankebersoll that is fine with me.
do we have one for (title, object) ?
where object could be several pieces that make up the signature data.
@niemyjski just found a nasty bug with the queue job… if you were using queue entry locks and it failed to get a lock, it wasn’t abandoning the queue item and so you’d end up with TONS of items in the work list and waiting to time out.
Blake Niemyjski
@niemyjski
Feb 25 2016 21:45 UTC
Dang
Unit tests for it?
Eric J. Smith
@ejsmith
Feb 25 2016 21:45 UTC
yeah, it was causing havok for is.
yeah, added some tests.
Frank Ebersoll
@frankebersoll
Feb 25 2016 22:09 UTC
same signature, different title. should that use the stack title of the first event with that signature in the UI?
Eric J. Smith
@ejsmith
Feb 25 2016 22:32 UTC
yeah, it will create the stack using the title given.
Frank Ebersoll
@frankebersoll
Feb 25 2016 22:33 UTC
i just found a problem that probably exists in node since I have joined the project
Eric J. Smith
@ejsmith
Feb 25 2016 22:33 UTC
doh
what’s going on?
Frank Ebersoll
@frankebersoll
Feb 25 2016 22:34 UTC
did the application ever quit? if we submit an event, it gets enqueued. the queue starts a timer. the timer continues forever
and the timer will block application exit.
[info] Exceptionless: Sent 1 events.
[info] Exceptionless: Finished processing queue.
[info] Exceptionless: Processing queue...
[info] Exceptionless: Processing queue...
[info] Exceptionless: Processing queue...
What we actually need is some kind of throttling, I think
Eric J. Smith
@ejsmith
Feb 25 2016 22:36 UTC
can we get any notification that the app is trying to exit?
Frank Ebersoll
@frankebersoll
Feb 25 2016 22:36 UTC
wait, I think we had that.
i don’t think so
the app isn’t trying to exit, actually...
index.js ends, but node can’t tell apart our timer from any other callback
for the runtime it’s just a bit of application code that is waiting in its event loop
Eric J. Smith
@ejsmith
Feb 25 2016 22:43 UTC
yeah, node isn’t my thing… not really sure how to handle that.
would have to start reading...
Frank Ebersoll
@frankebersoll
Feb 25 2016 22:44 UTC
we need a combination of timeouts and a „lastEventTimestamp“ field
it would queue up any events that come inside a short window, say 100 ms, up to the max batch size
that would mean: startup, 0 events, immediate shutdown
startup, 1 single event, it waits for other events for 200 ms, no events come in, submit, shutdown.
startup, multiple events inside the sliding 200 ms window, it submits them after the last event + 200 ms, shutdown
Eric J. Smith
@ejsmith
Feb 25 2016 22:48 UTC
yeah, think we just need to keep a handle to the timer and cancel it when the app is trying to shutdown?
Frank Ebersoll
@frankebersoll
Feb 25 2016 22:49 UTC
if the app doesn’t shutdown after the 200 ms (because it has work to do or is a server app) and we get events after sending the first batch, we would have to wait repeatedly until we get currentTime - lastSubmitTime > queueWaitTime
@ejsmith well, the app isn’t trying to shutdown. node will continue the process until there are 1) no pending timers and 2) no pending I/O operations
Eric J. Smith
@ejsmith
Feb 25 2016 22:51 UTC
yeah, but can we wire up to a spot where the code is done running and cancel the current timer?
I’m talking out of my ass here.
Frank Ebersoll
@frankebersoll
Feb 25 2016 22:51 UTC
that’s why we should only use timers if we have actual work to do (events in the queue)
Eric J. Smith
@ejsmith
Feb 25 2016 22:51 UTC
so what happens if we can’t send them to the server?
we will block forever?
Frank Ebersoll
@frankebersoll
Feb 25 2016 22:52 UTC
it looks like it, yes.
i’m not talking about Ctrl+C or SIGINT here, that’s working
i’m talking about „app has finished doing its work and should therefore exit"
currently, an app that does require(„exceptionless“) exits immediately. but one that does require(„exceptionless“).client.submitEvent(); will continue forever, if you don’t cancel it.
Eric J. Smith
@ejsmith
Feb 25 2016 22:56 UTC
ahh
so we start it when we get an event… and then if there are any events left...
otherwise its not going.
Frank Ebersoll
@frankebersoll
Feb 25 2016 22:58 UTC
yes
Eric J. Smith
@ejsmith
Feb 25 2016 22:58 UTC
that doesn’t help us with issues when the items can’t send for some reason and we need to retry later.
Frank Ebersoll
@frankebersoll
Feb 25 2016 22:58 UTC
yes. but how is that solved in .net?
Eric J. Smith
@ejsmith
Feb 25 2016 22:58 UTC
but if someone adds a new item then it can try again.
Frank Ebersoll
@frankebersoll
Feb 25 2016 22:58 UTC
say you add exceptionless to a console app
Eric J. Smith
@ejsmith
Feb 25 2016 22:58 UTC
in .net it’s a timer and it checks to see if anything is ready to be retried.
Frank Ebersoll
@frankebersoll
Feb 25 2016 22:58 UTC
will it quit, or will the queue block it?
timers in .net are on the ThreadPool
Eric J. Smith
@ejsmith
Feb 25 2016 22:59 UTC
actually… in a console app… you have to call ProcessQueue at the end
or it will leave items in the queue
Frank Ebersoll
@frankebersoll
Feb 25 2016 23:06 UTC
i will think about it. no idea yet :-|
ManualStackingInfo is checked in
Eric J. Smith
@ejsmith
Feb 25 2016 23:06 UTC
awesome
thanks man!
Frank Ebersoll
@frankebersoll
Feb 25 2016 23:07 UTC
my pleasure :-D