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

docs: add instructions on how to use idempotent runs in our docs #1286

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

leeederek
Copy link
Contributor

@leeederek leeederek commented Sep 11, 2023

Description:

We've enabled idempotent runs by default for a while now, but it turns out we don't actually have docs on how to actually use it. This PR adds a quick blurb in our docs to illustrate how a user would use idempotent runs for their enclave.

Is this change user facing?

YES

References (if applicable):

Related to, but a direct fix for: #1266

@cloudflare-pages
Copy link

cloudflare-pages bot commented Sep 11, 2023

Deploying with  Cloudflare Pages  Cloudflare Pages

Latest commit: 0fc96a7
Status: ✅  Deploy successful!
Preview URL: https://96a1c780.kurtosis-docs.pages.dev
Branch Preview URL: https://derek-add-idempotent-runs-to.kurtosis-docs.pages.dev

View logs

@leeederek leeederek enabled auto-merge (squash) September 11, 2023 22:12
@leeederek leeederek removed the request for review from galenmarchetti September 12, 2023 14:43
@leeederek leeederek added the DO NOT MERGE DO NOT MERGE THIS PR label Sep 12, 2023
@leeederek
Copy link
Contributor Author

Got some additional tidying up to do actually - will re-request a review once I add a code example

auto-merge was automatically disabled October 13, 2023 13:09

Merge queue setting changed

@adschwartz
Copy link
Contributor

@leeederek can we merge or close this PR?

@leeederek
Copy link
Contributor Author

Not yet - this is on my TODO list for adding code examples. Is there something that this PR is blocking?

@leeederek leeederek removed the DO NOT MERGE DO NOT MERGE THIS PR label Nov 3, 2023
@leeederek
Copy link
Contributor Author

@galenmarchetti i was going to add code examples but lets just do that in a separate PR

auto-merge was automatically disabled November 11, 2023 01:04

Merge queue setting changed

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.

None yet

4 participants