-
Notifications
You must be signed in to change notification settings - Fork 110
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
Implement process for upgrading old service instances on cloud.gov #3083
Comments
The CSB is implementing this capability upstream, and should be shipping it any day now. |
@nickumia-reisys can you add link to upstream documentation |
There actually isn't a good central location where this is documented. Various links,
|
This is relevant only if/when we move to EKS broker so moving to Icebox for now. |
As a summary of how this works:
Because we made this the default behavior, to achieve the old behavior of leaving the terraform code unchanged on This feature was critical in pushing updates to existing production instances without downtime.
Very much related to
|
🖖 Big shoutout to @mogul for keeping on top of the Brokerpak community and mentoring me through this! |
User Story
In order to ensure existing instances get updates relating to compliance or security, the SSB should include a mechanism for updating existing instances, or helping clients migrate between plans.
Acceptance Criteria
[ACs should be clearly demoable/verifiable whenever possible. Try specifying them using BDD.]
AND a the SSB is updated to include a new brokerpak version implementing a new feature
WHEN I run
cf update-service INSTANCENAME
THEN I can see the new feature is available in INSTANCENAME
Background
[Any helpful contextual notes or links to artifacts/evidence, if needed]
Security Considerations (required)
[Any security concerns that might be implicated in the change. "None" is OK, just be explicit here!]
Sketch
The CSB team is implementing this functionality upstream. See this epic in Pivotal Tracker and the set of stories attached to it.
The feature is now available, but is behind a feature flag we need to enable.
The text was updated successfully, but these errors were encountered: