Merge lp:~spiv/bzr/cross-format-stacking-fetch-562380-2.0 into lp:bzr/2.0

[Andrew Bennetts]

Don't allow RemoteRepository to stack on an incompatible-format repository. (#562380)

Backport to 2.0 of <https://code.edge.launchpad.net/~spiv/bzr/cross-format-stacking-fetch-562380/+merge/23526>: Don't allow RemoteRepository to stack on an incompatible-format repository. (#562380)

[Vincent Ladeuil (vila)]

review: Approve

[Vincent Ladeuil (vila)]

This can't merge by pqm, same root cause as jam's one:

test_selftest.TestSelftest.test_runner_class ERROR 9ms
    'AutoTimingTestResultDecorator' object has no attribute 'done'

======================================================================
ERROR: test_runner_class (bzrlib.tests.test_selftest.TestSelftest)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/net/bigmamac.local/Users/vila/src/bzr/integration/2.0/bzrlib/tests/test_selftest.py", line 1845, in test_runner_class
    test_suite_factory=self.factory)
  File "/net/bigmamac.local/Users/vila/src/bzr/integration/2.0/bzrlib/tests/test_selftest.py", line 1772, in run_selftest
    self.assertEqual(True, tests.selftest(stream=output, **kwargs))
  File "/net/bigmamac.local/Users/vila/src/bzr/integration/2.0/bzrlib/tests/__init__.py", line 3261, in selftest
    stream=stream,
  File "/net/bigmamac.local/Users/vila/src/bzr/integration/2.0/bzrlib/tests/__init__.py", line 2815, in run_suite
    result.done()
AttributeError: 'AutoTimingTestResultDecorator' object has no attribute 'done'

----------------------------------------------------------------------
Ran 21905 tests in 2064.942s

FAILED (errors=1, known_failure_count=30)

[Vincent Ladeuil (vila)]

Note that I had to run the test locally to get that result, pqm didn't issue any feedback, happily trying again adn againg (see pqm loops bug).

[bzr PQM (bzr-pqm)]

Successful steps
Failure output:
All lines of log output:
Executing star-merge lp:~spiv/bzr/cross-format-stacking-fetch-562380-2.0 at Thu Apr 29 04:26:54 2010
['Nothing to merge.']

