Bug 2218821

Summary:	Hammer accepts non-existent LCE in host update
Product:	Red Hat Satellite	Reporter:	Vladimír Sedmík <vsedmik>
Component:	Hosts - Content	Assignee:	Lucy Fu <lufu>
Status:	POST ---	QA Contact:	Satellite QE Team <sat-qe-bz-list>
Severity:	medium	Docs Contact:
Priority:	unspecified
Version:	6.14.0	CC:	chrobert, jlenz, rlavi
Target Milestone:	Unspecified	Keywords:	Triaged
Target Release:	Unused
Hardware:	x86_64
OS:	Linux
Whiteboard:
Fixed In Version:		Doc Type:	If docs needed, set a value
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:		Type:	Bug
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:
Embargoed:

Description Vladimír Sedmík 2023-06-30 07:26:37 UTC

Description of problem:
Hammer host update accepts non-existent LCE id and says 'Host updated.', but it's not.


Version-Release number of selected component (if applicable):
6.14.0 snap 4


How reproducible:
always


Steps to Reproduce:
1. Have a host registered to the Satellite
2. Run `hammer host update --id <host_id> --lifecycle-environment-id 999999`


Actual results:
[root@sat ~]# hammer host update --id 24 --lifecycle-environment-id 999999
Host updated.


Expected results:
Should fail with proper message (like "LCE does not exist", "LCE is not assigned to the content source", etc.)

Comment 3 Lucy Fu 2023-08-09 22:05:23 UTC

Created redmine issue https://projects.theforeman.org/issues/36667 from this bug

Comment 4 Bryan Kearney 2023-08-10 16:02:58 UTC

Upstream bug assigned to lufu

Comment 5 Bryan Kearney 2023-08-10 16:03:01 UTC

Upstream bug assigned to lufu

Comment 6 Bryan Kearney 2023-08-17 00:03:02 UTC

Moving this bug to POST for triage into Satellite since the upstream issue https://projects.theforeman.org/issues/36667 has been resolved.