Changeset 247069 in webkit


Ignore:
Timestamp:
Jul 2, 2019 12:51:50 PM (5 years ago)
Author:
aakash_jain@apple.com
Message:

[ews-build] Do not print worker environment variables in each build step [part 2]
​https://bugs.webkit.org/show_bug.cgi?id=199405

Reviewed by Jonathan Bedard.

  • BuildSlaveSupport/ews-build/steps.py:

(CheckOutSpecificRevision.init): Disabled logging of environment variables.
(CleanWorkingDirectory.init): Ditto.
(RunBindingsTests.init): Ditto.
(RunWebKitPerlTests.init): Ditto.
(RunWebKitPyTests.init): Ditto.
(CompileWebKit.init): Ditto.
(ArchiveTestResults.init): Ditto.

  • BuildSlaveSupport/ews-build/steps_unittest.py: Updated unit-tests.
Location:
trunk/Tools
Files:
3 edited

Legend:

Unmodified
Added
Removed

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

    r247049 r247069  
    115115    haltOnFailure = False
    116116
     117    def __init__(self, **kwargs):
     118        super(CheckOutSpecificRevision, self).__init__(logEnviron=False, **kwargs)
     119
    117120    def doStepIf(self, step):
    118121        return self.getProperty('ews_revision', False)
    … …  
    133136    haltOnFailure = True
    134137    command = ['Tools/Scripts/clean-webkit']
     138
     139    def __init__(self, **kwargs):
     140        super(CleanWorkingDirectory, self).__init__(logEnviron=False, **kwargs)
    135141
    136142
    … …  
    495501
    496502    def __init__(self, **kwargs):
    497         super(RunBindingsTests, self).__init__(timeout=5 * 60, **kwargs)
     503        super(RunBindingsTests, self).__init__(timeout=5 * 60, logEnviron=False, **kwargs)
    498504
    499505    def start(self):
    … …  
    542548
    543549    def __init__(self, **kwargs):
    544         super(RunWebKitPerlTests, self).__init__(timeout=2 * 60, **kwargs)
     550        super(RunWebKitPerlTests, self).__init__(timeout=2 * 60, logEnviron=False, **kwargs)
    545551
    546552
    … …  
    555561
    556562    def __init__(self, **kwargs):
    557         super(RunWebKitPyTests, self).__init__(timeout=2 * 60, **kwargs)
     563        super(RunWebKitPyTests, self).__init__(timeout=2 * 60, logEnviron=False, **kwargs)
    558564
    559565    def start(self):
    … …  
    613619    haltOnFailure = False
    614620    command = ['perl', 'Tools/Scripts/build-webkit', WithProperties('--%(configuration)s')]
     621
     622    def __init__(self, **kwargs):
     623        super(CompileWebKit, self).__init__(logEnviron=False, **kwargs)
    615624
    616625    def start(self):
    … …  
    11201129    haltOnFailure = True
    11211130
     1131    def __init__(self, **kwargs):
     1132        super(ArchiveTestResults, self).__init__(logEnviron=False, **kwargs)
     1133
    11221134
    11231135class UploadTestResults(transfer.FileUpload):

  • trunk/Tools/BuildSlaveSupport/ews-build/steps_unittest.py

    r247049 r247069  
    303303            ExpectShell(workdir='wkdir',
    304304                        timeout=300,
     305                        logEnviron=False,
    305306                        command=['Tools/Scripts/run-bindings-tests', '--json-output={0}'.format(self.jsonFileName)],
    306307                        logfiles={'json': self.jsonFileName},
    … …  
    316317            ExpectShell(workdir='wkdir',
    317318                        timeout=300,
     319                        logEnviron=False,
    318320                        command=['Tools/Scripts/run-bindings-tests', '--json-output={0}'.format(self.jsonFileName)],
    319321                        logfiles={'json': self.jsonFileName},
    … …  
    338340        self.expectRemoteCommands(
    339341            ExpectShell(workdir='wkdir',
     342                        logEnviron=False,
    340343                        command=['Tools/Scripts/test-webkitperl'],
    341344                        timeout=120,
    … …  
    350353        self.expectRemoteCommands(
    351354            ExpectShell(workdir='wkdir',
     355                        logEnviron=False,
    352356                        command=['Tools/Scripts/test-webkitperl'],
    353357                        timeout=120,
    … …  
    376380        self.expectRemoteCommands(
    377381            ExpectShell(workdir='wkdir',
     382                        logEnviron=False,
    378383                        command=['Tools/Scripts/test-webkitpy', '--json-output={0}'.format(self.jsonFileName)],
    379384                        logfiles={'json': self.jsonFileName},
    … …  
    389394        self.expectRemoteCommands(
    390395            ExpectShell(workdir='wkdir',
     396                        logEnviron=False,
    391397                        command=['Tools/Scripts/test-webkitpy', '--json-output={0}'.format(self.jsonFileName)],
    392398                        logfiles={'json': self.jsonFileName},
    … …  
    487493        self.expectRemoteCommands(
    488494            ExpectShell(workdir='wkdir',
     495                        logEnviron=False,
    489496                        command=['perl', 'Tools/Scripts/build-webkit', '--release'],
    490497                        )
    … …  
    500507        self.expectRemoteCommands(
    501508            ExpectShell(workdir='wkdir',
     509                        logEnviron=False,
    502510                        command=['perl', 'Tools/Scripts/build-webkit', '--debug'],
    503511                        )
    … …  
    524532        self.expectRemoteCommands(
    525533            ExpectShell(workdir='wkdir',
     534                        logEnviron=False,
    526535                        command=['perl', 'Tools/Scripts/build-webkit', '--release'],
    527536                        )
    … …  
    538547        self.expectRemoteCommands(
    539548            ExpectShell(workdir='wkdir',
     549                        logEnviron=False,
    540550                        command=['perl', 'Tools/Scripts/build-webkit', '--debug'],
    541551                        )
    … …  
    596606        self.expectRemoteCommands(
    597607            ExpectShell(workdir='wkdir',
     608                        logEnviron=False,
    598609                        command=['perl', 'Tools/Scripts/build-jsc', '--release'],
    599610                        )
    … …  
    609620        self.expectRemoteCommands(
    610621            ExpectShell(workdir='wkdir',
     622                        logEnviron=False,
    611623                        command=['perl', 'Tools/Scripts/build-jsc', '--debug'],
    612624                        )
    … …  
    633645        self.expectRemoteCommands(
    634646            ExpectShell(workdir='wkdir',
     647                        logEnviron=False,
    635648                        command=['perl', 'Tools/Scripts/build-jsc', '--release'],
    636649                        )
    … …  
    647660        self.expectRemoteCommands(
    648661            ExpectShell(workdir='wkdir',
     662                        logEnviron=False,
    649663                        command=['perl', 'Tools/Scripts/build-jsc', '--debug'],
    650664                        )
    … …  
    888902            ExpectShell(workdir='wkdir',
    889903                        timeout=1200,
     904                        logEnviron=False,
    890905                        command=['git', 'checkout', '1a3425cb92dbcbca12a10aa9514f1b77c76dc26'],
    891906                        )
    … …  
    902917            ExpectShell(workdir='wkdir',
    903918                        timeout=1200,
     919                        logEnviron=False,
    904920                        command=['git', 'checkout', '1a3425cb92dbcbca12a10aa9514f1b77c76dc26'],
    905921                        )
    … …  
    929945        self.expectRemoteCommands(
    930946            ExpectShell(workdir='wkdir',
     947                        logEnviron=False,
    931948                        command=['Tools/Scripts/clean-webkit'],
    932949                        )
    … …  
    940957        self.expectRemoteCommands(
    941958            ExpectShell(workdir='wkdir',
     959                        logEnviron=False,
    942960                        command=['Tools/Scripts/clean-webkit'],
    943961                        )
    … …  
    963981        self.expectRemoteCommands(
    964982            ExpectShell(workdir='wkdir',
     983                        logEnviron=False,
    965984                        command=['Tools/Scripts/clean-webkit'],
    966985                        )
    … …  
    976995        self.expectRemoteCommands(
    977996            ExpectShell(workdir='wkdir',
     997                        logEnviron=False,
    978998                        command=['Tools/Scripts/clean-webkit'],
    979999                        )
    … …  
    14311451        self.expectRemoteCommands(
    14321452            ExpectShell(workdir='wkdir',
     1453                        logEnviron=False,
    14331454                        command=['python', 'Tools/BuildSlaveSupport/test-result-archive', '--platform=ios-simulator',  '--release', 'archive'],
    14341455                        )
    … …  
    14451466        self.expectRemoteCommands(
    14461467            ExpectShell(workdir='wkdir',
     1468                        logEnviron=False,
    14471469                        command=['python', 'Tools/BuildSlaveSupport/test-result-archive', '--platform=mac',  '--debug', 'archive'],
    14481470                        )

  • trunk/Tools/ChangeLog

    r247059 r247069  
     12019-07-02  Aakash Jain  <aakash_jain@apple.com>
     2
     3        [ews-build] Do not print worker environment variables in each build step [part 2]
     4        https://bugs.webkit.org/show_bug.cgi?id=199405
     5
     6        Reviewed by Jonathan Bedard.
     7
     8        * BuildSlaveSupport/ews-build/steps.py:
     9        (CheckOutSpecificRevision.__init__): Disabled logging of environment variables.
     10        (CleanWorkingDirectory.__init__): Ditto.
     11        (RunBindingsTests.__init__): Ditto.
     12        (RunWebKitPerlTests.__init__): Ditto.
     13        (RunWebKitPyTests.__init__): Ditto.
     14        (CompileWebKit.__init__): Ditto.
     15        (ArchiveTestResults.__init__): Ditto.
     16        * BuildSlaveSupport/ews-build/steps_unittest.py: Updated unit-tests.
     17
    1182019-07-02  Jer Noble  <jer.noble@apple.com>
    219
Note: See TracChangeset for help on using the changeset viewer.