Skip to content

Commit

Permalink
cleanup
Browse files Browse the repository at this point in the history
  • Loading branch information
abuchanan-airbyte committed Sep 3, 2024
1 parent 80b4951 commit ff01187
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 35 deletions.
31 changes: 0 additions & 31 deletions internal/cmd/cmd.go
Original file line number Diff line number Diff line change
Expand Up @@ -16,37 +16,6 @@ import (
"k8s.io/client-go/tools/clientcmd"
)

// Help messages to display for specific error situations.
const (
// helpAirbyteDir is displayed if ErrAirbyteDir is ever returned
helpAirbyteDir = `The ~/.airbyte directory is inaccessible.
You may need to remove this directory before trying your command again.`

// helpCluster is displayed if ErrClusterNotFound is ever returned
helpCluster = `No cluster was found. If this is unexpected,
you may need to run the "local install" command again.`

// helpDocker is displayed if ErrDocker is ever returned
helpDocker = `An error occurred while communicating with the Docker daemon.
Ensure that Docker is running and is accessible. You may need to upgrade to a newer version of Docker.
For additional help please visit https://docs.docker.com/get-docker/`

// helpKubernetes is displayed if ErrKubernetes is ever returned
helpKubernetes = `An error occurred while communicating with the Kubernetes cluster.
If this error persists, you may need to run the uninstall command before attempting to run
the install command again.`

// helpIngress is displayed if ErrIngress is ever returned
helpIngress = `An error occurred while configuring ingress.
This could be in indication that the ingress port is already in use by a different application.
The ingress port can be changed by passing the flag --port.`

// helpPort is displayed if ErrPort is ever returned
helpPort = `An error occurred while verifying if the requested port is available.
This could be in indication that the ingress port is already in use by a different application.
The ingress port can be changed by passing the flag --port.`
)

func HandleErr(err error) {
if err == nil {
return
Expand Down
4 changes: 0 additions & 4 deletions internal/cmd/local/local_install.go
Original file line number Diff line number Diff line change
Expand Up @@ -59,10 +59,6 @@ func (i *InstallCmd) Run(ctx context.Context, provider k8s.Provider, telClient t
i.Port, err = getPort(ctx, provider.ClusterName)
if err != nil {
return err
// pterm.Warning.Printfln("Unable to determine which port the existing cluster was configured to use.\n" +
// "Installation will continue but may ultimately fail, in which case it will be necessarily to uninstall first.")
// since we can't verify the port is correct, push forward with the provided port
// i.Port = providedPort
}
if providedPort != i.Port {
pterm.Warning.Printfln("The existing cluster was found to be using port %d, which differs from the provided port %d.\n"+
Expand Down

0 comments on commit ff01187

Please sign in to comment.