Skip to content
This repository was archived by the owner on Jun 27, 2025. It is now read-only.

Use deployment watcher if there's a deploy ID #316

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

Conversation

Mongey
Copy link

@Mongey Mongey commented Dec 2, 2019

Previously levant would check the template contained an update stanza at
the job level, before watching for a service job deployment to complete.

With this change, we'll always wait for a deployment to complete. This
allows the Nomad server to be the one to determine if the template has a
"deployment" rather than levant.

Fixes #145

@Mongey Mongey force-pushed the cm-continue-when-deploy-id branch from 2439a12 to dd0f7f0 Compare December 18, 2020 01:17
Base automatically changed from master to main March 3, 2021 19:50
@Mongey Mongey force-pushed the cm-continue-when-deploy-id branch from 008799f to 0cddd2a Compare April 7, 2021 18:04
Previously levant would check the template contained an update stanza at
the job level, before watching for a service job deployment to complete.

With this change, we'll always wait for a deployment to complete. This
allows the Nomad server to be the one to determine if the template has a
"deployment" rather than levant.
Currently levant panics if there's a failed deployment for a job without
an update stanza.
@Mongey Mongey force-pushed the cm-continue-when-deploy-id branch from f80b31f to e04dd53 Compare January 24, 2022 17:38
@hashicorp-cla
Copy link

hashicorp-cla commented Mar 12, 2022

CLA assistant check
All committers have signed the CLA.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Levant Deployment Service Deployment: Update Stanza
2 participants