Project

General

Profile

Bug #6322

We need to clean up the Foreman environments when deleting a ContentViewEnvironment or ContentViewVersion

Added by David Davis over 6 years ago. Updated over 2 years ago.

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

Description

I think this code in KTEnvironment needs to be applied to content view version deletion actions:

http://goo.gl/Lhf8kZ

    self.content_views.each do |content_view|
      if foreman_env = Environment.find_by_katello_id(self.organization, self, content_view)
        foreman_env.destroy
      end
    end

Associated revisions

Revision 6421c79a (diff)
Added by Justin Sherrill over 6 years ago

fixes #6322 - clean up puppet environment on cv delete

Attempt to move content host's hosts. If there are others left
or host groups attached, throw a nice error. In the future
we should add host and host group puppet environment selection
to the content view version deletion

Revision f3d564d1
Added by Justin Sherrill about 6 years ago

Merge pull request #4414 from jlsherrill/6322-env_delete

fixes #6322 - clean up puppet environment on cv delete

History

#1 Updated by David Davis over 6 years ago

  • Subject changed from We need to clean up the foreman environment when deleting a ContentViewEnvironment or ContentViewVersion to We need to clean up the Foreman environments when deleting a ContentViewEnvironment or ContentViewVersion

#2 Updated by Justin Sherrill over 6 years ago

  • Status changed from New to Assigned
  • Assignee set to Justin Sherrill
  • Target version set to 48
  • Legacy Backlogs Release (now unused) set to 13
  • Difficulty set to easy

#3 Updated by Eric Helms over 6 years ago

  • Target version changed from 48 to 49

#4 Updated by Eric Helms over 6 years ago

  • Triaged changed from No to Yes

#5 Updated by Eric Helms about 6 years ago

  • Target version changed from 49 to 54

#6 Updated by The Foreman Bot about 6 years ago

  • Status changed from Assigned to Ready For Testing
  • Pull request https://github.com/Katello/katello/pull/4414 added

#7 Updated by Justin Sherrill about 6 years ago

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

Also available in: Atom PDF