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

Ability to configure Raw InferenceGraphs as private #521

Conversation

israel-hdez
Copy link

What this PR does / why we need it:

This adds the possibility to use the networking.kserve.io/visibility=cluster-local label to configure an InferenceGraph in Raw mode as private (not exposed).

Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):

https://issues.redhat.com/browse/RHOAIENG-20324

Type of changes
Please delete options that are not relevant.

  • New feature (non-breaking change which adds functionality)
  • This change requires a documentation update
    • It will be done in the cherry-pick to master branch.

Feature/Issue validation/testing:

Please describe the tests that you ran to verify your changes and relevant result summary. Provide instructions so it can be reproduced.
Please also list any relevant details for your test configuration.

  • Create InferenceGraph in Raw mode. Then, check that a route is created.
  • Add the networking.kserve.io/visibility=cluster-local label to the InferenceGraph. Then, check that the route is removed.
  • Logs

Checklist:

  • Have you added unit/e2e tests that prove your fix is effective or that this feature works?
  • Has code been commented, particularly in hard-to-understand areas?
  • Have you made corresponding changes to the documentation?
    • It will be done in the cherry-pick to master branch.

Copy link

openshift-ci bot commented Mar 7, 2025

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

@israel-hdez israel-hdez force-pushed the j20324-private-raw-ig branch from db7b831 to d410067 Compare March 7, 2025 17:50
Copy link

@VedantMahabaleshwarkar VedantMahabaleshwarkar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/approve

Copy link

openshift-ci bot commented Mar 7, 2025

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: israel-hdez, VedantMahabaleshwarkar

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:
  • OWNERS [VedantMahabaleshwarkar,israel-hdez]

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

This adds the possibility to use the `networking.kserve.io/visibility=cluster-local` label to configure an InferenceGraph in Raw mode as private (not exposed).

Signed-off-by: Edgar Hernández <[email protected]>
@israel-hdez israel-hdez force-pushed the j20324-private-raw-ig branch from d410067 to bb89560 Compare March 7, 2025 19:28
@israel-hdez
Copy link
Author

/retest

@VedantMahabaleshwarkar
Copy link

/lgtm

@openshift-ci openshift-ci bot added the lgtm label Mar 13, 2025
@openshift-merge-bot openshift-merge-bot bot merged commit 8837d5f into opendatahub-io:release-v0.14 Mar 13, 2025
27 checks passed
israel-hdez added a commit to israel-hdez/kserve that referenced this pull request Mar 14, 2025
This adds the possibility to use the `networking.kserve.io/visibility=cluster-local` label to configure an InferenceGraph in Raw mode as private (not exposed).

Signed-off-by: Edgar Hernández <[email protected]>
@israel-hdez israel-hdez deleted the j20324-private-raw-ig branch March 14, 2025 16:41
openshift-merge-bot bot pushed a commit that referenced this pull request Mar 14, 2025
This adds the possibility to use the `networking.kserve.io/visibility=cluster-local` label to configure an InferenceGraph in Raw mode as private (not exposed).

Signed-off-by: Edgar Hernández <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

3 participants