| *** openstack has joined #openstack-climate | 00:19 | |
| *** chandankumar_ has joined #openstack-climate | 03:33 | |
| *** chandankumar_ has quit IRC | 03:40 | |
| *** chandankumar_ has joined #openstack-climate | 03:55 | |
| *** chandankumar_ has quit IRC | 04:23 | |
| *** YorikSar has quit IRC | 05:37 | |
| *** chandan_kumar has quit IRC | 05:48 | |
| *** YorikSar has joined #openstack-climate | 06:32 | |
| *** saju_m has joined #openstack-climate | 07:30 | |
| *** saju_m has quit IRC | 08:50 | |
| *** saju_m has joined #openstack-climate | 08:52 | |
| *** YorikSar has quit IRC | 08:55 | |
| *** YorikSar has joined #openstack-climate | 08:56 | |
| *** chandan_kumar has joined #openstack-climate | 09:13 | |
| *** saju_m has quit IRC | 09:38 | |
| *** bauzas has joined #openstack-climate | 09:43 | |
| *** saju_m has joined #openstack-climate | 09:56 | |
| openstackgerrit | A change was merged to stackforge/climate: Updated from global requirements https://review.openstack.org/82240 | 10:52 |
|---|---|---|
| *** saju_m has quit IRC | 11:07 | |
| *** saju_m has joined #openstack-climate | 11:08 | |
| *** saju_m has quit IRC | 11:09 | |
| *** saju_m has joined #openstack-climate | 11:15 | |
| *** saju_m has quit IRC | 11:38 | |
| *** saju_m has joined #openstack-climate | 11:38 | |
| *** saju_m has quit IRC | 12:41 | |
| *** saju_m has joined #openstack-climate | 12:55 | |
| *** YorikSar has quit IRC | 12:56 | |
| *** YorikSar has joined #openstack-climate | 12:58 | |
| *** saju_m has quit IRC | 12:59 | |
| *** saju_m has joined #openstack-climate | 13:00 | |
| *** cmart has joined #openstack-climate | 13:04 | |
| *** saju_m has quit IRC | 13:04 | |
| *** saju_m has joined #openstack-climate | 13:05 | |
| cmart | Hello | 13:08 |
| cmart | DinaBelova, Are you there? | 13:08 |
| DinaBelova | o/ | 13:08 |
| DinaBelova | yep, I'm here :) | 13:08 |
| cmart | Hello! Long time no see! | 13:09 |
| DinaBelova | first of all, I'm really sorry I had no time to give you tempest instructions | 13:09 |
| DinaBelova | we had sooo many meetings, etc. | 13:09 |
| *** saju_m has quit IRC | 13:09 | |
| DinaBelova | I really feel bad about it :) | 13:10 |
| DinaBelova | How were your activities? | 13:10 |
| cmart | no problem at all... | 13:10 |
| DinaBelova | that you were working on? | 13:10 |
| *** saju_m has joined #openstack-climate | 13:10 | |
| DinaBelova | btw, on the last meeting | 13:11 |
| DinaBelova | http://eavesdrop.openstack.org/meetings/climate/2014/climate.2014-03-21-15.00.html | 13:11 |
| DinaBelova | it was discussed how to help you with tempest | 13:11 |
| cmart | I have an all-day training.. related to stuff that doesn't have anything to do with this | 13:11 |
| DinaBelova | :D | 13:11 |
| cmart | :D | 13:11 |
| DinaBelova | and it was decided to let you look in Manila project | 13:11 |
| DinaBelova | they have tempest impl in their contrib directory | 13:12 |
| DinaBelova | pretty much like the stuff we need | 13:12 |
| cmart | so I was kind of disconnected,.. | 13:12 |
| cmart | OK.. I'll check that | 13:12 |
| cmart | What is Manila? | 13:12 |
| DinaBelova | https://github.com/stackforge/manila/tree/master/contrib/tempest | 13:12 |
| DinaBelova | that's one of the endless stackforge projects)) | 13:12 |
| DinaBelova | aiming to provide network filesystem to OS | 13:13 |
| DinaBelova | as far as I remember at least | 13:13 |
| DinaBelova | cmart, btw, don't you know if I'll see Cristian & Diego today? | 13:14 |
| DinaBelova | I would like to discsuss smth with them | 13:14 |
| cmart | yes.. | 13:14 |
| DinaBelova | probably it's early for you guys now :) | 13:14 |
| cmart | I'll let them know.. | 13:14 |
| DinaBelova | ok, cool | 13:14 |
| DinaBelova | just smth about statuses :) | 13:15 |
| DinaBelova | nothing serious :) | 13:15 |
| cmart | done! | 13:15 |
| cmart | they will connect in a few minutes | 13:15 |
| DinaBelova | ok, cool | 13:15 |
| DinaBelova | :) | 13:15 |
| DinaBelova | thanks) | 13:15 |
| *** casanch1 has joined #openstack-climate | 13:15 | |
| casanch1 | hi | 13:16 |
| DinaBelova | casanch1 :) | 13:16 |
| DinaBelova | hello :) | 13:16 |
| casanch1 | DinaBelova hi | 13:16 |
| DinaBelova | casanch1, I just wanted to collect some status | 13:16 |
| DinaBelova | from what you're doing now | 13:16 |
| casanch1 | ok | 13:16 |
| DinaBelova | as far as I remember you have not only changes on review | 13:16 |
| DinaBelova | but also a task to start drafting volume reservation | 13:17 |
| casanch1 | yes | 13:17 |
| DinaBelova | and that's the thing I'm pretty interested in | 13:17 |
| casanch1 | last week I couldn't start it | 13:17 |
| casanch1 | I will start a etherpad today.. sorry but last week I was busy here | 13:17 |
| DinaBelova | np, good to know | 13:17 |
| DinaBelova | from my side I can promise you'll have less changes on review soon :) | 13:18 |
| DinaBelova | we're reviewing them right now :) | 13:18 |
| casanch1 | :) thanks | 13:18 |
| DinaBelova | And I'll have smth like this conversation with Diego when he'll come :D | 13:18 |
| *** ChanServ changes topic to "the gerrit event stream is currently hung, blocking all testing. troubleshooting is in progress (next update at 14:00 utc)" | 13:20 | |
| DinaBelova | BTW: I'll be afk ~15 minutes, please write me if you need me, I'm reading all the logs :) | 13:22 |
| casanch1 | sure, will do | 13:24 |
| *** pafuent has joined #openstack-climate | 13:28 | |
| openstackstatus | NOTICE: the issue with gerrit cleared on its own before any corrective action was taken | 13:29 |
| *** ChanServ changes topic to "Weekly meetings on Fridays 1500 UTC #openstack-meeting" | 13:29 | |
| openstackgerrit | Pablo Andres Fuente proposed a change to stackforge/climate: Support for DB migrations using Alembic https://review.openstack.org/75181 | 14:11 |
| *** YorikSar has quit IRC | 14:43 | |
| pafuent | Hi. I uploaded a new patch for https://review.openstack.org/#/c/75181/ (Alembic) | 14:51 |
| DinaBelova | pafuent, thanks! | 14:53 |
| DinaBelova | will look on it :) | 14:53 |
| pafuent | DinaBelova: Thanks | 14:53 |
| *** cmart_ has joined #openstack-climate | 15:07 | |
| *** cmart has quit IRC | 15:08 | |
| *** saju_m has quit IRC | 15:13 | |
| *** cmart_ has quit IRC | 15:16 | |
| *** YorikSar has joined #openstack-climate | 15:23 | |
| *** casanch1_ has joined #openstack-climate | 15:25 | |
| *** casanch1 has quit IRC | 15:25 | |
| *** casanch1_ is now known as casanch1 | 15:25 | |
| *** cmart_ has joined #openstack-climate | 15:45 | |
| *** bauzas has quit IRC | 16:40 | |
| *** cmart_ has quit IRC | 18:16 | |
| pafuent | DinaBelova: Are you around? | 18:19 |
| DinaBelova | pafuent, yep | 18:21 |
| pafuent | DinaBelova: Regarding trusts in PR plugin | 18:21 |
| DinaBelova | oh, yes | 18:21 |
| DinaBelova | have you prepared draft for it? | 18:22 |
| pafuent | DinaBelova: So, so | 18:22 |
| DinaBelova | :) | 18:22 |
| DinaBelova | well, ok :) | 18:22 |
| pafuent | DinaBelova: I'm testing it (manually) | 18:22 |
| DinaBelova | cool, it's nice :) | 18:22 |
| pafuent | DinaBelova: And I found that if I try to create a compute host, no context is available at RPC API level | 18:23 |
| DinaBelova | hm, at the moment you ask via Climate to create host? | 18:23 |
| pafuent | DinaBelova: So I added the trust creation at API level and pass the trust id as is done in create_lease | 18:23 |
| pafuent | DinaBelova: Yes | 18:23 |
| DinaBelova | well, that's smth that's done now | 18:23 |
| DinaBelova | currently trusts are created | 18:24 |
| DinaBelova | at the api level | 18:24 |
| DinaBelova | for virtual instances | 18:24 |
| DinaBelova | well, for the lease really | 18:24 |
| DinaBelova | probably I miss smth | 18:24 |
| DinaBelova | where did you add it? | 18:24 |
| pafuent | DinaBelova: I implemented it in the same way | 18:24 |
| DinaBelova | oh, but for the hosts as there is other maager for it | 18:25 |
| DinaBelova | manager* | 18:25 |
| DinaBelova | well, that's right I guess here | 18:25 |
| pafuent | DinaBelova: In oshost API, I added the trust creation and pass the trust_id in data parameter | 18:25 |
| pafuent | DinaBelova: The same that create_lease | 18:25 |
| pafuent | DinaBelova: But I need to do something similar for computehost_delete | 18:26 |
| DinaBelova | well, probably it'll be better for you to implement it as you feel better yourself now | 18:26 |
| DinaBelova | and we'll review code | 18:26 |
| DinaBelova | when we'll see it | 18:26 |
| pafuent | DinaBelova: The thing is that I don't have a dictionary to pass the trust_id | 18:26 |
| DinaBelova | because currently I can hardly imagine all problems and so on | 18:26 |
| pafuent | DinaBelova: So an option is to store the trust_id in the DB | 18:27 |
| DinaBelova | it's currently stored for leases, but not for the hosts | 18:27 |
| DinaBelova | am I right? | 18:27 |
| pafuent | DinaBelova: Yes | 18:27 |
| DinaBelova | well, probably it'll work here ok | 18:27 |
| DinaBelova | I guess that's nice first approximation | 18:28 |
| pafuent | DinaBelova: Ok | 18:28 |
| pafuent | DinaBelova: The sad part is that this patch will depend on the alembic patch | 18:28 |
| DinaBelova | and we'll se what might (if it is) be done better when we'll see code | 18:28 |
| DinaBelova | well, I guess it'll be merged oon anyway | 18:29 |
| DinaBelova | soon* | 18:29 |
| pafuent | DinaBelova: Yep | 18:29 |
| DinaBelova | so I guess it's ok | 18:29 |
| DinaBelova | as for the best solution here, I think it'll be better for you to present that first variant | 18:29 |
| pafuent | DinaBelova: The thing is that I don't know how to depend on two different patches | 18:29 |
| DinaBelova | and we'll together see if it can be improved | 18:29 |
| DinaBelova | what the other parent? | 18:30 |
| pafuent | DinaBelova: The scroiset_ one | 18:30 |
| pafuent | DinaBelova: For me that patch is ok | 18:30 |
| DinaBelova | ok, we'll ask Swann to rebase it | 18:31 |
| DinaBelova | once we'll merge alembic | 18:31 |
| DinaBelova | so you'll have two changes | 18:31 |
| pafuent | DinaBelova: So it could be merged | 18:31 |
| DinaBelova | or like this :) | 18:31 |
| DinaBelova | anyway, you'll be inherited from the last unmerged of them | 18:31 |
| pafuent | DinaBelova: Sorry If I didn't understand well, but the scroiset_ patch isn't merged because...? | 18:32 |
| DinaBelova | because some time ago I've -1'ed it due to your idea to test it with trusts, and forgot I did so | 18:33 |
| DinaBelova | as if I have mark on change | 18:33 |
| DinaBelova | I do not see it as "to be reviewed" in the list of changes | 18:33 |
| DinaBelova | sorry | 18:33 |
| pafuent | DinaBelova: So, you could change the -1 to a +2? | 18:33 |
| pafuent | DinaBelova: ;) | 18:33 |
| DinaBelova | I can, but after some final look tomorrow - as it's quite late here, so I can't really do that without a risk :D | 18:34 |
| pafuent | DinaBelova: :-D | 18:34 |
| DinaBelova | so now I +1'ed it :) | 18:35 |
| pafuent | DinaBelova: Ok, so after that patch is merged, I'll set my patch as dependent of alembic one in order to add the trust_id to ComputeHosts | 18:35 |
| DinaBelova | ok | 18:35 |
| DinaBelova | the only moment here will be the following: scroiset_, may you test your patch for nova client wrapper for working with different tenants / users? | 18:37 |
| DinaBelova | as I have some doubt s here about will it work correctly or not | 18:37 |
| DinaBelova | the best way to test it will be smth like "let simple user do smth with nova instances in climate and then change user to admin and try doing smth with hosts" | 18:38 |
| DinaBelova | I have some bad feeling it might not work in the way it looks now | 18:38 |
| DinaBelova | scroiset_, ^^ - take a look on it, please | 18:39 |
| DinaBelova | it'll be better to change both users and tenants, btw | 18:40 |
| pafuent | DinaBelova: Are you concern about the NovaClientWrapper singleton? | 18:52 |
| DinaBelova | yes | 18:56 |
| DinaBelova | this idea just came to my mind | 18:56 |
| pafuent | DinaBelova: I removed in my patch | 19:00 |
| DinaBelova | hm, so you've implemented in your patch smth that should be done in Swann's one? | 19:00 |
| DinaBelova | I mean | 19:01 |
| pafuent | pafuent: Maybe | 19:01 |
| DinaBelova | that's not really a good idea.. | 19:01 |
| pafuent | DinaBelova: Maybe | 19:01 |
| DinaBelova | probably you should coordinate with swann | 19:01 |
| DinaBelova | and add new patchset to his commit | 19:01 |
| DinaBelova | and ad you ass ca-author | 19:01 |
| DinaBelova | in this case | 19:01 |
| pafuent | DinaBelova: I only removed the singleton, so a co-author is too much | 19:02 |
| DinaBelova | because personally I prefer having, well, organised and solid changes in one | 19:02 |
| DinaBelova | may you please contact with scroiset_ and organise changes that should be done in his change really in his one? | 19:02 |
| pafuent | DinaBelova: OK, I will do it | 19:02 |
| DinaBelova | because when you removed singleton | 19:02 |
| DinaBelova | you also should check PR use case | 19:03 |
| pafuent | DinaBelova: Do you know at which time I can found him? | 19:03 |
| DinaBelova | as there are some nova client params to keep there | 19:03 |
| DinaBelova | well | 19:03 |
| DinaBelova | he's from France | 19:03 |
| DinaBelova | :( | 19:03 |
| DinaBelova | so probably it'll be better for you to write email to him | 19:03 |
| DinaBelova | he'll be reading logs, I'm pretty sure | 19:04 |
| DinaBelova | but still | 19:04 |
| DinaBelova | I suggest you to comment his change | 19:04 |
| DinaBelova | and leave there -1 with your comment | 19:04 |
| DinaBelova | and results of testing | 19:04 |
| DinaBelova | for example | 19:04 |
| DinaBelova | I guess it'll be better | 19:04 |
| pafuent | DinaBelova: OK | 19:05 |
| DinaBelova | and I'll ask him his day to take a look on it personally one more time | 19:05 |
| *** openstackgerrit has quit IRC | 19:18 | |
| *** openstackgerrit has joined #openstack-climate | 19:18 | |
| *** cmart has joined #openstack-climate | 19:47 | |
| *** pafuent has quit IRC | 21:08 | |
| *** casanch1 has quit IRC | 21:24 | |
| *** cmart has quit IRC | 21:42 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!