sensuctl entity update should support all of the managed entity fields - Githubissues

sensu / sensu-go

Simple. Scalable. Multi-cloud monitoring.

https://sensu.io

MIT License

1.03k stars 175 forks source link

sensuctl entity update should support all of the managed entity fields #4004

Open calebhailey opened 4 years ago

calebhailey commented 4 years ago

Feature Suggestion

Sensu 6.0 introduced the ability to manage agent entities via the Sensu API. The agent entity fields that are managed via the API are as follows:

entity.metadata.name
entity.metadata.namespace
entity.metadata.labels
entity.metadata.annotations
entity.spec.entity_class
entity.spec.user
entity.spec.subscriptions
entity.spec.deregister
entity.spec.deregistration.handler
entity.spec.keepalive_handlers
entity.spec.redact

At this time, the only fields supported by the interactive sensuctl entity update command are:

entity.spec.entity_class
entity.spec.subscriptions

Screenshot:

Let's support all of the editable fields!

Possible Implementation

Support all of the editable entity fields via sensuctl entity update.

Context

Comprehensive entity management via sensuctl.

hurrycaine commented 3 years ago

What about not-interactive, using flags. Is that something planned or you guys would accept PR for? I also could see an append flag being nice on the list type fields. That way you could add a subscription without having to pull and add the new one.

Something like

./sensuctl entity update [ID] -subscriptions newsub -A