| *** chandan_kumar has joined #openstack-climate | 04:35 | |
| *** chandan_kumar has quit IRC | 05:03 | |
| *** chandan_kumar has joined #openstack-climate | 05:03 | |
| *** scroiset has quit IRC | 06:29 | |
| *** scroiset has joined #openstack-climate | 06:30 | |
| *** bauzas has joined #openstack-climate | 07:33 | |
| *** chandan_kumar has quit IRC | 09:08 | |
| *** cmart has joined #openstack-climate | 11:35 | |
| *** pafuent has joined #openstack-climate | 12:37 | |
| *** casanch1 has joined #openstack-climate | 13:06 | |
| *** chandan_kumar has joined #openstack-climate | 13:36 | |
| casanch1 | bauzas thanks for reviewing the change from tenant to project | 13:56 |
|---|---|---|
| bauzas | no pb | 13:56 |
| casanch1 | but pep8 failed :( | 13:56 |
| bauzas | very limited access to reviews as of now :) | 13:56 |
| bauzas | ah ah | 13:56 |
| bauzas | http://sbauza.wordpress.com/2014/04/05/hook-for-tox-compliancy-before-a-git-review/ | 13:56 |
| bauzas | your welcome :) | 13:57 |
| casanch1 | because it failed the verification of the sample config file :( which worked ok before | 13:57 |
| casanch1 | I guess that there was some change in libs | 13:57 |
| bauzas | rebase issue probably | 13:57 |
| casanch1 | I'm looking into that now | 13:57 |
| bauzas | this gating stuff should be removed IMHO | 13:58 |
| bauzas | DinaBelova: you ok if I'm removing this check + the conf ? | 13:58 |
| DinaBelova | well... we had the bug earlier... | 13:59 |
| DinaBelova | so possibly it's okay to discuss it there | 13:59 |
| bauzas | DinaBelova: well, the consensus is quite clear within the ecosystem :) | 13:59 |
| DinaBelova | personally I believe that for the projects less than Nova these check might be useful)) | 13:59 |
| bauzas | DinaBelova: both check and etc/ file have to be removed | 13:59 |
| DinaBelova | bauzas, well - I saw no consensus last time I was there | 14:00 |
| DinaBelova | probably it was finally decided | 14:00 |
| bauzas | let me find the discussion :) | 14:00 |
| DinaBelova | ok, cool | 14:00 |
| bauzas | ok, as per the discussion, you're right, there is no consensus yet | 14:06 |
| bauzas | I was still reading when they were saying | 14:06 |
| bauzas | tox -egendocs | 14:07 |
| bauzas | + milestone flat conf | 14:07 |
| bauzas | but afterwards, some raised concerns about practicability | 14:07 |
| bauzas | and no clear answer was made | 14:07 |
| bauzas | at least from one TC member | 14:08 |
| bauzas | so -1 to my proposal | 14:10 |
| bauzas | or we isolate the check but put it non-voting | 14:10 |
| bauzas | DinaBelova: your thoughts ? | 14:10 |
| openstackgerrit | Cristian A Sanchez proposed a change to stackforge/climate: Change references of tenant to project https://review.openstack.org/82192 | 14:16 |
| casanch1 | there you go, I have changed the sample config file | 14:17 |
| casanch1 | why isn't this sample config file generated during tox execution? | 14:19 |
| bauzas | because it is in the repo | 14:20 |
| bauzas | ideally, it should be a gate | 14:20 |
| bauzas | when a change got merged, a post-gate would generate somewhere else the cofn | 14:20 |
| bauzas | but that requires to be outside the repo | 14:21 |
| casanch1 | but it could be somehow generated before tests execution | 14:21 |
| casanch1 | and then generated before packaging it at gate | 14:21 |
| bauzas | it can't be done by Jenkins in a check | 14:22 |
| bauzas | because Jenkins is creating VMs | 14:22 |
| bauzas | for checking | 14:22 |
| bauzas | Jenkins can only propose changes | 14:23 |
| bauzas | but can't merge by its own | 14:23 |
| DinaBelova | bauzas, I guess let's not touch this conf sample now | 14:24 |
| bauzas | DinaBelova: okay | 14:24 |
| DinaBelova | let's wait the decision :) | 14:24 |
| bauzas | DinaBelova: sure thing | 14:24 |
| bauzas | DinaBelova: I'm currently reviewing how things are done for statuses in Nova and Heat | 14:24 |
| bauzas | DinaBelova: the main difference I can see is that we don't have Climate objects yet for Leases | 14:25 |
| bauzas | DinaBelova: talking about https://blueprints.launchpad.net/climate/+spec/lease-state | 14:25 |
| DinaBelova | well, first of all we need to implement state management and new lease life workflow imho | 14:26 |
| DinaBelova | as it'll, well, make it feature rich | 14:26 |
| bauzas | DinaBelova: yup, I'm just saying it will be some helper | 14:26 |
| bauzas | helper method I mean | 14:27 |
| bauzas | DinaBelova: ok, will propose something | 14:27 |
| DinaBelova | possibly... Last time I had some doubts bout this change priority for now | 14:27 |
| DinaBelova | but possibly you'll make me think different) | 14:27 |
| bauzas | DinaBelova: let me see how I could do it in Climate as quick-win | 14:28 |
| bauzas | DinaBelova: that was basically a FYI | 14:28 |
| bauzas | DinaBelova: but I think our imperative model will become at some time heavy | 14:28 |
| bauzas | DinaBelova: but let's keep it for now, sure :)à | 14:28 |
| DinaBelova | ;) | 14:29 |
| *** chandan_kumar has quit IRC | 14:31 | |
| *** chandan_kumar has joined #openstack-climate | 14:58 | |
| casanch1 | DinaBelova: this patchset is ready for your approval :) https://review.openstack.org/#/c/82192/ | 15:06 |
| cmart | Hello! | 15:17 |
| cmart | I have a question regarding climate-nova.. How can I execute the DefaultReservationController' s create method? | 15:18 |
| *** bauzas has quit IRC | 15:23 | |
| cmart | from outside, I mean.. | 15:24 |
| cmart | nova boot with the hints params will execute ReservationController's create method.. | 15:25 |
| pafuent | cmart: Create a VM without the hints | 15:25 |
| cmart | I was wondering if there is a way of doing the same but for the DefaultReservationController | 15:25 |
| cmart | pafuent: OK.. and what else? | 15:26 |
| pafuent | cmart: Do a normal nova boot | 15:26 |
| *** chandan_kumar has quit IRC | 15:27 | |
| pafuent | cmart: That's why some Tempest test fail if DefaultReservationController is active. This controller adds the hints for you | 15:27 |
| *** chandan_kumar has joined #openstack-climate | 15:28 | |
| cmart | OK.. So the DRC (DefaultReservationController ) add the hints and then the reservationController takes them and create the lease, is that correct? | 15:29 |
| cmart | so bassically, if I have a proper configuration, the DRC will automatically add the hints with just calling nova boot without parameters, right? | 15:30 |
| cmart | * "a proper "= " the proper" | 15:30 |
| cmart | pafuent? | 15:35 |
| pafuent | cmart: Yes | 15:37 |
| cmart | OK! Thanks!!! | 15:37 |
| pafuent | cmart: ;-) | 15:38 |
| *** chandan_kumar has quit IRC | 15:42 | |
| openstackgerrit | Christian Martinez proposed a change to stackforge/climate: Add scenario testing support https://review.openstack.org/87352 | 15:44 |
| *** chandan_kumar has joined #openstack-climate | 16:02 | |
| *** pafuent has quit IRC | 16:09 | |
| *** pafuent has joined #openstack-climate | 16:39 | |
| DinaBelova | casanch1, done) | 16:46 |
| casanch1 | thank you | 16:48 |
| openstackgerrit | A change was merged to stackforge/climate: Change references of tenant to project https://review.openstack.org/82192 | 16:48 |
| *** chandan_kumar has quit IRC | 17:19 | |
| *** casanch1_ has joined #openstack-climate | 18:03 | |
| *** casanch1 has quit IRC | 18:06 | |
| *** casanch1_ has quit IRC | 18:07 | |
| *** chandan_kumar has joined #openstack-climate | 18:44 | |
| *** cmart has quit IRC | 19:01 | |
| openstackgerrit | Pablo Andres Fuente proposed a change to stackforge/climate: Added use of trusts to Physical Host plugin https://review.openstack.org/84158 | 19:24 |
| *** pafuent has left #openstack-climate | 19:30 | |
| openstackgerrit | Christian Martinez proposed a change to stackforge/climate: Add scenario testing support https://review.openstack.org/87352 | 20:08 |
| *** chandan_kumar has quit IRC | 20:22 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!