Skip to content

feat: support OCI image manifest#241

Merged
shizhMSFT merged 14 commits intonotaryproject:mainfrom
patrickzheng200:image
Jan 19, 2023
Merged

feat: support OCI image manifest#241
shizhMSFT merged 14 commits intonotaryproject:mainfrom
patrickzheng200:image

Conversation

@patrickzheng200
Copy link
Copy Markdown
Contributor

@patrickzheng200 patrickzheng200 commented Jan 12, 2023

This PR is the implementation of the spec.

Tested with [COSE, JWS] + [OCI artifact manifest, OCI image manifest] on ACR.

Patrick Zheng added 5 commits January 11, 2023 12:14
Signed-off-by: Patrick Zheng <patrickzheng@microsoft.com>
Signed-off-by: Patrick Zheng <patrickzheng@microsoft.com>
Signed-off-by: Patrick Zheng <patrickzheng@microsoft.com>
Signed-off-by: Patrick Zheng <patrickzheng@microsoft.com>
Signed-off-by: Patrick Zheng <patrickzheng@microsoft.com>
@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Jan 12, 2023

Codecov Report

Merging #241 (4e4d1f0) into main (e11bfd8) will decrease coverage by 0.05%.
The diff coverage is 16.21%.

@@            Coverage Diff             @@
##             main     #241      +/-   ##
==========================================
- Coverage   73.31%   73.25%   -0.06%     
==========================================
  Files          23       23              
  Lines        1746     1761      +15     
==========================================
+ Hits         1280     1290      +10     
- Misses        372      376       +4     
- Partials       94       95       +1     
Impacted Files Coverage Δ
registry/repository.go 39.18% <13.88%> (+6.98%) ⬆️
notation.go 59.86% <100.00%> (ø)

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

@shizhMSFT shizhMSFT changed the title Feat: added OCI image manifest support feat: support OCI image manifest Jan 12, 2023
Signed-off-by: Patrick Zheng <patrickzheng@microsoft.com>
@patrickzheng200 patrickzheng200 self-assigned this Jan 13, 2023
@patrickzheng200 patrickzheng200 added the enhancement New feature or request label Jan 13, 2023
@patrickzheng200 patrickzheng200 added this to the RC-2 milestone Jan 13, 2023
@patrickzheng200 patrickzheng200 linked an issue Jan 13, 2023 that may be closed by this pull request
Signed-off-by: Patrick Zheng <patrickzheng@microsoft.com>
Copy link
Copy Markdown
Contributor

@shizhMSFT shizhMSFT left a comment

Choose a reason for hiding this comment

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

LGTM

Patrick Zheng added 4 commits January 13, 2023 15:06
Signed-off-by: Patrick Zheng <patrickzheng@microsoft.com>
Signed-off-by: Patrick Zheng <patrickzheng@microsoft.com>
Signed-off-by: Patrick Zheng <patrickzheng@microsoft.com>
Patrick Zheng added 2 commits January 13, 2023 18:47
Signed-off-by: Patrick Zheng <patrickzheng@microsoft.com>
Signed-off-by: Patrick Zheng <patrickzheng@microsoft.com>
Signed-off-by: Patrick Zheng <patrickzheng@microsoft.com>
Copy link
Copy Markdown
Contributor

@shizhMSFT shizhMSFT 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
Copy Markdown
Contributor

@priteshbandi priteshbandi left a comment

Choose a reason for hiding this comment

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

LGTM

Although we will need to do some additional work for notaryproject/specifications#219 & notaryproject/specifications#220; this will act as a stepping stone.

@shizhMSFT shizhMSFT merged commit 36d97f7 into notaryproject:main Jan 19, 2023
@patrickzheng200 patrickzheng200 deleted the image branch January 19, 2023 02:31
priteshbandi pushed a commit to priteshbandi/notation-go that referenced this pull request Feb 1, 2023
Signed-off-by: Patrick Zheng <patrickzheng@microsoft.com>
Signed-off-by: Pritesh Bandi <pritesb@amazon.com>
priteshbandi pushed a commit to priteshbandi/notation-go that referenced this pull request Feb 1, 2023
Signed-off-by: Patrick Zheng <patrickzheng@microsoft.com>
Signed-off-by: Pritesh Bandi <pritesb@amazon.com>
priteshbandi pushed a commit to priteshbandi/notation-go that referenced this pull request Feb 1, 2023
Signed-off-by: Patrick Zheng <patrickzheng@microsoft.com>
Signed-off-by: Pritesh Bandi <pritesb@amazon.com>
priteshbandi pushed a commit to priteshbandi/notation-go that referenced this pull request Feb 3, 2023
Signed-off-by: Patrick Zheng <patrickzheng@microsoft.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

notation supports OCI Image manifest

4 participants