[#648] objsvc/delete: Remove redundant logs

We never propagate delete requests to the container node, because
tombstone broadcast is done via PUT. No need to pollute logs.

Signed-off-by: Evgenii Stratonikov <e.stratonikov@yadro.com>
This commit is contained in:
Evgenii Stratonikov 2023-08-25 10:43:49 +03:00 committed by Evgenii Stratonikov
parent 40b556fc19
commit d2084ece41
3 changed files with 2 additions and 16 deletions

View file

@ -82,7 +82,6 @@ const (
PersistentCouldNotCleanUpExpiredTokens = "could not clean up expired tokens"
ControllerReportIsAlreadyStarted = "report is already started"
TombstoneCouldNotGetTheTombstoneTheSource = "tombstone getter: could not get the tombstone the source"
DeleteRequestIsNotRolledOverToTheContainer = "request is not rolled over to the container"
DeleteCouldNotComposeSplitInfo = "could not compose split info"
DeleteNoSplitInfoObjectIsPHY = "no split info, object is PHY"
DeleteAssemblingChain = "assembling chain..."

View file

@ -1,7 +0,0 @@
package deletesvc
import "git.frostfs.info/TrueCloudLab/frostfs-node/internal/logs"
func (exec *execCtx) executeOnContainer() {
exec.log.Debug(logs.DeleteRequestIsNotRolledOverToTheContainer)
}

View file

@ -39,11 +39,10 @@ func (exec *execCtx) execute(ctx context.Context) {
// perform local operation
exec.executeLocal(ctx)
exec.analyzeStatus(true)
exec.analyzeStatus()
}
func (exec *execCtx) analyzeStatus(execCnr bool) {
func (exec *execCtx) analyzeStatus() {
// analyze local result
switch exec.status {
case statusOK:
@ -52,10 +51,5 @@ func (exec *execCtx) analyzeStatus(execCnr bool) {
exec.log.Debug(logs.OperationFinishedWithError,
zap.String("error", exec.err.Error()),
)
if execCnr {
exec.executeOnContainer()
exec.analyzeStatus(false)
}
}
}