gerrit.opnfv Code Review - opnfvdocs.git/log

George Paraskevopoulos [Fri, 24 Mar 2017 16:26:02 +0000 (18:26 +0200)]

Update git submodules

* Update docs/submodules/sfc from branch 'stable/danube'
  - Revert "Change from functest logger to python logging"

    This reverts commit cb311aad19c732fa006b50795ab66687e01b95a6.

    Change-Id: I123c30acd765df692483eb5f91fecd1478219201
Signed-off-by: George Paraskevopoulos <geopar@intracom-telecom.com>
    (cherry picked from commit e011a3d122888444c36fba6ee74b27b955356480)

  - Limit topologies to the ones working

    Due to the classifier rules only deployed where VNFs are, some use cases will
    fail because the client traffic never gets classified

    Change-Id: I77b10231b7a34497ed8ecd42d72e08e0b4481bfa
Signed-off-by: Manuel Buil <mbuil@suse.com>
  - Delete ACL when deleting a classifier

    Change-Id: I8e54892450e903aa51f888229940e6371a0710f7
Signed-off-by: George Paraskevopoulos <geopar@intracom-telecom.com>
    (cherry picked from commit d41124f9c8cd14cfab61d4848a70aa227e80eefb)

  - Added more logs to classification check

    The classification check is not working in CI and we don't understand why.
    I added a lot more logs to check

    Change-Id: I81bcc8874c5e5e843e0c301e1d66d5d11de092f3
Signed-off-by: Manuel Buil <mbuil@suse.com>
  - Perform cmd argument parsing when called from cli

    To be able to import run_tests as a module we must not try to parse
    command line arguments in global scope

    Also import the testcase using the full module path

    Change-Id: I1696800a16301d446bf9c926bdaf2b545435a2e7
Signed-off-by: George Paraskevopoulos <geopar@intracom-telecom.com>
    (cherry picked from commit 08aa86e1e7cb874aac72c831e239c003070e5ca3)

  - Bug fix in CI: security groups not working

    CI tests create a lot of security groups with the name default.
    Our SFs are assigned to a default security group but there are so
    many that the result is not predictable and most likely wrong.
    We add it now to example-sg sec. group

    Change-Id: I64984aa7b6114747c5bab5fb2e4b8d8d32865bb0
Signed-off-by: Manuel Buil <mbuil@suse.com>
    (cherry picked from commit 3c6e0efc2f4575ec2cf2e1304bdc1b66cdc08b06)

  - Bug fix in CI: security groups

    CI tests create a lot of security groups with the name default. Our SFs are
    assigned to a default security group but there are so many that the result is
    not predictable and most likely wrong. We add it now to example-sg sec. group

    Change-Id: I120765a33e55e47b78ef3c8664c82fb147ce9df0
Signed-off-by: Manuel Buil <mbuil@suse.com>
    (cherry picked from commit 6282c5cce71ae994aaece374375234cf207992f8)

  - Fix run_tests

    It was impossible to get a status='PASS'

    Change-Id: I016346c1acf38ae1a130afbfd132166b72d5e1af
Signed-off-by: Manuel Buil <mbuil@suse.com>
    (cherry picked from commit d63f9fc8f18f42cd80a32e3b93aed19091beccee)

  - Bug fix: Eth header build with wrong dest. mac

    JIRA: SFC-89

    The second test fetches the destination mac of the previous test because an
    entry with that IP is in the arp table. However, that table is outdated. We
    must flush the arp table before starting the new test.

    Change-Id: Ie00345bba13793dc68e8442421861a2555e5ba7d
Signed-off-by: Manuel Buil <mbuil@suse.com>
    (cherry picked from commit 4d07f0f7e2f03486c8d78a24282c5e4e7fc59253)

  - Add all topologies to the test

    We were limiting the topologies to the ones working but we believe
    all should work now

    Change-Id: I11921916105a149c585cd934150de2898286b142
Signed-off-by: Manuel Buil <mbuil@suse.com>
    (cherry picked from commit 1b9b0ad28243362b7953db12ebd072d9c6a99aff)

  - Cleanup IETF-ACL in OpenDaylight

    Change-Id: I6914a5fbe7eb30b2d76b89d8f38b7d3d16e7b44a
Signed-off-by: George Paraskevopoulos <geopar@intracom-telecom.com>
    (cherry picked from commit 3097fef23e4ae1c89b535c1d890e44710b33eea4)

  - Disable resource_finder_cache in controllers

    Heat caching somehow messes our tests so we disable it before the tests
    start.

    For instance, if we do:

    1. create example-net with NET_ID1
    2. create VNFD1 with example-net
    3. delete VNFD1
    4. delete NET_ID1
    5. create example-net with NET_ID2
    6. create VNFD2 with example-net

    heat has NET_ID1 cached as the id of example-net and tries to use that
    instead of NET_ID2 to create VNFD2. This off course leads to a crash
    since NET_ID1 does not exist anymore.

    In the future we should investigate further why heat cache does not play
    well for us (probably bc of tacker).

    Change-Id: Iac07d78641e2d1ff0c2d67e46752d551833f19fd
Signed-off-by: George Paraskevopoulos <geopar@intracom-telecom.com>
    (cherry picked from commit d0fec46322915ef8b49e1f7ab9c485ea2cdd399a)

  - VNFDs were not deleted

    Added vnfd_delete in the cleanup library

    Change-Id: Iaa3e9ecb33a15bf882585786271fef206a54ffa0
Signed-off-by: Manuel Buil <mbuil@suse.com>
    (cherry picked from commit 5342d88d799359ce9f92b0cef26382d26fce9dfb)

  - Give time to start the firewall process

    Sometimes test sfc_one is failing because the firewall is not started when
    the test begins. Waiting some seconds after starting the process corrects the
    problem

    Change-Id: Iebef292fa66338c3e27a325333cc5f1993facac5
Signed-off-by: Manuel Buil <mbuil@suse.com>
    (cherry picked from commit 891d42491fdabea0b3907cba8cbf220c507398b8)

  - Run tests in order

    Change-Id: I3cbbcd17182bf6f1f6558cb199577d989ac62ae6
Signed-off-by: George Paraskevopoulos <geopar@intracom-telecom.com>
    (cherry picked from commit d3b5ba3c16b0918060d1fc800bd47e688146f2c4)

  - Change from functest logger to python logging

    Change-Id: Ie7b2a63c301819a79768d1b286ee00693cd1129b
Signed-off-by: George Paraskevopoulos <geopar@intracom-telecom.com>
    (cherry picked from commit cb311aad19c732fa006b50795ab66687e01b95a6)

  - Add checks for cleanup utilities

    Change-Id: I1facd54b11ef6a090e39dd5977eb5cf7183147af
Signed-off-by: George Paraskevopoulos <geopar@intracom-telecom.com>
    (cherry picked from commit 98bb8ed7510d118a680079973aa8f7dce406be37)

  - Add ODL cleanup utilities

    ODL does not clean all resources properly after tacker deletions, so we
    add this as a post processing step

    Change-Id: Ic485ee32b195f72235922ebe613af6c010cb57db
Signed-off-by: George Paraskevopoulos <geopar@intracom-telecom.com>
    (cherry picked from commit e48b950bc77e37fbda611415577a7388e19c1ff6)

  - Add more logs to the CI

    We want to double check if the CI is using the correct tacker plugin
    version

    Change-Id: Ib6082014988daee316553d980c5e7f88d316fe8c
Signed-off-by: “Manuel Buil” <mbuil@suse.com>
    (cherry picked from commit 268fc4edfd7560bcf5b5e1248df05face402074a)

  - Implement delete.sh as a python module

    We need to call delete.sh after each test is run. The script is
    implemented as a python module and called inside run_tests.py

    Change-Id: I23a794be522b4713a44cc1d56587e0aadc6f1210
