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
Fixes #14703: Let rtf execute tests from a given ncf PR #159
Fixes #14703: Let rtf execute tests from a given ncf PR #159
Conversation
rtf
Outdated
@@ -115,6 +115,7 @@ Commands | |||
Setup the given platform, install ncf from git in the specified version and then run the ncf tests. | |||
When no cfengine version is specified, tests will be executed with the cfengine agent shipped with the | |||
Rudder agent in the same version than the ncf version specified. | |||
You can also test a give ncf pull request by remplacing the version parameter by the redmine number, like #14641 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
beware, # in a command line is a comment, can't we have something else ?
scenario/lib.py
Outdated
try: | ||
if live_output: | ||
print("+" + command) | ||
if hostname == 'localhost': | ||
return check_output("LANG=C " + command, shell=True) | ||
return ("0", check_output("LANG=C " + command, shell=True)) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
i think the return code should be an integer
Commit modified |
3 similar comments
Commit modified |
Commit modified |
Commit modified |
PR rebased |
e352a0f
to
dcc5352
Compare
Commit modified |
@@ -22,6 +22,7 @@ Usage: | |||
rtf scenario doc <scenario> | |||
rtf scenario env <platform> | |||
rtf scenario ncf_tests <platform> <version> [--cfengine-version=<cfengine_version>] | |||
rtf scenario ncf_tests <platform> <pull-request-number> [--cfengine-version=<cfengine_version>] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this won't work
Commit modified |
OK, squash merging this PR |
cf9bad9
to
a8c5c35
Compare
https://issues.rudder.io/issues/14703