Julia Pavlicova [Thu, 2 Apr 2015 11:35:42 +0000 (13:35 +0200)]
OpenFlow E2E Performance test for OpenDaylight
We was creating a test to run in Continuous Integration that uses
existing perf scripts to estimate the max flows/sec and stores
this value in a CSV file that can be later plotted by Jenkins
Change-Id: Ia54ed94467cbec386ed32e5b01373fccaec89bc5
Signed-off-by: Julia Pavlicova <jpavlico@cisco.com>
Vratko Polak [Fri, 27 Mar 2015 14:48:25 +0000 (15:48 +0100)]
Testplan, suite, variables and support library for basic PCEP testing.
https://wiki.opendaylight.org/view/BGP_LS_PCEP:Lithium_Feature_Tests#PCEP
specifies odl-bgpcep-pcep-all as an user-facing feature.
By M4, an automated test (not necessarily complete)
should be running in CI.
This is the fourth version of suite for testing basic PCEP,
this version is finally ready for merge.
Changes (added and missing) against previous patch sets:
+ Use space-only separation, formatted by RIDE
- Still using .robot extension
+ Documentation and docstrings added everywhere
+ Setup steps compacted to one Suite Setup keyword
+ Suite Teardown added, various log gathering happens there
+ Non-critical tests now can fail on Read Until
+ Start_Pcc_Mock now does tee to a file, its contents goes to Log
+ HfsJson library added, for sorting and indenting JSON strings
+ ${MININET} substituted to place where PCC IP address should be
+ Base64 code for tunnel name is computed in the variable file
+ Long JSON data is in the variable file, in readable form
+ PEP 8 compliant
+ pylint output addressed (not all, pylint is wrong sometimes)
+ Python files now do contain license headers
- actually two sections, with slight incompatibilities between them
+ Robot file now contains a single license section, in suite documentation
- No tests for TCPMD5, they will be added as another Change
Example Sandbox run for this patch set:
https://jenkins.opendaylight.org/sandbox/job/bgpcep-csit-1node-imds-basicpcep-only-master/17/
After this change is merged, separate change will introduce
csit job to releng/builder, triggered by upstream distribution jobs.
After that is merged, another change will add this suite
to the list of suites to be run in integration-verify job.
Until then, integration-verify job is insensitive to this suite.
Change-Id: I24a8704c07780a2a97904284eb3f14de1dde50b3
Signed-off-by: Vratko Polak <vrpolak@cisco.com>
Daniel Farrell [Tue, 31 Mar 2015 22:36:07 +0000 (22:36 +0000)]
Merge "Can't use SSHLibrary.File Should Exist right now :("
Jamo Luhrsen [Tue, 31 Mar 2015 20:43:16 +0000 (13:43 -0700)]
Can't use SSHLibrary.File Should Exist right now :(
Change-Id: I1bd0bbc0582c3fb347555acb48276f72ed70a271
Signed-off-by: Jamo Luhrsen <james.luhrsen@hp.com>
Daniel Farrell [Tue, 31 Mar 2015 20:03:52 +0000 (20:03 +0000)]
Merge "Adding libnetconf and netopeer to fedora"
Moiz Raja [Tue, 31 Mar 2015 19:42:21 +0000 (19:42 +0000)]
Merge "Fixing the verification, which was one less than actual create."
Luis Gomez [Tue, 31 Mar 2015 19:38:44 +0000 (19:38 +0000)]
Merge "Added Perf Test Plan and droptest config from robot"
Jamo Luhrsen [Tue, 31 Mar 2015 19:05:29 +0000 (12:05 -0700)]
Added Perf Test Plan and droptest config from robot
Change-Id: I6c69a948e7e83921e5bb4c348a9a139d12337b33
Signed-off-by: Jamo Luhrsen <james.luhrsen@hp.com>
Shaleen Saxena [Tue, 31 Mar 2015 18:48:36 +0000 (14:48 -0400)]
Fixing the verification, which was one less than actual create.
Change-Id: If4e70ce1c1480288d83d6136f5f1945dd7226925
Signed-off-by: Shaleen Saxena <ssaxena@brocade.com>
Patrick Laurin [Tue, 31 Mar 2015 18:27:17 +0000 (14:27 -0400)]
Adding libnetconf and netopeer to fedora
Change-Id: I8e39f9b94f1be47cb4053c79a28c872b05693266
Signed-off-by: Patrick Laurin <plaurin@inocybe.com>
Peter Gubka [Tue, 31 Mar 2015 08:23:51 +0000 (10:23 +0200)]
fixing unrecognized robot key word
Change-Id: Ia0bcfc39a0ccb82387c65279f33e5fb605522dc3
Signed-off-by: Peter Gubka <pgubka@cisco.com>
Luis Gomez [Sat, 28 Mar 2015 16:47:56 +0000 (16:47 +0000)]
Merge "Cluster sanity test using openflow plugin"
Jamo Luhrsen [Wed, 25 Mar 2015 22:53:44 +0000 (15:53 -0700)]
Cbench automation for CI
--changes after local review. better documentation, failure descriptions, etc
Change-Id: If3cb5cbf0e06270e326fb44c8c3a40f71e1a7473
Signed-off-by: Jamo Luhrsen <james.luhrsen@hp.com>
Luis Gomez [Fri, 27 Mar 2015 20:45:52 +0000 (20:45 +0000)]
Merge "Remove Packetcable from integration."
Daniel Farrell [Fri, 27 Mar 2015 16:18:52 +0000 (16:18 +0000)]
Merge "Use Fedora vs CentOS for tools VM, add Ubuntu cfg script"
Patrick Laurin [Fri, 27 Mar 2015 14:11:53 +0000 (10:11 -0400)]
Use Fedora vs CentOS for tools VM, add Ubuntu cfg script
Change-Id: If730d42fb2f798f94ad4df5ce1bf848277b3da70
Signed-off-by: Patrick Laurin <plaurin@inocybe.com>
Vratko Polak [Fri, 27 Mar 2015 15:42:39 +0000 (16:42 +0100)]
Remove Packetcable from integration.
Packetcable project in current state is preventing
Integration project to create ODL distribution builds.
Until https://git.opendaylight.org/gerrit/#/c/17222/ is merged,
packetcable features and repositories are commented out.
Change-Id: I1a6125586fe1d423a73f767483084efcaa5928e6
Signed-off-by: Vratko Polak <vrpolak@cisco.com>
Peter Gubka [Fri, 27 Mar 2015 12:03:12 +0000 (13:03 +0100)]
Cluster sanity test using openflow plugin
The test configures on each node 1 flow. Then it expects to see
these flows on all nodes in both datastores (config, operational).
Pep8 alligned.
Change-Id: Ic82358e43c53eefbb55eae913a13dfd1dfd0f7e9
Signed-off-by: Peter Gubka <pgubka@cisco.com>
Daniel Farrell [Thu, 26 Mar 2015 22:22:07 +0000 (22:22 +0000)]
Merge "Moved vagrant inline scripts to external scripts inside a new script directory"
Luis Gomez [Thu, 26 Mar 2015 21:41:28 +0000 (21:41 +0000)]
Merge "Revert "Add odlparent relativepath relative to autorelease""
Thanh Ha [Thu, 26 Mar 2015 18:58:22 +0000 (14:58 -0400)]
Revert "Add odlparent relativepath relative to autorelease"
Per discussion on mailing list thread we have decided to revert this
change and achieve relative path updating via scripting in autorelease.
https://lists.opendaylight.org/pipermail/release/2015-March/001754.html
This reverts commit
2da011ef1cb74556db256f76ddb23cf29fefbec3.
Change-Id: I557703bedc13bff0555784935116907d7e4f10ee
Signed-off-by: Thanh Ha <thanh.ha@linuxfoundation.org>
Patrick Laurin [Wed, 25 Mar 2015 21:17:28 +0000 (17:17 -0400)]
Moved vagrant inline scripts to external scripts inside a new script directory
Change-Id: Iff6471684f5d4a3e9eb91987ae7f0fa6318d0123
Signed-off-by: Patrick Laurin <plaurin@inocybe.com>
Luis Gomez [Thu, 26 Mar 2015 02:47:14 +0000 (19:47 -0700)]
Getting rid off test distribution + sorting features
Change-Id: I86486f8cab403764f6602f100f27162dfe5025a4
Signed-off-by: Luis Gomez <ecelgp@gmail.com>
Daniel Farrell [Tue, 24 Mar 2015 05:22:52 +0000 (01:22 -0400)]
Major update of tools VM README
* Add docs for installing dependences
* General refactoring for clarity
* General refactoring of README structure
Change-Id: Ia98a8593a39496f67a466cbbc8ffe447fa0ab746
Signed-off-by: Daniel Farrell <dfarrell@redhat.com>
Luis Gomez [Wed, 25 Mar 2015 02:23:18 +0000 (02:23 +0000)]
Merge "Install ODL in tools VM using Puppet provisioner"
Phillip Shea [Wed, 25 Mar 2015 00:37:39 +0000 (17:37 -0700)]
Removed a space from between library HTTP GET parameters.
This was causing the Jolokia query to fail.
Change-Id: Ieffb9d759770e6917de8d5b4966cebb10b21f6ca
Signed-off-by: Phillip Shea <phillip.shea@hp.com>
Daniel Farrell [Tue, 24 Mar 2015 05:16:36 +0000 (01:16 -0400)]
Install ODL in tools VM using Puppet provisioner
* Free fixes/updates as the Puppet mod is maintained
* Removes large amount of install complexity
* Provides ODL systemd config
* Better ODL install location
* General in-line docs cleanup in Vagrantfile
* Add files to support Puppet provisioning
* Update gitignore for new auto-gen'd files
Change-Id: I590bad2e1c6faedf497145e77d91a1fc4c2a7b52
Signed-off-by: Daniel Farrell <dfarrell@redhat.com>
Luis Gomez [Tue, 24 Mar 2015 00:59:39 +0000 (00:59 +0000)]
Merge "Update Dockerfile and Docker README for SR3"
Daniel Farrell [Mon, 23 Mar 2015 22:51:18 +0000 (18:51 -0400)]
Update Dockerfile and Docker README for SR3
Change-Id: Iee7e93208af16f96a6a8010338697cccf95b9b04
Signed-off-by: Daniel Farrell <dfarrell@redhat.com>
Daniel Farrell [Mon, 23 Mar 2015 22:22:31 +0000 (18:22 -0400)]
Update ODL RPM spec, docs, helper scripts to SR3
* Also added an uninstall.sh helper script
* Generally cleaned up the README
Change-Id: I9b765640b800b68c33099c6832fb052cfc876ecd
Signed-off-by: Daniel Farrell <dfarrell@redhat.com>
Patrick Laurin [Fri, 20 Mar 2015 18:19:47 +0000 (14:19 -0400)]
Add initial Integration team tools Vagrant box
Change-Id: I5421c392f8ca3dbe12e3dd8a9f1a8b2e7f85dfb1
Signed-off-by: Patrick Laurin <plaurin@inocybe.com>
Luis Gomez [Fri, 20 Mar 2015 01:14:17 +0000 (01:14 +0000)]
Merge "action for updated flows changed"
Luis Gomez [Fri, 20 Mar 2015 01:12:58 +0000 (01:12 +0000)]
Merge "Adding odl-sfc-ui and odl-sfc-sb-rest."
Vratko Polak [Thu, 19 Mar 2015 15:35:32 +0000 (16:35 +0100)]
Adding odl-sfc-ui and odl-sfc-sb-rest.
Those 2 features were left out of the previous change to update
the "odl-sfc-all" SFC feature name.
Also added features-sfc-sb-rest repository.
Change-Id: I4d993439dd5350ac4561a497dece50020b2bc14a
Signed-off-by: Brady Johnson <brady.allen.johnson@ericsson.com>
Signed-off-by: Vratko Polak <vrpolak@cisco.com>
Julia Pavlicova [Thu, 19 Mar 2015 14:30:48 +0000 (15:30 +0100)]
action for updated flows changed
there was conflict between match criteria and dec-mpls-ttl,
now this conflict was solved by not using this tag.
Changes XmlComparator in accordance with PEP8 requirements.
Change-Id: Ibc4aa5c0a89e62c7e4bd10f852d23a2b02a86732
Signed-off-by: Julia Pavlicova <jpavlico@cisco.com>
kalaiselvik [Thu, 19 Mar 2015 11:57:40 +0000 (17:27 +0530)]
Adding lacp feature
1)Feature Tests Pass:
https://jenkins.opendaylight.org/releng/job/lacp-merge-master/33/org.opendaylight.lacp$features-lacp/testReport/
2) Features Comply with Guidelines:
https://git.opendaylight.org/gerrit/gitweb?p=lacp.git;a=blob;f=features/src/main/features/features.xml
a) Feature Names: Prefixed with 'odl-' : odl-lacp-ui, odl-lacp-rest
b) Features Descriptions: Present
c) Features Start-Levels: No start-levels on bundle elements or feature elements
3) Features Tested: Features have been tested for functionality with
a) feature:install odl-lacp-ui
b) feature:install odl-integration-compatible-with-all
4) Integration Test Impact - none
Change-Id: I2cc7bdf237b75c40e113f1e53ca15ff3bb66e636
Signed-off-by: kalaiselvik <Kalaiselvi_K@Dell.com>
Vratko Polak [Wed, 18 Mar 2015 18:59:27 +0000 (19:59 +0100)]
Updated features/test list for SFC project
SFC project has changed its feature layout
https://git.opendaylight.org/gerrit/#/c/16621/3
Notably, odl-sfc-all feature is no more.
This patch makes a wild guess about
what the new list of user-facing features
from SFC project should look like.
Change-Id: I8859b74c1f9b3501c378e50d001f6ec8eade559c
Signed-off-by: Vratko Polak <vrpolak@cisco.com>
Luis Gomez [Wed, 18 Mar 2015 05:45:52 +0000 (05:45 +0000)]
Merge "Add basic test suite for usc"
Luis Gomez [Wed, 18 Mar 2015 05:43:42 +0000 (05:43 +0000)]
Merge "Adding tox files to .gitignore"
Luis Gomez [Wed, 18 Mar 2015 00:02:40 +0000 (17:02 -0700)]
Adding tox files to .gitignore
Change-Id: I3f34be7c94435a17e0842d784a8fa692a14ce97f
Signed-off-by: Luis Gomez <ecelgp@gmail.com>
A H [Wed, 18 Mar 2015 01:14:02 +0000 (18:14 -0700)]
Add basic test suite for usc
Change-Id: I74b0a6062a4247bfa22c5201f5f7db57215bd336
Signed-off-by: A H <an.ho@huawei.com>
Luis Gomez [Wed, 18 Mar 2015 04:12:34 +0000 (04:12 +0000)]
Merge "Remove dict comprehension for Python 2.6 support"
Thanh Ha [Wed, 18 Mar 2015 03:33:57 +0000 (23:33 -0400)]
Remove dict comprehension for Python 2.6 support
Change-Id: I7565874a75bbca5a587683ba6ea7a7430d0a06dc
Signed-off-by: Thanh Ha <thanh.ha@linuxfoundation.org>
Vishal Thapar [Tue, 17 Mar 2015 12:13:51 +0000 (17:43 +0530)]
Adding vpnservice features
1) Feature Tests Pass:
https://jenkins.opendaylight.org/releng/job/vpnservice-merge-master/org.opendaylight.vpnservice$vpnservice-features/5/testReport/
2) Features Comply with Guidelines:
https://git.opendaylight.org/gerrit/gitweb?p=vpnservice.git;a=blob;f=features/src/main/features/features.xml;hb=HEAD
a) Feature Names (prefixed with 'odl-'): odl-vpnservice-core
b) Features Descriptions: Present
c) Features Start-Levels: None
3) Features Tested: Features have been tested for functionality with
a) feature:install odl-vpnservice-core
b) feature:install odl-vpnservice-core,odl-integration-compatible-with-all
4) Integration Test Impact: None
Change-Id: I77d9187e7f5139dabbec698e7b92bcabfd84ecd6
Signed-off-by: Vishal Thapar <vishal.thapar@ericsson.com>
Luis Gomez [Tue, 17 Mar 2015 09:56:42 +0000 (09:56 +0000)]
Merge "Add missing comma to list, related to 16559"
Daniel Farrell [Tue, 17 Mar 2015 04:21:13 +0000 (04:21 +0000)]
Merge "Fix pep8 violations in base/run.py"
Daniel Farrell [Tue, 17 Mar 2015 04:20:50 +0000 (04:20 +0000)]
Merge "Fix pep8 violations in base"
Daniel Farrell [Tue, 17 Mar 2015 04:19:04 +0000 (04:19 +0000)]
Merge "Fix pep8 violations in testmodule.py"
Daniel Farrell [Tue, 17 Mar 2015 04:18:39 +0000 (04:18 +0000)]
Merge "Fix pep8 violations in restlib.py"
Daniel Farrell [Tue, 17 Mar 2015 04:18:00 +0000 (04:18 +0000)]
Merge "Fix pep8 violations in arp_handler.py"
Daniel Farrell [Tue, 17 Mar 2015 04:17:38 +0000 (04:17 +0000)]
Merge "Fix pep8 violations in container_manager.py"
Daniel Farrell [Tue, 17 Mar 2015 04:07:50 +0000 (04:07 +0000)]
Merge "Fix pep8 violations in forwarding_manager.py"
Daniel Farrell [Tue, 17 Mar 2015 04:07:26 +0000 (04:07 +0000)]
Merge "Fix pep8 violations in forwarding_rule_manager.py"
Daniel Farrell [Tue, 17 Mar 2015 04:06:55 +0000 (04:06 +0000)]
Merge "Fix pep8 violations in host_tracker.py"
Daniel Farrell [Tue, 17 Mar 2015 04:06:32 +0000 (04:06 +0000)]
Merge "Fix pep8 violations in statistics_manager.py"
Daniel Farrell [Tue, 17 Mar 2015 04:04:01 +0000 (04:04 +0000)]
Merge "Fix pep8 violations in topology_manager.py"
Daniel Farrell [Tue, 17 Mar 2015 04:02:09 +0000 (04:02 +0000)]
Merge "Fix pep8 violations in switch_manager.py"
Daniel Farrell [Tue, 17 Mar 2015 04:01:32 +0000 (04:01 +0000)]
Merge "Fix pep8 violations in Robot_Tool Variables.py"
Daniel Farrell [Tue, 17 Mar 2015 04:00:40 +0000 (04:00 +0000)]
Merge "Fix pep8 violations in test.py"
Daniel Farrell [Tue, 17 Mar 2015 04:00:15 +0000 (04:00 +0000)]
Merge "Fix pep8 violations in test1"
Daniel Farrell [Tue, 17 Mar 2015 03:59:53 +0000 (03:59 +0000)]
Merge "Fix pep8 violations in libconfig.py"
Daniel Farrell [Tue, 17 Mar 2015 03:58:15 +0000 (03:58 +0000)]
Merge "Fix pep8 violations in odl_tests.py"
Daniel Farrell [Tue, 17 Mar 2015 03:56:46 +0000 (03:56 +0000)]
Merge "Fix pep8 violations in Common"
Daniel Farrell [Tue, 17 Mar 2015 03:56:22 +0000 (03:56 +0000)]
Merge "Fix pep8 violations in ArpHandler"
Daniel Farrell [Tue, 17 Mar 2015 03:56:00 +0000 (03:56 +0000)]
Merge "Fix pep8 violations in ContainerManager"
Daniel Farrell [Tue, 17 Mar 2015 03:55:42 +0000 (03:55 +0000)]
Merge "Fix pep8 violations in HostTracker"
Daniel Farrell [Tue, 17 Mar 2015 03:55:06 +0000 (03:55 +0000)]
Merge "Fix pep8 violations in ForwardingManager"
Daniel Farrell [Tue, 17 Mar 2015 03:54:07 +0000 (03:54 +0000)]
Merge "Fix pep8 violations in ForwardingRuleManager"
Daniel Farrell [Tue, 17 Mar 2015 03:53:06 +0000 (03:53 +0000)]
Merge "Fix pep8 violations in StatisticsManager"
Daniel Farrell [Tue, 17 Mar 2015 03:52:39 +0000 (03:52 +0000)]
Merge "Fix pep8 violations in MininetHandler"
Daniel Farrell [Tue, 17 Mar 2015 03:51:52 +0000 (03:51 +0000)]
Merge "Fix pep8 violations in Topology"
Daniel Farrell [Tue, 17 Mar 2015 03:51:25 +0000 (03:51 +0000)]
Merge "Fix pep8 violations in SwitchManager"
Daniel Farrell [Tue, 17 Mar 2015 03:50:48 +0000 (03:50 +0000)]
Merge "Fix pep8 violations in TopologyManager"
Daniel Farrell [Tue, 17 Mar 2015 03:49:48 +0000 (03:49 +0000)]
Merge "Fix pep8 violations in RequestsLibrary"
Daniel Farrell [Tue, 17 Mar 2015 03:48:36 +0000 (03:48 +0000)]
Merge "Fix pep8 violations in restlib"
Daniel Farrell [Tue, 17 Mar 2015 03:46:40 +0000 (03:46 +0000)]
Merge "Fix pep8 violations in testmodule"
Daniel Farrell [Tue, 17 Mar 2015 03:42:01 +0000 (03:42 +0000)]
Merge "Fix pep8 violations in cluster-deployer (3/3)"
Daniel Farrell [Tue, 17 Mar 2015 03:37:44 +0000 (03:37 +0000)]
Merge "Fix pep8 violations in cluster-deployer (2/3)"
Daniel Farrell [Tue, 17 Mar 2015 03:36:38 +0000 (03:36 +0000)]
Merge "Fix pep8 violations in cluster-deployer (1/3)"
Daniel Farrell [Tue, 17 Mar 2015 03:36:00 +0000 (03:36 +0000)]
Merge "Fix pep8 violations in clustering-functional-test/settings"
Daniel Farrell [Tue, 17 Mar 2015 03:32:10 +0000 (03:32 +0000)]
Merge "Fix pep8 violations in util"
Daniel Farrell [Tue, 17 Mar 2015 03:30:48 +0000 (03:30 +0000)]
Merge "Fix pep8 violations in crud"
Daniel Farrell [Tue, 17 Mar 2015 03:27:46 +0000 (03:27 +0000)]
Merge "Fix pep8 violations in config_cleanup"
Daniel Farrell [Tue, 17 Mar 2015 03:27:18 +0000 (03:27 +0000)]
Merge "Fix pep8 violations in flow_add_delete_test"
Daniel Farrell [Tue, 17 Mar 2015 03:26:38 +0000 (03:26 +0000)]
Merge "Fix pep8 violations in flow_config_blaster_fle"
Daniel Farrell [Tue, 17 Mar 2015 03:26:02 +0000 (03:26 +0000)]
Merge "Fix pep8 violations ni flow_config_blaster"
Daniel Farrell [Tue, 17 Mar 2015 03:23:39 +0000 (03:23 +0000)]
Merge "Fix pep8 violations in inventory_crawler"
Daniel Farrell [Tue, 17 Mar 2015 03:22:45 +0000 (03:22 +0000)]
Merge "Fix pep8 violations in inventory_perf.py"
Daniel Farrell [Tue, 17 Mar 2015 03:18:54 +0000 (03:18 +0000)]
Merge "Fix pep8 violations in shard_perf_test"
Daniel Farrell [Tue, 17 Mar 2015 03:18:17 +0000 (03:18 +0000)]
Merge "Fix pep8 violations in csit/variables/Variables.py"
Thanh Ha [Mon, 16 Mar 2015 04:43:14 +0000 (00:43 -0400)]
Fix pep8 violations in base/run.py
Change-Id: Ieda881623913dcf88c34af8dccfe53c966949414
Signed-off-by: Thanh Ha <thanh.ha@linuxfoundation.org>
Thanh Ha [Mon, 16 Mar 2015 04:42:25 +0000 (00:42 -0400)]
Fix pep8 violations in base
Change-Id: I2d0171a9517fa79cf0476aa45809f395aff4bb6d
Signed-off-by: Thanh Ha <thanh.ha@linuxfoundation.org>
Daniel Farrell [Mon, 16 Mar 2015 21:28:36 +0000 (21:28 +0000)]
Merge "Fix pep8 violations in csit/suites/l2switch/topologies/customtopo.py"
Daniel Farrell [Mon, 16 Mar 2015 21:25:13 +0000 (21:25 +0000)]
Merge "Fix pep8 violations in csit/libraries/SwitchClasses/H3C.py"
Daniel Farrell [Mon, 16 Mar 2015 21:19:55 +0000 (21:19 +0000)]
Merge "Fix pep8 violations in csit/libraries/SwitchClasses/H3C_5920.py"
Daniel Farrell [Mon, 16 Mar 2015 21:19:34 +0000 (21:19 +0000)]
Merge "Fix pep8 violations in csit/libraries/SwitchClasses/BaseSwitch.py"
Daniel Farrell [Mon, 16 Mar 2015 21:15:55 +0000 (21:15 +0000)]
Merge "Fix pep8 violations in csit/libraries/SwitchClasses/Ovs.py"
Daniel Farrell [Mon, 16 Mar 2015 20:59:16 +0000 (16:59 -0400)]
Add missing comma to list, related to 16559
* PEP8 fix in 16559 exposed a bug-ish not-quite-failure
* Resulted in correct behavior, but mostly luckily accident
* Post-formatting-fix bug just needed a comma, which this adds
Change-Id: I8644dec30c092ca9a81102f9a0da2a47b5ae8e7c
Signed-off-by: Daniel Farrell <dfarrell@redhat.com>
Daniel Farrell [Mon, 16 Mar 2015 20:54:24 +0000 (20:54 +0000)]
Merge topic 'pep8'
* changes:
Fix pep8 violations in csit/libraries/SwitchClasses/ProVision_3800.py
Fix pep8 violations in csit/libraries/SwitchClasses/ProVision.py
Daniel Farrell [Mon, 16 Mar 2015 20:10:53 +0000 (20:10 +0000)]
Merge "Fix pep8 violations in csit/libraries/ClusterStateLibrary.py"
Code Review / integration.git / log