go-libp2p icon indicating copy to clipboard operation
go-libp2p copied to clipboard

identify: flaky TestLargeIdentifyMessage

Open marten-seemann opened this issue 3 years ago • 3 comments

  === RUN   TestLargeIdentifyMessage
      id_test.go:845: test peer1 has peer2 addrs correctly
      id_test.go:860: test peer2 has peer1 addrs correctly
      id_test.go:873: testing addrs just after disconnect
      id_test.go:876: 
          	Error Trace:	id_test.go:41
          	            				id_test.go:876
          	Error:      	elements differ
          	            	
          	            	extra elements in list A:
          	            	([]interface {}) (len=2) {
          	            	 (*multiaddr.multiaddr)({
          	            	  bytes: ([]uint8) (len=8) {
          	            	   00000000  04 7f 00 00 01 06 c3 e4                           |........|
          	            	  }
          	            	 }),
          	            	 (*multiaddr.multiaddr)({
          	            	  bytes: ([]uint8) (len=11) {
          	            	   00000000  04 7f 00 00 01 91 02 ed  e9 cc 03                 |...........|
          	            	  }
          	            	 })
          	            	}
          	            	
          	            	
          	            	listA:
          	            	([]multiaddr.Multiaddr) (len=2) {
          	            	 (*multiaddr.multiaddr)({
          	            	  bytes: ([]uint8) (len=8) {
          	            	   00000000  04 7f 00 00 01 06 c3 e4                           |........|
          	            	  }
          	            	 }),
          	            	 (*multiaddr.multiaddr)({
          	            	  bytes: ([]uint8) (len=11) {
          	            	   00000000  04 7f 00 00 01 91 02 ed  e9 cc 03                 |...........|
          	            	  }
          	            	 })
          	            	}
          	            	
          	            	
          	            	listB:
          	            	([]multiaddr.Multiaddr) {
          	            	}
          	Test:       	TestLargeIdentifyMessage
          	Messages:   	QmTYuoLmQHu34jyaN9ywjFA7r1bQLaapYZTddoRupT1ENg did not have addr for QmQ81pQuehk4dRkwkDGRrqpvQ4Ga6hZqzTkydGE3S1KkWG
      id_test.go:877: peerstore has no signed record for peer QmQ81pQuehk4dRkwkDGRrqpvQ4Ga6hZqzTkydGE3S1KkWG
  --- FAIL: TestLargeIdentifyMessage (9.11s)

marten-seemann avatar May 24 '22 14:05 marten-seemann

Failed run: https://github.com/libp2p/go-libp2p/runs/6681155403?check_suite_focus=true

marten-seemann avatar Jun 01 '22 11:06 marten-seemann

Re openeing since I think there are some fixes still left to do

MarcoPolo avatar Jun 02 '22 20:06 MarcoPolo

Picking this up as related to (or maybe duplicate of) https://github.com/libp2p/go-libp2p/issues/1164.

schomatis avatar Jun 24 '22 15:06 schomatis