| openstackgerrit | OpenStack Proposal Bot proposed openstack/ceilometer-powervm master: Updated from global requirements https://review.openstack.org/553168 | 06:48 |
|---|---|---|
| openstackgerrit | OpenStack Proposal Bot proposed openstack/networking-powervm master: Updated from global requirements https://review.openstack.org/552770 | 07:20 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/nova-powervm master: Updated from global requirements https://review.openstack.org/553212 | 07:27 |
| *** AlexeyAbashkin has joined #openstack-powervm | 08:13 | |
| *** adi______ has joined #openstack-powervm | 11:20 | |
| *** Alexey_Abashkin has joined #openstack-powervm | 11:20 | |
| *** adi_____ has quit IRC | 11:27 | |
| *** adi______ is now known as adi_____ | 11:27 | |
| *** AlexeyAbashkin has quit IRC | 11:28 | |
| *** adi_____ has quit IRC | 11:28 | |
| *** adi_____ has joined #openstack-powervm | 11:28 | |
| *** Alexey_Abashkin is now known as AlexeyAbashkin | 11:28 | |
| *** edmondsw has joined #openstack-powervm | 12:15 | |
| *** apearson has joined #openstack-powervm | 12:59 | |
| *** tjakobs has joined #openstack-powervm | 13:53 | |
| *** esberglu has joined #openstack-powervm | 14:00 | |
| -openstackstatus- NOTICE: POST_FAILURE results on Tempest-based jobs since the most recent Zuul security fixes are being investigated; rechecking those won't help for now but we'll keep you posted once a solution is identified | 14:10 | |
| *** ChanServ changes topic to "POST_FAILURE results on Tempest-based jobs since the most recent Zuul security fixes are being investigated; rechecking those won't help for now but we'll keep you posted once a solution is identified" | 14:10 | |
| openstackgerrit | Merged openstack/nova-powervm master: Updated from global requirements https://review.openstack.org/553212 | 14:11 |
| edmondsw | esberglu see thread starting with http://lists.openstack.org/pipermail/openstack-dev/2018-March/128310.html | 14:15 |
| efried | esberglu: Can you look into this failure when you get a chance please: https://review.openstack.org/#/c/552770/ | 14:21 |
| esberglu | efried: That's what the thread ^ is about | 14:22 |
| efried | oh, okay. | 14:22 |
| efried | I leave it in your capable hands. | 14:22 |
| edmondsw | this is also going to be interesting: http://lists.openstack.org/pipermail/openstack-dev/2018-March/128352.html | 14:23 |
| efried | edmondsw: It'll make it easier for us to bump pypowervm OOT. Otherwise, I don't really see a huge impact for us. | 14:31 |
| esberglu | efried: edmondsw: I'm getting this error on most of the UTs after following the info in that thread | 14:55 |
| esberglu | http://paste.openstack.org/show/701814/ | 14:55 |
| esberglu | https://github.com/openstack/networking-powervm/blob/master/networking_powervm/tests/unit/plugins/ibm/powervm/base.py#L89 | 14:55 |
| esberglu | ^ seems like we need to update that | 14:55 |
| efried | uh oh | 14:55 |
| efried | This may have something to do with the big "policy in code" movement that peaked in Queens. | 14:56 |
| efried | If you get stuck, ask lbragstad in -keystone | 14:56 |
| edmondsw | I can help with that | 14:56 |
| edmondsw | let me look | 14:56 |
| efried | Or ask edmondsw. | 14:56 |
| efried | :P | 14:56 |
| esberglu | edmondsw: Only changes I have made so far | 14:57 |
| esberglu | - remove neutron from deps in tox.ini | 14:57 |
| esberglu | - add neutron to requirements.txt | 14:57 |
| esberglu | that's it | 14:57 |
| edmondsw | k | 14:58 |
| edmondsw | esberglu when we were using git, etc/policy.json would be wherever the git dir was, so this would work | 15:02 |
| edmondsw | but pypi isn't going to install that file into site-packages | 15:03 |
| edmondsw | I'm not sure if it would put it in /etc/policy.json or just not install one at all | 15:03 |
| edmondsw | probably the former | 15:03 |
| edmondsw | I would try changing that method to look in /etc/policy.json instead of trying to build a path based on base.__file__ | 15:04 |
| edmondsw | then to efried's point we probably also need to handle the case that the file doesn't exist... but may not have to solve that to get this working | 15:06 |
| edmondsw | let me try it | 15:06 |
| *** apearson_ has joined #openstack-powervm | 15:31 | |
| *** apearson has quit IRC | 15:33 | |
| openstackgerrit | Eric Berglund proposed openstack/networking-powervm master: Move neutron to requirements.txt https://review.openstack.org/553459 | 16:10 |
| esberglu | edmondsw: efried: ^ should fix tox | 16:10 |
| openstackgerrit | Eric Fried proposed openstack/networking-powervm master: Updated from global requirements https://review.openstack.org/552770 | 16:14 |
| efried | esberglu: Thanks. Rebased ^ on top of it. | 16:14 |
| *** AlexeyAbashkin has quit IRC | 16:32 | |
| edmondsw | esberglu yep, there is a second version of the file there, and appears to be identical. interesting | 17:11 |
| edmondsw | esberglu sorry for the delay. Ben dropped in | 17:11 |
| openstackgerrit | Merged openstack/networking-powervm master: Move neutron to requirements.txt https://review.openstack.org/553459 | 17:24 |
| efried | woot | 17:25 |
| efried | edmondsw: want to pull https://review.openstack.org/#/c/552770/ out of the gate til it passes our CI? | 17:26 |
| edmondsw | efried it had already passed our CI on an earlier run... still think I should? | 17:26 |
| efried | I yanked it. | 17:27 |
| efried | and yes, definitely. | 17:27 |
| efried | cause now it's based on something else that, who knows, could affect it. | 17:27 |
| edmondsw | true | 17:27 |
| edmondsw | I guess... not sure how... but done | 17:27 |
| edmondsw | efried you can remove your -2 now | 17:28 |
| edmondsw | I added a CI recheck | 17:28 |
| efried | Actually don't know how the gate works on that. | 17:28 |
| edmondsw | efried oh, because the gate jobs had already started... yeah, might need to keep it, I dunno | 17:28 |
| efried | If it already went into the gate... right | 17:28 |
| edmondsw | I would hope not, but better safe I guess | 17:29 |
| efried | I don't know either, so I'll just leave it in place for now. | 17:29 |
| edmondsw | efried gate succeeded, so should be ok to remove the -2 now | 17:38 |
| efried | done | 17:39 |
| *** AlexeyAbashkin has joined #openstack-powervm | 18:54 | |
| *** AlexeyAbashkin has quit IRC | 18:58 | |
| *** esberglu has quit IRC | 19:07 | |
| *** esberglu has joined #openstack-powervm | 19:09 | |
| *** esberglu_ has joined #openstack-powervm | 19:12 | |
| openstackgerrit | Merged openstack/networking-powervm master: Updated from global requirements https://review.openstack.org/552770 | 19:13 |
| *** esberglu has quit IRC | 19:14 | |
| *** ChanServ changes topic to "This channel is for PowerVM-related development and discussion. For general OpenStack support, please use #openstack." | 19:25 | |
| -openstackstatus- NOTICE: The regression stemming from one of yesterday's Zuul security fixes has been rectified, and Devstack/Tempest jobs which encountered POST_FAILURE results over the past 24 hours can safely be rechecked now | 19:25 | |
| edmondsw | esberglu we're not running CI on neutron or ceilometer changes, only neutron-powervm and ceilometer-powervm changes? | 19:33 |
| edmondsw | remind me why? | 19:33 |
| esberglu_ | edmondsw: We actually do run on neutron/ceilometer, we just don't vote | 19:34 |
| esberglu_ | We would need to talk to those projects individually to get approved | 19:35 |
| edmondsw | you mean we don't publish results | 19:35 |
| esberglu_ | yeah | 19:35 |
| edmondsw | ok, I'll add a TODO for that | 19:35 |
| esberglu_ | There are also neutron and telemetry specific tempest plugins that we would probably have to be running in addition to or instead of base tempest | 19:36 |
| esberglu_ | I haven't really looked into those much | 19:36 |
| edmondsw | oh, we do have that on the TODO list :) | 19:38 |
| edmondsw | added the tempest plugin bit... didn't see that there | 19:39 |
| edmondsw | low priority | 19:39 |
| esberglu_ | edmondsw: efried: Is resize not supported if using ssp? | 20:25 |
| esberglu_ | Thought I saw some tempest tests hitting that, but I don't think it is | 20:25 |
| efried | uh | 20:26 |
| efried | a resize that includes disk, or any resize? | 20:26 |
| efried | Let me unpack my brain on you: | 20:26 |
| edmondsw | esberglu_ I believe resize is *only* supported if you are using ssp | 20:26 |
| esberglu_ | Looks like it doesn't work if you are resizing to a larger flavor | 20:27 |
| efried | I do recall that you can't resize an LU. If you do that, you end up with an LU of the new requested size... full of zeros. | 20:27 |
| edmondsw | larger in what sense? | 20:27 |
| edmondsw | larger disk? | 20:27 |
| esberglu_ | Larger disk, ssp doesn't have an extend_disk() method | 20:27 |
| efried | So if we were to support such resize, we would have to create a new LU and copy the data over. | 20:27 |
| edmondsw | ah... that may be | 20:27 |
| edmondsw | larger CPU or mem should be fine | 20:27 |
| efried | Or, yeah, convince the platform to support extend_disk. | 20:27 |
| efried | So right, knowing that, we may have blocked resize of SSP-backed VMs entirely. | 20:28 |
| efried | OR | 20:28 |
| efried | we might have been smarter and allowed resizes as long as they don't change disk size. | 20:28 |
| edmondsw | if we wanted to create a new LU and copy over, the good news there is that community resize is not live resize, which vastly simplifies that | 20:28 |
| efried | And I don't know offhand which we did. | 20:28 |
| esberglu_ | Looks like resize works for SSP only, but not if you want to extend the disk | 20:29 |
| esberglu_ | cool tx | 20:29 |
| efried | nice | 20:30 |
| edmondsw | https://github.com/openstack/nova-powervm/blob/master/nova_powervm/virt/powervm/driver.py#L1126 | 20:30 |
| edmondsw | https://github.com/openstack/nova-powervm/blob/master/nova_powervm/virt/powervm/driver.py#L1148 | 20:30 |
| edmondsw | looks like it would actually try to ExtendDisk... but we're saying that won't work? | 20:31 |
| edmondsw | kinda silly to try it then | 20:31 |
| esberglu_ | edmondsw: It would give you a not implemented error if you tried it | 20:31 |
| edmondsw | I guess that's there for the _is_booted_from_volume case | 20:32 |
| esberglu_ | But we could probably just raise the NotImplementedError as soon as we see the new size is bigger | 20:32 |
| edmondsw | but I don't think we support bfv for anything other than PowerVC? | 20:33 |
| edmondsw | efried do you know if bfv would work with nova-powervm and a non-PowerVC OpenStack (e.g. devstack)? | 20:33 |
| edmondsw | wonder if anyone has ever tried that | 20:34 |
| efried | I don't know. AFAIK, BFV is new in Q, so the answer is probably "we haven't tested it, no idea" | 20:34 |
| edmondsw | I don't think that's new in Q... | 20:34 |
| edmondsw | e.g. this is newton: https://docs.openstack.org/newton/user-guide/cli-nova-launch-instance-from-volume.html | 20:36 |
| efried | Hum, we have code around is_booted_from_volume. | 20:36 |
| edmondsw | all PowerVC VMs use a volume as the boot disk... but that's not what nova calls bfv | 20:36 |
| edmondsw | I wonder if that's what that method is covering today, though | 20:37 |
| edmondsw | we may have to try it and see | 20:37 |
| edmondsw | thorst may have some knowledge he can drop on us | 20:37 |
| edmondsw | esberglu_ I would recommend trying bfv with the in-tree vSCSI commit, and if that doesn't work we can simplify the code as we port resize in-tree | 20:41 |
| edmondsw | if it does work, then we need to keep the ExtendDisk for the bfv case | 20:41 |
| *** edmondsw has quit IRC | 21:02 | |
| *** apearson_ has quit IRC | 21:25 | |
| *** esberglu_ has quit IRC | 21:41 | |
| *** esberglu has joined #openstack-powervm | 21:44 | |
| *** esberglu_ has joined #openstack-powervm | 21:45 | |
| *** esberglu has quit IRC | 21:48 | |
| *** esberglu_ has quit IRC | 21:50 | |
| *** tjakobs has quit IRC | 21:59 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!