Skip to content

Commit 353f7ec

Browse files
committed
correctly delete app
1 parent e7a13f6 commit 353f7ec

File tree

1 file changed

+7
-1
lines changed

1 file changed

+7
-1
lines changed

pkg/oc/admin/diagnostics/diagnostics/cluster/app_create.go

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -541,14 +541,20 @@ func (d *AppCreate) cleanup() {
541541
func (d *AppCreate) cleanupApp() {
542542
errs := []error{}
543543
d.out.Debug("DCluAC010", fmt.Sprintf("%s: Deleting components of app '%s' if present.", now(), d.appName))
544-
appscmd.NewDeploymentConfigReaper(d.AppsClient, d.KubeClient)
545544

545+
// reap the DC's deployments first
546+
if err := appscmd.NewDeploymentConfigReaper(d.AppsClient, d.KubeClient).Stop(d.project, d.appName, time.Duration(1)*time.Second, nil); err != nil {
547+
errs = append(errs, err)
548+
}
549+
550+
// then delete the DC and its service
546551
if err := d.AppsClient.Apps().DeploymentConfigs(d.project).Delete(d.appName, nil); err != nil {
547552
errs = append(errs, err)
548553
}
549554
if err := d.KubeClient.Core().Services(d.project).Delete(d.appName, nil); err != nil {
550555
errs = append(errs, err)
551556
}
557+
552558
if len(errs) > 0 {
553559
d.out.Debug("DCluAC013", fmt.Sprintf("%s: Deleting components of app '%s' failed: %v", now(), d.appName, errs))
554560
}

0 commit comments

Comments
 (0)