Re: [smoke failure] Permission denied error while install-pygluypPYTHON

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Refreshed the patch to fix glupy, systemd and mount.glusterfs files.
http://review.gluster.org/14315
regards
Aravinda
On 05/13/2016 10:25 AM, Kaushal M wrote:
On Fri, May 13, 2016 at 9:59 AM, Aravinda <avishwan@xxxxxxxxxx> wrote:
Sent patch to fix glupy installation issue.
http://review.gluster.org/#/c/14315/

regards
Aravinda

On 05/12/2016 11:28 PM, Aravinda wrote:

Sorry miss from my side. Updated list of files/dir which do not honour
--prefix

usr/lib/
usr/lib/systemd
usr/lib/systemd/system
usr/lib/systemd/system/glusterd.service
usr/lib/python2.7
usr/lib/python2.7/site-packages
usr/lib/python2.7/site-packages/gluster
usr/lib/python2.7/site-packages/gluster/__init__.pyo
usr/lib/python2.7/site-packages/gluster/__init__.pyc
usr/lib/python2.7/site-packages/gluster/__init__.py
usr/lib/python2.7/site-packages/gluster/glupy
usr/lib/python2.7/site-packages/gluster/glupy/__init__.pyo
usr/lib/python2.7/site-packages/gluster/glupy/__init__.pyc
usr/lib/python2.7/site-packages/gluster/glupy/__init__.py
sbin/
sbin/mount.glusterfs

Thanks for identifying the list of paths. We need to fix all of this.
I've opened a bug [1] so that this can be correctly tracked and fixed.

[1] https://bugzilla.redhat.com/show_bug.cgi?id=1335717

Things I did to find above list.
./autogen.sh
./configure --prefix=/usr/local
DESTDIR=/tmp/glusterfs make install

Then listed all the files which are in /tmp/glusterfs except
/tmp/glusterfs/usr/local

regards
Aravinda

On 05/12/2016 08:56 PM, Aravinda wrote:


regards
Aravinda

On 05/12/2016 08:23 PM, Niels de Vos wrote:

On Thu, May 12, 2016 at 04:28:40PM +0530, Aravinda wrote:

regards
Aravinda

On 05/12/2016 04:08 PM, Kaushal M wrote:

The install path should be `$DESTDIR/$PREFIX/<path>`.

PREFIX should be the path under which the file is going to be installed.

Yes. That is substituted during ./configure if --prefix is passed, otherwise
generated Makefile will have $prefix variable. I think glupy need to
installed on /usr/lib/python2.6/site-packages/  to import python packages
globally while testing. Same rule is used to deploy systemd unit files.
(Prefix is not used)

I'm not convinced about this yet. If someone decides to use --prefix, I
think we should honour that everywhere. If that is not common, we can
introduce an additional ./configure option for the uncommon use-cases
like the Python site-packages.

Do you have a reference where the --prefix option explains that some
contents may not use it?

Following files/dirs are not honoring prefix, I am not sure about the exact
reason(for example, /var/log or /var/lib/glusterd)

sbin
sbin/mount.glusterfs
usr/lib/
usr/lib/systemd
usr/lib/systemd/system
usr/lib/systemd/system/glustereventsd.service
usr/lib/systemd/system/glusterd.service
usr/lib/python2.7
usr/lib/python2.7/site-packages
usr/lib/python2.7/site-packages/gluster
usr/lib/python2.7/site-packages/gluster/__init__.pyo
usr/lib/python2.7/site-packages/gluster/__init__.pyc
usr/lib/python2.7/site-packages/gluster/__init__.py
usr/lib/python2.7/site-packages/gluster/glupy
usr/lib/python2.7/site-packages/gluster/glupy/__init__.pyo
usr/lib/python2.7/site-packages/gluster/glupy/__init__.pyc
usr/lib/python2.7/site-packages/gluster/glupy/__init__.py
var/
var/lib
var/lib/glusterd
var/lib/glusterd/glusterfind
var/lib/glusterd/glusterfind/.keys
var/lib/glusterd/groups
var/lib/glusterd/groups/virt
var/lib/glusterd/hooks
var/lib/glusterd/hooks/1
var/lib/glusterd/hooks/1/delete
var/lib/glusterd/hooks/1/delete/post
var/lib/glusterd/hooks/1/delete/post/S57glusterfind-delete-post.py
var/lib/glusterd/hooks/1/gsync-create
var/lib/glusterd/hooks/1/gsync-create/post
var/lib/glusterd/hooks/1/gsync-create/post/S56glusterd-geo-rep-create-post.sh
var/lib/glusterd/hooks/1/reset
var/lib/glusterd/hooks/1/reset/post
var/lib/glusterd/hooks/1/reset/post/S31ganesha-reset.sh
var/lib/glusterd/hooks/1/stop
var/lib/glusterd/hooks/1/stop/pre
var/lib/glusterd/hooks/1/stop/pre/S30samba-stop.sh
var/lib/glusterd/hooks/1/stop/pre/S29CTDB-teardown.sh
var/lib/glusterd/hooks/1/start
var/lib/glusterd/hooks/1/start/post
var/lib/glusterd/hooks/1/start/post/S31ganesha-start.sh
var/lib/glusterd/hooks/1/start/post/S30samba-start.sh
var/lib/glusterd/hooks/1/start/post/S29CTDBsetup.sh
var/lib/glusterd/hooks/1/set
var/lib/glusterd/hooks/1/set/post
var/lib/glusterd/hooks/1/set/post/S32gluster_enable_shared_storage.sh
var/lib/glusterd/hooks/1/set/post/S30samba-set.sh
var/lib/glusterd/hooks/1/add-brick
var/lib/glusterd/hooks/1/add-brick/pre
var/lib/glusterd/hooks/1/add-brick/pre/S28Quota-enable-root-xattr-heal.sh
var/lib/glusterd/hooks/1/add-brick/post
var/lib/glusterd/hooks/1/add-brick/post/disabled-quota-root-xattr-heal.sh
var/log
var/log/glusterfs
var/run
var/run/gluster

Thanks,
Niels


DESTDIR is a way to make it easier to package builders to collect
installed files.
It shouldn't be used as an alternative to prefix. And I think software
generally shouldn't be run from DESTDIR.

More information is available at
https://www.gnu.org/software/automake/manual/html_node/DESTDIR.html

On Thu, May 12, 2016 at 3:55 PM, Aravinda <avishwan@xxxxxxxxxx> wrote:

regards
Aravinda

On 05/12/2016 02:33 PM, Niels de Vos wrote:

On Thu, May 12, 2016 at 02:01:43PM +0530, Aravinda wrote:

I checked the Makefile.am and configure.ac of glupy, looks good to me. I
don't think we have issue in glupy.

If we run make install with DESTDIR empty then
`${DESTDIR}/usr/lib/python2.6/site-packages/gluster` will become
/usr/lib/python2.6/site-packages/gluster. So we will get that error.

For example,
     DESTDIR= make install
     or
     make install DESTDIR=

Can we check how we are executing smoke test?

I think it is this script, no DESTDIR in there:


https://github.com/gluster/glusterfs-patch-acceptance-tests/blob/master/build.sh

My guess is that the --prefix ./configure option is not honoured?

DESTDIR will not get substitute during ./configure, it is used during make
install. Once we run ./autogen.sh and ./configure(with whatever prefix),
generated Makefile for glupy is


install-pyglupyPYTHON: $(pyglupy_PYTHON)
     @$(NORMAL_INSTALL)
     @list='$(pyglupy_PYTHON)'; dlist=; list2=; test -n "$(pyglupydir)" ||
list=; \
     if test -n "$$list"; then \
       echo " $(MKDIR_P) '$(DESTDIR)$(pyglupydir)'"; \
       $(MKDIR_P) "$(DESTDIR)$(pyglupydir)" || exit 1; \
     fi; \
     for p in $$list; do \
       if test -f "$$p"; then b=; else b="$(srcdir)/"; fi; \
       if test -f $$b$$p; then \
         $(am__strip_dir) \
         dlist="$$dlist $$f"; \
         list2="$$list2 $$b$$p"; \
       else :; fi; \
     done; \
     for file in $$list2; do echo $$file; done | $(am__base_list) | \
     while read files; do \
       echo " $(INSTALL_DATA) $$files '$(DESTDIR)$(pyglupydir)'"; \
       $(INSTALL_DATA) $$files "$(DESTDIR)$(pyglupydir)" || exit $$?; \
     done || exit $$?; \
     if test -n "$$dlist"; then \
       $(am__py_compile) --destdir "$(DESTDIR)" \
                         --basedir "$(pyglupydir)" $$dlist; \
     else :; fi

If you run `make install` without destdir then it will install to machine's
global path depending on prefix.(If this is the case then their is genuine
"permission denied" error in the machine I think.

If we are packaging or installing to custom target, we should pass DESTDIR.

DESTDIR=/build/install make install



Niels


regards
Aravinda

On 05/12/2016 12:29 PM, Niels de Vos wrote:

On Thu, May 12, 2016 at 01:14:07AM -0400, Raghavendra Gowdappa wrote:

https://build.gluster.org/job/smoke/27674/console

06:09:06 /bin/mkdir: cannot create directory
`/usr/lib/python2.6/site-packages/gluster': Permission denied
06:09:06 make[6]: *** [install-pyglupyPYTHON] Error 1

This definitely is a bug in the installation of glupy. Nothing should
get installed under /usr, teh installation process is instructed to do
its install under /build/install.

Did someone file a bug for this yet?

Thanks,
Niels

06:09:06 make[5]: *** [install-am] Error 2
06:09:06 make[4]: *** [install-recursive] Error 1
06:09:06 make[3]: *** [install-recursive] Error 1
06:09:06 make[2]: *** [install-recursive] Error 1
06:09:06 make[1]: *** [install-recursive] Error 1
06:09:06 make: *** [install-recursive] Error 1
06:09:06 Build step 'Execute shell' marked build as failure
06:09:06 Finished: FAILURE

regards,
Raghavendra
_______________________________________________
Gluster-devel mailing list
Gluster-devel@xxxxxxxxxxx
http://www.gluster.org/mailman/listinfo/gluster-devel


_______________________________________________
Gluster-devel mailing list
Gluster-devel@xxxxxxxxxxx
http://www.gluster.org/mailman/listinfo/gluster-devel



_______________________________________________
Gluster-devel mailing list
Gluster-devel@xxxxxxxxxxx
http://www.gluster.org/mailman/listinfo/gluster-devel




_______________________________________________
Gluster-devel mailing list
Gluster-devel@xxxxxxxxxxx
http://www.gluster.org/mailman/listinfo/gluster-devel




_______________________________________________
Gluster-devel mailing list
Gluster-devel@xxxxxxxxxxx
http://www.gluster.org/mailman/listinfo/gluster-devel



_______________________________________________
Gluster-devel mailing list
Gluster-devel@xxxxxxxxxxx
http://www.gluster.org/mailman/listinfo/gluster-devel

_______________________________________________
Gluster-devel mailing list
Gluster-devel@xxxxxxxxxxx
http://www.gluster.org/mailman/listinfo/gluster-devel

[Index of Archives]     [Gluster Users]     [Ceph Users]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux]     [Linux OMAP]     [Linux MIPS]     [eCos]     [Asterisk Internet PBX]     [Linux API]

  Powered by Linux