Code Monkey home page Code Monkey logo

Comments (4)

dwang159 avatar dwang159 commented on July 21, 2024

Can you give steps to repro? I think this only happens immediately after an incident is created, before the first notification is sent out (e.g. testing a plan, then immediately looking at the incident page). Here, "step 0" is a little confusing, but technically correct, since Iris hasn't started executing step 1 yet.

from iris.

wleese avatar wleese commented on July 21, 2024
  • Use 'Test Plan' with any plan
  • Switch to Incidents tab
  • Note that in the list view, your incident notes "Current Step: 0"
  • Wait until iris-sender comes by, note that in the list view "Current Step: 1"

In the individual incident view, "$planname: step $n" is used.

I think the confusion is with the language usage. The current wording implies "you are at step X". Which in my understanding means "you are anywhere between the start and the end of step X".

Really, what the current step value means, and the language should reflect: "Completed step X". By using such wording it would be silly to say "Completed step 0". Instead something like "To be processed" would be more appropriate.

If we can agree on that the current language usage suggests we are at step X rather than we've completed step X, I'd suggest keeping the wording and changing the step value displayed (current/"at" vs completed step number).

Thoughts?

from iris.

dwang159 avatar dwang159 commented on July 21, 2024

I'm a little confused by this. When I create a plan with multiple steps (specifically, one notification per step with wait == 1 and count == 2), things seem to behave as one would expect (i.e. "You are at step 1"). For example, after receiving the first message in step 1 but before receiving the second one, going to the incident page resulted in a "Current step: 1". Is the plan you're testing with a little more complicated?

Another nuance with plan steps is that the plan will proceed to step 2 after the first notification in step 1 completes. For example, if I have notification A in step 1 with count == 5, wait == 5 and notification B with count == 1, wait == 5, the plan will proceed to step 2 5 minutes after notification B is sent. Could this account for what you're seeing?

from iris.

wleese avatar wleese commented on July 21, 2024

For example, after receiving the first message in step 1 but before receiving the second one,
going to the incident page resulted in a "Current step: 1".
Is the plan you're testing with a little more complicated?

No, this is exactly what I'm seeing with a plan similar to what you describe.
What you're saying is completely correct. My point is that it's not intuitive - because there's no such thing as a "step 0"

It's like giving someone instructions and later you ask how far along they are. Currently Iris responds:
"I'm at step 0"
Where as a more logical response would be:
"Oh sorry John, I haven't started yet" ;)

Anyway, in the end this is just a nitpick. If I fail to convince anyone then that's fine too :)

from iris.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.