Skip to content
This repository has been archived by the owner on Sep 15, 2022. It is now read-only.

Remove pre-delete-job from AzureBroker #67

Merged
merged 1 commit into from
Feb 28, 2019
Merged

Remove pre-delete-job from AzureBroker #67

merged 1 commit into from
Feb 28, 2019

Conversation

adamwalach
Copy link
Contributor

@adamwalach adamwalach commented Feb 27, 2019 •
edited
Loading

Description

  • removed pre-delete-job

Related issue(s)

@adamwalach adamwalach added this to the Sprint_Gopher_12 milestone Feb 27, 2019
@adamwalach adamwalach added the area/service-management Issues or PRs related to service management label Feb 27, 2019
@adamwalach
Copy link
Contributor Author

I provisioned AzureBroker and deprovsioned it afterwards. Both actions were successful and I confirmed that pre-delete job was not triggered during deprovisioning action

kubectl -n qa get po -w
NAME                                                              READY     STATUS    RESTARTS   AGE
azure-service-broker-redis-0                                      1/1       Running   0          59s
hb-azure-service-broker-default-82793724-3a8a-11e9-b1-azurlr6zg   1/1       Running   0          59s
azure-service-broker-redis-0   1/1       Terminating   0         1m
hb-azure-service-broker-default-82793724-3a8a-11e9-b1-azurlr6zg   0/1       Error     0         1m
hb-azure-service-broker-default-82793724-3a8a-11e9-b1-azurlr6zg   0/1       Error     1         1m
azure-service-broker-redis-0   0/1       Terminating   0         1m
hb-azure-service-broker-default-82793724-3a8a-11e9-b1-azurlr6zg   0/1       CrashLoopBackOff   1         1m
azure-service-broker-redis-0   0/1       Terminating   0         1m
azure-service-broker-redis-0   0/1       Terminating   0         1m
hb-azure-service-broker-default-82793724-3a8a-11e9-b1-azurlr6zg   0/1       Terminating   1         1m
hb-azure-service-broker-default-82793724-3a8a-11e9-b1-azurlr6zg   0/1       Terminating   1         1m
hb-azure-service-broker-default-82793724-3a8a-11e9-b1-azurlr6zg   0/1       Terminating   1         1m
hb-azure-service-broker-default-82793724-3a8a-11e9-b1-azurlr6zg   0/1       Terminating   1         1m

@adamwalach adamwalach mentioned this pull request Feb 27, 2019
Closed
@adamwalach adamwalach merged commit 4567ecb into kyma-project:master Feb 28, 2019
@adamwalach adamwalach added the kind/feature Categorizes issue or PR as related to a new feature. label Feb 28, 2019
@adamwalach adamwalach removed the kind/feature Categorizes issue or PR as related to a new feature. label Apr 16, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@jasiu001 jasiu001 jasiu001 approved these changes

@aszecowka aszecowka Awaiting requested review from aszecowka

@mszostok mszostok Awaiting requested review from mszostok

@piotrmiskiewicz piotrmiskiewicz Awaiting requested review from piotrmiskiewicz piotrmiskiewicz is a code owner

@PK85 PK85 Awaiting requested review from PK85 PK85 is a code owner

@polskikiel polskikiel Awaiting requested review from polskikiel

Assignees
No one assigned
Labels
area/service-management Issues or PRs related to service management
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@adamwalach @jasiu001