Skip to content

Conversation

joelanford
Copy link
Member

@joelanford joelanford commented Oct 14, 2024

In order to give users of this library more information about the state of a release, we are adding the History method to the ActionClient interface.

Without this, the only information available via the ActionClient about a release is the latest revision (via the Get method). However this proves insufficient if callers need to know details of release history, for example if an upgrade failed and callers want to know about the most recently deployed release.

In order to give users of this library more information about the state
of a release, we are adding the `History` method to the `ActionClient`
interface.

Without this, the only information is available via the `ActionClient`
about a release is the latest revision (via the `Get` method). However
this proves insufficient if callers need to know details of release
history, for example if an upgrade failed and callers want to know about
the most recently deployed release.

Signed-off-by: Joe Lanford <joe.lanford@gmail.com>
@codecov-commenter
Copy link

Codecov Report

Attention: Patch coverage is 81.81818% with 4 lines in your changes missing coverage. Please review.

Project coverage is 79.69%. Comparing base (08ab7fb) to head (e3aa77e).
Report is 49 commits behind head on main.

Files with missing lines Patch % Lines
pkg/reconciler/internal/fake/actionclient.go 42.85% 3 Missing and 1 partial ⚠️

❗ There is a different number of reports uploaded between BASE (08ab7fb) and HEAD (e3aa77e). Click for more details.

HEAD has 1 upload less than BASE
Flag BASE (08ab7fb) HEAD (e3aa77e)
2 1
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #397      +/-   ##
==========================================
- Coverage   85.06%   79.69%   -5.37%     
==========================================
  Files          19       31      +12     
  Lines        1346     1980     +634     
==========================================
+ Hits         1145     1578     +433     
- Misses        125      313     +188     
- Partials       76       89      +13     

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

@everettraven everettraven added this pull request to the merge queue Oct 15, 2024
Merged via the queue into operator-framework:main with commit 67f6398 Oct 15, 2024
5 of 6 checks passed
@joelanford joelanford deleted the add-history branch October 15, 2024 15:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants