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

Added Contributing file #219

Merged
merged 2 commits into from
Oct 10, 2023
Merged

Conversation

ColmBhandal
Copy link
Contributor

No description provided.

@ColmBhandal ColmBhandal requested a review from a team as a code owner October 4, 2023 15:34
@ColmBhandal ColmBhandal changed the title Contributing Added Contributing file Oct 4, 2023
@i-chvets
Copy link
Contributor

i-chvets commented Oct 5, 2023

Approved. The changes in this PR do not affect any functionality. Seldon CI fails due to other reasons.
#203

@ColmBhandal ColmBhandal merged commit 4bab512 into main Oct 10, 2023
6 of 8 checks passed
@ColmBhandal ColmBhandal deleted the KF-4522-Seldon-Core-Contributing-File branch October 10, 2023 09:29
NohaIhab added a commit that referenced this pull request Oct 19, 2023
update tox.ini's update-requirements (#215)

Updates tox.ini's update-requirements as described in canonical/argo-operators#100
Ref canonical/kubeflow-ci#59

doc link (#214)

tests(integration): Apply seldondeployments in namespace `default` (#220)

* tests(integration): Refactor

Refactor integration tests after hitting issue #218. This is a
workaround for the issue. At the same time, it is needed to revert those
back since up until now, integration tests were applying
seldondeployments to the namespace created by Juju. In production, this
would be `kubeflow` and we would never apply something there.

* tests(integration): Rename DEFAULT_NAMESPACE to WORKLOADS_NAMESPACE

Contributing (#219)

Co-authored-by: Orfeas Kourkakis <orfeas.kourkakis@canonical.com>

chore: bump seldon charm 1.15 -> 1.17.1 for CKF release 1.8 (#216)

* bump seldon-core-operator 1.15.0 -> 1.17.1 for CKF 1.8
* manifests: Update CRD according to upstream manifests
* manifests: Update manifests roles names
* Use upstream images 1.17.1 for seldon servers
* tests(unit): Modify expected pebble plan
* tests(integration): Update expected huggingface output
* tests(integration): Update expected mlserver-mlflow output
* contributing.md: Add update manifests instructions
* gitignore: Ignore venv directory

reqs: refresh requirements-integration.txt
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.

3 participants