infinyon / fluvio-website

Website for Fluvio Project
Apache License 2.0
8 stars 32 forks source link

`www.fluvio.io/connectors/cdk/list-log/` doesn't look right and has errors #745

Closed mrasputin closed 11 months ago

mrasputin commented 11 months ago

1

image

the navigation menu

Install CDK
Generate a SmartConnector
Build and Test
Start and Shutdown
List and Logs
Publish to SmartConnector Hub

sits between Log and Changing the Log Level sections

2

the command

 $ cdk deploy --name log my-my-connector-test-connector

returns an error

❯ cdk deploy --name log my-wss-connector-test
error: unexpected argument '--name' found

  tip: 'log --name' exists

Usage: cdk deploy [OPTIONS] [-- <EXTRA_ARGUMENTS>...] <COMMAND>

For more information, try '--help'.

3

the name of the config file in command

cdk deploy start --config connector-config.yml --log-level debug

is not the same as the name of the config file created by cdk generate

❯ ls -l
total 7080
-rw-r--r-- 1 m m   86529 Dec 21 23:05 Cargo.lock
-rw-r--r-- 1 m m     445 Dec 21 23:03 Cargo.toml
-rw-r--r-- 1 m m     287 Dec 21 23:03 Connector.toml
-rw-r--r-- 1 m m      24 Dec 21 23:03 .gitignore
-rw-r--r-- 1 m m       6 Dec 21 23:03 README.md
-rw-r--r-- 1 m m     133 Dec 21 23:03 sample-config.yaml
drwxr-xr-x 1 m m      32 Dec 21 23:03 src
drwxr-xr-x 1 m m     120 Dec 21 23:05 target
-rw-r--r-- 1 m m 7135856 Dec 21 23:51 wss.log
digikata commented 11 months ago

Thanks for opening the issue @mrasputin , fix on the way

digikata commented 11 months ago

Thanks again, btw the navigation steps have a general consistency issue, fixing that in a separate PR...