Skip to content

Conversation

@reneighbor
Copy link

What is this change about?

Make ops-files and vars-files inputs optional

  • They are not used if the pipeline user is using default cf-deployment setup
  • This causes users to add dummy / unused resources and adds pipeline cruft

Please provide contextual information.

https://vmware.slack.com/archives/C01ERQAVD2L/p1631131092013600

Please check all that apply for this PR:

  • introduces a new task
  • changes an existing task
  • changes the Dockerfile
  • introduces a breaking change (other users will need to make manual changes when this is released)

Did you update the README as appropriate for this change?

  • YES
  • N/A

How should this change be described in release notes?

Something brief that conveys the change and is written with the component author audience in mind.

What is the level of urgency for publishing this change?

  • Urgent - unblocks current or future work
  • Slightly Less than Urgent

Tag your pair, your PM, and/or team!

@cloudfoundry/runtime

 * They are not used if the pipeline user is using default cf-deployment setup
 * This causes users to add dummy / unused resources and adds pipeline cruft
@linux-foundation-easycla
Copy link

linux-foundation-easycla bot commented Sep 8, 2021

CLA Signed

The committers are authorized under a signed CLA.

@davewalter
Copy link
Member

Hi @rroberts2222,

Could you please sign the CLA? You will need to close and reopen the PR to get the check to rerun.

Thanks,
Dave

@davewalter davewalter self-assigned this Oct 8, 2021
@davewalter davewalter merged commit 97f5a58 into cloudfoundry:main Oct 8, 2021
@davewalter
Copy link
Member

Thanks for the contribution!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants