Project

General

Profile

Bug #27508

Hammer package list with environment flag uses wrong api

Added by Chris Roberts 10 months ago. Updated 9 months ago.

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

Description

When doing

  1. hammer package list --help we see this option:

--environment ENVIRONMENT_NAME Lifecycle environment name to search by

here is mine:

[vagrant@centos7-hammer-devel hammer-cli-katello]$ hammer lifecycle-environment list
---|---------|--------
ID | NAME | PRIOR
---|---------|--------
2 | dev | Library
1 | Library |
---|---------|--------

but the command calls out to foreman instead of katello

[ INFO 2019-08-02T15:47:41 API] GET /api/environments

so when i do this

[vagrant@centos7-hammer-devel hammer-cli-katello]$ hammer package list --environment Library
Error: environment not found.

Associated revisions

Revision 0b0b878e (diff)
Added by Partha Aji 9 months ago

Fixes #27508 - use LCE for package list (#689)

  • Fixes #27508 - use LCE for package list

'hammer package list --environment=Library --organization-id=1'

This commit ensures that the --environment flag correctly points to
the lifecyle environment and not puppet env.

  • Refs #27508 - Adds unit test

History

#1 Updated by Chris Roberts 10 months ago

  • Triaged changed from No to Yes
  • Target version set to Katello Backlog

#2 Updated by The Foreman Bot 9 months ago

  • Assignee set to Partha Aji
  • Status changed from New to Ready For Testing
  • Pull request https://github.com/Katello/hammer-cli-katello/pull/689 added

#3 Updated by Partha Aji 9 months ago

  • Status changed from Ready For Testing to Closed

#4 Updated by Justin Sherrill 9 months ago

  • Triaged changed from Yes to No
  • Target version deleted (Katello Backlog)

#5 Updated by Chris Roberts 9 months ago

  • Target version set to Katello 3.13.0
  • Found in Releases Katello 3.12.3 added
  • Found in Releases deleted (Katello 3.12.0)

#6 Updated by Jonathon Turel 9 months ago

  • Triaged changed from No to Yes

Also available in: Atom PDF