-
Notifications
You must be signed in to change notification settings - Fork 13
periodic sync upstream KF to midstream ODH #215
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
Changes from all commits
8258979
095ec53
8b07b60
65a2e07
0130f9e
8e6e4f7
1d7b330
da0a708
eb7b82b
b7b4690
565f49e
e409d9f
01c9830
8292c4a
a0f41af
d7aa816
a2b83f7
764ca98
0096145
fdca36c
e83da4a
a9d4d7d
f9daf57
4a632cd
3825f99
be6792d
c8a3867
ab662e4
bf6cc97
9125316
cbf228f
8e3e6a5
8d48d2c
c58d14f
8b50a86
39d5e4f
949e24e
f54ea68
1fd212e
a098b60
f644f24
daa14ef
8a77105
62755e8
4033763
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,44 @@ | ||
| name: Welcome first-time contributors (Beta) | ||
| on: | ||
| pull_request_target: | ||
| types: | ||
| - opened | ||
| permissions: | ||
| issues: write | ||
| pull-requests: read | ||
|
|
||
| # do NOT: add actions/checkout to this flow, add-third party scripts, or auto-trigger CI jobs | ||
| # goal: this is meant to remind maintainers/approvers to add labels to ensure all tests are executed before merging | ||
| # and avoid merging without realizing that required tests has not been run; complementary to bulletpoint in template: https://github.com/kubeflow/model-registry/blame/de5f225d96a4daeca77506d233082b1c4ea5afa3/.github/pull_request_template.md#L21 | ||
| jobs: | ||
| welcome: | ||
| runs-on: ubuntu-latest | ||
| steps: | ||
| - name: Check contributor status | ||
| uses: actions/github-script@v7 | ||
| with: | ||
| script: | | ||
| const { data: pr } = await github.rest.pulls.get({ | ||
| owner: context.repo.owner, | ||
| repo: context.repo.repo, | ||
| pull_number: context.issue.number | ||
| }); | ||
| const isFirstTimeContributor = pr.user.contributions === 0; | ||
| console.log(`First-time contributor status: ${isFirstTimeContributor}`); | ||
| return { isFirstTimeContributor }; | ||
|
Comment on lines
+17
to
+29
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Missing step ID prevents output reference. The step that checks contributor status is missing an ID, but is referenced later by ID. Add an ID to the step: - - name: Check contributor status
+ - name: Check contributor status
+ id: check_author
uses: actions/github-script@v7
with:
script: |
const { data: pr } = await github.rest.pulls.get({
owner: context.repo.owner,
repo: context.repo.repo,
pull_number: context.issue.number
});
const isFirstTimeContributor = pr.user.contributions === 0;
console.log(`First-time contributor status: ${isFirstTimeContributor}`);
return { isFirstTimeContributor };🤖 Prompt for AI Agents |
||
| - name: Log contributor status | ||
| run: echo "First-time contributor status is ${{ steps.check_author.outputs.isFirstTimeContributor }}" | ||
|
|
||
| # - name: Add a comment to the PR if first time contributor | ||
| # if: steps.check_author.outputs.isFirstTimeContributor == 'true' | ||
| # uses: actions/github-script@v7 | ||
| # with: | ||
| # script: | | ||
| # github.rest.issues.createComment({ | ||
| # issue_number: context.issue.number, | ||
| # owner: context.repo.owner, | ||
| # repo: context.repo.repo, | ||
| # body: "Maintainers: let's ensure the label `ok-to-test` has been maintained and all the tests have been executed before merging.<br/><br/>Thank you for your first Pull Request! 🎉🎉" | ||
| # }) | ||
| # do NOT: add actions/checkout to this flow, add-third party scripts, or auto-trigger CI jobs | ||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -2,6 +2,11 @@ | |
|
|
||
| Below are the adopters of the Model Registry project. If you are using Model Registry, please add your name to the list by submitting a pull request: this will help foster the Kubeflow community. Kindly ensure the list remains in alphabetical order. | ||
|
|
||
| | Organization | Contact (GitHub User Name) | Environment | Description of Use | | ||
| | ------------- | ------------- | ------------- | ------------- | | ||
| | <company name here> | <your github handle here> | Production/Testing/Experimenting/etc | <any notes you'd like to share> | | ||
| | Organization | Contact (GitHub User Name) | Environment | Description of Use | | ||
| |-------------------------------|--------------------------------------------|--------------------------------------------|-----------------------------------------------------------------------| | ||
| | [Pepsico](https://www.pepsico.com/) | [@milosjava](https://github.com/milosjava) | Experimenting | Evaluating a transition from Azure Model Registry to Kubeflow Model Registry | | ||
| | [Red Hat](https://www.redhat.com) | [@rareddy](https://github.com/rareddy)| Production | Kubeflow Model Registry is part of [OpenShift AI](https://www.redhat.com/en/products/ai/openshift-ai) | | ||
| | [INFN (National Institute for Nuclear Physics)](https://www.infn.it/en/) | [@mginfn](https://github.com/mginfn) | Experimenting | Building a platform for running ML workflows | | ||
| | [YIQISOFT](https://www.yiqisoft.com) | [@jiekechoo](https://github.com/jiekechoo) | Production | Model Registry is part of [YiAI](https://www.yiqisoft.cn/products/)| | ||
| | [VMware](https://www.vmware.com) | [@tiansiyuan](https://github.com/tiansiyuan) | Testing | Model Registry is important for LLMs | | ||
| | < company name here> | < your github handle here > | <Production/Testing/Experimenting/etc> | <any notes you'd like to share> | | ||
|
Comment on lines
+5
to
+12
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Ensure alphabetical ordering of adopters The project guidelines specify that adopters must be listed in alphabetical order, but the current sequence ( 🧰 Tools🪛 LanguageTool[style] ~5-~5: It’s more common nowadays to write this noun as one word. (RECOMMENDED_COMPOUNDS) 🤖 Prompt for AI Agents |
||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,23 @@ | ||
| # Kubeflow Model Registry Roadmap | ||
|
|
||
| ## Overview | ||
|
|
||
| The Kubeflow Model Registry project roadmap is tracked using [GitHub Milestones](https://github.com/kubeflow/model-registry/milestones), following the Kubeflow Release Team's request: this approach has been recommended in order to maintain a detailed view of all the items intended for upcoming Kubeflow releases, organized by "themes". The roadmap is continuously updated as we make progress towards a Kubeflow Release. | ||
|
|
||
| ## Current Roadmap | ||
|
|
||
| The current roadmap and upcoming features are tracked in our [GitHub Milestones](https://github.com/kubeflow/model-registry/milestones). Each milestone tracks specific or future Kubeflow release and contains the planned features, enhancements, and bug fixes. For each milestone, a GitHub Issue is typically attached serving as a Tracker record to provide additional details. | ||
|
|
||
| ## Past Roadmaps | ||
|
|
||
| - Model Registry in Kubeflow 1.10 | ||
| - [Milestone](https://github.com/kubeflow/model-registry/milestone/1) | ||
| - [Tracker for details](https://github.com/kubeflow/model-registry/issues/175) | ||
| - Model Registry in Kubeflow 1.9 | ||
| - [Tracker for details](https://github.com/kubeflow/model-registry/issues/3) | ||
|
|
||
| ## See also | ||
|
|
||
| For more information about contributing to this project including the roadmap, refer to our [CONTRIBUTING.md](./CONTRIBUTING.md) file. | ||
|
|
||
| For more information about the release process, refer to our [RELEASE.md](./RELEASE.md) file. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fix output ID reference mismatch
The
Check contributor statusstep needs anidattribute to match the reference in the "Log contributor status" step.🤖 Prompt for AI Agents