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

Periodic job transitions to dead on update #2268

Closed
dadgar opened this issue Feb 1, 2017 · 7 comments · Fixed by #2486
Closed

Periodic job transitions to dead on update #2268

dadgar opened this issue Feb 1, 2017 · 7 comments · Fixed by #2486

Comments

@dadgar
Copy link
Contributor

dadgar commented Feb 1, 2017

Steps to reproduce

  1. nomad run example.nomad
  2. edit test -> test2
  3. nomad run example.nomad
  4. nomad status example: It will be dead.
job "example" {
  type = "batch"
  datacenters = ["dc1"]
  periodic {
     cron = "* * * * * *"
     prohibit_overlap = true
  }
  group "foo" {
    task "foo" {
      driver = "raw_exec"
      config {
        command = "echo"
        args = ["test2"]
      }
    }
  }
}
@jippi
Copy link
Contributor

jippi commented Feb 1, 2017

Haha, i've always thought that was the intended state for a periodic job waiting for something to execute... interesting :D

@sepulworld
Copy link
Contributor

Yeah, I was confused with the 'dead' status. Even with the 'dead' status the periodic job still appears to run.

@dadgar dadgar added this to the v0.6.0 milestone Mar 16, 2017
@briansorahan
Copy link

I wish I would have seen this issue a week ago!
We just started implementing a bulk update to all our periodic jobs and have run into this.

@briansorahan
Copy link

@dadgar Is this simply a matter of setting status correctly when a periodic job is updated? I'd like to submit a PR if so.
It seems to me like @sepulworld 's comment is correct. The periodic jobs that are marked as "dead" seem to be functioning correctly

dadgar added a commit that referenced this issue Mar 27, 2017
This PR fixes an issue in which a periodic job would incorrectly
transistion to status dead.

Fixes #2268
@dadgar
Copy link
Contributor Author

dadgar commented Mar 27, 2017

@briansorahan Appreciate the offer! I went ahead and fixed it as it is somewhat subtle! In the meantime, the periodic job will have the incorrect state but it will function correctly. We should have a release out soon though!

@briansorahan
Copy link

@dadgar Thank you!

@github-actions
Copy link

I'm going to lock this issue because it has been closed for 120 days ⏳. This helps our maintainers find and focus on the active issues.
If you have found a problem that seems similar to this, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Dec 14, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants