Fix okteto endpoints namespace detection from okteto.yml

cmd/context/utils.go

@@ -224,7 +224,7 @@ func LoadStackWithContext(ctx context.Context, name, namespace string, stackPath
 }
 
 // LoadContextFromPath initializes the okteto context taking into account command flags and manifest namespace/context fields
-func LoadContextFromPath(ctx context.Context, namespace, k8sContext, path string) error {
+func LoadContextFromPath(ctx context.Context, namespace, k8sContext, path string, defaultCtxOpts Options) error {
 	ctxResource, err := getCtxResource(path)
 	if err != nil {
 		return err
@@ -238,11 +238,9 @@ func LoadContextFromPath(ctx context.Context, namespace, k8sContext, path string
 		return err
 	}
 
-	ctxOptions := &Options{
-		Context:   ctxResource.Context,
-		Namespace: ctxResource.Namespace,
-		Show:      true,
-	}
+	ctxOptions := defaultCtxOpts
+	ctxOptions.Context = ctxResource.Context
+	ctxOptions.Namespace = ctxResource.Namespace
 
-	return NewContextCommand().Run(ctx, ctxOptions)
+	return NewContextCommand().Run(ctx, &ctxOptions)
 }

cmd/deploy/deploy.go

@@ -172,7 +172,7 @@ func Deploy(ctx context.Context, at analyticsTrackerInterface, ioCtrl *io.Contro
 			}
 
 			// Loads, updates and uses the context from path. If not found, it creates and uses a new context
-			if err := contextCMD.LoadContextFromPath(ctx, options.Namespace, options.K8sContext, options.ManifestPath); err != nil {
+			if err := contextCMD.LoadContextFromPath(ctx, options.Namespace, options.K8sContext, options.ManifestPath, contextCMD.Options{Show: true}); err != nil {
 				if err.Error() == fmt.Errorf(oktetoErrors.ErrNotLogged, okteto.CloudURL).Error() {
 					return err
 				}

cmd/deploy/endpoints.go

@@ -22,7 +22,6 @@ import (
 	"strings"
 
 	contextCMD "github.com/okteto/okteto/cmd/context"
-	"github.com/okteto/okteto/cmd/utils"
 	"github.com/okteto/okteto/pkg/devenvironment"
 	"github.com/okteto/okteto/pkg/endpoints"
 	oktetoErrors "github.com/okteto/okteto/pkg/errors"
@@ -32,6 +31,7 @@ import (
 	"github.com/okteto/okteto/pkg/log/io"
 	"github.com/okteto/okteto/pkg/model"
 	"github.com/okteto/okteto/pkg/okteto"
+	"github.com/spf13/afero"
 	"github.com/spf13/cobra"
 )
 
@@ -83,6 +83,7 @@ func NewEndpointGetter(k8sLogger *io.K8sLogger) (EndpointGetter, error) {
 // Endpoints deploys the okteto manifest
 func Endpoints(ctx context.Context, k8sLogger *io.K8sLogger) *cobra.Command {
 	options := &EndpointsOptions{}
+	fs := afero.NewOsFs()
 	cmd := &cobra.Command{
 		Use:   "endpoints",
 		Short: "Show endpoints for an environment",
@@ -93,32 +94,22 @@ func Endpoints(ctx context.Context, k8sLogger *io.K8sLogger) *cobra.Command {
 					return err
 				}
 				options.ManifestPath = model.GetManifestPathFromWorkdir(options.ManifestPath, workdir)
-			}
-
-			ctxResource, err := utils.LoadManifestContext(options.ManifestPath)
-			if err != nil {
-				if oktetoErrors.IsNotExist(err) {
-					ctxResource = &model.ContextResource{}
+				// check whether the manifest file provided by -f exists or not
+				if _, err := fs.Stat(options.ManifestPath); err != nil {
+					return oktetoErrors.UserError{
+						E:    fmt.Errorf("the okteto manifest file '%s' does not exist", options.ManifestPath),
+						Hint: "Check the path to the okteto manifest file",
+					}
 				}
 			}
 
-			if err := ctxResource.UpdateNamespace(options.Namespace); err != nil {
-				return err
-			}
-
-			if err := ctxResource.UpdateContext(options.K8sContext); err != nil {
-				return err
-			}
-
-			ctxOptions := &contextCMD.Options{
-				Context:   ctxResource.Context,
-				Namespace: ctxResource.Namespace,
-			}
-			if options.Output == "" {
-				ctxOptions.Show = true
-			}
-			if err := contextCMD.NewContextCommand().Run(ctx, ctxOptions); err != nil {
-				return err
+			// false for 'json' and 'md' to avoid breaking their syntax
+			showCtxHeader := options.Output == ""
+			// Loads, updates and uses the context from path. If not found, it creates and uses a new context
+			if err := contextCMD.LoadContextFromPath(ctx, options.Namespace, options.K8sContext, options.ManifestPath, contextCMD.Options{Show: showCtxHeader}); err != nil {
+				if err := contextCMD.NewContextCommand().Run(ctx, &contextCMD.Options{Namespace: options.Namespace, Show: showCtxHeader}); err != nil {
+					return err
+				}
 			}
 
 			eg, err := NewEndpointGetter(k8sLogger)

cmd/destroy/destroy.go

@@ -132,7 +132,7 @@ func Destroy(ctx context.Context, at analyticsTrackerInterface, ioCtrl *io.Contr
 				}
 				options.ManifestPath = uptManifestPath
 			}
-			if err := contextCMD.LoadContextFromPath(ctx, options.Namespace, options.K8sContext, options.ManifestPath); err != nil {
+			if err := contextCMD.LoadContextFromPath(ctx, options.Namespace, options.K8sContext, options.ManifestPath, contextCMD.Options{Show: true}); err != nil {
 				if err.Error() == fmt.Errorf(oktetoErrors.ErrNotLogged, okteto.CloudURL).Error() {
 					return err
 				}