Enable flake8 rule W292 No newline at end of file

.github/linters/.flake8

@@ -16,4 +16,4 @@
 # under the License.
 
 [flake8]
-select = W291,W293
+select = W291,W292,W293

api/test/integration/api/__init__.py

@@ -13,4 +13,4 @@
 # "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
 # KIND, either express or implied.  See the License for the
 # specific language governing permissions and limitations
-# under the License.
+# under the License.

api/test/integration/api/test/__init__.py

@@ -13,4 +13,4 @@
 # "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
 # KIND, either express or implied.  See the License for the
 # specific language governing permissions and limitations
-# under the License.
+# under the License.

api/test/integration/api/test/account/__init__.py

@@ -13,4 +13,4 @@
 # "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
 # KIND, either express or implied.  See the License for the
 # specific language governing permissions and limitations
-# under the License.
+# under the License.

plugins/hypervisors/baremetal/src/main/resources/security_group_agent/security_group_agent/xmlobject.py

@@ -94,4 +94,4 @@ def _loads(node):
 def loads(xmlstr):
     xmlstr = re.sub(r'xmlns=".*"', '', xmlstr)
     root = etree.fromstring(xmlstr)
-    return _loads(root)
+    return _loads(root)

plugins/hypervisors/ovm/src/main/scripts/vm/hypervisor/ovm/OvmLoggerModule.py

@@ -51,4 +51,4 @@ def error(self, func, msg=None):
     def warning(self, func, msg=None):
         assert callable(func), "%s is not a function"%func
         fmt = "[%s.%s]: "%(self.className, func.__name__)
-        self.logger.warning("%s%s"%(fmt,msg))
+        self.logger.warning("%s%s"%(fmt,msg))

plugins/hypervisors/ovm/src/main/scripts/vm/hypervisor/ovm/OvmObjectModule.py

@@ -20,4 +20,4 @@
 '''
 
 class OvmObject(object):
-   pass
+   pass

test/integration/broken/test_vpc_vm_life_cycle.py

@@ -789,4 +789,4 @@ def test_10_expunge_instance_in_network(self):
                                   id=self.lb_rule.id,
                                   listall=True
                                   )
-        return
+        return

test/integration/smoke/test_annotations.py

@@ -253,4 +253,4 @@ def test_06_add_adminsonly_and_update_annotation_visibility(self):
 
         # Remove the annotation
         self.removeAnnotation(annotation_id)
-        del self.added_annotations[-1]
+        del self.added_annotations[-1]

test/integration/smoke/test_network.py

@@ -2043,4 +2043,4 @@ def test_03_destroySharedNetwork(self):
             len(nic_ip_address),
             0,
             "Failed to find the placeholder IP"
-        )
+        )

test/integration/smoke/test_persistent_network.py

@@ -398,4 +398,4 @@ def validate_persistent_network_resources_created_on_host(self, network_vlan):
                     0,
                     "Failed to find bridge on the breth1-" + str(network_vlan))
         elif self.hypervisor.lower() in ["xenserver", "vmware"]:
-            self.verify_network_setup_on_host_per_cluster(self.hypervisor.lower(), network_vlan)
+            self.verify_network_setup_on_host_per_cluster(self.hypervisor.lower(), network_vlan)

test/integration/smoke/test_scale_vm.py

@@ -480,4 +480,4 @@ def test_03_scale_vm(self):
         else:
             self.fail("Expected an exception to be thrown, failing")
 
-        return
+        return

test/selenium/lib/initialize.py

@@ -43,4 +43,4 @@ def getMSip():
 
     for arg in sys.argv[1:]:
         MS_ip = arg
-        return MS_ip
+        return MS_ip