-
Notifications
You must be signed in to change notification settings - Fork 1k
improve the tests with real KF profile namespaces #3093
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.
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
Merged
google-oss-prow
merged 70 commits into
kubeflow:master
from
kunal-511:fix#3054/end-to-end-integration-tests-and-restructuring
Apr 14, 2025
Merged
Changes from all commits
Commits
Show all changes
70 commits
Select commit
Hold shift + click to select a range
43fc126
Added more tests, updated the two gha workflow to test in real KF pro…
kunal-511 2838f9b
resolved the wait error
kunal-511 9c01b31
Ensuring proper storage class configuration in the KinD environment
kunal-511 f81d069
Added logs to know more about the error
kunal-511 78e3c9c
Added more comprehensive logging and diagnostics
kunal-511 b59e8b4
Removed the training operator from katib tests andusing volume api di…
kunal-511 8ff3e66
Fixed the namespace error
kunal-511 918c6cc
Added echo for debugging the error
kunal-511 ae439db
Improved error handling and diagnostics
kunal-511 99524f4
removed the failing kubectl command
kunal-511 69143ec
removed the debuggers added
kunal-511 89668db
fixed the service account token issue
kunal-511 fd66098
removed the audience flag no needed
kunal-511 c0d31be
Fixed the unauth token probelm
kunal-511 03fe07c
Added echo for logging
kunal-511 7c7c64f
Removed echo statements and reduce the timeout
kunal-511 19b12f7
Added logs to understand more about the error
kunal-511 08b19fd
Removed the echo logs
kunal-511 209a3f5
Fixed crd issue in kserve test
kunal-511 29f9690
Fixed the timeout issue
kunal-511 a01e70b
Fixed the training operator test issue
kunal-511 48aff08
Fixed the training operator test issue
kunal-511 c0be276
Fixed the namespace error in kserve test
kunal-511 1800aef
Fixed lint error
kunal-511 c0eab55
Update .github/workflows/full_kubeflow_integration_test.yaml
kunal-511 0c1f39e
Updated the names as suggested
kunal-511 6d4f922
Updated the tests with tracking the success and removing sleep
kunal-511 d484a70
Fixed deployment name
kunal-511 8280549
Added to check the actual labels
kunal-511 9b86fa4
Added the echo to check the CSRF token issue
kunal-511 caee1cd
Added the echo to check the CSRF token issue
kunal-511 3612a94
Fixed the volume test issue
kunal-511 c159d7b
Added the result checker
kunal-511 ed6dd0b
Added logs to check the issue
kunal-511 f31c997
Fixed the lint issue
kunal-511 27df90a
Fixed the volume tests
kunal-511 0a45568
fixed issues as suggested
kunal-511 996c37c
Improved the katib tests
kunal-511 7c4921f
Update katib_test.yaml
juliusvonkohout 518aee3
Update full_kubeflow_integration_test.yaml
juliusvonkohout 46cc7c5
Update test_volumes_web_app.sh
juliusvonkohout 94667ba
Update and rename test_volumes_web_app.sh to test_volumes_web_applica…
juliusvonkohout bdc3a2a
Update full_kubeflow_integration_test.yaml
juliusvonkohout de967b0
Disabled the istio injection for katib
kunal-511 4410d6b
reverted back
kunal-511 1149c0b
Update and rename volumes_web_application_test.yaml to install_volume…
juliusvonkohout 03d663c
Rename install_volumes_web_application.yaml to test_volumes_web_appli…
juliusvonkohout 96e535c
Update and rename install_volumes_web_app.sh to install_volumes_web_a…
juliusvonkohout 5721b8a
Update test_volumes_web_application.yaml
juliusvonkohout 7390745
Rename test_volumes_web_application.yaml to volumes_web_application_t…
juliusvonkohout 451771d
Update volumes_web_application_test.yaml
juliusvonkohout 4f0ec88
Update test_volumes_web_application.sh
juliusvonkohout 9b81d65
Update test_volumes_web_application.sh
juliusvonkohout a7bf7fd
Update full_kubeflow_integration_test.yaml
juliusvonkohout 8856569
Update volumes_web_application_test.yaml
juliusvonkohout d67d26f
Rename full_kubeflow_integration_test.yaml to end-to-end_integration_…
juliusvonkohout da2319f
Rename end-to-end_integration_test.yaml to full_kubeflow_integration_…
juliusvonkohout 46eb8cf
Update test_volumes_web_application.sh
juliusvonkohout c9eb690
disable injection in katib objects only
kunal-511 7f3804b
Update test_volumes_web_application.sh
juliusvonkohout f96e9b9
Update test_volumes_web_application.sh
juliusvonkohout 8062a2c
Update test_volumes_web_application.sh
juliusvonkohout cb1c200
Directly changed the file to disable injection
kunal-511 ce7fe94
added namespace directly to the test file
kunal-511 9187f36
Fixed the XSRF-Token issue
kunal-511 3700611
Changed the Json payload a liitle bit
kunal-511 7316156
Added echo to logs the error
kunal-511 13857eb
fixed the access mode values
kunal-511 c147106
Fixing 403 error in PVC
kunal-511 7d6f4a7
removed the echo which was added to check the logs
kunal-511 File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
juliusvonkohout marked this conversation as resolved.
Show resolved
Hide resolved
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,12 +1,14 @@ | ||
#!/bin/bash | ||
set -euo pipefail | ||
echo "Installing training operator ..." | ||
|
||
cd apps/training-operator/upstream | ||
kustomize build overlays/kubeflow | kubectl apply --server-side --force-conflicts -f - | ||
kubectl wait --for=condition=Ready pods --all --all-namespaces --timeout=600s \ | ||
--field-selector=status.phase!=Succeeded | ||
kubectl wait --for=condition=Available deployment/training-operator -n kubeflow --timeout=10s | ||
kubectl get crd | grep -E 'tfjobs.kubeflow.org|pytorchjobs.kubeflow.org' | ||
|
||
kubectl wait --for=condition=Available deployment/training-operator -n kubeflow --timeout=180s | ||
|
||
|
||
kubectl get deployment -n kubeflow training-operator | ||
cd - | ||
kubectl get pods -n kubeflow -l app=training-operator | ||
kubectl get crd | grep -E 'tfjobs.kubeflow.org|pytorchjobs.kubeflow.org' | ||
|
||
cd - |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,8 @@ | ||
#!/bin/bash | ||
set -euxo pipefail | ||
|
||
cd apps/volumes-web-app/upstream | ||
kustomize build overlays/istio | kubectl apply -f - | ||
cd ../../../ | ||
|
||
kubectl wait --for=condition=Available deployment/volumes-web-app-deployment -n kubeflow --timeout=180s |
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.