Project

General

Profile

Feature #28294

add possibiliy to add ssl-stuff with the name

Added by Patrick Thyen 4 months ago. Updated about 2 months ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
Hammer
Target version:
Difficulty:
easy
Triaged:
Yes
Bugzilla link:

Description

With the command: "hammer repository create|update" you can add ssl-stuff only with the ID.
With the gpg-keys it is possible to add them via the name.
Please add the possibility to add them via the name, too.
(And there is a little typo in the description for --ssl-ca-cert-id)

Suggestion:
Before:
...
--ssl-ca-cert-id SSL_CA_CERT_ID Idenifier of the SSL CA Cert
--ssl-client-cert-id SSL_CLIENT_CERT_ID Identifier of the SSL Client Cert
--ssl-client-key-id SSL_CLIENT_KEY_ID Identifier of the SSL Client Key
...

After:
...
--ssl-ca-cert-id SSL_CA_CERT_ID Numeric identifier of the SSL CA Cert
--ssl-ca-cert SSL_CA_CERT_NAME SSL CA Cert name to search by
--ssl-client-cert-id SSL_CLIENT_CERT_ID Numeric identifier of the SSL Client Cert
--ssl-client-cert SSL_CLIENT_CERT_NAME SSL Cert name to search by
--ssl-client-key-id SSL_CLIENT_KEY_ID Numeric identifier of the SSL Client Key
--ssl-client-key SSL_CLIENT_KEY_NAME SSL KEY name to search by
...

Associated revisions

Revision b6f129a1 (diff)
Added by John Mitsch about 2 months ago

Fixes #28294 - Create repo with ssl cert name

This adds the ability to create a repository with a content credential names, not just ids

For example:

hammer repository create --product-id 3 --name my_repo --url https://example.com/myawesomerepo --content-type yum --ssl-client-key-name "my product key" --ssl-client-cert-name "my product cert" --ssl-ca-cert-name "my ca"

Revision 801649e1 (diff)
Added by John Mitsch about 2 months ago

Refs #28294 - Update API docs for repo create

It isn't very clear that the IDs for ssl-*-id params are content credentials, this commit adds some clarification and fixes a typo as well

History

#1 Updated by Martin Bacovsky 4 months ago

  • Category set to Hammer
  • Project changed from Hammer CLI to Katello

#2 Updated by Chris Roberts 4 months ago

  • Triaged changed from No to Yes
  • Difficulty set to easy
  • Target version set to Katello 3.15.0

#3 Updated by The Foreman Bot 2 months ago

  • Assignee set to John Mitsch
  • Status changed from New to Ready For Testing
  • Pull request https://github.com/Katello/katello/pull/8535 added

#4 Updated by The Foreman Bot 2 months ago

  • Pull request https://github.com/Katello/hammer-cli-katello/pull/720 added

#5 Updated by Chris Roberts 2 months ago

  • Pull request deleted (https://github.com/Katello/katello/pull/8535)

#6 Updated by The Foreman Bot about 2 months ago

  • Pull request https://github.com/Katello/katello/pull/8546 added

#7 Updated by John Mitsch about 2 months ago

  • Status changed from Ready For Testing to Closed

Also available in: Atom PDF