These are chat archives for exceptionless/Discuss

25th
Feb 2016
Eric J. Smith
@ejsmith
Feb 25 2016 03:55
aspnet/Logging#370
Really want them to separate their logging extensions into a separate project.
Blake Niemyjski
@niemyjski
Feb 25 2016 04:18
Seems they don't care
benmaina
@benmaina
Feb 25 2016 05:26
@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
woke up to this
blob
Jamie Clarke
@jamie94bc
Feb 25 2016 08:50
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
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
@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
Yup
Wonder if there are any azure templates / dicker images for this
Eric J. Smith
@ejsmith
Feb 25 2016 13:08
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
oh crap, we use mandrill too
Blake Niemyjski
@niemyjski
Feb 25 2016 13:12
Was just thinking what are we going to do for logging and the exceptionless client
Eric J. Smith
@ejsmith
Feb 25 2016 13:15
What do you mean @niemyjski ?
Blake Niemyjski
@niemyjski
Feb 25 2016 13:26
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
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
@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
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
Yeah
Eric J. Smith
@ejsmith
Feb 25 2016 13:31
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
that's strange!
Eric J. Smith
@ejsmith
Feb 25 2016 13:35
Yeah, our tests should probably be setup in a different way. Async testing is hard.
Blake Niemyjski
@niemyjski
Feb 25 2016 13:54
We should test the azure service bus too
Those are Bing skipped
Eric J. Smith
@ejsmith
Feb 25 2016 13:54
How?
I'm not paying for a servicebus account to test against.
Blake Niemyjski
@niemyjski
Feb 25 2016 13:56
We already are paying for a test one
Isn't it usage based
Eric J. Smith
@ejsmith
Feb 25 2016 13:58
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
Did you see that MS released an open source client for Kafka?
Blake Niemyjski
@niemyjski
Feb 25 2016 14:06
Cool
Blake Niemyjski
@niemyjski
Feb 25 2016 15:06
@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
@niemyjski we should do a blog post covering the usage chart you added
Blake Niemyjski
@niemyjski
Feb 25 2016 15:17
yeah
Blake Niemyjski
@niemyjski
Feb 25 2016 16:43
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
its for creating a tile
that is still a thing.
Blake Niemyjski
@niemyjski
Feb 25 2016 16:44
ok
Bill Henning
@BillHenning
Feb 25 2016 16:44
or you guys could just make a windows store wrapper web-based app for exceptionless :)
Blake Niemyjski
@niemyjski
Feb 25 2016 16:45
yeah
that would be a great pull request @BillHenning
Eric J. Smith
@ejsmith
Feb 25 2016 16:48
just tried it and it didn’t seem to respect any of those
Blake Niemyjski
@niemyjski
Feb 25 2016 16:48
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
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
yeah
Eric J. Smith
@ejsmith
Feb 25 2016 17:31
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
ok
Eric J. Smith
@ejsmith
Feb 25 2016 17:41
SyntaxEditor on Xbox!
Bill Henning
@BillHenning
Feb 25 2016 17:41
Code Writer on xbox!
Eric J. Smith
@ejsmith
Feb 25 2016 17:41
hehe
Blake Niemyjski
@niemyjski
Feb 25 2016 18:02
@ejsmith you got a big problem with the latest foundatio build
Eric J. Smith
@ejsmith
Feb 25 2016 18:02
what is that?
Blake Niemyjski
@niemyjski
Feb 25 2016 18:02
you started targeting 4.5.1 I think?
Eric J. Smith
@ejsmith
Feb 25 2016 18:02
yeah
why?
Blake Niemyjski
@niemyjski
Feb 25 2016 18:02
lib/net45 contains a single pdb and then /net451 contains a dll
Eric J. Smith
@ejsmith
Feb 25 2016 18:03
ahh
Blake Niemyjski
@niemyjski
Feb 25 2016 18:03
so when you upgrade a 4.5.0 package it actually gets uninstallled
Eric J. Smith
@ejsmith
Feb 25 2016 18:03
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
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
since I’m not using their package
Blake Niemyjski
@niemyjski
Feb 25 2016 18:03
ok
Eric J. Smith
@ejsmith
Feb 25 2016 18:03
I guess we can change to 4.5.2
Frank Ebersoll
@frankebersoll
Feb 25 2016 19:29
Hey there :-)
Blake Niemyjski
@niemyjski
Feb 25 2016 19:29
hey
Frank Ebersoll
@frankebersoll
Feb 25 2016 19:29
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
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
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
hey @frankebersoll
Frank Ebersoll
@frankebersoll
Feb 25 2016 19:34
hey eric
@niemyjski i’m looking at it
Blake Niemyjski
@niemyjski
Feb 25 2016 19:35
Thanks, how about you?
Frank Ebersoll
@frankebersoll
Feb 25 2016 19:35
how about me?
Blake Niemyjski
@niemyjski
Feb 25 2016 19:37
yeah
hows your day going
Frank Ebersoll
@frankebersoll
Feb 25 2016 19:38
ah! better than expected.
Eric J. Smith
@ejsmith
Feb 25 2016 19:41
/all how do you guys feel about having a slack channel instead of using gitter?
Frank Ebersoll
@frankebersoll
Feb 25 2016 19:41
never used slack
Eric J. Smith
@ejsmith
Feb 25 2016 19:42
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
yeah I like that too
and you can embed it into sites
Eric J. Smith
@ejsmith
Feb 25 2016 19:42
it is driving me nuts having to swap between 3 different chat apps.
Blake Niemyjski
@niemyjski
Feb 25 2016 19:42
and it’s picking up steam
lol
trillian :D
Eric J. Smith
@ejsmith
Feb 25 2016 19:43
2 different slack accounts, gitter and imessages
Bill Henning
@BillHenning
Feb 25 2016 19:43
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
really really wish slack had public channels
Bill Henning
@BillHenning
Feb 25 2016 19:44
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
yeah, they all suck
all require invite system
Blake Niemyjski
@niemyjski
Feb 25 2016 20:54
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
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
@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
Dang
Unit tests for it?
Eric J. Smith
@ejsmith
Feb 25 2016 21:45
yeah, it was causing havok for is.
yeah, added some tests.
Frank Ebersoll
@frankebersoll
Feb 25 2016 22:09
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
yeah, it will create the stack using the title given.
Frank Ebersoll
@frankebersoll
Feb 25 2016 22:33
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
doh
what’s going on?
Frank Ebersoll
@frankebersoll
Feb 25 2016 22:34
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
can we get any notification that the app is trying to exit?
Frank Ebersoll
@frankebersoll
Feb 25 2016 22:36
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
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
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
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
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
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
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
so what happens if we can’t send them to the server?
we will block forever?
Frank Ebersoll
@frankebersoll
Feb 25 2016 22:52
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
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
yes
Eric J. Smith
@ejsmith
Feb 25 2016 22:58
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
yes. but how is that solved in .net?
Eric J. Smith
@ejsmith
Feb 25 2016 22:58
but if someone adds a new item then it can try again.
Frank Ebersoll
@frankebersoll
Feb 25 2016 22:58
say you add exceptionless to a console app
Eric J. Smith
@ejsmith
Feb 25 2016 22:58
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
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
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
i will think about it. no idea yet :-|
ManualStackingInfo is checked in
Eric J. Smith
@ejsmith
Feb 25 2016 23:06
awesome
thanks man!
Frank Ebersoll
@frankebersoll
Feb 25 2016 23:07
my pleasure :-D