Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use yaml for buildbot steps #325

Merged
merged 12 commits into from Apr 20, 2016

Re-add pkill step

This automatically runs pkill for each build. It is cleaner to put this
in the main configuration instead of the lists of steps because this is
a CI-related cleanup task, not a necessary step to run the build + test
suite. Also, this ensures that the pkill step cannot be forgotten when
adding new builders.

Note: this doesn't work on Windows yet.
  • Loading branch information
aneeshusa committed Apr 20, 2016
commit d8955508eaf94aa235f682d08f78f6708b58d82e
@@ -2,6 +2,7 @@

from buildbot.plugins import steps, util
from buildbot.process import buildstep
from buildbot.status.results import SUCCESS

import environments as envs

@@ -58,9 +59,13 @@ def __init__(self, environment, commands):
print(str(e))
dynamic_steps = [BadConfigurationStep(e)]

# TODO: windows compatibility (use a custom script for this?)
pkill_step = [steps.ShellCommand(command=["pkill", "-x", "servo"],
decodeRC={0: SUCCESS, 1: SUCCESS})]

# util.BuildFactory is an old-style class so we cannot use super()
# but must hardcode the superclass here
ServoFactory.__init__(self, dynamic_steps)
ServoFactory.__init__(self, pkill_step + dynamic_steps)

def make_step(self, command):
step_kwargs = {}
ProTip! Use n and p to navigate between commits in a pull request.
You can’t perform that action at this time.