Skip to content

update(dashboard): do not set owner on CR - #224

Merged
zdtsw merged 2 commits into
red-hat-data-services:mainfrom
zdtsw-forking:jira/3963
Mar 20, 2024
Merged

zdtsw merged 2 commits into
red-hat-data-services:mainfrom
zdtsw-forking:jira/3963

Conversation

@zdtsw

@zdtsw zdtsw commented Mar 19, 2024

Copy link
Copy Markdown

ref: https://issues.redhat.com/browse/RHOAIENG-3963

live build: quay.io/wenzhou/rhods-operator-catalog:v2.10.3963
Screenshot from 2024-03-19 14-49-03
Screenshot from 2024-03-19 14-49-09

will need backport to ODH

Signed-off-by: Wen Zhou <wenzhou@redhat.com>
@VaishnaviHire

Copy link
Copy Markdown

Do we need to remove the ownerref for existing instances?

@VaishnaviHire

Copy link
Copy Markdown

Also we can include this in 2.9?

@zdtsw

zdtsw commented Mar 19, 2024

Copy link
Copy Markdown
Author

Do we need to remove the ownerref for existing instances?

good question.
from the issue itself, does not see as a blocker, maybe i can remove it for any upgrade case too.

Signed-off-by: Wen Zhou <wenzhou@redhat.com>

@VaishnaviHire VaishnaviHire left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

/lgtm

@VaishnaviHire

Copy link
Copy Markdown

I think this should be good to go in 2.9

@zdtsw
zdtsw merged commit e2ee57e into red-hat-data-services:main Mar 20, 2024
@zdtsw

zdtsw commented Mar 20, 2024 •

Copy link
Copy Markdown
Author

i will sync it to 2.9 #229

zdtsw added a commit to zdtsw-forking/rhods-operator that referenced this pull request Mar 20, 2024
* update(dashboard): do not set owner on CR

Signed-off-by: Wen Zhou <wenzhou@redhat.com>

* update: handle upgrade case to unset ownerreference if already exist

Signed-off-by: Wen Zhou <wenzhou@redhat.com>

---------

Signed-off-by: Wen Zhou <wenzhou@redhat.com>
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.

2 participants