test: Addition of ginkgo framework dependencies and TLS propagation test#409
test: Addition of ginkgo framework dependencies and TLS propagation test#409kaleemsiddiqu wants to merge 2 commits intoopenshift:masterfrom
Conversation
48a0740 to
5501ab7
Compare
|
Local run is successful ... |
|
@ricardomaraschini @ingvagabund @gangwgr please review this. |
cmd/cluster-openshift-controller-manager-operator-tests-ext/main.go
Outdated
Show resolved
Hide resolved
cmd/cluster-openshift-controller-manager-operator-tests-ext/main.go
Outdated
Show resolved
Hide resolved
cmd/cluster-openshift-controller-manager-operator-tests-ext/main.go
Outdated
Show resolved
Hide resolved
cmd/cluster-openshift-controller-manager-operator-tests-ext/main.go
Outdated
Show resolved
Hide resolved
|
make 2 commits, vendor changes should be in different commit |
8ffbd4c to
dd531db
Compare
cmd/cluster-openshift-controller-manager-operator-tests-ext/dependenciesmagnet.go
Show resolved
Hide resolved
cmd/cluster-openshift-controller-manager-operator-tests-ext/main.go
Outdated
Show resolved
Hide resolved
|
@kaleemsiddiqu there is gap in this |
Add vendor dependencies for the Ginkgo testing framework, openshift-tests-extension framework, and testify assertion library Signed-off-by: Kaleemullah Siddiqui <ksiddiqu@redhat.com>
dd531db to
0f02844
Compare
For this we have to create a test pod and try it from there otherwise this would fail as we are running test from outside of the cluster and dns resolution to service in test will fail. I think check on config propagation and operator status is sufficient to verify the issue. |
you don't need to create new pod try with ocm pods similar case we have in test-private repo in which we use kas pods |
Implement end-to-end test to verify that TLS security profile changes propagate from the APIServer to the OpenShift Controller Manager. Signed-off-by: Kaleemullah Siddiqui <ksiddiqu@redhat.com>
0f02844 to
fb545e3
Compare
@gangwgr |
|
/lgtm |
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: gangwgr, kaleemsiddiqu The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
|
@kaleemsiddiqu: all tests passed! Full PR test history. Your PR dashboard. DetailsInstructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here. |
|
/hold |
Test for changes done in #407