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

feat: add network TTL #94

Merged
merged 1 commit into from
Oct 12, 2023
Merged

feat: add network TTL #94

merged 1 commit into from
Oct 12, 2023

Conversation

nathanielc
Copy link
Collaborator

Now that we are programmatically creating networks (see ceramic-tests) we want to make sure we are not leaving lots of unused resources around.

With this feature the test driver can create networks with a generous TTL of a few hours (when we expect tests to run in a few minutes). If for some reason the test driver fails to clean up the network itself, eventually the operator will clean it up.

This feature remains optional so manually created networks are not affected.

@nathanielc nathanielc requested review from stbrody and smrz2001 October 6, 2023 18:50
operator/src/network/controller.rs Outdated Show resolved Hide resolved
operator/src/network/controller.rs Outdated Show resolved Hide resolved
operator/src/network/spec.rs Outdated Show resolved Hide resolved
Copy link
Collaborator

@smrz2001 smrz2001 left a comment

Choose a reason for hiding this comment

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

🚢

Now that we are programmatically creating networks (see ceramic-tests)
we want to make sure we are not leaving lots of unused resources around.

With this feature the test driver can create networks with a generous
TTL of a few hours (when we expect tests to run in a few minutes).
If for some reason the test driver fails to clean up the network itself,
eventually the operator will clean it up.

This feature remains optional so manually created networks are not
affected.
Copy link
Contributor

@stbrody stbrody left a comment

Choose a reason for hiding this comment

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

lgtm

@nathanielc nathanielc added this pull request to the merge queue Oct 12, 2023
Merged via the queue into main with commit 1dbeb31 Oct 12, 2023
5 checks passed
@nathanielc nathanielc deleted the feat/network_ttl branch October 16, 2023 15:01
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