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

As a Korifi app developer I want to see an error when I try to create a managed service instance of a disabled service plan #3506

Open
georgethebeatle opened this issue Oct 8, 2024 · 0 comments
Milestone

Comments

@georgethebeatle
Copy link
Member

georgethebeatle commented Oct 8, 2024

Background

Right now in Korifi we do set service plan visibility on the service plan resource, but the service instance controller does not observe it. This means that the instance will go to failed state and will never be able to recover.

Acceptance

GIVEN I have a plan that is not visible
WHEN I create a service instance for it
THEN tbd

Dev Notes

  • What is the behaviour of classic CF when we create an instance of a disabled plan? Can the instance recover or should it be re-created?
  • We should error when a user tries to create an instance of a disabled service plan
  • Existing instances in an errored state (as described above) should be able to recover once the plan is enabled.
  • Ready instances should remain ready when their plan is disabled.
  • All checks should happen in the controllers, so that kubectl users can have the experience as api users
  • Think about whether is makes sense to spin off a separate story for org visibility or if we should deliver all three visibilities with one story
  • For now space visibility is out of scope
@danail-branekov danail-branekov added this to the v0.13 milestone Oct 8, 2024
@georgethebeatle georgethebeatle modified the milestones: v0.13, v0.14 Oct 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: 🧊 Icebox
Development

No branches or pull requests

2 participants