Skip to content

Delete and Print actions for SignalCopy console applicatoin - #40

Closed
elv1s42 wants to merge 3 commits into
datalust:devfrom
elv1s42:dev
Closed

Delete and Print actions for SignalCopy console applicatoin#40
elv1s42 wants to merge 3 commits into
datalust:devfrom
elv1s42:dev

Conversation

@elv1s42

@elv1s42 elv1s42 commented Oct 19, 2017

Copy link
Copy Markdown

Hi!

Just want to share a couple of more actions for SignalCopy:
Action (--action) can be 'copy', 'print' or 'delete'. The delete action works by default for all signals, but you can also specify --signal=.

I my case I had a couple of deleted (via Seq Web GUI) signal, which were copied into another server. So I had to remove them via Seq API.

@nblumhardt

Copy link
Copy Markdown
Member

Thanks for the PR!

The purpose of signal-copy in this repo is to provide a codebase that can be extended and modified to fit different administrative scenarios, so I think adding more features to it might start to detract from that goal.

Your suggestions do raise the interesting possibility of building/shipping a more flexible Seq admin client... I can see things like:

# Creates a .seq-admin file with server details
seq-admin login -s https://my-seq -u admin -p somepass
seq-admin signal export -n "Production Errors" -o production-errors.json
seq-admin signal delete -n "Production Errors"
seq-admin signal import -i production-errors.json
...

This might be a bit of a pipe-dream, given current time and resource constraints, but there's the possibility we could create a new, mostly-empty repository and just chip away at it slowly. What do you think?

@elv1s42

elv1s42 commented Oct 20, 2017

Copy link
Copy Markdown
Author

Hi @nblumhardt,
I think this is a good idea and ready to contribute (at least for the commands, that may be useful to me).
At the moment I am using custom SignalCopy console app builded with net45 traget framework.
It would be great to have something officially approved by Datalust organization.

@nblumhardt

Copy link
Copy Markdown
Member

Moving to datalust/seqcli#2 :-)

@nblumhardt nblumhardt closed this Feb 9, 2018
@nblumhardt

Copy link
Copy Markdown
Member

Just a quick note, this has been published in the latest seqcli preview.

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