You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Invoking of porchctl repo reg without -n/--namespace fails
The intent is to register a new Git repo in the current namespace
$ porchctl repo reg https://gitlab.myprivaterepo.com/dimitar.georgievski/my-app.git --branch master
Error: an empty namespace may not be set when a resource name is provided
The same error is displayed in the case of porchctl repo unreg invocation
$ porchctl repo unreg my-app
Error: an empty namespace may not be set when a resource name is provided
While reviewing the fix for Porch support of private Git repos
#488
nephio-project/porch#58
I came across the following potential bugs in
porchctl repo
commandThe result of
porchctl repo get -A/--all-namespaces
misses the NAMESPACE columnA simple fix for adding the missing column could be like
https://github.com/dgeorgievski/porch/blob/fix-porchctl-repo/pkg/cli/commands/repo/get/command.go#L84-L91
Invoking of
porchctl repo reg
without-n/--namespace
failsThe intent is to register a new Git repo in the current namespace
$ porchctl repo reg https://gitlab.myprivaterepo.com/dimitar.georgievski/my-app.git --branch master Error: an empty namespace may not be set when a resource name is provided
The same error is displayed in the case of
porchctl repo unreg
invocation$ porchctl repo unreg my-app Error: an empty namespace may not be set when a resource name is provided
Again, these simple fixes would help address the issues
repo reg
: https://github.com/dgeorgievski/porch/blob/fix-porchctl-repo/pkg/cli/commands/repo/reg/command.go#L93-L100repo unreg
: https://github.com/dgeorgievski/porch/blob/fix-porchctl-repo/pkg/cli/commands/repo/unreg/command.go#L75-L82The text was updated successfully, but these errors were encountered: