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 #37948 - Update host output for MultiCV to match AK output #964

Merged
merged 2 commits into from
Nov 1, 2024

Conversation

jeremylenz
Copy link
Member

Requires Katello/katello#11194

  1. Add some tidbits to hammer host info, namely Content View Environment Labels and Label
Content Information:      
    Content View Environment Labels: lce2/cv2,lce1/cv1
    Content View Environments:       
     1) Content view:          
            Id:        3
            Name:      cv2
            Composite: no
        Lifecycle environment: 
            Id:   3
            Name: lce2
        Label:                 lce2/cv2
  1. Add Label to hammer activation-key info for consistency.

  2. Add columns to hammer host list (removing the CONTENT VIEW and LIFECYCLE ENVIRONMENT columns):

CONTENT VIEW ENVIRONMENTS | MULTI CONTENT VIEW ENVIRONMENT

@lfu
Copy link
Member

lfu commented Nov 1, 2024

$ hammer host info --id 3 
Content Information:      
    Content View Environment Labels: Library/cv2,test/cv1
    Content View Environments:       
     1) Content view:          
            Id:        3
            Name:      cv2
            Composite: no
        Lifecycle environment: 
            Id:   1
            Name: Library
        Label:                 Library/cv2
     2) Content view:          
            Id:        2
            Name:      cv1
            Composite: no
        Lifecycle environment: 
            Id:   2
            Name: test
        Label:                 test/cv1
$ hammer host list --organization-id 1
---|--------------------------------------------------|------------------|------------|--------------|-------------------|---------------|---------------------------|-------------------------------
ID | NAME                                             | OPERATING SYSTEM | HOST GROUP | IP           | MAC               | GLOBAL STATUS | CONTENT VIEW ENVIRONMENTS | MULTI CONTENT VIEW ENVIRONMENT
---|--------------------------------------------------|------------------|------------|--------------|-------------------|---------------|---------------------------|-------------------------------
2  | betty-meiler.rhos-01.prod.psi.rdu2.redhat.com    | RHEL 9.4         |            |              | 52:54:33:44:55:0c | Warning       |                           | no                            
1  | ip-10-0-167-112.rhos-01.prod.psi.rdu2.redhat.com | RHEL 9.4         |            | 10.0.167.112 | fa:16:3e:a8:81:31 | Warning       |                           | no                            
3  | ip-10-0-168-108.rhos-01.prod.psi.rdu2.redhat.com | RedHat 8.8       |            | 10.0.168.108 | fa:16:3e:4f:46:5e | Warning       | Library/cv1               | no                            
---|--------------------------------------------------|------------------|------------|--------------|-------------------|---------------|---------------------------|-------------------------------
$ hammer activation-key info --id 4 --organization-id 1
Name:                            ak4
Id:                              4
Host Limit:                      0 of Unlimited
Multi Content View Environment:  no
Content View Environment Labels: Library/cv2
Organization:                    
 1) Id:   1
    Name: Default Organization
Content View Environments:       
 1) Content View:          
        Id:                          3
        Name:                        cv2
        Version:                     1.0
        Content View version Id:     3
        Composite:                   no
        Content View Environment id: 4
    Lifecycle environment: 
        Id:   1
        Name: Library
    Label:                 Library/cv2

@lfu
Copy link
Member

lfu commented Nov 1, 2024

LGTM 👍🏻

@chris1984 chris1984 merged commit d2c7a35 into Katello:main Nov 1, 2024
11 of 12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants