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

feat: Add self monitoring to log gateway #1813

Merged
merged 23 commits into from
Feb 19, 2025
Merged

feat: Add self monitoring to log gateway #1813

merged 23 commits into from
Feb 19, 2025

Conversation

rakesh-garimella
Copy link
Contributor

@rakesh-garimella rakesh-garimella commented Feb 7, 2025 •
edited
Loading

Description

Changes proposed in this pull request (what was done and why):

  • Add selfmonitor to otel log pipeline

  • Add new conditions for otel log pipelines and abstract logpipleine conditions into common, flunetbit and otel logs

  • add e2e tests

  • Reason messages are not complete for otel logs as the doc links are missing. This would be handled later.
    Changes refer to particular issues, PRs or documents:

  • OTLP Logs: self-monitoring capabilities for log gateway #1800

Traceability

  • The PR is linked to a GitHub issue.
  • The follow-up issues (if any) are linked in the Related Issues section.
  • If the change is user-facing, the documentation has been adjusted.
  • If a CRD is changed, the corresponding Busola ConfigMap has been adjusted.
  • The feature is unit-tested.
  • The feature is e2e-tested.

@kyma-bot kyma-bot added the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Feb 7, 2025
@rakesh-garimella rakesh-garimella added area/logs LogPipeline and removed do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. labels Feb 7, 2025
@kyma-bot kyma-bot added cla: yes Indicates the PR's author has signed the CLA. size/L Denotes a PR that changes 100-499 lines, ignoring generated files. labels Feb 7, 2025
@rakesh-garimella rakesh-garimella changed the title Add self monitoring to log gateway [feat] Add self monitoring to log gateway Feb 7, 2025
@kyma-bot kyma-bot added the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Feb 7, 2025
@kyma-bot kyma-bot added size/XL Denotes a PR that changes 500-999 lines, ignoring generated files. and removed size/L Denotes a PR that changes 100-499 lines, ignoring generated files. labels Feb 12, 2025
@rakesh-garimella rakesh-garimella marked this pull request as ready for review February 12, 2025 16:27
@rakesh-garimella rakesh-garimella requested a review from a team as a code owner February 12, 2025 16:27
@kyma-bot kyma-bot removed the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Feb 12, 2025
@rakesh-garimella rakesh-garimella added this to the 1.34.0 milestone Feb 12, 2025
@kyma-bot kyma-bot added size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. and removed size/XL Denotes a PR that changes 500-999 lines, ignoring generated files. labels Feb 13, 2025
@rakesh-garimella rakesh-garimella added the kind/feature Categorizes issue or PR as related to a new feature. label Feb 13, 2025
@github-actions github-actions bot removed the kind/feature Categorizes issue or PR as related to a new feature. label Feb 13, 2025
@rakesh-garimella rakesh-garimella changed the title [feat] Add self monitoring to log gateway feat: Add self monitoring to log gateway Feb 13, 2025
@github-actions github-actions bot added the kind/feature Categorizes issue or PR as related to a new feature. label Feb 14, 2025
@rakesh-garimella rakesh-garimella added the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Feb 18, 2025
@kyma-bot kyma-bot added the lgtm Looks good to me! label Feb 18, 2025
@rakesh-garimella rakesh-garimella enabled auto-merge (squash) February 19, 2025 15:01
@rakesh-garimella rakesh-garimella modified the milestones: 1.34.0, 1.35.0 Feb 19, 2025
@kyma-bot kyma-bot removed the lgtm Looks good to me! label Feb 19, 2025
@kyma-bot
Copy link
Contributor

New changes are detected. LGTM label has been removed.

@rakesh-garimella rakesh-garimella removed the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Feb 19, 2025
@rakesh-garimella rakesh-garimella merged commit debbb0d into kyma-project:main Feb 19, 2025
54 of 56 checks passed
@rakesh-garimella rakesh-garimella deleted the add_selfmon_log_gw branch February 20, 2025 08:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@k15r k15r k15r approved these changes

Assignees

@k15r k15r

Labels
area/logs LogPipeline cla: yes Indicates the PR's author has signed the CLA. kind/feature Categorizes issue or PR as related to a new feature. size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files.
Projects
None yet
Milestone
1.35.0
Development

Successfully merging this pull request may close these issues.
3 participants
@rakesh-garimella @kyma-bot @k15r