| *** jonaspf has joined #openstack-freezer | 00:09 | |
| *** jonaspf has quit IRC | 00:24 | |
| *** jonaspf has joined #openstack-freezer | 01:22 | |
| *** jonaspf has quit IRC | 01:37 | |
| *** jonaspf has joined #openstack-freezer | 02:32 | |
| *** jonaspf has quit IRC | 02:47 | |
| *** jonaspf has joined #openstack-freezer | 03:30 | |
| *** jonaspf has quit IRC | 03:45 | |
| *** jonaspf has joined #openstack-freezer | 03:49 | |
| *** jonaspf has quit IRC | 04:04 | |
| *** jonaspf has joined #openstack-freezer | 04:40 | |
| *** jonaspf has quit IRC | 04:55 | |
| *** jonaspf has joined #openstack-freezer | 05:00 | |
| *** jonaspf has quit IRC | 05:15 | |
| *** jonaspf has joined #openstack-freezer | 05:47 | |
| *** jonaspf has quit IRC | 06:11 | |
| *** jonaspf has joined #openstack-freezer | 06:17 | |
| *** jonaspf has quit IRC | 06:33 | |
| *** jonaspf has joined #openstack-freezer | 06:35 | |
| *** jonaspf has quit IRC | 07:06 | |
| *** jonaspf has joined #openstack-freezer | 07:18 | |
| *** openstack has joined #openstack-freezer | 07:21 | |
| *** jonaspf has quit IRC | 07:32 | |
| *** jonaspf has joined #openstack-freezer | 07:56 | |
| *** jonaspf has quit IRC | 08:11 | |
| *** samuelBartel has joined #openstack-freezer | 08:22 | |
| openstackgerrit | Pierre Mathieu proposed openstack/freezer: Import pep3134daemon as local module https://review.openstack.org/248069 | 08:59 |
|---|---|---|
| *** reldan has joined #openstack-freezer | 08:59 | |
| openstackgerrit | Pierre Mathieu proposed openstack/freezer: Import pep3134daemon as local module https://review.openstack.org/248069 | 09:05 |
| *** szaher has joined #openstack-freezer | 09:13 | |
| *** szaher_ has joined #openstack-freezer | 09:13 | |
| *** reldan has quit IRC | 09:17 | |
| *** memogarcia has quit IRC | 09:37 | |
| *** jonaspf has joined #openstack-freezer | 09:52 | |
| *** jonaspf has quit IRC | 10:07 | |
| *** reldan has joined #openstack-freezer | 10:12 | |
| *** szaher_ has quit IRC | 10:13 | |
| *** szaher_ has joined #openstack-freezer | 10:14 | |
| *** daemontool has joined #openstack-freezer | 10:19 | |
| daemontool | Morning | 10:19 |
| daemontool | I think is critical that today we merge https://review.openstack.org/#/c/248069/ | 10:19 |
| Slashme | I agree | 10:22 |
| Slashme | And backport it to Kilo (also memogarcia patch on windows installation doc) | 10:22 |
| daemontool | m3m0, vannif reldan ffresh ^^ | 10:23 |
| daemontool | also looking for +A here https://review.openstack.org/#/c/246368/ and here https://review.openstack.org/#/c/246334/ | 10:25 |
| daemontool | vannif, any news on https://review.openstack.org/#/c/246993/ ? | 10:27 |
| reldan | https://review.openstack.org/#/c/248069/ I would like to have some explanation in README | 10:27 |
| reldan | why we are copying third part libary into our repo | 10:28 |
| reldan | Just copy commit message to README | 10:28 |
| vannif | fix for https://review.openstack.org/#/c/246993/ will be in today | 10:28 |
| reldan | daemontool: Do you agree with that? | 10:28 |
| daemontool | reldan, I think explaning why we do something is always a good thing | 10:29 |
| daemontool | so yes | 10:29 |
| *** jonaspf has joined #openstack-freezer | 10:29 | |
| reldan | https://review.openstack.org/#/c/246334/2 - seems good, but why we should change default branch? I just don’t know, if it is requirements for os projects - great | 10:30 |
| daemontool | reldan, because | 10:30 |
| reldan | But I suppose the major part of reviews will be in master not in stable/kilo | 10:30 |
| daemontool | if you don't do that | 10:30 |
| daemontool | yes | 10:31 |
| daemontool | that is only for .gitreview in stable/kilo | 10:31 |
| daemontool | that simplify the dev job | 10:31 |
| reldan | See now, sorry | 10:31 |
| daemontool | because by default when you execute git review the patchset goes in master | 10:31 |
| daemontool | while with that | 10:31 |
| daemontool | it goes by default on stable/kilo | 10:31 |
| reldan | Makes sense, yes. Sorry | 10:32 |
| daemontool | np | 10:32 |
| *** daemontool has quit IRC | 10:34 | |
| *** daemontool has joined #openstack-freezer | 10:35 | |
| *** jonaspf has quit IRC | 10:49 | |
| openstackgerrit | Fausto Marzi proposed openstack/freezer: Import pep3134daemon as local module https://review.openstack.org/248069 | 10:51 |
| daemontool | nope, there's somethign wrong there... | 10:56 |
| daemontool | the README.rst is the README from the Freezer API? | 10:58 |
| daemontool | I'm fixing it one sec | 10:59 |
| openstackgerrit | Fausto Marzi proposed openstack/freezer: Import pep3134daemon as local module https://review.openstack.org/248069 | 11:05 |
| openstackgerrit | Fausto Marzi proposed openstack/freezer: Import pep3134daemon as local module https://review.openstack.org/248069 | 11:10 |
| daemontool | should be OK now | 11:10 |
| daemontool | reldan, the info are added at the end under the MISC section | 11:11 |
| daemontool | we should do a table of content for that readme | 11:11 |
| daemontool | as it is in the api repo | 11:11 |
| daemontool | m3m0, this one can be merged? https://review.openstack.org/#/c/245737/ | 11:13 |
| reldan | Great thank you! | 11:14 |
| m3m0 | yes daemontool, please | 11:15 |
| daemontool | ok | 11:17 |
| daemontool | so https://review.openstack.org/#/c/248069/9 and https://review.openstack.org/#/c/245737/ please let's merge that today | 11:17 |
| daemontool | I mean... now :) | 11:18 |
| daemontool | unless there's something wrong | 11:18 |
| daemontool | vannif, m3m0 Slashme ffresh reldan https://review.openstack.org/#/c/248069/ | 11:23 |
| reldan | I gave +1 | 11:23 |
| daemontool | ah ok ty | 11:23 |
| vannif | if we need to move on quickly, it's ok. | 11:26 |
| *** reldan has quit IRC | 11:53 | |
| *** reldan has joined #openstack-freezer | 12:08 | |
| daemontool | vannif, yes we are blocked by that | 12:14 |
| openstackgerrit | Merged openstack/freezer: Import pep3134daemon as local module https://review.openstack.org/248069 | 12:24 |
| Slashme | Cherry-pick commit for stable/kilo: | 12:36 |
| Slashme | https://review.openstack.org/#/c/248660/ | 12:36 |
| Slashme | https://review.openstack.org/#/c/248664/ | 12:36 |
| daemontool | all, please review ^^ | 13:05 |
| *** daemontool has quit IRC | 13:25 | |
| *** jonaspf has joined #openstack-freezer | 13:36 | |
| Slashme | https://review.openstack.org/#/c/248664/ needs review again | 13:38 |
| *** daemontool has joined #openstack-freezer | 13:38 | |
| daemontool | Slashme, can we approve https://review.openstack.org/#/c/248664/ or do you need to send further patches? | 13:46 |
| Slashme | Nope, it is okay now. Can you check if what I did is not bad ? The cherry-pick commit I did is not only a cherry-pick anymore, is also bears a few very small changes needed for that to work on stable/kilo. | 13:49 |
| daemontool | ok, so we need to add in the commit message the files that has been modified | 13:51 |
| daemontool | Slashme, like here https://review.openstack.org/#/c/246460/ | 13:51 |
| daemontool | of course not the setup.cfg as the version always changes when backporting | 13:52 |
| Slashme | daemontool: done | 13:56 |
| *** jonaspf has quit IRC | 13:57 | |
| daemontool | Slashme, brilliant | 13:57 |
| openstackgerrit | Pierre Mathieu proposed openstack/freezer-api: Add propoerties that span across all action of a job https://review.openstack.org/243021 | 14:04 |
| daemontool | Slashme, ok, +2ed https://review.openstack.org/#/c/248664/ | 14:07 |
| *** daemontool has quit IRC | 14:12 | |
| *** daemontool has joined #openstack-freezer | 14:13 | |
| openstackgerrit | Pierre Mathieu proposed openstack/freezer-api: Add propoerties that span across all action of a job https://review.openstack.org/243021 | 14:50 |
| daemontool | Slashme, https://review.openstack.org/#/c/248664/ merged | 14:55 |
| daemontool | all, please review here when you can: https://review.openstack.org/#/c/245737/ | 14:56 |
| daemontool | Slashme, so now we need wait vannif changes to get this merged and we should be good, right? https://review.openstack.org/#/c/246993/ | 14:57 |
| daemontool | am I missing something? | 14:57 |
| vannif | yes | 14:57 |
| daemontool | vannif, ty | 14:58 |
| openstackgerrit | Fabrizio Vanni proposed openstack/freezer-api: remove elasticsearch method delete_by_query https://review.openstack.org/248744 | 14:58 |
| daemontool | there you go :) | 14:58 |
| daemontool | vannif, that's only for liberty if I remember well? | 14:59 |
| openstackgerrit | Fabrizio Vanni proposed openstack/freezer-api: remove elasticsearch method delete_by_query https://review.openstack.org/248744 | 15:00 |
| openstackgerrit | Merged openstack/freezer: Improvements for windows snapshots https://review.openstack.org/245737 | 15:00 |
| *** openstackgerrit has quit IRC | 15:02 | |
| *** openstackgerrit has joined #openstack-freezer | 15:02 | |
| vannif | no. for kilo also | 15:03 |
| vannif | kilo has elasticsearch>=1.3.0 | 15:03 |
| vannif | same as liberty | 15:03 |
| daemontool | ah ok | 15:14 |
| openstackgerrit | Pierre Mathieu proposed openstack/freezer-api: Add properties that span across all action of a job https://review.openstack.org/243021 | 15:17 |
| openstackgerrit | Pierre Mathieu proposed openstack/freezer-api: Add properties that span across all action of a job https://review.openstack.org/243021 | 15:20 |
| openstackgerrit | Pierre Mathieu proposed openstack/freezer-api: Add properties that span across all action of a job https://review.openstack.org/243021 | 15:37 |
| *** daemontool_ has joined #openstack-freezer | 15:48 | |
| *** daemontool has quit IRC | 15:52 | |
| openstackgerrit | Pierre Mathieu proposed openstack/freezer-api: Add properties that span across all action of a job https://review.openstack.org/243021 | 15:58 |
| *** dschroeder has joined #openstack-freezer | 16:07 | |
| *** memogarcia has joined #openstack-freezer | 16:11 | |
| *** memogarcia has quit IRC | 16:18 | |
| Slashme | Can I get some review on this: https://review.openstack.org/#/c/243021/ please ? | 16:23 |
| openstackgerrit | Memo Garcia proposed openstack/freezer: Freezer Scheduler for Windows https://review.openstack.org/248804 | 16:36 |
| daemontool_ | all, anyone can review https://review.openstack.org/#/c/248744/ that is also blocking? | 16:44 |
| openstackgerrit | Pierre Mathieu proposed openstack/freezer-api: Add support for properties that span across all action of a job https://review.openstack.org/243021 | 16:57 |
| openstackgerrit | Memo Garcia proposed openstack/freezer: Freezer Scheduler for Windows https://review.openstack.org/248804 | 17:03 |
| *** daemontool_ has quit IRC | 17:06 | |
| *** daemontool has joined #openstack-freezer | 17:08 | |
| *** samuelBartel has quit IRC | 17:26 | |
| *** reldan has quit IRC | 18:09 | |
| *** memogarcia has joined #openstack-freezer | 18:15 | |
| daemontool | m3m0, memogarcia https://review.openstack.org/#/c/248744/ :) | 18:25 |
| daemontool | Slashme, ffresh ^^ | 18:26 |
| *** szaher_ has quit IRC | 18:37 | |
| *** daemontool has quit IRC | 18:54 | |
| *** module000 has left #openstack-freezer | 18:59 | |
| *** memogarcia has quit IRC | 19:42 | |
| *** memogarcia has joined #openstack-freezer | 19:51 | |
| *** reldan has joined #openstack-freezer | 20:22 | |
| memogarcia | daemontool can I run a subprocess command after freezer is installed? | 20:52 |
| *** Slashme has quit IRC | 21:11 | |
| *** epheo has quit IRC | 21:11 | |
| *** Slashme has joined #openstack-freezer | 21:15 | |
| *** epheo has joined #openstack-freezer | 21:18 | |
| *** jonaspf has joined #openstack-freezer | 21:20 | |
| *** jonaspf has quit IRC | 21:35 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!