-
Notifications
You must be signed in to change notification settings - Fork 13
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
Feature request to specify products during apply-changes #46
Comments
We have created an issue in Pivotal Tracker to manage this. Unfortunately, the Pivotal Tracker project is private so you may be unable to view the contents of the story. The labels on this github issue will be updated when the story is started. |
There is a historical context of why the team chose to not allow this. We did not have guarantees that the Apply Changes would be idempotent with only specifying single products. From the end-user it is faster, but there were too many edge cases of "if something didn't fully install". |
@jtarchie I understand your comments, but our specific use case is for the credhub service broker tile that doesn't deploy any VMs. The CSB tile deploys an application in the post-deploy errand. We'd like to deploy this snowflake separate from other deployments. |
https://docs.pivotal.io/platform-automation/v5.0/tasks.html#stage-configure-apply allows this by wrapping other cheap, idempotent tasks to the front of the apply for reliability/consistency purposes. While it should be referenced from the Apply Changes task description so as to be more discoverable, I see it currently isn't. Does that task do what you need, or is there some mismatch for your situation there? |
URL or GitHub link to the page where you're having the issue
Is your request for a change or addition to content related to a problem? If so, please describe.
Feature Request.
Describe the solution you'd like
Updates to the apply-changes task to allow a pipeline developer to override the default (all) products that changes are applied.
Describe alternatives you've considered
None
The text was updated successfully, but these errors were encountered: