-
Notifications
You must be signed in to change notification settings - Fork 142
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Install GitOps Dashboard for gitops beta run #2469
Install GitOps Dashboard for gitops beta run #2469
Conversation
dab6a8c
to
26d958e
Compare
…hash. Add generating manifests for the GitOps Dashboard installation. Split `run.go` to several files.
…laying the prompt.
Add dashboard port flag. Add "Y" as a default value to the prompt.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM in general
thank you, @opudrovs 🥇
26d958e
to
dd2ed42
Compare
} | ||
|
||
// IsDashboardInstalled checks if the GitOps Dashboard is installed. | ||
func IsDashboardInstalled(log logger.Logger, ctx context.Context, kubeClient *kube.KubeHTTP) bool { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A small glitch here is that someone might install Weave GitOps with a different method.
But we can fix it later when needed.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Would another method also require them to use the same HelmChart? Is it possible to install the dashboard without this HelmChart? Or will the chart be the same just its name will be different?
I mean if using a HelmChart is required for installing the dashboard can we just use something else (not the name) for detecting that this HelmChart is present reliably?
@chanwit great, thank you! 🌞 |
Closes #2358
pkg/run/run.go
to separate files, because the existing file was getting huge.--dashboard-port
flag with default value 9001 and opening this port automatically for the dashboard.Notes: