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

Fix an output bug in paasta check #282

Merged
merged 1 commit into from
Mar 1, 2016
Merged

Fix an output bug in paasta check #282

merged 1 commit into from
Mar 1, 2016

Conversation

mjksmith
Copy link
Contributor

@mjksmith mjksmith commented Mar 1, 2016

Right now if there are chronos jobs that aren't listed in deploy.yaml, paasta check prints out a list of marathon instances instead.

Example:

✗ There are some instance(s) you have asked to run in chronos that
  do not have a corresponding entry in deploy.yaml:

  You should probably add entries to deploy.yaml for them so they
  are deployed to those clusters.

@mjksmith mjksmith self-assigned this Mar 1, 2016
@nhandler
Copy link
Contributor

nhandler commented Mar 1, 2016

Ship It!
Ship It!

mjksmith added a commit that referenced this pull request Mar 1, 2016
Fix an output bug in paasta check
@mjksmith mjksmith merged commit 30cddf5 into master Mar 1, 2016
@mjksmith mjksmith removed the in review label Mar 1, 2016
@mjksmith mjksmith deleted the fix_paasta_check branch March 1, 2016 21:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants