381
72%

Ran 07 Mar 2019 02:26PM UTC

Jobs 1

Files 494

Run time 52s

Badge

Embed ▾

pending completion

Build # 381

Build Type

push

jenkins

Committed by GitHub

Commit Message

Add validateConf to AbstractDNSToSwitchMapping

Descriptions of the changes in this PR:

- when setConf of AbstractDNSToSwitchMapping  is called it
should do sanity checking of the conf/env. and throw
RuntimeException if things are not valid.
- For RawScriptBasedMapping.validateConf, try executing
the script with no arguments for sanity check purpose.
Here it is expected that running script with no arguments
would do sanity check of the script and the env.

(there are 2 commits in this PR, but this PR is meant for the second commit and there is other PR for the first commit)

Reviewers: Sijie Guo <sijie@apache.org>

This closes #1965 from reddycharan/sanitycheckmappingconf

Run Details

26554 of 36608 relevant lines covered (72.54%)

0.73 hits per line

Jobs

ID	Job ID	Ran	Files	Coverage
1	381.1	07 Mar 2019 02:26PM UTC	0	72.54

Source Files on build 381

Detailed source file information is not available for this build.

apache / bookkeeper / 381
72%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 381

apache / bookkeeper / 381 72%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 381

apache / bookkeeper / 381
72%

README BADGES
x