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

Add new Executor property to config test #31

Merged
merged 1 commit into from
Sep 17, 2021

Conversation

Gerg
Copy link
Member

@Gerg Gerg commented Sep 13, 2021

Thank you for submitting a pull request to the rep repository!

We appreciate the contribution. To help us understand the context for your pull request, please fill out this template to the best of your ability.

Please make sure to complete all of the following steps

  1. Check the Contributing document on how to sign the CLA and run tests.
  2. Submit your PR to this repo.
  3. Submit an accompanying PR Review Request referencing this PR so the Diego Team knows to review your pull request.

Please provide the following information:

What is this change about?

Minor test update as part of cloudfoundry/diego-release#592.

What problem it is trying to solve?

See cloudfoundry/diego-release#592.

What is the impact if the change is not made?

The rep config test will not be up-to-date with executor config.

How should this change be described in diego-release release notes?

N/A

Please provide any contextual information.

Diego-release issue and associated PRs: cloudfoundry/diego-release#592
Root issue: cloudfoundry/routing-release#200

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.

2 participants