Skip to content
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

MG-2060 - Remove int32 type in policy interface #2131

Merged

Conversation

nyagamunene
Copy link
Contributor

@nyagamunene nyagamunene commented Apr 2, 2024

What type of PR is this?

This is a refactor because it changes the integer type of limit and count parameters in policyAgent and Authz interface .

What does this do?

It changes the the limit and count parameter in policyAgent and Authz interface from int32 to uint64.

Which issue(s) does this PR fix/relate to?

Have you included tests for your changes?

Yes. I have include test for my changes.

Did you document any new/modified feature?

No.

Notes

@nyagamunene nyagamunene force-pushed the MG-2060-RemoveInt32TypeInPolicyInterface branch from 43c4d0e to d028325 Compare April 2, 2024 12:09
@nyagamunene nyagamunene marked this pull request as ready for review April 2, 2024 14:36
auth/api/grpc/server.go Outdated Show resolved Hide resolved
@dborovcanin dborovcanin force-pushed the MG-2060-RemoveInt32TypeInPolicyInterface branch from 3c823a9 to 76a6315 Compare April 3, 2024 07:33
@nyagamunene nyagamunene force-pushed the MG-2060-RemoveInt32TypeInPolicyInterface branch from 76a6315 to 2c1fee9 Compare April 3, 2024 08:10
Copy link
Collaborator

@dborovcanin dborovcanin left a comment

Choose a reason for hiding this comment

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

Please fix CI.

auth.proto Outdated Show resolved Hide resolved
@dborovcanin dborovcanin force-pushed the MG-2060-RemoveInt32TypeInPolicyInterface branch from 25338e3 to f9d1e36 Compare April 3, 2024 10:41
@nyagamunene nyagamunene force-pushed the MG-2060-RemoveInt32TypeInPolicyInterface branch from f9d1e36 to b76ba4a Compare April 3, 2024 10:58
@dborovcanin dborovcanin force-pushed the MG-2060-RemoveInt32TypeInPolicyInterface branch from 3d823aa to 4f1fae0 Compare April 3, 2024 12:34
@nyagamunene nyagamunene force-pushed the MG-2060-RemoveInt32TypeInPolicyInterface branch from 4f1fae0 to df48909 Compare April 3, 2024 12:43
Copy link
Member

@rodneyosodo rodneyosodo left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

@arvindh123 arvindh123 left a comment

Choose a reason for hiding this comment

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

I have manually ran basic test for this PR, LGTM

@arvindh123 arvindh123 requested a review from dborovcanin April 3, 2024 15:44
@dborovcanin dborovcanin changed the title MG-2060-Remove int32 type in policy interface MG-2060 - Remove int32 type in policy interface Apr 4, 2024
@dborovcanin dborovcanin merged commit 987c853 into absmach:main Apr 4, 2024
6 checks passed
JeffMboya pushed a commit to JeffMboya/supermq that referenced this pull request Apr 7, 2024
JeffMboya pushed a commit to JeffMboya/supermq that referenced this pull request Apr 10, 2024
JeffMboya pushed a commit to JeffMboya/supermq that referenced this pull request Apr 11, 2024
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.

Remove int32 type in Policy agent interface
4 participants