Skip to content
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

feat: Integration tests for set of Seldon rocks #150

Merged
merged 5 commits into from
Jun 15, 2023

Conversation

i-chvets
Copy link
Contributor

NOTE: This PR description only contains information related to code changes and testing, if required. All detailed information about problem statement, design, implementation, technical discussions, etc. is tracked in corresponding Github issue indicated below. This will ensure that important information is reliably recorded and tracked and not scattered across PR(s).

Details
Refer to the following Github issue for more information on feature/fix that this PR is related to:
#133

Summary of changes:

  • Added integration test for Seldon prepackages servers: xgboost v1 and v2, mlflowserver v1.
  • Remove tests fails intermittently.

#133

Summary of changes:
- Added integration test for Seldon prepackages servers: xgboost v1 and
  v2, mlflowserver v1.
- Remove tests fails intermittently.
#133

Summary of changes:
- Bump resources for mlflowserver tests.
tests/integration/test_charm.py Outdated Show resolved Hide resolved
Ivan Chvets added 3 commits June 14, 2023 06:55
#133

Summary of changes:
- Use lightkube for deletion and testing.
- Verify remove test when juju/python-libjuju#877 is resolved.
#133

Summary of changes:
- Updated integration tests with server names to retrieve proper image
  name for testing.
- Added test for removal of workload deployed configmap.
Summary of changes:
- Updated workflow to remove pin of Juju agent version and use of
  2.9/stable channel.
- Reverted back to using remove_application() from model instead of
  subprocess.
@i-chvets
Copy link
Contributor Author

Updated with requested changes.

@i-chvets i-chvets merged commit 6113f66 into main Jun 15, 2023
@i-chvets i-chvets deleted the kf-2363-feat-rocks-integration-tests branch June 15, 2023 19:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants