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 ServiceBinding types instead of unstructured.Unstructured #4690

Closed
kadel opened this issue May 5, 2021 · 3 comments · Fixed by #4713
Closed

Use ServiceBinding types instead of unstructured.Unstructured #4690

kadel opened this issue May 5, 2021 · 3 comments · Fixed by #4713
Assignees
Labels
priority/Medium Nice to have issue. Getting it done before priority changes would be great.
Projects

Comments

@kadel
Copy link
Member

kadel commented May 5, 2021

To work around issues with a new service binding operator version we had to use unstructured.Unstructured for ServiceBinding (#4613) instead of using ServiceBiding struct types.

After #4648 is merged can move back to using ServiceBinding type.
This will make it easier to manage data in ServiecBinding CR and later it will be useful when implementing #4543

@kadel
Copy link
Member Author

kadel commented May 5, 2021

/area service-binding
/priority medium

@openshift-ci-robot openshift-ci-robot added area/service-binding priority/Medium Nice to have issue. Getting it done before priority changes would be great. labels May 5, 2021
@kadel kadel added this to For Consideration in Sprint 201 via automation May 5, 2021
@kadel
Copy link
Member Author

kadel commented May 5, 2021

This issue requires #4648 to be merged first.

@dharmit dharmit moved this from For Consideration to To Do in Sprint 201 May 6, 2021
@dharmit
Copy link
Member

dharmit commented May 6, 2021

/assign @feloy

@feloy feloy moved this from To Do to In Progress in Sprint 201 May 10, 2021
@feloy feloy moved this from In Progress to For Review in Sprint 201 May 18, 2021
Sprint 201 automation moved this from For Review to Done May 19, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
priority/Medium Nice to have issue. Getting it done before priority changes would be great.
Projects
No open projects
Development

Successfully merging a pull request may close this issue.

4 participants