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

Consolidated CLI, dead letter management #117

Merged
merged 7 commits into from Feb 13, 2017
Merged

Consolidated CLI, dead letter management #117

merged 7 commits into from Feb 13, 2017

Conversation

rclark
Copy link
Contributor

@rclark rclark commented Feb 12, 2017

Sets up a framework for a consolidated CLI, currently with 2 commands available:

> watchbot --help

  Build an AWS stack to do your work for you

  USAGE: watchbot <command> [OPTIONS]

  Commands:
    worker-capacity     assess available resources on the cluster
    dead-letter         triage messages in dead letter queue

  Options:
    -h, --help          show this help message
    -s, --stack-name    the full name of a watchbot stack
    -r, --region        the region of the stack (default us-east-1)

This incorporates @emilymdubois' worker-capacity function, and adds a command for interacting with the dead letter queue:

> watchbot dead-letter -r ap-northeast-1 -s ecs-telephone-rclark
? Would you like to: (Use arrow keys)
❯ Triage dead messages individually? 
  Print out all dead messages? 
  Return all dead messages to the work queue? 
  Purge the dead letter queue? 

These options are pretty self-explanatory. The individual triage pulls one message from the dead letter queue, prints it out for you, and asks you if you want to:

? Would you like to: (Use arrow keys)
❯ View this message's recent logs?
  Return this message to the work queue?,
  Return this message to the dead letter queue?,
  Delete this message entirely?,
  Stop individual triage?

Upon making a selection, it does that, then presents you with another dead message (if there is one).

TODO:

  • fix worker capacity tests
  • tests for new command
  • fix CLI description text
  • view recent logs for individual jobs
  • spinner / progress indication on long processes
  • try to improve cloudwatch log search speeds this will require some work in cwlogs.

@rclark rclark merged commit 7970895 into master Feb 13, 2017
@rclark rclark deleted the cli branch February 13, 2017 20:29
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.

None yet

1 participant