Skip to content
This repository has been archived by the owner on Jul 11, 2023. It is now read-only.

Commit

Permalink
test(e2e): Enable priv init container for OpenShift (#3480)
Browse files Browse the repository at this point in the history
Signed-off-by: Kalya Subramanian <kasubra@microsoft.com>
  • Loading branch information
ksubrmnn authored Jun 1, 2021
1 parent 0e5db70 commit ed30fa8
Showing 1 changed file with 11 additions and 1 deletion.
12 changes: 11 additions & 1 deletion tests/framework/common.go
Original file line number Diff line number Diff line change
Expand Up @@ -115,6 +115,8 @@ var (
defaultOSMLogLevel = "trace"
// Test folder base default value
testFolderBase = "/tmp"
// default enable privileged init container
defaultEnablePrivilegedInitContainer = false
)

// OSMDescribeInfo is a struct to represent the Tier and Bucket of a given e2e test
Expand Down Expand Up @@ -440,10 +442,16 @@ type InstallOSMOpts struct {
EnableDebugServer bool

SetOverrides []string

EnablePrivilegedInitContainer bool
}

// GetOSMInstallOpts initializes install options for OSM
func (td *OsmTestData) GetOSMInstallOpts() InstallOSMOpts {
enablePrivilegedInitContainer := defaultEnablePrivilegedInitContainer
if td.DeployOnOpenShift {
enablePrivilegedInitContainer = true
}
return InstallOSMOpts{
ControlPlaneNS: td.OsmNamespace,
CertManager: defaultCertManager,
Expand All @@ -467,6 +475,8 @@ func (td *OsmTestData) GetOSMInstallOpts() InstallOSMOpts {
OSMLogLevel: defaultOSMLogLevel,
EnableDebugServer: defaultEnableDebugServer,
SetOverrides: []string{},

EnablePrivilegedInitContainer: enablePrivilegedInitContainer,
}
}

Expand Down Expand Up @@ -598,6 +608,7 @@ func (td *OsmTestData) InstallOSM(instOpts InstallOSMOpts) error {
fmt.Sprintf("OpenServiceMesh.deployPrometheus=%v", instOpts.DeployPrometheus),
fmt.Sprintf("OpenServiceMesh.deployJaeger=%v", instOpts.DeployJaeger),
fmt.Sprintf("OpenServiceMesh.enableFluentbit=%v", instOpts.DeployFluentbit),
fmt.Sprintf("OpenServiceMesh.enablePrivilegedInitContainer=%v", instOpts.EnablePrivilegedInitContainer),
)

switch instOpts.CertManager {
Expand Down Expand Up @@ -1615,7 +1626,6 @@ func (td *OsmTestData) AddOpenShiftSCC(scc, serviceAccount, namespace string) er
if !td.DeployOnOpenShift {
return errors.Errorf("Tests are not configured for OpenShift. Try again with -deployOnOpenShift=true")
}
//oc adm policy add-scc-to-user privileged -z bookbuyer -n "$BOOKBUYER_NAMESPACE"
args := []string{"adm", "policy", "add-scc-to-user", scc, "-z", serviceAccount, "-n", namespace}
stdout, stderr, err := td.RunLocal("oc", args)
if err != nil {
Expand Down

0 comments on commit ed30fa8

Please sign in to comment.