[SECURESIGN-1228] add/update/remove old-style targets

[fghanmi]

No description provided.

[SequeI]

/approve

[bkabrda]

I think overall this looks good; I posted couple comments to improve certain things. We will also need these commands to modify the trusted_root.json file, but that can be improved in a different PR once the sigstore-rs PR is merged.

[bkabrda]

@fghanmi one last comment here - when I try to remove a non-existing target, the command silently passes not doing anything. Should removing a non-existing target rather error out? When I make a typo in the target name, this would silently pass and I might not notice because I would think that the command worked.

[fghanmi]

@fghanmi one last comment here - when I try to remove a non-existing target, the command silently passes not doing anything. Should removing a non-existing target rather error out? When I make a typo in the target name, this would silently pass and I might not notice because I would think that the command worked.

I was thinking it's okay to silently pass even if the target does not exist (as if it's deleted)
But I like more your idea. Thanks!

[bkabrda]

/approve

[openshift-ci]

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: bkabrda, fghanmi, SequeI

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

• OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment