Bug 786123 - audrey doesn't startup in fedora 16
Summary: audrey doesn't startup in fedora 16
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: CloudForms Cloud Engine
Classification: Retired
Component: aeolus-audrey-agent
Version: 1.0.0
Hardware: Unspecified
OS: Unspecified
unspecified
medium
Target Milestone: rc
Assignee: Ian McLeod
QA Contact: dgao
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2012-01-31 14:38 UTC by dgao
Modified: 2012-12-04 14:56 UTC (History)
6 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2012-12-04 14:56:06 UTC


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
Red Hat Product Errata RHEA-2012:1516 0 normal SHIPPED_LIVE CloudForms Cloud Engine 1.1 update 2012-12-04 19:51:45 UTC

Description dgao 2012-01-31 14:38:51 UTC
[root@localhost etc]# cat /etc/rc.local 

# This conditionally runs Audrey if it exists
[ -f /usr/bin/audrey ] && /usr/bin/audrey

[root@localhost etc]# ls /var/log/audrey.log
ls: cannot access /var/log/audrey.log: No such file or directory


[root@localhost etc]# ls -larht /etc/rc.local 
-rw-r--r--. 1 root root 89 Jan 30 18:57 /etc/rc.local

My hunch is that because f16 don't use /etc/rc.local by default anymore, it needs a chmod a+x in order for it to run during startup.

Comment 1 Greg Blomquist 2012-01-31 15:15:52 UTC
I suspect this is something that needs to be fixed in imagefactory (i.e., imgfac creates the rc.local file, so I think it probably needs to set the perms on the file, too)

Comment 2 Joe Vlcek 2012-01-31 16:02:39 UTC
dgao provided the credentials to the test instance.
I've successfully run the Audrey Agent from the command line.

/etc/rc.local is not executable
[root@localhost ~]# ls -aFl /etc/rc.local 
-rw-r--r--. 1 root root 89 Jan 30 18:57 /etc/rc.local

From the Fedora 16 release notes:
---------------------------------
 
http://docs.fedoraproject.org/en-US/Fedora/16/html/Release_Notes/sect-Release_Notes-Changes_for_Sysadmin.html

I've found:
-----------

3.2.4. rc.local no longer packaged
The /etc/rc.d/rc.local local customization script is no longer included by default. Administrators who need this functionality merely have to create this file, make it executable, and it will run on boot.
Upgrades are not affected by this change. 

The solution seem to be that when ImageFactory adds the invocation of
the Audrey Agent to /etc/rc.local that it also has to make /etc/rc.local
executable.

Reassigning to Ian McLeod <imcleod>

Comment 3 Ian McLeod 2012-01-31 16:33:45 UTC
Joe and Greg,

I believe we are all in violent agreement that our current method of smearing Audrey enablers between the factory and Audrey itself is sub-optimal.  I know Joe and I have explicitly discussed having the Audrey RPM put in place it's own startup script, ideally through the well established /etc/init.d mechanism.

Forcing the executable flag on our newly created rc.local seems like yet another hack to support this old scheme.

If it is essential that we support F16 as an Audrey target in 1.0, I can, of course, fix this with a few lines of patch.

What I'd much rather see is a migration to an Audrey RPM that does it's own boot-time startup and does not depend on us appending a line to rc.local in the factory.

Thoughts?

Comment 4 Joe Vlcek 2012-01-31 16:39:08 UTC
(In reply to comment #3)
> Joe and Greg,
> 
> I believe we are all in violent agreement that our current method of smearing
> Audrey enablers between the factory and Audrey itself is sub-optimal.  I know
> Joe and I have explicitly discussed having the Audrey RPM put in place it's own
> startup script, ideally through the well established /etc/init.d mechanism.
> 
> Forcing the executable flag on our newly created rc.local seems like yet
> another hack to support this old scheme.
> 
> If it is essential that we support F16 as an Audrey target in 1.0, I can, of
> course, fix this with a few lines of patch.
> 
> What I'd much rather see is a migration to an Audrey RPM that does it's own
> boot-time startup and does not depend on us appending a line to rc.local in the
> factory.
> 
> Thoughts?

agreed. And this is in the works but for post 1.0

For 1.0 I believe this should be fixed in ImageFactory.

Comment 5 wes hayutin 2012-01-31 17:03:04 UTC
My vote would be to not use imagefactory for laying down the rc.local config for audrey.

This might help in several areas.. that I wont mention in this bug to muck up the conversation

Comment 6 Ian McLeod 2012-01-31 17:04:43 UTC
Lads,

A quick fix is available here:

https://github.com/aeolusproject/imagefactory/tree/audrey_hack17

If this is found to be acceptable I'll apply it to the 1.0 release branch.

Comment 7 Joe Vlcek 2012-01-31 18:13:56 UTC

(In reply to comment #6)
> Lads,
> 
> A quick fix is available here:
> 
> https://github.com/aeolusproject/imagefactory/tree/audrey_hack17
> 
> If this is found to be acceptable I'll apply it to the 1.0 release branch.



Please apply to 1.0 branch.

Comment 8 Ian McLeod 2012-01-31 21:41:13 UTC
Done

Comment 9 James Laska 2012-02-10 14:09:52 UTC
Reassigning to proper 'aeolus-audrey-agent' component.

Comment 10 Mike Orazi 2012-08-09 15:53:21 UTC
Fedora issue, so we are dropping from 1.1.0

Comment 12 dgao 2012-09-25 16:06:23 UTC
No longer an issue as audrey now run in its own systemd/systemV script. Verified...

Comment 14 errata-xmlrpc 2012-12-04 14:56:06 UTC
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.

http://rhn.redhat.com/errata/RHEA-2012-1516.html


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