Thursday, 2017-01-05

*** thorst_ has quit IRC00:10
*** absubram has quit IRC00:20
*** jwcroppe has quit IRC00:27
*** jwcroppe has joined #openstack-watcher00:40
*** jwcroppe has quit IRC00:45
*** thorst_ has joined #openstack-watcher00:54
*** hidekazu has joined #openstack-watcher01:28
*** openstackgerrit has joined #openstack-watcher01:40
openstackgerritzhurong proposed openstack/watcher-dashboard: Add page title for some panel table  https://review.openstack.org/41679801:40
*** jwcroppe has joined #openstack-watcher01:47
*** jwcroppe has quit IRC02:14
*** harlowja has quit IRC03:05
*** thorst_ has quit IRC03:14
*** thorst_ has joined #openstack-watcher03:15
*** thorst_ has quit IRC03:19
*** thorst_ has joined #openstack-watcher03:25
*** thorst_ has quit IRC03:25
*** jwcroppe has joined #openstack-watcher03:54
*** thorst_ has joined #openstack-watcher04:01
*** thorst_ has quit IRC04:01
*** jwcroppe has quit IRC04:05
*** jwcroppe has joined #openstack-watcher05:43
*** jwcroppe has quit IRC05:54
*** thorst_ has joined #openstack-watcher06:02
*** thorst_ has quit IRC06:07
*** pksingh has joined #openstack-watcher06:18
*** pksingh has quit IRC06:19
*** thorst_ has joined #openstack-watcher06:28
*** thorst_ has quit IRC06:33
*** diga has joined #openstack-watcher06:45
*** jwcroppe has joined #openstack-watcher07:33
*** jwcroppe has quit IRC07:44
*** openstackgerrit has quit IRC07:50
*** Yumeng has joined #openstack-watcher08:04
*** zhurong has joined #openstack-watcher08:24
*** jwcroppe has joined #openstack-watcher08:28
*** thorst_ has joined #openstack-watcher08:29
*** thorst_ has quit IRC08:34
*** jwcroppe has quit IRC08:37
*** jwcroppe has joined #openstack-watcher09:22
*** jwcroppe has quit IRC09:33
*** diga has quit IRC09:36
*** zhurong has quit IRC09:40
*** openstackgerrit has joined #openstack-watcher10:04
openstackgerritMerged openstack/watcher-dashboard: Add reno for release notes management  https://review.openstack.org/41641910:04
*** openstack has joined #openstack-watcher10:14
*** jwcroppe has joined #openstack-watcher10:17
*** jwcroppe has quit IRC10:27
*** thorst_ has joined #openstack-watcher10:30
*** thorst_ has quit IRC10:35
*** vincentfrancoise has quit IRC10:55
*** diga has joined #openstack-watcher11:16
*** diga has quit IRC11:37
*** grumpycatt has quit IRC11:55
*** diga has joined #openstack-watcher12:00
*** jwcroppe has joined #openstack-watcher12:07
*** jwcroppe has quit IRC12:15
*** diga_ has joined #openstack-watcher12:21
*** diga has quit IRC12:23
*** thorst_ has joined #openstack-watcher12:46
*** Yumeng has left #openstack-watcher12:52
*** hidekazu has quit IRC12:55
*** vincentfrancoise has joined #openstack-watcher13:06
*** dtardivel has joined #openstack-watcher13:56
*** jwcroppe has joined #openstack-watcher13:56
*** jwcroppe has quit IRC14:08
*** pkoniszewski has quit IRC14:15
*** pkoniszewski has joined #openstack-watcher14:15
*** lamt has joined #openstack-watcher14:43
*** thorst_ is now known as thorst_afk14:50
*** jwcroppe has joined #openstack-watcher14:51
*** absubram has joined #openstack-watcher14:58
*** jwcroppe has quit IRC15:03
*** d34dh0r53 has quit IRC15:14
*** d34dh0r53 has joined #openstack-watcher15:14
*** lamt has quit IRC15:15
*** lamt has joined #openstack-watcher15:16
*** diga_ has quit IRC15:18
*** jwcroppe has joined #openstack-watcher15:22
*** Yumeng_ has joined #openstack-watcher15:31
*** thorst_afk is now known as thorst_15:53
*** Yumeng_ has quit IRC15:59
chrisspencersballe_: we are all setup on the systems16:00
sballe_chrisspencer: this is very cool! incluign collectd?16:01
sballe_can you check we are getting the right metrics like ll3_misses etc16:01
chrisspencerunfortunately we are blocked on that right now.  I have the request out to get access to the repo.16:01
chrisspencerI'll send an email to those guys to try and expedite the request16:02
*** Yumeng__ has joined #openstack-watcher16:09
vincentfrancoisehi sballe_16:49
vincentfrancoisedid you receive my bug suggestion last time?16:50
sballe_yes chrisspencer decided he wanted to work on it16:50
sballe_thx a lot for the suggesttion16:50
chrisspencervincentfrancoise: I just started looking into it16:50
vincentfrancoiseso just as an FYI16:51
vincentfrancoisethere's someone else that talked about this bug with me yesterday (let me find out who it was :p)16:51
chrisspencervincentfrancoise: ack.  is that someone else working on it now?16:56
vincentfrancoiseI don't know, did you assign it to yourself on launchpad?16:57
vincentfrancoise(can't find outwho it was)16:58
chrisspencernot yet, still unassigned16:58
chrisspencernow it is :)16:58
vincentfrancoisecan you send me the link pls?16:58
chrisspencerhttps://bugs.launchpad.net/watcher/+bug/164262316:58
openstackLaunchpad bug 1642623 in watcher "Add resource ID in nova notification logs " [Wishlist,Triaged] - Assigned to Chris Spencer (christopher-m-spencer)16:58
vincentfrancoisethere you go that was atuly that was talking with me about it16:59
vincentfrancoisehe wrote a comment on the bug BTW16:59
vincentfrancoisechrisspencer: do you understand what has to be done?17:02
vincentfrancoiseI'm about to leave so that would be the perfect moment to ask :)17:02
chrisspenceryes, I saw that comment.  Is his comment the fix? from the description it seemed like we wanted a more generic solution at a higher level. like for all NotificationEndpoints?17:02
vincentfrancoiseat the moment, we already have logs for the notifications17:02
vincentfrancoisebut since we receive the notifications concurrently17:03
chrisspenceralso notification IDs and messageIDs, in the code are the the publisher ID and instance ID?17:03
vincentfrancoisethe logs are unreadable because we don't know from which initial event/notification each line of log we see comes from17:03
chrisspencermakes sense17:04
vincentfrancoiseso when you see this log https://github.com/openstack/watcher/blob/master/watcher/decision_engine/model/notification/nova.py#L346-L35017:04
vincentfrancoiseyou have the metadata and you can track the initial event17:05
vincentfrancoisehowever when we handle the notification and get to https://github.com/openstack/watcher/blob/master/watcher/decision_engine/model/notification/nova.py#L4817:05
vincentfrancoisewe can't tell the initial message ID anymore17:06
vincentfrancoiseso the logger should be somehow aware of the message ID it references17:06
chrisspenceralright17:06
vincentfrancoiseand when I say generic17:07
vincentfrancoisethe idea is to make it the least intrusive we can17:07
vincentfrancoisebecause if it's well made, we may as well reuse the mechanism to do the logging of all event handlers17:08
chrisspencer+1  will start working on that17:09
chrisspencerthanks vincentfrancoise17:09
vincentfrancoiseis that ok for you? It's not the easiest starting point, but at least you don't really add a new feature so can be sure you won't break anything and you probably won't need to write unit tests17:10
chrisspenceryup that works, sounds like a good way to dig in!17:11
vincentfrancoiseemail/ping me if needed ;)17:11
vincentfrancoisebye all :)17:11
*** vincentfrancoise has quit IRC17:15
*** dtardivel has quit IRC19:04
*** harlowja has joined #openstack-watcher19:16
*** Yumeng__ has quit IRC19:31
*** thorst_ has quit IRC22:04
*** absubram has quit IRC23:28
*** jwcroppe has quit IRC23:58
*** jwcroppe has joined #openstack-watcher23:59

Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!