Use auth delegation to avoid oauth form parsing

Signed-off-by: Ruben Vargas <[email protected]> Signed-off-by: Ruben Vargas <[email protected]>
jaegertracing · Jan 14, 2020 · dc777fc · dc777fc
1 parent 853b677
commit dc777fc
Show file tree

Hide file tree

Showing 4 changed files with 157 additions and 238 deletions.
diff --git a/.ci/run-e2e-tests.sh b/.ci/run-e2e-tests.sh
@@ -42,12 +42,6 @@ then
 elif [ "${TEST_GROUP}" = "es-token-propagation" ]
 then
     echo "Running token propagation tests"
-    oc create user user-test-token
-    oc adm policy add-cluster-role-to-user cluster-admin user-test-token
-    # for ocp 4.2
-    htpasswd -c -B -b users.htpasswd user-test-token any
-    oc delete secret htpass-secret -n openshift-config
-    oc create secret generic htpass-secret --from-file=htpasswd=./users.htpasswd -n openshift-config
     make e2e-tests-token-propagation-es
 else
     echo "Unknown TEST_GROUP [${TEST_GROUP}]"; exit 1

diff --git a/deploy/crds/jaegertracing.io_jaegers_crd.yaml b/deploy/crds/jaegertracing.io_jaegers_crd.yaml
@@ -9434,6 +9434,10 @@ spec:
                     sar:
                       type: string
                   type: object
+                options:
+                  description: Options defines a common options parameter to the different
+                    structs
+                  type: object
                 resources:
                   description: ResourceRequirements describes the compute resource
                     requirements.

diff --git a/pkg/strategy/production.go b/pkg/strategy/production.go
@@ -24,7 +24,7 @@ import (
 	"github.com/jaegertracing/jaeger-operator/pkg/storage"
 )
 
-func newProductionStrategy(ctx context.Context, jaeger *v1.Jaeger) {
+func newProductionStrategy(ctx context.Context, jaeger *v1.Jaeger) S {
 	tracer := global.TraceProvider().GetTracer(v1.ReconciliationTracer)
 	ctx, span := tracer.Start(ctx, "newProductionStrategy")
 	defer span.End()
@@ -115,25 +115,6 @@ func newProductionStrategy(ctx context.Context, jaeger *v1.Jaeger) {
 		}
 		for i := range esRollover {
 			jobs = append(jobs, &esRollover[i].Spec.JobTemplate.Spec.Template.Spec)
-
-		err := es.CreateCerts()
-		if err != nil {
-			jaeger.Logger().WithError(err).Error("failed to create Elasticsearch certificates, Elasticsearch won't be deployed")
-		} else {
-			c.secrets = es.ExtractSecrets()
-			c.elasticsearches = append(c.elasticsearches, *es.Elasticsearch())
-
-			es.InjectStorageConfiguration(&queryDep.Spec.Template.Spec)
-			es.InjectStorageConfiguration(&cDep.Spec.Template.Spec)
-			if indexCleaner != nil {
-				es.InjectSecretsConfiguration(&indexCleaner.Spec.JobTemplate.Spec.Template.Spec)
-			}
-			for i := range esRollover {
-				es.InjectSecretsConfiguration(&esRollover[i].Spec.JobTemplate.Spec.Template.Spec)
-			}
-			for i := range c.dependencies {
-				es.InjectSecretsConfiguration(&c.dependencies[i].Spec.Template.Spec)
-			}
 		}
 		autoProvisionElasticsearch(&c, jaeger, jobs, []*appsv1.Deployment{queryDep, cDep})
 	}