aririikonen / oracle

Full Oracle cookbook for Chef. Please see the README.md for further details.
http://oraarir.blogspot.fi/
44 stars 50 forks source link

cli11R23.rsp.erb template file does not use the [:oracle][:ora_inventory] attribute #3

Open jsanghi opened 10 years ago

jsanghi commented 10 years ago

Hello, Your response file template does not use the ora_inventory attribute. This causes a problem when attempting to install in a custom location.

Line 4 is set: INVENTORY_LOCATION=/opt/oraInventory should be INVENTORY_LOCATION= node[:oracle][:ora_inventory]

Thanks, Jason

dsp-eoss commented 10 years ago

Confirmed. Setting node[:oracle][:ora_inventory] to another value than the default /opt/oraInventory, as a result, causes Chef Client to fail, as, apparently, runInstaller gets confused at the non-existence of /opt/oraInventory:

---- Begin output of "bash"  "/tmp/chef-script20141008-1565-vv4qlz" ----
STDOUT: 
STDERR: You do not have sufficient permissions to access the inventory '/opt/oraInventory'. Installation cannot continue. It is required that the primary group of the install user is same as the inventory owner group. Make sure that the install user is part of the inventory owner group and restart the installer.: Permission denied
---- End output of "bash"  "/tmp/chef-script20141008-1565-vv4qlz" ----

We'll add a fix for this in the next release of the cookbook- coming out within the next couple weeks.

Thanks for reporting this issue!

dsp-eoss commented 10 years ago

Having had a deeper look at the code, the real culprit is here: https://github.com/aririikonen/oracle/blob/master/recipes/clibin.rb#L71

IOW, the value of inventory_loc, in oraInst.loc, should be set dynamically, using node[:oracle][:ora_inventory], but is instead hardcoded to /opt/oraInventory. The value of INVENTORY_LOCATION in the response file is actually overridden by oraInst.loc.

jsanghi commented 10 years ago

Good find!