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

Use data connection secret to store s3 info, use storage-initializer mage from the cluster #97

Merged
merged 30 commits into from
Jan 20, 2025

Conversation

rnetser
Copy link
Collaborator

@rnetser rnetser commented Jan 19, 2025

by using data connection secrets, storage-config secret will be auto-generated by odh-model-controller and this is currently the recommended way (and the one used by the dashboard) to create a connection secret

Copy link

The following are automatically added/executed:

Available user actions:

  • To mark a PR as WIP, add /wip in a comment. To remove it from the PR comment /wip cancel to the PR.
  • To block merging of a PR, add /hold in a comment. To un-block merging of PR comment /hold cancel.
  • To mark a PR as approved, add /lgtm in a comment. To remove, add /lgtm cancel.
    lgtm label removed on each new commit push.
  • To mark PR as verified comment /verified to the PR, to un-verify comment /verified cancel to the PR.
    verified label removed on each new commit push.
Supported labels

{'/wip', '/hold', '/verified', '/lgtm'}

@rnetser
Copy link
Collaborator Author

rnetser commented Jan 19, 2025

/verified

@github-actions github-actions bot added the Verified Verified pr in Jenkins label Jan 19, 2025
@rnetser rnetser changed the title Use data connection secret to store s3 information Use data connection secret to store s3 info, use storage-initializer mage from the cluster Jan 19, 2025
Copy link
Collaborator

@lugi0 lugi0 left a comment

Choose a reason for hiding this comment

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

I'm not sure this is the right approach moving forward, we're coupling ourselves with dashboard's behaviour over which we have no control

utilities/constants.py Outdated Show resolved Hide resolved
utilities/infra.py Show resolved Hide resolved
@github-actions github-actions bot added size/xl and removed size/l Verified Verified pr in Jenkins labels Jan 20, 2025
@rnetser
Copy link
Collaborator Author

rnetser commented Jan 20, 2025

/verified

@github-actions github-actions bot added the Verified Verified pr in Jenkins label Jan 20, 2025
lugi0
lugi0 previously approved these changes Jan 20, 2025
@github-actions github-actions bot removed the Verified Verified pr in Jenkins label Jan 20, 2025
@rnetser
Copy link
Collaborator Author

rnetser commented Jan 20, 2025

/verified

@github-actions github-actions bot added the Verified Verified pr in Jenkins label Jan 20, 2025
@rnetser rnetser merged commit 6c00967 into opendatahub-io:main Jan 20, 2025
4 checks passed
mwaykole pushed a commit to mwaykole/opendatahub-tests that referenced this pull request Jan 29, 2025
…r mage from the cluster (opendatahub-io#97)

* Create size-labeler.yml

* Delete .github/workflows/size-labeler.yml

* get storage-in from cluster

* move to json-based secrets

* update storage path

* fix labels

* remove unused ci_s3_storage_uri

* refactor annottions class in constants
mwaykole pushed a commit to mwaykole/opendatahub-tests that referenced this pull request Jan 29, 2025
…r mage from the cluster (opendatahub-io#97)

* Create size-labeler.yml

* Delete .github/workflows/size-labeler.yml

* get storage-in from cluster

* move to json-based secrets

* update storage path

* fix labels

* remove unused ci_s3_storage_uri

* refactor annottions class in constants
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
size/xl Verified Verified pr in Jenkins
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants