Skip to content

Commit 5bf5eb2

Browse files
authored
Merge pull request #2098 from afumagalli98/feat/add-missing-db
feat: added unretrieved database to missing
2 parents 72b3712 + ff9081e commit 5bf5eb2

File tree

2 files changed

+18
-6
lines changed

2 files changed

+18
-6
lines changed

data-service/service/oracle_databases.go

Lines changed: 14 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -394,16 +394,28 @@ func (hds *HostDataService) ignorePreviousMissingDatabases(previous, new *model.
394394
}
395395
}
396396

397-
new.Features.Oracle.Database.MissingDatabases = []model.MissingDatabase{}
397+
missingDbs := make([]model.MissingDatabase, 0)
398+
398399
for _, v := range new.Features.Oracle.Database.UnlistedRunningDatabases {
399400
missingDB := model.MissingDatabase{Name: v}
400401
if _, ok := prevIgnoredMissingDBS[v]; ok {
401402
missingDB = prevIgnoredMissingDBS[v]
402403
}
403404

404-
new.Features.Oracle.Database.MissingDatabases = append(new.Features.Oracle.Database.MissingDatabases, missingDB)
405+
missingDbs = append(missingDbs, missingDB)
405406
}
406407

408+
for _, v := range new.Features.Oracle.Database.UnretrievedDatabases {
409+
missingDB := model.MissingDatabase{Name: v}
410+
if _, ok := prevIgnoredMissingDBS[v]; ok {
411+
missingDB = prevIgnoredMissingDBS[v]
412+
}
413+
414+
missingDbs = append(missingDbs, missingDB)
415+
}
416+
417+
new.Features.Oracle.Database.MissingDatabases = missingDbs
418+
407419
var alerts []model.Alert
408420

409421
for _, db := range new.Features.Oracle.Database.MissingDatabases {

model/oracle_database_feature.go

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -16,8 +16,8 @@
1616
package model
1717

1818
type OracleDatabaseFeature struct {
19-
Databases []OracleDatabase `json:"databases" bson:"databases"`
20-
MissingDatabases []MissingDatabase `json:"missingDatabases" bson:"missingDatabases"`
21-
22-
UnlistedRunningDatabases []string `json:"unlistedRunningDatabases,omitempty" bson:"-"`
19+
Databases []OracleDatabase `json:"databases" bson:"databases"`
20+
MissingDatabases []MissingDatabase `json:"missingDatabases" bson:"missingDatabases"`
21+
UnlistedRunningDatabases []string `json:"unlistedRunningDatabases,omitempty" bson:"-"`
22+
UnretrievedDatabases []string `json:"unretrievedDatabases,omitempty" bson:"-"`
2323
}

0 commit comments

Comments
 (0)