Skip to content

hello signing samples#78

Merged
NiazFK merged 22 commits intonotaryproject:mainfrom
SteveLasker:sample-hello-world
Sep 14, 2021
Merged

hello signing samples#78
NiazFK merged 22 commits intonotaryproject:mainfrom
SteveLasker:sample-hello-world

Conversation

@SteveLasker
Copy link
Contributor

These are the target commands, lifting the prototype-2 efforts to the nearly RC state of the artifacts-spec

Note: these commands don't work, yet. This is the target for what will work.
Please iterate any feedback here and we can incorporate into the work to support these commands.

Steve Lasker and others added 9 commits June 19, 2020 17:39
Signed-off-by: Steve Lasker <stevelasker@hotmail.com>
Signed-off-by: Steve Lasker <stevelasker@hotmail.com>
Signed-off-by: Steve Lasker <stevelasker@hotmail.com>
Signed-off-by: Steve Lasker <stevelasker@hotmail.com>
Signed-off-by: Steve Lasker <stevenlasker@hotmail.com>
Signed-off-by: Steve Lasker <stevenlasker@hotmail.com>
Signed-off-by: Steve Lasker <stevenlasker@hotmail.com>
Signed-off-by: Steve Lasker <stevenlasker@hotmail.com>
Signed-off-by: Steve Lasker <stevenlasker@hotmail.com>
Signed-off-by: Steve Lasker <stevenlasker@hotmail.com>
Signed-off-by: Steve Lasker <stevenlasker@hotmail.com>
@SteveLasker SteveLasker added the cli Issue or PR released to Notation CLI label Aug 30, 2021
Copy link

@nate-double-u nate-double-u left a comment

Choose a reason for hiding this comment

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

I've added some comments inline.

@shizhMSFT shizhMSFT mentioned this pull request Sep 3, 2021
Signed-off-by: Steve Lasker <stevenlasker@hotmail.com>
Signed-off-by: Steve Lasker <stevenlasker@hotmail.com>
@SteveLasker
Copy link
Contributor Author

PR feedback addressed. Please review latest updates.
To get the binary compatible with this doc, see: https://github.com/shizhMSFT/notation/releases/tag/v0.5.0

As #83 gets merged, and we start automating builds from main, I'll update the release location

Signed-off-by: Steve Lasker <stevenlasker@hotmail.com>
@SteveLasker SteveLasker added this to the alpha-1 milestone Sep 3, 2021
Signed-off-by: Steve Lasker <stevenlasker@hotmail.com>
@SteveLasker
Copy link
Contributor Author

👀 @notaryproject/notation-maintainers this is ready for review with https://github.com/shizhMSFT/notation/releases/tag/v0.5.2

Signed-off-by: Steve Lasker <stevenlasker@hotmail.com>
```
- Sign the container image
```bash
notation sign $IMAGE
Copy link
Contributor

Choose a reason for hiding this comment

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

I'd prefer being explicit about the key being used and not using a default, we can address that later.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

We did want to enable the simple/default scenarios but appreciate the concern.
@gokarnm, can you open an issue for this?

Copy link
Contributor

Choose a reason for hiding this comment

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

Opened #91

SteveLasker and others added 3 commits September 13, 2021 11:47
Co-authored-by: Milind Gokarn <milind81@gmail.com>
Signed-off-by: Steve Lasker <stevenlasker@hotmail.com>
…into sample-hello-world

Signed-off-by: Steve Lasker <stevenlasker@hotmail.com>
…into sample-hello-world

Signed-off-by: Steve Lasker <stevenlasker@hotmail.com>
Signed-off-by: Steve Lasker <stevenlasker@hotmail.com>
Copy link
Contributor

@NiazFK NiazFK left a comment

Choose a reason for hiding this comment

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

LGTM

@NiazFK NiazFK merged commit 91fde17 into notaryproject:main Sep 14, 2021
7h3-3mp7y-m4n pushed a commit to 7h3-3mp7y-m4n/notation that referenced this pull request Mar 29, 2025
* Notary v2 prototye process

Signed-off-by: Steve Lasker <stevelasker@hotmail.com>

* Notary v2 prototye process

Signed-off-by: Steve Lasker <stevelasker@hotmail.com>

* Notary v2 prototye process updates

Signed-off-by: Steve Lasker <stevelasker@hotmail.com>

* Rev of sketch, w/oci-index v3-index.config

Signed-off-by: Steve Lasker <stevelasker@hotmail.com>

* Hello world of notation

Signed-off-by: Steve Lasker <stevenlasker@hotmail.com>

* Add notation cli, updating from prototype-2

Signed-off-by: Steve Lasker <stevenlasker@hotmail.com>

* Add notation cli, updating from prototype-2

Signed-off-by: Steve Lasker <stevenlasker@hotmail.com>

* Add notation cli, updating from prototype-2

Signed-off-by: Steve Lasker <stevenlasker@hotmail.com>

* Incorporate feedback

Signed-off-by: Steve Lasker <stevenlasker@hotmail.com>

* Incorporate feedback

Signed-off-by: Steve Lasker <stevenlasker@hotmail.com>

* Update demo script to reflect latest CLI build

Signed-off-by: Steve Lasker <stevenlasker@hotmail.com>

* Incorporate PR feedback

Signed-off-by: Steve Lasker <stevenlasker@hotmail.com>

* PR Feedback

Signed-off-by: Steve Lasker <stevenlasker@hotmail.com>

* Updates with 0.5.2 alpha release

Signed-off-by: Steve Lasker <stevenlasker@hotmail.com>

* Add stubs for sbom and signature policy

Signed-off-by: Steve Lasker <stevenlasker@hotmail.com>

* Update hello-signing.md

Co-authored-by: Milind Gokarn <milind81@gmail.com>

* Add stubs for sbom and signature policy

Signed-off-by: Steve Lasker <stevenlasker@hotmail.com>

* Remvoing TBD files

Signed-off-by: Steve Lasker <stevenlasker@hotmail.com>

Co-authored-by: Steve Lasker <stevelasker@hotmail.com>
Co-authored-by: Milind Gokarn <milind81@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cli Issue or PR released to Notation CLI

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants

Comments