Couchbase Server

Preinstall scriptlet removes links from /opt/couchbase, even when RPM_INSTALL_PREFIX0 is set

Details

  • Type: Bug Bug
  • Status: Resolved Resolved
  • Priority: Major Major
  • Resolution: Fixed
  • Affects Version/s: 1.8.1
  • Fix Version/s: 2.0.1
  • Component/s: installer
  • Security Level: Public
  • Labels:
    None
  • Environment:
    Linux RPM

Description

I found the following in the pre-install scriptlet. I imagine that you should be using $RPM_INSTALL_PREFIX0 instead of /opt/couchbase here.

{code}
if [ -d /opt/couchbase ]
then
  find /opt/couchbase -maxdepth 1 -type l | xargs rm -f || true
fi
{code}

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Steve Yen added a comment -
thanks - good eyes!
Show
Steve Yen added a comment - thanks - good eyes!
Hide
Permalink
Bryce Jasmer added a comment -
I misstated the summary of this issue. The script removes /opt/couchbase if it is a symlink, not any symlinks inside of /opt/couchbase. (This is actually a problem for us since we want to install couchbase on a different partition by creating /opt/couchbase as a link to a different location, but that gets clobbered by preinstall.)
Show
Bryce Jasmer added a comment - I misstated the summary of this issue. The script removes /opt/couchbase if it is a symlink, not any symlinks inside of /opt/couchbase. (This is actually a problem for us since we want to install couchbase on a different partition by creating /opt/couchbase as a link to a different location, but that gets clobbered by preinstall.)
Hide
Permalink
Farshid Ghods added a comment -
deferring to 2.1 per bug scrub meeting ( Dipti & Farshid -December 7th )
Show
Farshid Ghods added a comment - deferring to 2.1 per bug scrub meeting ( Dipti & Farshid -December 7th )
Hide
Permalink
Steve Yen added a comment -
Hi Bin,
This is a centos-only issue. Would like to add to your list of misc. issues to hit.
Thanks,
Steve
Show
Steve Yen added a comment - Hi Bin, This is a centos-only issue. Would like to add to your list of misc. issues to hit. Thanks, Steve
Hide
Permalink
Bin Cui added a comment -
Please take a look at server-rpm.spec.tmpl and server-deb/preinst.tmpl for possible problem.

Project: voltron
Show
Bin Cui added a comment - Please take a look at server-rpm.spec.tmpl and server-deb/preinst.tmpl for possible problem. Project: voltron
Hide
Permalink
Pavel Paulau added a comment -
Just a reminder that we need this fix in 2.0.1 branch as well.
Show
Pavel Paulau added a comment - Just a reminder that we need this fix in 2.0.1 branch as well.
Hide
Permalink
Steve Yen added a comment -
Hi Xiaoqin,
Raising this to 2.0.1 on behalf of Alex Ma.
Steve
Show
Steve Yen added a comment - Hi Xiaoqin, Raising this to 2.0.1 on behalf of Alex Ma. Steve
Hide
Permalink
Dipti Borkar added a comment -
Is this fixed?

Why is fixversion 2.1 as well?
Show
Dipti Borkar added a comment - Is this fixed? Why is fixversion 2.1 as well?
Hide
Permalink
Xiaoqin Ma added a comment -
It was planned for 2.1 first. Then @Dipti and others asked to fix it in 2.0.1 too.
Show
Xiaoqin Ma added a comment - It was planned for 2.1 first. Then @Dipti and others asked to fix it in 2.0.1 too.
Hide
Permalink
Dipti Borkar added a comment -
So is it fixed in 2.0.1?
Show
Dipti Borkar added a comment - So is it fixed in 2.0.1?
Hide
Permalink
Xiaoqin Ma added a comment -
Yes. It is fixed both in 2.1 and 2.0.1.
Show
Xiaoqin Ma added a comment - Yes. It is fixed both in 2.1 and 2.0.1.

People

  • Assignee:
    Xiaoqin Ma
    Reporter:
    Bryce Jasmer
Vote (0)
Watch (2)

Dates

  • Created:
    Updated:
    Resolved: