-
Notifications
You must be signed in to change notification settings - Fork 55
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Merge branch 'clean-odh-main' into merge-odh-mr
- Loading branch information
Showing
205 changed files
with
65,231 additions
and
3 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,23 @@ | ||
--- | ||
name: Bug report | ||
about: Create a report to help us improve | ||
title: '' | ||
labels: 'bug' | ||
assignees: '' | ||
|
||
--- | ||
|
||
**Describe the bug** | ||
A clear and concise description of what the bug is. | ||
|
||
**To Reproduce** | ||
Steps to reproduce the behavior: | ||
1. Do this '...' | ||
2. Do that '....' | ||
3. See error | ||
|
||
**Expected behavior** | ||
A clear and concise description of what you expected to happen. | ||
|
||
**Additional context** | ||
Add any other context about the problem here. |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,8 @@ | ||
--- | ||
name: Chore issue template | ||
about: General purpose issues related to chores, project management, etc. | ||
title: '' | ||
labels: 'chore' | ||
assignees: '' | ||
|
||
--- |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,20 @@ | ||
--- | ||
name: Feature request | ||
about: Suggest an idea for this project | ||
title: '' | ||
labels: 'feature' | ||
assignees: '' | ||
|
||
--- | ||
|
||
**Is your feature request related to a problem? Please describe.** | ||
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...] | ||
|
||
**Describe the solution you'd like** | ||
A clear and concise description of what you want to happen. | ||
|
||
**Describe alternatives you've considered** | ||
A clear and concise description of any alternative solutions or features you've considered. | ||
|
||
**Additional context** | ||
Add any other context or screenshots about the feature request here. |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,23 @@ | ||
# To get started with Dependabot version updates, you'll need to specify which | ||
# package ecosystems to update and where the package manifests are located. | ||
# Please see the documentation for all configuration options: | ||
# https://docs.github.com/github/administering-a-repository/configuration-options-for-dependency-updates | ||
|
||
version: 2 | ||
updates: | ||
- package-ecosystem: "gomod" | ||
directory: "/" | ||
schedule: | ||
interval: "weekly" | ||
- package-ecosystem: "pip" | ||
directory: "/" | ||
schedule: | ||
interval: "weekly" | ||
- package-ecosystem: "docker" | ||
directory: "/" | ||
schedule: | ||
interval: "weekly" | ||
- package-ecosystem: "github-actions" | ||
directory: "/" | ||
schedule: | ||
interval: "weekly" |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,17 @@ | ||
<!--- Provide a general summary of your changes in the Title above --> | ||
|
||
## Description | ||
<!--- Describe your changes in detail --> | ||
|
||
## How Has This Been Tested? | ||
<!--- Please describe in detail how you tested your changes. --> | ||
<!--- Include details of your testing environment, and the tests you ran to --> | ||
<!--- see how your change affects other areas of the code, etc. --> | ||
|
||
## Merge criteria: | ||
<!--- This PR will be merged by any repository approver when it meets all the points in the checklist --> | ||
<!--- Go over all the following points, and put an `x` in all the boxes that apply. --> | ||
|
||
- [ ] The commits and have meaningful messages; the author will squash them [after approval](https://github.com/opendatahub-io/opendatahub-community/blob/main/contributor-cheatsheet.md#:~:text=Usually%20this%20is%20done%20in%20last%20phase%20of%20a%20PR%20revision) or will ask to merge with squash. | ||
- [ ] Testing instructions have been added in the PR body (for PRs involving changes that are not immediately obvious). | ||
- [ ] The developer has manually tested the changes and verified that the changes work |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,68 @@ | ||
name: Container image build and tag | ||
on: | ||
push: | ||
branches: | ||
- 'main' | ||
tags: | ||
- 'v*' | ||
paths-ignore: | ||
- 'LICENSE*' | ||
- '**.gitignore' | ||
- '**.md' | ||
- '**.txt' | ||
- '.github/ISSUE_TEMPLATE/**' | ||
- '.github/dependabot.yml' | ||
- 'docs/**' | ||
env: | ||
QUAY_ORG: opendatahub | ||
QUAY_IMG_REPO: model-registry | ||
QUAY_USERNAME: ${{ secrets.QUAY_USERNAME }} | ||
QUAY_PASSWORD: ${{ secrets.QUAY_PASSWORD }} | ||
PUSH_IMAGE: true | ||
jobs: | ||
build-image: | ||
runs-on: ubuntu-latest | ||
steps: | ||
# Assign context variable for various action contexts (tag, main, CI) | ||
- name: Assigning tag context | ||
if: github.head_ref == '' && startsWith(github.ref, 'refs/tags/v') | ||
run: echo "BUILD_CONTEXT=tag" >> $GITHUB_ENV | ||
- name: Assigning main context | ||
if: github.head_ref == '' && github.ref == 'refs/heads/main' | ||
run: echo "BUILD_CONTEXT=main" >> $GITHUB_ENV | ||
# checkout branch | ||
- uses: actions/checkout@v4 | ||
# set image version | ||
- name: Set main-branch environment | ||
if: env.BUILD_CONTEXT == 'main' | ||
run: | | ||
commit_sha=${{ github.event.after }} | ||
tag=main-${commit_sha:0:7} | ||
echo "VERSION=${tag}" >> $GITHUB_ENV | ||
- name: Set tag environment | ||
if: env.BUILD_CONTEXT == 'tag' | ||
run: | | ||
echo "VERSION=${{ github.ref_name }}" >> $GITHUB_ENV | ||
- name: Build and Push Image | ||
shell: bash | ||
run: ./scripts/build_deploy.sh | ||
- name: Tag Latest | ||
if: env.BUILD_CONTEXT == 'main' | ||
shell: bash | ||
env: | ||
IMG: quay.io/${{ env.QUAY_ORG }}/${{ env.QUAY_IMG_REPO }} | ||
BUILD_IMAGE: false | ||
run: | | ||
docker tag ${{ env.IMG }}:$VERSION ${{ env.IMG }}:latest | ||
# BUILD_IMAGE=false skip the build | ||
./scripts/build_deploy.sh | ||
- name: Tag Main | ||
if: env.BUILD_CONTEXT == 'main' | ||
shell: bash | ||
env: | ||
IMG: quay.io/${{ env.QUAY_ORG }}/${{ env.QUAY_IMG_REPO }} | ||
BUILD_IMAGE: false | ||
run: | | ||
docker tag ${{ env.IMG }}:$VERSION ${{ env.IMG }}:main | ||
# BUILD_IMAGE=false skip the build | ||
./scripts/build_deploy.sh |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,54 @@ | ||
name: Test container image build and deployment | ||
on: | ||
pull_request: | ||
paths-ignore: | ||
- 'LICENSE*' | ||
- '**.gitignore' | ||
- '**.md' | ||
- '**.txt' | ||
- '.github/ISSUE_TEMPLATE/**' | ||
- '.github/dependabot.yml' | ||
- 'docs/**' | ||
- 'clients/python/**' | ||
env: | ||
QUAY_IMG_REPO: model-registry | ||
PUSH_IMAGE: false | ||
BRANCH: ${{ github.base_ref }} | ||
jobs: | ||
build-and-test-image: | ||
runs-on: ubuntu-latest | ||
steps: | ||
- uses: actions/checkout@v4 | ||
- name: Generate Tag | ||
shell: bash | ||
id: tags | ||
run: | | ||
commit_sha=${{ github.event.after }} | ||
tag=main-${commit_sha:0:7} | ||
echo "tag=${tag}" >> $GITHUB_OUTPUT | ||
- name: Build Image | ||
shell: bash | ||
env: | ||
VERSION: ${{ steps.tags.outputs.tag }} | ||
run: ./scripts/build_deploy.sh | ||
- name: Start Kind Cluster | ||
uses: helm/[email protected] | ||
- name: Load Local Registry Test Image | ||
env: | ||
IMG: "quay.io/opendatahub/model-registry:${{ steps.tags.outputs.tag }}" | ||
run: | | ||
kind load docker-image -n chart-testing ${IMG} | ||
- name: Deploy Operator With Test Image | ||
env: | ||
IMG: "quay.io/opendatahub/model-registry:${{ steps.tags.outputs.tag }}" | ||
run: | | ||
echo "Deploying operator from model-registry-operator branch ${BRANCH}" | ||
kubectl apply -k "https://github.com/opendatahub-io/model-registry-operator.git/config/default?ref=${BRANCH}" | ||
kubectl set env -n model-registry-operator-system deployment/model-registry-operator-controller-manager REST_IMAGE="${IMG}" | ||
- name: Create Test Registry | ||
run: | | ||
kubectl apply -k "https://github.com/opendatahub-io/model-registry-operator.git/config/samples?ref=${BRANCH}" | ||
kubectl get mr | ||
- name: Wait for Test Registry Deployment | ||
run: | | ||
kubectl wait --for=condition=Available=true modelregistries/modelregistry-sample --timeout=5m |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,59 @@ | ||
name: Build | ||
on: | ||
push: | ||
branches: | ||
- 'main' | ||
pull_request: | ||
paths-ignore: | ||
- 'LICENSE*' | ||
- 'DOCKERFILE*' | ||
- '**.gitignore' | ||
- '**.md' | ||
- '**.txt' | ||
- '.github/ISSUE_TEMPLATE/**' | ||
- '.github/dependabot.yml' | ||
- 'docs/**' | ||
- 'scripts/**' | ||
jobs: | ||
build: | ||
runs-on: ubuntu-latest | ||
steps: | ||
- uses: actions/checkout@v4 | ||
- name: Setup Go | ||
uses: actions/setup-go@v5 | ||
with: | ||
go-version: '1.19' | ||
- name: Install Protoc | ||
uses: arduino/setup-protoc@v3 | ||
with: | ||
version: "24.3" | ||
repo-token: ${{ secrets.GITHUB_TOKEN }} | ||
- name: Set up Python | ||
uses: actions/setup-python@v5 | ||
with: | ||
python-version: 3.9 | ||
- name: Install makefile dependencies | ||
run: make deps | ||
- name: Clean | ||
run: make clean | ||
- name: Build | ||
run: make build | ||
- name: Check if there are uncommitted file changes | ||
run: | | ||
clean=$(git status --porcelain) | ||
if [[ -z "$clean" ]]; then | ||
echo "Empty git status --porcelain: $clean" | ||
else | ||
echo "Uncommitted file changes detected: $clean" | ||
git diff | ||
exit 1 | ||
fi | ||
- name: Unit tests | ||
run: make test-cover | ||
- name: Upload coverage to Codecov | ||
uses: codecov/[email protected] | ||
env: | ||
CODECOV_TOKEN: ${{ secrets.CODECOV_TOKEN }} | ||
with: | ||
files: coverage.txt | ||
fail_ci_if_error: true |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,5 @@ | ||
pip==23.3.1 | ||
nox==2023.04.22 | ||
nox-poetry==1.0.2 | ||
poetry==1.6.1 | ||
virtualenv==20.24.6 |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,50 @@ | ||
name: Release Python client | ||
on: | ||
push: | ||
tags: | ||
- py-v* | ||
workflow_dispatch: | ||
jobs: | ||
release: | ||
name: Release | ||
runs-on: ubuntu-latest | ||
permissions: | ||
id-token: write | ||
env: | ||
FORCE_COLOR: "1" | ||
steps: | ||
- name: Check out the repository | ||
uses: actions/checkout@v4 | ||
with: | ||
fetch-depth: 0 | ||
- name: Set up Python | ||
uses: actions/setup-python@v5 | ||
with: | ||
python-version: "3.10" | ||
- name: Upgrade pip | ||
run: | | ||
pip install --constraint=.github/workflows/constraints.txt pip | ||
pip --version | ||
- name: Install Poetry | ||
run: | | ||
pipx install --pip-args=--constraint=.github/workflows/constraints.txt poetry | ||
poetry --version | ||
- name: Check version | ||
working-directory: clients/python | ||
run: | | ||
set -o pipefail | ||
LATEST_TAG=$(git describe --tags --match="py-v*") | ||
if [[ "$LATEST_TAG" =~ $(poetry version | cut -d' ' -f1) ]]; then | ||
echo "::error title='$LATEST_TAG tag does not match project version'::" | ||
exit 1 | ||
fi | ||
- name: Build package | ||
working-directory: clients/python | ||
run: | | ||
poetry build --ansi | ||
- name: Publish package on PyPI | ||
uses: pypa/gh-action-pypi-publish@release/v1 | ||
with: | ||
verbose: true | ||
print-hash: true | ||
packages-dir: clients/python/dist/ |
Oops, something went wrong.