Skip to content

fix: updated capability response of the controller service#732

Merged
lukasmetzner merged 1 commit intomainfrom
controllerservice-capability-response
Oct 4, 2024
Merged

fix: updated capability response of the controller service#732
lukasmetzner merged 1 commit intomainfrom
controllerservice-capability-response

Conversation

@lukasmetzner
Copy link
Copy Markdown
Contributor

@lukasmetzner lukasmetzner commented Oct 2, 2024

See #327

SINGLE_NODE_MULTI_WRITER needs to be in the response of the ControllerGetCapabilities function to correctly integrate support for SINGLE_NODE_MULTI_WRITER access modes.

This was missing from #725.

BEGIN_COMMIT_OVERRIDE
chore: updated capability response of the controller service
END_COMMIT_OVERRIDE

@lukasmetzner lukasmetzner requested a review from a team as a code owner October 2, 2024 14:36
@codecov
Copy link
Copy Markdown

codecov bot commented Oct 2, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 34.52%. Comparing base (fe76006) to head (e224240).
Report is 5 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #732      +/-   ##
==========================================
+ Coverage   34.28%   34.52%   +0.24%     
==========================================
  Files          19       19              
  Lines        1616     1622       +6     
==========================================
+ Hits          554      560       +6     
  Misses       1028     1028              
  Partials       34       34              

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

@lukasmetzner lukasmetzner changed the title feat: Updated capability response of the controller service chore: Updated capability response of the controller service Oct 4, 2024
SINGLE_NODE_MULTI_WRITER needs to be in the response of the ControllerGetCapabilities function to correctly integrate support for SINGLE_NODE_MULTI_WRITER access modes.
@lukasmetzner lukasmetzner force-pushed the controllerservice-capability-response branch from 7f97b0b to e224240 Compare October 4, 2024 06:28
@lukasmetzner lukasmetzner changed the title chore: Updated capability response of the controller service fix: Updated capability response of the controller service Oct 4, 2024
@apricote apricote changed the title fix: Updated capability response of the controller service fix: updated capability response of the controller service Oct 4, 2024
@lukasmetzner lukasmetzner merged commit e806b80 into main Oct 4, 2024
@lukasmetzner lukasmetzner deleted the controllerservice-capability-response branch October 4, 2024 08:37
lukasmetzner added a commit that referenced this pull request Oct 10, 2024
See #327 

SINGLE_NODE_MULTI_WRITER needs to be in the response of the
ControllerGetCapabilities function to correctly integrate support for
SINGLE_NODE_MULTI_WRITER access modes.

This was missing from #725.
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