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

Fix error in the example for k8sgpt-local-ai #69

Closed
2 tasks done
bradmccoydev opened this issue May 9, 2023 · 0 comments · Fixed by #70
Closed
2 tasks done

Fix error in the example for k8sgpt-local-ai #69

bradmccoydev opened this issue May 9, 2023 · 0 comments · Fixed by #70
Assignees

Comments

@bradmccoydev
Copy link
Member

Checklist

  • I've searched for similar issues and couldn't find anything matching
  • I've discussed this feature request in the K8sGPT Slack and got positive feedback

Is this feature request related to a problem?

Yes

Problem Description

In the example for the K8sGPT there spec is wrong. The namespace doesn't exist under spec it should be in metadata.

apiVersion: core.k8sgpt.ai/v1alpha1
kind: K8sGPT
metadata:
  name: k8sgpt-local-ai
spec:
  namespace: default
  model: gpt-3.5-turbo
  backend: localai
  noCache: false
  version: v0.2.7
  enableAI: true

This should be fixed as per: chart/operator/templates/k8sgpt-crd.yaml

Solution Description

Move namespace under metadata not spec

Benefits

New people won't have errors when applying example.

Potential Drawbacks

No response

Additional Information

No response

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant