| *** thorst has joined #openstack-watcher | 00:13 | |
| *** thorst has quit IRC | 00:17 | |
| *** thorst has joined #openstack-watcher | 00:21 | |
| *** thorst_ has joined #openstack-watcher | 00:23 | |
| *** thorst has quit IRC | 00:26 | |
| *** thorst_ has quit IRC | 01:37 | |
| *** thorst has joined #openstack-watcher | 02:11 | |
| *** thorst has joined #openstack-watcher | 02:12 | |
| *** thorst has quit IRC | 02:20 | |
| *** thorst has joined #openstack-watcher | 03:18 | |
| *** thorst has quit IRC | 03:26 | |
| *** diga has joined #openstack-watcher | 03:44 | |
| *** thorst has joined #openstack-watcher | 04:25 | |
| *** thorst has quit IRC | 04:30 | |
| *** thorst has joined #openstack-watcher | 05:27 | |
| *** thorst has quit IRC | 05:36 | |
| *** diga has quit IRC | 05:44 | |
| *** thorst has joined #openstack-watcher | 06:35 | |
| *** thorst has quit IRC | 06:41 | |
| *** brunograz has joined #openstack-watcher | 07:17 | |
| *** alexchadin has joined #openstack-watcher | 07:21 | |
| *** vincentfrancoise has joined #openstack-watcher | 07:29 | |
| openstackgerrit | David TARDIVEL proposed openstack/watcher-dashboard: Updated from global requirements https://review.openstack.org/354329 | 07:32 |
|---|---|---|
| *** dtardivel has joined #openstack-watcher | 07:36 | |
| *** thorst has joined #openstack-watcher | 07:39 | |
| openstackgerrit | David TARDIVEL proposed openstack/watcher-dashboard: Updated from global requirements https://review.openstack.org/354329 | 07:42 |
| openstackgerrit | Vincent Françoise proposed openstack/watcher: Added tests on API hooks and related context https://review.openstack.org/357938 | 07:43 |
| *** thorst has quit IRC | 07:46 | |
| openstackgerrit | Merged openstack/watcher: Fixes to get cluster data model https://review.openstack.org/357295 | 07:51 |
| openstackgerrit | Merged openstack/watcher-dashboard: Updated from global requirements https://review.openstack.org/354329 | 08:03 |
| openstackgerrit | Vincent Françoise proposed openstack/watcher: Added strategy ID + Action Plan syncing https://review.openstack.org/325213 | 08:08 |
| openstackgerrit | David TARDIVEL proposed openstack/watcher-dashboard: Remove discover from test-requirements https://review.openstack.org/345214 | 08:12 |
| openstackgerrit | Vincent Françoise proposed openstack/watcher: Added start/end date params on ceilometer queries https://review.openstack.org/358465 | 08:15 |
| alexchadin | vincentfrancoise: hello! | 08:15 |
| vincentfrancoise | alexchadin: morning | 08:29 |
| alexchadin | I have some thoughts about respecting nova zones and aggregates | 08:30 |
| alexchadin | for example, we have 3 aggregates with 3 different metedata - ssd, sata, iscsi | 08:31 |
| alexchadin | and we set our scope to the ssd and iscsi aggregates, sata is ignored | 08:32 |
| alexchadin | then vms can be migrated between ssd and sata since nova live-migration mechanism doesn't call scheduler to apply scheduler hosts | 08:32 |
| alexchadin | do we place responsibility for migration to the watcher end user (admin)? | 08:33 |
| alexchadin | I won't mind but I think we need to take into account this moment | 08:34 |
| vincentfrancoise | what do you mean by placing responsibility for migration to the admin? | 08:36 |
| vincentfrancoise | The way I see it is that the strategy will produce the action plan as usual | 08:36 |
| vincentfrancoise | so the decision of performing the migration is already made by the admin | 08:37 |
| alexchadin | well, ok | 08:37 |
| alexchadin | :) | 08:37 |
| vincentfrancoise | and alter on when we will implement the automatic trigger of action plans, both will be possible | 08:39 |
| vincentfrancoise | later on* | 08:39 |
| *** thorst has joined #openstack-watcher | 08:44 | |
| *** thorst has quit IRC | 08:51 | |
| *** Kevin_Zheng has joined #openstack-watcher | 08:53 | |
| *** alexchadin has quit IRC | 09:01 | |
| *** alexchadin has joined #openstack-watcher | 09:02 | |
| *** thorst has joined #openstack-watcher | 09:48 | |
| *** thorst has quit IRC | 09:55 | |
| alexchadin | vincentfrancoise: I'd like to create reference to audit template table from audits | 10:51 |
| alexchadin | vincentfrancoise: what do you think about it? | 10:53 |
| *** aswadr_ has joined #openstack-watcher | 10:53 | |
| *** thorst has joined #openstack-watcher | 10:54 | |
| alexchadin | There is audit_template_id in watcher_db_schema_diagram.png but actually audit table doesn't have this field | 10:54 |
| *** thorst has quit IRC | 11:01 | |
| *** openstackgerrit has quit IRC | 11:03 | |
| *** openstackgerrit has joined #openstack-watcher | 11:03 | |
| *** thorst has joined #openstack-watcher | 11:49 | |
| *** alexchadin has quit IRC | 11:56 | |
| *** alexchadin has joined #openstack-watcher | 11:57 | |
| *** alexchadin has quit IRC | 12:01 | |
| vincentfrancoise | alexchadin: IMHO, you shouldn't make any reference to audit templates from audits | 12:11 |
| vincentfrancoise | the foreign key was removed recently because audit copies all the necessary information from its audit template counterpart so that it doesn't need the reference anymore | 12:12 |
| vincentfrancoise | see http://specs.openstack.org/openstack/watcher-specs/specs/newton/approved/persistent-audit-parameters.html | 12:12 |
| openstackgerrit | Merged openstack/watcher-dashboard: Remove discover from test-requirements https://review.openstack.org/345214 | 12:15 |
| *** alexchadin has joined #openstack-watcher | 13:27 | |
| alexchadin | vincentfrancoise: Do you suggest to copy scope to the audit? | 13:27 |
| vincentfrancoise | alexchadin: it's already the case | 13:27 |
| vincentfrancoise | alexchadin: https://github.com/openstack/watcher/blob/master/watcher/db/sqlalchemy/models.py#L164-L181 | 13:28 |
| alexchadin | okay, will add here scope | 13:28 |
| vincentfrancoise | alexchadin: not sure what's the best way | 13:29 |
| alexchadin | then we need to update image of db scheme | 13:29 |
| vincentfrancoise | but maybe you can reuse the host_aggregate DB field and rename it | 13:29 |
| alexchadin | I've already made audit_template_id | 13:29 |
| alexchadin | Didn't see your message about spec | 13:30 |
| alexchadin | I think we need to remove host_aggregate, it is redundant since we have scope | 13:30 |
| vincentfrancoise | alexchadin: that's why I was suggesting it :) | 13:31 |
| alexchadin | vincentfrancoise: I'm not sure that copying of scope from template to every audit is good idea | 13:32 |
| alexchadin | It is also redundant, IMHO | 13:32 |
| alexchadin | it is good to goal_id or something little | 13:32 |
| vincentfrancoise | the use case for the copy is the following | 13:32 |
| vincentfrancoise | imagine the admin creates an audit template with a given audit scope | 13:33 |
| vincentfrancoise | runs an audit with it | 13:33 |
| vincentfrancoise | but then he wants to change it because he , say, forgot to exclude a VM | 13:33 |
| *** esberglu has joined #openstack-watcher | 13:33 | |
| vincentfrancoise | if you didn't copy the audit scope in the audit table | 13:34 |
| vincentfrancoise | then later on, the admin may believe that the initial audit was executed with the new audit scope instead of the old | 13:34 |
| vincentfrancoise | which can be misleading | 13:34 |
| alexchadin | I see | 13:35 |
| vincentfrancoise | -> the copy allows the admin to change the template with impacting all underlying audits | 13:35 |
| vincentfrancoise | without* | 13:35 |
| *** bigdogstl has joined #openstack-watcher | 13:48 | |
| *** bigdogstl has quit IRC | 13:53 | |
| sballe_ | morning | 14:00 |
| alexchadin | hi | 14:00 |
| vincentfrancoise | hi | 14:00 |
| sballe_ | please let me know what I need to review code or specs. I have some time on hand today | 14:00 |
| *** hvprash has joined #openstack-watcher | 14:01 | |
| sballe_ | I'll look in launchpad as well but since I was gone for 3 weeks I am still ctaching up | 14:01 |
| *** diga has joined #openstack-watcher | 14:01 | |
| vincentfrancoise | sballe_: BP wise, there was wasn't much news except alexchadin that is working on the audit scope one I believe | 14:02 |
| *** hvprash has quit IRC | 14:03 | |
| sballe_ | I can see scoring moduel needs code review and also monasca support | 14:03 |
| sballe_ | I'll start with those teo | 14:03 |
| sballe_ | s/two | 14:03 |
| vincentfrancoise | sballe_: the monasca support is on hold for now | 14:04 |
| sballe_ | vincentfrancoise: ok txh for the heads up. alexchadin let me know if you want me to review anything code or spec | 14:05 |
| vincentfrancoise | we need to sync with the monasca team to see whether or not we decide to use ceilosca or monasca directly | 14:05 |
| diga | acabot: Hi | 14:05 |
| *** esberglu has left #openstack-watcher | 14:06 | |
| alexchadin | sballe_: I will add you to reviewers once PS is ready to publish | 14:06 |
| sballe_ | vincentfrancoise: ok make sense | 14:07 |
| vincentfrancoise | alexchadin: FYI, you can add all the cores in one go by adding the watcher-specs-core group to the review | 14:07 |
| alexchadin | wow | 14:07 |
| vincentfrancoise | alexchadin: same goes for watcher with the watcher-core group | 14:07 |
| sballe_ | When I talked to Roland several summits ago he said that Ceilosca had its limitations but he is buyest of course | 14:07 |
| alexchadin | vincentfrancoise: really cool information:) | 14:08 |
| *** hvprash has joined #openstack-watcher | 14:13 | |
| *** hvprash has quit IRC | 14:17 | |
| *** michaelgugino has joined #openstack-watcher | 14:30 | |
| *** bigdogstl has joined #openstack-watcher | 14:33 | |
| *** bigdogstl has quit IRC | 14:43 | |
| *** alexchadin has quit IRC | 15:04 | |
| *** Zucan has joined #openstack-watcher | 15:10 | |
| *** hvprash has joined #openstack-watcher | 15:12 | |
| *** hvprash has quit IRC | 15:12 | |
| *** bigdogstl has joined #openstack-watcher | 15:14 | |
| *** bigdogstl has quit IRC | 15:32 | |
| *** bigdogstl has joined #openstack-watcher | 15:33 | |
| acabot | diga : hi | 15:36 |
| diga | acabot: Hi | 15:38 |
| diga | acabot: https://review.openstack.org/#/c/342473/ | 15:39 |
| diga | acabot: gimme today's time, will update the implementation plan by tomorrow | 15:40 |
| acabot | diga : yes, any update for it ? | 15:40 |
| acabot | great | 15:40 |
| acabot | please do it before wednesday | 15:40 |
| diga | Sure | 15:40 |
| acabot | can you join us for the weekly meeting ? | 15:40 |
| diga | yes | 15:40 |
| diga | wednesday at 1600 UTC ?? | 15:41 |
| acabot | wednesday 14 UTC | 15:41 |
| acabot | #openstack-meeting-4 | 15:41 |
| diga | yes | 15:41 |
| acabot | https://wiki.openstack.org/wiki/Watcher#Meetings | 15:41 |
| diga | ohh sorry, 1400 UTC | 15:41 |
| acabot | I will add your updated spec to the agenda then | 15:41 |
| diga | yes | 15:41 |
| acabot | thx | 15:42 |
| diga | acabot: wc! | 15:43 |
| *** bigdogstl has quit IRC | 15:48 | |
| *** bigdogstl has joined #openstack-watcher | 15:56 | |
| *** pkoniszewski has quit IRC | 16:08 | |
| *** pkoniszewski has joined #openstack-watcher | 16:10 | |
| *** bigdogstl has quit IRC | 16:12 | |
| *** bigdogstl has joined #openstack-watcher | 16:14 | |
| *** bigdogstl has left #openstack-watcher | 16:24 | |
| *** vincentfrancoise has quit IRC | 16:26 | |
| *** diga has quit IRC | 16:46 | |
| *** tri2sing has joined #openstack-watcher | 16:53 | |
| *** dtardivel has quit IRC | 19:17 | |
| *** brunograz_ has joined #openstack-watcher | 21:00 | |
| *** brunograz has quit IRC | 21:01 | |
| *** thorst has quit IRC | 21:20 | |
| *** michaelgugino has quit IRC | 21:29 | |
| *** Zucan has quit IRC | 21:40 | |
| *** thorst has joined #openstack-watcher | 23:44 | |
| *** thorst_ has joined #openstack-watcher | 23:55 | |
| *** thorst has quit IRC | 23:58 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!