Alexey Serbin has posted comments on this change.

Change subject: [kudu-jepsen] fixed misprints in packaged mode
......................................................................


Patch Set 1:

(1 comment)

The regular Jenkins jobs overrides those properties -- it sets the names for 
nodes, etc.  Overall, the Jenkins job uses clojure:run maven target which uses 
separate run script: $KUDU_HOME/java/kudu-jepsen/src/utils/kudu_test_runner.clj.

However, there is another issue related to the 'packaged' kudu-jepsen runs: 
it's not possible to fetch those packages from outside of the internal Cloudera 
network (i.e. you either need to run that from the internal network or have 
Cloudera VPN enabled).  It would be nice to resolve that issue if we are about 
to push those changes into the Jepsen repo sometime.

http://gerrit.cloudera.org:8080/#/c/5769/1/java/kudu-jepsen/src/main/clojure/jepsen/kudu/util.clj
File java/kudu-jepsen/src/main/clojure/jepsen/kudu/util.clj:

PS1, Line 36: 1.2.0
> did you try this, there's no 1.2.0 version available in: http://repos.jenki
Yes, I tried that and it worked for me in Dockerized setup (jepsen repo).  The 
actual URL is 
http://repos.jenkins.cloudera.com/kudu1.2.0-nightly/debian/jessie/amd64/kudu/dists/
 -- it's set in line 39.

I put 1.2.0 because with 1.3.0 (or current) is that there is no regular scheme 
for URL and package versions: the URL does not contain the version but the 
'deb' line in should have that suffix.


-- 
To view, visit http://gerrit.cloudera.org:8080/5769
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I409f5fae9a65124afaeac8f6535daa9bba0aed01
Gerrit-PatchSet: 1
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: David Ribeiro Alves <[email protected]>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-HasComments: Yes

Reply via email to