lakectl autocomplete nouns

[nopcoder]

Have the completions code complete our "nouns" when relevant.
Example:

lakectl fs upload --source <tab> // should suggest files
lakectl repo delete <tab> // should suggest repositories names

[kichloo]

@nopcoder Though I am beginner in this, I can complete it with some help. So, I would like to take this issue.

[nopcoder]

Sure @kichloo, let me know if you have questions about the issue or the code.

[kichloo]

Yes, kindly help me regarding it!

[nopcoder]

@kichloo we are using Cobra, here you can find information about the current completion - https://github.com/spf13/cobra/blob/master/shell_completions.md
There you can also find information on how to add "Completion of nouns" to each command we mapped.
Our documentation https://docs.lakefs.io/reference/commands.html#commands-cli-reference holds the reference of all the command, but you can also find the same in the standard project structure based on Cobra.

[avats-dev]

@kichloo are you still working on this one? Let me know, I want to try this one.

[johnnyaug]

Hey @kichloo, still working on this?

[kichloo]

The issue is now open!

[fumblehool]

Hi, I can work on this if nobody is currently working on it.

[nopcoder]

@fumblehool still interested?

[fumblehool]

@nopcoder Unfortunately, I'm unable to work on it right now.

[whoAbhishekSah]

Hello @nopcoder I would like to work on this issue. Can you assign it to me?

[arielshaqed]

Unassigning so we can try to get it moving.
Thanks to all current participants!

[siddhant94]

hi @arielshaqed , is this issue up for grabs? I would like to take a shot if unassigned.
Also, I am trying to build just the lakectl binary using Dockerfile but I run into this error

Step 8/29 : RUN go build -ldflags "-X github.com/treeverse/lakefs/pkg/version.Version=${VERSION}" -o lakectl ./cmd/lakectl
 ---> Running in 9d62437d85d4
pkg/db/migration.go:17:2: no required module provides package github.com/treeverse/lakefs/pkg/ddl; to add it:
        go get github.com/treeverse/lakefs/pkg/ddl
pkg/api/ui_handler.go:18:2: no required module provides package github.com/treeverse/lakefs/pkg/webui; to add it:
        go get github.com/treeverse/lakefs/pkg/webui

[arielshaqed]

hi @arielshaqed , is this issue up for grabs? I would like to take a shot if unassigned.

Hi @siddhant94 ,

All yours, thanks for stepping up to this one! Please let us know if you need any help. Our Slack has a #dev channel that is really good if you prefer instant-messaging conversations.

Also, I am trying to build just the lakectl binary using Dockerfile but I run into this error

This is very odd; I just built the container successfully. Could you please send me:

1. git status
2. command used to initiate the build
3. an entire log of a failed build?

In the meantime: Most of us find it more convenient not to build lakeFS inside a container (we do use the container for deployments, of course). Instead the contribution guide should walk you through installing all the tools that you would need in order to run make build test on your development machine.

Step 8/29 : RUN go build -ldflags "-X github.com/treeverse/lakefs/pkg/version.Version=${VERSION}" -o lakectl ./cmd/lakectl
 ---> Running in 9d62437d85d4
pkg/db/migration.go:17:2: no required module provides package github.com/treeverse/lakefs/pkg/ddl; to add it:
        go get github.com/treeverse/lakefs/pkg/ddl
pkg/api/ui_handler.go:18:2: no required module provides package github.com/treeverse/lakefs/pkg/webui; to add it:
        go get github.com/treeverse/lakefs/pkg/webui