Changeset 243177 in webkit

Timestamp:

Mar 19, 2019, 4:02:26 PM (7 years ago)

Author:

aakash_jain@apple.com

Message:

[ews-build] Improve summary for PrintConfiguration step
​https://bugs.webkit.org/show_bug.cgi?id=195945

Reviewed by Lucas Forschler.

• BuildSlaveSupport/ews-build/steps.py:

(PrintConfiguration.getResultSummary): Override getResultSummary.

Location:

trunk/Tools

Files:

• BuildSlaveSupport/ews-build/steps.py (modified) (2 diffs)
• ChangeLog (modified) (1 diff)

trunk/Tools/BuildSlaveSupport/ews-build/steps.py

@@ -757 +757 @@
     name = 'configuration'
     description = ['configuration']
-    descriptionDone = ['configuration']
     haltOnFailure = False
     flunkOnFailure = False
     warnOnFailure = False
+    logEnviron = False
     command_list = [['hostname'],
                     ['df', '-hl'],
@@ -770 +770 @@
         super(PrintConfiguration, self).__init__(timeout=60, **kwargs)
         self.commands = []
+        self.log_observer = logobserver.BufferLogObserver(wantStderr=True)
+        self.addLogObserver('stdio', self.log_observer)
         # FIXME: Check platform before running platform specific commands.
         for command in self.command_list:
-            self.commands.append(util.ShellArg(command=command, logfile=command[0]))
+            self.commands.append(util.ShellArg(command=command, logfile='stdio'))
+
+    def convert_build_to_os_name(self, build):
+        if not build:
+            return 'Unknown'
+
+        build_to_name_mapping = {
+            '10.14': 'Mojave',
+            '10.13': 'High Sierra',
+            '10.12': 'Sierra',
+            '10.11': 'El Capitan',
+            '10.10': 'Yosemite',
+            '10.9': 'Maverick',
+            '10.8': 'Mountain Lion',
+            '10.7': 'Lion',
+            '10.6': 'Snow Leopard',
+            '10.5': 'Leopard',
+        }
+
+        for key, value in build_to_name_mapping.iteritems():
+            if build.startswith(key):
+                return value
+        return 'Unknown'
+
+    def getResultSummary(self):
+        if self.results != SUCCESS:
+            return {u'step': u'Failed to print configuration'}
+        logText = self.log_observer.getStdout() + self.log_observer.getStderr()
+        configuration = u''
+        match = re.search('ProductVersion:[ \t]*(.+?)\n', logText)
+        if match:
+            os_version = match.group(1).strip()
+            os_name = self.convert_build_to_os_name(os_version)
+            configuration = u'OS: {} ({})'.format(os_name, os_version)
+
+        sdk_re = 'MacOSX[\s\S]*?SDKVersion:[ \t]*(.+?)\n'
+        is_ios_builder = 'iOS' in self.getProperty('buildername', '')
+        if is_ios_builder:
+            sdk_re = 'iPhoneSimulator[\s\S]*?SDKVersion:[ \t]*(.+?)\n'
+        match = re.search(sdk_re, logText)
+        if match:
+            xcode_version = match.group(1).strip()
+            configuration += u', Xcode: {}'.format(xcode_version)
+        return {u'step': configuration}

trunk/Tools/ChangeLog

@@ +1 @@
+2019-03-19  Aakash Jain  <aakash_jain@apple.com>
+
+        [ews-build] Improve summary for PrintConfiguration step
+        https://bugs.webkit.org/show_bug.cgi?id=195945
+
+        Reviewed by Lucas Forschler.
+
+        * BuildSlaveSupport/ews-build/steps.py:
+        (PrintConfiguration.getResultSummary): Override getResultSummary.
+
 2019-03-19  Aakash Jain  <aakash_jain@apple.com>