-
Notifications
You must be signed in to change notification settings - Fork 712
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
Initial Commit of Basic Operator for Logstash #6404
Merged
Merged
Changes from 43 commits
Commits
Show all changes
46 commits
Select commit
Hold shift + click to select a range
ce56d6a
Initial Commit of ECK for Logstash
robbavey 02a5451
Comment out certs for HTTPS for now
robbavey b72182f
Fix up linter issus
robbavey 9cf0a45
Generate API docs
robbavey 4d62ab5
First set of unit tests
robbavey cf1379c
Fix goimports
robbavey e5a4621
Add version check
robbavey a74da1d
Add e2e tests
robbavey d3878a5
Temporarily take out probes
robbavey cd7f8f6
Revert "Temporarily take out probes"
robbavey 793ff28
Revert "Add e2e tests"
robbavey 1a83870
Fix linter
robbavey d3edd1e
Add logstash config details
robbavey 4e4303d
Fix up typos
robbavey 50e834e
Revert "Revert "Add e2e tests""
robbavey 2692661
Revert "Revert "Temporarily take out probes""
robbavey 865d06a
Tidying up
robbavey 84dd489
Add Logstash to sample and stack tests
robbavey 12430d9
Added basic logstash verification
robbavey d2ac277
Fix readiness probe
robbavey 214f63d
Tidy up
robbavey b9cb06a
Update config/crds/v1/patches/kustomization.yaml
kaisecheng 51f2bad
Update pkg/apis/logstash/v1alpha1/name.go
kaisecheng b52d9ee
change HTTPConfig to []LogstashService
kaisecheng 94b84b5
webhook check all rules
kaisecheng 1cfb725
remove building config from existing config in Secret
kaisecheng bb79c17
rename k8sutils EmitErrorEvent() to MaybeEmitErrorEvent()
kaisecheng 8ec1501
remove nil error in StatefulSet
kaisecheng aec57e4
remove APIError struct in test
kaisecheng b27445c
add license header
kaisecheng 746a307
lint
kaisecheng 822f0ab
reconcileConfig return error
kaisecheng 0a37afe
change the default Service to headless service
kaisecheng df96068
fix service test
kaisecheng 4c414bd
fix podTemplate and doc generation
kaisecheng 018dfbc
Fix Service tests
robbavey de1fb22
Update comments to reflect change from HTTPConfig to LogstashService
robbavey 41612f7
Update pkg/controller/logstash/service.go
robbavey a0d176a
remove duplicate err check
kaisecheng b9610da
Add support for overriding "default" service
robbavey 0c9409a
Replace 'default' service name with 'api'
robbavey 6167cb7
merge from feature/logstash
robbavey d4c5e05
Add tests and checks for services and endpoints
robbavey 1f77430
Update pkg/apis/logstash/v1alpha1/webhook.go
robbavey b677c65
Update pkg/controller/logstash/service.go
robbavey a797940
Addressing code review comments
robbavey File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Large diffs are not rendered by default.
Oops, something went wrong.
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
8,018 changes: 8,018 additions & 0 deletions
8,018
config/crds/v1/bases/logstash.k8s.elastic.co_logstashes.yaml
Large diffs are not rendered by default.
Oops, something went wrong.
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,7 @@ | ||
# Using `kubectl apply` stores the complete CRD file as an annotation, | ||
# which may be too big for the annotations size limit. | ||
# One way to mitigate this problem is to remove the (huge) podTemplate properties from the CRD. | ||
# It also avoids the problem of having any k8s-version specific field in the Pod schema, | ||
# that would maybe not match the user's k8s version. | ||
- op: remove | ||
path: /spec/versions/0/schema/openAPIV3Schema/properties/spec/properties/podTemplate/properties |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,27 @@ | ||
apiVersion: elasticsearch.k8s.elastic.co/v1 | ||
kind: Elasticsearch | ||
metadata: | ||
name: elasticsearch-sample | ||
spec: | ||
version: 8.6.1 | ||
nodeSets: | ||
- name: default | ||
count: 3 | ||
config: | ||
node.store.allow_mmap: false | ||
--- | ||
apiVersion: logstash.k8s.elastic.co/v1alpha1 | ||
kind: Logstash | ||
metadata: | ||
name: logstash-sample | ||
spec: | ||
count: 3 | ||
version: 8.6.1 | ||
config: | ||
log.level: info | ||
api.http.host: "0.0.0.0" | ||
queue.type: memory | ||
podTemplate: | ||
spec: | ||
containers: | ||
- name: logstash |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,46 @@ | ||
apiVersion: elasticsearch.k8s.elastic.co/v1 | ||
kind: Elasticsearch | ||
metadata: | ||
name: elasticsearch-sample | ||
spec: | ||
version: 8.6.1 | ||
nodeSets: | ||
- name: default | ||
count: 3 | ||
config: | ||
node.store.allow_mmap: false | ||
--- | ||
apiVersion: logstash.k8s.elastic.co/v1alpha1 | ||
kind: Logstash | ||
metadata: | ||
name: logstash-sample | ||
spec: | ||
count: 2 | ||
version: 8.6.1 | ||
config: | ||
log.level: info | ||
api.http.host: "0.0.0.0" | ||
queue.type: memory | ||
services: | ||
- name: api | ||
service: | ||
spec: | ||
type: ClusterIP | ||
ports: | ||
- port: 9600 | ||
name: "api" | ||
protocol: TCP | ||
targetPort: 9600 | ||
- name: beats | ||
service: | ||
spec: | ||
type: ClusterIP | ||
ports: | ||
- port: 5044 | ||
name: "filebeat" | ||
protocol: TCP | ||
targetPort: 5044 | ||
- port: 5045 | ||
name: "winlogbeat" | ||
protocol: TCP | ||
targetPort: 5045 |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The purpose of this Elasticsearch resource is not clear.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Will remove
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done