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

Add ai-lab model-server-sample chart owners #1739

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

thepetk
Copy link
Contributor

@thepetk thepetk commented Feb 25, 2025

The PR introduces the new ai-lab helm chart - the model-server-sample helm chart which deploys a model service based on the values provided by the user.

Once the owners have been added I'll open a new PR to add two new report files, one for chatbot v0.1.6 and one for model-server v0.1.6 chart versions.

Copy link
Contributor

Submission Facts Mismatch

The content of the OWNERS file and your submission path do not seem to match. Double check that
your vendor label and chart name values in your OWNERS file match with your submission path.

This comment was auto-generated by GitHub Actions.

Copy link
Contributor

Thank you for submitting PR #1739 for Helm Chart Certification!

Outcome:

Failed

Detail

One or more errors were found with the pull request:

[INFO] OWNERS file changes require manual review by maintainers.

Task Insights

Here are the outcomes of tasks driving this result.

task outcome
PR Content Check failure
Run Chart Verifier skipped
Result Verification skipped
OpenShift Client Installation skipped

For information on the certification process see:

/metadata {"vendor_label": "", "chart_name": ""}

@thepetk thepetk changed the title Add ai-lab model-server chart owners Add ai-lab model-server-sample chart owners Feb 25, 2025
Copy link
Contributor

🔓 Maintainers:

The OWNERS file contributed here has a chart name that is AVAILABLE!

The chart name 'model-server-sample' does not appear in our lockfile.

After reviewing this pull request, please re-run this workflow once more before merging.

This comment was auto-generated by GitHub Actions.

Copy link
Contributor

🔓 Maintainers:

The OWNERS file contributed here has a chart name that is AVAILABLE!

The chart name 'model-server-sample' does not appear in our lockfile.

After reviewing this pull request, please re-run this workflow once more before merging.

This comment was auto-generated by GitHub Actions.

Copy link
Contributor

🔓 Maintainers:

The OWNERS file contributed here has a chart name that is AVAILABLE!

The chart name 'model-server-sample' does not appear in our lockfile.

After reviewing this pull request, please re-run this workflow once more before merging.

This comment was auto-generated by GitHub Actions.

Copy link
Contributor

Thank you for submitting PR #1739 for Helm Chart Certification!

Outcome:

Failed

Detail

One or more errors were found with the pull request:

[INFO] OWNERS file changes require manual review by maintainers.

Task Insights

Here are the outcomes of tasks driving this result.

task outcome
PR Content Check failure
Run Chart Verifier skipped
Result Verification skipped
OpenShift Client Installation skipped

For information on the certification process see:

/metadata {"vendor_label": "", "chart_name": ""}

2 similar comments
Copy link
Contributor

Thank you for submitting PR #1739 for Helm Chart Certification!

Outcome:

Failed

Detail

One or more errors were found with the pull request:

[INFO] OWNERS file changes require manual review by maintainers.

Task Insights

Here are the outcomes of tasks driving this result.

task outcome
PR Content Check failure
Run Chart Verifier skipped
Result Verification skipped
OpenShift Client Installation skipped

For information on the certification process see:

/metadata {"vendor_label": "", "chart_name": ""}

Copy link
Contributor

Thank you for submitting PR #1739 for Helm Chart Certification!

Outcome:

Failed

Detail

One or more errors were found with the pull request:

[INFO] OWNERS file changes require manual review by maintainers.

Task Insights

Here are the outcomes of tasks driving this result.

task outcome
PR Content Check failure
Run Chart Verifier skipped
Result Verification skipped
OpenShift Client Installation skipped

For information on the certification process see:

/metadata {"vendor_label": "", "chart_name": ""}

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant