hash order dependency in fastimport tests

Bug #1808630 reported by Jelmer Vernooij on 2018-12-15
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Breezy
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)

Related branches

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

Other bug subscribers