Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

OCM-8679 | test: automated case id:72868 Separate the component route options when updating ingress #2157

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

OCM-8679 | test: automated case id:72868 Separate the component route options when updating ingress #2157

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
e8ecc5a
automate ingress component route changes - 72868
valeriiashapoval Jun 10, 2024
472cffe
Merge branch 'openshift:master' into automate-ingress-72868
valeriiashapoval Jul 17, 2024
d603e69
OCM-8679 | test: automated case id:72868 Separate the component route…
valeriiashapoval Jul 17, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
62 changes: 62 additions & 0 deletions tests/e2e/test_rosacli_ingress.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -9,6 +9,7 @@ import (

"github.com/openshift/rosa/tests/ci/labels"
"github.com/openshift/rosa/tests/utils/common"

"github.com/openshift/rosa/tests/utils/config"
"github.com/openshift/rosa/tests/utils/exec/rosacli"
ph "github.com/openshift/rosa/tests/utils/profilehandler"
Expand Down Expand Up @@ -403,4 +404,65 @@ var _ = Describe("Edit default ingress",
Expect(expectLabel).To(BeElementOf(ingressRouteSelectors))
}
})
It("can update ingress components (oauth, downloads, console) - [id:72868]",
labels.Medium,
labels.Runtime.Day2,
func() {

By("Record ingress default value")
output, err := rosaClient.Ingress.ListIngress(clusterID)
Expect(err).ToNot(HaveOccurred())
ingressList, err := rosaClient.Ingress.ReflectIngressList(output)
Expect(err).ToNot(HaveOccurred())
defaultIngress := ingressList.Ingresses[0]

By("Check edit ingress help message")
output, err = rosaClient.Ingress.EditIngress(clusterID, "-h")
Expect(err).ToNot(HaveOccurred())
Expect(output.String()).Should(ContainSubstring("--component-routes"))

By("Edit ingress with --component-routes")
componentRoutes := "oauth: hostname=oauth.hostname.com;tlsSecretRef=oauth-secret,downloads: hostname=downloads.hostname.com;tlsSecretRef=downloads-secret,console: hostname=console.hostname.com;tlsSecretRef=console-secret"
output, err = rosaClient.Ingress.EditIngress(clusterID,
defaultIngress.ID,
"--component-routes", componentRoutes,
)
Expect(err).ToNot(HaveOccurred())

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We usually define defer function to recover the updated value after the first edit command

_, err = rosaClient.Ingress.EditIngress(clusterID,
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Edit ingress with same parameter called twice.

defaultIngress.ID,
"--component-routes", componentRoutes,
)
Expect(err).ToNot(HaveOccurred())

By("List ingress to check")
output, err = rosaClient.Ingress.ListIngress(clusterID)
Expect(err).ToNot(HaveOccurred())
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Here just checked the error is not happening when list the ingress, but it didn't check the edited value showing


By("Edit ingress with --component-routes with incorrect syntax")
componentRoutes = "oauth: hostname:custom1;tlsSecretRef=custom1,downloads: hostname=custom2;tlsSecretRef=custom2,console: hostname=custom3;tlsSecretRef=custom3"
output, err = rosaClient.Ingress.EditIngress(clusterID,
defaultIngress.ID,
"--component-routes", componentRoutes,
)
Expect(err).To(HaveOccurred())
Expect(output.String()).Should(ContainSubstring("An error occurred whilst parsing the supplied component routes: only the name of the component should be followed by ':'"))

By("List ingress to check")
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If last steps is expecting an error happen, no need to list the ingress for double checking

output, err = rosaClient.Ingress.ListIngress(clusterID)
Expect(err).ToNot(HaveOccurred())

By("Edit ingress with --component-routes with incorrect number of components")
componentRoutes = "oauth: hostname=custom1;tlsSecretRef=custom1"
output, err = rosaClient.Ingress.EditIngress(clusterID,
defaultIngress.ID,
"--component-routes", componentRoutes,
)
Expect(err).To(HaveOccurred())
Expect(output.String()).Should(ContainSubstring("An error occurred whilst parsing the supplied component routes: the expected amount of component routes is 3, but 1 have been supplied"))

By("List ingress to check")
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Same as above

_, err = rosaClient.Ingress.ListIngress(clusterID)
Expect(err).ToNot(HaveOccurred())
})
})
1 change: 1 addition & 0 deletions tests/utils/exec/rosacli/ingress_service.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -64,6 +64,7 @@ type Ingress struct {
ExcludeNamespace string `yaml:"Exclude Namespce,omitempty" json:"EXCLUDED NAMESPACE,omitempty"`
WildcardPolicy string `yaml:"Wildcard Policy,omitempty" json:"WILDCARD POLICY,omitempty"`
NamespaceOwnershipPolicy string `yaml:"Namespace Ownership Policy,omitempty" json:"NAMESPACE OWNERSHIP,omitempty"`
ComponentRoutes string `yaml:"Component Routes,omitempty" json:"COMPONENT ROUTES,omitempty"`
}

// Get specified ingress by ingress id
Expand Down