Skip to content
This repository has been archived by the owner on Jan 19, 2023. It is now read-only.

Unbound PVC still shown as "OK" #2641

Closed
davidmirror-ops opened this issue Jul 12, 2021 · 8 comments · Fixed by #2696
Closed

Unbound PVC still shown as "OK" #2641

davidmirror-ops opened this issue Jul 12, 2021 · 8 comments · Fixed by #2696
Assignees
Labels
enhancement New feature or request

Comments

@davidmirror-ops
Copy link
Contributor

davidmirror-ops commented Jul 12, 2021

What steps did you take and what happened:
[A clear and concise description of what the bug is, and what commands you ran.)

I deployed a workload in a local cluster and introduced an error: the PVC spec calls a Storage Class that doesn't exist. The workload is not working, the PVC is not bound but Octant shows it in a "OK" state which can be confusing for troubleshooting.

image

What did you expect to happen:
Octant should reflect the accurate status of a PVC
Anything else you would like to add:

Environment:

  • Octant version (use octant version): 0.21.0
  • Kubernetes version (use kubectl version): 1.21
  • OS (macOS 10.15, Windows 10, Ubuntu 19.10 etc): macOS 11.3.1
@wwitzel3 wwitzel3 added the bug Something isn't working label Jul 13, 2021
@wwitzel3
Copy link
Contributor

wwitzel3 commented Jul 14, 2021

We may benefit from doing #381 first? If we had the list of storage classes, this would let us hint that it was non-existent.

@wwitzel3 wwitzel3 added enhancement New feature or request and removed bug Something isn't working labels Jul 15, 2021
@wwitzel3
Copy link
Contributor

Extend the PVC printer to look at status and explicitly inform the user that their PVC is Unbound and potentially corelate that with recent events.

@wwitzel3
Copy link
Contributor

wwitzel3 commented Jul 15, 2021

3 areas for UI

  • resource viewer
  • summary
  • list

who ever picks this up, please work with @mklanjsek to determine a good starting point for UI especially with resource viewer

@wwitzel3 wwitzel3 added hold and removed hold labels Jul 15, 2021
@ftovaro
Copy link
Contributor

ftovaro commented Jul 15, 2021

To be honest this one looks really interesting, if is possible I'd like to take it and work it together with @mklanjsek if he's ok with it

@davidmirror-ops
Copy link
Contributor Author

davidmirror-ops commented Jul 15, 2021

This is what i see from kubectl:

davidespejo@daespejo-a01 code % kubectl get pvc
NAME        STATUS    VOLUME   CAPACITY   ACCESS MODES STORAGECLASS       AGE
www-web-0   Pending                                      my-storage-class   16s

@wwitzel3
Copy link
Contributor

@ftovaro Checking to make sure you have all the info you need to start on this issue?

@ftovaro
Copy link
Contributor

ftovaro commented Jul 27, 2021

@davidmirror-ops is it possible to share the config file of the pvc?

@davidmirror-ops
Copy link
Contributor Author

sure @ftovaro

kind: PersistentVolumeClaim
apiVersion: v1
metadata:
 name: www-web-0
  
spec:
 storageClassName: "my-storage-class"
 accessModes:
  - ReadWriteOnce
 resources:
  requests:
   storage: 2Gi

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants