Project

General

Profile

Actions

Bug #15963

closed

katello-installer --help typos

Added by Stephen Benjamin almost 8 years ago. Updated almost 6 years ago.

Status:
Resolved
Priority:
Low
Assignee:
-
Category:
Installer
Target version:

Description

Cloned from https://bugzilla.redhat.com/show_bug.cgi?id=1175412

Description of problem:
There are typos and general clean-up that should happen with the output for katello-installer, I've added notes when necessary at the end of that line's description in [ ] brackets.
  1. katello-installer --help
    --reset This option will drop the Katello database and clear all subsequent backend data stores.You will lose all data! Unfortunately we
    can't detect a failure at the moment so you should verify the success
    manually. e.g. dropping can fail when DB is currently in use. (default: false) [this is wrapped messily, needs a space after "data stores.", needs a comma after "at the moment"]
    --capsule-pulp-admin-password passowrd for the Pulp admin user.It should be left blank so that random password is generated (default: "kpU2mLK9oBCpNpmMSGEz4jzJuopcEBhq") [password is spelled wrong, no space before next sentence]
    --certs-ca-expiration Ca expiration attribute for managed certificates (default: "36500") [some CA references are capitalized, others aren't, acronyms should be capitalized in most cases]
    --certs-deploy Deploy the certs on the configured system. False means
    we want apply it on a different system (default: true) [missing to, should add it after "want"]
    --certs-log-dir When the log files should go (default: "/var/log/certs") [When should be Where]
    --certs-regenerate Force regeneration of the certificates (excluding
    ca certificates) (default: false) [ca should be capitalized]
    --certs-regenerate-ca Force regeneration of the ca certificate (default: false) [ca not capitalized]
    --certs-server-cert Path to the ssl certificate for https
    if not specified, the default CA will generate one (default: nil) [capitalize ssl, these are separate thoughts, should use a period between them]
    --certs-server-cert-req Path to the ssl certificate request for https (default: nil) [capitalize ssl, https]
    --certs-server-key Path to the ssl key for https [acronyms should be capitalized, ssl and https]
    --foreman-authentication Enable users authentication (default user:admin pw:changeme) (default: true) [should users be singular?]
    --foreman-configure-scl-repo If disabled the the SCL repo will not be configured on Red Hat clone systems.
    (Currently only installs repos for CentOS and Scientific) (default: false) ["the" is repeated, should be a comma after disabled]
    --foreman-custom-repo No need to change anything here by default
    if set to true, no repo will be added by this module, letting you to
    set it to some custom location. (default: true) [can remove "to" after "letting you", probably should end "No need to change anything here by default" with a period]
    --foreman-db-sslmode Database 'production' ssl mode (default: nil) [capitalize ssl]
    --foreman-passenger-start-timeout Amount of seconds to wait for Ruby application boot. (default: "600") ["Amount" should be "Number"]
    --foreman-selinux when undef, foreman-selinux will be installed if SELinux is enabled
    setting to false/true will override this check (e.g. set to false on 1.1) (default: nil) [seems like these are separate thoughts, end "is enabled" with a period, other descriptions here start with a capital letter in the first word, we don't here]
Version-Release number of selected component (if applicable):
katello-utils-1.5.16-1.el6sat.noarch
Additional info:
Actions #1

Updated by Justin Sherrill over 7 years ago

  • Subject changed from katello-installer --help typos to katello-installer --help typos
  • Assignee set to Chris Roberts
  • translation missing: en.field_release set to 114
  • Difficulty set to easy
Actions #2

Updated by The Foreman Bot almost 6 years ago

  • Status changed from New to Ready For Testing
  • translation missing: en.field_release deleted (114)
  • Pull request https://github.com/Katello/katello-installer/pull/636 added
Actions #3

Updated by The Foreman Bot almost 6 years ago

  • Pull request https://github.com/theforeman/puppet-certs/pull/203 added
Actions #4

Updated by Ewoud Kohl van Wijngaarden almost 6 years ago

  • Status changed from Ready For Testing to Resolved

Both PRs were merged so I'm closing this.

Actions #5

Updated by It's Bill almost 6 years ago

  • Pull request https://github.com/theforeman/puppet-foreman/pull/641 added
Actions #6

Updated by Andrew Kofink almost 6 years ago

  • translation missing: en.field_release set to 338
Actions #7

Updated by Chris Roberts almost 6 years ago

  • Assignee deleted (Chris Roberts)
Actions

Also available in: Atom PDF