Even though permissions are successfully set in check_path (according to adrian), some warnings are generated when run:

Created <code><em>sites/test.hostmaster.koumbit.net/modules</em>
Changed ownership of <em>sites/test.hostmaster.koumbit.net/modules</em>
Could not change group ownership <em>sites/test.hostmaster.koumbit.net/modules</em> (500 != 33)
Could not change permissions <em>sites/test.hostmaster.koumbit.net/modules</em> to 1512 (750 != 2750)

Comments

adrian’s picture

adrian commented
Version: » 5.x-0.1-alpha1

While busy working on http://drupal.org/node/237555 i needed to add more provision_check_path operations, so I went over the entire thing with a fine tooth comb.

I ended up splitting the big ugly switch into small provision_path_$op functions, and i pass a third variable to these functions as a reference, so that these functions can provide more accurate description of the issues.

Also added some helper functions to help checking for #245664. So now the chmod functions check that the user and groups exist, and that the user is in the group it is trying to change the file to, before attempting it.

still need to wrap my head around what is causing this specific case, but it's much closer and the testing is much more flexible now.

Also, it's exhaustively documented.

Will commit after some more testing.

adrian’s picture

adrian commented
Status: Active » Fixed

fixed in http://drupal.org/cvs?commit=120930

Anonymous’s picture

Anonymous (not verified) commented
Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for two weeks with no activity.

  • Commit 76e90b7 on debian, dev-dns, dev-envobject, dev-koumbit, dev-log_directory, dev-migrate_aliases, dev-multiserver-install, dev-newhooks, dev-nginx, dev-platform_management, dev-ports, dev-purgebackup, dev-restore, dev-services, dev-simplerinstaller, dev-site_rename, dev-ssl, dev_716166_apache_conf, dev_dns, dev_server_verify, prod-koumbit, ssl, dev-ssl-ip-allocation-refactor, dev-1205458-move_sites_out_of_platforms, 7.x-3.x, dev-subdir-multiserver, 6.x-2.x-backports, dev-helmo-3.x authored by adrian: 
    Practically completely rewrote the provision_check_path function.

It is...