Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update volume entry if exists #615

Merged
merged 2 commits into from
Mar 7, 2022
Merged

Update volume entry if exists #615

merged 2 commits into from
Mar 7, 2022

Conversation

0sewa0
Copy link
Contributor

@0sewa0 0sewa0 commented Mar 7, 2022

Description

In some cases (node restart) kubelet might ask to mount a previously mounted volume (volumeID is the same), which causes a clash in the database.

How can this be tested?

On node restart everything should work

Checklist

  • Unit tests have been updated/added
  • PR is labeled accordingly

@0sewa0 0sewa0 added the bug Something isn't working label Mar 7, 2022
@0sewa0 0sewa0 requested a review from a team March 7, 2022 12:57
@0sewa0 0sewa0 enabled auto-merge (squash) March 7, 2022 13:55
@chrismuellner chrismuellner changed the title Makes volume insert safer Update volume entry if exists Mar 7, 2022
@0sewa0 0sewa0 merged commit c69b1d0 into master Mar 7, 2022
@0sewa0 0sewa0 deleted the bugfix/csi-volume-update branch March 7, 2022 13:59
chrismuellner pushed a commit that referenced this pull request Mar 8, 2022
(cherry picked from commit c69b1d0)
chrismuellner pushed a commit that referenced this pull request Mar 9, 2022
(cherry picked from commit c69b1d0)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants