Skip to content

Commit

Permalink
Change trace logs format in dca pld handler
Browse files Browse the repository at this point in the history
  • Loading branch information
adel121 committed Mar 3, 2025
1 parent c571fd3 commit d7c2c90
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 12 deletions.
4 changes: 2 additions & 2 deletions cmd/cluster-agent/api/v1/languagedetection/handler.go
Original file line number Diff line number Diff line change
Expand Up @@ -112,9 +112,9 @@ func (handler *languageDetectionHandler) leaderHandler(w http.ResponseWriter, r

ownersLanguagesFromRequest := getOwnersLanguages(requestData, time.Now().Add(handler.cfg.languageTTL))

log.Tracef("Owner Languages state pre merge-and-flush:\n %s", handler.ownersLanguages.String())
log.Tracef("Owner Languages state pre merge-and-flush: %s", handler.ownersLanguages.String())
err = handler.ownersLanguages.mergeAndFlush(ownersLanguagesFromRequest, handler.wlm)
log.Tracef("Owner Languages state post merge-and-flush:\n %s", handler.ownersLanguages.String())
log.Tracef("Owner Languages state post merge-and-flush: %s", handler.ownersLanguages.String())
if err != nil {
http.Error(w, fmt.Sprintf("failed to store some (or all) languages in workloadmeta store: %s", err), http.StatusInternalServerError)
ProcessedRequests.Inc(statusError)
Expand Down
16 changes: 6 additions & 10 deletions cmd/cluster-agent/api/v1/languagedetection/util.go
Original file line number Diff line number Diff line change
Expand Up @@ -77,23 +77,19 @@ func (ownersLanguages *OwnersLanguages) String() string {
ownersLanguages.mutex.Lock()
defer ownersLanguages.mutex.Unlock()

state := ""
state := "["
for owner, langs := range ownersLanguages.containersLanguages {
state += fmt.Sprintf("===== %s/%s/%s ====\n", owner.Namespace, owner.Kind, owner.Name)
if langs.dirty {
state += "dirty\n"
} else {
state += "clean\n"
}

state += fmt.Sprintf("(%s/%s/%s, %v,[", owner.Namespace, owner.Kind, owner.Name, langs.dirty)
for container, languageSet := range langs.languages {
state += container.Name + ": "
state += container.Name + ": ("
for languagename := range languageSet {
state += fmt.Sprintf("%s,", languagename)
}
state += "\n"
state += "),"
}
state += "]),"
}
state += "]"
return state
}

Expand Down

0 comments on commit d7c2c90

Please sign in to comment.