| *** bauzas has quit IRC | 00:35 | |
| *** chandankumar has joined #openstack-climate | 04:40 | |
| *** SergeyLukjanov has joined #openstack-climate | 04:56 | |
| *** SergeyLukjanov has quit IRC | 05:36 | |
| *** SergeyLukjanov has joined #openstack-climate | 05:44 | |
| *** SergeyLukjanov has quit IRC | 05:49 | |
| openstackgerrit | Nikolay Starodubtsev proposed a change to stackforge/climate: Implement keystone trust support https://review.openstack.org/48002 | 07:41 |
|---|---|---|
| openstackgerrit | Nikolay Starodubtsev proposed a change to stackforge/climate: Add utils to work with openstack components https://review.openstack.org/52296 | 07:41 |
| openstackgerrit | Nikolay Starodubtsev proposed a change to stackforge/climate: Implement basic plugin for VM management https://review.openstack.org/49142 | 07:41 |
| *** bauzas has joined #openstack-climate | 08:28 | |
| *** chandankumar has quit IRC | 08:41 | |
| *** chandankumar has joined #openstack-climate | 09:06 | |
| *** DinaBelova has joined #openstack-climate | 09:07 | |
| *** SergeyLukjanov has joined #openstack-climate | 09:13 | |
| DinaBelova | o/ | 09:51 |
| DinaBelova | guys, I remind you we have small meating today :) | 09:51 |
| DinaBelova | wow | 09:51 |
| DinaBelova | sorry | 09:51 |
| DinaBelova | meeting :) | 09:51 |
| DinaBelova | I suppose we'll have it in this channel :) | 09:52 |
| Nikolay_St | yo | 10:00 |
| DinaBelova | bauzas, are you here? | 10:00 |
| DinaBelova | SergeyLukjanov, f_rossigneux :) | 10:01 |
| bauzas | yup | 10:01 |
| f_rossigneux | Hi! | 10:01 |
| bauzas | sorry, was beating with regexps | 10:01 |
| bauzas | I finally won | 10:01 |
| DinaBelova | guys, I'm glad you're here :) | 10:01 |
| DinaBelova | bauzas, congrats! | 10:01 |
| DinaBelova | ok | 10:01 |
| DinaBelova | so me, Nick, Sylvain and Francois are here | 10:02 |
| scroiset | à/ | 10:02 |
| DinaBelova | scroiset, hello! :) | 10:02 |
| scroiset | Hi Dina | 10:02 |
| DinaBelova | Nice to hear from you :) | 10:02 |
| DinaBelova | ok :) | 10:02 |
| DinaBelova | I think we may start :) | 10:02 |
| scroiset | yep | 10:03 |
| bauzas | SergeyLukjanov around ? | 10:03 |
| SergeyLukjanov | yup | 10:03 |
| SergeyLukjanov | hi folks | 10:03 |
| DinaBelova | great :) | 10:03 |
| bauzas | great | 10:03 |
| bauzas | ok, up to you, DinaBelova | 10:03 |
| DinaBelova | ok, so for our first release | 10:03 |
| DinaBelova | https://launchpad.net/climate/+milestone/0.1.0 | 10:03 |
| DinaBelova | these are things we were planning to deliver | 10:03 |
| DinaBelova | I remind you, that next Tuesday we have code freeze | 10:04 |
| DinaBelova | So this small discussion should have a result - what things we will deliver | 10:04 |
| DinaBelova | and what not | 10:04 |
| bauzas | okay | 10:04 |
| DinaBelova | let's start with BPs | 10:04 |
| bauzas | sure | 10:04 |
| DinaBelova | As you might see, all essential BPs are either implemented or waiting for review | 10:05 |
| DinaBelova | as for essential ones we have 4 needing code review | 10:05 |
| bauzas | amending for client | 10:06 |
| DinaBelova | bauzas, sorry? | 10:06 |
| bauzas | (Launchpad is not considering python-climateclient) | 10:06 |
| bauzas | so the whiteboard is incorrect | 10:06 |
| DinaBelova | I've fixed it | 10:06 |
| bauzas | there will be 3 patches for this BP, not only one | 10:06 |
| DinaBelova | yep, I added my one | 10:07 |
| DinaBelova | I'll ask you to add yours | 10:07 |
| bauzas | a 3rd one goes to the whiteboard by 1 hour | 10:07 |
| scroiset | FYI i'll start to review today or tomorrow | 10:07 |
| DinaBelova | great, guys | 10:07 |
| DinaBelova | okay | 10:07 |
| DinaBelova | we are testing virtual use case with Nick | 10:08 |
| bauzas | on our side, we are ready for being reviewed | 10:08 |
| bauzas | we hardly tested the PR usecase | 10:08 |
| DinaBelova | sure :) | 10:08 |
| bauzas | there is only the last patch for creating a lease using the client to be sent | 10:08 |
| DinaBelova | And I think almost everything is nice with your changes | 10:08 |
| DinaBelova | bauzas, I'm really glad to read that | 10:09 |
| *** chandankumar has quit IRC | 10:09 | |
| DinaBelova | also we have two high priority BPs on review now | 10:09 |
| bauzas | ok, then let's consider these patches are mandatory for 0.1 | 10:09 |
| DinaBelova | I think so | 10:09 |
| bauzas | keystone trust is reading for reviewing ? | 10:10 |
| bauzas | I saw last patches by yesterday | 10:10 |
| bauzas | Nikolay_St ? | 10:10 |
| Nikolay_St | bauzas: are you using client from climate/utils/openstack? | 10:10 |
| DinaBelova | bauzas, yes it is | 10:10 |
| Nikolay_St | bauzas: yeap | 10:10 |
| DinaBelova | we have only some questions with full workflow and we are testing that | 10:10 |
| DinaBelova | utils and trusts are ready for review | 10:11 |
| bauzas | Nikolay_St: when I'm speaking about client, I'm speaking about python-climateclient for creating a lease: ) | 10:11 |
| DinaBelova | no-no | 10:11 |
| bauzas | sorry if I misunderstood, what was the question ? | 10:12 |
| DinaBelova | Nick is typing right now :) | 10:12 |
| Nikolay_St | bauzas: one minute | 10:12 |
| Nikolay_St | bauzas: we've merged a patch that import nova(keystone)client directly | 10:12 |
| Nikolay_St | bauzas: not from climate/utils/openstack | 10:13 |
| Nikolay_St | bauzas: I think it will be better to fix it | 10:13 |
| Nikolay_St | that's what I mean | 10:13 |
| bauzas | I'm sorry, from which BP are we talking about ? | 10:13 |
| bauzas | or which review ? | 10:13 |
| bauzas | I'm lost :) | 10:14 |
| DinaBelova | we merged PR use case path | 10:14 |
| scroiset | I'm lost too | 10:14 |
| DinaBelova | patch** | 10:14 |
| Nikolay_St | one minute | 10:14 |
| Nikolay_St | I try to find it | 10:14 |
| DinaBelova | that contained working with OS clients directly | 10:14 |
| bauzas | ah, you mean about the reservation pools ? | 10:14 |
| DinaBelova | yep | 10:14 |
| bauzas | gotcha | 10:14 |
| Nikolay_St | yeap | 10:15 |
| bauzas | indeed, as the review for your client was not yet ready | 10:15 |
| DinaBelova | so I think we need to fix that after utils got ready | 10:15 |
| bauzas | I think that would be a risk for delivering to 0.1 | 10:15 |
| bauzas | we could log a bug tho | 10:15 |
| DinaBelova | we may just have a bug | 10:15 |
| DinaBelova | yep | 10:15 |
| bauzas | and tag it for 0.1.1 | 10:15 |
| DinaBelova | +1 | 10:15 |
| DinaBelova | We also have Replace oslo-incubator/rpc usage with oslo-messaging and Implements API Policies... I think I'll test policies after VR use case and try to fix that to 0.1 | 10:16 |
| DinaBelova | as for messaging it's ready and waiting for your review :) | 10:16 |
| Nikolay_St | okay | 10:16 |
| bauzas | Nikolay_St: could you please open the bug once your patch got merged ? | 10:16 |
| Nikolay_St | yeap | 10:16 |
| DinaBelova | bauzas, are you ready to review messaging? :) | 10:17 |
| bauzas | you could address the change in your PR, but that's risky for 0.1.1, here is why I propose you create the bug :) | 10:17 |
| bauzas | DinaBelova: I already began to review it :) | 10:17 |
| bauzas | could we stick to the miletstone dashboard ? | 10:17 |
| bauzas | https://blueprints.launchpad.net/climate/+spec/host-filter | 10:18 |
| DinaBelova | bauzas, i think I lost you | 10:18 |
| bauzas | is High | 10:18 |
| bauzas | I mean, we began discussing about Medium BPs, while High BPs were not yet discussed :) | 10:18 |
| bauzas | we agreed on Essential BPs | 10:18 |
| DinaBelova | sorry, missed that BP :) | 10:18 |
| bauzas | oh ok | 10:19 |
| bauzas | no worries | 10:19 |
| bauzas | ok, so please consider it for 0.1 | 10:19 |
| DinaBelova | I marked it as essential now for 0.1 :) | 10:19 |
| bauzas | cool thanks | 10:20 |
| DinaBelova | :) | 10:20 |
| bauzas | the first one Medium I have is https://blueprints.launchpad.net/climate/+spec/hosts-api-docs | 10:20 |
| DinaBelova | yes | 10:20 |
| bauzas | I propose to descope it | 10:20 |
| bauzas | to 0.1.1 | 10:20 |
| DinaBelova | +1 | 10:20 |
| scroiset | +1 | 10:20 |
| bauzas | DinaBelova: could you please tag it for 0.1.1 then ? | 10:20 |
| DinaBelova | yes, I'll add it today | 10:20 |
| DinaBelova | now I'll mark it as next | 10:20 |
| bauzas | ok | 10:21 |
| bauzas | we need to create the milestone indeed :) | 10:21 |
| SergeyLukjanov | how are you planning to schedule 0.1.1 release? | 10:21 |
| bauzas | not yet | 10:21 |
| bauzas | we need to review the scope of it | 10:21 |
| SergeyLukjanov | when everithing will be ready or some specific date | 10:21 |
| SergeyLukjanov | I mean which approach | 10:21 |
| SergeyLukjanov | make a scope to fit date or set the right date | 10:22 |
| DinaBelova | I think we should discuss that next week | 10:22 |
| DinaBelova | and create scope, date, etc. | 10:22 |
| bauzas | DinaBelova: +1 | 10:22 |
| bauzas | we also need to set our roadmap | 10:22 |
| bauzas | :) | 10:22 |
| DinaBelova | next one BP i see is https://blueprints.launchpad.net/climate/+spec/climate-policy :) | 10:22 |
| DinaBelova | bauzas, sure | 10:22 |
| bauzas | Policies already have 2 +2 | 10:23 |
| DinaBelova | As I said I need test that, although I've added my +2 | 10:23 |
| bauzas | should have been approved since a while ;) | 10:23 |
| DinaBelova | yep | 10:23 |
| DinaBelova | so I think we'll have that in 0.1 | 10:23 |
| DinaBelova | as for messaging | 10:23 |
| DinaBelova | https://blueprints.launchpad.net/climate/+spec/oslo-messaging | 10:23 |
| bauzas | please do it for policies | 10:23 |
| bauzas | this review is waiting for so long | 10:23 |
| bauzas | with 2x +2 ;) | 10:23 |
| DinaBelova | :D | 10:23 |
| DinaBelova | sure :) | 10:24 |
| DinaBelova | as for messaging change | 10:24 |
| DinaBelova | it's also waiting a while | 10:24 |
| DinaBelova | https://review.openstack.org/#/c/57880/ | 10:24 |
| bauzas | indeed, but younger than policies :p | 10:24 |
| DinaBelova | at least I added you my +2 :D | 10:25 |
| *** chandankumar has joined #openstack-climate | 10:25 | |
| DinaBelova | you - not :D:D:D | 10:25 |
| bauzas | uh uh | 10:25 |
| bauzas | I'm a busy man :p | 10:25 |
| bauzas | and French... :D | 10:25 |
| bauzas | ok, will do asap | 10:25 |
| DinaBelova | are you ok with having this feature in 0.1? | 10:25 |
| bauzas | btw. we also need to consider balancing in between giving -1 or putting to trunk and raise bug | 10:26 |
| bauzas | DinaBelova: :) | 10:26 |
| bauzas | yes | 10:26 |
| bauzas | :d | 10:26 |
| DinaBelova | ok | 10:26 |
| DinaBelova | bauzas, I think that -1 and bugs stuff depends on every CR in separate | 10:26 |
| bauzas | yeah, I'm just saying | 10:27 |
| DinaBelova | that depends, really | 10:27 |
| bauzas | regarding the delivery date for 0.1 which is close | 10:27 |
| DinaBelova | yep | 10:27 |
| bauzas | regarding the things to change | 10:27 |
| bauzas | if the delta is little, I would have a preference for raising a bug rather than asking for a new PR | 10:27 |
| DinaBelova | I think we should merge as much as possible here, if bugs are not show stoppers | 10:28 |
| bauzas | provided it doesn't break anything | 10:28 |
| bauzas | exactly | 10:28 |
| DinaBelova | +1 | 10:28 |
| DinaBelova | let's move on | 10:28 |
| DinaBelova | https://blueprints.launchpad.net/climate/+spec/pylint-support | 10:28 |
| DinaBelova | low priority Bp that was forgot :) | 10:28 |
| bauzas | you missed i18n | 10:28 |
| DinaBelova | let's descope it :) | 10:28 |
| DinaBelova | it's merged | 10:28 |
| DinaBelova | already to trunk | 10:28 |
| DinaBelova | so I think we'll have it :) | 10:28 |
| bauzas | oh dear ! | 10:28 |
| DinaBelova | :D | 10:29 |
| bauzas | ok for descoping pylint BP | 10:29 |
| bauzas | tag it as next | 10:29 |
| bauzas | 2 only BPs descoped, not high rate :) | 10:29 |
| DinaBelova | done | 10:29 |
| DinaBelova | :) | 10:29 |
| bauzas | so, for the bugs | 10:29 |
| bauzas | filter per tenant | 10:29 |
| bauzas | although it's high | 10:29 |
| DinaBelova | https://bugs.launchpad.net/climate/+bug/1256117 | 10:30 |
| DinaBelova | I think i may change my mind here | 10:30 |
| bauzas | I won't have bandwidth for addressing it | 10:30 |
| bauzas | DinaBelova: what are your thoughts ? | 10:30 |
| bauzas | that's matter of priority | 10:30 |
| DinaBelova | I think I may give it +2, but we have BP regarding to tenant -> project change | 10:31 |
| DinaBelova | so that will be done separately | 10:31 |
| DinaBelova | now let's just fix this moment | 10:31 |
| DinaBelova | ok? | 10:31 |
| SergeyLukjanov | tenant->project rename sounds like low prio | 10:31 |
| DinaBelova | SergeyLukjanov, yes | 10:31 |
| SergeyLukjanov | bug/issue, not bp | 10:32 |
| DinaBelova | that's Climate next | 10:32 |
| DinaBelova | low one thing | 10:32 |
| bauzas | yup | 10:32 |
| bauzas | the thing is | 10:32 |
| bauzas | https://bugs.launchpad.net/climate/+bug/1256117 | 10:32 |
| DinaBelova | but now we have bug Currently, DB queries don't filter on project_id, so any user can request all DB entries. | 10:32 |
| bauzas | I was having a -1 because of the non-use of Oslo lib | 10:32 |
| bauzas | as far as I remember | 10:33 |
| DinaBelova | as i remember, I set -1 because of "Please use everywhere project_id, not tenant_id." | 10:33 |
| DinaBelova | and that's low priority thing now | 10:33 |
| DinaBelova | so let's just fix bug and that's it | 10:33 |
| DinaBelova | *giving my +2* | 10:34 |
| DinaBelova | And the last unimplemented bug | 10:34 |
| DinaBelova | https://bugs.launchpad.net/climate/+bug/1263709 | 10:34 |
| DinaBelova | that occurred while client fixing | 10:35 |
| DinaBelova | as you remember our discussions about prolong-for stuff | 10:35 |
| DinaBelova | it needs code review now to work with client correctly | 10:36 |
| DinaBelova | bauzas, Nikolay_St, may you review that? | 10:36 |
| bauzas | ok, will try :) | 10:37 |
| DinaBelova | that is almost about date string format, nothing more | 10:37 |
| bauzas | we also have potentially a bug with updating the lease for PR | 10:37 |
| DinaBelova | bauzas, what is it about? | 10:38 |
| bauzas | Francois will provide a patch for it by eob today | 10:38 |
| bauzas | if we have time for reviewing it, that goes to 0.1 | 10:38 |
| DinaBelova | +1 | 10:38 |
| bauzas | if not, we will descope to 0.1.1 and change it as bug | 10:38 |
| DinaBelova | agreed | 10:38 |
| DinaBelova | So that's it :) | 10:38 |
| bauzas | cool | 10:39 |
| bauzas | thanks buddies | 10:39 |
| DinaBelova | I think we may do all described on https://launchpad.net/climate/+milestone/0.1.0 for our 1st release :) | 10:39 |
| DinaBelova | much work done | 10:39 |
| DinaBelova | thanks :) | 10:39 |
| DinaBelova | so let's do it :) | 10:39 |
| DinaBelova | #end_of_meeting :) | 10:39 |
| scroiset | ++ | 10:39 |
| bauzas | +++ | 10:41 |
| openstackgerrit | Sylvain Bauza proposed a change to stackforge/python-climateclient: Implement Lease creation for Physical reservations https://review.openstack.org/66810 | 11:31 |
| openstackgerrit | Sylvain Bauza proposed a change to stackforge/python-climateclient: Implement Lease creation for Physical reservations https://review.openstack.org/66810 | 11:33 |
| SergeyLukjanov | folks, which rtfd id are you planning to use for docs? | 11:50 |
| SergeyLukjanov | Nikolay_St, AFAIK you've worked on it | 11:51 |
| Nikolay_St | we have space on readthedocs | 11:52 |
| SergeyLukjanov | Nikolay_St, could you please post a link? | 11:54 |
| SergeyLukjanov | oh, I find it - https://readthedocs.org/projects/openstack-climate/ | 11:55 |
| SergeyLukjanov | looks like build is still failing | 11:56 |
| SergeyLukjanov | if you'll add me to the project admins, I can take a look on why it happens | 11:56 |
| SergeyLukjanov | my username is slukjanov | 11:56 |
| bauzas | I would prefer having all docs referring to http://wiki.openstack.org/wiki/Climate | 13:00 |
| bauzas | readthedocs is cool, but for projects planning to be incubated, that's better to be in wiki | 13:01 |
| *** bauzas has quit IRC | 13:05 | |
| SergeyLukjanov | bauzas, all links should be to the wiki where some list of useful links should provided | 13:26 |
| SergeyLukjanov | but all your docs should be in sphinx (at rtfd for example) and docbook | 13:27 |
| *** SergeyLukjanov_ has joined #openstack-climate | 14:35 | |
| *** SergeyLukjanov has quit IRC | 14:39 | |
| *** SergeyLukjanov_ is now known as SergeyLukjanov | 14:39 | |
| *** SergeyLukjanov has quit IRC | 14:44 | |
| *** bauzas has joined #openstack-climate | 14:45 | |
| *** SergeyLukjanov_ has joined #openstack-climate | 14:45 | |
| *** SergeyLukjanov_ is now known as SergeyLukjanov | 14:45 | |
| *** SergeyLukjanov is now known as SergeyLukjanov_ | 14:53 | |
| *** SergeyLukjanov_ is now known as SergeyLukjanov | 14:54 | |
| *** SergeyLukjanov has quit IRC | 15:09 | |
| *** SergeyLukjanov_ has joined #openstack-climate | 15:10 | |
| *** SergeyLukjanov_ is now known as SergeyLukjanov | 15:10 | |
| *** SergeyLukjanov has quit IRC | 15:19 | |
| *** SergeyLukjanov has joined #openstack-climate | 15:20 | |
| *** SergeyLukjanov has quit IRC | 15:22 | |
| openstackgerrit | Nikolay Starodubtsev proposed a change to stackforge/climate: Implement keystone trust support https://review.openstack.org/48002 | 15:22 |
| openstackgerrit | Nikolay Starodubtsev proposed a change to stackforge/climate: Add utils to work with openstack components https://review.openstack.org/52296 | 15:22 |
| openstackgerrit | Nikolay Starodubtsev proposed a change to stackforge/climate: Implement basic plugin for VM management https://review.openstack.org/49142 | 15:22 |
| *** SergeyLukjanov has joined #openstack-climate | 15:23 | |
| Nikolay_St | ola | 15:24 |
| Nikolay_St | guys | 15:24 |
| Nikolay_St | we finish with testing vm_plugin and related changes | 15:25 |
| Nikolay_St | so you can take a look on it | 15:25 |
| *** chandankumar has quit IRC | 15:35 | |
| openstackgerrit | Dina Belova proposed a change to stackforge/climate-nova: Reservation extensions to Nova API https://review.openstack.org/63097 | 15:35 |
| DinaBelova | wow, final version of extensions :) | 15:36 |
| Nikolay_St | bauzas: yo, we end with vm_plugin | 15:37 |
| Nikolay_St | bauzas: please take a look at it asap | 15:37 |
| bauzas | I'm currently rebasing the Nova Filter | 15:37 |
| bauzas | with the renaming from Dina | 15:37 |
| bauzas | once it's done, I'll review again | 15:38 |
| bauzas | DinaBelova: fyi, scroiset and I gave +2/A to the renaming of climate to climatenova | 15:38 |
| DinaBelova | yep, I saw that :) | 15:38 |
| DinaBelova | great :) | 15:38 |
| *** chandankumar has joined #openstack-climate | 15:39 | |
| bauzas | Jenkins is having a long delay for gating | 15:39 |
| bauzas | my sources are telling me around 24h to get merged a patch | 15:39 |
| *** SergeyLukjanov is now known as SergeyLukjanov_a | 15:39 | |
| *** SergeyLukjanov_a is now known as SergeyLukjanov_ | 15:40 | |
| DinaBelova | :D | 15:43 |
| *** SergeyLukjanov_ is now known as SergeyLukjanov | 15:43 | |
| DinaBelova | we'll wait :D | 15:43 |
| DinaBelova | that's nothing we can do here :) | 15:43 |
| *** DinaBelova has quit IRC | 15:43 | |
| *** SergeyLukjanov is now known as SergeyLukjanov_ | 15:44 | |
| openstackgerrit | A change was merged to stackforge/climate-nova: Fix packages naming https://review.openstack.org/66552 | 15:48 |
| bauzas | my sources were wrong :D | 15:49 |
| *** SergeyLukjanov_ is now known as SergeyLukjanov | 16:13 | |
| openstackgerrit | Sylvain Bauza proposed a change to stackforge/climate-nova: Implement Nova filter for Host Reservation Pools https://review.openstack.org/63448 | 16:30 |
| *** DinaBelova has joined #openstack-climate | 16:47 | |
| *** chandankumar has quit IRC | 17:01 | |
| bauzas | folks, leaving now | 17:04 |
| bauzas | I'll review this evening | 17:04 |
| *** bauzas has quit IRC | 17:09 | |
| *** openstackgerrit has quit IRC | 18:27 | |
| *** openstackgerrit has joined #openstack-climate | 18:27 | |
| *** bauzas has joined #openstack-climate | 20:10 | |
| *** DinaBelova has quit IRC | 20:42 | |
| *** SergeyLukjanov is now known as SergeyLukjanov_ | 21:08 | |
| *** bauzas has quit IRC | 22:53 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!