comment and close

diff --git a/doc/bugs/assistant_tries_to_drop_all_files_on_startup.mdwn b/doc/bugs/assistant_tries_to_drop_all_files_on_startup.mdwn
index 11f1cbd45144a6c373e286cb593751f8a56fac47..8c0b561f0dadd31650506b77615bf529f1481513 100644 (file)
--- a/doc/bugs/assistant_tries_to_drop_all_files_on_startup.mdwn
+++ b/doc/bugs/assistant_tries_to_drop_all_files_on_startup.mdwn
@@ -51,3 +51,4 @@ failed
 ### Have you had any luck using git-annex before? (Sometimes we get tired of reading bug reports all day and a lil' positive end note does wonders)
 
 
+> notabug per my comment; [[done]] --[[Joey]]

diff --git a/doc/bugs/assistant_tries_to_drop_all_files_on_startup/comment_1_17f31c4230b84709b7c9ae777b2ebbd0._comment b/doc/bugs/assistant_tries_to_drop_all_files_on_startup/comment_1_17f31c4230b84709b7c9ae777b2ebbd0._comment
new file mode 100644 (file)
index 0000000..9143651
--- /dev/null
+++ b/doc/bugs/assistant_tries_to_drop_all_files_on_startup/comment_1_17f31c4230b84709b7c9ae777b2ebbd0._comment
@@ -0,0 +1,33 @@
+[[!comment format=mdwn
+ username="joey"
+ subject="""comment 1"""
+ date="2021-05-10T14:53:34Z"
+ content="""
+Currently the assistant simply tries to drop when it thinks it's possible
+that the content may be droppable. It doesn't check if the drop is allowed
+before trying to drop. Because that would be redundant with the check
+that's done when actually dropping.
+
+I don't see anything dangerous about this, same as running `git annex drop`
+on lots of files and having them be preserved by numcopies is not
+dangerous.
+
+If it's a bug at all, it's only that it should be silent if
+it is unable to perform the drop due to numcopies. However, 
+making it silent about that also seems like it would make it harder to
+figure out what is preventing things from being dropping in situations
+where you do expect drops to happen.
+
+As in, ironically, your specific case! You have a transfer remote, which is
+having files pile up on it, despite them apparently having been transferred
+from it to both of the repos you want it to transfer them to. Since your
+local repo cannot access the other repo, it cannot verify it has the
+content and so leaves a copy on the transfer remote.
+
+If you had a problem with that transfer remote filling up, and nothing was
+ever logged about why it was not dropping from it, it would be hard to
+understand what was going on and how to fix it (eg by trusting the other
+repo, or adding it as a remote, or lowering numcopies to 1).
+
+So the logging seems like a good thing to me.
+"""]]