search

microsoft / vsts-cloudfoundry

Azure DevOps Extension for Cloud Foundry
MIT License
20 stars 23 forks source link

Allow cmdline overrides to provided manifest #2

Closed seanmck closed 7 years ago

seanmck commented 7 years ago

Currently, the task forces a choice between using a manifest and providing cmdline arguments manually. However, the CF CLI itself allows for both, with cmdline arguments overriding conflicting manifest values. Ideally, the VSTS task would mirror that behavior.

https://docs.cloudfoundry.org/devguide/deploy-apps/manifest.html#precedence

madhurig commented 7 years ago

@seanmck: Additional arguments can be passed when specifying a manifest file or when specifying properties. Is this not sufficient? cfoptions

seanmck commented 7 years ago

I will double check - I can't remember if I tried that. Either way, I think it's useful to have first-class UI for the common parameters.

madhurig commented 7 years ago

@seanmck : Closing since overriding values from the manifest is possible. We can add more UI but it might be confusing and we can't add UI for all the common parameters anyway.