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

Implement exponential backoff for service fetch operations (CC API v2) #2680

Conversation

svkrieger
Copy link
Contributor

@svkrieger svkrieger commented Feb 15, 2022

Thanks for contributing to cloud_controller_ng. To speed up the process of reviewing your pull request please provide us with:

This PR depends on the above one.

  • I have reviewed the contributing guide

  • I have viewed, signed, and submitted the Contributor License Agreement

  • I have made this pull request to the main branch

  • I have run all the unit tests using bundle exec rake

  • I have run CF Acceptance Tests

@svkrieger
Copy link
Contributor Author

svkrieger commented Feb 15, 2022

Regarding the tests we would be glad about feedback, as they are quite different from the v3 tests. The tests can be replicated to the service binding test file after we found some good ones.

Tests are failing because this PR depends on #2667

@svkrieger svkrieger force-pushed the implement-exponential-backoff-for-sb-operations-v2 branch from c137330 to 9749856 Compare February 15, 2022 10:07
@svkrieger svkrieger force-pushed the implement-exponential-backoff-for-sb-operations-v2 branch from 9749856 to fa6daa7 Compare February 16, 2022 18:02
@iaftab-alam iaftab-alam force-pushed the implement-exponential-backoff-for-sb-operations-v2 branch from fa6daa7 to 27cc0d5 Compare February 17, 2022 16:12
@svkrieger svkrieger force-pushed the implement-exponential-backoff-for-sb-operations-v2 branch from 27cc0d5 to 66f8016 Compare February 17, 2022 16:17
@svkrieger svkrieger marked this pull request as ready for review February 18, 2022 08:44
@philippthun philippthun self-requested a review February 18, 2022 16:05
@philippthun philippthun self-assigned this Feb 22, 2022
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