mirror of
https://github.com/Stone-Red-Code/dvc.git
synced 2026-11-11 07:18:28 +01:00
This patch introduces `.dvc/cache/stages` that is used to store previous runs and their results, which could then be reused later when we stumble upon the same command with the same deps and outs. Format of build cache entries is single-line json, which is readable by humans and might also be used for lock files discussed in #1871. Related to #1871 Local part of #1234
1681 lines
50 KiB
Python
1681 lines
50 KiB
Python
import filecmp
|
|
import getpass
|
|
import os
|
|
import posixpath
|
|
import re
|
|
import shutil
|
|
import uuid
|
|
from pathlib import Path
|
|
from subprocess import PIPE
|
|
from subprocess import Popen
|
|
from urllib.parse import urljoin
|
|
from unittest import SkipTest
|
|
|
|
import boto3
|
|
import paramiko
|
|
import pytest
|
|
from flaky.flaky_decorator import flaky
|
|
from google.cloud import storage as gc
|
|
from mock import patch
|
|
|
|
from dvc.compat import fspath
|
|
from dvc.exceptions import CyclicGraphError
|
|
from dvc.exceptions import ReproductionError
|
|
from dvc.exceptions import StagePathAsOutputError
|
|
from dvc.main import main
|
|
from dvc.output.base import BaseOutput
|
|
from dvc.path_info import URLInfo
|
|
from dvc.remote.local import LocalRemote
|
|
from dvc.repo import Repo as DvcRepo
|
|
from dvc.stage import Stage
|
|
from dvc.dvcfile import Dvcfile
|
|
from dvc.stage.exceptions import StageFileDoesNotExistError
|
|
from dvc.system import System
|
|
from dvc.utils import file_md5
|
|
from dvc.utils import relpath
|
|
from dvc.utils.stage import dump_stage_file
|
|
from dvc.utils.stage import load_stage_file
|
|
from dvc.utils.fs import remove
|
|
from tests.basic_env import TestDvc
|
|
from tests.remotes import (
|
|
GCP,
|
|
HDFS,
|
|
Local,
|
|
S3,
|
|
SSH,
|
|
SSHMocked,
|
|
TEST_AWS_REPO_BUCKET,
|
|
TEST_GCP_REPO_BUCKET,
|
|
)
|
|
from tests.utils.httpd import StaticFileServer, ContentMD5Handler
|
|
|
|
|
|
class SingleStageRun:
|
|
def _run(self, **kwargs):
|
|
kwargs.pop("name", None)
|
|
return self.dvc.run(**kwargs)
|
|
|
|
@staticmethod
|
|
def _get_stage_target(stage):
|
|
return stage.path
|
|
|
|
|
|
class TestRepro(SingleStageRun, TestDvc):
|
|
def setUp(self):
|
|
super().setUp()
|
|
|
|
stages = self.dvc.add(self.FOO)
|
|
self.assertEqual(len(stages), 1)
|
|
self.foo_stage = stages[0]
|
|
self.assertTrue(self.foo_stage is not None)
|
|
|
|
self.file1 = "file1"
|
|
self.file1_stage = self.file1 + ".dvc"
|
|
self.stage = self._run(
|
|
fname=self.file1_stage,
|
|
outs=[self.file1],
|
|
deps=[self.FOO, self.CODE],
|
|
cmd="python {} {} {}".format(self.CODE, self.FOO, self.file1),
|
|
name="run1",
|
|
)
|
|
|
|
|
|
class TestReproFail(TestRepro):
|
|
def test(self):
|
|
os.unlink(self.CODE)
|
|
|
|
ret = main(["repro", self._get_stage_target(self.stage)])
|
|
self.assertNotEqual(ret, 0)
|
|
|
|
|
|
class TestReproCyclicGraph(SingleStageRun, TestDvc):
|
|
def test(self):
|
|
self._run(
|
|
deps=[self.FOO],
|
|
outs=["bar.txt"],
|
|
cmd="echo bar > bar.txt",
|
|
name="copybarbar.txt",
|
|
)
|
|
|
|
self._run(
|
|
deps=["bar.txt"],
|
|
outs=["baz.txt"],
|
|
cmd="echo baz > baz.txt",
|
|
name="copybazbaz.txt",
|
|
)
|
|
|
|
stage_dump = {
|
|
"cmd": "echo baz > foo",
|
|
"deps": [{"path": "baz.txt"}],
|
|
"outs": [{"path": self.FOO}],
|
|
}
|
|
dump_stage_file("cycle.dvc", stage_dump)
|
|
|
|
with self.assertRaises(CyclicGraphError):
|
|
self.dvc.reproduce("cycle.dvc")
|
|
|
|
|
|
class TestReproWorkingDirectoryAsOutput(TestDvc):
|
|
"""
|
|
| stage.cwd | out.path | cwd as output |
|
|
|:-----------:|:---------:|:-------------:|
|
|
| dir | dir | True |
|
|
| dir/subdir/ | dir | True |
|
|
| dir | dir-1 | False |
|
|
| . | something | False |
|
|
"""
|
|
|
|
def test(self):
|
|
# File structure:
|
|
# .
|
|
# |-- dir1
|
|
# | |__ dir2.dvc (out.path == ../dir2)
|
|
# |__ dir2
|
|
# |__ something.dvc (stage.cwd == ./dir2)
|
|
|
|
os.mkdir(os.path.join(self.dvc.root_dir, "dir1"))
|
|
|
|
self.dvc.run(
|
|
fname=os.path.join("dir1", "dir2.dvc"),
|
|
wdir="dir1",
|
|
outs=[os.path.join("..", "dir2")],
|
|
cmd="mkdir {path}".format(path=os.path.join("..", "dir2")),
|
|
)
|
|
|
|
faulty_stage_path = os.path.join("dir2", "something.dvc")
|
|
|
|
output = os.path.join("..", "something")
|
|
stage_dump = {
|
|
"cmd": "echo something > {}".format(output),
|
|
"outs": [{"path": output}],
|
|
}
|
|
dump_stage_file(faulty_stage_path, stage_dump)
|
|
|
|
with self.assertRaises(StagePathAsOutputError):
|
|
self.dvc.reproduce(faulty_stage_path)
|
|
|
|
def test_nested(self):
|
|
# .
|
|
# |-- a
|
|
# | |__ nested
|
|
# | |__ dir
|
|
# | |__ error.dvc (stage.cwd == 'a/nested/dir')
|
|
# |__ b
|
|
# |__ nested.dvc (stage.out == 'a/nested')
|
|
dir1 = "b"
|
|
dir2 = "a"
|
|
|
|
os.mkdir(dir1)
|
|
os.mkdir(dir2)
|
|
|
|
nested_dir = os.path.join(dir2, "nested")
|
|
out_dir = relpath(nested_dir, dir1)
|
|
|
|
nested_stage = self.dvc.run(
|
|
fname=os.path.join(dir1, "b.dvc"),
|
|
wdir=dir1,
|
|
outs=[out_dir], # ../a/nested
|
|
cmd="mkdir {path}".format(path=out_dir),
|
|
)
|
|
|
|
os.mkdir(os.path.join(nested_dir, "dir"))
|
|
|
|
error_stage_path = os.path.join(nested_dir, "dir", "error.dvc")
|
|
|
|
output = os.path.join("..", "..", "something")
|
|
stage_dump = {
|
|
"cmd": "echo something > {}".format(output),
|
|
"outs": [{"path": output}],
|
|
}
|
|
dump_stage_file(error_stage_path, stage_dump)
|
|
|
|
# NOTE: os.walk() walks in a sorted order and we need dir2 subdirs to
|
|
# be processed before dir1 to load error.dvc first.
|
|
self.dvc.stages = [
|
|
nested_stage,
|
|
Dvcfile(self.dvc, error_stage_path).stage,
|
|
]
|
|
|
|
with patch.object(self.dvc, "_reset"): # to prevent `stages` resetting
|
|
with self.assertRaises(StagePathAsOutputError):
|
|
self.dvc.reproduce(error_stage_path)
|
|
|
|
def test_similar_paths(self):
|
|
# File structure:
|
|
#
|
|
# .
|
|
# |-- something.dvc (out.path == something)
|
|
# |-- something
|
|
# |__ something-1
|
|
# |-- a
|
|
# |__ a.dvc (stage.cwd == something-1)
|
|
|
|
self.dvc.run(outs=["something"], cmd="mkdir something")
|
|
|
|
os.mkdir("something-1")
|
|
|
|
stage = os.path.join("something-1", "a.dvc")
|
|
|
|
stage_dump = {"cmd": "echo a > a", "outs": [{"path": "a"}]}
|
|
dump_stage_file(stage, stage_dump)
|
|
|
|
try:
|
|
self.dvc.reproduce(stage)
|
|
except StagePathAsOutputError:
|
|
self.fail("should not raise StagePathAsOutputError")
|
|
|
|
|
|
class TestReproDepUnderDir(SingleStageRun, TestDvc):
|
|
def test(self):
|
|
stages = self.dvc.add(self.DATA_DIR)
|
|
self.assertEqual(len(stages), 1)
|
|
self.dir_stage = stages[0]
|
|
self.assertTrue(self.dir_stage is not None)
|
|
|
|
self.file1 = "file1"
|
|
stage = self._run(
|
|
fname=self.file1 + ".dvc",
|
|
outs=[self.file1],
|
|
deps=[self.DATA, self.CODE],
|
|
cmd="python {} {} {}".format(self.CODE, self.DATA, self.file1),
|
|
name="copy-data-file1",
|
|
)
|
|
|
|
self.assertTrue(filecmp.cmp(self.file1, self.DATA, shallow=False))
|
|
|
|
os.unlink(self.DATA)
|
|
shutil.copyfile(self.FOO, self.DATA)
|
|
|
|
stages = self.dvc.reproduce(self._get_stage_target(stage))
|
|
self.assertEqual(len(stages), 2)
|
|
self.assertTrue(filecmp.cmp(self.file1, self.FOO, shallow=False))
|
|
|
|
|
|
class TestReproDepDirWithOutputsUnderIt(SingleStageRun, TestDvc):
|
|
def test(self):
|
|
stages = self.dvc.add(self.DATA)
|
|
self.assertEqual(len(stages), 1)
|
|
self.assertTrue(stages[0] is not None)
|
|
|
|
stages = self.dvc.add(self.DATA_SUB)
|
|
self.assertEqual(len(stages), 1)
|
|
self.assertTrue(stages[0] is not None)
|
|
|
|
stage = self.dvc.run(
|
|
fname="dvcfile2.dvc", deps=[self.DATA, self.DATA_SUB]
|
|
)
|
|
self.assertTrue(stage is not None)
|
|
|
|
file1 = "file1"
|
|
file1_stage = file1 + ".dvc"
|
|
stage = self._run(
|
|
fname=file1_stage,
|
|
deps=[self.DATA_DIR],
|
|
outs=[file1],
|
|
cmd="python {} {} {}".format(self.CODE, self.DATA, file1),
|
|
name="copy-data-file1",
|
|
)
|
|
self.assertTrue(stage is not None)
|
|
|
|
os.unlink(self.DATA)
|
|
shutil.copyfile(self.FOO, self.DATA)
|
|
|
|
stages = self.dvc.reproduce(self._get_stage_target(stage))
|
|
self.assertEqual(len(stages), 2)
|
|
|
|
|
|
class TestReproNoDeps(TestRepro):
|
|
def test(self):
|
|
out = "out"
|
|
code_file = "out.py"
|
|
stage_file = "out.dvc"
|
|
code = (
|
|
'import uuid\nwith open("{}", "w+") as fd:\n'
|
|
"\tfd.write(str(uuid.uuid4()))\n".format(out)
|
|
)
|
|
with open(code_file, "w+") as fd:
|
|
fd.write(code)
|
|
stage = self._run(
|
|
fname=stage_file,
|
|
outs=[out],
|
|
cmd="python {}".format(code_file),
|
|
name="uuid",
|
|
)
|
|
|
|
stages = self.dvc.reproduce(self._get_stage_target(stage))
|
|
self.assertEqual(len(stages), 1)
|
|
|
|
|
|
class TestReproForce(TestRepro):
|
|
def test(self):
|
|
stages = self.dvc.reproduce(
|
|
self._get_stage_target(self.stage), force=True
|
|
)
|
|
self.assertEqual(len(stages), 2)
|
|
|
|
|
|
class TestReproChangedCode(TestRepro):
|
|
def test(self):
|
|
self.swap_code()
|
|
|
|
stages = self.dvc.reproduce(self._get_stage_target(self.stage))
|
|
|
|
self.assertTrue(filecmp.cmp(self.file1, self.BAR, shallow=False))
|
|
self.assertEqual(len(stages), 1)
|
|
|
|
def swap_code(self):
|
|
os.unlink(self.CODE)
|
|
new_contents = self.CODE_CONTENTS
|
|
new_contents += "\nshutil.copyfile('{}', " "sys.argv[2])\n".format(
|
|
self.BAR
|
|
)
|
|
self.create(self.CODE, new_contents)
|
|
|
|
|
|
class TestReproChangedData(TestRepro):
|
|
def test(self):
|
|
self.swap_foo_with_bar()
|
|
|
|
stages = self.dvc.reproduce(self._get_stage_target(self.stage))
|
|
|
|
self.assertTrue(filecmp.cmp(self.file1, self.BAR, shallow=False))
|
|
self.assertEqual(len(stages), 2)
|
|
|
|
def swap_foo_with_bar(self):
|
|
os.unlink(self.FOO)
|
|
shutil.copyfile(self.BAR, self.FOO)
|
|
|
|
|
|
class TestReproDry(TestReproChangedData):
|
|
def test(self):
|
|
self.swap_foo_with_bar()
|
|
|
|
stages = self.dvc.reproduce(
|
|
self._get_stage_target(self.stage), dry=True
|
|
)
|
|
|
|
self.assertTrue(len(stages), 2)
|
|
self.assertFalse(filecmp.cmp(self.file1, self.BAR, shallow=False))
|
|
|
|
ret = main(["repro", "--dry", self._get_stage_target(self.stage)])
|
|
self.assertEqual(ret, 0)
|
|
self.assertFalse(filecmp.cmp(self.file1, self.BAR, shallow=False))
|
|
|
|
|
|
class TestReproUpToDate(TestRepro):
|
|
def test(self):
|
|
ret = main(["repro", self._get_stage_target(self.stage)])
|
|
self.assertEqual(ret, 0)
|
|
|
|
|
|
class TestReproDryNoExec(TestDvc):
|
|
def test(self):
|
|
deps = []
|
|
for d in range(3):
|
|
idir = "idir{}".format(d)
|
|
odir = "odir{}".format(d)
|
|
|
|
deps.append("-d")
|
|
deps.append(odir)
|
|
|
|
os.mkdir(idir)
|
|
|
|
f = os.path.join(idir, "file")
|
|
with open(f, "w+") as fobj:
|
|
fobj.write(str(d))
|
|
|
|
ret = main(
|
|
[
|
|
"run",
|
|
"--no-exec",
|
|
"-d",
|
|
idir,
|
|
"-o",
|
|
odir,
|
|
"python -c 'import shutil; "
|
|
'shutil.copytree("{}", "{}")\''.format(idir, odir),
|
|
]
|
|
)
|
|
self.assertEqual(ret, 0)
|
|
|
|
ret = main(["run", "--no-exec", "-f", "Dvcfile"] + deps)
|
|
self.assertEqual(ret, 0)
|
|
|
|
ret = main(["repro", "--dry"])
|
|
self.assertEqual(ret, 0)
|
|
|
|
|
|
class TestReproChangedDeepData(TestReproChangedData):
|
|
def setUp(self):
|
|
super().setUp()
|
|
|
|
self.file2 = "file2"
|
|
self.stage = self._run(
|
|
fname=self.file2 + ".dvc",
|
|
outs=[self.file2],
|
|
deps=[self.file1, self.CODE],
|
|
cmd="python {} {} {}".format(self.CODE, self.file1, self.file2),
|
|
name="copy-file-file2",
|
|
)
|
|
|
|
def test(self):
|
|
self.swap_foo_with_bar()
|
|
|
|
stages = self.dvc.reproduce(self._get_stage_target(self.stage))
|
|
|
|
self.assertTrue(filecmp.cmp(self.file1, self.BAR, shallow=False))
|
|
self.assertTrue(filecmp.cmp(self.file2, self.BAR, shallow=False))
|
|
self.assertEqual(len(stages), 3)
|
|
|
|
|
|
class TestReproIgnoreBuildCache(TestDvc):
|
|
def test(self):
|
|
stages = self.dvc.add(self.FOO)
|
|
self.assertEqual(len(stages), 1)
|
|
foo_stage = stages[0]
|
|
self.assertTrue(foo_stage is not None)
|
|
|
|
code1 = "code1.py"
|
|
shutil.copyfile(self.CODE, code1)
|
|
file1 = "file1"
|
|
file1_stage = self.dvc.run(
|
|
outs=[file1],
|
|
deps=[self.FOO, code1],
|
|
cmd="python {} {} {}".format(code1, self.FOO, file1),
|
|
)
|
|
self.assertTrue(file1_stage is not None)
|
|
|
|
code2 = "code2.py"
|
|
shutil.copyfile(self.CODE, code2)
|
|
file2 = "file2"
|
|
file2_stage = self.dvc.run(
|
|
outs=[file2],
|
|
deps=[file1, code2],
|
|
cmd="python {} {} {}".format(code2, file1, file2),
|
|
)
|
|
self.assertTrue(file2_stage is not None)
|
|
|
|
code3 = "code3.py"
|
|
shutil.copyfile(self.CODE, code3)
|
|
file3 = "file3"
|
|
file3_stage = self.dvc.run(
|
|
outs=[file3],
|
|
deps=[file2, code3],
|
|
cmd="python {} {} {}".format(code3, file2, file3),
|
|
)
|
|
self.assertTrue(file3_stage is not None)
|
|
|
|
with open(code2, "a") as fobj:
|
|
fobj.write("\n\n")
|
|
|
|
stages = self.dvc.reproduce(file3_stage.path, ignore_build_cache=True)
|
|
self.assertEqual(len(stages), 2)
|
|
self.assertEqual(stages[0].path, file2_stage.path)
|
|
self.assertEqual(stages[1].path, file3_stage.path)
|
|
|
|
|
|
class TestReproPipeline(TestReproChangedDeepData):
|
|
def test(self):
|
|
stages = self.dvc.reproduce(
|
|
self._get_stage_target(self.stage), force=True, pipeline=True
|
|
)
|
|
self.assertEqual(len(stages), 3)
|
|
|
|
def test_cli(self):
|
|
ret = main(
|
|
["repro", "--pipeline", "-f", self._get_stage_target(self.stage)]
|
|
)
|
|
self.assertEqual(ret, 0)
|
|
|
|
|
|
class TestReproPipelines(SingleStageRun, TestDvc):
|
|
def setUp(self):
|
|
super().setUp()
|
|
|
|
stages = self.dvc.add(self.FOO)
|
|
self.assertEqual(len(stages), 1)
|
|
self.foo_stage = stages[0]
|
|
self.assertTrue(self.foo_stage is not None)
|
|
|
|
stages = self.dvc.add(self.BAR)
|
|
self.assertEqual(len(stages), 1)
|
|
self.bar_stage = stages[0]
|
|
self.assertTrue(self.bar_stage is not None)
|
|
|
|
self.file1 = "file1"
|
|
self.file1_stage = self.dvc.run(
|
|
fname=self.file1 + ".dvc",
|
|
outs=[self.file1],
|
|
deps=[self.FOO, self.CODE],
|
|
cmd="python {} {} {}".format(self.CODE, self.FOO, self.file1),
|
|
)
|
|
|
|
self.file2 = "file2"
|
|
self.file2_stage = self._run(
|
|
fname=self.file2 + ".dvc",
|
|
outs=[self.file2],
|
|
deps=[self.BAR, self.CODE],
|
|
cmd="python {} {} {}".format(self.CODE, self.BAR, self.file2),
|
|
name="copy-BAR-file2",
|
|
)
|
|
|
|
def test(self):
|
|
stages = self.dvc.reproduce(all_pipelines=True, force=True)
|
|
self.assertEqual(len(stages), 4)
|
|
self.assertTrue(self.file1_stage in stages)
|
|
self.assertTrue(self.file2_stage in stages)
|
|
|
|
def test_cli(self):
|
|
ret = main(["repro", "-f", "-P"])
|
|
self.assertEqual(ret, 0)
|
|
|
|
|
|
class TestReproLocked(TestReproChangedData):
|
|
def test(self):
|
|
file2 = "file2"
|
|
file2_stage = self._run(
|
|
fname=file2 + ".dvc",
|
|
outs=[file2],
|
|
deps=[self.file1, self.CODE],
|
|
cmd="python {} {} {}".format(self.CODE, self.file1, file2),
|
|
name="copy-file1-file2",
|
|
)
|
|
|
|
self.swap_foo_with_bar()
|
|
|
|
ret = main(["lock", self._get_stage_target(file2_stage)])
|
|
self.assertEqual(ret, 0)
|
|
stages = self.dvc.reproduce(self._get_stage_target(file2_stage))
|
|
self.assertEqual(len(stages), 0)
|
|
|
|
ret = main(["unlock", self._get_stage_target(file2_stage)])
|
|
self.assertEqual(ret, 0)
|
|
stages = self.dvc.reproduce(self._get_stage_target(file2_stage))
|
|
self.assertTrue(filecmp.cmp(self.file1, self.BAR, shallow=False))
|
|
self.assertTrue(filecmp.cmp(file2, self.BAR, shallow=False))
|
|
self.assertEqual(len(stages), 3)
|
|
|
|
def test_non_existing(self):
|
|
with self.assertRaises(StageFileDoesNotExistError):
|
|
self.dvc.lock_stage("Dvcfile")
|
|
self.dvc.lock_stage("pipelines.yaml")
|
|
self.dvc.lock_stage("pipelines.yaml:name")
|
|
self.dvc.lock_stage("Dvcfile:name")
|
|
self.dvc.lock_stage("stage.dvc")
|
|
self.dvc.lock_stage("stage.dvc:name")
|
|
self.dvc.lock_stage("not-existing-stage.json")
|
|
|
|
ret = main(["lock", "non-existing-stage"])
|
|
self.assertNotEqual(ret, 0)
|
|
|
|
|
|
class TestReproLockedCallback(SingleStageRun, TestDvc):
|
|
def test(self):
|
|
file1 = "file1"
|
|
file1_stage = file1 + ".dvc"
|
|
# NOTE: purposefully not specifying dependencies
|
|
# to create a callback stage.
|
|
stage = self._run(
|
|
fname=file1_stage,
|
|
outs=[file1],
|
|
cmd="python {} {} {}".format(self.CODE, self.FOO, file1),
|
|
name="copy-FOO-file1",
|
|
)
|
|
self.assertTrue(stage is not None)
|
|
|
|
stages = self.dvc.reproduce(self._get_stage_target(stage))
|
|
self.assertEqual(len(stages), 1)
|
|
|
|
self.dvc.lock_stage(self._get_stage_target(stage))
|
|
stages = self.dvc.reproduce(self._get_stage_target(stage))
|
|
self.assertEqual(len(stages), 0)
|
|
|
|
self.dvc.lock_stage(self._get_stage_target(stage), unlock=True)
|
|
stages = self.dvc.reproduce(self._get_stage_target(stage))
|
|
self.assertEqual(len(stages), 1)
|
|
|
|
|
|
class TestReproLockedUnchanged(TestRepro):
|
|
def test(self):
|
|
"""
|
|
Check that locking/unlocking doesn't affect stage state
|
|
"""
|
|
target = self._get_stage_target(self.stage)
|
|
self.dvc.lock_stage(target)
|
|
stages = self.dvc.reproduce(target)
|
|
self.assertEqual(len(stages), 0)
|
|
|
|
self.dvc.lock_stage(target, unlock=True)
|
|
stages = self.dvc.reproduce(target)
|
|
self.assertEqual(len(stages), 0)
|
|
|
|
|
|
class TestReproMetricsAddUnchanged(TestDvc):
|
|
def test(self):
|
|
"""
|
|
Check that adding/removing metrics doesn't affect stage state
|
|
"""
|
|
stages = self.dvc.add(self.FOO)
|
|
self.assertEqual(len(stages), 1)
|
|
self.assertTrue(stages[0] is not None)
|
|
|
|
file1 = "file1"
|
|
file1_stage = file1 + ".dvc"
|
|
self.dvc.run(
|
|
fname=file1_stage,
|
|
outs_no_cache=[file1],
|
|
deps=[self.FOO, self.CODE],
|
|
cmd="python {} {} {}".format(self.CODE, self.FOO, file1),
|
|
)
|
|
|
|
stages = self.dvc.reproduce(file1_stage)
|
|
self.assertEqual(len(stages), 0)
|
|
|
|
self.dvc.metrics.add(file1)
|
|
stages = self.dvc.reproduce(file1_stage)
|
|
self.assertEqual(len(stages), 0)
|
|
|
|
self.dvc.metrics.remove(file1)
|
|
stages = self.dvc.reproduce(file1_stage)
|
|
self.assertEqual(len(stages), 0)
|
|
|
|
|
|
class TestReproPhony(TestReproChangedData):
|
|
def test(self):
|
|
stage = self._run(
|
|
cmd="cat " + self.file1, deps=[self.file1], name="no_cmd?"
|
|
)
|
|
|
|
self.swap_foo_with_bar()
|
|
|
|
self.dvc.reproduce(self._get_stage_target(stage))
|
|
|
|
self.assertTrue(filecmp.cmp(self.file1, self.BAR, shallow=False))
|
|
|
|
|
|
class TestNonExistingOutput(TestRepro):
|
|
def test(self):
|
|
os.unlink(self.FOO)
|
|
|
|
with self.assertRaises(ReproductionError):
|
|
self.dvc.reproduce(self._get_stage_target(self.stage))
|
|
|
|
|
|
class TestReproDataSource(TestReproChangedData):
|
|
def test(self):
|
|
self.swap_foo_with_bar()
|
|
|
|
stages = self.dvc.reproduce(self.foo_stage.path)
|
|
|
|
self.assertTrue(filecmp.cmp(self.FOO, self.BAR, shallow=False))
|
|
self.assertEqual(stages[0].outs[0].checksum, file_md5(self.BAR)[0])
|
|
|
|
|
|
class TestReproChangedDir(SingleStageRun, TestDvc):
|
|
def test(self):
|
|
file_name = "file"
|
|
shutil.copyfile(self.FOO, file_name)
|
|
|
|
dir_name = "dir"
|
|
dir_code = "dir.py"
|
|
code = (
|
|
'import os; import shutil; os.mkdir("{}"); '
|
|
'shutil.copyfile("{}", os.path.join("{}", "{}"))'
|
|
)
|
|
|
|
with open(dir_code, "w+") as fd:
|
|
fd.write(code.format(dir_name, file_name, dir_name, file_name))
|
|
|
|
stage = self._run(
|
|
outs=[dir_name],
|
|
deps=[file_name, dir_code],
|
|
cmd="python {}".format(dir_code),
|
|
name="copy-in-dir",
|
|
)
|
|
target = self._get_stage_target(stage)
|
|
|
|
stages = self.dvc.reproduce(target)
|
|
self.assertEqual(len(stages), 0)
|
|
|
|
os.unlink(file_name)
|
|
shutil.copyfile(self.BAR, file_name)
|
|
|
|
stages = self.dvc.reproduce(target)
|
|
self.assertEqual(len(stages), 1)
|
|
|
|
|
|
class TestReproChangedDirData(SingleStageRun, TestDvc):
|
|
def test(self):
|
|
dir_name = "dir"
|
|
dir_code = "dir_code.py"
|
|
|
|
with open(dir_code, "w+") as fd:
|
|
fd.write(
|
|
"import os; import sys; import shutil; "
|
|
"shutil.copytree(sys.argv[1], sys.argv[2])"
|
|
)
|
|
|
|
stage = self._run(
|
|
outs=[dir_name],
|
|
deps=[self.DATA_DIR, dir_code],
|
|
cmd="python {} {} {}".format(dir_code, self.DATA_DIR, dir_name),
|
|
name="copy-dir",
|
|
)
|
|
target = self._get_stage_target(stage)
|
|
|
|
self.assertTrue(stage is not None)
|
|
|
|
stages = self.dvc.reproduce(target)
|
|
self.assertEqual(len(stages), 0)
|
|
|
|
with open(self.DATA_SUB, "a") as fd:
|
|
fd.write("add")
|
|
|
|
stages = self.dvc.reproduce(target)
|
|
self.assertEqual(len(stages), 1)
|
|
self.assertTrue(stages[0] is not None)
|
|
|
|
# Check that dvc indeed registers changed output dir
|
|
shutil.move(self.BAR, dir_name)
|
|
stages = self.dvc.reproduce(target)
|
|
self.assertEqual(len(stages), 1)
|
|
self.assertTrue(stages[0] is not None)
|
|
|
|
# Check that dvc registers mtime change for the directory.
|
|
System.hardlink(self.DATA_SUB, self.DATA_SUB + ".lnk")
|
|
stages = self.dvc.reproduce(target)
|
|
self.assertEqual(len(stages), 1)
|
|
self.assertTrue(stages[0] is not None)
|
|
|
|
|
|
class TestReproMissingMd5InStageFile(TestRepro):
|
|
def test(self):
|
|
d = load_stage_file(self.file1_stage)
|
|
del d[Stage.PARAM_OUTS][0][LocalRemote.PARAM_CHECKSUM]
|
|
del d[Stage.PARAM_DEPS][0][LocalRemote.PARAM_CHECKSUM]
|
|
dump_stage_file(self.file1_stage, d)
|
|
|
|
stages = self.dvc.reproduce(self.file1_stage)
|
|
self.assertEqual(len(stages), 1)
|
|
|
|
|
|
class TestCmdRepro(TestReproChangedData):
|
|
def test(self):
|
|
self.swap_foo_with_bar()
|
|
|
|
ret = main(["status"])
|
|
self.assertEqual(ret, 0)
|
|
|
|
ret = main(["repro", self._get_stage_target(self.stage)])
|
|
self.assertEqual(ret, 0)
|
|
|
|
ret = main(["repro", "non-existing-file"])
|
|
self.assertNotEqual(ret, 0)
|
|
|
|
|
|
class TestCmdReproChdir(TestDvc):
|
|
def test(self):
|
|
dname = "dir"
|
|
os.mkdir(dname)
|
|
foo = os.path.join(dname, self.FOO)
|
|
bar = os.path.join(dname, self.BAR)
|
|
code = os.path.join(dname, self.CODE)
|
|
shutil.copyfile(self.FOO, foo)
|
|
shutil.copyfile(self.CODE, code)
|
|
|
|
ret = main(
|
|
[
|
|
"run",
|
|
"-f",
|
|
"{}/Dvcfile".format(dname),
|
|
"-w",
|
|
"{}".format(dname),
|
|
"-d",
|
|
self.FOO,
|
|
"-o",
|
|
self.BAR,
|
|
"python {} {} {}".format(self.CODE, self.FOO, self.BAR),
|
|
]
|
|
)
|
|
self.assertEqual(ret, 0)
|
|
self.assertTrue(os.path.isfile(foo))
|
|
self.assertTrue(os.path.isfile(bar))
|
|
self.assertTrue(filecmp.cmp(foo, bar, shallow=False))
|
|
|
|
os.unlink(bar)
|
|
|
|
ret = main(["repro", "-c", dname])
|
|
self.assertEqual(ret, 0)
|
|
self.assertTrue(os.path.isfile(foo))
|
|
self.assertTrue(os.path.isfile(bar))
|
|
self.assertTrue(filecmp.cmp(foo, bar, shallow=False))
|
|
|
|
|
|
class TestReproExternalBase(SingleStageRun, TestDvc):
|
|
cache_type = None
|
|
|
|
@staticmethod
|
|
def should_test():
|
|
return False
|
|
|
|
@property
|
|
def cache_scheme(self):
|
|
return self.scheme
|
|
|
|
@property
|
|
def scheme(self):
|
|
return None
|
|
|
|
@property
|
|
def scheme_sep(self):
|
|
return "://"
|
|
|
|
@property
|
|
def sep(self):
|
|
return "/"
|
|
|
|
def check_already_cached(self, stage):
|
|
stage.outs[0].remove()
|
|
|
|
patch_download = patch.object(
|
|
stage.deps[0], "download", wraps=stage.deps[0].download
|
|
)
|
|
|
|
patch_checkout = patch.object(
|
|
stage.outs[0], "checkout", wraps=stage.outs[0].checkout
|
|
)
|
|
|
|
patch_run = patch.object(stage, "_run", wraps=stage._run)
|
|
|
|
with self.dvc.lock, self.dvc.state:
|
|
with patch_download as mock_download:
|
|
with patch_checkout as mock_checkout:
|
|
with patch_run as mock_run:
|
|
stage.locked = False
|
|
stage.run()
|
|
stage.locked = True
|
|
|
|
mock_run.assert_not_called()
|
|
mock_download.assert_not_called()
|
|
mock_checkout.assert_called_once()
|
|
|
|
@patch("dvc.prompt.confirm", return_value=True)
|
|
def test(self, mock_prompt):
|
|
if not self.should_test():
|
|
raise SkipTest(
|
|
"Test {} is disabled".format(self.__class__.__name__)
|
|
)
|
|
|
|
cache = (
|
|
self.scheme
|
|
+ self.scheme_sep
|
|
+ self.bucket
|
|
+ self.sep
|
|
+ str(uuid.uuid4())
|
|
)
|
|
|
|
ret = main(["config", "cache." + self.cache_scheme, "myrepo"])
|
|
self.assertEqual(ret, 0)
|
|
ret = main(["remote", "add", "myrepo", cache])
|
|
self.assertEqual(ret, 0)
|
|
if self.cache_type:
|
|
ret = main(["remote", "modify", "myrepo", "type", self.cache_type])
|
|
self.assertEqual(ret, 0)
|
|
|
|
remote_name = "myremote"
|
|
remote_key = str(uuid.uuid4())
|
|
remote = (
|
|
self.scheme + self.scheme_sep + self.bucket + self.sep + remote_key
|
|
)
|
|
|
|
ret = main(["remote", "add", remote_name, remote])
|
|
self.assertEqual(ret, 0)
|
|
if self.cache_type:
|
|
ret = main(
|
|
["remote", "modify", remote_name, "type", self.cache_type]
|
|
)
|
|
self.assertEqual(ret, 0)
|
|
|
|
self.dvc = DvcRepo(".")
|
|
|
|
foo_key = remote_key + self.sep + self.FOO
|
|
bar_key = remote_key + self.sep + self.BAR
|
|
|
|
foo_path = (
|
|
self.scheme + self.scheme_sep + self.bucket + self.sep + foo_key
|
|
)
|
|
bar_path = (
|
|
self.scheme + self.scheme_sep + self.bucket + self.sep + bar_key
|
|
)
|
|
|
|
# Using both plain and remote notation
|
|
out_foo_path = "remote://" + remote_name + "/" + self.FOO
|
|
out_bar_path = bar_path
|
|
|
|
self.write(self.bucket, foo_key, self.FOO_CONTENTS)
|
|
|
|
import_stage = self.dvc.imp_url(out_foo_path, "import")
|
|
|
|
self.assertTrue(os.path.exists("import"))
|
|
self.assertTrue(filecmp.cmp("import", self.FOO, shallow=False))
|
|
self.assertEqual(self.dvc.status([import_stage.path]), {})
|
|
self.check_already_cached(import_stage)
|
|
|
|
import_remote_stage = self.dvc.imp_url(
|
|
out_foo_path, out_foo_path + "_imported"
|
|
)
|
|
self.assertEqual(self.dvc.status([import_remote_stage.path]), {})
|
|
|
|
cmd_stage = self._run(
|
|
outs=[out_bar_path],
|
|
deps=[out_foo_path],
|
|
cmd=self.cmd(foo_path, bar_path),
|
|
name="external-base",
|
|
)
|
|
|
|
self.assertEqual(self.dvc.status([cmd_stage.addressing]), {})
|
|
self.assertEqual(self.dvc.status(), {})
|
|
self.check_already_cached(cmd_stage)
|
|
|
|
self.write(self.bucket, foo_key, self.BAR_CONTENTS)
|
|
|
|
self.assertNotEqual(self.dvc.status(), {})
|
|
|
|
self.dvc.update(import_stage.path)
|
|
self.assertTrue(os.path.exists("import"))
|
|
self.assertTrue(filecmp.cmp("import", self.BAR, shallow=False))
|
|
self.assertEqual(self.dvc.status([import_stage.path]), {})
|
|
|
|
self.dvc.update(import_remote_stage.path)
|
|
self.assertEqual(self.dvc.status([import_remote_stage.path]), {})
|
|
|
|
stages = self.dvc.reproduce(cmd_stage.addressing)
|
|
self.assertEqual(len(stages), 1)
|
|
self.assertEqual(self.dvc.status([cmd_stage.addressing]), {})
|
|
|
|
self.assertEqual(self.dvc.status(), {})
|
|
self.dvc.gc(workspace=True)
|
|
self.assertEqual(self.dvc.status(), {})
|
|
|
|
self.dvc.remove(cmd_stage.path, outs_only=True)
|
|
self.assertNotEqual(self.dvc.status([cmd_stage.addressing]), {})
|
|
|
|
self.dvc.checkout([cmd_stage.path], force=True)
|
|
self.assertEqual(self.dvc.status([cmd_stage.addressing]), {})
|
|
|
|
|
|
@pytest.mark.skipif(os.name == "nt", reason="temporarily disabled on windows")
|
|
class TestReproExternalS3(S3, TestReproExternalBase):
|
|
@property
|
|
def scheme(self):
|
|
return "s3"
|
|
|
|
@property
|
|
def bucket(self):
|
|
return TEST_AWS_REPO_BUCKET
|
|
|
|
def cmd(self, i, o):
|
|
return "aws s3 cp {} {}".format(i, o)
|
|
|
|
def write(self, bucket, key, body):
|
|
s3 = boto3.client("s3")
|
|
s3.put_object(Bucket=bucket, Key=key, Body=body)
|
|
|
|
|
|
class TestReproExternalGS(GCP, TestReproExternalBase):
|
|
@property
|
|
def scheme(self):
|
|
return "gs"
|
|
|
|
@property
|
|
def bucket(self):
|
|
return TEST_GCP_REPO_BUCKET
|
|
|
|
def cmd(self, i, o):
|
|
return "gsutil cp {} {}".format(i, o)
|
|
|
|
def write(self, bucket, key, body):
|
|
client = gc.Client()
|
|
bucket = client.bucket(bucket)
|
|
bucket.blob(key).upload_from_string(body)
|
|
|
|
|
|
class TestReproExternalHDFS(HDFS, TestReproExternalBase):
|
|
@property
|
|
def scheme(self):
|
|
return "hdfs"
|
|
|
|
@property
|
|
def bucket(self):
|
|
return "{}@127.0.0.1".format(getpass.getuser())
|
|
|
|
def cmd(self, i, o):
|
|
return "hadoop fs -cp {} {}".format(i, o)
|
|
|
|
def write(self, bucket, key, body):
|
|
url = self.scheme + "://" + bucket + "/" + key
|
|
p = Popen(
|
|
"hadoop fs -rm -f {}".format(url),
|
|
shell=True,
|
|
executable=os.getenv("SHELL"),
|
|
stdin=PIPE,
|
|
stdout=PIPE,
|
|
stderr=PIPE,
|
|
)
|
|
p.communicate()
|
|
|
|
p = Popen(
|
|
"hadoop fs -mkdir -p {}".format(posixpath.dirname(url)),
|
|
shell=True,
|
|
executable=os.getenv("SHELL"),
|
|
stdin=PIPE,
|
|
stdout=PIPE,
|
|
stderr=PIPE,
|
|
)
|
|
out, err = p.communicate()
|
|
if p.returncode != 0:
|
|
print(out)
|
|
print(err)
|
|
self.assertEqual(p.returncode, 0)
|
|
|
|
with open("tmp", "w+") as fd:
|
|
fd.write(body)
|
|
|
|
p = Popen(
|
|
"hadoop fs -copyFromLocal {} {}".format("tmp", url),
|
|
shell=True,
|
|
executable=os.getenv("SHELL"),
|
|
stdin=PIPE,
|
|
stdout=PIPE,
|
|
stderr=PIPE,
|
|
)
|
|
out, err = p.communicate()
|
|
if p.returncode != 0:
|
|
print(out)
|
|
print(err)
|
|
self.assertEqual(p.returncode, 0)
|
|
|
|
|
|
@flaky(max_runs=3, min_passes=1)
|
|
class TestReproExternalSSH(SSH, TestReproExternalBase):
|
|
_dir = None
|
|
cache_type = "copy"
|
|
|
|
@property
|
|
def scheme(self):
|
|
return "ssh"
|
|
|
|
@property
|
|
def bucket(self):
|
|
if not self._dir:
|
|
self._dir = self.mkdtemp()
|
|
return "{}@127.0.0.1:{}".format(getpass.getuser(), self._dir)
|
|
|
|
def cmd(self, i, o):
|
|
prefix = "ssh://"
|
|
assert i.startswith(prefix) and o.startswith(prefix)
|
|
i = i[len(prefix) :]
|
|
o = o[len(prefix) :]
|
|
return "scp {} {}".format(i, o)
|
|
|
|
def write(self, bucket, key, body):
|
|
path = posixpath.join(self._dir, key)
|
|
|
|
ssh = paramiko.SSHClient()
|
|
ssh.load_system_host_keys()
|
|
ssh.set_missing_host_key_policy(paramiko.AutoAddPolicy())
|
|
ssh.connect("127.0.0.1")
|
|
|
|
sftp = ssh.open_sftp()
|
|
try:
|
|
sftp.stat(path)
|
|
sftp.remove(path)
|
|
except IOError:
|
|
pass
|
|
|
|
stdin, stdout, stderr = ssh.exec_command(
|
|
"mkdir -p $(dirname {})".format(path)
|
|
)
|
|
self.assertEqual(stdout.channel.recv_exit_status(), 0)
|
|
|
|
with sftp.open(path, "w+") as fobj:
|
|
fobj.write(body)
|
|
|
|
|
|
class TestReproExternalLOCAL(Local, TestReproExternalBase):
|
|
cache_type = "hardlink"
|
|
|
|
def setUp(self):
|
|
super().setUp()
|
|
self.tmpdir = self.mkdtemp()
|
|
ret = main(["config", "cache.type", "hardlink"])
|
|
self.assertEqual(ret, 0)
|
|
self.dvc = DvcRepo(".")
|
|
|
|
@property
|
|
def cache_scheme(self):
|
|
return "local"
|
|
|
|
@property
|
|
def scheme(self):
|
|
return ""
|
|
|
|
@property
|
|
def scheme_sep(self):
|
|
return ""
|
|
|
|
@property
|
|
def sep(self):
|
|
return os.sep
|
|
|
|
@property
|
|
def bucket(self):
|
|
return self.tmpdir
|
|
|
|
def cmd(self, i, o):
|
|
if os.name == "nt":
|
|
return "copy {} {}".format(i, o)
|
|
return "cp {} {}".format(i, o)
|
|
|
|
def write(self, bucket, key, body):
|
|
path = os.path.join(bucket, key)
|
|
dname = os.path.dirname(path)
|
|
|
|
if not os.path.exists(dname):
|
|
os.makedirs(dname)
|
|
|
|
with open(path, "w+") as fd:
|
|
fd.write(body)
|
|
|
|
|
|
class TestReproExternalHTTP(TestReproExternalBase):
|
|
_external_cache_id = None
|
|
|
|
@staticmethod
|
|
def get_remote(port):
|
|
return "http://localhost:{}/".format(port)
|
|
|
|
@property
|
|
def local_cache(self):
|
|
return os.path.join(self.dvc.dvc_dir, "cache")
|
|
|
|
def test(self):
|
|
# Import
|
|
with StaticFileServer() as httpd:
|
|
import_url = urljoin(self.get_remote(httpd.server_port), self.FOO)
|
|
import_output = "imported_file"
|
|
import_stage = self.dvc.imp_url(import_url, import_output)
|
|
|
|
self.assertTrue(os.path.exists(import_output))
|
|
self.assertTrue(filecmp.cmp(import_output, self.FOO, shallow=False))
|
|
|
|
self.dvc.remove("imported_file.dvc")
|
|
|
|
with StaticFileServer(handler_class=ContentMD5Handler) as httpd:
|
|
import_url = urljoin(self.get_remote(httpd.server_port), self.FOO)
|
|
import_output = "imported_file"
|
|
import_stage = self.dvc.imp_url(import_url, import_output)
|
|
|
|
self.assertTrue(os.path.exists(import_output))
|
|
self.assertTrue(filecmp.cmp(import_output, self.FOO, shallow=False))
|
|
|
|
# Run --deps
|
|
with StaticFileServer() as httpd:
|
|
remote = self.get_remote(httpd.server_port)
|
|
|
|
cache_id = str(uuid.uuid4())
|
|
cache = urljoin(remote, cache_id)
|
|
|
|
ret1 = main(["remote", "add", "mycache", cache])
|
|
ret2 = main(["remote", "add", "myremote", remote])
|
|
self.assertEqual(ret1, 0)
|
|
self.assertEqual(ret2, 0)
|
|
|
|
self.dvc = DvcRepo(".")
|
|
|
|
run_dependency = urljoin(remote, self.BAR)
|
|
run_output = "remote_file"
|
|
cmd = 'open("{}", "w+")'.format(run_output)
|
|
|
|
with open("create-output.py", "w") as fd:
|
|
fd.write(cmd)
|
|
|
|
run_stage = self._run(
|
|
deps=[run_dependency],
|
|
outs=[run_output],
|
|
cmd="python create-output.py",
|
|
name="http_run",
|
|
)
|
|
self.assertTrue(run_stage is not None)
|
|
|
|
self.assertTrue(os.path.exists(run_output))
|
|
|
|
# Pull
|
|
self.dvc.remove(import_stage.path, outs_only=True)
|
|
self.assertFalse(os.path.exists(import_output))
|
|
|
|
shutil.move(self.local_cache, cache_id)
|
|
self.assertFalse(os.path.exists(self.local_cache))
|
|
|
|
self.dvc.pull([import_stage.path], remote="mycache")
|
|
|
|
self.assertTrue(os.path.exists(import_output))
|
|
|
|
|
|
class TestReproShell(TestDvc):
|
|
def test(self):
|
|
if os.name == "nt":
|
|
return
|
|
|
|
fname = "shell.txt"
|
|
stage = fname + ".dvc"
|
|
|
|
self.dvc.run(
|
|
fname=stage, outs=[fname], cmd="echo $SHELL > {}".format(fname)
|
|
)
|
|
|
|
with open(fname, "r") as fd:
|
|
self.assertEqual(os.getenv("SHELL"), fd.read().strip())
|
|
|
|
os.unlink(fname)
|
|
|
|
self.dvc.reproduce(stage)
|
|
|
|
with open(fname, "r") as fd:
|
|
self.assertEqual(os.getenv("SHELL"), fd.read().strip())
|
|
|
|
|
|
class TestReproAllPipelines(SingleStageRun, TestDvc):
|
|
def test(self):
|
|
stages = [
|
|
self._run(
|
|
fname="start.dvc",
|
|
outs=["start.txt"],
|
|
cmd="echo start > start.txt",
|
|
name="start",
|
|
),
|
|
self._run(
|
|
fname="middle.dvc",
|
|
deps=["start.txt"],
|
|
outs=["middle.txt"],
|
|
cmd="echo middle > middle.txt",
|
|
name="middle",
|
|
),
|
|
self._run(
|
|
fname="final.dvc",
|
|
deps=["middle.txt"],
|
|
outs=["final.txt"],
|
|
cmd="echo final > final.txt",
|
|
name="final",
|
|
),
|
|
self._run(
|
|
fname="disconnected.dvc",
|
|
outs=["disconnected.txt"],
|
|
cmd="echo other > disconnected.txt",
|
|
name="disconnected",
|
|
),
|
|
]
|
|
|
|
from dvc.state import StateNoop
|
|
|
|
self.dvc.state = StateNoop()
|
|
|
|
with patch.object(
|
|
Stage, "reproduce", side_effect=stages
|
|
) as mock_reproduce:
|
|
ret = main(["repro", "--all-pipelines"])
|
|
self.assertEqual(ret, 0)
|
|
self.assertEqual(mock_reproduce.call_count, 4)
|
|
|
|
|
|
class TestReproNoCommit(TestRepro):
|
|
def test(self):
|
|
remove(self.dvc.cache.local.cache_dir)
|
|
ret = main(
|
|
["repro", self._get_stage_target(self.stage), "--no-commit"]
|
|
)
|
|
self.assertEqual(ret, 0)
|
|
self.assertEqual(
|
|
os.listdir(self.dvc.cache.local.cache_dir), ["stages"]
|
|
)
|
|
|
|
|
|
class TestReproAlreadyCached(TestRepro):
|
|
def test(self):
|
|
stage = self._run(
|
|
fname="datetime.dvc",
|
|
deps=[],
|
|
outs=["datetime.txt"],
|
|
cmd='python -c "import time; print(time.time())" > datetime.txt',
|
|
name="datetime",
|
|
)
|
|
run_out = stage.outs[0]
|
|
repro_out = self.dvc.reproduce(self._get_stage_target(stage))[0].outs[
|
|
0
|
|
]
|
|
|
|
self.assertNotEqual(run_out.checksum, repro_out.checksum)
|
|
|
|
def test_force_with_dependencies(self):
|
|
run_out = self.dvc.run(
|
|
fname="datetime.dvc",
|
|
deps=[self.FOO],
|
|
outs=["datetime.txt"],
|
|
cmd='python -c "import time; print(time.time())" > datetime.txt',
|
|
).outs[0]
|
|
|
|
ret = main(["repro", "--force", "datetime.dvc"])
|
|
self.assertEqual(ret, 0)
|
|
|
|
repro_out = Dvcfile(self.dvc, "datetime.dvc").stage.outs[0]
|
|
|
|
self.assertNotEqual(run_out.checksum, repro_out.checksum)
|
|
|
|
def test_force_import(self):
|
|
ret = main(["import-url", self.FOO, self.BAR])
|
|
self.assertEqual(ret, 0)
|
|
|
|
patch_download = patch.object(
|
|
LocalRemote,
|
|
"download",
|
|
side_effect=LocalRemote.download,
|
|
autospec=True,
|
|
)
|
|
|
|
patch_checkout = patch.object(
|
|
BaseOutput,
|
|
"checkout",
|
|
side_effect=BaseOutput.checkout,
|
|
autospec=True,
|
|
)
|
|
|
|
with patch_download as mock_download:
|
|
with patch_checkout as mock_checkout:
|
|
assert main(["unlock", "bar.dvc"]) == 0
|
|
ret = main(["repro", "--force", "bar.dvc"])
|
|
self.assertEqual(ret, 0)
|
|
self.assertEqual(mock_download.call_count, 1)
|
|
self.assertEqual(mock_checkout.call_count, 0)
|
|
|
|
|
|
class TestShouldDisplayMetricsOnReproWithMetricsOption(TestDvc):
|
|
def test(self):
|
|
metrics_file = "metrics_file"
|
|
metrics_value = 0.123489015
|
|
ret = main(
|
|
[
|
|
"run",
|
|
"-m",
|
|
metrics_file,
|
|
"echo {} >> {}".format(metrics_value, metrics_file),
|
|
]
|
|
)
|
|
self.assertEqual(0, ret)
|
|
|
|
self._caplog.clear()
|
|
|
|
from dvc.dvcfile import DVC_FILE_SUFFIX
|
|
|
|
ret = main(
|
|
["repro", "--force", "--metrics", metrics_file + DVC_FILE_SUFFIX]
|
|
)
|
|
self.assertEqual(0, ret)
|
|
|
|
expected_metrics_display = "{}: {}".format(metrics_file, metrics_value)
|
|
self.assertIn(expected_metrics_display, self._caplog.text)
|
|
|
|
|
|
@pytest.fixture
|
|
def repro_dir(tmp_dir, dvc, run_copy):
|
|
# Creates repo with following structure:
|
|
# data_dir/dir_file origin_data
|
|
# | | |
|
|
# | | origin_copy.dvc
|
|
# unrelated2.dvc | | |
|
|
# | | unrelated1.dvc
|
|
# dir/subdir/dir_file_copy.dvc |
|
|
# | |
|
|
# | dir/origin_copy_2.dvc
|
|
# | |
|
|
# \ /
|
|
# \ /
|
|
# dir/Dvcfile
|
|
tmp_dir.gen(
|
|
{
|
|
"origin_data": "origin data content",
|
|
"data_dir": {"dir_file": "dir file content"},
|
|
"dir": {"subdir": {}},
|
|
}
|
|
)
|
|
|
|
stages = {}
|
|
|
|
origin_copy = tmp_dir / "origin_copy"
|
|
stage = run_copy("origin_data", fspath(origin_copy))
|
|
assert stage is not None
|
|
assert origin_copy.read_text() == "origin data content"
|
|
stages["origin_copy"] = stage
|
|
|
|
origin_copy_2 = tmp_dir / "dir" / "origin_copy_2"
|
|
stage = run_copy(
|
|
fspath(origin_copy),
|
|
fspath(origin_copy_2),
|
|
fname=fspath(origin_copy_2) + ".dvc",
|
|
)
|
|
assert stage is not None
|
|
assert origin_copy_2.read_text() == "origin data content"
|
|
stages["origin_copy_2"] = stage
|
|
|
|
dir_file_path = tmp_dir / "data_dir" / "dir_file"
|
|
dir_file_copy = tmp_dir / "dir" / "subdir" / "dir_file_copy"
|
|
stage = run_copy(
|
|
fspath(dir_file_path),
|
|
fspath(dir_file_copy),
|
|
fname=fspath(dir_file_copy) + ".dvc",
|
|
)
|
|
assert stage is not None
|
|
assert dir_file_copy.read_text() == "dir file content"
|
|
stages["dir_file_copy"] = stage
|
|
|
|
last_stage = tmp_dir / "dir" / "Dvcfile"
|
|
stage = dvc.run(
|
|
fname=fspath(last_stage),
|
|
deps=[fspath(origin_copy_2), fspath(dir_file_copy)],
|
|
)
|
|
assert stage is not None
|
|
stages["last_stage"] = stage
|
|
|
|
# Unrelated are to verify that reproducing `dir` will not trigger them too
|
|
assert run_copy(fspath(origin_copy), "unrelated1") is not None
|
|
assert run_copy(fspath(dir_file_path), "unrelated2") is not None
|
|
|
|
yield stages
|
|
|
|
|
|
def _rewrite_file(path_elements, new_content):
|
|
if isinstance(path_elements, str):
|
|
path_elements = [path_elements]
|
|
file = Path(os.sep.join(path_elements))
|
|
file.unlink()
|
|
file.write_text(new_content)
|
|
|
|
|
|
def _read_out(stage):
|
|
return Path(stage.outs[0].fspath).read_text()
|
|
|
|
|
|
def test_recursive_repro_default(dvc, repro_dir):
|
|
"""
|
|
Test recursive repro on dir after a dep outside this dir has changed.
|
|
"""
|
|
_rewrite_file("origin_data", "new origin data content")
|
|
|
|
stages = dvc.reproduce("dir", recursive=True)
|
|
|
|
# Check that the dependency ("origin_copy") and the dependent stages
|
|
# inside the folder have been reproduced ("origin_copy_2", "last_stage")
|
|
assert stages == [
|
|
repro_dir["origin_copy"],
|
|
repro_dir["origin_copy_2"],
|
|
repro_dir["last_stage"],
|
|
]
|
|
assert _read_out(repro_dir["origin_copy"]) == "new origin data content"
|
|
assert _read_out(repro_dir["origin_copy_2"]) == "new origin data content"
|
|
|
|
|
|
def test_recursive_repro_single(dvc, repro_dir):
|
|
"""
|
|
Test recursive single-item repro on dir
|
|
after a dep outside this dir has changed.
|
|
"""
|
|
_rewrite_file("origin_data", "new origin content")
|
|
_rewrite_file(["data_dir", "dir_file"], "new dir file content")
|
|
|
|
stages = dvc.reproduce("dir", recursive=True, single_item=True)
|
|
# Check that just stages inside given dir
|
|
# with changed direct deps have been reproduced.
|
|
# This means that "origin_copy_2" stage should not be reproduced
|
|
# since it depends on "origin_copy".
|
|
# Also check that "dir_file_copy" stage was reproduced before "last_stage"
|
|
assert stages == [repro_dir["dir_file_copy"], repro_dir["last_stage"]]
|
|
assert _read_out(repro_dir["dir_file_copy"]) == "new dir file content"
|
|
|
|
|
|
def test_recursive_repro_single_force(dvc, repro_dir):
|
|
"""
|
|
Test recursive single-item force repro on dir
|
|
without any dependencies changing.
|
|
"""
|
|
stages = dvc.reproduce("dir", recursive=True, single_item=True, force=True)
|
|
# Check that all stages inside given dir have been reproduced
|
|
# Also check that "dir_file_copy" stage was reproduced before "last_stage"
|
|
# and that "origin_copy" stage was reproduced before "last_stage" stage
|
|
assert len(stages) == 3
|
|
assert set(stages) == {
|
|
repro_dir["origin_copy_2"],
|
|
repro_dir["dir_file_copy"],
|
|
repro_dir["last_stage"],
|
|
}
|
|
assert stages.index(repro_dir["origin_copy_2"]) < stages.index(
|
|
repro_dir["last_stage"]
|
|
)
|
|
assert stages.index(repro_dir["dir_file_copy"]) < stages.index(
|
|
repro_dir["last_stage"]
|
|
)
|
|
|
|
|
|
def test_recursive_repro_empty_dir(tmp_dir, dvc):
|
|
"""
|
|
Test recursive repro on an empty directory
|
|
"""
|
|
(tmp_dir / "emptydir").mkdir()
|
|
|
|
stages = dvc.reproduce("emptydir", recursive=True, force=True)
|
|
assert stages == []
|
|
|
|
|
|
def test_recursive_repro_recursive_missing_file(dvc):
|
|
"""
|
|
Test recursive repro on a missing file
|
|
"""
|
|
with pytest.raises(StageFileDoesNotExistError):
|
|
dvc.reproduce("notExistingStage.dvc", recursive=True)
|
|
with pytest.raises(StageFileDoesNotExistError):
|
|
dvc.reproduce("notExistingDir/", recursive=True)
|
|
|
|
|
|
def test_recursive_repro_on_stage_file(dvc, repro_dir):
|
|
"""
|
|
Test recursive repro on a stage file instead of directory
|
|
"""
|
|
stages = dvc.reproduce(
|
|
repro_dir["origin_copy_2"].relpath, recursive=True, force=True
|
|
)
|
|
assert stages == [repro_dir["origin_copy"], repro_dir["origin_copy_2"]]
|
|
|
|
|
|
def test_dvc_formatting_retained(tmp_dir, dvc, run_copy):
|
|
tmp_dir.dvc_gen("foo", "foo content")
|
|
stage = run_copy("foo", "foo_copy", fname="foo_copy.dvc")
|
|
stage_path = tmp_dir / stage.relpath
|
|
|
|
# Add comments and custom formatting to DVC-file
|
|
lines = list(map(_format_dvc_line, stage_path.read_text().splitlines()))
|
|
lines.insert(0, "# Starting comment")
|
|
stage_text = "".join(line + "\n" for line in lines)
|
|
stage_path.write_text(stage_text)
|
|
|
|
# Rewrite data source and repro
|
|
(tmp_dir / "foo").write_text("new foo")
|
|
dvc.reproduce("foo_copy.dvc", force=True)
|
|
|
|
assert _hide_md5(stage_text) == _hide_md5(stage_path.read_text())
|
|
|
|
|
|
def _format_dvc_line(line):
|
|
# Add line comment for all cache and md5 keys
|
|
if "cache:" in line or "md5:" in line:
|
|
return line + " # line comment"
|
|
# Format command as one word per line
|
|
elif line.startswith("cmd: "):
|
|
pre, command = line.split(None, 1)
|
|
return pre + " >\n" + "\n".join(" " + s for s in command.split())
|
|
else:
|
|
return line
|
|
|
|
|
|
def _hide_md5(text):
|
|
return re.sub(r"\b[a-f0-9]{32}\b", "<md5>", text)
|
|
|
|
|
|
def test_downstream(dvc):
|
|
# The dependency graph should look like this:
|
|
#
|
|
# E
|
|
# / \
|
|
# D F
|
|
# / \ \
|
|
# B C G
|
|
# \ /
|
|
# A
|
|
#
|
|
assert main(["run", "-o", "A", "echo A>A"]) == 0
|
|
assert main(["run", "-d", "A", "-o", "B", "echo B>B"]) == 0
|
|
assert main(["run", "-d", "A", "-o", "C", "echo C>C"]) == 0
|
|
assert main(["run", "-d", "B", "-d", "C", "-o", "D", "echo D>D"]) == 0
|
|
assert main(["run", "-o", "G", "echo G>G"]) == 0
|
|
assert main(["run", "-d", "G", "-o", "F", "echo F>F"]) == 0
|
|
assert main(["run", "-d", "D", "-d", "F", "-o", "E", "echo E>E"]) == 0
|
|
|
|
# We want the evaluation to move from B to E
|
|
#
|
|
# E
|
|
# /
|
|
# D
|
|
# /
|
|
# B
|
|
#
|
|
evaluation = dvc.reproduce("B.dvc", downstream=True, force=True)
|
|
|
|
assert len(evaluation) == 3
|
|
assert evaluation[0].relpath == "B.dvc"
|
|
assert evaluation[1].relpath == "D.dvc"
|
|
assert evaluation[2].relpath == "E.dvc"
|
|
|
|
# B, C should be run (in any order) before D
|
|
# See https://github.com/iterative/dvc/issues/3602
|
|
evaluation = dvc.reproduce("A.dvc", downstream=True, force=True)
|
|
|
|
assert len(evaluation) == 5
|
|
assert evaluation[0].relpath == "A.dvc"
|
|
assert {evaluation[1].relpath, evaluation[2].relpath} == {"B.dvc", "C.dvc"}
|
|
assert evaluation[3].relpath == "D.dvc"
|
|
assert evaluation[4].relpath == "E.dvc"
|
|
|
|
|
|
@pytest.mark.skipif(
|
|
os.name == "nt",
|
|
reason="external output scenario is not supported on Windows",
|
|
)
|
|
def test_ssh_dir_out(tmp_dir, dvc, ssh_server):
|
|
tmp_dir.gen({"foo": "foo content"})
|
|
|
|
# Set up remote and cache
|
|
user = ssh_server.test_creds["username"]
|
|
port = ssh_server.port
|
|
keyfile = ssh_server.test_creds["key_filename"]
|
|
|
|
remote_url = SSHMocked.get_url(user, port)
|
|
assert main(["remote", "add", "upstream", remote_url]) == 0
|
|
assert main(["remote", "modify", "upstream", "keyfile", keyfile]) == 0
|
|
|
|
cache_url = SSHMocked.get_url(user, port)
|
|
assert main(["remote", "add", "sshcache", cache_url]) == 0
|
|
assert main(["config", "cache.ssh", "sshcache"]) == 0
|
|
assert main(["remote", "modify", "sshcache", "keyfile", keyfile]) == 0
|
|
|
|
# Recreating to reread configs
|
|
repo = DvcRepo(dvc.root_dir)
|
|
|
|
# To avoid "WARNING: UNPROTECTED PRIVATE KEY FILE" from ssh
|
|
os.chmod(keyfile, 0o600)
|
|
|
|
(tmp_dir / "script.py").write_text(
|
|
"import sys, pathlib\n"
|
|
"path = pathlib.Path(sys.argv[1])\n"
|
|
"dir_out = path / 'dir-out'\n"
|
|
"dir_out.mkdir()\n"
|
|
"(dir_out / '1.txt').write_text('1')\n"
|
|
"(dir_out / '2.txt').write_text('2')\n"
|
|
)
|
|
|
|
url_info = URLInfo(remote_url)
|
|
repo.run(
|
|
cmd="python {} {}".format(tmp_dir / "script.py", url_info.path),
|
|
outs=["remote://upstream/dir-out"],
|
|
deps=["foo"], # add a fake dep to not consider this a callback
|
|
)
|
|
|
|
repo.reproduce("dir-out.dvc")
|
|
repo.reproduce("dir-out.dvc", force=True)
|