*** seroyer has quit IRC | 00:01 | |
*** thorst has joined #openstack-powervm | 00:19 | |
*** tjakobs has quit IRC | 00:19 | |
*** thorst has quit IRC | 00:21 | |
*** efried has quit IRC | 00:35 | |
*** efried has joined #openstack-powervm | 00:47 | |
*** svenkat has joined #openstack-powervm | 01:11 | |
*** YuYangWang has quit IRC | 01:14 | |
*** thorst has joined #openstack-powervm | 01:22 | |
*** thorst has quit IRC | 01:27 | |
openstackgerrit | Sridhar Venkat proposed openstack/nova-powervm: WIP: Migration scenario, SR-IOV vif contains source physical ports https://review.openstack.org/412474 | 01:42 |
---|---|---|
*** thorst has joined #openstack-powervm | 01:42 | |
*** thorst has quit IRC | 01:43 | |
*** svenkat has quit IRC | 02:00 | |
*** thorst has joined #openstack-powervm | 02:44 | |
*** thorst has quit IRC | 02:48 | |
*** tjakobs has joined #openstack-powervm | 03:19 | |
*** edmondsw has joined #openstack-powervm | 03:31 | |
*** edmondsw has quit IRC | 03:36 | |
*** thorst has joined #openstack-powervm | 04:45 | |
*** thorst has quit IRC | 04:50 | |
*** tlian has quit IRC | 04:50 | |
*** nbante has joined #openstack-powervm | 04:56 | |
*** kylek3h has quit IRC | 05:46 | |
*** kylek3h has joined #openstack-powervm | 05:46 | |
*** kylek3h has quit IRC | 05:51 | |
*** Sujeet_ has joined #openstack-powervm | 06:27 | |
*** tjakobs has quit IRC | 06:29 | |
*** thorst has joined #openstack-powervm | 06:46 | |
*** kylek3h has joined #openstack-powervm | 06:47 | |
*** thorst has quit IRC | 06:50 | |
*** kylek3h has quit IRC | 06:51 | |
*** thorst has joined #openstack-powervm | 08:47 | |
*** thorst has quit IRC | 08:52 | |
*** thorst has joined #openstack-powervm | 10:48 | |
*** thorst has quit IRC | 10:52 | |
*** XiaYan has joined #openstack-powervm | 10:56 | |
*** xia has joined #openstack-powervm | 10:56 | |
*** nbante has quit IRC | 11:49 | |
*** Cartoon has joined #openstack-powervm | 12:02 | |
*** Sujeet_ has quit IRC | 12:03 | |
*** seroyer has joined #openstack-powervm | 12:35 | |
*** smatzek has joined #openstack-powervm | 12:36 | |
seroyer | BorD__ I got an answer. They weren't real happy with the idea and strongly recommend you figure out the depenecy issue instead. But here you go: http://pastebin.com/VDXVEPXL | 12:41 |
*** thorst has joined #openstack-powervm | 12:46 | |
*** esberglu has quit IRC | 12:49 | |
*** jwcroppe has joined #openstack-powervm | 12:59 | |
*** chmod666org has quit IRC | 13:02 | |
*** chmod666org has joined #openstack-powervm | 13:03 | |
*** edmondsw has joined #openstack-powervm | 13:03 | |
*** tblakes has joined #openstack-powervm | 13:07 | |
*** svenkat has joined #openstack-powervm | 13:14 | |
*** sujeet_ has joined #openstack-powervm | 13:20 | |
*** esberglu has joined #openstack-powervm | 13:40 | |
*** apearson has joined #openstack-powervm | 13:49 | |
*** nbante has joined #openstack-powervm | 13:50 | |
*** wangqwsh has joined #openstack-powervm | 13:52 | |
*** kylek3h has joined #openstack-powervm | 13:59 | |
esberglu | #startmeeting powervm_driver_meeting | 14:00 |
openstack | Meeting started Tue Jan 17 14:00:03 2017 UTC and is due to finish in 60 minutes. The chair is esberglu. Information about MeetBot at http://wiki.debian.org/MeetBot. | 14:00 |
openstack | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 14:00 |
openstack | The meeting name has been set to 'powervm_driver_meeting' | 14:00 |
efried | o/ | 14:01 |
wangqwsh | :) | 14:01 |
efried | adreznec thorst | 14:01 |
thorst | o/ | 14:02 |
esberglu | #topic Status | 14:02 |
xia | Hi all, it's my first time to join this meeting, I'm Yan Xia,wangqw's team member | 14:02 |
thorst | hey xia! | 14:03 |
efried | Welcome. | 14:03 |
esberglu | Hi! | 14:03 |
xia | Thanks!Glad to meet you all! | 14:03 |
esberglu | Anyone have status? I'm still a bit out of the loop for where we are on this | 14:03 |
efried | I'll do a quick summary of the in-tree driver work. | 14:04 |
efried | We have four change sets out there. | 14:04 |
efried | https://review.openstack.org/391288 https://review.openstack.org/409401 https://review.openstack.org/409402 https://review.openstack.org/409404 | 14:04 |
efried | ...in that order. | 14:04 |
thorst | efried: we still need a vif one to be seeded? | 14:05 |
efried | The first one is _almost_ ready to go. I'm working on a few tweaks suggested by thorst. | 14:05 |
efried | Yes, I think we could start a vif change set and an SSP change set in parallel. | 14:05 |
nbante | Hi, I am Nilesh Bante..I am joining this meeting first time | 14:05 |
efried | Welcome nbante | 14:05 |
thorst | nbante will be helping with testing OSA for PowerVM :-) | 14:06 |
efried | The vif and SSP change sets could be a good thing for wangqwsh and xia to get started on. | 14:06 |
wangqwsh | wlc | 14:06 |
thorst | efried: the vif one needs a little bit of research | 14:06 |
efried | wangqwsh and xia, do you have systems to test on? | 14:06 |
thorst | because we need to understand where 'os-vif' is in core nova. | 14:06 |
wangqwsh | i have neo4 | 14:06 |
efried | thorst: start with OVS? | 14:06 |
thorst | well, yeah, but more importantly, do we do OVS as we have out of tree | 14:07 |
thorst | or has ocata changed the 'vif' model a bit | 14:07 |
thorst | I guess initially, lets move in what we have, and I'll do some os-vif research | 14:07 |
efried | #action thorst to research the status of os-vif in core nova | 14:08 |
efried | wangqwsh, xia - y'all good with starting on OVS and SSP change sets? | 14:08 |
wangqwsh | sure | 14:09 |
thorst | to clarify, two separate change sets. To copy/paste in the code for the vif/ssp logic from the out of tree into the in-tree driver. But only port over the stuff we need specific to those | 14:09 |
thorst | we're trying to create very small change sets for the nova team to review | 14:09 |
thorst | but at the same time keep it in sync with our out-of-tree driver | 14:10 |
efried | #action wangqwsh and xia to seed next two change sets: 1) PlugVifs and supporting code for OVS only; 2) SSP disk driver and supporting code. | 14:10 |
wangqwsh | got, | 14:10 |
*** mdrabe has joined #openstack-powervm | 14:10 | |
efried | Let me know if you need help/guidance there. | 14:10 |
wangqwsh | ok | 14:10 |
thorst | efried: I'm assuming we hold off on the in-tree CI discussion till later part of meeting? | 14:10 |
efried | I wrote the SSP driver, so I'm pretty familiar with it; thorst did the OVS work, so he's the SME. | 14:10 |
efried | thorst: sure, fine with me. | 14:11 |
efried | One more note on status: | 14:11 |
wangqwsh | i will try to read the code first. any issue i will let you know. | 14:11 |
efried | As we get code reviews on the ported code, some changes/fixes/improvements need to be flushed back to nova-powervm. That's what I've been working on while fixing up the first in-tree change set. | 14:12 |
efried | One of the things we're trying to avoid in the in-tree driver is unit tests that dig too deeply into pypowervm. | 14:12 |
thorst | +1 | 14:12 |
efried | Among other things, this means that wherever we're using pypowervm's test data files, we need to convert over to using plain ol' mocks instead. | 14:13 |
BorD__ | seroyer thanks a ton! things got wonky after upgrading rpm to the version that works with yum.. | 14:13 |
efried | Also be on the lookout for places where we're validating pypowervm lib functions - those should be replaced with mock.patches of the pypowervtm lib methods instead. | 14:14 |
efried | Sometimes we can let the pypowervm methods run, but only if we don't have to do a bunch of extra mocking and coercion in the UT to make them "work". | 14:14 |
efried | Any questions for now? | 14:15 |
wangqwsh | no | 14:15 |
thorst | agree with that. No Questions | 14:15 |
efried | esberglu: next topic? | 14:15 |
esberglu | What else do you guys want to talk about? | 14:16 |
efried | If you don't have anything else queued up, we can get into the dual-CI business. | 14:16 |
esberglu | Okay | 14:17 |
efried | #topic CI for in-tree driver | 14:17 |
efried | So the challenge here is that the in-tree driver has a tiny subset of the functionality we're testing in our existing CI. | 14:17 |
efried | We can't stop testing the out-of-tree driver | 14:17 |
esberglu | So we want a second set of tests whitelisted for the in tree | 14:18 |
efried | but we need to be able to show a passing CI for the in-tree driver before we can expect to merge it (per nova core direction, and common sense) | 14:18 |
efried | Yes, we'll need a separate environment to test in-tree. It'll involve both a whitelist for the tests around the functionality we have, and a separate conf setup to load up and configure the in-tree driver properly. | 14:19 |
efried | That env will have to proceed in lock-step with the change sets as they merge. | 14:19 |
thorst | I want to be clear | 14:19 |
thorst | It won't be a separate environment | 14:19 |
efried | So right now, it needs to be set up to target just the first (spawn/delete) change set (https://review.openstack.org/391288) | 14:19 |
thorst | it'll be a different VM in the existing CI environment | 14:19 |
thorst | which runs a different setup/test job | 14:20 |
thorst | so if nova has a patch, we kick off two jobs. One out of tree, one in tree. | 14:20 |
efried | Once that change set merges, we'll need to modify the conf & whitelist to support the second. And so on. | 14:20 |
thorst | efried: that whitelist needs to be flexible | 14:21 |
efried | thorst: what do you mean? | 14:21 |
thorst | because you'll be working on change set 3, then need to make a rev on change set 1. | 14:21 |
esberglu | Are you saying we want 2 types of ready nodes? Or just one type of ready node with diff. config and setup after | 14:21 |
thorst | esberglu: one ready node...two different jobs | 14:21 |
esberglu | Okay good | 14:21 |
thorst | one "type" of ready node | 14:21 |
efried | Sure, the whitelist will need to keep up with change set 1 - but I don't see us trying to maintain whitelists for multiple in-flight change sets at once. | 14:22 |
thorst | efried: fair enough...we'll see how that goes | 14:22 |
efried | I mean, we _could_ try to do that - but the CI infra would have to keep a mapping of change sets to conf/whitelists. That would be complicated, and probably not worth the trouble. | 14:23 |
*** tlian has joined #openstack-powervm | 14:23 | |
thorst | lets start with easy | 14:23 |
thorst | move up from there | 14:23 |
efried | At any given point, the setup should still pass for all in-flight change sets. But we can't add new tests until a change set merges. | 14:24 |
thorst | esberglu: priority wise, this second CI is probably right after getting our main CI up and running again | 14:24 |
efried | +1 | 14:24 |
efried | That and core reviews are the last inhibitor to getting the first change set merged (once I've put up the last patch set with thorst's suggested changes). | 14:25 |
esberglu | Main CI pretty much is up and running, just was failing on those 2 tests added to the skip list | 14:25 |
efried | Now, is it going to be easier to fix the CI as-is, and then do the split, or do both at once? | 14:25 |
esberglu | So I can start looking into the 2nd CI today | 14:25 |
efried | okay, I guess you answered that. | 14:25 |
efried | #action esberglu to start work on the split-CI | 14:26 |
efried | Anything else on CI for now? | 14:26 |
thorst | I'd like a status update with the main CI | 14:26 |
efried | #topic Main CI status | 14:27 |
esberglu | It seems to be working pretty well. I upgraded to zuul 2.5.1 yesterday, 2.5.0 was missing a change and was causing issues | 14:27 |
efried | are we passing? | 14:28 |
esberglu | There were 2 tests that were failing and got added to the skip list. Pretty much every failure I saw yesterday was just those 2 | 14:28 |
esberglu | A run hasn't finished since the skip list change merged | 14:28 |
efried | Are the two tests appropriate to skip, or do they need to be investigated for possible code fixes? | 14:28 |
esberglu | Not supported | 14:28 |
thorst | appropriate to skip | 14:29 |
thorst | DVR | 14:29 |
efried | beaut | 14:29 |
thorst | are we working in parallel getting the staging environment up and running? | 14:29 |
efried | Do we have some way of knowing when new tests are added so we can maybe be proactive about skipping unsupported stuff? | 14:29 |
esberglu | thorst: I was going to deploy staging this morning. I really want to get us back on the latest devstack master, the longer we wait on that the more difficult its going to be | 14:30 |
esberglu | So I was going to hit staging with that while I start working on the code for the 2nd CI | 14:30 |
thorst | esberglu: before you chase that, could you try out that DNS server that adreznec set up? | 14:31 |
thorst | I'd love to just knock that out. | 14:31 |
esberglu | adreznec: Is that change set ready? I still need to look through it | 14:31 |
adreznec | esberglu: Yeah, like I said the client-side changes to actually set clients to use the DNS mirror/apt cache aren't tested | 14:32 |
thorst | adreznec in/ | 14:32 |
adreznec | But it's ready for review/testing anyway | 14:33 |
thorst | so ready for staging | 14:33 |
adreznec | Right | 14:33 |
adreznec | There might be some wonkiness in the variables we need to handle | 14:33 |
esberglu | Okay. I will deploy with that as well then | 14:33 |
adreznec | If we're going to share one between staging and prod | 14:33 |
thorst | esberglu: we're thinking that can dramatically speed up CI runs | 14:33 |
thorst | maybe another 10-20 min | 14:33 |
esberglu | #action esberglu: Test dns / apt on staging environment | 14:33 |
esberglu | That would be sweet | 14:33 |
adreznec | Though I suppose we could have it such that in staging it just installs those things onto the management nodes | 14:34 |
thorst | yeah, and make it more stable... | 14:34 |
adreznec | Since the management node isn't getting a lot of volume there | 14:34 |
esberglu | Let's see what happens when I deploy and go from there? | 14:36 |
esberglu | Anything else for CI? | 14:37 |
thorst | not from my end. | 14:37 |
adreznec | Sure. Was just a heads up that it might not work right out of the box | 14:37 |
esberglu | Other topics? | 14:38 |
esberglu | Btw, the first run with the new skip list just went through and passed | 14:38 |
efried | sweet | 14:38 |
efried | Do we have some way of knowing when new tests are added so we can maybe be proactive about skipping unsupported stuff? | 14:38 |
esberglu | I can look into it | 14:39 |
thorst | woo | 14:39 |
efried | Cool. Low priority, below other stuff above. | 14:39 |
esberglu | #action: esberglu: Find a way to get notified of new tempest tests | 14:39 |
esberglu | efried: thorst: I think you guys wanted further discussion on the in-tree driver? | 14:41 |
thorst | nope | 14:41 |
efried | I think that was the split-CI discussion | 14:41 |
esberglu | Oh okay | 14:41 |
esberglu | Anything final topics before I end the meeting? | 14:42 |
thorst | just one future facing thing | 14:42 |
thorst | we're going to need to start looking at supporting that fileio disk/volume connector | 14:42 |
thorst | I'm trying to figure out who/how we do that...but I don't think it'll be too tough TBH | 14:42 |
thorst | probably start with cinder. | 14:42 |
thorst | target will be pike | 14:43 |
efried | need a bp? | 14:44 |
thorst | prob | 14:45 |
thorst | out of tree bp though | 14:45 |
esberglu | Alright looks like that covers it. Thanks for joining | 14:48 |
thorst | that's all I had | 14:48 |
thorst | thanks! | 14:49 |
esberglu | #endmeeting | 14:49 |
openstack | Meeting ended Tue Jan 17 14:49:03 2017 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 14:49 |
openstack | Minutes: http://eavesdrop.openstack.org/meetings/powervm_driver_meeting/2017/powervm_driver_meeting.2017-01-17-14.00.html | 14:49 |
openstack | Minutes (text): http://eavesdrop.openstack.org/meetings/powervm_driver_meeting/2017/powervm_driver_meeting.2017-01-17-14.00.txt | 14:49 |
openstack | Log: http://eavesdrop.openstack.org/meetings/powervm_driver_meeting/2017/powervm_driver_meeting.2017-01-17-14.00.log.html | 14:49 |
*** wangqwsh has quit IRC | 15:02 | |
*** burgerk has joined #openstack-powervm | 15:19 | |
*** tjakobs has joined #openstack-powervm | 15:33 | |
*** mdrabe has quit IRC | 15:49 | |
*** burgerk has quit IRC | 15:54 | |
*** mdrabe has joined #openstack-powervm | 16:00 | |
*** sujeet_ has quit IRC | 16:07 | |
*** xia has quit IRC | 16:22 | |
*** burgerk has joined #openstack-powervm | 16:26 | |
*** XiaYan has quit IRC | 16:27 | |
openstackgerrit | Sridhar Venkat proposed openstack/nova-powervm: WIP: Migration scenario, SR-IOV vif contains source physical ports https://review.openstack.org/412474 | 16:45 |
*** smatzek has quit IRC | 16:48 | |
*** smatzek has joined #openstack-powervm | 16:50 | |
openstackgerrit | Taylor Jakobson proposed openstack/nova-powervm: First pass at imagecache https://review.openstack.org/408758 | 17:02 |
openstackgerrit | Taylor Jakobson proposed openstack/nova-powervm: First pass at imagecache https://review.openstack.org/408758 | 17:10 |
*** seroyer has quit IRC | 17:39 | |
*** seroyer has joined #openstack-powervm | 17:54 | |
openstackgerrit | Sridhar Venkat proposed openstack/nova-powervm: Migration scenario, SR-IOV vif contains source physical ports https://review.openstack.org/412474 | 18:16 |
*** smatzek has quit IRC | 18:16 | |
*** nbante has quit IRC | 18:54 | |
openstackgerrit | Eric Fried proposed openstack/nova-powervm: Mock data cleanup 1 https://review.openstack.org/421448 | 19:01 |
efried | thorst: We now have pypowervm 1.0.0.4 on pypi. | 19:10 |
efried | I would like to update the global requirements, but we'll have to justify it: http://git.openstack.org/cgit/openstack/requirements/tree/README.rst#n327 | 19:10 |
efried | Can we point to any show-stopping differences from .3.9? | 19:10 |
thorst | efried: I can't | 19:18 |
thorst | so lets stick with 1.0.0.3.9 for now... | 19:18 |
efried | k, I guess we wait til the in-tree driver breaks on somethnig. | 19:18 |
thorst | fair enough | 19:19 |
adreznec | efried: Hmm... well I guess it depends where we want to go. If we want to say things like SR-IOV support, etc | 19:19 |
adreznec | Oh wait | 19:19 |
adreznec | I think iSCSI is broken in 1.0.0.3.9 | 19:19 |
adreznec | and there are the upload bits as well thorst | 19:20 |
adreznec | That aren't in there | 19:20 |
thorst | we don't use iSCSI yet. | 19:20 |
thorst | but true. | 19:20 |
adreznec | Yeah | 19:20 |
adreznec | was just thinking of major items | 19:20 |
efried | ibmi console... | 19:21 |
thorst | that was in 1.0.0.3.9 | 19:21 |
adreznec | Can we just update it on the grounds that the version number is evil? | 19:22 |
efried | x509... | 19:22 |
adreznec | I think this is the diff https://github.com/powervm/pypowervm/compare/cba2106c107f23165bdc3ab02064a35d9f1152ee...release/1.0.0.4 | 19:23 |
efried | PCI device ID (whatever that is) | 19:24 |
efried | I don't see SR-IOV differences. | 19:25 |
efried | adreznec: you and your fancy URLs. I downloaded the tarballs and diff'd 'em. | 19:25 |
efried | thorst: in response to your comment about exceptions: There's 20 in the oot driver, including a couple of abstracts (we can probably get rid of one of those). | 19:29 |
efried | No other driver seems to have its own exception classes, so I guess we should probably look toward adding them to the main one. | 19:30 |
*** Cartoon has quit IRC | 19:31 | |
thorst | efried: agree. | 19:32 |
efried | That said, I only find one nova exception that mentions a specific driver. | 19:32 |
*** apearson has quit IRC | 20:01 | |
*** apearson has joined #openstack-powervm | 20:03 | |
thorst | efried: something up with ut's? | 20:16 |
thorst | all these change sets I'm seeing proposed are failing ut.... | 20:16 |
efried | which ones? | 20:16 |
thorst | but I think its just people not running tox on their code | 20:16 |
thorst | and not real problems | 20:16 |
thorst | yeah, verified...your stuff is clean and prestine | 20:16 |
efried | :) | 20:16 |
efried | It does look like core nova moved some stuff around recently, which could be causing issues. At least I believe that's the problem with svenkat's | 20:17 |
svenkat | efried: I figured the problem… upadting UT code. I was mocking wrong API class. | 20:18 |
efried | cool | 20:18 |
openstackgerrit | Merged openstack/ceilometer-powervm: Update to match latest global-requirements https://review.openstack.org/418618 | 20:27 |
*** tjakobs has quit IRC | 20:29 | |
*** tjakobs has joined #openstack-powervm | 20:33 | |
*** apearson has quit IRC | 20:37 | |
*** apearson has joined #openstack-powervm | 20:39 | |
openstackgerrit | Sridhar Venkat proposed openstack/nova-powervm: Migration scenario, SR-IOV vif contains source physical ports https://review.openstack.org/412474 | 20:44 |
efried | thorst: https://review.openstack.org/#/c/421515/ | 20:49 |
efried | (global requirements update) | 20:49 |
openstackgerrit | Eric Fried proposed openstack/nova-powervm: Mock data cleanup 1 https://review.openstack.org/421448 | 20:56 |
thorst | adreznec: did we ever merge the bp for the image cache? | 21:35 |
thorst | I don't think we did... | 21:35 |
adreznec | Hmm | 21:35 |
adreznec | I don't think so | 21:35 |
adreznec | Kind of forgot about it | 21:35 |
thorst | yeah...I ask because I'm starting on this bp | 21:37 |
thorst | https://blueprints.launchpad.net/nova-powervm/+spec/file-io-cinder-connector | 21:37 |
adreznec | Ok | 21:50 |
adreznec | Well we should clean it up and merge it then I guess | 21:50 |
adreznec | and probably be more diligent in the future | 21:50 |
adreznec | Sigh | 21:50 |
*** thorst has quit IRC | 21:59 | |
openstackgerrit | Taylor Jakobson proposed openstack/nova-powervm: BP: Add image cache to nova-powervm https://review.openstack.org/371946 | 22:01 |
*** svenkat has quit IRC | 22:04 | |
*** burgerk has quit IRC | 22:09 | |
*** thorst has joined #openstack-powervm | 22:24 | |
*** thorst has quit IRC | 22:24 | |
*** seroyer has quit IRC | 22:24 | |
*** seroyer has joined #openstack-powervm | 22:24 | |
*** tblakes has quit IRC | 22:30 | |
openstackgerrit | Eric Fried proposed openstack/nova-powervm: BP: Add image cache to nova-powervm https://review.openstack.org/371946 | 22:32 |
*** tlian has quit IRC | 22:34 | |
*** edmondsw has quit IRC | 22:42 | |
*** thorst has joined #openstack-powervm | 22:46 | |
*** thorst has quit IRC | 22:53 | |
*** dwayne__ has quit IRC | 22:58 | |
*** tlian has joined #openstack-powervm | 22:58 | |
*** mdrabe has quit IRC | 23:00 | |
*** seroyer has quit IRC | 23:25 | |
*** edmondsw has joined #openstack-powervm | 23:25 | |
*** edmondsw has quit IRC | 23:30 | |
*** apearson has quit IRC | 23:31 | |
openstackgerrit | Taylor Jakobson proposed openstack/nova-powervm: First pass at imagecache https://review.openstack.org/408758 | 23:37 |
*** tjakobs has quit IRC | 23:42 | |
*** esberglu has quit IRC | 23:44 | |
*** esberglu has joined #openstack-powervm | 23:45 | |
*** esberglu has quit IRC | 23:49 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!