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.

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

skip and log invalid k8s watch events #2099

Merged
merged 1 commit into from
Jun 5, 2023

Conversation

zubenkoivan
Copy link
Contributor

@zubenkoivan zubenkoivan commented Jun 5, 2023

Sometimes invalid watch events are sent by k8s, currently there is no way to skip them and they generate a lot of logs in Google Cloud Logging system.

@codecov
Copy link

codecov bot commented Jun 5, 2023

Codecov Report

Merging #2099 (63d23a0) into master (b908c8a) will decrease coverage by 22.43%.
The diff coverage is 0.00%.

Additional details and impacted files

Impacted file tree graph

@@             Coverage Diff             @@
##           master    #2099       +/-   ##
===========================================
- Coverage   92.15%   69.72%   -22.43%     
===========================================
  Files          41       41               
  Lines        7317     7324        +7     
  Branches     1090     1091        +1     
===========================================
- Hits         6743     5107     -1636     
- Misses        429     2101     +1672     
+ Partials      145      116       -29     
Flag Coverage Δ
integration ?
unit 69.72% <0.00%> (-0.07%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
platform_api/orchestrator/kube_client.py 66.35% <0.00%> (-22.93%) ⬇️

... and 26 files with indirect coverage changes

Copy link
Contributor

@YevheniiSemendiak YevheniiSemendiak left a comment

Choose a reason for hiding this comment

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

LGTM

@zubenkoivan zubenkoivan merged commit c048fdb into master Jun 5, 2023
@zubenkoivan zubenkoivan deleted the iz/skip-and-log-invalid-watch-events branch June 5, 2023 13:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants