*** edmondsw has joined #openstack-powervm | 00:31 | |
*** edmondsw has quit IRC | 00:36 | |
*** esberglu has quit IRC | 01:26 | |
*** tonyb has quit IRC | 01:47 | |
*** tonyb has joined #openstack-powervm | 01:49 | |
*** edmondsw has joined #openstack-powervm | 02:19 | |
*** edmondsw has quit IRC | 02:24 | |
*** edmondsw has joined #openstack-powervm | 04:07 | |
*** edmondsw has quit IRC | 04:12 | |
*** chhavi has joined #openstack-powervm | 05:02 | |
*** edmondsw has joined #openstack-powervm | 05:56 | |
*** edmondsw has quit IRC | 06:00 | |
*** esberglu has joined #openstack-powervm | 06:40 | |
*** esberglu has quit IRC | 06:40 | |
*** esberglu has joined #openstack-powervm | 06:40 | |
*** edmondsw has joined #openstack-powervm | 07:44 | |
*** edmondsw has quit IRC | 07:48 | |
*** edmondsw has joined #openstack-powervm | 09:32 | |
*** edmondsw has quit IRC | 09:37 | |
*** edmondsw has joined #openstack-powervm | 11:20 | |
*** edmondsw has quit IRC | 11:24 | |
*** edmondsw has joined #openstack-powervm | 11:36 | |
*** chhavi has quit IRC | 11:40 | |
*** edmondsw has quit IRC | 12:15 | |
*** edmondsw has joined #openstack-powervm | 13:13 | |
*** edmondsw has quit IRC | 13:18 | |
*** apearson has quit IRC | 13:35 | |
*** edmondsw has joined #openstack-powervm | 14:37 | |
*** tjakobs has joined #openstack-powervm | 15:10 | |
*** chhavi has joined #openstack-powervm | 15:29 | |
openstackgerrit | Merged openstack/networking-powervm master: Avoid tox_install.sh for constraints support https://review.openstack.org/528779 | 15:32 |
---|---|---|
*** apearson has joined #openstack-powervm | 15:34 | |
esberglu | edmondsw: Regarding your latest comment on the SEA IT CI patch | 16:00 |
esberglu | Right now ALL branches use the same blacklist | 16:01 |
esberglu | Which works alright for OOT because there isn't that much difference in functionality between the branches | 16:01 |
esberglu | For IT not so much | 16:01 |
esberglu | I'm planning on proposing additional CI patches that differentiate the lists based on which branch is being used | 16:02 |
edmondsw | hmm... so update the SEA patch's commit message to note that there will be a followup for this? | 16:02 |
esberglu | But for now I say we go ahead and push this CI change through so we can start getting some volume on SEA CI tempest testing | 16:03 |
esberglu | Will do | 16:03 |
edmondsw | esberglu what is the value of the SEA CI tempest testing if it's not actually testing networking? | 16:03 |
edmondsw | wondering if we should hold off that patch and rebase it on the upcoming patches you mentioned | 16:04 |
esberglu | edmondsw: It is testing networking. Just not the handful of networking tests that aren't getting skipped due to tempest config on pike | 16:05 |
esberglu | edmondsw: http://184.172.12.213/manual/master_it_blacklist_v2/powervm_os_ci.html.gz | 16:08 |
esberglu | Networking tests are at the bottom of that run | 16:08 |
esberglu | Plus all of the tests that spawn servers hit the plug/unplug paths | 16:08 |
*** openstackgerrit has quit IRC | 16:19 | |
esberglu | edmondsw: See my latest on 6228 | 16:57 |
esberglu | Then I can remove that set of networking tests from the base blacklist and put them in the pike blacklist in the SEA CI change | 16:59 |
*** chhavi has quit IRC | 17:05 | |
edmondsw | esberglu might be better to have a complete blacklist per release, so you could e.g. disable something in master that is NOT disabled in pike | 17:09 |
edmondsw | what do you think? | 17:09 |
edmondsw | e.g. if we had ever supported cells v1, we would want to test that in older releases and not in newer ones | 17:10 |
edmondsw | esberglu I guess we could always do that later if/when it's needed | 17:13 |
esberglu | edmondsw: That's the type of thing that we _should_ be able to configure through the tempest conf. But no guarantee of that | 17:14 |
esberglu | I just don't like the idea of trying to manage a bunch of blacklists I guess | 17:15 |
edmondsw | esberglu sounds good. I just added a couple easy comments and then I think we're good here | 17:15 |
edmondsw | esberglu 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 it | 17:51 |
esberglu | edmondsw: ack | 18:04 |
esberglu | Addressed your comments on the blacklist change | 18:05 |
edmondsw | esberglu +2 | 18:12 |
*** ChanServ sets mode: +r | 18:21 | |
*** efried has quit IRC | 19:09 | |
*** edmondsw has quit IRC | 21:22 | |
*** apearson has quit IRC | 23:20 | |
*** esberglu has quit IRC | 23:38 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!