Skip to content

t0040: test ipfs add with stdin - #1285

Merged
jbenet merged 1 commit into
masterfrom
t0040-stdin-add-tests
May 24, 2015
Merged

t0040: test ipfs add with stdin#1285
jbenet merged 1 commit into
masterfrom
t0040-stdin-add-tests

Conversation

@chriscool

Copy link
Copy Markdown
Contributor

Let's test 'ipfs add' with stdin input too.

License: MIT
Signed-off-by: Christian Couder chriscool@tuxfamily.org

Let's test 'ipfs add' with stdin input too.

License: MIT
Signed-off-by: Christian Couder <chriscool@tuxfamily.org>
@jbenet jbenet added the status/in-progress In progress label May 23, 2015
@chriscool

Copy link
Copy Markdown
Contributor Author

The errors are:

race: limit on 8192 simultaneously alive goroutines is exceeded, dying
FAIL    github.com/ipfs/go-ipfs/exchange/bitswap    4.227s
expecting success: 
    test_must_fail ipfs init 2> init_fail_out

initializing ipfs node at /workspace/ipfs_test_all_commits_on_branch/src/github.com/ipfs/go-ipfs/test/sharness/trash directory.t0020-init.sh/.badipfs
generating 2048-bit RSA keypair...done
peer identity: QmNUG47KsEkMVDcxVLceRmjTYYjn8WVBZRkmaxogHvG2TY
to get started, enter:

    ipfs cat /ipfs/Qmcqtw8FfrVSBaRmbWwHxt3AuySBhJLcvmFYi3Lbc4xnwj/readme

not ok 2 - ipfs init fails
#   
#       test_must_fail ipfs init 2> init_fail_out
#   
expecting success: 
    export IPFS_PATH="$IPTB_ROOT/3" &&
    NODE2_ID=`iptb get id 2` &&
    ipfs cat /ipns/$NODE2_ID > output
Error: context deadline exceeded
not ok 5 - cat that entry on a third node
#   
#       export IPFS_PATH="$IPTB_ROOT/3" &&
#       NODE2_ID=`iptb get id 2` &&
#       ipfs cat /ipns/$NODE2_ID > output
#   
expecting success: 
    export IPFS_PATH="$IPTB_ROOT/2" &&
    ipfs cat $FILEA_HASH >fileb
Error: context deadline exceeded
not ok 3 - cat that file on node2
#   
#       export IPFS_PATH="$IPTB_ROOT/2" &&
#       ipfs cat $FILEA_HASH >fileb
#   

@whyrusleeping

Copy link
Copy Markdown
Member

digging into the iptb failures, i'm beginning to think its a DHT issue. the keyspace randomly gets into a position where there are no nodes closer to a given key other than yourself, so you end up failing to do a get/put

@chriscool

Copy link
Copy Markdown
Contributor Author

@whyrusleeping anyway it is not related to the PR so please merge it.

@jbenet

jbenet commented May 24, 2015

Copy link
Copy Markdown
Member

@chriscool LGTM, thanks

jbenet added a commit that referenced this pull request May 24, 2015
@jbenet
jbenet merged commit dacdeb1 into master May 24, 2015
@jbenet
jbenet deleted the t0040-stdin-add-tests branch May 24, 2015 16:24
@jbenet jbenet removed the status/in-progress In progress label May 24, 2015
@wking wking mentioned this pull request Jun 12, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants