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

feat: add policy command #333

Merged
merged 3 commits into from
Oct 10, 2022
Merged

Conversation

binbin-li
Copy link
Contributor

@binbin-li binbin-li commented Sep 8, 2022

What?

Add policy commands. It supports adding/updating/reading/deleting policies.
Spec can be found in https://hackmd.io/AP9Xz9AbRpOy6ASpbok23A?view#Manage-Trust-Policy.

Test

Manually tested commands. Test cases can be checked in: https://hackmd.io/@H7a8_rG4SuaKwzu4NLT-9Q/S1j2Yspxj

Notes

This PR depends on notaryproject/notation-go#132, so the build will fail until it's merged.

Signed-off-by: Binbin Li libinbin@microsoft.com

@binbin-li binbin-li changed the title [WIP] feat: add policy command feat: add policy command Sep 13, 2022
cmd/notation/policy.go Show resolved Hide resolved
cmd/notation/policy.go Outdated Show resolved Hide resolved
cmd/notation/policy.go Outdated Show resolved Hide resolved
cmd/notation/policy.go Outdated Show resolved Hide resolved
cmd/notation/policy.go Outdated Show resolved Hide resolved
cmd/notation/policy.go Outdated Show resolved Hide resolved
cmd/notation/policy.go Outdated Show resolved Hide resolved
cmd/notation/policy.go Outdated Show resolved Hide resolved
cmd/notation/policy.go Outdated Show resolved Hide resolved
@shizhMSFT shizhMSFT added the dev Issue or PR related to dev branches label Sep 21, 2022
@binbin-li binbin-li force-pushed the support-policy branch 2 times, most recently from 3044065 to 348a110 Compare September 27, 2022 06:03
@codecov-commenter
Copy link

codecov-commenter commented Sep 27, 2022

Codecov Report

Merging #333 (e41bb47) into dev-rc.1 (115dccf) will increase coverage by 0.67%.
The diff coverage is 36.00%.

@@             Coverage Diff              @@
##           dev-rc.1     #333      +/-   ##
============================================
+ Coverage     31.15%   31.82%   +0.67%     
============================================
  Files            26       27       +1     
  Lines          1711     1986     +275     
============================================
+ Hits            533      632      +99     
- Misses         1161     1332     +171     
- Partials         17       22       +5     
Impacted Files Coverage Δ
cmd/notation/main.go 0.00% <0.00%> (ø)
cmd/notation/policy.go 36.13% <36.13%> (ø)

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@binbin-li binbin-li force-pushed the support-policy branch 4 times, most recently from 82aed08 to d2bcec7 Compare September 29, 2022 06:33
Copy link
Contributor

@patrickzheng200 patrickzheng200 left a comment

Choose a reason for hiding this comment

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

LGTM

Signed-off-by: Binbin Li <libinbin@microsoft.com>
Signed-off-by: Binbin Li <libinbin@microsoft.com>
@vaninrao10 vaninrao10 modified the milestones: RC-1, RC-2 Oct 6, 2022
@dtzar
Copy link
Contributor

dtzar commented Oct 6, 2022

Now that notaryproject/notation-go#132 is merged, does this make sense to try to PR to main and kick off a review from everyone else?

Signed-off-by: Binbin Li <libinbin@microsoft.com>
@binbin-li binbin-li merged commit d75d027 into notaryproject:dev-rc.1 Oct 10, 2022
@binbin-li
Copy link
Contributor Author

Now that notaryproject/notation-go#132 is merged, does this make sense to try to PR to main and kick off a review from everyone else?

Once we fully test the dev-rc.1 branch, we can make the PR to main.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dev Issue or PR related to dev branches
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants