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.

Sign up for GitHub

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

Jump to bottom

RMS: Compliance api #742

Merged
merged 2 commits into from
Oct 18, 2024
Merged

RMS: Compliance api #742

merged 2 commits into from
Oct 18, 2024

Conversation

artem-lifshits
Copy link
Member

What this PR does / why we need it

Which issue this PR fixes

Acceptance tests

=== RUN   TestAllPoliciesList
--- PASS: TestAllPoliciesList (0.83s)
=== RUN   TestComplianceList
--- PASS: TestComplianceList (4.51s)
=== RUN   TestComplianceLifecycle
--- PASS: TestComplianceLifecycle (7.64s)
PASS

Process finished with the exit code 0

@artem-lifshits artem-lifshits self-assigned this Oct 18, 2024
Aloento
Aloento previously approved these changes Oct 18, 2024
View reviewed changes
@artem-lifshits artem-lifshits mentioned this pull request Oct 18, 2024
Closed
7 tasks
@artem-lifshits artem-lifshits added the gate Merge PR label Oct 18, 2024
@otc-zuul otc-zuul bot merged commit 3da4d76 into devel Oct 18, 2024
8 checks passed
@otc-zuul otc-zuul bot deleted the rms_compliance branch October 18, 2024 14:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@anton-sidelnikov anton-sidelnikov anton-sidelnikov approved these changes

@Aloento Aloento Aloento left review comments

@muneeb-jan muneeb-jan Awaiting requested review from muneeb-jan

Assignees

@artem-lifshits artem-lifshits

Labels
gate Merge PR
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@artem-lifshits @Aloento @anton-sidelnikov