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 Notation task #19228

Merged
merged 42 commits into from
Nov 22, 2023

Conversation

JeyJeyGao
Copy link
Collaborator

Task name: NotationV0

Description: Notation task is for container image signing and verifying

Documentation changes required: (Y/N) Y

Added unit tests: (Y/N) Y

Attached related issue: (Y/N) N

Checklist:

  • Task version was bumped - please check instruction how to do it
  • Checked that applied changes work as expected

@JeyJeyGao JeyJeyGao force-pushed the users/junjie/feat_notation_task branch from 89a4882 to c8a4c7d Compare November 7, 2023 13:48
@JeyJeyGao JeyJeyGao marked this pull request as ready for review November 7, 2023 13:49
Copy link
Contributor

@DmitriiBobreshev DmitriiBobreshev left a comment

Choose a reason for hiding this comment

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

Also please add the task in the make-options file to build the task during ci/cd and to the codeowners file

Tasks/NotationV0/package.json Outdated Show resolved Hide resolved
Tasks/NotationV0/package.json Outdated Show resolved Hide resolved
Tasks/NotationV0/task.json Show resolved Hide resolved
@mmrazik
Copy link
Collaborator

mmrazik commented Nov 7, 2023

FYI -- this needs way more conversation about ownership, (on call) support, CSS involvement, etc. We already have a thread on it.

Tasks/NotationV0/task.json Outdated Show resolved Hide resolved
Tasks/NotationV0/task.json Outdated Show resolved Hide resolved
.github/CODEOWNERS Outdated Show resolved Hide resolved
make-options.json Outdated Show resolved Hide resolved
Signed-off-by: Junjie Gao <junjiegao@microsoft.com>
Signed-off-by: Junjie Gao <junjiegao@microsoft.com>
Signed-off-by: Junjie Gao <junjiegao@microsoft.com>
Signed-off-by: Junjie Gao <junjiegao@microsoft.com>
Signed-off-by: Junjie Gao <junjiegao@microsoft.com>
Signed-off-by: Junjie Gao <junjiegao@microsoft.com>
Signed-off-by: Junjie Gao <junjiegao@microsoft.com>
Signed-off-by: Junjie Gao <junjiegao@microsoft.com>
Signed-off-by: Junjie Gao <junjiegao@microsoft.com>
Signed-off-by: Junjie Gao <junjiegao@microsoft.com>
Signed-off-by: Junjie Gao <junjiegao@microsoft.com>
Signed-off-by: Junjie Gao <junjiegao@microsoft.com>
Signed-off-by: Junjie Gao <junjiegao@microsoft.com>
Signed-off-by: Junjie Gao <junjiegao@microsoft.com>
Signed-off-by: Junjie Gao <junjiegao@microsoft.com>
Signed-off-by: Junjie Gao <junjiegao@microsoft.com>
Signed-off-by: Junjie Gao <junjiegao@microsoft.com>
Signed-off-by: Junjie Gao <junjiegao@microsoft.com>
@JeyJeyGao JeyJeyGao force-pushed the users/junjie/feat_notation_task branch from ad70a74 to 4cd6540 Compare November 8, 2023 13:13
JeyJeyGao and others added 15 commits November 9, 2023 07:35
Signed-off-by: Junjie Gao <junjiegao@microsoft.com>
Signed-off-by: Junjie Gao <junjiegao@microsoft.com>
Signed-off-by: Junjie Gao <junjiegao@microsoft.com>
Signed-off-by: Junjie Gao <junjiegao@microsoft.com>
Signed-off-by: Junjie Gao <junjiegao@microsoft.com>
Signed-off-by: Junjie Gao <junjiegao@microsoft.com>
Signed-off-by: Junjie Gao <junjiegao@microsoft.com>
Signed-off-by: Junjie Gao <junjiegao@microsoft.com>
Signed-off-by: Junjie Gao <junjiegao@microsoft.com>
Signed-off-by: Junjie Gao <junjiegao@microsoft.com>
@DergachevE DergachevE merged commit 0839553 into microsoft:master Nov 22, 2023
8 checks passed
@JeyJeyGao JeyJeyGao deleted the users/junjie/feat_notation_task branch August 13, 2024 07:36
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.

7 participants