search

vultr / vultr-cli

Official command line tool for Vultr services
Apache License 2.0
472 stars 75 forks source link

[BUG] - Proxy protocol flag error on load balancer create #435

Closed michelgirault closed 6 months ago

michelgirault commented 6 months ago

Describe the bug vultr-cli version: Vultr-CLI v3.0.3

It is not possible to create a load balancer with the command : "vultr-cli load-balancer create --region ams" error message: "Error: error parsing flag 'proxy-protocol' for load balancer create : trying to get bool value of flag of type string"

Additional comments it works with the vultr-cli version : Vultr-cli v2.21.0

To Reproduce Steps to reproduce the behavior: open the shell command and install the vulr-cli 3.0.3 run the basic load balancer command : vultr-cli load-balancer create --region "selected region"

Expected behavior to have an output of the creation of the load-balancer in the selected region (example AMS)

Desktop (please complete the following information where applicable:

michelgirault commented 6 months ago

I changed StringP to BoolP and added false as default and it works locally after compiling

create.Flags().BoolP("proxy-protocol", "p", false, "(optional) if true, you must configure backend nodes to accept Proxy protocol.")

and

update.Flags().BoolP("proxy-protocol", "p", false, "(optional) if true, you must configure backend nodes to accept Proxy protocol.")