| AndyWojo | VIO is a requirement for NovaLink right? | 00:23 |
|---|---|---|
| AndyWojo | My P812L shows no VIO capabilities. | 00:29 |
| AndyWojo | Nvm... stupid 812L doesn't have PowerVM enabled. | 00:47 |
| *** chhavi has joined #openstack-powervm | 01:11 | |
| *** thorst has joined #openstack-powervm | 01:15 | |
| *** chhavi has quit IRC | 02:13 | |
| *** dwayne has joined #openstack-powervm | 02:22 | |
| *** thorst has quit IRC | 02:41 | |
| *** chhavi has joined #openstack-powervm | 02:46 | |
| *** tjakobs_ has joined #openstack-powervm | 02:49 | |
| *** tjakobs_ has quit IRC | 02:56 | |
| *** shyama_ has joined #openstack-powervm | 03:03 | |
| *** dwayne has quit IRC | 03:11 | |
| *** shyama has quit IRC | 03:11 | |
| *** dwayne has joined #openstack-powervm | 03:18 | |
| *** thorst has joined #openstack-powervm | 03:41 | |
| *** thorst has quit IRC | 03:45 | |
| *** k0da has joined #openstack-powervm | 04:05 | |
| *** chhavi has quit IRC | 04:08 | |
| *** thorst has joined #openstack-powervm | 04:12 | |
| *** nbante has joined #openstack-powervm | 04:22 | |
| *** thorst has quit IRC | 04:30 | |
| *** arunman has joined #openstack-powervm | 05:08 | |
| *** nbante has quit IRC | 05:43 | |
| *** thorst has joined #openstack-powervm | 06:27 | |
| *** thorst has quit IRC | 06:32 | |
| *** arunman has quit IRC | 06:52 | |
| *** arunman has joined #openstack-powervm | 07:02 | |
| *** thorst has joined #openstack-powervm | 07:28 | |
| *** thorst has quit IRC | 07:33 | |
| *** k0da has quit IRC | 07:56 | |
| openstackgerrit | Shyama proposed openstack/nova-powervm master: Allow vios to be filtered for rebuild https://review.openstack.org/460723 | 08:27 |
| *** thorst has joined #openstack-powervm | 08:29 | |
| *** k0da has joined #openstack-powervm | 08:43 | |
| *** adreznec has quit IRC | 08:45 | |
| *** adreznec has joined #openstack-powervm | 08:47 | |
| *** thorst has quit IRC | 08:49 | |
| *** k0da has quit IRC | 10:13 | |
| *** arunman has quit IRC | 10:31 | |
| *** arunman has joined #openstack-powervm | 10:33 | |
| *** thorst has joined #openstack-powervm | 10:46 | |
| *** smatzek has joined #openstack-powervm | 10:48 | |
| *** thorst has quit IRC | 10:51 | |
| *** k0da has joined #openstack-powervm | 11:02 | |
| *** edmondsw has quit IRC | 11:16 | |
| *** thorst has joined #openstack-powervm | 11:33 | |
| *** svenkat has joined #openstack-powervm | 11:36 | |
| *** jpasqualetto has joined #openstack-powervm | 11:57 | |
| *** edmondsw has joined #openstack-powervm | 12:17 | |
| *** smatzek has quit IRC | 12:17 | |
| *** jpasqualetto has quit IRC | 12:49 | |
| *** chhavi has joined #openstack-powervm | 12:52 | |
| chhavi | thorst: https://review.openstack.org/#/c/461863/ | 12:52 |
| chhavi | do u agree with this comment | 12:53 |
| *** jpasqualetto has joined #openstack-powervm | 12:54 | |
| shyama_ | thorst: efried https://review.openstack.org/#/c/460723 if we are mixing fileio and fc types we need to go back and change pypowervm code to accommodate that. let me know your thoughts | 12:55 |
| thorst | chhavi: responded. | 12:58 |
| thorst | shyama_: that makes sense. But I thought the rebuild map had that logic already? Because it would 'discover' the volumes and if one of the vioses couldn't discover the volume then it was kicked out | 12:59 |
| *** thorst is now known as thorst_afk | 12:59 | |
| shyama_ | thorst_afk: ok I had not done that for fileio. may be i should do that | 13:00 |
| *** jpasqualetto has quit IRC | 13:05 | |
| chhavi | thorst: in the code _ISCSI_LOOKUP_COMPLETE is we are marking true, even if the initiator is discovered or not. don't we need to mark it true only when its discovered. | 13:17 |
| chhavi | or here we are considering that we need to run the initiator discovery only once, | 13:18 |
| *** jpasqualetto has joined #openstack-powervm | 13:18 | |
| chhavi | thorst: if its not there, then it will never be there. do we mean that | 13:18 |
| chhavi | thorst_afk: i think i got it :) since its initiator, it get initialized during open-iscsi installation. which should never be changed | 13:20 |
| chhavi | thorst_afk: need workflow for this https://review.openstack.org/#/c/461863/ | 13:21 |
| *** jpasqualetto has quit IRC | 13:26 | |
| *** jpasqualetto has joined #openstack-powervm | 13:39 | |
| *** esberglu has joined #openstack-powervm | 13:44 | |
| *** dwayne has quit IRC | 13:47 | |
| efried | chhavi thorst_afk Let me ask this: is RebuildSlotMap smart enough to skip a VIOS that doesn't have any slots of the particular type it's trying to populate? | 13:58 |
| efried | I mean, why do we need to filter at all at this level? Why not just keep all the VIOSes in there, and let RebuildSlotMap figure it out. | 13:59 |
| esberglu | efried: CI is back up. Just need to update the skip list. About to spawn a node so that I can test out 1.1.3 for you | 14:08 |
| efried | esberglu your skip list change set is +2 | 14:09 |
| efried | And thanks. | 14:09 |
| *** tjakobs_ has joined #openstack-powervm | 14:11 | |
| shyama_ | efried: the vios members should be given by the volume driver is what we were thinkgin | 14:12 |
| *** jpasqualetto has quit IRC | 14:12 | |
| efried | You mean ignore the ftsk feed entirely? | 14:13 |
| shyama_ | no filter the feed based on the uuids coming from the driver | 14:16 |
| shyama_ | if there are mixed types then we provide a union of the vios and then let the is_volume_on_vios filter further | 14:22 |
| openstackgerrit | Shyama proposed openstack/nova-powervm master: Allow vios to be filtered for rebuild https://review.openstack.org/460723 | 14:30 |
| openstackgerrit | Taylor Jakobson proposed openstack/nova-powervm master: Add support for file-backed ephemeral disk https://review.openstack.org/462248 | 14:31 |
| shyama_ | efried: thorst_afk i added one more patch. let me know what you think | 14:31 |
| efried | shyama_ Right, so why bother filtering at this level at all? Just let the is_volume_on_vios do that. | 14:32 |
| *** kylek3h has joined #openstack-powervm | 14:33 | |
| shyama_ | efried: yeah that was my initial option :) so we will still keep the vios_uuids property? | 14:43 |
| openstackgerrit | Ravi Kumar Kota proposed openstack/nova-powervm master: Translation files for supported languages https://review.openstack.org/462579 | 14:45 |
| efried | shyama_ Is it necessary for it to be an instance variable? | 14:49 |
| *** smatzek has joined #openstack-powervm | 14:53 | |
| *** k0da has quit IRC | 15:14 | |
| *** dwayne has joined #openstack-powervm | 15:23 | |
| shyama_ | efried: we need it to be same for all volume drivers so I added a property | 15:42 |
| efried | Okay, I'm getting pretty seriously confused here. Especially about where the VIOS uuid/wrapper is actually being consumed. | 15:43 |
| thorst_afk | doesn't this just go back to my original proposal | 15:44 |
| thorst_afk | in the volume driver | 15:44 |
| esberglu | efried: Develop pypowervm passed on my manual run using OOT driver | 15:45 |
| thorst_afk | if the vios_uuid passed in != mgmt partition (because that's the only one we have file system access to), don't do the file system lookup | 15:45 |
| efried | esberglu Thanks. Let's move forward with 1.1.3, then? Anyone have anything else they need in? thorst_afk adreznec ? | 15:45 |
| thorst_afk | nope | 15:46 |
| adreznec | nothing here | 15:46 |
| shyama_ | we have 2 options 1. filter the vios in the volume_on_vios method in the driver. the vios_uuids is not needed in this case. 2. add the vios_uuid property in the volume driver and use it to filter in the rebuild flow and also in deploy flow in future(if needed) | 15:51 |
| shyama_ | I think option 1 is good for now. efried thorst_afk | 15:51 |
| thorst_afk | I was never opposed to option 1. | 16:05 |
| thorst_afk | efried: ? | 16:05 |
| efried | Yeah, that seems fine to me. Means we take this out entirely, yes? https://review.openstack.org/#/c/460723/3/nova_powervm/virt/powervm/slot.py@130 | 16:06 |
| thorst_afk | I think so | 16:08 |
| thorst_afk | shyama_: please confirm? | 16:08 |
| shyama_ | efried: yes | 16:08 |
| efried | Yeah, so I like that idea. That chunk of code is offensive to mine eyes. | 16:09 |
| shyama_ | ok :) | 16:09 |
| thorst_afk | efried: thoughts on https://review.openstack.org/#/c/461863/? | 16:39 |
| efried | thorst_afk What about it? | 16:40 |
| openstackgerrit | Shyama proposed openstack/nova-powervm master: Allow vios to be filtered for rebuild https://review.openstack.org/460723 | 16:40 |
| thorst_afk | review plz? | 16:40 |
| efried | thorst_afk Done. | 16:42 |
| thorst_afk | thx | 16:42 |
| shyama_ | efried: thorst_afk updated the changeset https://review.openstack.org/460723 | 16:44 |
| *** arunman has quit IRC | 17:03 | |
| thorst_afk | shyama_: I need help. Put in a comment, but I'm confused why the entire change isn't just a 2 liner | 17:07 |
| *** smatzek has quit IRC | 17:51 | |
| *** smatzek has joined #openstack-powervm | 18:22 | |
| *** smatzek has quit IRC | 18:30 | |
| *** k0da has joined #openstack-powervm | 18:56 | |
| esberglu | efried: thorst: adreznec: Turns out the services are getting started already with that systemd change. Looks like it might be a problem with pypowervm remote stuff | 18:57 |
| efried | Oh, goodie. We love dealing with that. | 18:57 |
| esberglu | http://paste.openstack.org/show/608900/ | 18:58 |
| esberglu | At least that's what it appears to be from the above | 18:58 |
| esberglu | efried: You have any time to help debug? Otherwise I can start and let you know if I find anything | 18:59 |
| efried | esberglu Yeah, fact that it says localhost implies that our session config isn't being called out. | 19:00 |
| efried | Is that happening *just* in the SEA agent? Or in all the services that use pypowervm? | 19:00 |
| esberglu | Yeah all pypowervm, not just sea | 19:02 |
| esberglu | Man it's gonna be weird getting used to the new logging system | 19:03 |
| *** jpasqualetto has joined #openstack-powervm | 19:04 | |
| *** chhavi has quit IRC | 19:08 | |
| efried | esberglu Do you need help? | 19:25 |
| esberglu | efried: Got distracted by some CI stuff for a second. About to jump back in | 19:26 |
| esberglu | But I suspect I will | 19:26 |
| efried | esberglu First thing to look for is the log messages in local2remote.py | 19:27 |
| efried | 'Successfully loaded config ...' or 'Setting Session host to: ...' | 19:28 |
| efried | If neither is there, you ain't getting local2remote. | 19:28 |
| efried | ooo, I have an idea what might be the issue. | 19:28 |
| efried | If I start a service with systemd, does it load up /etc/environment? | 19:29 |
| efried | Ima guess that's the problem. | 19:29 |
| efried | thorst_afk Hey, can we get this one through? https://review.openstack.org/#/c/456741/ | 19:30 |
| *** smatzek has joined #openstack-powervm | 19:31 | |
| thorst_afk | efried: looking | 19:33 |
| efried | edmondsw How strongly do you feel about https://review.openstack.org/#/c/460331/3/nova_powervm/virt/powervm/tasks/base.py@34 ? | 19:35 |
| efried | esberglu you too ^ | 19:35 |
| thorst_afk | efried: +2 | 19:35 |
| efried | thx | 19:35 |
| edmondsw | efried I was thinking more about not overwriting something that might be different than about efficiency | 19:37 |
| edmondsw | but maybe it being different would be a bug? | 19:37 |
| efried | edmondsw Most likely, yeah. | 19:37 |
| edmondsw | I don't feel too strongly about it | 19:37 |
| efried | k, I'm gonna push it. | 19:37 |
| esberglu | efried: Yeah looks like I will have to specify the env in the systemd files | 19:39 |
| esberglu | And cool | 19:39 |
| efried | esberglu Makes sense. | 19:39 |
| efried | thorst_afk Hacking rules in networking-powervm appear to be getting inherited from a mainline project, which is forbidding log translation. | 19:42 |
| thorst_afk | o boy | 19:42 |
| efried | Do we a) break away from mainline hacking rules, or b) remove translation? | 19:42 |
| thorst_afk | lets fix it then | 19:42 |
| thorst_afk | remove translation | 19:42 |
| thorst_afk | :-/ | 19:42 |
| efried | on it. | 19:42 |
| openstackgerrit | Merged openstack/nova-powervm master: Get flavor from instance https://review.openstack.org/456741 | 19:42 |
| esberglu | efried: yep that was it | 19:48 |
| efried | esberglu cool | 19:48 |
| esberglu | Seeing if anything else will fail us now | 19:49 |
| efried | esberglu https://review.openstack.org/#/c/461147/ needs a rebase | 20:02 |
| efried | Probably fault of 'get flavor from instance'. | 20:03 |
| efried | esberglu Ditto https://review.openstack.org/#/c/460331/ | 20:03 |
| esberglu | efried: on it | 20:12 |
| openstackgerrit | Eric Berglund proposed openstack/nova-powervm master: In Tree Backport: Exception handling #1 https://review.openstack.org/461147 | 20:40 |
| openstackgerrit | Eric Berglund proposed openstack/nova-powervm master: In-Tree Backport: TaskFlow https://review.openstack.org/460331 | 20:52 |
| *** thorst_afk has quit IRC | 20:59 | |
| openstackgerrit | Eric Fried proposed openstack/networking-powervm master: Remove log t9n, py34->35, pep8, other tox fixes https://review.openstack.org/462737 | 21:00 |
| openstackgerrit | Eric Fried proposed openstack/networking-powervm master: Re-home neutron-lib constants https://review.openstack.org/462303 | 21:01 |
| *** svenkat has quit IRC | 21:02 | |
| *** jpasqualetto has quit IRC | 21:03 | |
| *** thorst_afk has joined #openstack-powervm | 21:03 | |
| edmondsw | efried CI passed for https://review.openstack.org/#/c/460331 | 21:05 |
| efried | edmondsw How can you tell? | 21:05 |
| edmondsw | efried oh is it more than seeing Jenkins +1 ? | 21:06 |
| efried | You watching jenkins? | 21:06 |
| efried | edmondsw Yeah, our CI jobs run tempest tests with our driver. | 21:06 |
| efried | It's, like, esberglu's whole job. | 21:06 |
| edmondsw | so how do you tell, if it doesn't update the review? | 21:07 |
| edmondsw | I assumed our CI was one of the jenkins gate jobs listed in the review as SUCCESS | 21:07 |
| *** thorst_afk has quit IRC | 21:08 | |
| edmondsw | efried isn't that the point of the CI, to post results where folks can see them? | 21:08 |
| efried | edmondsw It does update the review. | 21:08 |
| efried | edmondsw See e.g. https://review.openstack.org/#/c/443189/ | 21:09 |
| efried | Under "IBM PowerVM CI" | 21:09 |
| edmondsw | efried ahhh... k | 21:09 |
| efried | edmondsw It's considerd a "third-party" CI. | 21:10 |
| edmondsw | efried right... I just hadn't thought that through apparently | 21:10 |
| efried | It doesn't get to vote for nova proper - but it does gate changes in our *-powervm projects. | 21:10 |
| esberglu | edmodsw: Yeah it just takes way longer than the jenkins gating jobs | 21:10 |
| adreznec | Yeah, that's because the jenkins gating jobs for the *-powervm projects don't run devstack, just pep8/flake/etc | 21:11 |
| esberglu | The systemd stuff is going well. Got the remote pypowervm working. I think the only other major change due to that is going to be log publishing | 21:13 |
| efried | esberglu Yeah, that's gonna take some getting used to. | 21:15 |
| *** jpasqualetto has joined #openstack-powervm | 21:15 | |
| efried | esberglu sdague sent a note with some pointers to the devstack functions that pull the relevant logs from journald | 21:15 |
| esberglu | Yep. Saw that, haven't tried any of it yet though | 21:15 |
| efried | k | 21:15 |
| *** smatzek has quit IRC | 21:27 | |
| *** dwayne has quit IRC | 22:02 | |
| *** edmondsw has quit IRC | 22:12 | |
| *** edmondsw has joined #openstack-powervm | 22:15 | |
| *** edmondsw has quit IRC | 22:20 | |
| *** jpasqualetto has quit IRC | 22:21 | |
| *** tjakobs_ has quit IRC | 22:44 | |
| *** k0da has quit IRC | 22:57 | |
| *** tjakobs_ has joined #openstack-powervm | 23:08 | |
| *** tjakobs_ has quit IRC | 23:13 | |
| *** edmondsw has joined #openstack-powervm | 23:25 | |
| *** edmondsw has quit IRC | 23:30 | |
| *** svenkat has joined #openstack-powervm | 23:55 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!