Skip to content
This repository was archived by the owner on Dec 12, 2025. It is now read-only.

Conversation

@chatton
Copy link
Contributor

@chatton chatton commented Jul 14, 2020

All Submissions:

  • Have you signed our CLA?
  • Have you checked to ensure there aren't other open Pull Requests for the same update/change?

This PR fetches the MongoDB resources and dumps the config for each one. This should work for tests that deal with multiple MongoDB resources.

@chatton chatton requested a review from bznein July 14, 2020 14:14
Copy link
Contributor

@bznein bznein left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great improvement! LGTM once we see on the tests that the dump works fine!

@chatton chatton merged commit 963f2d1 into master Jul 14, 2020
@chatton chatton deleted the remove_hard_coded_name branch July 14, 2020 17:28
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants