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

Call arguments for exec missing (but present in confirm message). #10

Open
sangee2004 opened this issue Jul 23, 2024 · 1 comment
Open
Labels
bug Something isn't working

Comments

@sangee2004
Copy link

clio version - v0.1.1

Steps to reproduce the problem

  1. Chat with k8s agent and ask it to delete an existing deployment
  2. Notice that the call arguments for exec is missing in the TUI (but present in confirm message).
> delete the deployment nginx-deployment

    ┌─────────────────┐
    │ Call Arguments: │
    │                 │
    │ exec {"         │
    └─────────────────┘
                       
Run "kubectl delete deployment nginx-deployment" (or allow all "kubectl delete ..." commands)
Confirm (y/n/a)>  

After I confirm , I see it have the actual command for exec

> delete the deployment nginx-deployment

    ┌───────────────────────────────────────────────────────────────┐
    │ Call Arguments:                                               │
    │                                                               │
    │ exec {"command":"kubectl delete deployment nginx-deployment"} │
    └───────────────────────────────────────────────────────────────┘
                                                                     
  Running  kubectl delete deployment nginx-deployment                                                                                                                                          

    ┌────────────────────────────────────────────┐
    │ deployment.apps "nginx-deployment" deleted │
    └────────────────────────────────────────────┘
                                                  
  The deployment  nginx-deployment  has been successfully deleted.                                                                                                                             
                                                                                                                                                                                               
  Would you like to verify the deletion or perform any other actions?                                                                                                                          

@K8s>  

Anther instance on when this happens:

 > yes

    ┌─────────────────┐
    │ Call Arguments: │
    │                 │
    │ exec {"         │
    └─────────────────┘
                       
Run "kubectl get deployments" (or allow all "kubectl get ..." commands)
Confirm (y/n/a)>  

On confirmation:

> list all deployments
> list all deployments

    ┌─────────────────────────────────────────────────────────────┐
    │ Call Arguments:                                             │
    │                                                             │
    │ exec {"command":"kubectl get deployments --all-namespaces"} │
    └─────────────────────────────────────────────────────────────┘
                                                                   
  Running  kubectl get deployments --all-namespaces        

Expected Behavior:
Call arguments should not be missing for exec calls.

@sangee2004 sangee2004 added the bug Something isn't working label Jul 23, 2024
@sangee2004
Copy link
Author

Similar truncation issue is also seen when testing with gptscript -gptscript-ai/gptscript#659

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant