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

periodic sync upstream KF to midstream ODH #32

Merged

Conversation

tarilabs and others added 7 commits March 18, 2024 14:17
... replaces quay.io use in CI/CD/container-registry-use
which was available originally through quay.io

Signed-off-by: Matteo Mortari <[email protected]>
* py: provide default URI builder for S3 in register_model

Signed-off-by: Isabella Basso do Amaral <[email protected]>

* py: provide overloads for type-checking register_model

Signed-off-by: Isabella Basso do Amaral <[email protected]>

* py: verify overload signatures at runtime

Signed-off-by: Isabella Basso do Amaral <[email protected]>

---------

Signed-off-by: Isabella Basso do Amaral <[email protected]>
* add Kustomize 5.2.1 in Kind GHA testing

Signed-off-by: Matteo Mortari <[email protected]>

* fix GHA syntax for dependent env

Signed-off-by: Matteo Mortari <[email protected]>

* use kustomize install binary instead of dedicated GHA step

Signed-off-by: Matteo Mortari <[email protected]>

* kustomize seems installed by default in GHA

Signed-off-by: Matteo Mortari <[email protected]>

---------

Signed-off-by: Matteo Mortari <[email protected]>
...handle gin-generated code by openapi-codegen internally,
so to make propert string ptr semantic as expected by core lib,
as a query param might not even be passed during requests

Signed-off-by: Matteo Mortari <[email protected]>
…o 24.0.9+incompatible (kubeflow#47)

Bumps [github.com/docker/docker](https://github.com/docker/docker) from 24.0.7+incompatible to 24.0.9+incompatible.
- [Release notes](https://github.com/docker/docker/releases)
- [Commits](moby/moby@v24.0.7...v24.0.9)

---
updated-dependencies:
- dependency-name: github.com/docker/docker
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
@tarilabs tarilabs requested a review from a team March 26, 2024 08:16
@openshift-ci openshift-ci bot requested review from lampajr and rareddy March 26, 2024 08:16
@codecov-commenter
Copy link

codecov-commenter commented Mar 26, 2024

Codecov Report

Attention: Patch coverage is 67.94872% with 25 lines in your changes are missing coverage. Please review.

Project coverage is 76.09%. Comparing base (bb0a432) to head (0ebf88d).

Files Patch % Lines
clients/python/src/model_registry/_utils.py 53.19% 20 Missing and 2 partials ⚠️
clients/python/src/model_registry/utils.py 89.28% 0 Missing and 3 partials ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #32      +/-   ##
==========================================
- Coverage   76.43%   76.09%   -0.35%     
==========================================
  Files          17       19       +2     
  Lines        2054     2129      +75     
  Branches       74       93      +19     
==========================================
+ Hits         1570     1620      +50     
- Misses        299      319      +20     
- Partials      185      190       +5     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link

openshift-ci bot commented Mar 26, 2024

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: rkubis, tarilabs

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@openshift-merge-bot openshift-merge-bot bot merged commit dd093b7 into opendatahub-io:main Mar 26, 2024
11 checks passed
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.

4 participants