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

Fixes #30747 - Make fuzzy search work in hammer shell #335

Merged
merged 1 commit into from
Sep 4, 2020

Conversation

ofedoren
Copy link
Member

@ofedoren ofedoren commented Aug 31, 2020

Needs to be CPed into 2.1-stable

Copy link
Member

@chris1984 chris1984 left a comment

Choose a reason for hiding this comment

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

ACK.

Works good, able to do both content commands without an error now:

[vagrant@hammer ~]$ hammer shell
Welcome to the hammer interactive shell.
Type 'help' for usage information.
hammer> capsule content
Usage:
    capsule [OPTIONS] SUBCOMMAND [ARG] ...

Parameters:
 SUBCOMMAND                    Subcommand
 [ARG] ...                     Subcommand arguments

Subcommands:
 add-lifecycle-environment        Add lifecycle environments to the capsule
 available-lifecycle-environments List the lifecycle environments not attached to the capsule
 cancel-synchronization           Cancel running capsule synchronization
 info                             Get current capsule synchronization status
 lifecycle-environments           List the lifecycle environments attached to the capsule
 remove-lifecycle-environment     Remove lifecycle environments from the capsule
 synchronization-status           Get current capsule synchronization status
 synchronize                      Synchronize the content to the capsule

Options:
 -h, --help                    Print help


hammer> content-credentials 
Usage:
    content-credentials [OPTIONS] SUBCOMMAND [ARG] ...

Parameters:
 SUBCOMMAND                    Subcommand
 [ARG] ...                     Subcommand arguments

Subcommands:
 create                        Create a content credential
 delete                        Destroy a content credential
 info                          Show a content credential
 list                          List content credentials
 update                        Update a content credential

Options:
 -h, --help                    Print help


hammer> 

@ofedoren ofedoren merged commit 3e5df06 into theforeman:master Sep 4, 2020
ofedoren added a commit that referenced this pull request Sep 4, 2020
ofedoren added a commit that referenced this pull request Sep 4, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants