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

feat: emit Kubernetes events for error conditions #598

Merged
merged 8 commits into from
Jan 12, 2024
Merged

feat: emit Kubernetes events for error conditions #598

merged 8 commits into from
Jan 12, 2024

Conversation

apricote
Copy link
Member

@apricote apricote commented Jan 8, 2024

k/cloud-provider emits Events for most errors we return. In some conditions we previously hid the errors from k/cloud-provider for various reasons and only logged a warning. This is hard to discover for users. With this change, we now also emit Kubernetes Events for these cases.

@apricote apricote added the enhancement New feature or request label Jan 8, 2024
Copy link

codecov bot commented Jan 8, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

❗ No coverage uploaded for pull request base (main@f303a5a). Click here to learn what that means.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #598   +/-   ##
=======================================
  Coverage        ?   74.74%           
=======================================
  Files           ?       21           
  Lines           ?     2926           
  Branches        ?        0           
=======================================
  Hits            ?     2187           
  Misses          ?      561           
  Partials        ?      178           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@phm07 phm07 marked this pull request as ready for review January 11, 2024 14:30
@phm07 phm07 requested a review from a team as a code owner January 11, 2024 14:30
Copy link
Member Author

@apricote apricote left a comment

Choose a reason for hiding this comment

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

LGTM from my side

@apricote apricote requested a review from jooola January 12, 2024 11:29
@apricote apricote merged commit e8f9199 into main Jan 12, 2024
8 checks passed
@apricote apricote deleted the events branch January 12, 2024 11:37
apricote pushed a commit that referenced this pull request Jul 8, 2024
🤖 I have created a release *beep* *boop*
---


##
[1.20.0](v1.19.0...v1.20.0)
(2024-07-08)


### Features

* add support & tests for Kubernetes 1.29
([#600](#600))
([e8fabda](e8fabda))
* add support & tests for Kubernetes 1.30
([#679](#679))
([0748b6e](0748b6e))
* drop tests for kubernetes v1.25
([#597](#597))
([58261ec](58261ec))
* drop tests for kubernetes v1.26
([#680](#680))
([9c4be01](9c4be01))
* emit Kubernetes events for error conditions
([#598](#598))
([e8f9199](e8f9199))
* **helm,manifests:** only specify container args instead of command
([#691](#691))
([2ba4058](2ba4058))
* **helm:** allow setting affinity for deployment
([#686](#686))
([1a8ea95](1a8ea95))
* read HCLOUD_TOKEN from file
([#652](#652))
([a4343b8](a4343b8))


### Bug Fixes

* **routes:** many requests for outdated routes by rate limiting
([#675](#675))
([e283b7d](e283b7d))

---
This PR was generated with [Release
Please](/~https://github.com/googleapis/release-please). See
[documentation](/~https://github.com/googleapis/release-please#release-please).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants