fix(warning): resolve formatting, unreachable code, and exception logic

[yunkon-kim]

This PR fixes the warning.

This will help contributors read and review the code in their IDEs and reduce future errors.

Updates

• Refactored error messages to use consistent formatting with fmt.Errorf
• Applied proper formatting to log.Error() calls
• Removed unreachable codeblocks
• Added missing error checks after file operations (e.g., os.OpenFile)
• Note
  • Updated multiple files in infra, resource, and interface/rest/server modules
  • Enhanced code readability and maintainability

[seokho-son]

Thank you @yunkon-kim !

[yunkon-kim]

I'm just sharing one thing I couldn't update.

cb-tumblebug/src/core/infra/manageInfo.go

Lines 2226 to 2250 in 91efe44

	func UpdateInfraInfo(nsId string, infraInfoData model.InfraInfo) {
	infraInfoMutex.Lock()
	defer infraInfoMutex.Unlock()
	infraInfoData.Node = nil
	key := common.GenInfraKey(nsId, infraInfoData.Id, "")
	// Check existence of the key. If no key, no update.
	keyValue, exists, err := kvstore.GetKv(key)
	if !exists || err != nil {
	return
	}
	infraTmp := model.InfraInfo{}
	json.Unmarshal([]byte(keyValue.Value), &infraTmp)
	if !reflect.DeepEqual(infraTmp, infraInfoData) {
	val, _ := json.Marshal(infraInfoData)
	err = kvstore.Put(key, string(val))
	if err != nil {
	log.Error().Err(err).Msg("")
	}
	}
	}

avoid using reflect.DeepEqual with errors in

[yunkon-kim]

@seokho-son, I will merge it once the ongoing tests (system > global-infra-test) are completed.

[yunkon-kim]

[yunkon-kim]

/approve