Monday, 2016-09-19

*** thorst has joined #openstack-watcher00:29
*** thorst has quit IRC00:37
*** thorst has joined #openstack-watcher01:14
*** thorst has quit IRC01:15
*** thorst has joined #openstack-watcher01:16
*** thorst has quit IRC01:24
*** thorst has joined #openstack-watcher02:22
*** thorst has quit IRC02:29
*** thorst has joined #openstack-watcher03:26
*** thorst has quit IRC03:34
*** jwcroppe has joined #openstack-watcher04:16
*** thorst has joined #openstack-watcher04:32
*** thorst has quit IRC04:39
*** thorst has joined #openstack-watcher05:37
*** thorst has quit IRC05:45
openstackgerritDavid TARDIVEL proposed openstack/watcher-dashboard: Add new audit fields in dashboard tables  https://review.openstack.org/37167206:23
*** dtardivel has joined #openstack-watcher06:24
sballe_Morning06:37
sballe_I just landed in PDX so I am not sure I'll make 5:30 pacific but I'll try06:39
sballe_dtardivel: acabot ^^^^^06:39
sballe_Is gfhelma online ?06:40
dtardivelsballe_: hi Susanne, he is not online for now06:41
*** thorst has joined #openstack-watcher06:42
sballe_Okey make sense but you'll be at 5:30 pacific to debug Monasca right? It is in about 5 hours 45 mins06:44
sballe_dtardivel:06:44
sballe_ dtardivel ^^^^06:46
dtardivelsballe_: Yes I will be connected :)06:47
sballe_Perfect06:47
*** thorst has quit IRC06:49
sballe_dtardivel: I am hoping that since Watcher has been kola-ized that install will be easy and that we will be able to start testing this week. Let's also talk ceilosca later today with gfhelma06:58
dtardivelsballe_: we did not been involved into kolla watcher integration ... I had a quick look on kolla commit related to watcher. There is no configuration file ... We will have to inject it manually ...07:05
*** diga has joined #openstack-watcher07:17
*** jwcroppe has quit IRC07:21
*** vincentfrancoise has joined #openstack-watcher07:35
openstackgerritDavid TARDIVEL proposed openstack/watcher-dashboard: interval param should be set to None by default  https://review.openstack.org/37226907:39
*** jwcroppe has joined #openstack-watcher07:41
*** jwcroppe_ has joined #openstack-watcher07:44
*** jwcroppe has quit IRC07:46
*** thorst has joined #openstack-watcher07:47
*** alexchadin has joined #openstack-watcher07:52
*** thorst has quit IRC07:55
alexchadinhi vincentfrancoise08:23
*** jwcroppe_ has quit IRC08:24
openstackgerritCarlos Camacho proposed openstack/puppet-watcher: Test latest puppet-lint  https://review.openstack.org/36969808:25
*** jwcroppe has joined #openstack-watcher08:32
openstackgerritDavid TARDIVEL proposed openstack/watcher-dashboard: interval param should be set to None by default  https://review.openstack.org/37226908:35
vincentfrancoisealexchadin: morning08:46
*** thorst has joined #openstack-watcher08:51
openstackgerritMerged openstack/watcher-dashboard: interval param should be set to None by default  https://review.openstack.org/37226908:54
openstackgerritVincent Fran├žoise proposed openstack/watcher: Test code tidy up  https://review.openstack.org/37232008:54
alexchadinvincentfrancoise: I have stange error with datetime_or_none function. When I pass utcnow() datetime to last_seen_up field, it doesn't have tzinfo, but then datetime_or_none adds iso8601.iso8601.Utc(). last_seen_up field can store only timestamp without timezone.08:56
alexchadinstrange*08:56
vincentfrancoisealexchadin: this will change soon with the versionedobjects BP that is currently in review08:58
alexchadin        if 'last_seen_up' in values:08:58
alexchadin            values['last_seen_up'] = values['last_seen_up'].replace(08:58
alexchadin                tzinfo=None)08:58
alexchadinI can fix it on db layer, when service info is updating -08:58
vincentfrancoiseso maybe your problem will be fixed straight away08:58
alexchadinI looked at oslo.versionedobjects code08:58
alexchadinwait a sec08:58
*** thorst has quit IRC08:58
alexchadinseems that it does the same thing08:59
alexchadinhttps://github.com/openstack/oslo.versionedobjects/blob/master/oslo_versionedobjects/fields.py#L45908:59
vincentfrancoisealexchadin: BTW, why do you want to avoid remove the timezone?09:02
alexchadinActually, I don't want to have timezone in DB09:02
alexchadinIt should be in UTC format09:03
alexchadincause all created_at, deleted_at, updated_at fields are stored without timezone09:03
alexchadinoh09:04
alexchadinhm09:04
alexchadinoslo.versionedobjects set tzinfo_aware=True by default09:04
alexchadinIt fits my needs:)09:05
vincentfrancoiseyes but with my oslo.versionedobjects refactoring, we'll have these fields as ovo_fields.DateTimeField(nullable=True)09:05
vincentfrancoiseso I guess that means they'll have a timezone09:05
vincentfrancoisesee https://review.openstack.org/#/c/359065/8/watcher/objects/base.py09:06
alexchadinhttps://github.com/openstack/oslo.versionedobjects/blob/master/oslo_versionedobjects/fields.py#L104709:06
alexchadinShould it?09:06
alexchadinLooks like it also has tzinfo_aware=True by default09:06
vincentfrancoiseIMHO it's better to add the timezone09:06
*** edleafe has quit IRC09:07
*** edleafe has joined #openstack-watcher09:07
alexchadinall the projects I've met store their datetime fields without tz09:07
vincentfrancoisenova seems to be use timezones: https://github.com/openstack/nova/blob/master/nova/objects/base.py#L205-L20609:08
alexchadinit doesn't overwrite tzinfo_aware field09:10
alexchadinSo nova awares tzinfo by default09:10
vincentfrancoiseseems like it, yes09:11
vincentfrancoiseto keep it consistent09:12
vincentfrancoisedo it without the timezone09:12
vincentfrancoiseand when my OVO refactoring gets merged it will add TZ for all datetime fields09:12
vincentfrancoisewhat do you think?09:13
alexchadincould you show me where it adds tz to datetime?09:13
vincentfrancoisealexchadin: not sure I understand what you want to see: isn't it the link you sent me with the tz_aware thing that sets it in oslo.versionedobjects lib?09:15
*** jwcroppe has quit IRC09:16
alexchadinWe have found out that oslo.versionedobjects lib is set tz_aware argument to True by default. You say that your patch will add TZ for all datetime field so it should mean that tz_aware field is somewhere overwritten to False. I'd like to see this place09:18
vincentfrancoiseAh I see: we currently do not rely on OVO, that's why it doesn't have any TZ09:20
*** jwcroppe has joined #openstack-watcher09:23
alexchadinvincentfrancoise: I have sent me the link to objects/base.py from your PS and on 101-103 lines the same situation as in the Nova09:24
alexchadinYou*09:24
vincentfrancoisealexchadin: in my refatoring you have the datetime field that is a standard OVO datetime field (https://review.openstack.org/#/c/359065/8/watcher/objects/fields.py@30)09:26
alexchadinvincentfrancoise: yes, that's great:) It looks like we won't have tz09:26
vincentfrancoisealexchadin: we will have TZ -> because the standard OVO datetime field is the same as the link you sent earlier: https://github.com/openstack/oslo.versionedobjects/blob/master/oslo_versionedobjects/fields.py#L45909:29
alexchadinvincentfrancoise: yeah, my fault09:32
openstackgerritVincent Fran├žoise proposed openstack/watcher: Added Tempest API tests for /scoring_engines  https://review.openstack.org/37155809:32
alexchadinsqlalchemy creates datetime field without timezone during initialization of watcher db09:37
*** thorst has joined #openstack-watcher10:08
alexchadinvincentfrancoise: for datetime field we should use dt_serializer10:08
alexchadinvincentfrancoise: to convert datetime to str10:08
alexchadinvincentfrancoise: so many problems with this dt...10:09
*** jwcroppe has quit IRC10:10
vincentfrancoisealexchadin: I didn't have any datetime issue up until now, so show me what you changed at the end10:12
alexchadinYou will see it when you get invite to review ;P10:13
vincentfrancoisealexchadin: that was my point :p10:14
*** jwcroppe has joined #openstack-watcher10:14
*** thorst has quit IRC10:35
*** jwcroppe has quit IRC10:55
*** jwcroppe has joined #openstack-watcher11:02
*** alexchadin has quit IRC11:22
openstackgerritavnish proposed openstack/python-watcherclient: Update home page link in cfg file  https://review.openstack.org/37242611:32
*** jwcroppe has quit IRC11:41
*** thorst has joined #openstack-watcher11:42
*** thorst has quit IRC11:46
*** jwcroppe has joined #openstack-watcher11:49
*** alexchadin has joined #openstack-watcher12:04
*** jwcroppe has quit IRC12:07
*** jwcroppe has joined #openstack-watcher12:08
*** jwcroppe has quit IRC12:26
sballe_Morning12:27
*** alexchadin has quit IRC12:27
*** alexchadin has joined #openstack-watcher12:28
*** jwcroppe has joined #openstack-watcher12:31
sballe_gfhelma are you here12:33
acabotMorning12:35
sballe_Hi12:35
openstackgerritAntoine Cabot proposed openstack/watcher: Update Watcher description  https://review.openstack.org/37246813:01
*** alexchadin has quit IRC13:04
*** alexchadin has joined #openstack-watcher13:05
openstackgerritMerged openstack/watcher: Update reno for stable/newton  https://review.openstack.org/37151813:11
*** jwcroppe has quit IRC13:12
sballe_Gfhelma lets talk in this channel13:14
*** gfhellma has joined #openstack-watcher13:14
gfhellmaHello13:15
dtardivelgfhellma: Hi13:15
sballe_Good we are all here :-)13:15
gfhellmaI did the manual install within the Horizon Docker Container13:16
dtardivelgfhellma: How are you ? Could you give us a status about the Monasca deployment and Watcher ?13:16
gfhellmainstall on monasc-ui13:17
*** thorst_ has joined #openstack-watcher13:17
gfhellmathe monasca install on the host we call 4.6 went ok as there are no Docker Containers on that host13:17
gfhellmajust a notmal install13:17
dtardivelgfhellma: Do you access now onto the 'Monitoring' Panel on Horizon ?13:18
gfhellmaThe lasy ichecked that was not showing up mainly because of th config issue that mentioned13:18
gfhellmaApache with the docker container is et up differently than on anormal howizon apache host13:19
gfhellmathe main difference is that on a regular host horizon is in the sites13:19
gfhellmafolder13:20
gfhellmawithin the docker container it is folded into the 000-default conf file13:20
dtardivelgfhellma: which distrib do you use for horion ? Ubuntu ? CentOS ?13:21
gfhellmai have to pull that apart so that i can include the monasca-ui without over-riding the base Horizon install.13:21
gfhellmathese hosts have been done with ubuntu 14.0413:21
*** thorst_ has quit IRC13:24
gfhellmadtardvel: can you meet me on a 1:113:25
*** michaelgugino has joined #openstack-watcher13:30
openstackgerritMerged openstack/watcher: Update Watcher description  https://review.openstack.org/37246813:33
*** gfhellma has quit IRC14:04
*** thorst_ has joined #openstack-watcher14:13
openstackgerritAntoine Cabot proposed openstack/watcher-specs: Add team priorities for Ocata  https://review.openstack.org/37252814:30
*** thorst_ has quit IRC14:33
*** thorst_ has joined #openstack-watcher14:34
*** hvprash_ has joined #openstack-watcher14:35
*** openstackgerrit has quit IRC14:37
*** openstackgerrit has joined #openstack-watcher14:37
*** hvprash_ has quit IRC14:42
*** thorst_ has quit IRC14:42
*** thorst_ has joined #openstack-watcher14:43
*** hvprash1 has joined #openstack-watcher14:51
*** tri2sing has joined #openstack-watcher14:51
openstackgerritDavid TARDIVEL proposed openstack/watcher-dashboard: Update table's actions  https://review.openstack.org/37256015:05
openstackgerritAlexander Chadin proposed openstack/watcher: [WIP] Add service supervisor  https://review.openstack.org/37163215:07
alexchadinvincentfrancoise: take a look https://review.openstack.org/#/c/371632/2/watcher/objects/base.py15:08
alexchadinvincentfrancoise: https://review.openstack.org/#/c/371632/2/watcher/objects/service.py line 3515:09
alexchadinvincentfrancoise: That's how issue we have spoken about was resolved.15:09
vincentfrancoisealexchadin: FYI to jump to a line in gerrit links you can add an '@' like https://review.openstack.org/#/c/371632/2/watcher/objects/service.py@35 in your case15:11
alexchadinah, thanks15:11
alexchadindidn't know about it15:11
vincentfrancoisealexchadin: the _attr_last_seen_up_to_primitive is kind of ugly but I don't see any better solution...15:14
alexchadinThe mechanism is ugly - to store datetime it converts utc to iso8601 (just adds +00:00) which will be converted to string and saved as string15:15
alexchadincreated_at, updated_at, deleted_at used the same way15:16
*** alexchadin has quit IRC15:24
*** alexchadin has joined #openstack-watcher15:24
openstackgerritAntoine Cabot proposed openstack/watcher-specs: Add team priorities for Ocata  https://review.openstack.org/37252815:26
*** thorst_ has quit IRC15:31
*** thorst_ has joined #openstack-watcher15:31
*** alexchadin has quit IRC15:32
openstackgerritDavid TARDIVEL proposed openstack/watcher-dashboard: Clean the code by renaming 'type' parameters  https://review.openstack.org/37258315:32
sballe_dtardivel: where you and gfhellman able to get things working15:50
vincentfrancoisesballe_: dtardivel already left the office15:52
vincentfrancoisesballe_: I was following from afar but seems like Gustav wasn't stuck anymor15:56
sballe_Cool thx15:56
sballe_vincentfrancoise: did you or dtardivel give him the watcher conf file he needs if he deploys watcher with kola15:59
*** tri2sing has quit IRC16:00
sballe_Gfhellman how are you doing? Moving forward installing the next service?16:00
*** acabot has quit IRC16:00
vincentfrancoisesballe_: I don't know about what they did other than their conversation logs16:02
*** thorst_ has quit IRC16:18
*** vincentfrancoise has quit IRC16:18
*** thorst_ has joined #openstack-watcher16:18
*** thorst_ has quit IRC16:25
*** thorst_ has joined #openstack-watcher16:26
*** thorst_ has quit IRC16:41
*** thorst_ has joined #openstack-watcher16:48
*** diga has quit IRC17:02
*** thorst_ has quit IRC17:05
*** gfhellma has joined #openstack-watcher17:10
*** thorst_ has joined #openstack-watcher17:11
*** thorst_ has quit IRC17:16
*** tri2sing has joined #openstack-watcher17:27
*** harlowja has joined #openstack-watcher17:30
*** jwcroppe has joined #openstack-watcher17:51
*** jwcroppe_ has joined #openstack-watcher17:52
*** jwcropp__ has joined #openstack-watcher17:53
*** jwcroppe has quit IRC17:56
*** jwcroppe_ has quit IRC17:57
*** jwcropp__ has quit IRC18:43
*** dtardivel has quit IRC18:57
*** jwcroppe has joined #openstack-watcher19:13
*** jwcroppe has quit IRC19:43
*** jwcroppe has joined #openstack-watcher19:49
*** thorst_ has joined #openstack-watcher19:54
*** thorst_ has quit IRC20:10
*** gfhellma has quit IRC20:13
*** thorst_ has joined #openstack-watcher20:25
*** gfhellma has joined #openstack-watcher20:28
*** jwcroppe has quit IRC20:44
*** thorst_ has quit IRC20:54
*** jwcroppe has joined #openstack-watcher20:55
*** jwcroppe has quit IRC21:11
*** jwcroppe has joined #openstack-watcher21:36
*** jwcroppe has quit IRC21:48
*** harlowja has quit IRC21:51
*** michaelgugino has quit IRC21:52
*** harlowja has joined #openstack-watcher21:56
*** jwcroppe has joined #openstack-watcher22:06
*** tri2sing has quit IRC22:26
*** jwcroppe has quit IRC22:40
*** jwcroppe has joined #openstack-watcher22:51
*** jwcroppe has quit IRC22:56
*** jwcroppe has joined #openstack-watcher23:06
*** jwcroppe has quit IRC23:11
*** thorst_ has joined #openstack-watcher23:14
*** gfhellma has quit IRC23:24
*** thorst_ has quit IRC23:25
*** gfhellma has joined #openstack-watcher23:26
*** jwcroppe has joined #openstack-watcher23:29
*** jwcroppe has quit IRC23:41
*** openstackgerrit has quit IRC23:42
*** openstackgerrit has joined #openstack-watcher23:46
*** jwcroppe has joined #openstack-watcher23:48
*** openstackgerrit has quit IRC23:51
*** openstackgerrit has joined #openstack-watcher23:54
*** jwcroppe has quit IRC23:57

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