You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@camel.apache.org by pc...@apache.org on 2023/02/27 17:30:14 UTC

[camel-k] 10/19: fix: remove local operator installation

This is an automated email from the ASF dual-hosted git repository.

pcongiusti pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/camel-k.git

commit aeeb397ffcfd0f1f1a8638ad6e91e2bc7a306bf9
Author: Pasquale Congiusti <pa...@gmail.com>
AuthorDate: Thu Feb 23 10:50:34 2023 +0100

    fix: remove local operator installation
---
 e2e/README.md               | 48 +++++++++++++++++++++++++++++++++++++++++++++
 e2e/knative/kamelet_test.go |  2 +-
 e2e/knative/knative_test.go |  2 --
 3 files changed, 49 insertions(+), 3 deletions(-)

diff --git a/e2e/README.md b/e2e/README.md
new file mode 100644
index 000000000..c38203b7c
--- /dev/null
+++ b/e2e/README.md
@@ -0,0 +1,48 @@
+# Camel K End To End tests
+
+This directory contains the suite of test that are run on a CI to ensure the stability of the product and no regression are introduced at each PR. The full documentation can be found at https://camel.apache.org/camel-k/next/contributing/e2e.html
+
+## Structure of the directory
+
+NOTE: dear contributor, please, keep this organization as clean as you can, updating any documentation if any change is done.
+
+* builder
+* common
+* commonwithcustominstall
+* install
+* knative
+* native
+* telemetry
+* yaks
+
+### Builder
+
+Contains a basic set of tests required to validate each builder strategy we offer. Ideally we don't want to test complex features but only a few test to validate any builder we offer is working correctly.
+
+### Common
+
+Full set of test to validate the main project feature. This test will assume the presence of a namespaced operator (installation provided by the same test execution suite). Insert here any test that has to validate any new feature required.
+
+### Commonwithcustominstall
+
+Additional set of test that cover the main common features but that requires some particular operator configuration. In this test suite you must take care of installing the operator as well.
+
+### Install
+
+Test suite that cover the different installation procedures we offer and any upgrade scenario.
+
+### KNative
+
+Test suite that cover the features associated with KNative. This test will assume the presence of a namespaced operator (installation provided by the same test execution suite) togheter with KNative operator configuration.
+
+### Native
+
+Test suite that cover the Quarkus Native build. As it is high resource consuming, we just validate that a native build for the supported DSLs is working.
+
+### Telemetry
+
+Test suite that cover the features associated with Telemetry feature. The test execution takes care of installing the required configuration.
+
+### Yaks
+
+Test suite that cover certain KNative features togheter with YAKS operator.
diff --git a/e2e/knative/kamelet_test.go b/e2e/knative/kamelet_test.go
index 8e48a8710..8c236ea0c 100644
--- a/e2e/knative/kamelet_test.go
+++ b/e2e/knative/kamelet_test.go
@@ -70,7 +70,7 @@ func TestKameletChange(t *testing.T) {
 
 	Eventually(IntegrationPodPhase(ns, timerBinding), TestTimeoutLong).Should(Equal(corev1.PodRunning))
 	Eventually(IntegrationConditionStatus(ns, "timer-binding", v1.IntegrationConditionReady), TestTimeoutShort).Should(Equal(corev1.ConditionTrue))
-	Eventually(IntegrationLogs(ns, "display"), TestTimeoutShort).Should(ContainSubstring("message is Hello"))
+	Eventually(IntegrationLogs(ns, "test-kamelet-display"), TestTimeoutShort).Should(ContainSubstring("message is Hello"))
 
 	Eventually(KameletBindingCondition(ns, timerBinding, v1alpha1.KameletBindingConditionReady), TestTimeoutMedium).Should(And(
 		WithTransform(KameletBindingConditionStatusExtract, Equal(corev1.ConditionTrue)),
diff --git a/e2e/knative/knative_test.go b/e2e/knative/knative_test.go
index a9810564a..7bf04010f 100644
--- a/e2e/knative/knative_test.go
+++ b/e2e/knative/knative_test.go
@@ -42,8 +42,6 @@ func TestKnative(t *testing.T) {
 	knChannelWords := "words"
 	Expect(CreateKnativeChannel(ns, knChannelMessages)()).To(Succeed())
 	Expect(CreateKnativeChannel(ns, knChannelWords)()).To(Succeed())
-	operatorID := fmt.Sprintf("camel-k-%s", ns)
-	Expect(KamelInstallWithID(operatorID, ns, "--trait-profile", "knative").Execute()).To(Succeed())
 
 	t.Run("Service combo", func(t *testing.T) {
 		Expect(KamelRunWithID(operatorID, ns, "files/knative2.groovy").Execute()).To(Succeed())