updated the get_catalog.sh to check weather maven is present or not.#6642
updated the get_catalog.sh to check weather maven is present or not.#6642anandyadav3559 wants to merge 3 commits into
Conversation
…if not prompt user to isntall it. during make build I came around this issue and system default promopt is not very understandable
There was a problem hiding this comment.
issue: #6631
Motivation: Right now we leverage the presence of the make target to perform installation of the older operator version during the upgrade E2E test. This forces the test to pull the entire GitHub project locally just to run the Makefile. We can instead install directly from the tag via a Kustomize procedure, which is faster and cleaner.
Modifications:
Removed the logic in e2e/install/upgrade/upgrade_test.go that creates a temporary directory, executes git clone, checks out the target tag, and runs make install-k8s-global.
Replaced it with a temporary kustomization.yaml that dynamically applies the generated test namespace alongside the remote GitHub repository base (github.com/apache/camel-k/install/overlays/kubernetes/descoped?ref=v).
Executed installation using kubectl apply -k --server-side.
Testing Verification:
✅ make fmt and make lint completed successfully with no issues.
There was a problem hiding this comment.
Please, don't mix commits and stick to the scope of this PR. If you want, feel free to open a separated PR to address any other fix. For the upgrade test, mind that this is already fixed by another running PR, so, no need to work on it.
The approved code was modified. Please, change it accordingly.
if maven not installed prompt user to install it. during make build I came around this issue and system default promopt is not very understandable.