Skip to content

Commit

Permalink
configmap: Remove installation configmap
Browse files Browse the repository at this point in the history
Installation configmap can significantly complicate new installation
process and has multiple issues with its design (and related tests).
Remove installation congigmap
Redesign related tests or make them always skip until further
redesign would be done in the scope of different change

Refs: #2153
Signed-off-by: Konstantin Yarovoy <[email protected]>
  • Loading branch information
Konstantin Yarovoy committed Oct 7, 2024
1 parent 9713630 commit 9fcd974
Show file tree
Hide file tree
Showing 11 changed files with 143 additions and 324 deletions.
24 changes: 2 additions & 22 deletions spec/5g/ran_spec.cr
Original file line number Diff line number Diff line change
Expand Up @@ -14,31 +14,11 @@ describe "5g" do


it "'oran_e2_connection' should pass if the ORAN enabled RAN connects to the RIC using the e2 standard", tags: ["oran"] do
begin
ShellCmd.cnf_setup("cnf-config=sample-cnfs/sample_srsran_ueauth_open5gs/cnf-testsuite.yml")
ShellCmd.cnf_setup("cnf-config=sample-cnfs/sample-oran-ric/cnf-testsuite.yml")
result = ShellCmd.run_testsuite("oran_e2_connection verbose")
(/(PASSED).*(RAN connects to a RIC using the e2 standard interface)/ =~ result[:output]).should_not be_nil
ensure
result = ShellCmd.run_testsuite("cnf_cleanup cnf-config=sample-cnfs/sample_srsran_ueauth_open5gs/cnf-testsuite.yml")
result[:status].success?.should be_true
result = ShellCmd.run_testsuite("cnf_cleanup cnf-config=sample-cnfs/sample-oran-ric/cnf-testsuite.yml")
result[:status].success?.should be_true
end
# (kosstennbl) TODO: Test and specs for 'oran_e2_connection' should be redesigned. Check #2153 for more info. Spec was using sample_srsran_ueauth_open5gs and sample-oran-ric.
end

it "'oran_e2_connection' should fail if the ORAN enabled RAN does not connect to the RIC using the e2 standard", tags: ["oran"] do
begin
ShellCmd.cnf_setup("cnf-config=sample-cnfs/sample_srsran_ueauth_open5gs/cnf-testsuite.yml")
ShellCmd.cnf_setup("cnf-config=sample-cnfs/sample-oran-noric/cnf-testsuite.yml")
result = ShellCmd.run_testsuite("oran_e2_connection verbose")
(/(FAILED).*(RAN does not connect to a RIC using the e2 standard interface)/ =~ result[:output]).should_not be_nil
ensure
result = ShellCmd.run_testsuite("cnf_cleanup cnf-config=sample-cnfs/sample_srsran_ueauth_open5gs/cnf-testsuite.yml")
result[:status].success?.should be_true
result = ShellCmd.run_testsuite("cnf_cleanup cnf-config=sample-cnfs/sample-oran-noric/cnf-testsuite.yml")
result[:status].success?.should be_true
end
# (kosstennbl) TODO: Test and specs for 'oran_e2_connection' should be redesigned. Check #2153 for more info. Spec was using sample_srsran_ueauth_open5gs and sample-oran-noric.
end

end
2 changes: 1 addition & 1 deletion spec/workload/installability_spec.cr
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@ describe CnfTestSuite do
ShellCmd.cnf_setup("cnf-path=./sample-cnfs/k8s-non-helm")
result = ShellCmd.run_testsuite("helm_deploy verbose")
result[:status].success?.should be_true
(/(FAILED).*(Helm deploy failed)/ =~ result[:output]).should_not be_nil
(/(FAILED).*(CNF has deployemnts, that are not installed with helm)/ =~ result[:output]).should_not be_nil
ensure
result = ShellCmd.run_testsuite("cnf_cleanup cnf-path=./sample-cnfs/k8s-non-helm verbose")
end
Expand Down
29 changes: 2 additions & 27 deletions spec/workload/observability_spec.cr
Original file line number Diff line number Diff line change
Expand Up @@ -161,35 +161,10 @@ describe "Observability" do
end

it "'tracing' should fail if tracing is not used", tags: ["observability_jaeger_fail"] do
Log.info { "Installing Jaeger " }
JaegerManager.install

ShellCmd.cnf_setup("cnf-config=sample-cnfs/sample-coredns-cnf/cnf-testsuite.yml")
result = ShellCmd.run_testsuite("tracing")
(/(FAILED).*(Tracing not used)/ =~ result[:output]).should_not be_nil
ensure
result = ShellCmd.run_testsuite("cnf_cleanup cnf-config=sample-cnfs/sample-coredns-cnf/cnf-testsuite.yml")
JaegerManager.uninstall
KubectlClient::Get.resource_wait_for_uninstall("Statefulset", "jaeger-cassandra")
KubectlClient::Get.resource_wait_for_uninstall("Deployment", "jaeger-collector")
KubectlClient::Get.resource_wait_for_uninstall("Deployment", "jaeger-query")
KubectlClient::Get.resource_wait_for_uninstall("Daemonset", "jaeger-agent")
# (kosstennbl) TODO: Test and specs for 'tracing' should be redesigned. Check #2153 for more info. Spec was using sample-coredns-cnf CNF.
end

it "'tracing' should pass if tracing is used", tags: ["observability_jaeger_pass"] do
Log.info { "Installing Jaeger " }
JaegerManager.install

ShellCmd.cnf_setup("cnf-config=sample-cnfs/sample-tracing/cnf-testsuite.yml")
result = ShellCmd.run_testsuite("tracing")
(/(PASSED).*(Tracing used)/ =~ result[:output]).should_not be_nil
ensure
result = ShellCmd.run_testsuite("cnf_cleanup cnf-config=sample-cnfs/sample-tracing/cnf-testsuite.yml")
JaegerManager.uninstall
KubectlClient::Get.resource_wait_for_uninstall("Statefulset", "jaeger-cassandra")
KubectlClient::Get.resource_wait_for_uninstall("Deployment", "jaeger-collector")
KubectlClient::Get.resource_wait_for_uninstall("Deployment", "jaeger-query")
KubectlClient::Get.resource_wait_for_uninstall("Daemonset", "jaeger-agent")
# (kosstennbl) TODO: Test and specs for 'tracing' should be redesigned. Check #2153 for more info. Spec was using sample-tracing CNF.
end

end
Loading

0 comments on commit 9fcd974

Please sign in to comment.