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
Projects

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 wwitzel3 added this to Incoming in Backlog via automation Jul 13, 2021
@wwitzel3 wwitzel3 moved this from Incoming to Themes in Backlog Jul 14, 2021
@GuessWhoSamFoo GuessWhoSamFoo added this to To do in 0.23 via automation Jul 14, 2021
@GuessWhoSamFoo GuessWhoSamFoo removed this from Themes in Backlog Jul 14, 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?

@wwitzel3 wwitzel3 moved this from To do to In progress in 0.23 Jul 26, 2021
@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

@wwitzel3 wwitzel3 moved this from In progress to Review in progress in 0.23 Jul 31, 2021
0.23 automation moved this from Review in progress to Done Jul 31, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Projects
No open projects
0.23
Done
Development

Successfully merging a pull request may close this issue.

4 participants