Ansible fail early when path in api URL (#4987)

* modified files

Signed-off-by: laxmikantbpandhare <[email protected]>

* fix for the path issue - #4980

Signed-off-by: laxmikantbpandhare <[email protected]>

* added changelog

Signed-off-by: laxmikantbpandhare <[email protected]>

* worked on review comments

Signed-off-by: laxmikantbpandhare <[email protected]>

* testdata

Signed-off-by: laxmikantbpandhare <[email protected]>

* modified error

Signed-off-by: laxmikantbpandhare <[email protected]>

* modified error

Signed-off-by: laxmikantbpandhare <[email protected]>

* modified changelog

Signed-off-by: laxmikantbpandhare <[email protected]>

* changelog change

Signed-off-by: laxmikantbpandhare <[email protected]>

* Update internal/cmd/ansible-operator/run/cmd.go

Signed-off-by: Fabian von Feilitzsch <[email protected]>

Co-authored-by: Fabian von Feilitzsch <[email protected]>

Author: laxmikantbpandhare

changelog/fragments/ansible-path-issue-fix.yaml

@@ -0,0 +1,9 @@
+entries:
+  - description: >
+      For Ansible operators, if an API endpoint has path component in it then the ansible-operator binary will terminate early with an error, due to a bug in the proxy that would cause the operator to target the wrong cluster.
+    kind: change
+    breaking: true
+    migration:
+      header: The ansible-operator binary will reject the kubeconfig if the server URL contains a path.
+      body: >
+        There is currently no workaround other than running the operator as a pod in the cluster (where it will use the internal endpoint). The fix for this issue is currently blocked waiting on a fix to the apimachinery package. Once the upstream issue is merged then a proper fix will be done for ansible-operator. Work on this issue is being tracked here: https://github.com/operator-framework/operator-sdk/issues/4925

internal/cmd/ansible-operator/run/cmd.go

@@ -18,6 +18,7 @@ import (
 	"errors"
 	"flag"
 	"fmt"
+	"net/url"
 	"os"
 	"runtime"
 	"strconv"
@@ -103,6 +104,18 @@ func run(cmd *cobra.Command, f *flags.Flags) {
 		os.Exit(1)
 	}
 
+	urlPath, err := url.Parse(cfg.Host)
+
+	if err != nil {
+		log.Error(err, "Failed to Parse the Path URL.")
+		os.Exit(1)
+	}
+
+	if urlPath != nil && urlPath.Path != "" {
+		log.Error(fmt.Errorf("api endpoint '%s' contains a path component, which the proxy server is currently unable to handle properly. Work on this issue is being tracked here: https://github.com/operator-framework/operator-sdk/issues/4925", cfg.Host), "")
+		os.Exit(1)
+	}
+
 	// TODO(2.0.0): remove
 	// Deprecated: OPERATOR_NAME environment variable is an artifact of the
 	// legacy operator-sdk project scaffolding. Flag `--leader-election-id`