hash order dependency in fastimport tests

Bug #1808630 reported by Jelmer Vernooij
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Breezy
Fix Released
High
Jelmer Vernooij

Bug Description

This is a recent regression:

% PYTHONHASHSEED=5 python3 ./brz selftest breezy.plugins.fastimport.tests.test_commands.TestFastExport.test_baseline_option
======================================================================
FAIL: breezy.plugins.fastimport.tests.test_commands.TestFastExport.test_baseline_option
----------------------------------------------------------------------
Traceback (most recent call last):
testtools.testresult.real._StringException: log: {{{
1.537 Using Bazaar configuration directory (/home/jelmer/.bazaar)
1.538 creating repository in file:///tmp/testbzr-42b3_k57.tmp/.bzr/.
1.538 Using Bazaar configuration directory (/home/jelmer/.bazaar)
1.539 Using Bazaar configuration directory (/home/jelmer/.bazaar)
1.540 Using Bazaar configuration directory (/home/jelmer/.bazaar)
1.540 creating branch <breezy.bzr.branch.BzrBranchFormat7 object at 0x7ff92fb810b8> in file:///tmp/testbzr-42b3_k57.tmp/
1.540 Using Bazaar configuration directory (/home/jelmer/.bazaar)
1.542 Using Bazaar configuration directory (/home/jelmer/.bazaar)
1.542 Using Bazaar configuration directory (/home/jelmer/.bazaar)
1.543 Using Bazaar configuration directory (/home/jelmer/.bazaar)
1.544 Using Bazaar configuration directory (/home/jelmer/.bazaar)
1.544 Using Bazaar configuration directory (/home/jelmer/.bazaar)
1.545 Using Bazaar configuration directory (/home/jelmer/.bazaar)
1.545 Using Bazaar configuration directory (/home/jelmer/.bazaar)
1.545 trying to create missing lock '/tmp/testbzr-42b3_k57.tmp/.bzr/checkout/dirstate'
1.546 opening working tree '/tmp/testbzr-42b3_k57.tmp'
1.547 Using Bazaar configuration directory (/home/jelmer/.bazaar)
1.547 Using Bazaar configuration directory (/home/jelmer/.bazaar)
1.547 Using Bazaar configuration directory (/home/jelmer/.bazaar)
1.547 Using Bazaar configuration directory (/home/jelmer/.bazaar)
1.552 creating repository in file:///tmp/testbzr-42b3_k57.tmp/breezy.plugins.fastimport.tests.test_commands.TestFastExport.test_baseline_option/work/bl/.bzr/.
1.553 creating branch <breezy.bzr.branch.BzrBranchFormat7 object at 0x7ff9313e0c18> in file:///tmp/testbzr-42b3_k57.tmp/breezy.plugins.fastimport.tests.test_commands.TestFastExport.test_baseline_option/work/bl/
1.558 trying to create missing lock '/tmp/testbzr-42b3_k57.tmp/breezy.plugins.fastimport.tests.test_commands.TestFastExport.test_baseline_option/work/bl/.bzr/checkout/dirstate'
1.558 opening working tree '/tmp/testbzr-42b3_k57.tmp/breezy.plugins.fastimport.tests.test_commands.TestFastExport.test_baseline_option/work/bl'
1.567 preparing to commit
    INFO Committing to: /tmp/testbzr-42b3_k57.tmp/breezy.plugins.fastimport.tests.test_commands.TestFastExport.test_baseline_option/work/bl/
1.568 Selecting files for commit with filter None
1.569 failed to load compiled extension: No module named 'breezy._readdir_pyx'
    INFO added a
    INFO Committed revision 1.
1.575 Committed revid <email address hidden>' as revno 1.
1.579 preparing to commit
    INFO Committing to: /tmp/testbzr-42b3_k57.tmp/breezy.plugins.fastimport.tests.test_commands.TestFastExport.test_baseline_option/work/bl/
1.580 Selecting files for commit with filter None
    INFO modified a
    INFO added b
    INFO Committed revision 2.
1.586 Committed revid <email address hidden>' as revno 2.
    INFO removed b
1.591 preparing to commit
    INFO Committing to: /tmp/testbzr-42b3_k57.tmp/breezy.plugins.fastimport.tests.test_commands.TestFastExport.test_baseline_option/work/bl/
1.592 Selecting files for commit with filter None
    INFO deleted b
    INFO added c
    INFO Committed revision 3.
1.600 Committed revid <email address hidden>' as revno 3.
1.602 preparing to commit
    INFO Committing to: /tmp/testbzr-42b3_k57.tmp/breezy.plugins.fastimport.tests.test_commands.TestFastExport.test_baseline_option/work/bl/
1.603 Selecting files for commit with filter None
    INFO modified a
    INFO Committed revision 4.
1.609 Committed revid <email address hidden>' as revno 4.
1.614 preparing to commit
    INFO Committing to: /tmp/testbzr-42b3_k57.tmp/breezy.plugins.fastimport.tests.test_commands.TestFastExport.test_baseline_option/work/bl/
1.615 Selecting files for commit with filter None
    INFO added d
    INFO Committed revision 5.
1.621 Committed revid <email address hidden>' as revno 5.
1.621 run brz: ['fast-export', '--baseline', '-r', '3..', 'bl']
1.622 breezy version: 3.0a2
1.622 brz arguments: ['fast-export', '--baseline', '-r', '3..', 'bl']
    INFO 14:00:16 Calculating the revisions to include ...
    INFO 14:00:16 Calculating the revisions to exclude ...
    INFO 14:00:16 Starting export of 3 revisions ...
    INFO 14:00:16 Exported 3 revisions in 0:00:00
1.631 output:
'reset refs/heads/master\ncommit refs/heads/master\nmark :1\ncommitter <email address hidden> <> 1544882416 +0000\ndata 15\nadd c, remove b\nM 644 inline c\ndata 6\ntest 4\nM 644 inline a\ndata 13\ntest 1\ntest 3\ncommit refs/heads/master\nmark :2\ncommitter <email address hidden> <> 1544882416 +0000\ndata 14\nmodify a again\nfrom :1\nM 644 inline a\ndata 20\ntest 1\ntest 3\ntest 5\ncommit refs/heads/master\nmark :3\ncommitter <email address hidden> <> 1544882416 +0000\ndata 5\nadd d\nfrom :2\nM 644 inline d\ndata 6\ntest 6\n'
1.631 errors:
'14:00:16 Calculating the revisions to include ...\n14:00:16 Calculating the revisions to exclude ...\n14:00:16 Starting export of 3 revisions ...\n14:00:16 Exported 3 revisions in 0:00:00\n'
}}}

Traceback (most recent call last):
  File "/home/jelmer/src/breezy.old/pending/info-colocated/breezy/plugins/fastimport/tests/test_commands.py", line 191, in test_baseline_option
    self.assertEquals(fast_export_baseline_data, data)
  File "/home/jelmer/src/breezy.old/pending/info-colocated/breezy/tests/__init__.py", line 1317, in assertEqual
    pprint.pformat(a), pprint.pformat(b)))
AssertionError: not equal:
a = ('reset refs/heads/master\n'
 'commit refs/heads/master\n'
 'mark :1\n'
 'committer\n'
 'data 15\n'
 'add c, remove b\n'
 'M 644 inline a\n'
 'data 13\n'
 'test 1\n'
 'test 3\n'
 'M 644 inline c\n'
 'data 6\n'
 'test 4\n'
 'commit refs/heads/master\n'
 'mark :2\n'
 'committer\n'
 'data 14\n'
 'modify a again\n'
 'from :1\n'
 'M 644 inline a\n'
 'data 20\n'
 'test 1\n'
 'test 3\n'
 'test 5\n'
 'commit refs/heads/master\n'
 'mark :3\n'
 'committer\n'
 'data 5\n'
 'add d\n'
 'from :2\n'
 'M 644 inline d\n'
 'data 6\n'
 'test 6\n')
b = ('reset refs/heads/master\n'
 'commit refs/heads/master\n'
 'mark :1\n'
 'committer\n'
 'data 15\n'
 'add c, remove b\n'
 'M 644 inline c\n'
 'data 6\n'
 'test 4\n'
 'M 644 inline a\n'
 'data 13\n'
 'test 1\n'
 'test 3\n'
 'commit refs/heads/master\n'
 'mark :2\n'
 'committer\n'
 'data 14\n'
 'modify a again\n'
 'from :1\n'
 'M 644 inline a\n'
 'data 20\n'
 'test 1\n'
 'test 3\n'
 'test 5\n'
 'commit refs/heads/master\n'
 'mark :3\n'
 'committer\n'
 'data 5\n'
 'add d\n'
 'from :2\n'
 'M 644 inline d\n'
 'data 6\n'
 'test 6\n')

----------------------------------------------------------------------
Ran 1 test in 0.107s

FAILED (failures=1)

Tags: selftest

Related branches

Jelmer Vernooij (jelmer)
Changed in brz:
status: Triaged → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.