Skip to content

Remove the CanStreamData RPC#87

Merged
tiyash-basu-frequenz merged 1 commit intofrequenz-floss:v0.x.xfrom
tiyash-basu-frequenz:remove_can_stream_data
Aug 9, 2023
Merged

Remove the CanStreamData RPC#87
tiyash-basu-frequenz merged 1 commit intofrequenz-floss:v0.x.xfrom
tiyash-basu-frequenz:remove_can_stream_data

Conversation

@tiyash-basu-frequenz
Copy link
Copy Markdown
Contributor

It can be checked by calling the StreamComponentData method, and checking if the response is an error or not.

@tiyash-basu-frequenz tiyash-basu-frequenz added this to the v0.16.0 milestone Aug 8, 2023
@tiyash-basu-frequenz tiyash-basu-frequenz self-assigned this Aug 8, 2023
@tiyash-basu-frequenz tiyash-basu-frequenz requested a review from a team as a code owner August 8, 2023 12:10
@github-actions github-actions Bot added part:docs Affects the documentation part:protobuf Affects the protocol buffer definition files labels Aug 8, 2023
@tiyash-basu-frequenz
Copy link
Copy Markdown
Contributor Author

Marking it as blocked for now until we finalise the release plan for v0.15.2.

@tiyash-basu-frequenz tiyash-basu-frequenz added the status:blocked Other issues must be resolved before this can be worked on label Aug 8, 2023
@tiyash-basu-frequenz tiyash-basu-frequenz linked an issue Aug 8, 2023 that may be closed by this pull request
@llucax llucax removed the status:blocked Other issues must be resolved before this can be worked on label Aug 9, 2023
@llucax
Copy link
Copy Markdown
Contributor

llucax commented Aug 9, 2023

Unblocking as v0.15.2 was released 🎉

Copy link
Copy Markdown
Contributor

@llucax llucax left a comment

Choose a reason for hiding this comment

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

LGTM

llucax
llucax previously approved these changes Aug 9, 2023
It can be checked by calling the `StreamComponentData` method, and checking
if the response is an error or not.

Signed-off-by: Tiyash Basu <tiyash.basu@frequenz.com>
@tiyash-basu-frequenz
Copy link
Copy Markdown
Contributor Author

tiyash-basu-frequenz commented Aug 9, 2023

Rebased on v0.x.x to resolve conflicts.
Ready for another look.

@tiyash-basu-frequenz tiyash-basu-frequenz added this pull request to the merge queue Aug 9, 2023
Merged via the queue into frequenz-floss:v0.x.x with commit b509335 Aug 9, 2023
@tiyash-basu-frequenz tiyash-basu-frequenz deleted the remove_can_stream_data branch August 9, 2023 10:13
Copy link
Copy Markdown

@thomas-nicolai-frequenz thomas-nicolai-frequenz left a comment

Choose a reason for hiding this comment

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

Looks good from my side.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

part:docs Affects the documentation part:protobuf Affects the protocol buffer definition files

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Remove gRPC method CanStreamData

4 participants