Signed-off-by: George Paraskevopoulos <geopar@intracom-telecom.com>
    (cherry picked from commit 8869d175363dfec69cc70fd8cb9bef56e8ebc115)

  - Remove SFF_filter

    After further investigation we found out that the classifier is deployed in all
    computes. When deploying several tests, ODL starts to function very slowly and
    sometimes the classifier takes a long time to be deployed in one compute. That
    tricked us and made us think that it was not deployed everywhere

    Change-Id: I474a05f77a6342be741d14ffbdb674ef7f970908
Signed-off-by: “Manuel Buil” <mbuil@suse.com>
    (cherry picked from commit 70445ba93cb07164933ed0c5aac8db32f6f652d4)

  - Run all testcases even if one throws an exception

    If the first testcase throws an exception, the rest were not being
    executed

    Change-Id: I5fdc521da840ffdbc7a0bde120d7bfe7907b881c
Signed-off-by: “Manuel Buil” <mbuil@suse.com>
    (cherry picked from commit 84f3ea23ccc9f6e19a86748519deb600b2ac95e3)

  - Log credentials

    We have to make sure that we source the correct credentials in the CI

    Change-Id: I4d988943588bbc99a3118cb89eabfea37c41daf6
Signed-off-by: George Paraskevopoulos <geopar@intracom-telecom.com>
    (cherry picked from commit 1d72729b1513e419ab1d97f294410a5cfdd1c62f)

  - Bug fix: rsps and first_RSPs must be restarted

    JIRA: SFC-88

    Not a blocking bug but when the classifier is implemented in several computes,
    we loop all computes. By the time the second compute is inspected, more than
    10 seconds passed and thus, the classification rules are already there.
    first_RSP should not be evaluated and rsps must be restarted

    Change-Id: I70858d6b423eebd8088a79aebebb4950f031878d
Signed-off-by: “Manuel Buil” <mbuil@suse.com>
    (cherry picked from commit 189642fff7428457d3992c365eab9a9f5c0bea90)

  - Bug fix: disable non-working topologies

    JIRA: SFC-87

    Due to ODL bugs, some topologies do not work

    Change-Id: Ia2fcfe5948c308ce2348b52c01aa3ce58ac318d6
Signed-off-by: “Manuel Buil” <mbuil@suse.com>
    (cherry picked from commit 184f787f3a8a57695fa3b9b8f17c65b9aeee1904)

  - Remove security group exists error logs

    We copied the create_secgroup_rule function from
    functest.openstack_utils and removed the logger.exception("Security
    group exists") line.

    This is very ugly and we should investigate an upstream fix after D
    release.

    Change-Id: I1674a77ebc3b3e3132b2b9e1f16a84106b2350eb
Signed-off-by: George Paraskevopoulos <geopar@intracom-telecom.com>
    (cherry picked from commit 4585d8fe2a4b40f62b96bdd38dc1a14f32510c93)

  - Bug fix: Classification check failing

    If both SFs are in one host, the classification rules are not implemented
    in all the hosts

    Change-Id: Ifea131a86f49d33fdc6110a13d5fdc6c2796f7fa
Signed-off-by: Manuel Buil <mbuil@suse.com>
    (cherry picked from commit da6d1d2e018e358f71ad734766580a7343829c29)

  - Bug fix: classif. check not working for one chain

    In the test case with only one chain the wait_classification function was not
    working

    Change-Id: I4cff1e61130b6fc68d736108a0da048909dad750
Signed-off-by: Manuel Buil <mbuil@suse.com>
    (cherry picked from commit 4b082d08b46941da06630926a117054881a8aa1b)

  - Bug fix: client and server not using shuffler

    Client and server were not being created based on the topology shuffler and
    thus the topology was not followed

    Change-Id: I1d7228d2d75e4ecf57ff8eb1c4226b963b2b9dd4
Signed-off-by: “Manuel Buil” <mbuil@suse.com>