-
Notifications
You must be signed in to change notification settings - Fork 33
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
Refresh auth context while polling for clustergroup backupschedule #393
Conversation
@rnarenpujari, you must sign every commit in this pull request acknowledging our Developer Certificate of Origin before your changes are merged. This can be done by adding
|
Signed-off-by: Rahul Naren Pujari <rnarenpujari@gmail.com>
1ca7d5a
to
ff99714
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #393 +/- ##
=======================================
Coverage 25.23% 25.23%
=======================================
Files 195 195
Lines 16908 16908
=======================================
Hits 4267 4267
Misses 12423 12423
Partials 218 218 ☔ View full report in Codecov by Sentry. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
I'm going to lock this pull request because it has been closed for 30 days. This helps our maintainers find and focus on the active issues. If you have found a problem that seems related to this change, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further. |
Missed one case in this earlier PR: #390
The acceptance tests for this resource are failing for a different reason and that is being tracked separately. This is a minor change and should not cause any regression.
Which issue(s) this PR fixes
Additional information
Special notes for your reviewer