Skip to content

Add condition field to Keycloak log datastream - #11213

Merged
andrewkroh merged 4 commits into
elastic:mainfrom
Alphayeeeet:feature-keycloak-add-condition
Sep 26, 2024
Merged

Add condition field to Keycloak log datastream#11213
andrewkroh merged 4 commits into
elastic:mainfrom
Alphayeeeet:feature-keycloak-add-condition

Conversation

@Alphayeeeet

Copy link
Copy Markdown
Contributor

Please label as Enhancement

This PR adds the concition field as configurable option to the Keycloak log datastream

@Alphayeeeet
Alphayeeeet requested a review from a team as a code owner September 23, 2024 15:46
@andrewkroh andrewkroh added bugfix Pull request that fixes a bug issue Integration:keycloak Keycloak Team:Security-Service Integrations Security Service Integrations team [elastic/security-service-integrations] labels Sep 23, 2024
@elasticmachine

Copy link
Copy Markdown

Pinging @elastic/security-service-integrations (Team:Security-Service Integrations)

@Alphayeeeet

Copy link
Copy Markdown
Contributor Author

@andrewkroh FYI this PR is not a bugfix, but rather an enhancement.

Comment thread packages/keycloak/changelog.yml Outdated
Comment thread packages/keycloak/data_stream/log/manifest.yml Outdated
@andrewkroh andrewkroh added enhancement New feature or request and removed bugfix Pull request that fixes a bug issue labels Sep 23, 2024
@Alphayeeeet

Alphayeeeet commented Sep 24, 2024

Copy link
Copy Markdown
Contributor Author

@andrewkroh Could you please run CI again?

Comment thread packages/keycloak/changelog.yml Outdated
@andrewkroh

Copy link
Copy Markdown
Contributor

/test

@Alphayeeeet

Copy link
Copy Markdown
Contributor Author

@andrewkroh Package tests look alright, but CI has a problem when stopping test environment. Is this an issue of the integration or of the CI environment?

Co-authored-by: Andrew Kroh <andrew.kroh@elastic.co>
@andrewkroh

Copy link
Copy Markdown
Contributor

/test

@andrewkroh

Copy link
Copy Markdown
Contributor

It's a CI issue. I am told that #11227 is testing a fix.

@andrewkroh

Copy link
Copy Markdown
Contributor

/test

@elastic-vault-github-plugin-prod

Copy link
Copy Markdown
Contributor

🚀 Benchmarks report

To see the full report comment with /test benchmark fullreport

@elasticmachine

Copy link
Copy Markdown

💚 Build Succeeded

History

@andrewkroh
andrewkroh merged commit 5205b4a into elastic:main Sep 26, 2024
@elastic-vault-github-plugin-prod

Copy link
Copy Markdown
Contributor

Package keycloak - 1.23.0 containing this change is available at https://epr.elastic.co/search?package=keycloak

harnish-crest-data pushed a commit to chavdaharnish/integrations that referenced this pull request Feb 4, 2025
Make condition configurable to control when to apply the log data stream.
The elastic agent evaluates the condition to determine whether or not to
execute the input.
harnish-crest-data pushed a commit to chavdaharnish/integrations that referenced this pull request Feb 5, 2025
Make condition configurable to control when to apply the log data stream.
The elastic agent evaluates the condition to determine whether or not to
execute the input.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request Integration:keycloak Keycloak Team:Security-Service Integrations Security Service Integrations team [elastic/security-service-integrations]

3 participants