This is resolved by aa8a25592f442ab640708115e144e6b8d95cb664, give it a try!
C.
On Sun, Feb 6, 2011 at 10:02 PM, Colin McCabe <cmccabe@xxxxxxxxxxxxxx> wrote:
> On Sun, Feb 6, 2011 at 5:33 AM, DongJin Lee <dongjin.lee@xxxxxxxxxxxxxx> wrote:
>> Hi all:
>>
>> commit 09c38e8edadb3a5bf470ff9b0409172e0c61e012
>> About introducing the new tmp directory.
>>
>> TDIR=`mktemp -d -t mkcephfs.XXXXXXXXXX` || exit 1
>> trap "rm -rf ${TDIR}; exit" INT TERM EXIT
>>
>> I think this works if running all on the same nodes.
>> but other nodes don't know this directory name as they don't yet have
>> this newly (and randomly) created directory.
>
> Yes, the scp should be fixed. Good find.
>
>> Also, in the line 193, scp -q $conf $host:$conf
>> I don't think we check whether nodes have config directory, e.g., for
>> -c /etc/ceph/ceph.conf, we don't check /etc/ceph directory?
>> and so the other custom directory would also fail, too I think.
>
> I suppose we should do a mkdir -p before copying over the configuration file.
>
> cheers,
> Colin
>
--
To unsubscribe from this list: send the line "unsubscribe ceph-devel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
