Skip to content

Commit

Permalink
Not return error when multiple application use the same backing service.
Browse files Browse the repository at this point in the history
  • Loading branch information
rujche committed Feb 18, 2025
1 parent 81b3da1 commit e8eaaff
Showing 1 changed file with 3 additions and 4 deletions.
7 changes: 3 additions & 4 deletions analyzer/project_analysis_result.go
Original file line number Diff line number Diff line change
Expand Up @@ -118,12 +118,11 @@ func addApplicationRelatedBackingServiceToResult(result *ProjectAnalysisResult,
if result.Services == nil {
result.Services = make(map[string]Service)
}
// todo: support multiple application use same backing Service,
// merge properties (like database name) instead of return error
if _, ok := result.Services[backingServiceName]; ok {
return fmt.Errorf("backingServiceName %s already exists", backingServiceName)
// todo: merge properties (like database name)
} else {
result.Services[backingServiceName] = backingService
}
result.Services[backingServiceName] = backingService
// 3. Add Application to backing Service mapping
if result.ApplicationToBackingService == nil {
result.ApplicationToBackingService = make(map[string]map[string]interface{})
Expand Down

0 comments on commit e8eaaff

Please sign in to comment.