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.

Sign up for GitHub

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

Jump to bottom

[Merged by Bors] - Enable logging #407

Closed
wants to merge 28 commits into from
Closed

[Merged by Bors] - Enable logging #407

wants to merge 28 commits into from

Conversation

siegfriedweber
Copy link
Member

@siegfriedweber siegfriedweber commented Feb 27, 2023 •
edited by maltesander
Loading

Description

Add log aggregation

Closes #364

Integration tests

Build Status

Review Checklist

  • Code contains useful comments
  • CRD change approved (or not applicable)
  • (Integration-)Test cases added (or not applicable)
  • Documentation added (or not applicable)
  • Changelog updated (or not applicable)
  • Cargo.toml only contains references to git tags (not specific commits or branches)
  • Helm chart can be installed and deployed operator works (or not applicable)

Once the review is done, comment bors r+ (or bors merge) to merge. Further information

@siegfriedweber
Implement configuration merge
77067d0
@siegfriedweber
Add replicas and selector to CommonConfig
e4dd4f7
@siegfriedweber
Remove unused error variants
cb1cd8c
@siegfriedweber
Update changelog
f1fae47
@siegfriedweber
Upgrade operator-rs to version 0.35.0
ba11a8f
@siegfriedweber
Add code comments
b4dd7fe
@siegfriedweber
Enable log aggregation
ccc5f6d
@siegfriedweber
Add logging test
7eded34
@siegfriedweber
Enable log aggregation in authorizer test
c49233c
@siegfriedweber
Enable log aggregation in hdfs-deep-storage test
ede7f9a
@siegfriedweber
Enable log aggregation in ingestion-no-s3-ext test
3a1c662
@siegfriedweber
Enable log aggregation in ingestion-s3-ext test
5f17132
@siegfriedweber
Enable log aggregation in ldap-authentication test
3cd0edb
@siegfriedweber
Enable log aggregation in orphand-resources test
35ccdba
@siegfriedweber
Enable log aggregation in resources test
16257c3
@siegfriedweber
Enable log aggregation in s3-deep-storage test
eb052ef
@siegfriedweber
Enable log aggregation in smoke test
012f54a
@siegfriedweber
Enable log aggregation in tls test
b8183b4
@siegfriedweber
Fix custom configuration in logging test
9389036
@siegfriedweber
Regenerate charts
f95bb6b
@siegfriedweber
Upgrade stackableVersions of images
0942915
@siegfriedweber
Add section about log aggregation to the documentation
2694fe2
@siegfriedweber
Update changelog
19dbb67
@siegfriedweber
Merge branch 'main' into enable-logging
c2bdda5
@siegfriedweber siegfriedweber self-assigned this Feb 27, 2023
@siegfriedweber siegfriedweber marked this pull request as ready for review February 28, 2023 11:03
@vsupalov vsupalov mentioned this pull request Feb 28, 2023
Closed
7 tasks
maltesander
maltesander reviewed Feb 28, 2023
View reviewed changes
Copy link
Member

@maltesander maltesander left a comment

Choose a reason for hiding this comment

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

Minor things, will continue tomorrow!

rust/operator-binary/src/druid_controller.rs Outdated Show resolved Hide resolved
rust/crd/src/lib.rs Show resolved Hide resolved
maltesander
maltesander approved these changes Mar 1, 2023
View reviewed changes
Copy link
Member

@maltesander maltesander left a comment

Choose a reason for hiding this comment

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

LGTM!

@siegfriedweber
Copy link
Member Author

bors merge

bors bot pushed a commit that referenced this pull request Mar 1, 2023
@siegfriedweber
Enable logging (#407)
d29ee11 
# Description

Add log aggregation
@bors
Copy link
Contributor

bors bot commented Mar 1, 2023

Pull request successfully merged into main.

Build succeeded:

@bors bors bot changed the title Enable logging [Merged by Bors] - Enable logging Mar 1, 2023
@bors bors bot closed this Mar 1, 2023
@bors bors bot deleted the enable-logging branch March 1, 2023 11:23
@sbernauer sbernauer mentioned this pull request Mar 3, 2023
Closed
7 tasks
bors bot pushed a commit that referenced this pull request Mar 6, 2023
@sbernauer
Fix failing ldap test (#410)
35c7fce 
# Description

During resolving conflicts between #407 and #408 this line was not removed, thus creating (and overwriting!) the truststore *after* the ldap tls cert was added. Result was the ldap tls cert missing from the truststore
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@maltesander maltesander maltesander approved these changes

Assignees

@siegfriedweber siegfriedweber

Labels
None yet
Projects
Stackable Engineering
Archived in project
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Enable Log Aggregation for Druid
2 participants
@siegfriedweber @maltesander