`-h` and `--host` don't behave the same

[dveeden]

`-h` and `--host` don't behave the same

[lichunzhu]

That's a tiup issue. Try:

tiup -- dumpling -h host

[fubinzh]

/severity minor

[dveeden]

So this is the same problem as pingcap/tiup#1488 ?

[lichunzhu]

They are similar but are not totally the same.

[dveeden]

Looks like tiup -- dumpling doesn't work anymore.

[dvaneeden@dve-carbon ~]$ tiup -v
1.5.6 tiup
Go Version: go1.16.7
Git Ref: v1.5.6
GitHash: 1d1478dbec0d06ef5e909d3323331914538d4cb0
[dvaneeden@dve-carbon ~]$ tiup dumpling -u foo
Starting component `dumpling`: /home/dvaneeden/.tiup/components/dumpling/v5.2.1/dumpling -u foo
Release version: v5.2.1
Git commit hash: 4e012e54ce89fab9551e7ff58ebcffe57590244a
Git branch:      heads/refs/tags/v5.2.1
Build timestamp: 2021-09-08 02:31:43Z
Go version:      go version go1.16.4 linux/amd64

[2021/09/30 09:54:13.876 +02:00] [INFO] [versions.go:55] ["Welcome to dumpling"] ["Release Version"=v5.2.1] ["Git Commit Hash"=4e012e54ce89fab9551e7ff58ebcffe57590244a] ["Git Branch"=heads/refs/tags/v5.2.1] ["Build timestamp"="2021-09-08 02:31:43"] ["Go Version"="go version go1.16.4 linux/amd64"]

create dumper failed: sql: SELECT version(): Error 1045: Access denied for user 'foo'@'127.0.0.1' (using password: NO)
Error: run `/home/dvaneeden/.tiup/components/dumpling/v5.2.1/dumpling` (wd:/home/dvaneeden/.tiup/data/SkTl80D) failed: exit status 1
[dvaneeden@dve-carbon ~]$ tiup dumpling -- -u foo
Starting component `dumpling`: /home/dvaneeden/.tiup/components/dumpling/v5.2.1/dumpling -- -u foo
Release version: v5.2.1
Git commit hash: 4e012e54ce89fab9551e7ff58ebcffe57590244a
Git branch:      heads/refs/tags/v5.2.1
Build timestamp: 2021-09-08 02:31:43Z
Go version:      go version go1.16.4 linux/amd64


meet some unparsed arguments, please check again: [-u foo]
Error: run `/home/dvaneeden/.tiup/components/dumpling/v5.2.1/dumpling` (wd:/home/dvaneeden/.tiup/data/SkTlB5U) failed: exit status 1

[dveeden]

Looks like pingcap/tiup#1549 might have caused tiup to pass the -- to the component it is calling

[dveeden]

diff --git a/pkg/exec/run.go b/pkg/exec/run.go
index 0d19614..4450da8 100644
--- a/pkg/exec/run.go
+++ b/pkg/exec/run.go
@@ -261,6 +261,12 @@ func launchComponent(ctx context.Context, component string, version utils.Versio
                instanceDir = env.LocalPath(localdata.DataParentDir, tag)
        }
 
+       if len(args) > 0 {
+               if args[0] == "--" {
+                       args = args[1:]
+               }
+       }
+
        params := &PrepareCommandParams{
                Ctx:         ctx,
                Component:   component,

This patch for tiup seems to restore the way -- works

[chengshihard]

That's a tiup issue. Try:

tiup -- dumpling -h host

thx is work for me

[lichunzhu]

Close this now because it's a tiup issue.