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

Wash is unable to inspect volume that's already attached (and has ReadWriteOnce access) #758

Closed
MikaelSmith opened this issue Mar 16, 2020 · 0 comments · Fixed by #762
Closed

Wash is unable to inspect volume that's already attached (and has ReadWriteOnce access) #758

MikaelSmith opened this issue Mar 16, 2020 · 0 comments · Fixed by #762
Assignees
@MikaelSmith
Labels
bug Something isn't working

Comments

@MikaelSmith
Copy link
Contributor

I've setup a CD4PE install using http://github.com/MikaelSmith/cd4pe-k8s. When I try to cd to the persistent volume it creates, it times out. Looking at events for the temporary pod it created, I see

  Normal   Scheduled    25s               default-scheduler     Successfully assigned cd4pe-1/washjf7kc to node-1
  Warning  FailedMount  8s (x6 over 24s)  kubelet, node-1  MountVolume.SetUp failed for volume "pvc-e7d42aee-51ba-4ae9-a216-9aeaf566bd11" : mount command failed, status: Failure, reason: Rook: Mount volume failed: failed to attach volume pvc-e7d42aee-51ba-4ae9-a216-9aeaf566bd11 for pod cd4pe-1/washjf7kc. Volume is already attached by pod cd4pe-1/cd4pe-0. Status Running

There's two bugs here. We should be able to find that warning in Wash logs or output. And we should be able to inspect a volume that's already mounted (ideally by execing in the container that's mounted it, which will be faster than scheduling a temporary container).
@MikaelSmith MikaelSmith added the bug Something isn't working label Mar 16, 2020
@MikaelSmith MikaelSmith self-assigned this Mar 26, 2020
MikaelSmith added a commit to MikaelSmith/wash that referenced this issue Mar 27, 2020
@MikaelSmith
Inspect volume on existing pod if already mounted
09b3adf 
This speeds up access and avoids errors trying to mount a volume in a
new pod when it can only be mounted once.

Fixes puppetlabs-toy-chest#758.

Signed-off-by: Michael Smith <michael.smith@puppet.com>
@MikaelSmith MikaelSmith mentioned this issue Mar 27, 2020
Merged
MikaelSmith added a commit to MikaelSmith/wash that referenced this issue Mar 27, 2020
@MikaelSmith
Inspect volume on existing pod if already mounted
6996a76 
This speeds up access and avoids errors trying to mount a volume in a
new pod when it can only be mounted once.

Fixes puppetlabs-toy-chest#758.

Signed-off-by: Michael Smith <michael.smith@puppet.com>
MikaelSmith added a commit to MikaelSmith/wash that referenced this issue Mar 30, 2020
@MikaelSmith
Inspect volume on existing pod if already mounted
b12588b 
This speeds up access and avoids errors trying to mount a volume in a
new pod when it can only be mounted once.

Fixes puppetlabs-toy-chest#758.

Signed-off-by: Michael Smith <michael.smith@puppet.com>
MikaelSmith added a commit to MikaelSmith/wash that referenced this issue Mar 30, 2020
@MikaelSmith
Inspect volume on existing pod if already mounted
0fa0316 
This speeds up access and avoids errors trying to mount a volume in a
new pod when it can only be mounted once.

Fixes puppetlabs-toy-chest#758.

Signed-off-by: Michael Smith <michael.smith@puppet.com>
MikaelSmith added a commit to MikaelSmith/wash that referenced this issue Mar 31, 2020
@MikaelSmith
Inspect volume on existing pod if already mounted
0244923 
This speeds up access and avoids errors trying to mount a volume in a
new pod when it can only be mounted once.

Creates the containerBase type to represent a specific Kubernetes
container and provide methods to execute a command and asynchronously
stream its output. This reduces some of the duplication across container
and persistent volume claim implementations.

Simplify execution logic by using the same method on an existing
container or when we create a new temporaryContainer.

Fixes puppetlabs-toy-chest#758.

Signed-off-by: Michael Smith <michael.smith@puppet.com>
MikaelSmith added a commit to MikaelSmith/wash that referenced this issue Mar 31, 2020
@MikaelSmith
Inspect volume on existing pod if already mounted
aa6fbf8 
This speeds up access and avoids errors trying to mount a volume in a
new pod when it can only be mounted once.

Creates the containerBase type to represent a specific Kubernetes
container and provide methods to execute a command and asynchronously
stream its output. This reduces some of the duplication across container
and persistent volume claim implementations.

Simplify execution logic by using the same method on an existing
container or when we create a new temporaryContainer.

Fixes puppetlabs-toy-chest#758.

Signed-off-by: Michael Smith <michael.smith@puppet.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@MikaelSmith MikaelSmith

Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Improve exploring persistent volumes in Kubernetes MikaelSmith/wash
1 participant
@MikaelSmith