summaryrefslogtreecommitdiff
path: root/src/import/pull-tar.c
diff options
context:
space:
mode:
authorLennart Poettering <lennart@poettering.net>2018-10-11 10:27:59 +0200
committerLennart Poettering <lennart@poettering.net>2018-11-26 18:09:01 +0100
commit5f7ecd610c7918bb89468ce068bfcf68114dc53e (patch)
tree84ede3c89ef2b3f28e04bf353b9debf6a1f6f016 /src/import/pull-tar.c
parente21b7229ff776891443fe3200ebf4648f3f2940b (diff)
downloadsystemd-5f7ecd610c7918bb89468ce068bfcf68114dc53e.tar.gz
import: drop logic of setting up /var/lib/machines as btrfs loopback mount
Let's simplify things and drop the logic that /var/lib/machines is setup as auto-growing btrfs loopback file /var/lib/machines.raw. THis was done in order to make quota available for machine management, but quite frankly never really worked properly, as we couldn't grow the file system in sync with its use properly. Moreover philosophically it's problematic overriding the admin's choice of file system like this. Let's hence drop this, and simplify things. Deleting code is a good feeling. Now that regular file systems provide project quota we could probably add per-machine quota support based on that, hence the btrfs quota argument is not that interesting anymore (though btrfs quota is a bit more powerful as it allows recursive quota, i.e. that the machine pool gets an overall quota in addition to per-machine quota).
Diffstat (limited to 'src/import/pull-tar.c')
-rw-r--r--src/import/pull-tar.c6
1 files changed, 0 insertions, 6 deletions
diff --git a/src/import/pull-tar.c b/src/import/pull-tar.c
index 67ea8813e1..b19d4ea391 100644
--- a/src/import/pull-tar.c
+++ b/src/import/pull-tar.c
@@ -52,7 +52,6 @@ struct TarPull {
char *local;
bool force_local;
- bool grow_machine_directory;
bool settings;
pid_t tar_pid;
@@ -112,7 +111,6 @@ int tar_pull_new(
_cleanup_(sd_event_unrefp) sd_event *e = NULL;
_cleanup_(tar_pull_unrefp) TarPull *i = NULL;
_cleanup_free_ char *root = NULL;
- bool grow;
int r;
assert(ret);
@@ -121,8 +119,6 @@ int tar_pull_new(
if (!root)
return -ENOMEM;
- grow = path_startswith(root, "/var/lib/machines");
-
if (event)
e = sd_event_ref(event);
else {
@@ -143,7 +139,6 @@ int tar_pull_new(
.on_finished = on_finished,
.userdata = userdata,
.image_root = TAKE_PTR(root),
- .grow_machine_directory = grow,
.event = TAKE_PTR(e),
.glue = TAKE_PTR(g),
};
@@ -512,7 +507,6 @@ int tar_pull_start(
i->tar_job->on_open_disk = tar_pull_job_on_open_disk_tar;
i->tar_job->on_progress = tar_pull_job_on_progress;
i->tar_job->calc_checksum = verify != IMPORT_VERIFY_NO;
- i->tar_job->grow_machine_directory = i->grow_machine_directory;
r = pull_find_old_etags(url, i->image_root, DT_DIR, ".tar-", NULL, &i->tar_job->old_etags);
if (r < 0)