Skip to content

Fix: PullLayer and CompleteUpload for OCI conformance#27

Merged
jay-dee7 merged 1 commit intomasterfrom
OCI-Conformance
Sep 11, 2021
Merged

Fix: PullLayer and CompleteUpload for OCI conformance#27
jay-dee7 merged 1 commit intomasterfrom
OCI-Conformance

Conversation

@guacamole
Copy link
Member

Signed-off-by: guacamole [email protected]

  • Removed headers from PullLayer which were causing the fail during OCI tests
  • Updated the store.Update() function to overwrite the Manifest.config if the tag is duplicate
  • Total of 57 cases are passing, 2 failed, 3 skipped, following the snapshot of same

Screenshot 2021-09-11 at 11 38 26 AM

@guacamole guacamole requested a review from jay-dee7 September 11, 2021 06:11
@guacamole guacamole self-assigned this Sep 11, 2021
Copy link
Member

@jay-dee7 jay-dee7 left a comment

Choose a reason for hiding this comment

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

@guacamole LGTM 🥇 Good work and thanks for getting this done. This was a big one

@jay-dee7 jay-dee7 merged commit 8363f15 into master Sep 11, 2021
@jay-dee7 jay-dee7 deleted the OCI-Conformance branch September 11, 2021 06:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants