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

Hardware Tokens doc missing subcommand piv-tool #1849

Closed
nealmcb opened this issue May 6, 2022 · 1 comment · Fixed by #1850
Closed

Hardware Tokens doc missing subcommand piv-tool #1849

nealmcb opened this issue May 6, 2022 · 1 comment · Fixed by #1850
Labels
bug Something isn't working

Comments

@nealmcb
Copy link
Contributor

nealmcb commented May 6, 2022

Description

cosign/TOKENS.md at main · sigstore/cosign says:

... run the cosign generate-key command ...

Which yields this error:

$ cosign generate-key
Error: unknown command "generate-key" for "cosign"

The documentation is simply missing the piv-tool sub-command.

@nealmcb nealmcb added the bug Something isn't working label May 6, 2022
nealmcb added a commit to nealmcb/cosign that referenced this issue May 6, 2022
[cosign/TOKENS\.md at main · sigstore/cosign](https://github.com/sigstore/cosign/blob/main/TOKENS.md) says:

... *run the* `cosign generate-key` *command* ...

Which yields this error:

```
$ cosign generate-key
Error: unknown command "generate-key" for "cosign"
```

The documentation is simply missing the `piv-tool` sub-command.

Resolves: sigstore#1849

Signed-off-by: Neal McBurnett <neal@mcburnett.org>
nealmcb added a commit to nealmcb/cosign that referenced this issue May 7, 2022
[cosign/TOKENS\.md at main · sigstore/cosign](https://github.com/sigstore/cosign/blob/main/TOKENS.md) says:

... *run the* `cosign generate-key` *command* ...

Which yields this error:

```
$ cosign generate-key
Error: unknown command "generate-key" for "cosign"
```

The documentation is simply missing the `piv-tool` sub-command.

Resolves: sigstore#1849

Also fix make target for custom build: was make cosign-pivkey
now make cosign-pivkey-pkcs11key.
Drop old, brittle reference to line in Makefile.

Signed-off-by: Neal McBurnett <neal@mcburnett.org>
@nealmcb
Copy link
Contributor Author

nealmcb commented May 7, 2022

Proceeding step-by-step, we see that there is also an obsolete Makefilie target being used in the doc. A fix for that is now also included in the PR.

nealmcb added a commit to nealmcb/cosign that referenced this issue May 7, 2022
[cosign/TOKENS\.md at main · sigstore/cosign](https://github.com/sigstore/cosign/blob/main/TOKENS.md) says:

... *run the* `cosign generate-key` *command* ...

Which yields this error:

```
$ cosign generate-key
Error: unknown command "generate-key" for "cosign"
```

The documentation is simply missing the `piv-tool` sub-command.

Resolves: sigstore#1849

Also fix make target for custom build: was make cosign-pivkey
now make cosign-pivkey-pkcs11key.
Drop old, brittle reference to line in Makefile.

Also note need to `cd cmd/cosign`.

See also: sigstore#1853

Signed-off-by: Neal McBurnett <neal@mcburnett.org>
nealmcb added a commit to nealmcb/cosign that referenced this issue May 7, 2022
[cosign/TOKENS\.md at main · sigstore/cosign](https://github.com/sigstore/cosign/blob/main/TOKENS.md) says:

... *run the* `cosign generate-key` *command* ...

Which yields this error:

```
$ cosign generate-key
Error: unknown command "generate-key" for "cosign"
```

The documentation is simply missing the `piv-tool` sub-command.

Resolves: sigstore#1849

Also fix make target for custom build: was make cosign-pivkey
now make cosign-pivkey-pkcs11key.
Drop old, brittle reference to line in Makefile.

Also make `go build` also work the same way, from the root directory.

See also: sigstore#1853

Signed-off-by: Neal McBurnett <neal@mcburnett.org>
dlorenc pushed a commit that referenced this issue May 7, 2022
[cosign/TOKENS\.md at main · sigstore/cosign](https://github.com/sigstore/cosign/blob/main/TOKENS.md) says:

... *run the* `cosign generate-key` *command* ...

Which yields this error:

```
$ cosign generate-key
Error: unknown command "generate-key" for "cosign"
```

The documentation is simply missing the `piv-tool` sub-command.

Resolves: #1849

Also fix make target for custom build: was make cosign-pivkey
now make cosign-pivkey-pkcs11key.
Drop old, brittle reference to line in Makefile.

Also make `go build` also work the same way, from the root directory.

See also: #1853

Signed-off-by: Neal McBurnett <neal@mcburnett.org>
pxp928 pushed a commit to pxp928/cosign that referenced this issue May 12, 2022
[cosign/TOKENS\.md at main · sigstore/cosign](https://github.com/sigstore/cosign/blob/main/TOKENS.md) says:

... *run the* `cosign generate-key` *command* ...

Which yields this error:

```
$ cosign generate-key
Error: unknown command "generate-key" for "cosign"
```

The documentation is simply missing the `piv-tool` sub-command.

Resolves: sigstore#1849

Also fix make target for custom build: was make cosign-pivkey
now make cosign-pivkey-pkcs11key.
Drop old, brittle reference to line in Makefile.

Also make `go build` also work the same way, from the root directory.

See also: sigstore#1853

Signed-off-by: Neal McBurnett <neal@mcburnett.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant