Bugzilla will be upgraded to version 5.0. The upgrade date is tentatively scheduled for 2 December 2018, pending final testing and feedback.
Bug 1192560 - Update foreman-discovery-image to 2.0
Update foreman-discovery-image to 2.0
Status: CLOSED ERRATA
Product: Red Hat Satellite 6
Classification: Red Hat
Component: Packaging (Show other bugs)
6.1.0
Unspecified Unspecified
high Severity high (vote)
: Beta
: Unused
Assigned To: Daniel Mach
Sachin Ghai
: Triaged
Depends On:
Blocks: 1193977
  Show dependency treegraph
 
Reported: 2015-02-13 12:10 EST by Lukas Zapletal
Modified: 2017-02-23 15:31 EST (History)
6 users (show)

See Also:
Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Environment:
Last Closed: 2015-08-12 01:25:41 EDT
Type: Bug
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---


Attachments (Terms of Use)
screenshort of available old discovery image in recent snap (142.30 KB, image/png)
2015-03-13 11:10 EDT, Sachin Ghai
no flags Details
foreman-disocvery-image for rhel6 repo (49.57 KB, image/png)
2015-03-27 06:19 EDT, Sachin Ghai
no flags Details
issue with image in "foreman-discovery-image" rpm build for rhel66 (28.41 KB, image/png)
2015-03-27 07:44 EDT, Sachin Ghai
no flags Details
Not switching root: /sysroot does not seem to be an OS tree. /etc/os-release is missing (32.88 KB, image/png)
2015-03-27 07:55 EDT, Sachin Ghai
no flags Details


External Trackers
Tracker ID Priority Status Summary Last Updated
Red Hat Product Errata RHSA-2015:1592 normal SHIPPED_LIVE Important: Red Hat Satellite 6.1.1 on RHEL 6 2015-08-12 05:04:35 EDT

  None (edit)
Description Lukas Zapletal 2015-02-13 12:10:55 EST
Upstream changed the way the image is being built. We need to upgrade Makefile and SPEC of the foreman-discovery-image repo in dist-git.

I want to change naming conventions, the way the image is built and simplify the process.

PHASE 1: Get the ISO out of brew

I would like to ask mbridon to get the build out of brew first, then send me the commands which were used to do the build. I will change our Makefile to do that. I expect something like:

  $ ./build-livecd fdi-empty.ks
  $ koji spin-livecd \
    fdi-image-rhel_7_0 \
    $(cat root/usr/share/fdi/VERSION) \
    --release $(cat root/usr/share/fdi/RELEASE) \
    --repo=http://my.repo/1 \
    --scratch \
    my-tag-image
    x86_64 \
    fdi-image.ks

I want the name of the resulting ISO image to be fdi-image-rhel_7_0 or similar. We must be able to release fdi-image based off 7_1 or 8_0.

PHASE 2: Modify the SPEC

I can do this one easily as I do not need any special permissions in koji. I will modify the SPEC to wrap the fdi-image correctly. I want to change the way we ship the image, instead of including the ISO in the RPM I will extract just the image/kernel files.
Comment 5 Lukas Zapletal 2015-02-16 09:43:37 EST
By the way if we are fine to add the upstream foreman-discovery-image kickstarts/shell scripts as SOURCE2, the tarballs are published by github automatically here:

https://github.com/theforeman/foreman-discovery-image/releases

The download that is relevant is:

https://github.com/theforeman/foreman-discovery-image/archive/2.1.0.tar.gz
Comment 8 Lukas Zapletal 2015-03-13 09:29:23 EDT
Image should have been updated in both RHEL6 and RHEL7 composes now.
Comment 9 Sachin Ghai 2015-03-13 11:08:53 EDT
In latest sat6.1 beta snap6 compose2.. in rhel6 compose image is still old. This needs to be updated.
Comment 10 Sachin Ghai 2015-03-13 11:10:02 EDT
Created attachment 1001417 [details]
screenshort of available old discovery image in recent snap
Comment 11 Sachin Ghai 2015-03-13 11:10:49 EDT
based on comment 9 and 10, moving to assigned state.
Comment 13 Lukas Zapletal 2015-03-19 06:02:57 EDT
Hello,

there is no rel-eng ticket for this, let me know if you need one. We've worked directly with dmach over e-mail in getting RHEL7 build since there were some changes in "distill" tool.

This is high priority for us. Thanks!
Comment 15 Mike McCune 2015-03-24 13:09:55 EDT
Build                                     Tag                   Built by
----------------------------------------  --------------------  ----------------
foreman-discovery-image-2.1.0-11          satellite-6.1.0-rhel-6-candidate  mbridon
Build                                     Tag                   Built by
----------------------------------------  --------------------  ----------------
foreman-discovery-image-2.1.0-9           satellite-6.1.0-rhel-7-candidate  tmlcoch
Comment 17 Sachin Ghai 2015-03-27 06:19:06 EDT
Verified with Sat61 beta snap8.  We have now foreman-discovery-image available in rhel6 repo. I didn't give it try yet. As soon as I discover a host with it.. will move this bz to verified state.
Comment 18 Sachin Ghai 2015-03-27 06:19:48 EDT
Created attachment 1007165 [details]
foreman-disocvery-image for rhel6 repo
Comment 19 Sachin Ghai 2015-03-27 07:43:07 EDT
So I just checked.. Looks like image has some issue.. I'm attaching the screenshot.

--
[root@hp-ms-01-c03 ~]# rpm -qa | grep discovery-image
foreman-discovery-image-2.1.0-11.el6_6sat.noarch
[root@hp-ms-01-c03 ~]# cat /etc/redhat-release 
Red Hat Enterprise Linux Server release 6.6 (Santiago)
[root@hp-ms-01-c03 ~]# 
--
Comment 20 Sachin Ghai 2015-03-27 07:44:39 EDT
Created attachment 1007189 [details]
issue with image in "foreman-discovery-image" rpm build for rhel66
Comment 21 Sachin Ghai 2015-03-27 07:55:22 EDT
Created attachment 1007202 [details]
Not switching root: /sysroot does not seem to be an OS tree. /etc/os-release is missing
Comment 22 Lukas Zapletal 2015-03-30 06:34:44 EDT
I have tested this build and it works fine:

https://brewweb.devel.redhat.com/buildinfo?buildID=427312

Please retag this into 6.1 RHEL6 and 7 and we are good to go.
Comment 23 Mathieu Bridon 2015-03-30 06:43:34 EDT
(In reply to Lukas Zapletal from comment #22)
> I have tested this build and it works fine:
> 
> https://brewweb.devel.redhat.com/buildinfo?buildID=427312
> 
> Please retag this into 6.1 RHEL6 and 7 and we are good to go.

To get it tagged, it needs to be added to the errata.
Comment 27 Sachin Ghai 2015-04-01 08:45:32 EDT
Verified with snap9(Satellite-6.1.0-RHEL-6-20150331.1).

Following version of discovery_image works with sat6 on rhel6.

foreman-discovery-image.noarch 0:2.1.0-13.el7sat
Comment 29 Bryan Kearney 2015-08-11 09:30:26 EDT
This bug is slated to be released with Satellite 6.1.
Comment 30 errata-xmlrpc 2015-08-12 01:25:41 EDT
Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory, and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

https://access.redhat.com/errata/RHSA-2015:1592

Note You need to log in before you can comment on or make changes to this bug.