Wednesday, 2018-01-31

*** toan has quit IRC03:24
*** toan has joined #openstack-powervm03:26
*** zerick_ has quit IRC05:13
*** zerick has joined #openstack-powervm05:15
*** AlexeyAbashkin has joined #openstack-powervm07:51
*** AlexeyAbashkin has quit IRC07:51
*** AlexeyAbashkin has joined #openstack-powervm07:52
*** edmondsw has joined #openstack-powervm08:50
*** edmondsw has quit IRC08:54
*** AlexeyAbashkin has quit IRC11:09
*** AlexeyAbashkin has joined #openstack-powervm11:15
*** edmondsw has joined #openstack-powervm13:22
*** apearson has joined #openstack-powervm13:28
*** svenkat has joined #openstack-powervm14:17
*** svenkat_ has joined #openstack-powervm14:22
*** svenkat has quit IRC14:23
*** svenkat_ is now known as svenkat14:23
*** esberglu has joined #openstack-powervm14:30
openstackgerritEric Berglund proposed openstack/nova-powervm master: Add missing capabilities
*** esberglu_ has joined #openstack-powervm15:36
*** esberglu has quit IRC15:40
*** esberglu_ is now known as esberglu15:48
openstackgerritEric Berglund proposed openstack/nova-powervm master: Update the SEA local.conf
*** efried_hexchat has quit IRC16:08
esbergluedmondsw: Regarding the etcd version for devstack. The devstack default is currently 3.1.10. The closest version to that for power is 3.2.316:25
*** cjvolzka has joined #openstack-powervm16:25
esbergluThe latest release is 3.2.1516:25
esbergluI think we should go latest, but thought I would get your opinion16:25
*** cjvolzka has quit IRC16:25
efriedwasn't there some reason we needed to go older?16:27
edmondswesberglu we needed at least 3.2.016:27
edmondswI mean efried16:27
esbergluefried: I don't think we needed to go older. The problem was that the version devstack defaults to doesn't support power16:27
edmondswesberglu if 3.2.0 has released, can we just propose devstack move up to that as a min?16:28
efriedIf we have passing test runs at the latest, go with the latest.16:28
efriededmondsw: Not now.16:28
efriededmondsw: Not until Rocky at this point.16:28
edmondswefried right, not for queens... but very soon16:28
efriedIs there something broken about whatever we're using right now, or is this just because you happened to be touching it in your local.conf change set?16:29
edmondswefried it was just that I noticed there was something newer when looking at the TODO comment... not really related to this change16:29
edmondswthough it's odd to be using a rc when there's been a release16:30
efriedSo let's leave the version whatever it is in this change set, wait for Rocky to open up, propose the latest to g-r, and then we can remove that line entirely, right?16:31
esbergluWe can move up to 3.2.0 now, or we can just leave it alone until we can get the devstack min bumped16:31
esbergluAdded it to the pad16:33
edmondswwas just about to do that... tx16:34
edmondswefried is there a way to have git realize the file was renamed instead of deleted and a new file added?16:41
edmondswreferring to
edmondswit notices the rename on the ovs file, but not for sea16:42
efriededmondsw: It usually does, whether you tell it so or not.  In this case it did, but it only "recognized" one of them.16:42
efriededmondsw: In this case because we created a new copy of the same file.16:42
efriedThere's a 'git mv' but I suspect it's just an alias for mv + git add16:43
efriedi.e. I don't think it does anything special metadata-wise to indicate the origin of the file.16:43
efriedI think git figures that out on the fly.16:43
efriedbased on how similar the files were.16:44
edmondswyeah... esberglu did you create a new file and delete the old one, or rename?16:44
edmondswwould be nice to have it recognize rename for history and diffs16:44
esbergluI renamed both today. I might have created a new file in the first patch yesterday though? No idea16:46
efriededmondsw: If you look at 3..4, you can see they both show up as renamed.16:47
efriededmondsw: But if you look against base, you can see one rename, one add.16:48
efriedWhich is all as it should be.16:48
edmondswyep, so that proves you can both rename and change the file contents in the same patch16:48
edmondswefried i.e. it's not all as it should be16:48
edmondswshould be 2 renames16:48
efriededmondsw: From base?  No.16:48
edmondswwhy not?16:49
efriedFrom base, we renamed one file and added another.16:49
efriedoh, I think I see what you're saying.16:49
edmondswright... and deleted one. Whereas we should have renamed that one instead of deleting it and adding another16:49
efriedRight, so you can't rename a file *and* edit the contents and have it show up as a rename *and* a diff.16:49 the same patch set.16:50
edmondswseems like you can, based on 3..4 diff16:50
efriedIf you do it across multiple patch sets, that would work.16:50
efriedmm, okay.16:50
edmondsw3..4 shows a rename and changes in the same file16:50
efriedWell, whatever.  I'm not gonna lose sleep over it.16:50
edmondswyeah :)16:50
*** efried is now known as efried_rollin17:02
openstackgerritMerged openstack/nova-powervm master: Update the SEA local.conf
*** AlexeyAbashkin has quit IRC17:23
*** AlexeyAbashkin has joined #openstack-powervm18:17
*** AlexeyAbashkin has quit IRC18:51
openstackgerritEric Berglund proposed openstack/nova-powervm master: Add missing capabilities
openstackgerritEric Berglund proposed openstack/nova-powervm master: Add missing capabilities
*** efried_rollin is now known as efried20:42
*** svenkat has quit IRC22:16
*** edmondsw has quit IRC22:32
*** edmondsw has joined #openstack-powervm22:33
*** edmondsw has quit IRC22:37
*** esberglu has quit IRC23:07
*** svenkat has joined #openstack-powervm23:09
openstackgerritMerged openstack/nova-powervm master: Add missing capabilities
*** apearson has quit IRC23:18
*** svenkat has quit IRC23:52

Generated by 2.15.3 by Marius Gedminas - find it at!