*** dschroeder has quit IRC | 00:15 | |
*** samuelBartel has quit IRC | 00:36 | |
*** reldan has quit IRC | 01:43 | |
*** reldan has joined #openstack-freezer | 02:47 | |
*** memogarcia has quit IRC | 04:00 | |
*** reldan has quit IRC | 04:03 | |
openstackgerrit | Einst Crazy proposed openstack/freezer-api: Replace assertEqual(*, None) with assertIsNone in tests https://review.openstack.org/264654 | 07:58 |
---|---|---|
*** EinstCrazy has joined #openstack-freezer | 08:06 | |
openstackgerrit | Einst Crazy proposed openstack/freezer: use openstack command to create user and project https://review.openstack.org/264665 | 08:37 |
*** samuelBartel has joined #openstack-freezer | 08:39 | |
openstackgerrit | Einst Crazy proposed openstack/freezer: use openstack command to create user and project https://review.openstack.org/264665 | 08:43 |
*** samuelBartel has quit IRC | 08:45 | |
*** samuelBartel has joined #openstack-freezer | 08:58 | |
*** daemontool has joined #openstack-freezer | 09:32 | |
daemontool | Morning | 09:33 |
*** vannif_ has quit IRC | 09:52 | |
openstackgerrit | Vishal kumar mahajan proposed openstack/freezer: Fix the parameter order of assertEqual. https://review.openstack.org/264028 | 10:04 |
*** EinstCrazy has quit IRC | 10:07 | |
*** reldan has joined #openstack-freezer | 10:08 | |
openstackgerrit | Eldar Nugaev proposed openstack/freezer: Parallel backup https://review.openstack.org/247840 | 10:27 |
openstackgerrit | Eldar Nugaev proposed openstack/freezer: Parallel backup https://review.openstack.org/247840 | 10:43 |
openstackgerrit | Eldar Nugaev proposed openstack/freezer: Parallel backup https://review.openstack.org/247840 | 10:51 |
daemontool | reldan, can I retest https://review.openstack.org/247840 ? is number 17 intended to be the last patchset? | 10:53 |
reldan | daemontool: Yes, I have fixed problem with incremental swift and creation default containers. But let’s wait until jenkins shows that all tests passed | 10:54 |
daemontool | ok | 10:54 |
daemontool | if anyone have time please review the bp https://review.openstack.org/#/c/263776/ | 10:55 |
daemontool | specs for python-freezerclient | 10:55 |
daemontool | I'll test it today | 10:58 |
*** memogarcia has joined #openstack-freezer | 11:06 | |
daemontool | memogarcia, :) | 11:07 |
memogarcia | hey daemontool, anything for review? | 11:12 |
daemontool | memogarcia, https://review.openstack.org/#/c/247840/ | 11:14 |
daemontool | I did some test last week | 11:14 |
daemontool | there's an etherpad link in the comments | 11:15 |
daemontool | now should be fixed by reldan if you want to help me restesting it | 11:15 |
daemontool | also the first 8 commits here https://review.openstack.org/#/q/project:openstack/freezer+status:open | 11:16 |
daemontool | the specs | 11:16 |
daemontool | also | 11:16 |
memogarcia | do you have the link at hand for the specs? | 11:19 |
daemontool | https://review.openstack.org/#/c/263776/ https://review.openstack.org/#/c/259905/ | 11:20 |
memogarcia | grazie | 11:22 |
daemontool | >) | 11:23 |
memogarcia | +1 in both specs, I'll wait for more reviews to give a +2 | 11:29 |
reldan | I’m watching, one second :) | 11:31 |
daemontool | ok | 11:32 |
reldan | For both +2 | 11:36 |
reldan | But memo has several comments here https://review.openstack.org/#/c/259905/ | 11:37 |
memogarcia | nothing out of the ordinary | 11:43 |
*** reldan has quit IRC | 11:45 | |
*** reldan has joined #openstack-freezer | 11:59 | |
*** memogarcia has quit IRC | 12:23 | |
*** reldan has quit IRC | 12:36 | |
jokke_ | daemontool: ping | 12:37 |
daemontool | jokke_,here I'm | 12:37 |
jokke_ | daemontool: happy new year | 12:37 |
jokke_ | daemontool: you had pinged me during holidays ... what's up? | 12:38 |
daemontool | jokke_, all good solved | 12:45 |
daemontool | thank you | 12:45 |
daemontool | happy new year you too! | 12:46 |
jokke_ | good stuff | 12:47 |
daemontool | >) | 12:47 |
jokke_ | I got nice 3 week break from work, OpenStack, mostly internet, etc. | 12:48 |
jokke_ | did good | 12:48 |
*** reldan has joined #openstack-freezer | 12:51 | |
daemontool | fantastic | 12:52 |
daemontool | me too | 12:53 |
daemontool | reldan, can we make a little bit more verbose the bp for parallel? https://blueprints.launchpad.net/freezer/+spec/parallel-backup-config | 12:59 |
reldan | Yes, sure. Give me a minute please | 12:59 |
daemontool | np | 12:59 |
daemontool | something like https://blueprints.launchpad.net/freezer/+spec/freezerclient | 13:00 |
reldan | I see, I see | 13:02 |
reldan | Thank you | 13:02 |
reldan | https://blueprints.launchpad.net/freezer/+spec/parallel-backup-config | 13:12 |
*** memogarcia has joined #openstack-freezer | 13:14 | |
daemontool | reldan, (thumbsup) | 13:17 |
reldan | Thank you! | 13:17 |
*** memogarcia has quit IRC | 13:31 | |
daemontool | reldan, Slashme m3m0_ are we still using apscheduler? | 13:46 |
*** vannif has joined #openstack-freezer | 13:47 | |
reldan | Slashme is still on holiday | 13:56 |
reldan | I don’t know actually about apscheduler | 13:57 |
m3m0_ | yeah we are, daemontool | 13:58 |
*** memogarcia has joined #openstack-freezer | 13:59 | |
*** EinstCrazy has joined #openstack-freezer | 14:01 | |
vannif | hi | 14:02 |
daemontool | vannif, I'm about to split | 14:05 |
daemontool | the python-freezerclient related code | 14:05 |
daemontool | please review https://blueprints.launchpad.net/freezer/+spec/freezerclient when you can | 14:05 |
daemontool | also reldan fixed the issues foudn with testing last week | 14:05 |
daemontool | Im' retesting it today | 14:06 |
daemontool | if you want to restest it too would be good | 14:06 |
daemontool | so we can merge it today | 14:06 |
*** EinstCrazy has quit IRC | 14:09 | |
openstackgerrit | Memo Garcia proposed openstack/freezer-web-ui: Add visual clues for mandatory fields in actions https://review.openstack.org/264785 | 14:10 |
openstackgerrit | Memo Garcia proposed openstack/freezer-web-ui: Add keystone 3 support for freezer ui https://review.openstack.org/256417 | 14:18 |
*** EinstCrazy has joined #openstack-freezer | 14:30 | |
*** daemontool_ has joined #openstack-freezer | 14:33 | |
*** daemontool has quit IRC | 14:34 | |
*** EinstCrazy has quit IRC | 14:40 | |
daemontool_ | vannif, ping | 15:03 |
vannif | it seems fine to me. so the freezer-agent, freezer-scheduler and freezer-api will use oslo log/conf. python-freezerclient will contain the freezerclient library (formerly known as apiclient) and the cli tool which will use cliff | 15:06 |
daemontool_ | ok | 15:16 |
daemontool_ | all: do we have quorum today | 15:37 |
daemontool_ | to do the meeting? | 15:37 |
daemontool_ | or do we want to restart next Thursday 11? | 15:38 |
daemontool_ | I'm available to do it | 15:42 |
daemontool_ | m3m0_, do you want to be the chair? | 15:42 |
daemontool_ | right now we have nothing done with cliff right? | 15:51 |
vannif | memo is afk | 15:58 |
vannif | that's right. we're not using cliff, yet | 15:58 |
*** reldan has quit IRC | 15:59 | |
daemontool_ | I think the creatino of the freezerclient needs to be done in at least 3 steps | 16:00 |
daemontool_ | 1) repo creation, code split | 16:00 |
daemontool_ | 2) change code according the new namespace | 16:00 |
daemontool_ | 3) port to cliff | 16:01 |
daemontool_ | not sure if until #3 the module is usable | 16:01 |
*** reldan has joined #openstack-freezer | 16:02 | |
memogarcia | I think we have in the calendar that the meetings will restart next week | 16:06 |
daemontool_ | ah ok | 16:06 |
daemontool_ | good | 16:06 |
daemontool_ | np | 16:06 |
memogarcia | but if everyone agrees, we can have one short for today | 16:07 |
daemontool_ | ++ | 16:07 |
daemontool_ | i-m basically working on testing parallels | 16:08 |
daemontool_ | freezerclient creation | 16:08 |
daemontool_ | and move freezer-api totally to testr | 16:08 |
memogarcia | on my side I'm enabling keystone v3 in horizon and planning the parallel upload from the ui | 16:12 |
memogarcia | it will be a major change | 16:12 |
memogarcia | now that I remember I need to move the freezer repos for windows to the new github page | 16:13 |
daemontool_ | memogarcia, good | 16:14 |
vannif | regarding the python-freezerclient: I'd like to work on the cliff stuff. how do we manage the split ? shall we leave the scheduler usable as a freezerclient until step #3 ? | 16:15 |
*** dschroeder has joined #openstack-freezer | 16:21 | |
daemontool_ | mmmhhhh | 16:25 |
daemontool_ | probably we should make the changes before in the repo, and when he code works split | 16:25 |
daemontool_ | it | 16:25 |
daemontool_ | because it will confuse a lot having a module that doesn't work | 16:25 |
daemontool_ | vannif, memogarcia reldan Slashme wha do you think? | 16:27 |
daemontool_ | frescof, ^^ | 16:27 |
daemontool_ | vannif, can we do the changes in the freezer repo first? | 16:28 |
vannif | ok,so | 16:28 |
vannif | 1) is change namespace/imports in the freezer repo (freezer-scheduler works the same) | 16:29 |
vannif | 2) implement freezerclient using cliff | 16:29 |
vannif | 3) split and move freezerclient to its own repo | 16:30 |
daemontool_ | I think that make sense | 16:30 |
vannif | step #1 impacts on freezer-web-ui | 16:30 |
daemontool_ | rather split a non working code, create a new repo and then work on the code of the new repo | 16:30 |
memogarcia | yes it does affect the ui but shiuld't be so drastic | 16:30 |
daemontool_ | it affects also the scheduler | 16:31 |
daemontool_ | or do we want to split first? | 16:31 |
daemontool_ | the think is that we are going to have code in a new repo that does't work | 16:32 |
vannif | another approach would be to start the freezer-apiclient in its own repo, and when it's working, remove the functionality from the freezer repo | 16:32 |
daemontool_ | do you mean the python-freezerclient ? | 16:32 |
vannif | the problem in this case is that we replicate the apiclient functions | 16:32 |
vannif | yes, sorry | 16:32 |
daemontool_ | yes that was what I was thinking of | 16:33 |
vannif | i meant the python-freezerclient | 16:33 |
daemontool_ | at first instance | 16:33 |
daemontool_ | what code do we place initially in that repo_ | 16:33 |
daemontool_ | ? | 16:33 |
daemontool_ | the issue is, that is not possible to create a new repo openstack/* | 16:33 |
daemontool_ | without a pypi module avaialble | 16:33 |
daemontool_ | infra wont approve | 16:34 |
vannif | ah ok ^^, I thought you were considering removing the working code from the freezer-agent repo in the first place | 16:34 |
memogarcia | it's not a problem if we have duplicate code there as long as we don't break the compatibility at any point | 16:34 |
daemontool_ | memogarcia, ++ | 16:34 |
daemontool_ | vannif, yes that was the second thought | 16:34 |
daemontool_ | are we going to place a broken module on pypi? | 16:34 |
daemontool_ | called python-freezerclient ? | 16:35 |
daemontool_ | mmhhhh | 16:35 |
daemontool_ | or we can create an empty repo | 16:35 |
daemontool_ | stating in the Readme that it's not yet ready for use | 16:35 |
daemontool_ | and not have the pypi module | 16:35 |
daemontool_ | probably I need to talk with Doug or someone to ask for some feedback | 16:36 |
vannif | we can *duplicate* most of the the freezerclient code from the freezer-agent repo to the new repo, have it working quickly, then make the transition to cliff | 16:38 |
daemontool_ | vannif, ++ | 16:39 |
daemontool_ | that's a good one | 16:39 |
daemontool_ | we just split our current code as it is | 16:39 |
daemontool_ | now | 16:39 |
daemontool_ | in a dedicated repo | 16:39 |
daemontool_ | and have it working | 16:39 |
daemontool_ | then rework with cliff | 16:39 |
daemontool_ | we need to update the namespace in web ui and scheduler | 16:39 |
daemontool_ | anyway | 16:39 |
daemontool_ | but we have to do less changes | 16:39 |
daemontool_ | Vanni I splitted most of the code | 16:40 |
daemontool_ | just need to do some change to have it work | 16:40 |
vannif | yes. but the namespace change in the web-ui and scheduler should be valid even after the switch to cliff | 16:40 |
vannif | when in doubt, let your laziness guide you to the best solution :) | 16:42 |
daemontool_ | lol | 16:43 |
daemontool_ | take alook here | 16:43 |
daemontool_ | https://github.com/daemontool/python-freezerclient | 16:43 |
daemontool_ | we need anyway an intermediate repo that will seed the openstack/python-freezerclient repo | 16:43 |
daemontool_ | vannif, there's something wrong there | 16:44 |
*** pennerc has joined #openstack-freezer | 16:44 | |
daemontool_ | Hi pennerc :), Fausto is here | 16:44 |
daemontool_ | vannif, http://paste.openstack.org/show/jaYVY6AhAhMgOeF1sKqj/ | 16:48 |
*** vannif has quit IRC | 16:51 | |
*** vannif has joined #openstack-freezer | 16:53 | |
daemontool_ | all: this is important https://review.openstack.org/264865 and https://review.openstack.org/#/c/264863/ | 16:57 |
daemontool_ | vannif, reldan memogarcia jokke_ frescof Slashme take a looks if you get the chance ^^ | 16:57 |
jokke_ | will do | 16:58 |
daemontool_ | that adds 2 new modules to openstack/requirements/global-requirements.txt | 16:59 |
daemontool_ | apscheduler and pep3143daemon | 16:59 |
daemontool_ | and the freezer projects as well on the projects.txt file | 16:59 |
daemontool_ | ok have to go now | 17:01 |
daemontool_ | vannif, we'll see the freezerclient thing tomorrow | 17:02 |
*** daemontool_ has quit IRC | 17:06 | |
openstackgerrit | Memo Garcia proposed openstack/freezer-web-ui: Add keystone 3 support for freezer ui https://review.openstack.org/256417 | 17:08 |
*** samuelBartel has quit IRC | 17:11 | |
pennerc | rcoy | 17:15 |
pennerc | wrong room | 17:15 |
*** pennerc has left #openstack-freezer | 17:15 | |
openstackgerrit | Memo Garcia proposed openstack/freezer-web-ui: Add keystone 3 support for freezer ui https://review.openstack.org/256417 | 17:23 |
*** memogarcia has quit IRC | 17:28 | |
*** reldan has quit IRC | 17:55 | |
*** memogarcia has joined #openstack-freezer | 18:18 | |
memogarcia | o/ | 18:21 |
*** dschroeder has quit IRC | 18:45 | |
*** dschroeder has joined #openstack-freezer | 18:46 | |
*** reldan has joined #openstack-freezer | 19:34 | |
*** openstackgerrit has quit IRC | 20:23 | |
*** openstackgerrit has joined #openstack-freezer | 20:25 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!