Skip to content

Commit

Permalink
Merge pull request #1528 from flanksource/fix-del-topology
Browse files Browse the repository at this point in the history
fix: gorm query for deleting topology
  • Loading branch information
moshloop authored Dec 26, 2023
2 parents 98b8811 + 45136fa commit 97db195
Showing 1 changed file with 2 additions and 4 deletions.
6 changes: 2 additions & 4 deletions pkg/db/topology.go
Original file line number Diff line number Diff line change
Expand Up @@ -310,12 +310,10 @@ func UpdateStatusAndSummaryForComponent(id uuid.UUID, status types.ComponentStat

func DeleteTopology(t *v1.Topology) error {
logger.Infof("Deleting topology %s/%s", t.Namespace, t.Name)
model := pkg.TopologyFromV1(t)
deleteTime := time.Now()

tx := Gorm.Table("topologies").Find(model, "id = ?", t.GetPersistedID()).UpdateColumn("deleted_at", deleteTime)
if tx.Error != nil {
return tx.Error
if err := Gorm.Table("topologies").Where("id = ?", t.GetPersistedID()).UpdateColumn("deleted_at", deleteTime).Error; err != nil {
return fmt.Errorf("failed to update deleted_at for topology[%s]: %w", t.GetPersistedID(), err)
}
return DeleteComponentsOfTopology(t.GetPersistedID(), deleteTime)
}
Expand Down

0 comments on commit 97db195

Please sign in to comment.