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

Show wrong ConfigMap data #2167

Closed
chivalryq opened this issue Jul 21, 2023 · 0 comments
Closed

Show wrong ConfigMap data #2167

chivalryq opened this issue Jul 21, 2023 · 0 comments
Labels
bug Something isn't working InProgress Marks an issue has being worked on

Comments

@chivalryq
Copy link




Describe the bug
Use command to show a ConfigMap, the content is different with the actual value (velua from with command or kubectl get)

Wrong content

content, notice the "ClassInSpec"[] near to end:
image

Now if you the content, you'll get the same wrong configmap content in clipboard.
image

Actual content

True configmap content, shown by command:

image

It's same with kuebctl get

kubectl get cm -n vela-system trait-schema-gateway-v6 -oyaml
apiVersion: v1
data:
  openapi-v3-json-schema: '{"properties":{"class":{"default":"nginx","description":"Specify
    the class of ingress to use","title":"class","type":"string"},"classInSpec":{"default":false,"description":"Set
    ingress class in ''.spec.ingressClassName'' instead of ''kubernetes.io/ingress.class''
    annotation.","title":"classInSpec","type":"boolean"},"domain":{"description":"Specify
    the domain you want to expose","title":"domain","type":"string"},"gatewayHost":{"description":"Specify
    the host of the ingress gateway, which is used to generate the endpoints when
    the host is empty.","title":"gatewayHost","type":"string"},"http":{"additionalProperties":{"type":"integer"},"description":"Specify
    the mapping relationship between the http path and the workload port","title":"http","type":"object"},"secretName":{"description":"Specify
    the secret name you want to quote to use tls.","title":"secretName","type":"string"}},"required":["http","class","classInSpec"],"type":"object"}'
kind: ConfigMap
metadata:
  annotations:
    definition.oam.dev/appliedWorkloads: deployments.apps,statefulsets.apps
  creationTimestamp: "2023-07-21T06:33:15Z"
  labels:
    definition.oam.dev: schema
    definition.oam.dev/name: gateway-v6
  name: trait-schema-gateway-v6
  namespace: vela-system
  ownerReferences:
  - apiVersion: core.oam.dev/v1beta1
    blockOwnerDeletion: true
    controller: true
    kind: DefinitionRevision
    name: gateway-v6
    uid: 4ecdf779-a181-43b3-8d0c-9cbd9bb91899
  resourceVersion: "25244925"
  uid: 7d79e8ec-1cdf-4df8-8c1d-c96dd33d726e

To Reproduce
Steps to reproduce the behavior:

  1. Use this configmap:
apiVersion: v1
data:
  openapi-v3-json-schema: '{"properties":{"class":{"default":"nginx","description":"Specify
    the class of ingress to use","title":"class","type":"string"},"classInSpec":{"default":false,"description":"Set
    ingress class in ''.spec.ingressClassName'' instead of ''kubernetes.io/ingress.class''
    annotation.","title":"classInSpec","type":"boolean"},"domain":{"description":"Specify
    the domain you want to expose","title":"domain","type":"string"},"gatewayHost":{"description":"Specify
    the host of the ingress gateway, which is used to generate the endpoints when
    the host is empty.","title":"gatewayHost","type":"string"},"http":{"additionalProperties":{"type":"integer"},"description":"Specify
    the mapping relationship between the http path and the workload port","title":"http","type":"object"},"secretName":{"description":"Specify
    the secret name you want to quote to use tls.","title":"secretName","type":"string"}},"required":["http","class","classInSpec"],"type":"object"}'
kind: ConfigMap
metadata:
  name: a-schema-configmap
  namespace: default
  1. Follow the step mentioned above.

Expected behavior
A clear and concise description of what you expected to happen.
The view show the right ConfigMap content.

Screenshots
If applicable, add screenshots to help explain your problem.

Versions (please complete the following information):

  • OS: OSX
  • K9s: 0.27.4
  • K8s: 1.26.3

Additional context
Add any other context about the problem here.

@derailed derailed added bug Something isn't working InProgress Marks an issue has being worked on labels Nov 22, 2023
derailed added a commit that referenced this issue Dec 7, 2023
* Feat: Move shell pod cluster config to general config

> BREAKING CHANGE! K9s configuration breaking change! Shellpod specification will
no longer reside with a cluster configuration. It is now part of the
global K9s configuration object.

Shellpod configuration should be part of k9s config. Clusters admins
will most likely use the same image and config to run shells on their
nodes. Each cluster in turn will have the option to either
enable/disable shelling into nodes. This not only DRYs up the k9s config
but also allows user to consolidate their shell pod configuration in one
central place.

