Bug 2052757: [4.9] PVs are not being cleaned up after PVC deletion #40

dobsonj · 2022-02-09T23:32:50Z

This PR:

Rebases the openshift fork on the commit being used by LSO today.
Adds a carry patch with a fix for https://bugzilla.redhat.com/show_bug.cgi?id=2032924 on top of the rebase

Comparison to upstream:
kubernetes-sigs/sig-storage-local-static-provisioner@master...dobsonj:rebase-4.9-20210414
/cc @openshift/storage

Add option EKS to getting-started.md

…rk-v2 Helm chart rework v2

Switch to go modules and update k8s code to 1.17.0

Change link to absolute path in "Getting Started"

Docs: fix broken link in "Getting Started"

fix comments typo

…tching feat: add namePattern field in discovery PV

doc: add github template

feat: add helm support on namePattern field

fix comments

test: add travis CI to get code coverage data

add code coverage and license check

doc: add code coverage

…ample doc: add namePattern example

update get-started page with AKS

openshift-bot · 2022-02-17T01:00:42Z

/bugzilla refresh

Recalculating validity in case the underlying Bugzilla bug has changed.

openshift-ci · 2022-02-17T01:00:48Z

@openshift-bot: This pull request references Bugzilla bug 2052757, which is invalid:

expected dependent Bugzilla bug 2052756 to be in one of the following states: VERIFIED, RELEASE_PENDING, CLOSED (ERRATA), CLOSED (CURRENTRELEASE), but it is POST instead

Comment /bugzilla refresh to re-evaluate validity if changes to the Bugzilla bug are made, or edit the title of this pull request to link to a different bug.

In response to this:

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

openshift-bot · 2022-02-18T01:01:09Z

/bugzilla refresh

Recalculating validity in case the underlying Bugzilla bug has changed.

openshift-ci · 2022-02-18T01:01:15Z

@openshift-bot: This pull request references Bugzilla bug 2052757, which is invalid:

expected dependent Bugzilla bug 2052756 to be in one of the following states: VERIFIED, RELEASE_PENDING, CLOSED (ERRATA), CLOSED (CURRENTRELEASE), but it is POST instead

Comment /bugzilla refresh to re-evaluate validity if changes to the Bugzilla bug are made, or edit the title of this pull request to link to a different bug.

In response to this:

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

openshift-bot · 2022-02-19T01:00:20Z

/bugzilla refresh

Recalculating validity in case the underlying Bugzilla bug has changed.

openshift-ci · 2022-02-19T01:00:25Z

@openshift-bot: This pull request references Bugzilla bug 2052757, which is invalid:

expected dependent Bugzilla bug 2052756 to be in one of the following states: VERIFIED, RELEASE_PENDING, CLOSED (ERRATA), CLOSED (CURRENTRELEASE), but it is POST instead

Comment /bugzilla refresh to re-evaluate validity if changes to the Bugzilla bug are made, or edit the title of this pull request to link to a different bug.

In response to this:

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

openshift-bot · 2022-02-20T01:00:57Z

/bugzilla refresh

Recalculating validity in case the underlying Bugzilla bug has changed.

openshift-ci · 2022-02-20T01:01:02Z

@openshift-bot: This pull request references Bugzilla bug 2052757, which is invalid:

expected dependent Bugzilla bug 2052756 to be in one of the following states: VERIFIED, RELEASE_PENDING, CLOSED (ERRATA), CLOSED (CURRENTRELEASE), but it is POST instead

Comment /bugzilla refresh to re-evaluate validity if changes to the Bugzilla bug are made, or edit the title of this pull request to link to a different bug.

In response to this:

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

openshift-bot · 2022-02-21T01:00:57Z

/bugzilla refresh

Recalculating validity in case the underlying Bugzilla bug has changed.

openshift-ci · 2022-02-21T01:01:02Z

@openshift-bot: This pull request references Bugzilla bug 2052757, which is invalid:

expected dependent Bugzilla bug 2052756 to be in one of the following states: VERIFIED, RELEASE_PENDING, CLOSED (ERRATA), CLOSED (CURRENTRELEASE), but it is POST instead

Comment /bugzilla refresh to re-evaluate validity if changes to the Bugzilla bug are made, or edit the title of this pull request to link to a different bug.

In response to this:

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

openshift-bot · 2022-02-22T01:02:02Z

/bugzilla refresh

Recalculating validity in case the underlying Bugzilla bug has changed.

openshift-ci · 2022-02-22T01:02:08Z

@openshift-bot: This pull request references Bugzilla bug 2052757, which is invalid:

expected dependent Bugzilla bug 2052756 to be in one of the following states: VERIFIED, RELEASE_PENDING, CLOSED (ERRATA), CLOSED (CURRENTRELEASE), but it is POST instead

Comment /bugzilla refresh to re-evaluate validity if changes to the Bugzilla bug are made, or edit the title of this pull request to link to a different bug.

