Project

General

Profile

Actions

Bug #10732

closed

Permissions problems when exporting / importing global archive

Added by I C almost 7 years ago. Updated almost 2 years ago.

Status:
Resolved
Priority:
N/A
Category:
Web - Config management
Target version:
Severity:
Major - prevents use of part of Rudder | no simple workaround
UX impact:
User visibility:
Operational - other Techniques | Technique editor | Rudder settings
Effort required:
Medium
Priority:
0
Name check:
Fix check:
Regression:

Description

Hi,

Technique editor return a permission error when saving a technique after exporting and importing a global archive from a server to another server.

It seems that permissions on files inside /var/rudder/configuration-repository/ncf/ncf-hooks.d/ was not group executable :

root@ic-rudder:/var/rudder/configuration-repository/ncf# ll ncf-hooks.d/
total 8.0K
-rw-r--r-- 1 root rudder 1.6K Apr 27 22:28 post.write_technique.commit.sh
-rw-r--r-- 1 root rudder 2.7K Apr 27 22:28 post.write_technique.rudderify.sh

Permissions on the source server was correct.

Here is the error message in the technique editor when saving :

Cause: [Errno 13] Permission denied
Traceback (most recent call last):
  File "/usr/share/ncf/api/ncf_api_flask_app/views.py", line 155, in create_technique
    result = ncf.write_technique(technique,path)
  File "/usr/share/ncf/tools/ncf.py", line 635, in write_technique
    execute_hooks("post", action, path, bundle_name)
  File "/usr/share/ncf/tools/ncf.py", line 410, in execute_hooks
    check_output([hookfile,path,bundle_name])
  File "/usr/share/ncf/tools/ncf.py", line 82, in check_output
    process = subprocess.Popen(command, shell=False, stdout=subprocess.PIPE, stderr=subprocess.PIPE, universal_newlines=True, env=command_env)
  File "/usr/lib/python2.7/subprocess.py", line 710, in __init__
    errread, errwrite)
  File "/usr/lib/python2.7/subprocess.py", line 1335, in _execute_child
    raise child_exception
OSError: [Errno 13] Permission denied

Actions #1

Updated by François ARMAND almost 7 years ago

  • Category set to Web - Config management
  • Target version set to 3.1.20
  • Severity set to Minor - inconvenience | misleading | easy workaround
  • User visibility set to Operational - other Techniques | Technique editor | Rudder settings
  • Effort required set to Medium

Thanks for reporting. Indeed, we are not keeping the file permissions in the archive, and so we are not correctly restoring them (obviously:)

From an implementation point of view, we need to look how to 1/ get the perms information from jgit, 2/ find the correct options (if needeed) in zip to store perms.

Actions #2

Updated by François ARMAND almost 7 years ago

  • Translation missing: en.field_tag_list set to Sponsored
Actions #3

Updated by François ARMAND almost 7 years ago

  • Severity changed from Minor - inconvenience | misleading | easy workaround to Major - prevents use of part of Rudder | no simple workaround
  • Priority changed from 0 to 60
Actions #4

Updated by Vincent MEMBRÉ almost 7 years ago

  • Target version changed from 3.1.20 to 3.1.21
Actions #5

Updated by Benoît PECCATTE almost 7 years ago

  • Assignee set to François ARMAND

We should create a script to fix permissions.
This script could be used as post-import and probably for other problems too (post-install, rudderify ...).
Then document to use this script after import.

Actions #6

Updated by Benoît PECCATTE almost 7 years ago

  • Assignee changed from François ARMAND to Vincent MEMBRÉ
Actions #7

Updated by Vincent MEMBRÉ almost 7 years ago

  • Target version changed from 3.1.21 to 3.1.22
Actions #8

Updated by Vincent MEMBRÉ over 6 years ago

  • Target version changed from 3.1.22 to 3.1.23
  • Priority changed from 60 to 58
Actions #9

Updated by Vincent MEMBRÉ over 6 years ago

  • Target version changed from 3.1.23 to 3.1.24
  • Priority changed from 58 to 57
Actions #10

Updated by Vincent MEMBRÉ over 6 years ago

  • Target version changed from 3.1.24 to 3.1.25
  • Priority changed from 57 to 56
Actions #11

Updated by Benoît PECCATTE over 6 years ago

  • Priority changed from 56 to 55
Actions #12

Updated by Vincent MEMBRÉ over 6 years ago

  • Target version changed from 3.1.25 to 387
  • Priority changed from 55 to 61
Actions #13

Updated by Vincent MEMBRÉ over 6 years ago

  • Target version changed from 387 to 4.1.10
