| *** apearson has joined #openstack-powervm | 00:40 | |
| *** mdrabe has quit IRC | 00:57 | |
| thorst | adreznec: need to figure out https://review.openstack.org/#/c/452220/ | 01:01 |
|---|---|---|
| thorst | is the bot running now or do we just need to update the manual requirements again? | 01:01 |
| adreznec | thorst: Ah, I see | 01:02 |
| adreznec | I bet it's the pypowervm version | 01:02 |
| adreznec | We moved it to 1.1.1 in requirements but that patchset is out of date | 01:02 |
| adreznec | 1 sec | 01:02 |
| thorst | adreznec: you weren't supposed to respond tonight... | 01:02 |
| adreznec | lol | 01:02 |
| openstackgerrit | Adam Reznechek proposed openstack/networking-powervm master: Manual sync with upstream requirements https://review.openstack.org/452220 | 01:06 |
| openstackgerrit | Adam Reznechek proposed openstack/ceilometer-powervm master: Manual sync with upstream requirements https://review.openstack.org/452222 | 01:07 |
| thorst | adreznec: these ceilometer-powervm ones will be tricky | 01:16 |
| thorst | given that we didn't merge them when gordc proposed them, I think we're going to have to wrap many of them into a single change set | 01:16 |
| adreznec | Yeah... kind of a pain. We could probably rework them into a patch series if we wanted | 01:19 |
| adreznec | But probably not worth the effort | 01:19 |
| thorst | well, I'm just picking up gordc's first | 01:20 |
| thorst | and then working from there | 01:20 |
| thorst | I should connect up with him and thank him | 01:20 |
| adreznec | Yeah... downside to not having a more thorough CI on ceilometer-powervm | 01:21 |
| thorst | we'll get there someday | 01:21 |
| thorst | ok...I think I have them sorted. | 01:24 |
| openstackgerrit | Drew Thorstensen (thorst) proposed openstack/ceilometer-powervm master: use new instance polling interface https://review.openstack.org/448134 | 01:28 |
| *** thorst has quit IRC | 01:30 | |
| *** tjakobs__ has joined #openstack-powervm | 01:30 | |
| *** thorst has joined #openstack-powervm | 01:30 | |
| *** thorst has quit IRC | 01:34 | |
| *** edmondsw has joined #openstack-powervm | 01:39 | |
| *** tjakobs__ has quit IRC | 01:39 | |
| *** edmondsw has quit IRC | 01:43 | |
| *** esberglu has joined #openstack-powervm | 01:58 | |
| *** esberglu has quit IRC | 02:03 | |
| *** apearson has quit IRC | 02:13 | |
| *** thorst has joined #openstack-powervm | 02:31 | |
| *** thorst has quit IRC | 02:51 | |
| *** esberglu has joined #openstack-powervm | 03:47 | |
| *** esberglu has quit IRC | 03:52 | |
| *** shyama has joined #openstack-powervm | 04:24 | |
| *** tjakobs__ has joined #openstack-powervm | 04:40 | |
| *** thorst has joined #openstack-powervm | 04:48 | |
| *** thorst has quit IRC | 04:52 | |
| *** tjakobs__ has quit IRC | 04:54 | |
| *** jay1_ has joined #openstack-powervm | 05:32 | |
| *** esberglu has joined #openstack-powervm | 05:37 | |
| *** esberglu has quit IRC | 05:41 | |
| *** thorst has joined #openstack-powervm | 05:48 | |
| *** jay2__ has joined #openstack-powervm | 05:50 | |
| *** jay2__ has quit IRC | 05:50 | |
| *** thorst has quit IRC | 05:53 | |
| *** k0da has joined #openstack-powervm | 06:11 | |
| *** thorst has joined #openstack-powervm | 06:49 | |
| *** thorst has quit IRC | 06:54 | |
| *** thorst has joined #openstack-powervm | 07:50 | |
| *** thorst has quit IRC | 08:09 | |
| *** openstackgerrit has quit IRC | 09:03 | |
| *** thorst has joined #openstack-powervm | 09:06 | |
| *** thorst has quit IRC | 09:11 | |
| *** esberglu has joined #openstack-powervm | 09:14 | |
| *** esberglu has quit IRC | 09:19 | |
| *** k0da has quit IRC | 10:43 | |
| *** smatzek has joined #openstack-powervm | 11:07 | |
| *** thorst has joined #openstack-powervm | 11:08 | |
| *** thorst has quit IRC | 11:13 | |
| *** shyama has quit IRC | 11:26 | |
| *** thorst has joined #openstack-powervm | 11:47 | |
| *** edmondsw has joined #openstack-powervm | 12:12 | |
| *** shyama has joined #openstack-powervm | 12:13 | |
| *** jpasqualetto has joined #openstack-powervm | 12:24 | |
| *** nbante has joined #openstack-powervm | 12:58 | |
| *** apearson has joined #openstack-powervm | 13:00 | |
| *** esberglu has joined #openstack-powervm | 13:00 | |
| efried | less dudiss | 13:01 |
| esberglu | #startmeeting powervm_driver_meeting | 13:01 |
| openstack | Meeting started Tue Apr 11 13:01:14 2017 UTC and is due to finish in 60 minutes. The chair is esberglu. Information about MeetBot at http://wiki.debian.org/MeetBot. | 13:01 |
| openstack | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 13:01 |
| openstack | The meeting name has been set to 'powervm_driver_meeting' | 13:01 |
| efried | \o | 13:01 |
| thorst | o/ | 13:02 |
| esberglu | #topic In Tree Driver | 13:02 |
| thorst | efried: review this in background | 13:02 |
| thorst | https://review.openstack.org/#/c/448134/ | 13:02 |
| efried | ack | 13:02 |
| esberglu | efried: Need anything from us in-tree? | 13:03 |
| esberglu | Other than reviewing once the rebases all go through? | 13:03 |
| efried | esberglu I don't think so. Assume CI will be a separate topic. | 13:03 |
| esberglu | yeah | 13:03 |
| *** jay1_ has quit IRC | 13:04 | |
| esberglu | #topic OOT Driver | 13:04 |
| esberglu | thorst: Are there any reviews that we are still trying to push through CI OOT? | 13:05 |
| esberglu | I think at least most of them have a CI +1 now | 13:05 |
| thorst | esberglu: tons | 13:05 |
| thorst | we need to do a full on scrub of things | 13:05 |
| esberglu | Yeah I think that's the best bet | 13:06 |
| esberglu | Gonna be a pain to redeploy until that packaging stuff while stacking is fixed though | 13:06 |
| thorst | well, I think that we have a few things that we should chat through and then once we're relatively solid...we can then go ahead and redeploy OOT | 13:07 |
| thorst | OOT CI... | 13:07 |
| thorst | net, there are about 6 or so change sets we need to scrub through. | 13:07 |
| *** mdrabe has joined #openstack-powervm | 13:07 | |
| esberglu | Links? I will keep an eye and try to recheck as soon as any fail | 13:09 |
| thorst | well, some will take effort. But here's a first pass (and they're not just blocked by CI, but also reviews) | 13:09 |
| thorst | https://review.openstack.org/448134 | 13:10 |
| *** newbie|0 has joined #openstack-powervm | 13:10 | |
| thorst | https://review.openstack.org/448249 | 13:10 |
| thorst | https://review.openstack.org/447762 | 13:10 |
| thorst | https://review.openstack.org/453024 | 13:10 |
| thorst | https://review.openstack.org/452712 | 13:10 |
| thorst | https://review.openstack.org/447763 | 13:10 |
| thorst | https://review.openstack.org/416667 | 13:11 |
| *** mdrabe has quit IRC | 13:11 | |
| thorst | I think that's it | 13:12 |
| thorst | from my end | 13:12 |
| esberglu | Okay. I'm kicking off rechecks on the ones that haven't had a run yet | 13:12 |
| *** nbante has quit IRC | 13:12 | |
| *** newbie|0 has quit IRC | 13:13 | |
| esberglu | Okay is that it for OOT (CI separate topic) | 13:14 |
| *** nbante has joined #openstack-powervm | 13:14 | |
| thorst | longer term... | 13:14 |
| adreznec | The other two that will need reviews are https://review.openstack.org/#/c/452220/ and https://review.openstack.org/#/c/452222/ | 13:14 |
| thorst | I would like to see us start to look at PCI direct attach | 13:14 |
| adreznec | the latter is gated behind those ceilometer changes thorst posted | 13:14 |
| thorst | in case anyone has spare cycles. | 13:14 |
| adreznec | But the former should be good, though it appears to have failed tempest with different tests than before the recheck... | 13:15 |
| esberglu | Are you saying it failed different tests? Or ran different tests? | 13:15 |
| adreznec | Failed | 13:15 |
| esberglu | Whew okay good | 13:16 |
| adreznec | Haha yeah, the latter would definitely be more confusing to deal with | 13:16 |
| adreznec | esberglu: Do you know if the CI was down/didn't run on a patch when it first came in and I do a powervm: recheck comment on it, will CI run on that patch? | 13:17 |
| esberglu | Yeah it will | 13:17 |
| adreznec | Talking https://review.openstack.org/#/c/454506/ in particular | 13:17 |
| adreznec | Ok | 13:17 |
| esberglu | thorst: You have anything else you wanted to say about PCI direct attach, or just for awareness at this point? | 13:19 |
| thorst | just awareness...start thinking about it | 13:20 |
| efried | Yeah, that was a bit of a bomb there, thorst | 13:20 |
| efried | We talking queens? | 13:20 |
| thorst | yeah | 13:20 |
| thorst | long term game | 13:20 |
| efried | ight | 13:20 |
| thorst | I suspect the code for that right now is very kvm centric | 13:21 |
| efried | Oh hell yeah. | 13:21 |
| thorst | so I'd like our team to build an understanding of what's there...does it fit...do we have changes needed in the core driver...etc... | 13:21 |
| efried | #agreed PCI direct attach is a big fish, long term. | 13:21 |
| thorst | core driver --> manager | 13:21 |
| efried | (been looking for an opportunity to use #agreed) | 13:22 |
| thorst | heh | 13:22 |
| efried | (oh, it only works for the chair. Rats) | 13:22 |
| thorst | anywho, that's all I had | 13:22 |
| esberglu | Okay lets move on | 13:22 |
| esberglu | #topic CI | 13:22 |
| esberglu | So we already talked about pushing the backlog through | 13:22 |
| esberglu | And then at that point I think the plans is to wipe the mgmt node and redeploy from scratch | 13:23 |
| esberglu | Cuz this whole saving a sinking ship with a bucket thing isn't working | 13:23 |
| esberglu | Other than that we have efrieds local2remote changes | 13:25 |
| esberglu | I know you want to have that installed as part of the base image template | 13:25 |
| efried | Well, that's open for discussion, but I think that's the right place to do it. | 13:25 |
| esberglu | But we would still have to reinstall during the prep_devstack part | 13:25 |
| thorst | and we have a long term plan to go 'patchless'? | 13:25 |
| efried | We shouldn't have to reinstall it at all then. | 13:25 |
| esberglu | We don't know which version of pypowervm we want until then | 13:26 |
| efried | You mean because newton and ocata are still on the table at that point? | 13:26 |
| efried | Doesn't matter. | 13:26 |
| efried | If we diddle /etc/environment in the base image, and have the .py present, pre-1.1.2 will simply ignore, and we'll still have to do the old style local2remote patching if newton/ocata. | 13:27 |
| efried | But 1.1.2 will just work. | 13:27 |
| efried | To summarize: the new stuff I'm proposing can be part of the base image template; and the old stuff will need to change to be conditioned on pre-pike-only once 1.1.2 is released. | 13:28 |
| esberglu | Well we can't pull the version from requirements then | 13:28 |
| efried | When requirements says 1.1.2, we can. | 13:28 |
| efried | Cause we should be pulling from the requirements of the branch we're testing. | 13:29 |
| esberglu | What if requirements changes between the nightly build and the CI run? | 13:29 |
| efried | Yeah, we still pull from requirements of whatever branch we're testing. | 13:29 |
| esberglu | We don't know which branch we are testing until way after the image build | 13:29 |
| esberglu | Maybe I'm missing something here | 13:30 |
| efried | Doesn't matter if we flipflop back and forth between 1.1.2 and pre-1.1.2. | 13:30 |
| esberglu | Yeah but we still have to reinstall at that point is what I'm saying | 13:30 |
| efried | Only if we're testing pre-pike. | 13:30 |
| efried | Whole point of this is we don't have to patch pypowervm at 1.1.2. | 13:30 |
| efried | ...which should be concomitant with pike. And pre-1.1.2 with pre-pike. | 13:31 |
| esberglu | My point about the reqs changing between the image build and the run still stands | 13:31 |
| esberglu | Edge case, but still | 13:32 |
| efried | reqs of pypowervm specifically? | 13:32 |
| efried | I'm still not seeing the problem. | 13:32 |
| esberglu | Yeah, what if we bump the reqs up to 1.1.3? We would then have to wipe all of the nodes out | 13:32 |
| esberglu | And build from scratch | 13:32 |
| efried | no, we shouldn't. | 13:32 |
| efried | Cause 1.1.3 will then get pulled in when we stack. | 13:33 |
| efried | And 1.1.3 will still just work when the env var & .py are present. | 13:33 |
| esberglu | Ohhh duhhhh | 13:33 |
| *** smatzek has quit IRC | 13:33 | |
| efried | Whole reason we're doing things this way. | 13:33 |
| esberglu | Yeah my b | 13:33 |
| efried | Cool | 13:33 |
| *** tjakobs__ has joined #openstack-powervm | 13:34 | |
| esberglu | I still owe you a patch / review on that stuff | 13:34 |
| efried | Yeah, but this is background work, as there are lulls in the flushing-CI-backlog business. | 13:35 |
| efried | As excited as I am about trying it out and getting it working... | 13:35 |
| esberglu | Yep. I would like to get it in for when we do the full CI redeploy | 13:35 |
| esberglu | So we don't have to redeploy again right after | 13:36 |
| efried | For the sake of having it documented, we will ultimately need: o) patches to deploy the .py and /etc/environment change; o) image template rebuild; o) 1.1.2 released; o) g-r and nova project requirements bumps | 13:36 |
| efried | I think in that order. | 13:37 |
| efried | the image template rebuild & CI redeploy could happen at any point after the neo-os-ci change is merged. | 13:38 |
| esberglu | Yeah I think so | 13:39 |
| efried | oh, and o) separate neo-os-ci (or powervm-ci?) change to condition manual local2remote patch on pre-pike. I think that needs to come in sync with the reqs bumps. | 13:39 |
| efried | which will be fiddly. | 13:39 |
| efried | Cause the manual local2remote patch will simply fail on 1.1.2. | 13:40 |
| efried | merge conflict there. | 13:40 |
| efried | I guess we could change that right now to condition on the pypowervm version. | 13:40 |
| efried | rather than the openstack branch. | 13:40 |
| efried | Then no fiddly. | 13:40 |
| esberglu | Sure I like that | 13:41 |
| efried | Will probably have to use pbr to do version compare, groan. | 13:41 |
| efried | Okay, let's move on. | 13:41 |
| esberglu | I didn't have anything else base CI | 13:41 |
| esberglu | thorst: Did you want to talk OSA CI here? Or separate discussion on that? | 13:42 |
| thorst | lets have a separate discussion on that when we stabilize | 13:42 |
| esberglu | Okay | 13:43 |
| esberglu | That's it from my end then | 13:43 |
| *** tjakobs__ has quit IRC | 13:43 | |
| esberglu | #topic Other Discussion | 13:44 |
| esberglu | nbante: You have anything you are working through? | 13:44 |
| efried | #action thorst As able, take a look at https://review.openstack.org/#/c/438729/12..13 (tasks/* bits - the others are rebase) and let's move toward getting this back to nova-powervm for https://bugs.launchpad.net/nova-powervm/+bug/1680947 | 13:44 |
| openstack | Launchpad bug 1680947 in nova-powervm "Use PrintingDurationListener, get rid of PowerVMTask" [Wishlist,New] | 13:44 |
| efried | Low priority. | 13:44 |
| nbante | I tried basic tests with tempest and those are passed successfully. Now working to configure external storage then we can try deploy test cases | 13:45 |
| thorst | it looks like I need to figure out how to get our v7k on a private IP address for nbante | 13:46 |
| thorst | it'll take some wiring :-/ | 13:46 |
| nbante | meanwhile I'll check,what are configuration files need to modify and once all changes done. I'll redploy to get cinder configured | 13:47 |
| esberglu | nbante: Cool sounds good | 13:48 |
| esberglu | Doesn't look like jay is on | 13:48 |
| esberglu | Any final topics before I close? | 13:48 |
| esberglu | #endmeeting | 13:50 |
| thorst | efried: do you know if we've fixed the arping thing? | 13:50 |
| openstack | Meeting ended Tue Apr 11 13:50:50 2017 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 13:50 |
| openstack | Minutes: http://eavesdrop.openstack.org/meetings/powervm_driver_meeting/2017/powervm_driver_meeting.2017-04-11-13.01.html | 13:50 |
| openstack | Minutes (text): http://eavesdrop.openstack.org/meetings/powervm_driver_meeting/2017/powervm_driver_meeting.2017-04-11-13.01.txt | 13:50 |
| openstack | Log: http://eavesdrop.openstack.org/meetings/powervm_driver_meeting/2017/powervm_driver_meeting.2017-04-11-13.01.log.html | 13:50 |
| efried | thorst Sylvain has proposed the fix. Hopefully it goes through today. | 13:51 |
| thorst | lets raise hell to make sure it does | 13:51 |
| efried | thorst 5117 | 13:51 |
| thorst | rabble rabble rabble rabble | 13:51 |
| efried | Looks like it got +2 from nvcastet. | 13:51 |
| efried | Let's merge that sucker. | 13:51 |
| efried | Seems to be having trouble in jenkins. | 13:52 |
| efried | So we may have to rabble rabble jfoliva | 13:52 |
| efried | I kicked off a recheck ~45m ago. | 13:52 |
| thorst | k | 13:52 |
| *** jpasqualetto has quit IRC | 13:57 | |
| *** smatzek has joined #openstack-powervm | 14:02 | |
| *** jpasqualetto has joined #openstack-powervm | 14:09 | |
| *** k0da has joined #openstack-powervm | 14:16 | |
| -openstackstatus- NOTICE: latest base images have mistakenly put python3 in some places expecting python2 causing widespread failure of docs patches - fixes are underway | 14:27 | |
| *** mdrabe has joined #openstack-powervm | 14:44 | |
| -openstackstatus- NOTICE: we have rolled back centos-7, fedora-25 and ubuntu-xenial images to the previous days release. Feel free to recheck your jobs now. | 14:47 | |
| esberglu | efried: One thing we will be giving up with the local2remote change is the ability to add additional pypowervm patches without rebuilding the image template | 14:48 |
| esberglu | Not sure if you were aware of that | 14:48 |
| esberglu | Actually I have an idea to get around that | 14:53 |
| esberglu | Stay tuned | 14:54 |
| *** nbante has quit IRC | 14:56 | |
| *** tjakobs__ has joined #openstack-powervm | 15:03 | |
| *** openstackgerrit has joined #openstack-powervm | 15:16 | |
| openstackgerrit | Eric Berglund proposed openstack/networking-powervm master: Switch to manual service enablement for devstack plugins https://review.openstack.org/416667 | 15:16 |
| *** tjakobs__ has quit IRC | 15:21 | |
| *** tjakobs has joined #openstack-powervm | 15:21 | |
| efried | esberglu Not sure how we lose that ability. We just need to keep the pre-pike local2remote patch separate. | 15:32 |
| esberglu | Because we aren't reinstalling pypowervm for master. So if we want to test a pypowervm patch quick we can't just add it to the jenkins job parameter and reload the jenkins job | 15:35 |
| esberglu | I have a solution I think hold tight | 15:35 |
| efried | ight | 15:39 |
| esberglu | I'll walk you through my solution quick | 15:42 |
| esberglu | Leave the -p flag in the prep_devstack script (pypowervm_patch_list) so we can still pass in additional patches from the jenkins job | 15:44 |
| esberglu | In prep_devstack | 15:44 |
| esberglu | If on newton or ocata | 15:44 |
| esberglu | pypowervm_patch_list+=2382 | 15:44 |
| esberglu | fi | 15:44 |
| esberglu | if pypowervm_patch_list isn't empty | 15:44 |
| esberglu | checkout the correct pypowervm from requirements | 15:44 |
| esberglu | apply all patches from the patch list | 15:44 |
| esberglu | install pypowervm | 15:45 |
| esberglu | wipe pypowervm from requirements so stack won't overwrite | 15:45 |
| esberglu | fi | 15:45 |
| esberglu | if the pypowervm_patch_list is empty we just proceed as normal and don't worry about anything | 15:45 |
| efried | esberglu That sounds okay. | 15:48 |
| efried | Long as "from requirements" is branch-gnostic. | 15:48 |
| esberglu | Yeah we checkout the correct branch for all projects before any of this happens | 15:49 |
| esberglu | We just need to make sure that the env. var. for pypowervm either survives to this point or set it again | 15:50 |
| efried | esberglu If it's in /etc/environment in the image template, it should survive just fine. | 15:55 |
| efried | Just need to make sure the .py is present if the env var is set - else kablooey. | 15:56 |
| efried | Which is as it should be - I don't think we want to ignore a failed import there. | 15:56 |
| efried | thorst Has this ceilometer-powervm stuff been tested live? | 15:57 |
| esberglu | thorst: Most of the reviews from that first list you posted have passed CI now excluding | 16:01 |
| esberglu | https://review.openstack.org/#/c/416667/ -- This needs a local.conf change in neo-os-ci | 16:01 |
| esberglu | https://review.openstack.org/#/c/452712/ -- Has been running for 3 hours, will check out logs when complete | 16:02 |
| openstackgerrit | Eric Fried proposed openstack/ceilometer-powervm master: Manual sync with upstream requirements https://review.openstack.org/452222 | 16:18 |
| openstackgerrit | Merged openstack/ceilometer-powervm master: use new instance polling interface https://review.openstack.org/448134 | 16:22 |
| thorst | efried: gautam has been testing I believe | 16:31 |
| openstackgerrit | Merged openstack/nova-powervm master: Remove INSTALL_PYPOWERVM https://review.openstack.org/454345 | 17:05 |
| openstackgerrit | Drew Thorstensen (thorst) proposed openstack/ceilometer-powervm master: Remove INSTALL_PYPOWERVM https://review.openstack.org/454344 | 17:18 |
| openstackgerrit | Merged openstack/networking-powervm master: Manual sync with upstream requirements https://review.openstack.org/452220 | 17:29 |
| openstackgerrit | Merged openstack/ceilometer-powervm master: Manual sync with upstream requirements https://review.openstack.org/452222 | 17:29 |
| openstackgerrit | Merged openstack/ceilometer-powervm master: Remove INSTALL_PYPOWERVM https://review.openstack.org/454344 | 17:31 |
| efried | Howzat backlog looking? | 17:34 |
| thorst | I was just looking | 17:36 |
| thorst | its much better | 17:36 |
| thorst | but I'm debugging one of adreznec's changes | 17:36 |
| thorst | it looks like it failed to start networking-powervm | 17:36 |
| thorst | o duh, because his change turns that off | 17:38 |
| thorst | adreznec: https://review.openstack.org/#/c/416667 | 17:38 |
| thorst | this won't pass CI unless we have a CI change? | 17:38 |
| thorst | esberglu: ^^ | 17:38 |
| esberglu | We would need to add the | 17:40 |
| esberglu | enable_service pvm-q-sea-agt pvm-q-sriov-agt | 17:40 |
| esberglu | bit to the local.conf | 17:40 |
| thorst | yeah, I'm not sure I even buy the change though | 17:40 |
| thorst | if you're running networking-powervm...you'd want those services enabled I would think | 17:40 |
| esberglu | Perhaps the thinking was that you would only want 1 or the other enabled instead of both? | 17:41 |
| thorst | meh | 17:43 |
| thorst | maybe...I know we had a reason for it...but it's escaping my mind right now | 17:43 |
| thorst | so I -1'd it | 17:43 |
| esberglu | K. Just let me know if you change your mind and I will update CI accordingly | 17:43 |
| *** jay1_ has joined #openstack-powervm | 17:44 | |
| openstackgerrit | Drew Thorstensen (thorst) proposed openstack/ceilometer-powervm master: Optimize the link address for fetching git code https://review.openstack.org/447762 | 17:45 |
| thorst | efried: you finally OK with that one above??? | 17:46 |
| thorst | :-) | 17:46 |
| openstackgerrit | Drew Thorstensen (thorst) proposed openstack/networking-powervm master: Optimize the link address for fetching git code https://review.openstack.org/447763 | 17:48 |
| openstackgerrit | Drew Thorstensen (thorst) proposed openstack/ceilometer-powervm master: Remove support for py34 for ceilometer-powervm https://review.openstack.org/430619 | 17:50 |
| openstackgerrit | Drew Thorstensen (thorst) proposed openstack/ceilometer-powervm master: Remove support for py34 for ceilometer-powervm https://review.openstack.org/430619 | 17:51 |
| *** nbante has joined #openstack-powervm | 17:59 | |
| *** nbante has quit IRC | 18:19 | |
| *** shyama has quit IRC | 18:32 | |
| *** jay1_ has quit IRC | 18:54 | |
| efried | thorst I'm confused about those "optimize link address" ones. | 18:55 |
| efried | Basically, why tf do we need to do that? | 18:56 |
| thorst | https is better than http | 18:56 |
| thorst | but to me...its a little thing | 18:56 |
| thorst | I'm indifferent. | 18:57 |
| efried | meh, whatever. | 18:57 |
| efried | esberglu thorst As for enabling the agents, we need the SEA agent for networking to work in the CI env. Having the SR-IOV one enabled should be unnecessary, but harmless | 18:58 |
| efried | At least, I'm assuming we have SEA set up on our VIOSes in the CI env. (I know we don't have SR-IOV.0 | 18:59 |
| thorst | yeah...but I'm wondering why we need to change it | 19:00 |
| esberglu | I think thorst was just confused about why this change needs to go in at all | 19:00 |
| esberglu | yeah | 19:00 |
| efried | why, what is it now? | 19:01 |
| thorst | if you install networking-powervm, and set the virt driver to powervm, you get the SEA | 19:01 |
| thorst | if you set the virt driver to powervm, and don't install networking-powervm...you can use OVS or something | 19:01 |
| thorst | maybe it was to bypass the case where you had installed networking-powervm, but wanted to flip a future devstack to OVS... | 19:01 |
| openstackgerrit | Merged openstack/ceilometer-powervm master: Optimize the link address for fetching git code https://review.openstack.org/447762 | 19:02 |
| efried | If you install networking-powervm and set the virt driver to powervm, you get the SEA? | 19:02 |
| efried | How? | 19:02 |
| thorst | that code looks at the nova compute virt driver attribute, and if set to powervm, it turns on SEA | 19:02 |
| openstackgerrit | Merged openstack/networking-powervm master: Optimize the link address for fetching git code https://review.openstack.org/447763 | 19:04 |
| efried | thorst "that code" what code? | 19:04 |
| thorst | ugh...of course you ask for details | 19:05 |
| efried | of course. | 19:05 |
| thorst | https://review.openstack.org/#/c/416667/2/devstack/settings | 19:05 |
| thorst | jerk wad | 19:05 |
| efried | Okay, right, so with that part deleted, we need to enable those services explicitly via local.conf | 19:06 |
| efried | right? | 19:06 |
| efried | Am I missing something fundamental? | 19:06 |
| thorst | correct | 19:06 |
| thorst | and I'm just asserting..do we really need to make them jump through that hoop? | 19:07 |
| efried | I would think so. They should need to decide whether to enable one, both, or neither. | 19:07 |
| efried | And ultimately if we have others, those too. | 19:07 |
| efried | If for no other reason than that they shouldn't have services running that they don't need. | 19:08 |
| thorst | ok...well, we can't merge until CI has that setting done | 19:08 |
| efried | I'm okay with that. | 19:08 |
| esberglu | I will put a neo-os-ci patch quick | 19:09 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/nova-powervm master: Updated from global requirements https://review.openstack.org/454506 | 19:14 |
| *** jpasqualetto has quit IRC | 19:16 | |
| *** jpasqualetto has joined #openstack-powervm | 19:29 | |
| *** jpasqualetto has quit IRC | 19:41 | |
| *** smatzek has quit IRC | 19:53 | |
| *** jpasqualetto has joined #openstack-powervm | 19:54 | |
| openstackgerrit | Merged openstack/nova-powervm master: Updated from global requirements https://review.openstack.org/454506 | 20:37 |
| *** thorst has quit IRC | 21:07 | |
| *** apearson has quit IRC | 21:11 | |
| *** esberglu has quit IRC | 21:19 | |
| *** esberglu has joined #openstack-powervm | 21:20 | |
| *** esberglu has quit IRC | 21:24 | |
| *** thorst has joined #openstack-powervm | 21:30 | |
| *** edmondsw has quit IRC | 21:34 | |
| *** thorst has quit IRC | 21:34 | |
| *** edmondsw has joined #openstack-powervm | 21:35 | |
| *** esberglu has joined #openstack-powervm | 21:35 | |
| *** edmondsw has quit IRC | 21:39 | |
| *** esberglu has quit IRC | 21:53 | |
| *** dwayne has quit IRC | 22:00 | |
| *** edmondsw has joined #openstack-powervm | 22:04 | |
| *** mdrabe has quit IRC | 22:04 | |
| *** edmondsw has quit IRC | 22:08 | |
| *** jpasqualetto has quit IRC | 22:27 | |
| *** tjakobs has quit IRC | 22:43 | |
| *** jpasqualetto has joined #openstack-powervm | 22:44 | |
| *** k0da has quit IRC | 23:07 | |
| *** thorst has joined #openstack-powervm | 23:20 | |
| *** thorst has quit IRC | 23:23 | |
| *** thorst has joined #openstack-powervm | 23:53 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!