In response to this:

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

openshift-bot · 2022-02-23T01:01:03Z

/bugzilla refresh

Recalculating validity in case the underlying Bugzilla bug has changed.

openshift-ci · 2022-02-23T01:01:10Z

@openshift-bot: This pull request references Bugzilla bug 2052757, which is invalid:

expected dependent Bugzilla bug 2052756 to be in one of the following states: VERIFIED, RELEASE_PENDING, CLOSED (ERRATA), CLOSED (CURRENTRELEASE), but it is POST instead

Comment /bugzilla refresh to re-evaluate validity if changes to the Bugzilla bug are made, or edit the title of this pull request to link to a different bug.

In response to this:

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

openshift-bot · 2022-02-24T01:00:25Z

/bugzilla refresh

Recalculating validity in case the underlying Bugzilla bug has changed.

openshift-ci · 2022-02-24T01:00:35Z

@openshift-bot: This pull request references Bugzilla bug 2052757, which is invalid:

expected dependent Bugzilla bug 2052756 to be in one of the following states: VERIFIED, RELEASE_PENDING, CLOSED (ERRATA), CLOSED (CURRENTRELEASE), but it is POST instead

Comment /bugzilla refresh to re-evaluate validity if changes to the Bugzilla bug are made, or edit the title of this pull request to link to a different bug.

In response to this:

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

dobsonj · 2022-02-24T15:13:54Z

/assign @chao007
for cherry-pick-approved once the 4.10 fix is verified

openshift-bot · 2022-02-25T01:00:26Z

/bugzilla refresh

Recalculating validity in case the underlying Bugzilla bug has changed.

openshift-ci · 2022-02-25T01:00:35Z

@openshift-bot: This pull request references Bugzilla bug 2052757, which is invalid:

expected dependent Bugzilla bug 2052756 to be in one of the following states: VERIFIED, RELEASE_PENDING, CLOSED (ERRATA), CLOSED (CURRENTRELEASE), but it is ON_QA instead

Comment /bugzilla refresh to re-evaluate validity if changes to the Bugzilla bug are made, or edit the title of this pull request to link to a different bug.

In response to this:

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

chao007 · 2022-02-25T09:53:29Z

/label cherry-pick-approved

dobsonj · 2022-02-25T14:46:09Z

/bugzilla refresh

openshift-ci · 2022-02-25T14:46:19Z

@dobsonj: This pull request references Bugzilla bug 2052757, which is valid. The bug has been moved to the POST state. The bug has been updated to refer to the pull request using the external bug tracker.

6 validation(s) were run on this bug

bug is open, matching expected state (open)
bug target release (4.9.z) matches configured target release for branch (4.9.z)
bug is in the state ASSIGNED, which is one of the valid states (NEW, ASSIGNED, ON_DEV, POST, POST)
dependent bug Bugzilla bug 2052756 is in the state VERIFIED, which is one of the valid states (VERIFIED, RELEASE_PENDING, CLOSED (ERRATA), CLOSED (CURRENTRELEASE))
dependent Bugzilla bug 2052756 targets the "4.10.0" release, which is one of the valid target releases: 4.10.0
bug has dependents

No GitHub users were found matching the public email listed for the QA contact in Bugzilla (wduan@redhat.com), skipping review request.

In response to this:

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

openshift-ci · 2022-02-25T14:51:41Z

@dobsonj: All pull requests linked via external trackers have merged:

openshift/sig-storage-local-static-provisioner#40

Bugzilla bug 2052757 has been moved to the MODIFIED state.

In response to this:

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

openshift-bot · 2023-11-25T01:15:46Z

[ART PR BUILD NOTIFIER]

This PR has been included in build local-storage-static-provisioner-container-v4.9.0-202311250023.p0.g3b8b7b5.assembly.stream for distgit local-storage-static-provisioner.
All builds following this will include this PR.

brunsgaard and others added 30 commits December 16, 2019 22:40

Add option EKS to getting-started.md

b02823e

changed chart config and lables to support more configuration values

0c28d96

added helm v3/v2 to generation and verify script

7516792

added new generated examples for helm v2/v3

ffb87a2

adjusted example helm values

3635d9b

adjusted changelog and readme for helm v2/v3

1b102dc

changed image version to v2.3.4

3004ec8

Merge pull request kubernetes-sigs#162 from brunsgaard/master

25f1e2b

Add option EKS to getting-started.md

changed lib script helm for template generation

1c1d3dc

Merge pull request kubernetes-sigs#179 from HaveFun83/helm-chart-rewo…

73cda4c

…rk-v2 Helm chart rework v2

use go mod: vendor changes

ce8f17d

use go mod: non-vendor changes

bb5c950

Merge pull request kubernetes-sigs#184 from cofyc/gomods

81b1957

Switch to go modules and update k8s code to 1.17.0

Fix broken link in "Getting Started"

6d9b17c

Change link to absolute path in "Getting Started"

Merge pull request kubernetes-sigs#183 from AliMirlou/patch-1

9790017

Docs: fix broken link in "Getting Started"

feat: add namePattern field

f46da07

fix comments typo

Merge pull request kubernetes-sigs#187 from andyzhangx/device-name-ma…

cd56ec6

…tching feat: add namePattern field in discovery PV

doc: add github template

88a8c03

feat: add helm support on namePattern field

edf7c64

Merge pull request kubernetes-sigs#188 from andyzhangx/add-template

965a271

doc: add github template

Merge pull request kubernetes-sigs#189 from andyzhangx/namepattern-doc

0c83bc3

feat: add helm support on namePattern field

test: add CI to enable code coverage data

02b1138

fix comments

Merge pull request kubernetes-sigs#190 from andyzhangx/travis

80cc8e6

test: add travis CI to get code coverage data

doc: add code coverage and license check

33b1ca3

add code coverage and license check

doc: remove license check first

df7c8e9

Merge pull request kubernetes-sigs#191 from andyzhangx/patch-1

7c788f2

doc: add code coverage

doc: add namePattern example

0b6059d

update get-started with AKS

1267a9f

Merge pull request kubernetes-sigs#192 from andyzhangx/namePattern-ex…

0a747ab

…ample doc: add namePattern example

Merge pull request kubernetes-sigs#193 from andyzhangx/patch-2

e70b6eb

update get-started page with AKS

openshift-ci bot assigned chao007 Feb 24, 2022

openshift-ci bot added the cherry-pick-approved Indicates a cherry-pick PR into a release branch has been approved by the release branch manager. label Feb 25, 2022

openshift-ci bot added bugzilla/valid-bug Indicates that a referenced Bugzilla bug is valid for the branch this PR is targeting. and removed bugzilla/invalid-bug Indicates that a referenced Bugzilla bug is invalid for the branch this PR is targeting. labels Feb 25, 2022

openshift-merge-robot merged commit 3b8b7b5 into openshift:release-4.9 Feb 25, 2022

dobsonj mentioned this pull request Feb 25, 2022

Bug 2052757: [4.9] PVs are not being cleaned up after PVC deletion openshift/local-storage-operator#326

Merged

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Bug 2052757: [4.9] PVs are not being cleaned up after PVC deletion #40

Bug 2052757: [4.9] PVs are not being cleaned up after PVC deletion #40

dobsonj commented Feb 9, 2022

openshift-bot commented Feb 17, 2022

openshift-ci bot commented Feb 17, 2022

openshift-bot commented Feb 18, 2022

openshift-ci bot commented Feb 18, 2022

openshift-bot commented Feb 19, 2022

openshift-ci bot commented Feb 19, 2022

openshift-bot commented Feb 20, 2022

openshift-ci bot commented Feb 20, 2022

openshift-bot commented Feb 21, 2022

openshift-ci bot commented Feb 21, 2022

openshift-bot commented Feb 22, 2022

openshift-ci bot commented Feb 22, 2022

openshift-bot commented Feb 23, 2022

openshift-ci bot commented Feb 23, 2022

openshift-bot commented Feb 24, 2022

openshift-ci bot commented Feb 24, 2022

dobsonj commented Feb 24, 2022

openshift-bot commented Feb 25, 2022

openshift-ci bot commented Feb 25, 2022

chao007 commented Feb 25, 2022

dobsonj commented Feb 25, 2022

openshift-ci bot commented Feb 25, 2022

openshift-ci bot commented Feb 25, 2022

openshift-bot commented Nov 25, 2023

Bug 2052757: [4.9] PVs are not being cleaned up after PVC deletion #40

Bug 2052757: [4.9] PVs are not being cleaned up after PVC deletion #40

Conversation

dobsonj commented Feb 9, 2022

openshift-bot commented Feb 17, 2022

openshift-ci bot commented Feb 17, 2022

openshift-bot commented Feb 18, 2022

openshift-ci bot commented Feb 18, 2022

openshift-bot commented Feb 19, 2022

openshift-ci bot commented Feb 19, 2022

openshift-bot commented Feb 20, 2022

openshift-ci bot commented Feb 20, 2022

openshift-bot commented Feb 21, 2022

openshift-ci bot commented Feb 21, 2022

openshift-bot commented Feb 22, 2022

openshift-ci bot commented Feb 22, 2022

openshift-bot commented Feb 23, 2022

openshift-ci bot commented Feb 23, 2022

openshift-bot commented Feb 24, 2022

openshift-ci bot commented Feb 24, 2022

dobsonj commented Feb 24, 2022

openshift-bot commented Feb 25, 2022

openshift-ci bot commented Feb 25, 2022

chao007 commented Feb 25, 2022

dobsonj commented Feb 25, 2022

openshift-ci bot commented Feb 25, 2022

openshift-ci bot commented Feb 25, 2022

openshift-bot commented Nov 25, 2023