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

Provide a way to allow analyses to complete the experiment #248

Closed
jessesuen opened this issue Oct 31, 2019 · 0 comments · Fixed by #345
Closed

Provide a way to allow analyses to complete the experiment #248

jessesuen opened this issue Oct 31, 2019 · 0 comments · Fixed by #345
Labels
enhancement New feature or request experiment Related to Experiment CRD S

Comments

@jessesuen
Copy link
Member

Currently Experiments either run forever, or complete after a specified duration. A new mode should be that a user can decide that an Experiment should run until one or more analyses complete.

We will probably need a way to mark individual analyses as required for completion. e.g.:

spec:
  analyses:
  # success-rate is background analysis that we want running until experiment completes
  - name: success-rate
    templateName: success-rate

  # test1 and test2 are tests which we want to complete and terminate the experiment after
  - name: test1
    templateName: test1
    requiredForCompletion: true
  - name: test2
    templateName: test2
    requiredForCompletion: true
@jessesuen jessesuen added analysis Related to Analysis CRD experiment Related to Experiment CRD enhancement New feature or request labels Oct 31, 2019
@jessesuen jessesuen changed the title Provide a way to allow analyses to terminate the experiment Provide a way to allow analyses to complete the experiment Oct 31, 2019
@malinafong malinafong added S and removed analysis Related to Analysis CRD labels Dec 2, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request experiment Related to Experiment CRD S
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants