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

Pause on simulation delay trigger now action #1395

Closed
RomuDeuxfois opened this issue Sep 6, 2024 · 2 comments · Fixed by #1424
Closed

Pause on simulation delay trigger now action #1395

RomuDeuxfois opened this issue Sep 6, 2024 · 2 comments · Fixed by #1424
Assignees
Labels
bug use for describing something not working as expected solved The issue has been solved
Milestone

Comments

@RomuDeuxfois
Copy link
Member

Description

If I launch my simulation and then pause it for 30 minutes, when I restart my simulation everything goes well.
On the other hand, if after this restart I decide to trigger now on this inject, then it will execute X time later X being the duration of the pause that took place (30 minutes for example).

Reproducible Steps

Steps to create the smallest reproducible scenario:

  1. Create a simulation with injects (different trigger time)
  2. Launch the simulation
  3. Pause the simulation before all injects is executed
  4. Resume the simulation
  5. Trigger now one inject
  6. Observe

Expected Output

The inject is executed now

Actual Output

The inject is executed after X minutes

@RomuDeuxfois RomuDeuxfois added bug use for describing something not working as expected needs triage use to identify issue needing triage from Filigran Product team and removed needs triage use to identify issue needing triage from Filigran Product team labels Sep 6, 2024
@RomuDeuxfois RomuDeuxfois added this to the Bugs backlog milestone Sep 6, 2024
@savacano28 savacano28 self-assigned this Sep 12, 2024
@savacano28
Copy link
Contributor

I can see: Image

@savacano28
Copy link
Contributor

Currently, we include pauses from exercise to define the date of execution, so we have to add a condition in order to ignore pauses when we make a trigger now (instant 0): Image

@MarineLeM MarineLeM linked a pull request Sep 16, 2024 that will close this issue
5 tasks
MarineLeM added a commit that referenced this issue Sep 18, 2024
@MarineLeM MarineLeM added solved The issue has been solved and removed solved The issue has been solved labels Sep 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug use for describing something not working as expected solved The issue has been solved
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants