direct: plan: when action=delete, do not fail deps on read errors (#4462)

## Why
It's intended to work like that: if we plan to delete the resource we're
reading it to see in case it already deleted. If we get error during
read, we should delegate to resource's DoDelete() method - some of them
are intentionally no-ops, so they will succeed.

Fixes
acceptance/bundle/resources/permissions/jobs/destroy_without_mgmtperms/with_permissions

Author: denik

acceptance/bundle/resources/permissions/jobs/destroy_without_mgmtperms/with_permissions/out.destroy.direct.txt

@@ -1,9 +1,12 @@
 
 >>> errcode as-test-sp [CLI] bundle destroy --auto-approve
 Warn: reading resources.jobs.foo.permissions id="/jobs/[NUMID]": [UUID] does not have Manage permissions on Job with ID: ElasticJobId([NUMID]). Please contact the owner or an administrator for access.
-Error: cannot plan resources.jobs.foo: dependency failed: resources.jobs.foo.permissions
+Warn: reading resources.jobs.foo id="[NUMID]": User [UUID] does not have View or Admin or Manage Run or Owner permissions on job [NUMID]
+The following resources will be deleted:
+  delete resources.jobs.foo
 
-Error: planning failed
+All files and directories at the following location will be deleted: /Workspace/Users/[UUID]/.bundle/test-bundle-[UNIQUE_NAME]/default
 
-
-Exit code: 1
+Warn: Ignoring permission error when deleting resources.jobs.foo id=[NUMID]: User [UUID] does not have Admin or Owner permissions on job [NUMID]
+Deleting files...
+Destroy complete!

bundle/direct/bundle_plan.go

@@ -180,7 +180,7 @@ func (b *DeploymentBundle) CalculatePlan(ctx context.Context, client *databricks
 					plan.RemoveEntry(resourceKey)
 				} else {
 					log.Warnf(ctx, "reading %s id=%q: %s", resourceKey, dbentry.ID, err)
-					return false
+					// This is not an error during deletion, so don't return false here
 				}
 			}