| *** catmando has joined #openstack-powervm | 14:53 | |
| *** catmando_ has joined #openstack-powervm | 14:56 | |
| *** efried has joined #openstack-powervm | 15:02 | |
| catmando | hey all] | 15:07 |
|---|---|---|
| catmando | been having a look at powervc 1.4.2 | 15:08 |
| catmando | i'm trying to find the "Fibre Channel tape pluggable driver" option in the interface somewhere: any ideas? | 15:08 |
| efried | catmando: You may want to wait for mdrabe or edmondsw to show up. | 15:08 |
| *** edmondsw has joined #openstack-powervm | 15:09 | |
| catmando | i shall | 15:09 |
| catmando | no rush, thank you | 15:09 |
| edmondsw | catmando I'm waiting for eavesdrop to catch up, but I heard you might have a question for me? | 15:12 |
| catmando | powervc 1.4.2: i'm trying to find the "Fibre Channel tape pluggable driver" option in the interface somewhere: any ideas? | 15:13 |
| catmando | edmondsw: ^ | 15:17 |
| edmondsw | well that's weird... it should have that info in the Knowledge Center but I'm not finding it | 15:18 |
| edmondsw | give me a minute | 15:18 |
| edmondsw | I assume you're asking where the CLI command is to configure tapes? | 15:20 |
| edmondsw | I believe it's in /opt/ibm/powervc/bin | 15:20 |
| edmondsw | something like powervc-tape-register | 15:20 |
| edmondsw | catmando ^ | 15:20 |
| catmando | edmondsw: aha, let me have a look at that | 15:21 |
| catmando | i was hoping that there would be something like the port load balancing | 15:22 |
| edmondsw | that registers an FC tape driver, so I think the port configuration stuff would work the same as other FC drivers | 15:25 |
| edmondsw | but I'm not sure about that | 15:27 |
| edmondsw | powervc-tape-register -n DISPLAY_NAME -p TARGET_PORTS [-c CAPACITY] | 15:27 |
| edmondsw | or something like that | 15:27 |
| edmondsw | catmando I believe chhagarw designed/wrote that, so I've pinged her to get on IRC if/when she's available. | 15:35 |
| edmondsw | and pinged our docs folks about why it's not in the KC | 15:35 |
| *** chhagarw has joined #openstack-powervm | 15:40 | |
| chhagarw | catmando: Seems, you have a question for FC Tape pluggable driver ? | 15:45 |
| chhagarw | Hope you are able to find powervc-tape-register, this CLI can be used to register the FC Storage target ports to PowerVC. | 15:46 |
| catmando | chhagarw: yes, i am looking at that now | 16:32 |
| catmando | but there's no docs and I thought there might be a frontend (similar to the one for port groups) | 16:35 |
| catmando | chhagarw: the cli docs state: "specifies the comma separated target ports of the FC tape storage device."; does this mean the "target" as in the port on the vios? | 16:42 |
| chhagarw | catmando: we do not have it documented in KC currently, we will check on that. | 16:42 |
| catmando | i assume it does not mean the actual target port on the remote side | 16:42 |
| chhagarw | CLI, requires target ports list, which are the FC Tape device target ports, | 16:43 |
| chhagarw | once its registered using CLI, it will get listed in the PowerVC GUI as the storage device. | 16:44 |
| catmando | so, the wwpn of the remote side, as opposed to the wwpn on the vios... | 16:44 |
| chhagarw | You can consider, FC Tape as a pluggable storage device, so while registering user needs to provide the remote storage target ports, not the vios wwpns. | 16:47 |
| chhagarw | once registered these target ports wwpns will be used to zone the NPIV VM wwpns. | 16:47 |
| catmando | that makes sense | 16:48 |
| catmando | chhagarw: i'll have a play around with it in our lab and get back to you when i see you online if anything seems odd. many thanks for the help and for getting this into powervc | 16:50 |
| chhagarw | chhagarw: sure, will look forward for your response. | 16:51 |
| *** catmando_ has quit IRC | 17:04 | |
| *** chhagarw has quit IRC | 17:31 | |
| openstackgerrit | Matthew Edmonds proposed openstack/networking-powervm master: Update tests for policy-in-code https://review.openstack.org/628012 | 17:39 |
| edmondsw | efried ^ that should fix UTs for networking-powervm, which are currently failing | 17:39 |
| openstackgerrit | Matthew Edmonds proposed openstack/networking-powervm master: Update tests for policy-in-code https://review.openstack.org/628012 | 17:40 |
| edmondsw | had to fix a mistake in the commit message... | 17:40 |
| efried | edmondsw: Seem to remember we're not supposed to import things from neutron. Doesn't neutronlib expose a base test case? | 17:50 |
| edmondsw | I don't know... I can look | 17:50 |
| edmondsw | that wouldn't be new, but we could certainly clean it up while we're in here if you're right | 17:51 |
| edmondsw | neutron has a _base.py, which means it is not exposed | 17:52 |
| edmondsw | I mean neutron-lib | 17:52 |
| edmondsw | the most correct thing to do may be to create our own BaseTestCase class in networking-powervm, but I don't know that I want to bite that off right now | 17:57 |
| edmondsw | that's what networking-hyperv does | 17:57 |
| efried | there's a hacking rule that we have on our ignore list. | 17:58 |
| efried | # N530 direct neutron imports not allowed | 17:58 |
| edmondsw | yeah, I'm not disagreeing with you... just saying a) I'm not introducing that and b) let's worry about that later in another commit | 17:59 |
| edmondsw | all I'm doing here is removing the middleman | 17:59 |
| edmondsw | BasePVMTestClass | 17:59 |
| efried | I understand. Cleanest thing would probably be to switch to | 18:01 |
| efried | class BasePVMTestCase(neutron.tests.base.BaseWhatever): | 18:01 |
| efried | pass | 18:01 |
| efried | so we don't have to switch those other things again when we figure out what we're supposed to be doing. | 18:01 |
| edmondsw | ok | 18:01 |
| edmondsw | that's easy enough | 18:01 |
| efried | fyi, asked in -neutron and the answer is that a neutron-lib base test case is on the to-do list; for now import from neutron. But yeah, I think doing the empty local class for now - possibly with a TODO about ^ - is the right thing. | 18:03 |
| openstackgerrit | Andreas Jaeger proposed openstack/networking-powervm master: Use template for lower-constraints https://review.openstack.org/626709 | 18:08 |
| openstackgerrit | Matthew Edmonds proposed openstack/networking-powervm master: Update tests for policy-in-code https://review.openstack.org/628012 | 18:38 |
| edmondsw | efried ^ | 18:39 |
| efried | ... | 18:40 |
| openstackgerrit | Eric Fried proposed openstack/networking-powervm master: Use template for lower-constraints https://review.openstack.org/626709 | 18:41 |
| edmondsw | efried while you're looking at that lc one, there's an identical change in nova-powervm that needs your +2 | 18:48 |
| edmondsw | https://review.openstack.org/#/c/626714/ | 18:48 |
| efried | done | 18:49 |
| edmondsw | efried and when you get a chance, these: https://review.openstack.org/#/q/status:open+branch:master+topic:py36+owner:edmondsw%2540us.ibm.com | 18:51 |
| openstackgerrit | Merged openstack/networking-powervm master: Update tests for policy-in-code https://review.openstack.org/628012 | 19:02 |
| openstackgerrit | Merged openstack/nova-powervm master: Use template for lower-constraints https://review.openstack.org/626714 | 19:03 |
| openstackgerrit | Merged openstack/networking-powervm master: Use template for lower-constraints https://review.openstack.org/626709 | 19:06 |
| efried | edmondsw: So before https://review.openstack.org/#/c/623739/ the py36 job would fail? | 19:25 |
| edmondsw | it would fail locally, but it was somehow working for zuul.... I'm not sure how | 19:26 |
| efried | ugh. I'm looking for some justification for these changes to be combined into one. | 19:32 |
| edmondsw | that's the justification... neither makes sense without the other | 19:33 |
| efried | ight | 19:33 |
| edmondsw | if I just changed the UT, what's the point if the reason for the change was for py36 local UTs and we don't have that enabled in our tox.ini | 19:33 |
| edmondsw | and if I just do the tox.ini change, what's the point if I still can't run the UTs locally because I'm missing the UT change | 19:34 |
| edmondsw | but I am concerned, now that you mention it, that the py36 zuul job hasn't been failing all this time... | 19:34 |
| efried | I'm +2 on those three changes | 19:34 |
| efried | IIUC, the failure would be intermittent/unpredictable based on random hash ordering or something similar. | 19:34 |
| efried | Said hashing/ordering is affected by things, including env vars like PYTHONHASHSEED. | 19:35 |
| efried | Which for all we know might be (un)set on the zuul server and vice versa when we run locally. | 19:35 |
| efried | in any case, changing it to sorted() so it's predictable is goodness. | 19:35 |
| edmondsw | that helps... tx | 19:36 |
| edmondsw | infra may have set PYTHONHASHSEED earlier for some other reason and it has masked this for us | 19:36 |
| openstackgerrit | Merged openstack/networking-powervm master: More py3.x support https://review.openstack.org/623739 | 20:06 |
| openstackgerrit | Merged openstack/nova-powervm master: More py3.x support https://review.openstack.org/623084 | 20:31 |
| openstackgerrit | Merged openstack/ceilometer-powervm master: More py3.x support https://review.openstack.org/624241 | 20:53 |
| *** efried has quit IRC | 21:21 | |
| *** efried has joined #openstack-powervm | 21:29 | |
| *** efried has quit IRC | 22:39 | |
| *** catmando has quit IRC | 23:08 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!