=== modified file 'NEWS'
--- NEWS	2010-04-01 14:10:47 +0000
+++ NEWS	2010-04-23 05:34:16 +0000
@@ -26,6 +26,12 @@
   history no longer crash when deleted files are involved.
   (Vincent Ladeuil, John Arbash Meinel, #375898)
 
+* Repositories accessed via a smart server now reject being stacked on a
+  repository in an incompatible format, as is the case when accessing them
+  via other methods.  This was causing fetches from those repositories via
+  a smart server (e.g. using ``bzr branch``) to receive invalid data.
+  (Andrew Bennetts, #562380)
+
 bzr 2.0.5
 #########
 

=== modified file 'bzrlib/remote.py'
--- bzrlib/remote.py	2009-10-07 05:33:23 +0000
+++ bzrlib/remote.py	2010-04-23 05:34:16 +0000
@@ -27,6 +27,7 @@
     lock,
     lockdir,
     repository,
+    repository as _mod_repository,
     revision,
     revision as _mod_revision,
     symbol_versioning,
@@ -1151,6 +1152,7 @@
             # state, so always add a lock here. If a caller passes us a locked
             # repository, they are responsible for unlocking it later.
             repository.lock_read()
+        self._check_fallback_repository(repository)
         self._fallback_repositories.append(repository)
         # If self._real_repository was parameterised already (e.g. because a
         # _real_branch had its get_stacked_on_url method called), then the
@@ -1161,6 +1163,16 @@
             if repository.bzrdir.root_transport.base not in fallback_locations:
                 self._real_repository.add_fallback_repository(repository)
 
+    def _check_fallback_repository(self, repository):
+        """Check that this repository can fallback to repository safely.
+
+        Raise an error if not.
+
+        :param repository: A repository to fallback to.
+        """
+        return _mod_repository.InterRepository._assert_same_model(
+            self, repository)
+
     def add_inventory(self, revid, inv, parents):
         self._ensure_real()
         return self._real_repository.add_inventory(revid, inv, parents)

=== modified file 'bzrlib/tests/per_repository_reference/__init__.py'
--- bzrlib/tests/per_repository_reference/__init__.py	2009-07-15 17:51:40 +0000
+++ bzrlib/tests/per_repository_reference/__init__.py	2010-04-23 05:34:16 +0000
@@ -23,6 +23,7 @@
 """
 
 from bzrlib import (
+    errors,
     repository,
     remote,
     )
@@ -66,6 +67,22 @@
             self.repository_format.__class__)
 
 
+class TestIncompatibleStacking(TestCaseWithRepository):
+
+    def test_add_fallback_repository_rejects_incompatible(self):
+        # Repository.add_fallback_repository raises IncompatibleRepositories if
+        # you take two repositories in different serializations and try to
+        # stack them.
+        if self.make_repository('test')._format.supports_chks:
+            different_fmt = '1.9'
+        else:
+            different_fmt = '2a'
+        repo = self.make_repository('repo', format=different_fmt)
+        referring = self.make_repository('referring')
+        self.assertRaises(errors.IncompatibleRepositories,
+                referring.add_fallback_repository, repo)
+
+
 def external_reference_test_scenarios():
     """Generate test scenarios for repositories supporting external references.
     """

=== modified file 'bzrlib/tests/per_repository_reference/test_default_stacking.py'
--- bzrlib/tests/per_repository_reference/test_default_stacking.py	2009-06-10 03:56:49 +0000
+++ bzrlib/tests/per_repository_reference/test_default_stacking.py	2010-04-23 05:34:16 +0000
@@ -15,7 +15,6 @@
 # Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
 
 
-from bzrlib.smart import server
 from bzrlib.tests.per_repository import TestCaseWithRepository
 
 

=== modified file 'bzrlib/tests/test_remote.py'
--- bzrlib/tests/test_remote.py	2009-10-07 05:33:23 +0000
+++ bzrlib/tests/test_remote.py	2010-04-23 05:34:16 +0000
@@ -1164,8 +1164,8 @@
             len(branch.repository._real_repository._fallback_repositories))
 
     def test_get_stacked_on_real_branch(self):
-        base_branch = self.make_branch('base', format='1.6')
-        stacked_branch = self.make_branch('stacked', format='1.6')
+        base_branch = self.make_branch('base')
+        stacked_branch = self.make_branch('stacked')
         stacked_branch.set_stacked_on_url('../base')
         reference_format = self.get_repo_format()
         network_name = reference_format.network_name()
@@ -1176,7 +1176,7 @@
             'success', ('branch', branch_network_name))
         client.add_expected_call(
             'BzrDir.find_repositoryV3', ('stacked/',),
-            'success', ('ok', '', 'no', 'no', 'yes', network_name))
+            'success', ('ok', '', 'yes', 'no', 'yes', network_name))
         # called twice, once from constructor and then again by us
         client.add_expected_call(
             'Branch.get_stacked_on_url', ('stacked/',),
@@ -2115,12 +2115,13 @@
         """
         # Make a repo with a fallback repo, both using a FakeClient.
         format = remote.response_tuple_to_repo_format(
-            ('yes', 'no', 'yes', 'fake-network-name'))
+            ('yes', 'no', 'yes', self.get_repo_format().network_name()))
         repo, client = self.setup_fake_client_and_repository('quack')
         repo._format = format
         fallback_repo, ignored = self.setup_fake_client_and_repository(
             'fallback')
         fallback_repo._client = client
+        fallback_repo._format = format
         repo.add_fallback_repository(fallback_repo)
         # First the client should ask the primary repo
         client.add_expected_call(