Here's a patch for the two style fixes that are needed:
diff --git a/cloudinit/reporting/handlers.py b/cloudinit/reporting/handlers.py index c77bf2d1..e12d8ea8 100644 --- a/cloudinit/reporting/handlers.py +++ b/cloudinit/reporting/handlers.py @@ -284,7 +284,8 @@ class HyperVKvpReportingHandler(ReportingHandler): try: if not os.path.exists(self._kvp_file_path): LOG.warning( - "skip writing event %s to %s because file not exists.", + "skip writing event %s to %s because " + + "file not exists.", event.as_string(), self._kvp_file_path) encoded_event = self._encode_event(event) diff --git a/tests/unittests/test_reporting_hyperv.py b/tests/unittests/test_reporting_hyperv.py index 0b62506a..2e64c6c7 100644 --- a/tests/unittests/test_reporting_hyperv.py +++ b/tests/unittests/test_reporting_hyperv.py @@ -5,7 +5,6 @@ from cloudinit.reporting import handlers
import json import os -import tempfile
from cloudinit import util from cloudinit.tests.helpers import CiTestCase
« Back to merge proposal
Here's a patch for the two style fixes that are needed:
diff --git a/cloudinit/ reporting/ handlers. py b/cloudinit/ reporting/ handlers. py reporting/ handlers. py reporting/ handlers. py ingHandler( ReportingHandle r):
try:
if not os.path. exists( self._kvp_ file_path) :
LOG. warning(
event. as_string( ),
self._ kvp_file_ path)
encoded_ event = self._encode_ event(event) unittests/ test_reporting_ hyperv. py b/tests/ unittests/ test_reporting_ hyperv. py unittests/ test_reporting_ hyperv. py unittests/ test_reporting_ hyperv. py
index c77bf2d1..e12d8ea8 100644
--- a/cloudinit/
+++ b/cloudinit/
@@ -284,7 +284,8 @@ class HyperVKvpReport
- "skip writing event %s to %s because file not exists.",
+ "skip writing event %s to %s because " +
+ "file not exists.",
diff --git a/tests/
index 0b62506a..2e64c6c7 100644
--- a/tests/
+++ b/tests/
@@ -5,7 +5,6 @@ from cloudinit.reporting import handlers
import json
import os
-import tempfile
from cloudinit import util tests.helpers import CiTestCase
from cloudinit.