Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Wrong access rights of pcmdi_metrics-v1.1.1-py2.7.egg-info in 2.8.0! #496

Closed
jypeter opened this issue May 16, 2017 · 8 comments
Closed

Comments

@jypeter
Copy link

jypeter commented May 16, 2017

@doutriaux1 and @durack1 with my usual luck, I have found an unusual problem...

I have installed PMP in 2.8.0, just in case, with
conda install -n cdatm14 -c conda-forge -c pcmdi pcmdi_metrics

I have not used PMP, so I assume it is OK, but it has the following weird side effect: if another person uses my shared CDAT installation, AND wants to use ipython, then there is a huge traceback due to the fact that pcmdi_metrics-v1.1.1-py2.7.egg-info can be read only by the owner (the person who installed cdat, i.e me)

(cdatm14) bash-4.1$ ls -l /home/share/unix_files/cdat/miniconda2/envs/cdatm14/lib/python2.7/site-packages/pcmdi_metrics-v1.1.1-py2.7.egg-info
-rw------- 2 jypeter lsce 228 21 juil. 2016 /home/share/unix_files/cdat/miniconda2/envs/cdatm14/lib/python2.7/site-packages/pcmdi_metrics-v1.1.1-py2.7.egg-info

I have checked the content of envs/cdatm14/lib/python2.7/site-packages/ on 2 different servers where I have installed 2.8.0 and pcmdi_metrics-v1.1.1-py2.7.egg-info is the only file that is not readable by group+other.

I guess that ipython scans the content of site-packages when it starts (maybe that's one of the reasons it is so much slower to start than the regular python interpreter, and why I don't use ipython) and breaks when it can't access one of the files. This almost sounds like an ipython bug to me, but I won't take care of reporting it

