Attachment 368335 Details for Bug 197319 – Patch

[patch] Patch

bug-197319-20190426134651.patch (text/plain), 10.42 KB, created by Aakash Jain on 2019-04-26 10:46:52 PDT

(hide)

Description:

Filename:

MIME Type:

Creator: Aakash Jain

Created: 2019-04-26 10:46:52 PDT

Size: 10.42 KB

patch

obsolete

>Index: Tools/ChangeLog
>===================================================================
>--- Tools/ChangeLog	(revision 244698)
>+++ Tools/ChangeLog	(working copy)
>@@ -1,3 +1,20 @@
>+2019-04-26  Aakash Jain  <aakash_jain@apple.com>
>+
>+        [ews-build] Do not print worker environment variables in each build step
>+        https://bugs.webkit.org/show_bug.cgi?id=197319
>+
>+        Reviewed by NOBODY (OOPS!).
>+
>+        * BuildSlaveSupport/ews-build/steps.py:
>+        (CheckOutSource.__init__): Disabled logging of environment variables.
>+        (ApplyPatch.__init__): Ditto.
>+        (KillOldProcesses.__init__): Ditto.
>+        (ArchiveBuiltProduct.__init__): Ditto.
>+        (DownloadBuiltProduct.__init__): Ditto.
>+        (ExtractBuiltProduct.__init__): Ditto.
>+        (RunAPITests.__init__): Ditto.
>+        * BuildSlaveSupport/ews-build/steps_unittest.py: Updated unit-tests accordingly.
>+
> 2019-04-26  Alex Christensen  <achristensen@webkit.org>
> 
>         Add ENABLE(CONTENT_EXTENSIONS) and namespace ContentExtensions to ResourceLoadInfo.h
>Index: Tools/BuildSlaveSupport/ews-build/steps.py
>===================================================================
>--- Tools/BuildSlaveSupport/ews-build/steps.py	(revision 244696)
>+++ Tools/BuildSlaveSupport/ews-build/steps.py	(working copy)
>@@ -96,6 +96,7 @@ class CheckOutSource(git.Git):
>                                                 retry=self.CHECKOUT_DELAY_AND_MAX_RETRIES_PAIR,
>                                                 timeout=2 * 60 * 60,
>                                                 alwaysUseLatest=True,
>+                                                logEnviron=False,
>                                                 method='clean',
>                                                 progress=True,
>                                                 **kwargs)
>@@ -124,6 +125,9 @@ class ApplyPatch(shell.ShellCommand, Com
>     haltOnFailure = True
>     command = ['Tools/Scripts/svn-apply', '--force', '.buildbot-diff']
> 
>+    def __init__(self, **kwargs):
>+        super(ApplyPatch, self).__init__(timeout=5 * 60, logEnviron=False, **kwargs)
>+
>     def _get_patch(self):
>         sourcestamp = self.build.getSourceStamp(self.getProperty('codebase', ''))
>         if not sourcestamp or not sourcestamp.patch:
>@@ -636,7 +640,7 @@ class KillOldProcesses(shell.Compile):
>     command = ["python", "Tools/BuildSlaveSupport/kill-old-processes", "buildbot"]
> 
>     def __init__(self, **kwargs):
>-        super(KillOldProcesses, self).__init__(timeout=60, **kwargs)
>+        super(KillOldProcesses, self).__init__(timeout=60, logEnviron=False, **kwargs)
> 
> 
> class RunWebKitTests(shell.Test):
>@@ -680,6 +684,9 @@ class ArchiveBuiltProduct(shell.ShellCom
>     descriptionDone = ['Archived built product']
>     haltOnFailure = True
> 
>+    def __init__(self, **kwargs):
>+        super(ArchiveBuiltProduct, self).__init__(logEnviron=False, **kwargs)
>+
> 
> class UploadBuiltProduct(transfer.FileUpload):
>     name = 'upload-built-product'
>@@ -719,6 +726,9 @@ class DownloadBuiltProduct(shell.ShellCo
>     haltOnFailure = True
>     flunkOnFailure = True
> 
>+    def __init__(self, **kwargs):
>+        super(DownloadBuiltProduct, self).__init__(logEnviron=False, **kwargs)
>+
> 
> class ExtractBuiltProduct(shell.ShellCommand):
>     command = ['python', 'Tools/BuildSlaveSupport/built-product-archive',
>@@ -729,6 +739,9 @@ class ExtractBuiltProduct(shell.ShellCom
>     haltOnFailure = True
>     flunkOnFailure = True
> 
>+    def __init__(self, **kwargs):
>+        super(ExtractBuiltProduct, self).__init__(logEnviron=False, **kwargs)
>+
> 
> class RunAPITests(TestWithFailureCount):
>     name = 'run-api-tests'
>@@ -740,6 +753,9 @@ class RunAPITests(TestWithFailureCount):
>                WithProperties('--%(configuration)s'), '--verbose', '--json-output={0}'.format(jsonFileName)]
>     failedTestsFormatString = '%d api test%s failed or timed out'
> 
>+    def __init__(self, **kwargs):
>+        super(RunAPITests, self).__init__(logEnviron=False, **kwargs)
>+
>     def start(self):
>         appendCustomBuildFlags(self, self.getProperty('platform'), self.getProperty('fullPlatform'))
>         return TestWithFailureCount.start(self)
>Index: Tools/BuildSlaveSupport/ews-build/steps_unittest.py
>===================================================================
>--- Tools/BuildSlaveSupport/ews-build/steps_unittest.py	(revision 244696)
>+++ Tools/BuildSlaveSupport/ews-build/steps_unittest.py	(working copy)
>@@ -404,6 +404,7 @@ class TestKillOldProcesses(BuildStepMixi
>         self.expectRemoteCommands(
>             ExpectShell(workdir='wkdir',
>                         command=['python', 'Tools/BuildSlaveSupport/kill-old-processes', 'buildbot'],
>+                        logEnviron=False,
>                         timeout=60,
>                         )
>             + 0,
>@@ -416,6 +417,7 @@ class TestKillOldProcesses(BuildStepMixi
>         self.expectRemoteCommands(
>             ExpectShell(workdir='wkdir',
>                         command=['python', 'Tools/BuildSlaveSupport/kill-old-processes', 'buildbot'],
>+                        logEnviron=False,
>                         timeout=60,
>                         )
>             + ExpectShell.log('stdio', stdout='Unexpected error.')
>@@ -922,6 +924,7 @@ class TestArchiveBuiltProduct(BuildStepM
>         self.setProperty('configuration', 'release')
>         self.expectRemoteCommands(
>             ExpectShell(workdir='wkdir',
>+                        logEnviron=False,
>                         command=['python', 'Tools/BuildSlaveSupport/built-product-archive', '--platform=ios-simulator',  '--release', 'archive'],
>                         )
>             + 0,
>@@ -935,6 +938,7 @@ class TestArchiveBuiltProduct(BuildStepM
>         self.setProperty('configuration', 'debug')
>         self.expectRemoteCommands(
>             ExpectShell(workdir='wkdir',
>+                        logEnviron=False,
>                         command=['python', 'Tools/BuildSlaveSupport/built-product-archive', '--platform=mac-sierra',  '--debug', 'archive'],
>                         )
>             + ExpectShell.log('stdio', stdout='Unexpected failure.')
>@@ -1012,6 +1016,7 @@ class TestDownloadBuiltProduct(BuildStep
>         self.setProperty('patch_id', '1234')
>         self.expectRemoteCommands(
>             ExpectShell(workdir='wkdir',
>+                        logEnviron=False,
>                         command=['python', 'Tools/BuildSlaveSupport/download-built-product', '--platform=ios',  '--release', 'https://ews-build.webkit.org/archives/ios-simulator-12-x86_64-release/1234.zip'],
>                         )
>             + 0,
>@@ -1028,6 +1033,7 @@ class TestDownloadBuiltProduct(BuildStep
>         self.setProperty('patch_id', '123456')
>         self.expectRemoteCommands(
>             ExpectShell(workdir='wkdir',
>+                        logEnviron=False,
>                         command=['python', 'Tools/BuildSlaveSupport/download-built-product', '--platform=mac',  '--debug', 'https://ews-build.webkit.org/archives/mac-sierra-x86_64-debug/123456.zip'],
>                         )
>             + ExpectShell.log('stdio', stdout='Unexpected failure.')
>@@ -1051,6 +1057,7 @@ class TestExtractBuiltProduct(BuildStepM
>         self.setProperty('configuration', 'release')
>         self.expectRemoteCommands(
>             ExpectShell(workdir='wkdir',
>+                        logEnviron=False,
>                         command=['python', 'Tools/BuildSlaveSupport/built-product-archive', '--platform=ios-simulator',  '--release', 'extract'],
>                         )
>             + 0,
>@@ -1064,6 +1071,7 @@ class TestExtractBuiltProduct(BuildStepM
>         self.setProperty('configuration', 'debug')
>         self.expectRemoteCommands(
>             ExpectShell(workdir='wkdir',
>+                        logEnviron=False,
>                         command=['python', 'Tools/BuildSlaveSupport/built-product-archive', '--platform=mac-sierra',  '--debug', 'extract'],
>                         )
>             + ExpectShell.log('stdio', stdout='Unexpected failure.')
>@@ -1090,6 +1098,7 @@ class TestRunAPITests(BuildStepMixinAddi
> 
>         self.expectRemoteCommands(
>             ExpectShell(workdir='wkdir',
>+                        logEnviron=False,
>                         command=['python', 'Tools/Scripts/run-api-tests', '--no-build', '--release', '--verbose', '--json-output={0}'.format(self.jsonFileName)],
>                         logfiles={'json': self.jsonFileName},
>                         )
>@@ -1117,6 +1126,7 @@ All tests successfully passed!
> 
>         self.expectRemoteCommands(
>             ExpectShell(workdir='wkdir',
>+                        logEnviron=False,
>                         command=['python', 'Tools/Scripts/run-api-tests', '--no-build', '--debug', '--verbose', '--json-output={0}'.format(self.jsonFileName), '--ios-simulator'],
>                         logfiles={'json': self.jsonFileName},
>                         )
>@@ -1144,6 +1154,7 @@ All tests successfully passed!
> 
>         self.expectRemoteCommands(
>             ExpectShell(workdir='wkdir',
>+                        logEnviron=False,
>                         command=['python', 'Tools/Scripts/run-api-tests', '--no-build', '--debug', '--verbose', '--json-output={0}'.format(self.jsonFileName)],
>                         logfiles={'json': self.jsonFileName},
>                         )
>@@ -1185,6 +1196,7 @@ Testing completed, Exit status: 3
> 
>         self.expectRemoteCommands(
>             ExpectShell(workdir='wkdir',
>+                        logEnviron=False,
>                         command=['python', 'Tools/Scripts/run-api-tests', '--no-build', '--debug', '--verbose', '--json-output={0}'.format(self.jsonFileName)],
>                         logfiles={'json': self.jsonFileName},
>                         )
>@@ -1240,6 +1252,7 @@ Testing completed, Exit status: 3
> 
>         self.expectRemoteCommands(
>             ExpectShell(workdir='wkdir',
>+                        logEnviron=False,
>                         command=['python', 'Tools/Scripts/run-api-tests', '--no-build', '--debug', '--verbose', '--json-output={0}'.format(self.jsonFileName)],
>                         logfiles={'json': self.jsonFileName},
>                         )
>@@ -1257,6 +1270,7 @@ Testing completed, Exit status: 3
> 
>         self.expectRemoteCommands(
>             ExpectShell(workdir='wkdir',
>+                        logEnviron=False,
>                         command=['python', 'Tools/Scripts/run-api-tests', '--no-build', '--debug', '--verbose', '--json-output={0}'.format(self.jsonFileName)],
>                         logfiles={'json': self.jsonFileName},
>                         )

Actions: View | Formatted Diff | Diff

Attachments on bug 197319: 368335 | 372993