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]: from ODH to RHOAI #236

Merged
merged 8 commits into from
Apr 5, 2024

Conversation

zdtsw
Copy link

@zdtsw zdtsw commented Apr 3, 2024

bartoszmajsak and others added 8 commits April 3, 2024 09:13
after introducing validating webhook these are not needed anymore.
This functionality has not been aligned with the functions in deploy.go and is currently not utilized anywhere within the codebase.
- jupyter-install-python-packages OdhDocument
- jupyter OdhApplication
- deploy-python-model and create-jupyter-notebook OdhQuickStart

Signed-off-by: Wen Zhou <[email protected]>
* chore: split upgrade.go into 3 files

Signed-off-by: Wen Zhou <[email protected]>

* update: code review

Signed-off-by: Wen Zhou <[email protected]>

* revert: keep return error for the case of readfile etc

Signed-off-by: Wen Zhou <[email protected]>

* Update pkg/cluster/cluster_config.go

Co-authored-by: Bartosz Majsak <[email protected]>

---------

Signed-off-by: Wen Zhou <[email protected]>
Co-authored-by: Bartosz Majsak <[email protected]>
we cannot apply this change in RHOAI, because the webhook part is not
here
This reverts commit c35320b.
…#941)

This change introduces types for easy re-use of labels.

The idea is not only to extract "magic strings" and avoid
repetitions, but primarily to provide structured and unified way for
re-using common labels which can be easily discovered and assisted
by the code completion.
* fix: properly updates config map

Previously the desired configmap state was overwritten by existing
version fetched using client.Get.

Includes test which fails without the patched function.

* chore(lint): fixes formatting

Signed-off-by: Wen Zhou <[email protected]>
Copy link

@VaishnaviHire VaishnaviHire left a comment

Choose a reason for hiding this comment

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

/lgtm

@VaishnaviHire VaishnaviHire merged commit 388e69e into red-hat-data-services:main Apr 5, 2024
5 checks passed
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.

4 participants