Project

General

Profile

Bug #17784

API/CLI - ISE on Deleting proxy: Can't modify frozen hash

Added by Andrew Kofink over 4 years ago. Updated almost 3 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
Foreman Proxy Content
Target version:
Difficulty:
Triaged:
Bugzilla link:
Fixed in Releases:
Found in Releases:

Description

Cloned from https://bugzilla.redhat.com/show_bug.cgi?id=1398695
Description of problem:
Server returning err500 - "Can't modify frozen hash" every time user deletes a capsule using cli or api.
The capsule seems to be removed anyway.

This is a regression in 6.3
Version-Release number of selected component (if applicable):
satellite-6.3.0-6.1.beta.el7sat.noarch
How reproducible:
always
Steps to Reproduce:
1. add a capsule (you can forward port 9090 of the stallite to some other port to simulate a different capsule):
ncat -kl -p 11018 -c "ncat sat63.com 9090 &" (you might need to set a proper selinux context for the ports or simply disable it)
hammer proxy create --name foo --url="https://sat63.com:11018"
2. use CLI or API to delete it:
  1. hammer -u admin -p changeme proxy delete --id 11
    Could not delete the proxy:
    Can't modify frozen hash
or
$ curl -ku admin:changeme -X DELETE "https://sat63.com/api/v2/smart_proxies/11" 
{
"error": {"message":"Can't modify frozen hash"}
}
Actual results:
error returned:"Can't modify frozen hash" but proxy removed
Expected results:
no errors
Additional info:

Associated revisions

Revision 3d8abda4 (diff)
Added by Andrew Kofink over 4 years ago

Fixes #17784 - Check prxy persisted? before update

Check that the SmartProxy is persisted? before updating the puppet_path

History

#1 Updated by Justin Sherrill over 4 years ago

  • Subject changed from API/CLI - ISE on Deleting proxy: Can't modify frozen hash to API/CLI - ISE on Deleting proxy: Can't modify frozen hash
  • Legacy Backlogs Release (now unused) set to 114

#2 Updated by Justin Sherrill over 4 years ago

  • Priority changed from High to Normal

#3 Updated by Andrew Kofink over 4 years ago

  • Target version set to 151

#4 Updated by Andrew Kofink over 4 years ago

  • Assignee set to Andrew Kofink

#5 Updated by The Foreman Bot over 4 years ago

  • Status changed from New to Ready For Testing
  • Legacy Backlogs Release (now unused) deleted (114)
  • Pull request https://github.com/Katello/katello/pull/6510 added

#6 Updated by Andrew Kofink over 4 years ago

  • Status changed from Ready For Testing to Closed
  • % Done changed from 0 to 100

#7 Updated by Justin Sherrill over 4 years ago

  • Legacy Backlogs Release (now unused) set to 188

Also available in: Atom PDF