| *** thorst has joined #openstack-watcher | 00:29 | |
| *** thorst has quit IRC | 00:37 | |
| *** thorst has joined #openstack-watcher | 01:14 | |
| *** thorst has quit IRC | 01:15 | |
| *** thorst has joined #openstack-watcher | 01:16 | |
| *** thorst has quit IRC | 01:24 | |
| *** thorst has joined #openstack-watcher | 02:22 | |
| *** thorst has quit IRC | 02:29 | |
| *** thorst has joined #openstack-watcher | 03:26 | |
| *** thorst has quit IRC | 03:34 | |
| *** jwcroppe has joined #openstack-watcher | 04:16 | |
| *** thorst has joined #openstack-watcher | 04:32 | |
| *** thorst has quit IRC | 04:39 | |
| *** thorst has joined #openstack-watcher | 05:37 | |
| *** thorst has quit IRC | 05:45 | |
| openstackgerrit | David TARDIVEL proposed openstack/watcher-dashboard: Add new audit fields in dashboard tables https://review.openstack.org/371672 | 06:23 |
|---|---|---|
| *** dtardivel has joined #openstack-watcher | 06:24 | |
| sballe_ | Morning | 06:37 |
| sballe_ | I just landed in PDX so I am not sure I'll make 5:30 pacific but I'll try | 06:39 |
| sballe_ | dtardivel: acabot ^^^^^ | 06:39 |
| sballe_ | Is gfhelma online ? | 06:40 |
| dtardivel | sballe_: hi Susanne, he is not online for now | 06:41 |
| *** thorst has joined #openstack-watcher | 06:42 | |
| sballe_ | Okey make sense but you'll be at 5:30 pacific to debug Monasca right? It is in about 5 hours 45 mins | 06:44 |
| sballe_ | dtardivel: | 06:44 |
| sballe_ | dtardivel ^^^^ | 06:46 |
| dtardivel | sballe_: Yes I will be connected :) | 06:47 |
| sballe_ | Perfect | 06:47 |
| *** thorst has quit IRC | 06: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 gfhelma | 06:58 |
| dtardivel | sballe_: 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-watcher | 07:17 | |
| *** jwcroppe has quit IRC | 07:21 | |
| *** vincentfrancoise has joined #openstack-watcher | 07:35 | |
| openstackgerrit | David TARDIVEL proposed openstack/watcher-dashboard: interval param should be set to None by default https://review.openstack.org/372269 | 07:39 |
| *** jwcroppe has joined #openstack-watcher | 07:41 | |
| *** jwcroppe_ has joined #openstack-watcher | 07:44 | |
| *** jwcroppe has quit IRC | 07:46 | |
| *** thorst has joined #openstack-watcher | 07:47 | |
| *** alexchadin has joined #openstack-watcher | 07:52 | |
| *** thorst has quit IRC | 07:55 | |
| alexchadin | hi vincentfrancoise | 08:23 |
| *** jwcroppe_ has quit IRC | 08:24 | |
| openstackgerrit | Carlos Camacho proposed openstack/puppet-watcher: Test latest puppet-lint https://review.openstack.org/369698 | 08:25 |
| *** jwcroppe has joined #openstack-watcher | 08:32 | |
| openstackgerrit | David TARDIVEL proposed openstack/watcher-dashboard: interval param should be set to None by default https://review.openstack.org/372269 | 08:35 |
| vincentfrancoise | alexchadin: morning | 08:46 |
| *** thorst has joined #openstack-watcher | 08:51 | |
| openstackgerrit | Merged openstack/watcher-dashboard: interval param should be set to None by default https://review.openstack.org/372269 | 08:54 |
| openstackgerrit | Vincent Françoise proposed openstack/watcher: Test code tidy up https://review.openstack.org/372320 | 08:54 |
| alexchadin | vincentfrancoise: 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 |
| alexchadin | strange* | 08:56 |
| vincentfrancoise | alexchadin: this will change soon with the versionedobjects BP that is currently in review | 08: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 |
| alexchadin | I can fix it on db layer, when service info is updating - | 08:58 |
| vincentfrancoise | so maybe your problem will be fixed straight away | 08:58 |
| alexchadin | I looked at oslo.versionedobjects code | 08:58 |
| alexchadin | wait a sec | 08:58 |
| *** thorst has quit IRC | 08:58 | |
| alexchadin | seems that it does the same thing | 08:59 |
| alexchadin | https://github.com/openstack/oslo.versionedobjects/blob/master/oslo_versionedobjects/fields.py#L459 | 08:59 |
| vincentfrancoise | alexchadin: BTW, why do you want to avoid remove the timezone? | 09:02 |
| alexchadin | Actually, I don't want to have timezone in DB | 09:02 |
| alexchadin | It should be in UTC format | 09:03 |
| alexchadin | cause all created_at, deleted_at, updated_at fields are stored without timezone | 09:03 |
| alexchadin | oh | 09:04 |
| alexchadin | hm | 09:04 |
| alexchadin | oslo.versionedobjects set tzinfo_aware=True by default | 09:04 |
| alexchadin | It fits my needs:) | 09:05 |
| vincentfrancoise | yes but with my oslo.versionedobjects refactoring, we'll have these fields as ovo_fields.DateTimeField(nullable=True) | 09:05 |
| vincentfrancoise | so I guess that means they'll have a timezone | 09:05 |
| vincentfrancoise | see https://review.openstack.org/#/c/359065/8/watcher/objects/base.py | 09:06 |
| alexchadin | https://github.com/openstack/oslo.versionedobjects/blob/master/oslo_versionedobjects/fields.py#L1047 | 09:06 |
| alexchadin | Should it? | 09:06 |
| alexchadin | Looks like it also has tzinfo_aware=True by default | 09:06 |
| vincentfrancoise | IMHO it's better to add the timezone | 09:06 |
| *** edleafe has quit IRC | 09:07 | |
| *** edleafe has joined #openstack-watcher | 09:07 | |
| alexchadin | all the projects I've met store their datetime fields without tz | 09:07 |
| vincentfrancoise | nova seems to be use timezones: https://github.com/openstack/nova/blob/master/nova/objects/base.py#L205-L206 | 09:08 |
| alexchadin | it doesn't overwrite tzinfo_aware field | 09:10 |
| alexchadin | So nova awares tzinfo by default | 09:10 |
| vincentfrancoise | seems like it, yes | 09:11 |
| vincentfrancoise | to keep it consistent | 09:12 |
| vincentfrancoise | do it without the timezone | 09:12 |
| vincentfrancoise | and when my OVO refactoring gets merged it will add TZ for all datetime fields | 09:12 |
| vincentfrancoise | what do you think? | 09:13 |
| alexchadin | could you show me where it adds tz to datetime? | 09:13 |
| vincentfrancoise | alexchadin: 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 IRC | 09:16 | |
| alexchadin | We 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 place | 09:18 |
| vincentfrancoise | Ah I see: we currently do not rely on OVO, that's why it doesn't have any TZ | 09:20 |
| *** jwcroppe has joined #openstack-watcher | 09:23 | |
| alexchadin | vincentfrancoise: I have sent me the link to objects/base.py from your PS and on 101-103 lines the same situation as in the Nova | 09:24 |
| alexchadin | You* | 09:24 |
| vincentfrancoise | alexchadin: 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 |
| alexchadin | vincentfrancoise: yes, that's great:) It looks like we won't have tz | 09:26 |
| vincentfrancoise | alexchadin: 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#L459 | 09:29 |
| alexchadin | vincentfrancoise: yeah, my fault | 09:32 |
| openstackgerrit | Vincent Françoise proposed openstack/watcher: Added Tempest API tests for /scoring_engines https://review.openstack.org/371558 | 09:32 |
| alexchadin | sqlalchemy creates datetime field without timezone during initialization of watcher db | 09:37 |
| *** thorst has joined #openstack-watcher | 10:08 | |
| alexchadin | vincentfrancoise: for datetime field we should use dt_serializer | 10:08 |
| alexchadin | vincentfrancoise: to convert datetime to str | 10:08 |
| alexchadin | vincentfrancoise: so many problems with this dt... | 10:09 |
| *** jwcroppe has quit IRC | 10:10 | |
| vincentfrancoise | alexchadin: I didn't have any datetime issue up until now, so show me what you changed at the end | 10:12 |
| alexchadin | You will see it when you get invite to review ;P | 10:13 |
| vincentfrancoise | alexchadin: that was my point :p | 10:14 |
| *** jwcroppe has joined #openstack-watcher | 10:14 | |
| *** thorst has quit IRC | 10:35 | |
| *** jwcroppe has quit IRC | 10:55 | |
| *** jwcroppe has joined #openstack-watcher | 11:02 | |
| *** alexchadin has quit IRC | 11:22 | |
| openstackgerrit | avnish proposed openstack/python-watcherclient: Update home page link in cfg file https://review.openstack.org/372426 | 11:32 |
| *** jwcroppe has quit IRC | 11:41 | |
| *** thorst has joined #openstack-watcher | 11:42 | |
| *** thorst has quit IRC | 11:46 | |
| *** jwcroppe has joined #openstack-watcher | 11:49 | |
| *** alexchadin has joined #openstack-watcher | 12:04 | |
| *** jwcroppe has quit IRC | 12:07 | |
| *** jwcroppe has joined #openstack-watcher | 12:08 | |
| *** jwcroppe has quit IRC | 12:26 | |
| sballe_ | Morning | 12:27 |
| *** alexchadin has quit IRC | 12:27 | |
| *** alexchadin has joined #openstack-watcher | 12:28 | |
| *** jwcroppe has joined #openstack-watcher | 12:31 | |
| sballe_ | gfhelma are you here | 12:33 |
| acabot | Morning | 12:35 |
| sballe_ | Hi | 12:35 |
| openstackgerrit | Antoine Cabot proposed openstack/watcher: Update Watcher description https://review.openstack.org/372468 | 13:01 |
| *** alexchadin has quit IRC | 13:04 | |
| *** alexchadin has joined #openstack-watcher | 13:05 | |
| openstackgerrit | Merged openstack/watcher: Update reno for stable/newton https://review.openstack.org/371518 | 13:11 |
| *** jwcroppe has quit IRC | 13:12 | |
| sballe_ | Gfhelma lets talk in this channel | 13:14 |
| *** gfhellma has joined #openstack-watcher | 13:14 | |
| gfhellma | Hello | 13:15 |
| dtardivel | gfhellma: Hi | 13:15 |
| sballe_ | Good we are all here :-) | 13:15 |
| gfhellma | I did the manual install within the Horizon Docker Container | 13:16 |
| dtardivel | gfhellma: How are you ? Could you give us a status about the Monasca deployment and Watcher ? | 13:16 |
| gfhellma | install on monasc-ui | 13:17 |
| *** thorst_ has joined #openstack-watcher | 13:17 | |
| gfhellma | the monasca install on the host we call 4.6 went ok as there are no Docker Containers on that host | 13:17 |
| gfhellma | just a notmal install | 13:17 |
| dtardivel | gfhellma: Do you access now onto the 'Monitoring' Panel on Horizon ? | 13:18 |
| gfhellma | The lasy ichecked that was not showing up mainly because of th config issue that mentioned | 13:18 |
| gfhellma | Apache with the docker container is et up differently than on anormal howizon apache host | 13:19 |
| gfhellma | the main difference is that on a regular host horizon is in the sites | 13:19 |
| gfhellma | folder | 13:20 |
| gfhellma | within the docker container it is folded into the 000-default conf file | 13:20 |
| dtardivel | gfhellma: which distrib do you use for horion ? Ubuntu ? CentOS ? | 13:21 |
| gfhellma | i have to pull that apart so that i can include the monasca-ui without over-riding the base Horizon install. | 13:21 |
| gfhellma | these hosts have been done with ubuntu 14.04 | 13:21 |
| *** thorst_ has quit IRC | 13:24 | |
| gfhellma | dtardvel: can you meet me on a 1:1 | 13:25 |
| *** michaelgugino has joined #openstack-watcher | 13:30 | |
| openstackgerrit | Merged openstack/watcher: Update Watcher description https://review.openstack.org/372468 | 13:33 |
| *** gfhellma has quit IRC | 14:04 | |
| *** thorst_ has joined #openstack-watcher | 14:13 | |
| openstackgerrit | Antoine Cabot proposed openstack/watcher-specs: Add team priorities for Ocata https://review.openstack.org/372528 | 14:30 |
| *** thorst_ has quit IRC | 14:33 | |
| *** thorst_ has joined #openstack-watcher | 14:34 | |
| *** hvprash_ has joined #openstack-watcher | 14:35 | |
| *** openstackgerrit has quit IRC | 14:37 | |
| *** openstackgerrit has joined #openstack-watcher | 14:37 | |
| *** hvprash_ has quit IRC | 14:42 | |
| *** thorst_ has quit IRC | 14:42 | |
| *** thorst_ has joined #openstack-watcher | 14:43 | |
| *** hvprash1 has joined #openstack-watcher | 14:51 | |
| *** tri2sing has joined #openstack-watcher | 14:51 | |
| openstackgerrit | David TARDIVEL proposed openstack/watcher-dashboard: Update table's actions https://review.openstack.org/372560 | 15:05 |
| openstackgerrit | Alexander Chadin proposed openstack/watcher: [WIP] Add service supervisor https://review.openstack.org/371632 | 15:07 |
| alexchadin | vincentfrancoise: take a look https://review.openstack.org/#/c/371632/2/watcher/objects/base.py | 15:08 |
| alexchadin | vincentfrancoise: https://review.openstack.org/#/c/371632/2/watcher/objects/service.py line 35 | 15:09 |
| alexchadin | vincentfrancoise: That's how issue we have spoken about was resolved. | 15:09 |
| vincentfrancoise | alexchadin: 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 case | 15:11 |
| alexchadin | ah, thanks | 15:11 |
| alexchadin | didn't know about it | 15:11 |
| vincentfrancoise | alexchadin: the _attr_last_seen_up_to_primitive is kind of ugly but I don't see any better solution... | 15:14 |
| alexchadin | The mechanism is ugly - to store datetime it converts utc to iso8601 (just adds +00:00) which will be converted to string and saved as string | 15:15 |
| alexchadin | created_at, updated_at, deleted_at used the same way | 15:16 |
| *** alexchadin has quit IRC | 15:24 | |
| *** alexchadin has joined #openstack-watcher | 15:24 | |
| openstackgerrit | Antoine Cabot proposed openstack/watcher-specs: Add team priorities for Ocata https://review.openstack.org/372528 | 15:26 |
| *** thorst_ has quit IRC | 15:31 | |
| *** thorst_ has joined #openstack-watcher | 15:31 | |
| *** alexchadin has quit IRC | 15:32 | |
| openstackgerrit | David TARDIVEL proposed openstack/watcher-dashboard: Clean the code by renaming 'type' parameters https://review.openstack.org/372583 | 15:32 |
| sballe_ | dtardivel: where you and gfhellman able to get things working | 15:50 |
| vincentfrancoise | sballe_: dtardivel already left the office | 15:52 |
| vincentfrancoise | sballe_: I was following from afar but seems like Gustav wasn't stuck anymor | 15:56 |
| sballe_ | Cool thx | 15:56 |
| sballe_ | vincentfrancoise: did you or dtardivel give him the watcher conf file he needs if he deploys watcher with kola | 15:59 |
| *** tri2sing has quit IRC | 16:00 | |
| sballe_ | Gfhellman how are you doing? Moving forward installing the next service? | 16:00 |
| *** acabot has quit IRC | 16:00 | |
| vincentfrancoise | sballe_: I don't know about what they did other than their conversation logs | 16:02 |
| *** thorst_ has quit IRC | 16:18 | |
| *** vincentfrancoise has quit IRC | 16:18 | |
| *** thorst_ has joined #openstack-watcher | 16:18 | |
| *** thorst_ has quit IRC | 16:25 | |
| *** thorst_ has joined #openstack-watcher | 16:26 | |
| *** thorst_ has quit IRC | 16:41 | |
| *** thorst_ has joined #openstack-watcher | 16:48 | |
| *** diga has quit IRC | 17:02 | |
| *** thorst_ has quit IRC | 17:05 | |
| *** gfhellma has joined #openstack-watcher | 17:10 | |
| *** thorst_ has joined #openstack-watcher | 17:11 | |
| *** thorst_ has quit IRC | 17:16 | |
| *** tri2sing has joined #openstack-watcher | 17:27 | |
| *** harlowja has joined #openstack-watcher | 17:30 | |
| *** jwcroppe has joined #openstack-watcher | 17:51 | |
| *** jwcroppe_ has joined #openstack-watcher | 17:52 | |
| *** jwcropp__ has joined #openstack-watcher | 17:53 | |
| *** jwcroppe has quit IRC | 17:56 | |
| *** jwcroppe_ has quit IRC | 17:57 | |
| *** jwcropp__ has quit IRC | 18:43 | |
| *** dtardivel has quit IRC | 18:57 | |
| *** jwcroppe has joined #openstack-watcher | 19:13 | |
| *** jwcroppe has quit IRC | 19:43 | |
| *** jwcroppe has joined #openstack-watcher | 19:49 | |
| *** thorst_ has joined #openstack-watcher | 19:54 | |
| *** thorst_ has quit IRC | 20:10 | |
| *** gfhellma has quit IRC | 20:13 | |
| *** thorst_ has joined #openstack-watcher | 20:25 | |
| *** gfhellma has joined #openstack-watcher | 20:28 | |
| *** jwcroppe has quit IRC | 20:44 | |
| *** thorst_ has quit IRC | 20:54 | |
| *** jwcroppe has joined #openstack-watcher | 20:55 | |
| *** jwcroppe has quit IRC | 21:11 | |
| *** jwcroppe has joined #openstack-watcher | 21:36 | |
| *** jwcroppe has quit IRC | 21:48 | |
| *** harlowja has quit IRC | 21:51 | |
| *** michaelgugino has quit IRC | 21:52 | |
| *** harlowja has joined #openstack-watcher | 21:56 | |
| *** jwcroppe has joined #openstack-watcher | 22:06 | |
| *** tri2sing has quit IRC | 22:26 | |
| *** jwcroppe has quit IRC | 22:40 | |
| *** jwcroppe has joined #openstack-watcher | 22:51 | |
| *** jwcroppe has quit IRC | 22:56 | |
| *** jwcroppe has joined #openstack-watcher | 23:06 | |
| *** jwcroppe has quit IRC | 23:11 | |
| *** thorst_ has joined #openstack-watcher | 23:14 | |
| *** gfhellma has quit IRC | 23:24 | |
| *** thorst_ has quit IRC | 23:25 | |
| *** gfhellma has joined #openstack-watcher | 23:26 | |
| *** jwcroppe has joined #openstack-watcher | 23:29 | |
| *** jwcroppe has quit IRC | 23:41 | |
| *** openstackgerrit has quit IRC | 23:42 | |
| *** openstackgerrit has joined #openstack-watcher | 23:46 | |
| *** jwcroppe has joined #openstack-watcher | 23:48 | |
| *** openstackgerrit has quit IRC | 23:51 | |
| *** openstackgerrit has joined #openstack-watcher | 23:54 | |
| *** jwcroppe has quit IRC | 23:57 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!