Project

General

Profile

Bug #11195

rendered kickstarts do not include ks url when content view is used

Added by Justin Sherrill about 6 years ago. Updated about 3 years ago.

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

Description

Steps:

1. create a host and select: * content source * lifecycle environment * content view (that contains a ks tree) * operating system * arch

2. fill out other information needed for the host
3. render the provisioning template for the host

template will not contain any sort of url at all for the kickstart tree


Related issues

Related to Foreman - Feature #11035: Allow all templates preview during their creation and editingClosed2015-07-07

Associated revisions

Revision 63de0095 (diff)
Added by Stephen Benjamin about 6 years ago

fixes #11195 - handle foreman restructuring of kickstart_attributes

Revision e5e6f58c
Added by Stephen Benjamin about 6 years ago

Merge pull request #5403 from stbenjam/11195

fixes #11195 - handle foreman restructuring of kickstart_attributes

History

#2 Updated by Justin Sherrill about 6 years ago

  • Subject changed from rendered kickstart do not include ks url when content view is used to rendered kickstarts do not include ks url when content view is used

#3 Updated by Dominic Cleal about 6 years ago

  • Related to Feature #11035: Allow all templates preview during their creation and editing added

#4 Updated by Stephen Benjamin about 6 years ago

  • Status changed from New to Need more information

I'm not sure how we handle this, the broken commit isn't slated to be in Foreman 1.9 GA, but rather 1.9.1, so we can't get the fix into 2.3.0 GA.

The only option would really to be to make sure Katello 2.3.1 is released concurrently with 1.9.1, however we have no formal agreement to synchronize releases. Foreman would have to guarantee they release 1.9.1 on Katello's schedule.

Or the relevant commit doesn't get added to 1.9.1 at all -- it's a plugin API change to move methods around like that in a patch release.

#5 Updated by Marek Hulán about 6 years ago

I thought Justin had a fix for this that we discussed few days ago.

#6 Updated by The Foreman Bot about 6 years ago

  • Status changed from Need more information to Ready For Testing
  • Pull request https://github.com/Katello/katello/pull/5403 added
  • Pull request deleted ()

#7 Updated by Anonymous about 6 years ago

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

Also available in: Atom PDF