[zorg] r357721 - [LLDB] Remove all abandoned LLDB bots

Reid Kleckner via llvm-commits llvm-commits at lists.llvm.org
Fri Apr 5 11:29:57 PDT 2019


It would, I just forgot, since I already noted that via Phab, but I do
think it's ultimately best to record the author in the commit message,
which is a more reliable system of record.

On Thu, Apr 4, 2019 at 3:24 PM Shoaib Meenai <smeenai at fb.com> wrote:

> Would it be customary to include the "Patch by Zach Turner" here?
>
> On 4/4/19, 1:27 PM, "llvm-commits on behalf of Reid Kleckner via
> llvm-commits" <llvm-commits-bounces at lists.llvm.org on behalf of
> llvm-commits at lists.llvm.org> wrote:
>
>     Author: rnk
>     Date: Thu Apr  4 13:28:43 2019
>     New Revision: 357721
>
>     URL:
> https://urldefense.proofpoint.com/v2/url?u=http-3A__llvm.org_viewvc_llvm-2Dproject-3Frev-3D357721-26view-3Drev&d=DwIGaQ&c=5VD0RTtNlTh3ycd41b3MUw&r=o3kDXzdBUE3ljQXKeTWOMw&m=QKCZC7_UHAA40H02VuKG43vTwhmf-JTRG8tKkyEWnsI&s=uKio4tY85gPY3WehKTNqyLjSByecM10H6HRd76uUZ6Q&e=
>     Log:
>     [LLDB] Remove all abandoned LLDB bots
>
>     Summary: All of these bots have been broken for months -- in some
> cases years.  Removing them from the waterfall.
>
>     Reviewers: gkistanova, labath, serge-sans-paille, stella.stamenova,
> srhines
>
>     Reviewed By: gkistanova, labath, stella.stamenova, srhines
>
>     Subscribers: javed.absar, jdoerfert, jankratochvil, srhines,
> stella.stamenova, echristo, llvm-commits
>
>     Differential Revision:
> https://urldefense.proofpoint.com/v2/url?u=https-3A__reviews.llvm.org_D57911&d=DwIGaQ&c=5VD0RTtNlTh3ycd41b3MUw&r=o3kDXzdBUE3ljQXKeTWOMw&m=QKCZC7_UHAA40H02VuKG43vTwhmf-JTRG8tKkyEWnsI&s=mv4jNvp4Mc5CU3wzaYHbfxIEKu3avcABBiUpFbkzkhg&e=
>
>     Removed:
>         zorg/trunk/zorg/buildbot/schedulers/LLDBTriggerable.py
>     Modified:
>         zorg/trunk/buildbot/osuosl/master/config/builders.py
>         zorg/trunk/buildbot/osuosl/master/config/slaves.py
>         zorg/trunk/buildbot/osuosl/master/config/status.py
>         zorg/trunk/buildbot/osuosl/master/master.cfg
>
>     Modified: zorg/trunk/buildbot/osuosl/master/config/builders.py
>     URL:
> https://urldefense.proofpoint.com/v2/url?u=http-3A__llvm.org_viewvc_llvm-2Dproject_zorg_trunk_buildbot_osuosl_master_config_builders.py-3Frev-3D357721-26r1-3D357720-26r2-3D357721-26view-3Ddiff&d=DwIGaQ&c=5VD0RTtNlTh3ycd41b3MUw&r=o3kDXzdBUE3ljQXKeTWOMw&m=QKCZC7_UHAA40H02VuKG43vTwhmf-JTRG8tKkyEWnsI&s=PhhUY1pBwmhg1cF2XIR1vVtufBtiZ4furVu0EMzvuno&e=
>
> ==============================================================================
>     --- zorg/trunk/buildbot/osuosl/master/config/builders.py (original)
>     +++ zorg/trunk/buildbot/osuosl/master/config/builders.py Thu Apr  4
> 13:28:43 2019
>     @@ -937,39 +937,6 @@ def _get_rev_iter_builders():
>      # LLDB builders.
>      def _get_lldb_builders():
>          return [
>     -        {'name': "lldb-x86-windows-msvc2015",
>     -         'slavenames': ["zturner-win2008"],
>     -         'builddir': "lldb-windows-x86",
>     -         'factory': LLDBBuilder.getLLDBCMakeBuildFactory(
>     -                    test=False,
>     -                    python_source_dir=r'C:/Python35',
>     -                    vs=r"""%VS140COMNTOOLS%""")},
>     -        # Disable the builder till we fix the cmake configuration
>     -        #{'name': "lldb-x86-win7-msvc",
>     -        #'slavenames': ["hexagon-build-01"],
>     -        #'builddir': "builddir/lldb-win7-msvc",
>     -        #'factory': LLDBBuilder.getLLDBCMakeBuildFactory(
>     -        #           config='Debug',
>     -        #           python_source_dir=r'C:/Python35',
>     -        #           vs=r"""%VS140COMNTOOLS%""")},
>     -        {'name': "lldb-x86_64-ubuntu-14.04-buildserver",
>     -         'slavenames': ["lldb-linux-android-buildserver"],
>     -         'builddir': "lldb-android-buildserver",
>     -         'category' : 'lldb',
>     -         'factory': LLDBBuilder.getLLDBScriptCommandsFactory(
>     -                    downloadBinary=False,
>     -                    buildAndroid=True,
>     -                    runTest=False,
>     -
> extra_cmake_args=['-DLLVM_ENABLE_ASSERTIONS=ON'])},
>     -        {'name': "lldb-x86_64-ubuntu-14.04-cmake",
>     -         'slavenames': ["lldb-build1-ubuntu-1404"],
>     -         'builddir': "buildWorkingDir",
>     -         'category' : 'lldb',
>     -         'factory': LLDBBuilder.getLLDBScriptCommandsFactory(
>     -                    downloadBinary=False,
>     -                    buildAndroid=False,
>     -                    runTest=True,
>     -
> extra_cmake_args=['-DLLVM_ENABLE_ASSERTIONS=ON'])},
>              {'name': "lldb-amd64-ninja-freebsd11",
>               'slavenames': ["lldb-amd64-ninja-freebsd11"],
>               'builddir': "scratch",
>     @@ -1599,34 +1566,6 @@ def _get_experimental_scheduled_builders
>                           enable_thrust_tests=False,
>               ),
>               'category' : 'clang'},
>     -
>     -        # lldb builders
>     -        {'name': "lldb-x86_64-darwin-13.4",
>     -         'slavenames': ["lldb-x86_64-darwin-13.4"],
>     -         'builddir': "buildDir",
>     -         'category' : 'lldb',
>     -         'factory': LLDBBuilder.getLLDBScriptCommandsFactory(
>     -                    downloadBinary=True,
>     -                    buildAndroid=False,
>     -                    runTest=True)},
>     -        {'name': "lldb-x86_64-ubuntu-14.04-android",
>     -         'slavenames': ["lldb-lab-linux01"],
>     -         'builddir': "lldb-cross-compile",
>     -         'category' : 'lldb',
>     -         'factory': LLDBBuilder.getLLDBScriptCommandsFactory(
>     -                    downloadBinary=True,
>     -                    buildAndroid=False,
>     -                    runTest=True)},
>     -        {'name': "lldb-windows7-android",
>     -         'slavenames': ["lldb-lab-win01"],
>     -         'builddir': "lldb-win7-android",
>     -         'category': "lldb",
>     -         'factory': LLDBBuilder.getLLDBScriptCommandsFactory(
>     -                    downloadBinary=True,
>     -                    buildAndroid=False,
>     -                    runTest=True,
>     -                    scriptExt='.bat')},
>     -
>              ]
>
>      # Builders responsible building Sphinix documentation
>
>     Modified: zorg/trunk/buildbot/osuosl/master/config/slaves.py
>     URL:
> https://urldefense.proofpoint.com/v2/url?u=http-3A__llvm.org_viewvc_llvm-2Dproject_zorg_trunk_buildbot_osuosl_master_config_slaves.py-3Frev-3D357721-26r1-3D357720-26r2-3D357721-26view-3Ddiff&d=DwIGaQ&c=5VD0RTtNlTh3ycd41b3MUw&r=o3kDXzdBUE3ljQXKeTWOMw&m=QKCZC7_UHAA40H02VuKG43vTwhmf-JTRG8tKkyEWnsI&s=kxK58MEOPVi16G3j0EJlKzwZu-Ihhuorr8bE75LyZcI&e=
>
> ==============================================================================
>     --- zorg/trunk/buildbot/osuosl/master/config/slaves.py (original)
>     +++ zorg/trunk/buildbot/osuosl/master/config/slaves.py Thu Apr  4
> 13:28:43 2019
>     @@ -189,8 +189,6 @@ def get_build_slaves():
>              create_slave('libcxx-cloud5', properties={'jobs': 64},
> max_builds=1),
>
>
>     -        # Windows Server 2008 R2, Quad 2.6GHz Intel Xeon(R) 4GB RAM
>     -        create_slave("zturner-win2008", properties={'jobs': 4},
> max_builds=1),
>              # Windows Server 2012 x86_64 16-core GCE instance
>              create_slave("sanitizer-windows", properties={'jobs': 16},
> max_builds=1),
>              create_slave("windows-gcebot1", properties={'jobs': 8},
> max_builds=1),
>     @@ -199,17 +197,6 @@ def get_build_slaves():
>              # Windows Server 2016 x86_64 16-core GCE instance
>              create_slave("windows-lld-thinlto-1", max_builds=1),
>
>     -        # Ubuntu x86-64, 51GiB System memory Intel(R) Xeon(R) CPU @
> 2.60GHz
>     -        create_slave("lldb-build1-ubuntu-1404", properties={'jobs':
> 32, 'loadaverage':32}, max_builds=1),
>     -        # Darwin 13.4.0, x86_64
>     -        create_slave("lldb-x86_64-darwin-13.4", properties={'jobs':
> 16, 'loadaverage':32}, max_builds=1),
>     -        # Ubuntu 14.04 x86_64, Intel(R) Xeon(R) CPU E5-2680 v2 @
> 2.80GHz
>     -        create_slave("lldb-lab-linux01", properties={'jobs': 40},
> max_builds=1),
>     -
>     -        # Windows 7 x86_64
>     -        create_slave("lldb-lab-win01", properties={'jobs': 40},
> max_builds=1),
>     -        # Ubuntu 14.04 x86_64
>     -        create_slave("lldb-linux-android-buildserver",
> properties={'jobs': 40}, max_builds=1),
>              # Ubuntu 14.04 x86_64, Intel(R) Xeon(R) CPU L5520 @ 2.27GHz
>              create_slave("bpf-build-slave01", properties={'jobs': 16},
> max_builds=1),
>
>
>     Modified: zorg/trunk/buildbot/osuosl/master/config/status.py
>     URL:
> https://urldefense.proofpoint.com/v2/url?u=http-3A__llvm.org_viewvc_llvm-2Dproject_zorg_trunk_buildbot_osuosl_master_config_status.py-3Frev-3D357721-26r1-3D357720-26r2-3D357721-26view-3Ddiff&d=DwIGaQ&c=5VD0RTtNlTh3ycd41b3MUw&r=o3kDXzdBUE3ljQXKeTWOMw&m=QKCZC7_UHAA40H02VuKG43vTwhmf-JTRG8tKkyEWnsI&s=1tQnrwUKGzZd0G4fsuxUF1JVqJMe41d7ndwSVTBPV-M&e=
>
> ==============================================================================
>     --- zorg/trunk/buildbot/osuosl/master/config/status.py (original)
>     +++ zorg/trunk/buildbot/osuosl/master/config/status.py Thu Apr  4
> 13:28:43 2019
>     @@ -145,20 +145,6 @@ def get_status_targets(standard_builders
>              InformativeMailNotifier(
>                  fromaddr = "llvm.buildmaster at lab.llvm.org",
>                  sendToInterestedUsers= False,
>     -            extraRecipients = ["
> lldb-build-police-gmt at grotations.appspotmail.com",
>     -                               "
> lldb-build-police-pst at grotations.appspotmail.com"],
>     -            subject="Build %(builder)s Failure",
>     -            mode = "failing",
>     -            builders = ["lldb-x86_64-ubuntu-14.04-cmake",
>     -                        "lldb-x86_64-ubuntu-14.04-buildserver",
>     -                        "lldb-windows7-android",
>     -                        "lldb-x86_64-darwin-13.4",
>     -                        "lldb-x86_64-ubuntu-14.04-android"],
>     -            addLogs=False,
>     -            num_lines = 15),
>     -        InformativeMailNotifier(
>     -            fromaddr = "llvm.buildmaster at lab.llvm.org",
>     -            sendToInterestedUsers= False,
>                  extraRecipients = ["sunil_srivastava at playstation.sony.com
> ",
>                                     "warren_ristow at playstation.sony.com"],
>                  subject="Build %(builder)s Failure",
>
>     Modified: zorg/trunk/buildbot/osuosl/master/master.cfg
>     URL:
> https://urldefense.proofpoint.com/v2/url?u=http-3A__llvm.org_viewvc_llvm-2Dproject_zorg_trunk_buildbot_osuosl_master_master.cfg-3Frev-3D357721-26r1-3D357720-26r2-3D357721-26view-3Ddiff&d=DwIGaQ&c=5VD0RTtNlTh3ycd41b3MUw&r=o3kDXzdBUE3ljQXKeTWOMw&m=QKCZC7_UHAA40H02VuKG43vTwhmf-JTRG8tKkyEWnsI&s=cSDcZll5f_Vq9-GIRq4ERn0426qRzMwfhdmGhgcYITI&e=
>
> ==============================================================================
>     --- zorg/trunk/buildbot/osuosl/master/master.cfg (original)
>     +++ zorg/trunk/buildbot/osuosl/master/master.cfg Thu Apr  4 13:28:43
> 2019
>     @@ -178,19 +178,9 @@ c['schedulers'].append(SingleBranchSched
>                                      "llvm",
>                                      "lld"])))
>
>     -# lldb android builders will be triggered by completion of
> buildserver instead of by source change
>     -lldb_android_builders=["lldb-x86_64-ubuntu-14.04-android",
>     -                       "lldb-windows7-android",
>     -                       "lldb-x86_64-darwin-13.4"]
>     -from zorg.buildbot.schedulers.LLDBTriggerable import LLDBTriggerable
>     -c['schedulers'].append((LLDBTriggerable(
>     -                                  projects=['lldb', 'cfe', 'llvm'],
>     -                                  name="lldb_android_scheduler",
>     -
> builderNames=lldb_android_builders)))
>     -
>      c['schedulers'].append(SingleBranchScheduler(name="lldb_scheduler",
>                                  treeStableTimer=2*60,
>     -
> builderNames=list(set(get_all_for("lldb"))-set(lldb_android_builders)),
>     +                            builderNames=get_all_for("lldb"),
>                                  change_filter=depends_on([
>                                      "llvm",
>                                      "cfe",
>
>     Removed: zorg/trunk/zorg/buildbot/schedulers/LLDBTriggerable.py
>     URL:
> https://urldefense.proofpoint.com/v2/url?u=http-3A__llvm.org_viewvc_llvm-2Dproject_zorg_trunk_zorg_buildbot_schedulers_LLDBTriggerable.py-3Frev-3D357720-26view-3Dauto&d=DwIGaQ&c=5VD0RTtNlTh3ycd41b3MUw&r=o3kDXzdBUE3ljQXKeTWOMw&m=QKCZC7_UHAA40H02VuKG43vTwhmf-JTRG8tKkyEWnsI&s=Uk2VBbNIrxzc4n3VnaZp4TPKR7bpWwruUdO985XLZuc&e=
>
> ==============================================================================
>     --- zorg/trunk/zorg/buildbot/schedulers/LLDBTriggerable.py (original)
>     +++ zorg/trunk/zorg/buildbot/schedulers/LLDBTriggerable.py (removed)
>     @@ -1,84 +0,0 @@
>     -from twisted.python import log
>     -from twisted.internet import defer
>     -from buildbot.schedulers.triggerable import Triggerable
>     -from buildbot.process.properties import Properties
>     -
>     -class LLDBTriggerable(Triggerable):
>     -    """
>     -    This is the scheduler used for lldb android builders,
>     -    Overwrite trigger function, so the triggered builder will
>     -    get changelist based on new changes since their last build.
>     -    The origianl implementation takes changelist from upstream
>     -    triggerer builder, this will be inaccurate in the case that
>     -    some builds in upstream didn't trigger new builds due to
>     -    failure in their early steps.
>     -    """
>     -
>     -    def __init__(self, projects, maxChange=100, **kwargs):
>     -        Triggerable.__init__(self, **kwargs)
>     -        self.projects = projects
>     -        self.lastRevision = None
>     -        self.maxChange = maxChange
>     -
>     -    def trigger(self, ssid, set_props=None):
>     -        """Trigger this scheduler with the given sourcestamp ID.
> Returns a
>     -        deferred that will fire when the buildset is finished."""
>     -        # properties for this buildset are composed of our own
> properties,
>     -        # potentially overridden by anything from the triggering build
>     -        props = Properties()
>     -        props.updateFromProperties(self.properties)
>     -        if set_props:
>     -            props.updateFromProperties(set_props)
>     -
>     -        newRevision = [None]
>     -        def getRevision(ss):
>     -            newRevision[0] = ss['revision']
>     -            return ss['revision']
>     -
>     -        def getRecentChanges(newRev):
>     -            if self.lastRevision is None:
>     -                return None
>     -            return
> self.master.db.changes.getRecentChanges(self.maxChange)
>     -
>     -        # check the last x changeset and pick up the ones that are
> between
>     -        # last revision and current revision and belong to interested
> projects
>     -        def selectChangeSet(changes):
>     -            changeids = []
>     -            if changes is not None:
>     -                for change in changes:
>     -                    if change['revision'] > newRevision[0] or
> change['revision'] <= self.lastRevision or change['project'] not in
> self.projects:
>     -                        continue
>     -                    changeids.append(change['changeid'])
>     -            log.msg("LLDBTriggerable: last revision change from %s to
> %s" % (self.lastRevision, newRevision[0]))
>     -            self.lastRevision = newRevision[0]
>     -            return changeids
>     -
>     -        def addBuildset(changeids):
>     -            if changeids:
>     -                log.msg("LLDBTriggerable: addBuildsetForChanges,
> changeids: %s" % changeids)
>     -                return self.addBuildsetForChanges(reason=self.reason,
> changeids=changeids, properties=props)
>     -            elif ssid:
>     -                # if this is the first build after master startup,
> use the source stamp from triggerer build
>     -                # it's possible to write last revision to a file on
> master, so after master reconfig we could
>     -                # pick up the correct last revision.
>     -                # It's not implemented here because 1) the cases are
> rare that first build after master restart
>     -                # is preceded by failing builds on triggerer builder,
> 2) avoid polluting master with project
>     -                # specific cache files
>     -                log.msg("LLDBTriggerable: addBuildsetForSourceStamp")
>     -                return
> self.addBuildsetForSourceStamp(reason=self.reason, ssid=ssid,
> properties=props)
>     -            else:
>     -                return self.addBuildsetForLatest(reason=self.reason,
> properties=props)
>     -
>     -        def setup_waiter((bsid,brids)):
>     -            self._waiters[bsid] = d = defer.Deferred()
>     -            self._updateWaiters()
>     -            return d
>     -
>     -        d = self.master.db.sourcestamps.getSourceStamp(ssid)
>     -        d.addCallback(getRevision)
>     -        d.addCallback(getRecentChanges)
>     -        d.addCallback(selectChangeSet)
>     -        d.addCallback(addBuildset)
>     -        d.addCallback(setup_waiter)
>     -        return d
>     -
>
>
>     _______________________________________________
>     llvm-commits mailing list
>     llvm-commits at lists.llvm.org
>
> https://urldefense.proofpoint.com/v2/url?u=https-3A__lists.llvm.org_cgi-2Dbin_mailman_listinfo_llvm-2Dcommits&d=DwIGaQ&c=5VD0RTtNlTh3ycd41b3MUw&r=o3kDXzdBUE3ljQXKeTWOMw&m=QKCZC7_UHAA40H02VuKG43vTwhmf-JTRG8tKkyEWnsI&s=lCRlHLiqh3vBe_lke6GvRECfCUHLj5t4hoSwgodpFTI&e=
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20190405/bd009745/attachment.html>


More information about the llvm-commits mailing list