Skip to content

Conversation

@alyssacgoins
Copy link

@alyssacgoins alyssacgoins commented Nov 17, 2025

Add script validate_metadata.py to the scripts directory. Validates metadata schema with the formatting below, and includes a Github Actions workflow.

tier: core | third_party
name: <string>
stability: alpha | beta | stable
dependencies:
  kubeflow:
    - name: Pipelines  # Kubeflow Pipelines version is required
      version: '>=2.5'
    - name: Trainer  # Other official Kubeflow components required. This is a validated list enforced by CI.
      version: '>=2.0'
  external_services:  # A free form of external service dependencies
    - name: Argo Workflows
      version: "3.6"
tags:  # Optional and may be used for tooling built around the catalog in the future
  - training
  - evaluation
lastVerified: 2025-03-15T00:00:00Z
ci:
  skip_dependency_probe: false
  pytest: optional
links:
  documentation: https://kubeflow.org/components/<name>
  issue_tracker: https://github.com/kubeflow/kfp-components/issues

@alyssacgoins alyssacgoins force-pushed the metadata-validation-workflow branch 3 times, most recently from 8115f83 to 52131e5 Compare November 18, 2025 13:58
@alyssacgoins alyssacgoins marked this pull request as ready for review November 18, 2025 14:25
@google-oss-prow google-oss-prow bot requested a review from mprahl November 18, 2025 14:25
@alyssacgoins alyssacgoins force-pushed the metadata-validation-workflow branch 9 times, most recently from c287ac8 to 0efbdc2 Compare November 19, 2025 16:05
@alyssacgoins alyssacgoins force-pushed the metadata-validation-workflow branch 2 times, most recently from 8e3044d to f881126 Compare November 19, 2025 16:35
@google-oss-prow
Copy link

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
Once this PR has been reviewed and has the lgtm label, please ask for approval from humairak. For more information see the Kubernetes Code Review Process.

The full list of commands accepted by this bot can be found 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

@alyssacgoins alyssacgoins force-pushed the metadata-validation-workflow branch 3 times, most recently from 20e5f23 to 73d353e Compare November 20, 2025 23:50
@alyssacgoins alyssacgoins force-pushed the metadata-validation-workflow branch 19 times, most recently from 307b1f1 to 2dc5b12 Compare November 24, 2025 21:20
@alyssacgoins alyssacgoins force-pushed the metadata-validation-workflow branch from 2355365 to c171861 Compare November 25, 2025 16:46
for item in $NEW_ITEMS_ARRAY; do
FILE_PATH="$GITHUB_WORKSPACE/$item"
echo "Processing item: $item"
python "${{ env.VALIDATION_SCRIPT_PATH }}" --item $FILE_PATH

Choose a reason for hiding this comment

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

Suggested change
python "${{ env.VALIDATION_SCRIPT_PATH }}" --item $FILE_PATH
python "${{ env.VALIDATION_SCRIPT_PATH }}" --component $FILE_PATH

I see that the validate_metadata.py script accepts --component flag.

Copy link
Author

Choose a reason for hiding this comment

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

Updated this argument to --dir

@alyssacgoins alyssacgoins force-pushed the metadata-validation-workflow branch from c171861 to c36c399 Compare November 25, 2025 22:04
@google-oss-prow
Copy link

The following users are mentioned in OWNERS file(s) but are untrusted for the following reasons. One way to make the user trusted is to add them as members of the kubeflow org. You can then trigger verification by writing /verify-owners in a comment.

  • sample-approver
    • User is not a member of the org. Satisfy at least one of these conditions to make the user trusted.
    • scripts/validate_metadata/test_data/directories_metadata/valid/OWNERS

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