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

feat: pull in auth code from Kapacitor Enterprise #2512

Merged
merged 1 commit into from Mar 29, 2021

Conversation

lesam
Copy link
Contributor

@lesam lesam commented Mar 24, 2021

Closes #2505

And add a config option to enable it.

Required for all non-trivial PRs
  • Rebased/mergable
  • Tests pass
  • CHANGELOG.md updated

)

type Config struct {
Enabled bool `toml:"enabled"`
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Note Enabled is added

And add a config option to enable it.
@lesam lesam marked this pull request as ready for review March 24, 2021 19:32
Copy link
Contributor

@docmerlin docmerlin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks good so far, lets see how manual testing goes before merging.

@docmerlin docmerlin self-requested a review March 29, 2021 14:58
@docmerlin docmerlin merged commit 4da08aa into v1.6-master Mar 29, 2021
@docmerlin docmerlin deleted the pull-in-auth branch March 29, 2021 14:58
docmerlin added a commit that referenced this pull request Apr 7, 2021
* feat: add Zenoss event handler (#2484)

* feat(kafka): route alerts to partitions by ID, allow for configuring the hashing strategy (#2493)

* test: protoc failing on build, skip broken tests (#2506)

* fix: Zenoss testing (#2509)

* fix: Zenoss testing

* test: add missing test

* feat: pull in auth code from Kapacitor Enterprise (#2512)

And add a config option to enable it.

* fix: external Zenoss testability (#2514)

* chore: go mod tidy

* tests: fix tests and go vet from merge

* chore: go fmt

* Update Changelog

* Update Changelog

* Update CHANGELOG.md

Co-authored-by: Daniel Moran <danxmoran@gmail.com>

* Update CHANGELOG.md

Co-authored-by: Daniel Moran <danxmoran@gmail.com>

Co-authored-by: alespour <42931850+alespour@users.noreply.github.com>
Co-authored-by: Daniel Moran <danxmoran@gmail.com>
Co-authored-by: Sam Arnold <sarnold@influxdata.com>
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

2 participants