Actions #14

Updated by Vincent MEMBRÉ about 6 years ago

  • Target version changed from 4.1.10 to 4.1.11
  • Priority changed from 61 to 59
Actions #15

Updated by Vincent MEMBRÉ almost 6 years ago

  • Target version changed from 4.1.11 to 4.1.12
  • Priority changed from 59 to 57
Actions #16

Updated by Vincent MEMBRÉ almost 6 years ago

  • Target version changed from 4.1.12 to 4.1.13
  • Priority changed from 57 to 56
Actions #17

Updated by Benoît PECCATTE almost 6 years ago

  • Target version changed from 4.1.13 to 411
Actions #18

Updated by Benoît PECCATTE almost 6 years ago

  • Target version changed from 411 to 4.1.13
Actions #19

Updated by Vincent MEMBRÉ over 5 years ago

  • Target version changed from 4.1.13 to 4.1.14
Actions #20

Updated by Benoît PECCATTE over 5 years ago

  • Target version changed from 4.1.14 to 4.1.15
Actions #21

Updated by Vincent MEMBRÉ over 5 years ago

  • Target version changed from 4.1.15 to 4.1.16
Actions #22

Updated by Vincent MEMBRÉ over 5 years ago

  • Target version changed from 4.1.16 to 4.1.17
Actions #23

Updated by Vincent MEMBRÉ over 5 years ago

  • Target version changed from 4.1.17 to 4.1.18
  • Priority changed from 56 to 0
Actions #24

Updated by Vincent MEMBRÉ about 5 years ago

  • Target version changed from 4.1.18 to 4.1.19
Actions #25

Updated by Alexis Mousset about 5 years ago

  • Target version changed from 4.1.19 to 4.1.20
Actions #26

Updated by François ARMAND about 5 years ago

  • Target version changed from 4.1.20 to 4.1.21
Actions #27

Updated by Vincent MEMBRÉ almost 5 years ago

  • Target version changed from 4.1.21 to 4.1.22
Actions #28

Updated by Vincent MEMBRÉ almost 5 years ago

  • Target version changed from 4.1.22 to 4.1.23
Actions #29

Updated by Vincent MEMBRÉ almost 5 years ago

  • Target version changed from 4.1.23 to 4.1.24
Actions #30

Updated by Vincent MEMBRÉ over 4 years ago

  • Target version changed from 4.1.24 to 588
Actions #31

Updated by Benoît PECCATTE over 4 years ago

  • Target version changed from 588 to 5.0.13
Actions #32

Updated by Vincent MEMBRÉ over 4 years ago

  • Target version changed from 5.0.13 to 5.0.14
Actions #33

Updated by Vincent MEMBRÉ over 4 years ago

  • Target version changed from 5.0.14 to 5.0.15
Actions #34

Updated by Vincent MEMBRÉ over 4 years ago

  • Target version changed from 5.0.15 to 5.0.16
Actions #35

Updated by Alexis Mousset about 4 years ago

  • Target version changed from 5.0.16 to 5.0.17
Actions #36

Updated by Vincent MEMBRÉ almost 4 years ago

  • Target version changed from 5.0.17 to 5.0.18
Actions #37

Updated by Vincent MEMBRÉ over 3 years ago

  • Target version changed from 5.0.18 to 5.0.19
Actions #38

Updated by Vincent MEMBRÉ over 3 years ago

  • Target version changed from 5.0.19 to 5.0.20
Actions #39

Updated by Vincent MEMBRÉ over 3 years ago

  • Target version changed from 5.0.20 to 797
Actions #40

Updated by Benoît PECCATTE over 2 years ago

  • Target version changed from 797 to 6.1.14
Actions #41

Updated by Vincent MEMBRÉ over 2 years ago

  • Target version changed from 6.1.14 to 6.1.15
Actions #42

Updated by Vincent MEMBRÉ over 2 years ago

  • Target version changed from 6.1.15 to 6.1.16
Actions #43

Updated by Vincent MEMBRÉ over 2 years ago

  • Target version changed from 6.1.16 to 6.1.17
Actions #44

Updated by Vincent MEMBRÉ over 2 years ago

  • Target version changed from 6.1.17 to 6.1.18
Actions #45

Updated by Vincent MEMBRÉ over 2 years ago

  • Target version changed from 6.1.18 to 6.1.19
Actions #46

Updated by François ARMAND about 2 years ago

  • Status changed from New to Resolved

As of 7.0 at least, this is not the case anymore.

Actions

Also available in: Atom PDF