Skip to content
This repository has been archived by the owner on Apr 12, 2022. It is now read-only.

Feature keystore #149

Open
wants to merge 9 commits into
base: main
Choose a base branch
from
Open

Feature keystore #149

wants to merge 9 commits into from

Conversation

mirkenstein
Copy link

See the README_KEYSTORE_SSL.md file.
This pull request adds support for uploading SSL Certificate Authority files for the output Elasticsearch Cluster and the monitored cluster.
Adds support for beats keystore secrets.

@elasticmachine
Copy link
Collaborator

Since this is a community submitted pull request, a Jenkins build has not been kicked off automatically. Can an Elastic organization member please verify the contents of this patch and then kick off a build manually?

@cla-checker-service
Copy link

cla-checker-service bot commented Jun 1, 2021 •
edited
Loading

❌ Author of the following commits did not sign a Contributor Agreement:
, , 15a858c, , , ,

Please, read and sign the above mentioned agreement if you want to contribute to this project

@mirkenstein mirkenstein mentioned this pull request Jun 1, 2021
Closed
mirkenstein
mirkenstein commented Jun 1, 2021
View reviewed changes
Copy link
Author

@mirkenstein mirkenstein left a comment

Choose a reason for hiding this comment

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

Typo correction SLL->SSL
15a858c

defaults/main.yml Outdated Show resolved Hide resolved
mirkenstein
mirkenstein commented Jun 1, 2021
View reviewed changes
Copy link
Author

@mirkenstein mirkenstein left a comment

Choose a reason for hiding this comment

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

I Approve

@mirkenstein
Copy link
Author

Already signed the contributor agreement.

@jmlrt
Copy link
Member

jmlrt commented Jun 23, 2021

jenkins test this please

@jmlrt
Copy link
Member

jmlrt commented Jun 23, 2021

cla/check

@jmlrt
Copy link
Member

jmlrt commented Jul 6, 2021

cla/check

@jmlrt
Copy link
Member

jmlrt commented Jul 6, 2021

jenkins test this please

@jmlrt jmlrt added the feature label Jul 6, 2021
@cutler-scott-newrelic
Copy link

Seems like the CLA checker is broken? I'm still hoping to see this merged eventually as well! I got an email saying this was stale, hopefully it dosen't get dropped.

miromihaylov
miromihaylov previously approved these changes Oct 23, 2021
View reviewed changes
@mirkenstein
Copy link
Author

mirkenstein commented Oct 23, 2021 •
edited
Loading

Can someone remove the extra file tasks/beats-ssl . This file already exists as tasks/beats-ssl.yml

@mirkenstein mirkenstein mentioned this pull request Dec 10, 2021
Open
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@cutler-scott-newrelic cutler-scott-newrelic cutler-scott-newrelic left review comments

@miromihaylov miromihaylov miromihaylov left review comments

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
feature
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@mirkenstein @elasticmachine @jmlrt @cutler-scott-newrelic @miromihaylov