Wednesday, 2018-01-03

*** edmondsw has joined #openstack-powervm00:31
*** edmondsw has quit IRC00:36
*** esberglu has quit IRC01:26
*** tonyb has quit IRC01:47
*** tonyb has joined #openstack-powervm01:49
*** edmondsw has joined #openstack-powervm02:19
*** edmondsw has quit IRC02:24
*** edmondsw has joined #openstack-powervm04:07
*** edmondsw has quit IRC04:12
*** chhavi has joined #openstack-powervm05:02
*** edmondsw has joined #openstack-powervm05:56
*** edmondsw has quit IRC06:00
*** esberglu has joined #openstack-powervm06:40
*** esberglu has quit IRC06:40
*** esberglu has joined #openstack-powervm06:40
*** edmondsw has joined #openstack-powervm07:44
*** edmondsw has quit IRC07:48
*** edmondsw has joined #openstack-powervm09:32
*** edmondsw has quit IRC09:37
*** edmondsw has joined #openstack-powervm11:20
*** edmondsw has quit IRC11:24
*** edmondsw has joined #openstack-powervm11:36
*** chhavi has quit IRC11:40
*** edmondsw has quit IRC12:15
*** edmondsw has joined #openstack-powervm13:13
*** edmondsw has quit IRC13:18
*** apearson has quit IRC13:35
*** edmondsw has joined #openstack-powervm14:37
*** tjakobs has joined #openstack-powervm15:10
*** chhavi has joined #openstack-powervm15:29
openstackgerritMerged openstack/networking-powervm master: Avoid tox_install.sh for constraints support  https://review.openstack.org/52877915:32
*** apearson has joined #openstack-powervm15:34
esbergluedmondsw: Regarding your latest comment on the SEA IT CI patch16:00
esbergluRight now ALL branches use the same blacklist16:01
esbergluWhich works alright for OOT because there isn't that much difference in functionality between the branches16:01
esbergluFor IT not so much16:01
esbergluI'm planning on proposing additional CI patches that differentiate the lists based on which branch is being used16:02
edmondswhmm... so update the SEA patch's commit message to note that there will be a followup for this?16:02
esbergluBut for now I say we go ahead and push this CI change through so we can start getting some volume on SEA CI tempest testing16:03
esbergluWill do16:03
edmondswesberglu what is the value of the SEA CI tempest testing if it's not actually testing networking?16:03
edmondswwondering if we should hold off that patch and rebase it on the upcoming patches you mentioned16:04
esbergluedmondsw: It is testing networking. Just not the handful of networking tests that aren't getting skipped due to tempest config on pike16:05
esbergluedmondsw: http://184.172.12.213/manual/master_it_blacklist_v2/powervm_os_ci.html.gz16:08
esbergluNetworking tests are at the bottom of that run16:08
esbergluPlus all of the tests that spawn servers hit the plug/unplug paths16:08
*** openstackgerrit has quit IRC16:19
esbergluedmondsw: See my latest on 622816:57
esbergluThen I can remove that set of networking tests from the base blacklist and put them in the pike blacklist in the SEA CI change16:59
*** chhavi has quit IRC17:05
edmondswesberglu might be better to have a complete blacklist per release, so you could e.g. disable something in master that is NOT disabled in pike17:09
edmondswwhat do you think?17:09
edmondswe.g. if we had ever supported cells v1, we would want to test that in older releases and not in newer ones17:10
edmondswesberglu I guess we could always do that later if/when it's needed17:13
esbergluedmondsw: That's the type of thing that we _should_ be able to configure through the tempest conf. But no guarantee of that17:14
esbergluI just don't like the idea of trying to manage a bunch of blacklists I guess17:15
edmondswesberglu sounds good. I just added a couple easy comments and then I think we're good here17:15
edmondswesberglu fyi, cinder has merged support for service tokens, so if we see any timeout issues with cinder calling another service's APIs we could use that to fix it17:51
esbergluedmondsw: ack18:04
esbergluAddressed your comments on the blacklist change18:05
edmondswesberglu +218:12
*** ChanServ sets mode: +r 18:21
*** efried has quit IRC19:09
*** edmondsw has quit IRC21:22
*** apearson has quit IRC23:20
*** esberglu has quit IRC23:38

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