fix: disable adding multiple openai provider

[gyliu513]

Closes #1169

📑 Description

✅ Checks

• My pull request adheres to the code style of this project
• My code requires changes to the documentation
• I have updated the documentation as required
• All the tests have passed

ℹ Additional Information

[gyliu513]

/cc @AlexsJones

[gyliu513]

gyliu@guangyas-air k8sgpt % ./bin/k8sgpt auth add
Warning: backend input is empty, will use the default value: openai
Warning: model input is empty, will use the default value: gpt-3.5-turbo
Enter openai Key: openai added to the AI backend provider list

gyliu@guangyas-air k8sgpt % ./bin/k8sgpt auth add
Warning: backend input is empty, will use the default value: openai
Warning: model input is empty, will use the default value: gpt-3.5-turbo
Enter openai Key: Provider with same name already exists.

gyliu@Guangyas-MacBook-Air k8sgpt % cat k8sgpt.yaml
active_filters:
    - ValidatingWebhookConfiguration
    - Deployment
    - PersistentVolumeClaim
    - InstanaSelfHostedInstallCheck
    - StatefulSet
    - CronJob
    - ReplicaSet
    - Service
    - Node
    - MutatingWebhookConfiguration
    - InstanaSelfHostedValidate
    - Ingress
    - Pod
ai:
    providers:
        - name: openai
          model: gpt-3.5-turbo
          password: "123456"
          temperature: 0.7
          topp: 0.5
          topk: 50
          maxtokens: 2048
          customheaders: []
    defaultprovider: ""
kubeconfig: ""
kubecontext: ""

[AlexsJones]

Can you help me understand this issue?

[gyliu513]

@AlexsJones if you take a look at https://github.com/k8sgpt-ai/k8sgpt/blob/main/cmd/auth/add.go#L64 , you will see the backend is always empty if I did not set backend when adding auth, here I was trying to get backend first before adding auth to k8sgpt.yaml, is this clear for you? Thanks

[JuHyung-Son]

your intend is to make disabling auth add if there is existing one. right?

[gyliu513]

your intend is to make disabling auth add if there is existing one. right?

@JuHyung-Son This is the design behavior for all other LLM backends, I was trying to enable this behavior for openai as well.

[gyliu513]

@AlexsJones @JuHyung-Son any further comments for this? Thanks!

[matthisholleville]

Hi @gyliu513 ! I just tested it, and it works well. However, the CLI still prompts me to enter an API key before returning an error, since the backend already exists. Shouldn't we perform this check before displaying the Enter openai Key ?

[gyliu513]

@matthisholleville done

gyliu@Guangyas-MacBook-Air k8sgpt % ./bin/k8sgpt auth add
Warning: backend input is empty, will use the default value: openai
Provider with same name already exists.

[matthisholleville]

lgtm ! Great work