Skip to content

Add "Best Effort" Service Level#27

Merged
eric-murray merged 5 commits intocamaraproject:mainfrom
eric-murray:eric-murray-patch-1
Nov 3, 2025
Merged

Add "Best Effort" Service Level#27
eric-murray merged 5 commits intocamaraproject:mainfrom
eric-murray:eric-murray-patch-1

Conversation

@eric-murray
Copy link
Contributor

@eric-murray eric-murray commented Jul 8, 2025

What type of PR is this?

  • enhancement/feature

What this PR does / why we need it:

Adds a "Best Effort" service level with no specific service level guarantee

Which issue(s) this PR fixes:

Part fix for issue 26

Special notes for reviewers:

None

Changelog input

 release-note
 - Add "Best Effort" service level

Additional documentation

None

@hdamker
Copy link
Contributor

hdamker commented Sep 2, 2025

@eric-murray @albertoramosmonagas is it planned to add this PR to the Fall25 release? If no, please set to draft so that RM reviewers have a clear situation.

@albertoramosmonagas albertoramosmonagas marked this pull request as draft September 2, 2025 13:30
Updated the description of the BEST_EFFORT service level to clarify its qualitative nature and lack of guarantees.
@eric-murray eric-murray marked this pull request as ready for review October 28, 2025 09:53
Copy link
Contributor

@albertoramosmonagas albertoramosmonagas left a comment

Choose a reason for hiding this comment

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

LGTM! On the other hand, I modified the part about errors and moving the version to wip in my PR. If you want to merge it, don't close the issue associated with this PR so that I can use it in mine too.

@eric-murray
Copy link
Contributor Author

OK, thanks

Before merging, we need to reset version numbers to wip. I'll raise a separate PR for that,

@eric-murray eric-murray merged commit 4cd9cf5 into camaraproject:main Nov 3, 2025
2 checks passed
@eric-murray
Copy link
Contributor Author

Leaving issue #26 open for additional error message required when support for some service levels is optional

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.

3 participants