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

Sync master from odh v1.9-branch to get the latest changes ready for rhoai-nightly #125

Merged
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
18 commits
Select commit Hold shift + click to select a range
1775c80
NO-JIRA: chore(nbcs): run `make generate` in `components/notebook-con…
jiridanek Sep 25, 2024
e7ff614
Merge pull request #407 from openshift-cherrypick-robot/cherry-pick-3…
openshift-merge-bot[bot] Oct 2, 2024
1b2dd4f
NO-JIRA: chore(nbcs): update tooling in controller's Makefiles (#391)
jiridanek Oct 2, 2024
92f30d9
Renamed the GHA fixing a typo
jstourac Oct 4, 2024
7986030
Merge pull request #414 from jstourac/renamegha
openshift-merge-bot[bot] Oct 4, 2024
5ddebb2
chore(gha): bump peter-evans/create-pull-request from 6.1.0 to 7.0.5
dependabot[bot] Oct 7, 2024
92d07c3
Update odh and notebook-controller with image main-7986030
github-actions[bot] Oct 7, 2024
cba3e5d
Merge pull request #416 from opendatahub-io/temp-11211801603
openshift-merge-bot[bot] Oct 7, 2024
d24d840
Merge pull request #415 from opendatahub-io/dependabot/github_actions…
openshift-merge-bot[bot] Oct 7, 2024
741dcfe
NO-JIRA: fix(gha): replace oc with kubectl in notebook_controller_int…
jiridanek Oct 15, 2024
385767d
Merge pull request #417 from jiridanek/oc_kubectl
openshift-merge-bot[bot] Oct 16, 2024
ed317a7
NO-JIRA: fix(gha) improve robustness by checking for resource existence
jiridanek Oct 15, 2024
83fd2c8
Revert "[v1.9-branch] RHOAIENG-10827: feat(nbcs): update ose-oauth-pr…
jiridanek Oct 16, 2024
130d758
Merge pull request #418 from jiridanek/waiting
openshift-merge-bot[bot] Oct 16, 2024
7afca5b
Merge branch 'main' of https://github.com/opendatahub-io/kubeflow int…
jiridanek Oct 16, 2024
c3d9c8b
Merge pull request #419 from jiridanek/jd_revert_oauth_proxy_update
openshift-merge-bot[bot] Oct 16, 2024
16bc9fa
Merge pull request #420 from jiridanek/jd_sync_v1.9-branch_from_main
openshift-merge-bot[bot] Oct 16, 2024
7f21e13
Merge pull request #422 from opendatahub-io/main
openshift-merge-bot[bot] Oct 16, 2024
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Original file line number Diff line number Diff line change
Expand Up @@ -89,7 +89,7 @@ jobs:
export PR_NOTEBOOK_IMG=localhost/${{env.IMG}}:${{env.TAG}}
kustomize edit set image ${CURRENT_NOTEBOOK_IMG}=${PR_NOTEBOOK_IMG}
cat <<EOF | oc apply -f -
cat <<EOF | kubectl apply -f -
---
apiVersion: v1
kind: ConfigMap
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -136,7 +136,7 @@ jobs:
kustomize edit set image ${CURRENT_NOTEBOOK_IMG}=${PR_NOTEBOOK_IMG}
# configure culler
cat <<EOF | oc apply -f -
cat <<EOF | kubectl apply -f -
---
apiVersion: v1
kind: ConfigMap
Expand Down Expand Up @@ -169,7 +169,7 @@ jobs:
echo "odh-notebook-controller-image=localhost/${{env.IMG}}:${{env.TAG}}" > params.env
cat <<EOF | oc apply -f -
cat <<EOF | kubectl apply -f -
---
apiVersion: v1
kind: ConfigMap
Expand Down Expand Up @@ -225,7 +225,7 @@ jobs:
- name: Create notebook and check it, this is from kubeflow readme
run: |
notebook_namespace=default
cat <<EOF | oc apply -f -
cat <<EOF | kubectl apply -f -
---
apiVersion: kubeflow.org/v1
kind: Notebook
Expand Down Expand Up @@ -271,6 +271,8 @@ jobs:
port: notebook-port
EOF
# wait for the statefulset to be created
timeout 100 bash -c -- 'until kubectl get statefulset minimal-notebook; do sleep 1; done'
# wait for the good result
kubectl rollout status --watch statefulset minimal-notebook
kubectl wait statefulset minimal-notebook --for=jsonpath='{.spec.replicas}'=1 --timeout=100s
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -35,7 +35,7 @@ jobs:
echo "branch=$SYNC_BRANCH" >> $GITHUB_OUTPUT

- name: Create pull request
uses: peter-evans/create-pull-request@c5a7806660adbe173f04e3e038b0ccdcd758773c # v6.1.0
uses: peter-evans/create-pull-request@5e914681df9dc83aa4e4905692ca88beb2f9e91f # v7.0.5
with:
branch: ${{ steps.prepare.outputs.branch }}
title: "Sync `${{ github.event.inputs.target }}` branch with `${{ github.event.inputs.source }}` branch"
Expand Down
8 changes: 4 additions & 4 deletions components/notebook-controller/Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -143,18 +143,18 @@ $(LOCALBIN): ## Ensure that the directory exists
CONTROLLER_GEN = $(LOCALBIN)/controller-gen
.PHONY: controller-gen
controller-gen: ## Download controller-gen locally if necessary.
GOBIN=$(LOCALBIN) go install sigs.k8s.io/controller-tools/cmd/controller-gen@v0.8.0
GOBIN=$(LOCALBIN) go install sigs.k8s.io/controller-tools/cmd/controller-gen@v0.16.3

KUSTOMIZE = $(shell pwd)/bin/kustomize
.PHONY: kustomize
kustomize: ## Download kustomize locally if necessary.
$(call go-get-tool,$(KUSTOMIZE),sigs.k8s.io/kustomize/v3/cmd/kustomize@v3.2.0)
$(call go-get-tool,$(KUSTOMIZE),sigs.k8s.io/kustomize/v3/cmd/kustomize@v5.0.2)

ENVTEST = $(shell pwd)/bin/setup-envtest
ENVTEST_VERSION?=release-0.14
ENVTEST_VERSION?=v0.0.0-20240923090159-236e448db12c

.PHONY: envtest
envtest: ## Download envtest-setup locally if necessary.
envtest: ## Download setup-envtest locally if necessary.
$(call go-get-tool,$(ENVTEST),sigs.k8s.io/controller-runtime/tools/setup-envtest@$(ENVTEST_VERSION))

# go-get-tool will 'go get' any package $2 and install it to $1.
Expand Down

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

Loading