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

Implement Sensu Go plugin to allow Terraform to use Sensu secrets #4337

Open
mgibson323 opened this issue Jun 25, 2021 · 4 comments
Open

Implement Sensu Go plugin to allow Terraform to use Sensu secrets #4337

mgibson323 opened this issue Jun 25, 2021 · 4 comments
Labels
component:secrets Sensu Secrets Management
Milestone

Comments

@mgibson323
Copy link

Right now, it looks like the Sensu Go client library omits the secrets type in the check/checkconfig definition.

Feature Suggestion

A plugin that allows Terraform to use Sensu secrets.

Possible Implementation

Context

This is related to this PR.

@calebhailey
Copy link

If I'm following this correctly, I believe we'd need to update every resource in the Sensu Go client package that supports secrets, including:

  • core/v2.CheckConfig
  • core/v2.Handler

@calebhailey calebhailey added the component:secrets Sensu Secrets Management label Aug 2, 2021
@calebhailey calebhailey added this to the 6.6.0 milestone Aug 17, 2021
@calebhailey calebhailey modified the milestones: 6.next, 6.6.0 Sep 24, 2021
@calebhailey calebhailey modified the milestones: 6.6.0, 6.next Nov 15, 2021
@calebhailey calebhailey modified the milestones: 6.next, 7.next Aug 19, 2022
@devinlyons
Copy link

@calebhailey, I see this feature keeps getting pushed back. Is this something I can contribute?

@calebhailey
Copy link

@devinlyons I'm no longer with the company (since September 2022). Sensu was acquired by Sumo Logic in June 2021, and again by a private equity firm last month. I'm not sure who's heading up product development these days, but I'm sure I wouldn't be speaking out of turn to say that contributions are always welcome! 😊

@devinlyons
Copy link

devinlyons commented Jun 15, 2023

It seems like this feature is already implemented. The sensu/core library already has the correct types and they are being used in the current version of the client.

Does this issue just need to be closed?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component:secrets Sensu Secrets Management
Projects
None yet
Development

No branches or pull requests

3 participants