* Fix #2290 - Add freebsd assets

* Maintenance cleaning up

* Fix #2166 - Add taint tracking column to node view

* Fix #2009: Update screendump file names to contain resource info

* Maintenance: Cleanup errror messages

* Fix #1513: Change log default to tail vs last 5min

* Fix #2166: Add taint indicator on node view

* Fix #2165: Track init co restarts

* Fix #2308: Fix rbac auth checks

* Fix #2036: Fix npe on filtering CRDs

* Fix #2219: Turn on TTY option on shellpod

* Fix #2167: Update color escape sequence on copy

* Fix #2297: Enable multi select on nodes

* Cleanup headers

* Fix #2162: Allow edit when describing/viewing

* Feat: Add helm release history support

* Fix #2039: Command Arrow up/down + enter support

* Small refactor

* Add img vulenerability scans support

* Change skin loading and support

- Move skin specification to k9s cluster config section
- Load skins for skins dir

* Release v0.29.0 docs
thejoeejoee pushed a commit to thejoeejoee/k9s that referenced this issue Feb 23, 2024
* Feat: Move shell pod cluster config to general config

> BREAKING CHANGE! K9s configuration breaking change! Shellpod specification will
no longer reside with a cluster configuration. It is now part of the
global K9s configuration object.

Shellpod configuration should be part of k9s config. Clusters admins
will most likely use the same image and config to run shells on their
nodes. Each cluster in turn will have the option to either
enable/disable shelling into nodes. This not only DRYs up the k9s config
but also allows user to consolidate their shell pod configuration in one
central place.

* Fix derailed#2290 - Add freebsd assets

* Maintenance cleaning up

* Fix derailed#2166 - Add taint tracking column to node view

* Fix derailed#2009: Update screendump file names to contain resource info

* Maintenance: Cleanup errror messages

* Fix derailed#1513: Change log default to tail vs last 5min

* Fix derailed#2166: Add taint indicator on node view

* Fix derailed#2165: Track init co restarts

* Fix derailed#2308: Fix rbac auth checks

* Fix derailed#2036: Fix npe on filtering CRDs

* Fix derailed#2219: Turn on TTY option on shellpod

* Fix derailed#2167: Update color escape sequence on copy

* Fix derailed#2297: Enable multi select on nodes

* Cleanup headers

* Fix derailed#2162: Allow edit when describing/viewing

* Feat: Add helm release history support

* Fix derailed#2039: Command Arrow up/down + enter support

* Small refactor

* Add img vulenerability scans support

* Change skin loading and support

- Move skin specification to k9s cluster config section
- Load skins for skins dir

* Release v0.29.0 docs
placintaalexandru pushed a commit to placintaalexandru/k9s that referenced this issue Apr 3, 2024
* Feat: Move shell pod cluster config to general config

> BREAKING CHANGE! K9s configuration breaking change! Shellpod specification will
no longer reside with a cluster configuration. It is now part of the
global K9s configuration object.

Shellpod configuration should be part of k9s config. Clusters admins
will most likely use the same image and config to run shells on their
nodes. Each cluster in turn will have the option to either
enable/disable shelling into nodes. This not only DRYs up the k9s config
but also allows user to consolidate their shell pod configuration in one
central place.

* Fix derailed#2290 - Add freebsd assets

* Maintenance cleaning up

* Fix derailed#2166 - Add taint tracking column to node view

* Fix derailed#2009: Update screendump file names to contain resource info

* Maintenance: Cleanup errror messages

* Fix derailed#1513: Change log default to tail vs last 5min

* Fix derailed#2166: Add taint indicator on node view

* Fix derailed#2165: Track init co restarts

* Fix derailed#2308: Fix rbac auth checks

* Fix derailed#2036: Fix npe on filtering CRDs

* Fix derailed#2219: Turn on TTY option on shellpod

* Fix derailed#2167: Update color escape sequence on copy

* Fix derailed#2297: Enable multi select on nodes

* Cleanup headers

* Fix derailed#2162: Allow edit when describing/viewing

* Feat: Add helm release history support

* Fix derailed#2039: Command Arrow up/down + enter support

* Small refactor

* Add img vulenerability scans support

* Change skin loading and support

- Move skin specification to k9s cluster config section
- Load skins for skins dir

* Release v0.29.0 docs
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working InProgress Marks an issue has being worked on
Projects
None yet
Development

No branches or pull requests

2 participants