Just to be sure, I have checked if some other files were having the same problem in 2.8.0, but it seems only files in PMP have this problem (you may want to check if I used the correct value for -perm below

(cdatm14) [jypmce@ciclad-ng envs]$ find cdatm14 ! -perm /g+r,o+r

cdatm14/share/pmp/obs_info_dictionary.json
cdatm14/share/pmp/disclaimer.txt
cdatm14/share/pmp/demo/pmp_input_parameters_demo1.py
cdatm14/share/pmp/demo/pmp_input_parameters_demo2_save_interpolated_clims.py
cdatm14/share/pmp/default_regions.py
cdatm14/share/pmp/pcmdi_metrics_table
cdatm14/share/pmp/graphics/vcs/portraits.scr
cdatm14/lib/python2.7/site-packages/pcmdi_metrics/pcmdi/rms_0.pyc
cdatm14/lib/python2.7/site-packages/pcmdi_metrics/pcmdi/seasonal_mean.py
cdatm14/lib/python2.7/site-packages/pcmdi_metrics/pcmdi/annual_mean.pyc
cdatm14/lib/python2.7/site-packages/pcmdi_metrics/pcmdi/io.pyc
cdatm14/lib/python2.7/site-packages/pcmdi_metrics/pcmdi/cor_xyt.pyc
cdatm14/lib/python2.7/site-packages/pcmdi_metrics/pcmdi/zonal_mean.pyc
cdatm14/lib/python2.7/site-packages/pcmdi_metrics/pcmdi/std_xyt.py
cdatm14/lib/python2.7/site-packages/pcmdi_metrics/pcmdi/__init__.py
cdatm14/lib/python2.7/site-packages/pcmdi_metrics/pcmdi/meanabs_xy.py
cdatm14/lib/python2.7/site-packages/pcmdi_metrics/pcmdi/seasonal_mean.pyc
cdatm14/lib/python2.7/site-packages/pcmdi_metrics/pcmdi/rms_xy.py
cdatm14/lib/python2.7/site-packages/pcmdi_metrics/pcmdi/mean_climate_metrics_calculations.py
cdatm14/lib/python2.7/site-packages/pcmdi_metrics/pcmdi/cor_xy.py
cdatm14/lib/python2.7/site-packages/pcmdi_metrics/pcmdi/std_xy.pyc
cdatm14/lib/python2.7/site-packages/pcmdi_metrics/pcmdi/annual_mean.py
cdatm14/lib/python2.7/site-packages/pcmdi_metrics/pcmdi/bias.py
cdatm14/lib/python2.7/site-packages/pcmdi_metrics/pcmdi/io.py
cdatm14/lib/python2.7/site-packages/pcmdi_metrics/pcmdi/rms_xyt.py
cdatm14/lib/python2.7/site-packages/pcmdi_metrics/pcmdi/cor_xyt.py
cdatm14/lib/python2.7/site-packages/pcmdi_metrics/pcmdi/std_xyt.pyc
cdatm14/lib/python2.7/site-packages/pcmdi_metrics/pcmdi/mean_climate_metrics_calculations.pyc
cdatm14/lib/python2.7/site-packages/pcmdi_metrics/pcmdi/cor_xy.pyc
cdatm14/lib/python2.7/site-packages/pcmdi_metrics/pcmdi/meanabs_xy.pyc
cdatm14/lib/python2.7/site-packages/pcmdi_metrics/pcmdi/bias.pyc
cdatm14/lib/python2.7/site-packages/pcmdi_metrics/pcmdi/rms_xyt.pyc
cdatm14/lib/python2.7/site-packages/pcmdi_metrics/pcmdi/rms_xy.pyc
cdatm14/lib/python2.7/site-packages/pcmdi_metrics/pcmdi/rms_0.py
cdatm14/lib/python2.7/site-packages/pcmdi_metrics/pcmdi/__init__.pyc
cdatm14/lib/python2.7/site-packages/pcmdi_metrics/pcmdi/zonal_mean.py
cdatm14/lib/python2.7/site-packages/pcmdi_metrics/pcmdi/std_xy.py
cdatm14/lib/python2.7/site-packages/pcmdi_metrics/__init__.py
cdatm14/lib/python2.7/site-packages/pcmdi_metrics/io/__init__.py
cdatm14/lib/python2.7/site-packages/pcmdi_metrics/io/__init__.pyc
cdatm14/lib/python2.7/site-packages/pcmdi_metrics/io/base.pyc
cdatm14/lib/python2.7/site-packages/pcmdi_metrics/io/base.py
cdatm14/lib/python2.7/site-packages/pcmdi_metrics/version.pyc
cdatm14/lib/python2.7/site-packages/pcmdi_metrics/__init__.pyc
cdatm14/lib/python2.7/site-packages/pcmdi_metrics/graphics/__init__.py
cdatm14/lib/python2.7/site-packages/pcmdi_metrics/graphics/portraits.pyc
cdatm14/lib/python2.7/site-packages/pcmdi_metrics/graphics/__init__.pyc
cdatm14/lib/python2.7/site-packages/pcmdi_metrics/graphics/portraits.py
cdatm14/lib/python2.7/site-packages/pcmdi_metrics/version.py
cdatm14/lib/python2.7/site-packages/pcmdi_metrics-v1.1.1-py2.7.egg-info
@durack1
Copy link
Collaborator

durack1 commented May 16, 2017

@jypeter thanks for this feedback, when the conda packages are being built we might have to add in a perm wash to ensure that similar issues don't occur in the future.. It's the first time that we've received such a specific use case issue, some other folks using the PMP are doing so in a shared install env

@jypeter
Copy link
Author

jypeter commented May 16, 2017

Can you check what access rights you have for this file (and the others), ideally in a 2.8.0 version?

Also, the date of the pcmdi_metrics-v1.1.1-py2.7.egg-info file is 21 juil. 2016, which seems kind of old to me, but I don't know what it should be. Could it be that I got a wrong and outdated version?

@doutriaux1
Copy link
Contributor

@jypeter when I install for users here I always do some sort of rechmod to make sure users can read files. I don't think it's due to pmp.

@jypeter
Copy link
Author

jypeter commented May 16, 2017

Not due to PMP? Why is it that I have this access rights problem only with PMP (see my find command above), on 2 separate servers? I have installed PMP the same way I have installed other packages. Look for "PMP" in my 2.8.0 installation notes

Can one of you guys check the access rights of the mentionned PMP file(s) in one of your installation (where you have not done a chmod -R a+rX cdat)

@doutriaux1
Copy link
Contributor

@durack1 @zshaheen where is the pmp conda package built at? crunchy and ocenaonly have weird permission mask by dsefault, could that be it?

@zshaheen
Copy link
Contributor

@doutriaux1 @durack1 Nightlies are done on Travis. I didn't build the conda packages for v1.1.1, but I usually use aims4, however I never worked on a shared env.

@jypeter
Copy link
Author

jypeter commented Aug 31, 2017

@doutriaux1 @zshaheen The problem seems to have been solved in pcmdi_metrics 1.1.2 used in uvcdat 2.10

My initial problem was with pcmdi_metrics 1.1.1 used in uvcdat 2.8.0

How did you solve this in the end?

@durack1
Copy link
Collaborator

durack1 commented Nov 21, 2017

This issue was solved in 1.1.2 so closing

@durack1 durack1 closed this as completed Nov 21, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants