Monday, 2017-10-23

*** edmondsw has joined #openstack-powervm00:02
*** thorst has joined #openstack-powervm00:03
*** edmondsw has quit IRC00:06
*** thorst has quit IRC00:07
*** thorst has joined #openstack-powervm00:07
*** thorst has quit IRC00:08
*** edmondsw has joined #openstack-powervm00:33
*** edmondsw has quit IRC00:56
*** thorst has joined #openstack-powervm01:11
*** thorst has quit IRC01:11
*** fried_rice is now known as efried01:32
*** edmondsw has joined #openstack-powervm01:55
*** edmondsw has quit IRC01:59
*** thorst has joined #openstack-powervm03:02
*** thorst has quit IRC03:08
*** edmondsw has joined #openstack-powervm03:43
*** edmondsw has quit IRC03:47
*** edmondsw has joined #openstack-powervm05:31
*** edmondsw has quit IRC05:36
*** edmondsw has joined #openstack-powervm07:19
*** edmondsw has quit IRC07:24
*** AlexeyAbashkin has joined #openstack-powervm07:37
*** AlexeyAbashkin has quit IRC08:12
*** AlexeyAbashkin has joined #openstack-powervm08:12
*** edmondsw has joined #openstack-powervm09:08
*** edmondsw has quit IRC09:12
*** thorst has joined #openstack-powervm11:59
*** thorst_ has joined #openstack-powervm12:00
*** thorst_ has quit IRC12:01
*** thorst has quit IRC12:04
*** thorst has joined #openstack-powervm12:04
*** edmondsw has joined #openstack-powervm12:21
*** edmondsw has quit IRC12:21
*** edmondsw has joined #openstack-powervm12:22
*** kylek3h has joined #openstack-powervm13:35
*** esberglu has joined #openstack-powervm13:58
*** k0da has joined #openstack-powervm14:49
esbergluefried: https://review.openstack.org/#/c/409404/35/nova/virt/powervm/media.py14:54
esbergluCan you weigh in on edmondsw's comment about get_vm_id vs. get_pvm_uuid here? (L183)14:54
esbergluhttps://github.com/powervm/pypowervm/blob/develop/pypowervm/tasks/scsi_mapper.py#L45014:55
efriedesberglu Will do.14:56
esberglu^ sounds like it accepts either from that parameter in pypowervm14:56
edmondswefried it's probably fine based on the comment in pypowervm, but appreciate you double checking14:58
*** tjakobs has joined #openstack-powervm15:14
*** k0da has quit IRC15:16
openstackgerritEric Berglund proposed openstack/nova-powervm master: DNM: ci check  https://review.openstack.org/32831515:16
efriedesberglu edmondsw commented; it's fine.15:19
edmondswcool, t15:19
edmondswtx15:19
efried        is_uuid, client_id = uuid.id_or_uuid(client_lpar_id)15:19
esberglutx15:19
openstackgerritTyler Blakeslee proposed openstack/nova-powervm master: Raise catachable error when VNC terminal fails to open.  https://review.openstack.org/51383315:20
*** AlexeyAbashkin has quit IRC15:37
openstackgerritTyler Blakeslee proposed openstack/nova-powervm master: Raise catachable error when VNC terminal fails to open.  https://review.openstack.org/51383316:17
*** AlexeyAbashkin has joined #openstack-powervm17:13
*** AlexeyAbashkin has quit IRC17:17
openstackgerritMatthew Edmonds proposed openstack/nova-powervm master: update translations  https://review.openstack.org/51439917:37
openstackgerritMatthew Edmonds proposed openstack/networking-powervm master: update translations  https://review.openstack.org/51440317:45
openstackgerritMatthew Edmonds proposed openstack/ceilometer-powervm master: update translations  https://review.openstack.org/51440517:52
*** kylek3h has quit IRC17:56
*** kylek3h has joined #openstack-powervm17:56
efriededmondsw Translations suck.  I am certain I have spent more time looking at these messages than all customers from those locales combined.18:05
edmondswefried yep18:37
edmondswonce we merge in master, we'll have to backport to pike18:37
edmondswbugs me that we still have all those files for logs18:38
efriededmondsw We shouldn't.  We should delete them.  Is there some reason we're keeping them?19:05
edmondswefried I'll throw up a separate patch to remove them. I'm just not sure how to make sure it doesn't break something19:05
efriededmondsw What would it break?19:06
edmondswefried that's the question19:06
efriedIf a translation can't be found, it uses the English, end of story.19:06
efriededmondsw Don't we have some kind of TVT anyway?19:06
efriededmondsw btw, you saw my patch to rip out those orphaned info log translations, yah?19:07
edmondswefried already +2'd19:07
efriedCool, I should look before I ask :)19:07
edmondswnp :)19:07
edmondswtx for that19:08
edmondswefried and your comments... I'm leery of some of them, but I'll give it a shot19:08
efriededmondsw Happy to explain if you like.19:08
edmondswhave been burned in the past when I tried to edit translated files and file encodings got messed up19:08
efriedmm, I can see that.  Maybe leave the Japanese one alone, then.  The others ought to be fine.19:09
edmondswI don't question your comments, just whether I can safely edit the files19:09
efriededmondsw Dig.19:09
edmondswefried why would japanese be the only one that's special?19:09
efriededmondsw TBH, it's pretty unimportant to me whether those changes get made.  Some of them (like missing spaces from one line to the next) are clearly wrong.  But like I said, nobody looks at these messages anyway.19:09
edmondswthat was my thought :)19:10
efriededmondsw Because that one would be using unicode.  The Western ones are going to be using ascii or something similar enough as makes no difference.19:10
edmondswwell, there's simplified chinese, traditional chinese, korean...19:11
edmondswI'd rather just leave them if ok with you19:11
edmondswefried ^19:12
edmondswnot our job to correct the translator's mistakes19:12
efriededmondsw But I didn't suggest any edits to those19:12
efriedoh, I guess some spacing.19:13
efriedand that one %s<korean stuff> that seemed wrong.19:13
efriededmondsw sfine, at least I'm on record noting the mistakes.19:15
edmondsw:) tx19:15
efriedThough I really do feel we should be deleting those log-related ones.19:15
openstackgerritMatthew Edmonds proposed openstack/nova-powervm master: remove log translation files  https://review.openstack.org/51443719:21
openstackgerritMatthew Edmonds proposed openstack/networking-powervm master: remove log translation files  https://review.openstack.org/51443919:31
openstackgerritMatthew Edmonds proposed openstack/ceilometer-powervm master: remove log translations  https://review.openstack.org/51444119:38
openstackgerritMatthew Edmonds proposed openstack/ceilometer-powervm master: remove log translations  https://review.openstack.org/51444119:40
edmondswefried ^19:41
efriededmondsw ack19:42
*** AlexeyAbashkin has joined #openstack-powervm19:58
*** AlexeyAbashkin has quit IRC20:02
*** thorst has quit IRC21:02
*** edmondsw has quit IRC21:20
*** thorst has joined #openstack-powervm21:27
openstackgerritTyler Blakeslee proposed openstack/nova-powervm master: Raise catachable error when VNC terminal fails to open.  https://review.openstack.org/51383321:27
*** thorst has quit IRC21:31
openstackgerritTyler Blakeslee proposed openstack/nova-powervm master: Raise catachable error when VNC terminal fails to open.  https://review.openstack.org/51383321:40
*** esberglu has quit IRC21:49
*** esberglu has joined #openstack-powervm21:49
*** esberglu_ has joined #openstack-powervm21:51
*** esberglu has quit IRC21:54
*** esberglu_ has quit IRC21:56
*** esberglu has joined #openstack-powervm22:01
openstackgerritTyler Blakeslee proposed openstack/nova-powervm master: Raise catachable error when VNC terminal fails to open.  https://review.openstack.org/51383322:06
*** esberglu has quit IRC22:25
*** thorst has joined #openstack-powervm22:46
*** edmondsw has joined #openstack-powervm22:48
*** thorst has quit IRC22:51
*** edmondsw has quit IRC22:52
*** tjakobs has quit IRC23:03

Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!