Bug #18393

Updated by Nathan Cutler about 6 years ago

objecter_requests workunit takes an argument "test" specifying a download URL, e.g:

test: ";a=blob_plain;f=src/test/admin_socket/objecter_requests;hb={branch}"

Since ceph-ci.git has been split off from ceph.git, that download will fail for wip branches since those are found only in ceph-ci.git.

This can be seen in the wild, e.g., here: (pertinent log excerpt follows)

2017-01-02T15:58:06.733 'sudo adjust-ulimits ceph-coverage /home/ubuntu/cephtest/archive/coverage timeout 30 ceph --admin-daemon /var/run/ceph/ceph-osd.5.asok dump_blocked_ops'
2017-01-02T15:58:06.741 DEBUG:tasks.admin_socket:Testing objecter_requests with config {'test': ';a=blob_plain;f=src/test/admin_socket/objecter_requests;hb={branch}', 'branch': 'wip-hammer-backports'}
2017-01-02T15:58:06.744 "wget -q -O /home/ubuntu/cephtest/admin_socket_client.0/objecter_requests -- ';a=blob_plain;f=src/test/admin_socket/objecter_requests;hb=wip-hammer-backports' && chmod u=rx -- /home/ubuntu/cephtest/admin_socket_client.0/objecter_requests"
CommandFailedError: Command failed on smithi056 with status 8: "wget -q -O /home/ubuntu/cephtest/admin_socket_client.0/objecter_requests -- ';a=blob_plain;f=src/test/admin_socket/objecter_requests;hb=wip-hammer-backports' && chmod u=rx -- /home/ubuntu/cephtest/admin_socket_client.0/objecter_requests"

Following the ceph.git/ceph-ci.git split, this URL is unreliable:

test: "{repo};a=blob_plain;f=src/test/admin_socket/objecter_requests;hb={branch}"

Some branches are *only* in ceph.git, others *only* in ceph-ci.git. In this case, the branch is in ceph-ci.git, but "repo" evaluates to ceph.git.

I guess it will be rare for someone's wip branch to include changes to src/test/admin_socket/objecter_requests so we can just download from the appropriate branch of ceph/ceph.git. If someone needed to download the test from their wip branch, they could override the URL when running the test.