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

Refs #36401 - Update info command to show multiple cv envs #894

Merged
merged 1 commit into from Jun 20, 2023

Conversation

chris1984
Copy link
Member

@chris1984 chris1984 commented Jun 13, 2023

Needs Katello PR: Katello/katello#10604

Output with PR:

Additional info:
    Owner:      Admin User
    Owner Id:   4
    Owner Type: User
    Enabled:    yes
    Model:      VMware Virtual Platform
    Comment:
Content Information:
    Content view environments:
     1) Content View:
            Id:        2
            Name:      Zoo
            Composite: no
        Lifecycle environment:
            Id:   1
            Name: Library
     2) Content View:
            Id:        3
            Name:      Plumbing
            Composite: no
        Lifecycle environment:
            Id:   2
            Name: Custom
    Content Source:

@theforeman-bot
Copy link

Issues: #36401

Copy link
Member

@jeremylenz jeremylenz left a comment

Choose a reason for hiding this comment

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

Small capitalization nits below, but this works great!

Content Information:      
    Content View Environments: 
     1) Content View:          
            Id:        2
            Name:      cv1
            Composite: no
        Lifecycle environment: 
            Id:   1
            Name: Library
     2) Content View:          
            Id:        4
            Name:      cv2
            Composite: no
        Lifecycle environment: 
            Id:   1
            Name: Library

Also tested with a single env which works fine.

lib/hammer_cli_katello/host_extensions.rb Outdated Show resolved Hide resolved
lib/hammer_cli_katello/host_extensions.rb Outdated Show resolved Hide resolved
@chris1984 chris1984 merged commit 94b39e0 into Katello:master Jun 20, 2023
3 checks passed
@chris1984 chris1984 deleted the fixcv branch June 20, 2023 18:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants