nodeshift --watch fails to set permissions on remote directory

[lance]

When running nodeshift --watch, I get an error shown below. This is because the oc rsync command is trying to set permissions on the transferred files. The --no-perms=true flag (noted here) would fix this.

~/s/m/mywebapp (master *=) oc rsync --no-perms=true ./src/ mywebapp-1-k47tq:.
building file list ... done

sent 150 bytes  received 20 bytes  113.33 bytes/sec
total size is 9775  speedup is 57.50
~/s/m/mywebapp (master *=) nodeshift watch --strictSSL=false --no-perms=true
2019-01-13T17:12:51.150Z INFO loading configuration
2019-01-13T17:12:51.167Z INFO using namespace myproject at https://192.168.99.100:8443
2019-01-13T17:12:51.230Z INFO Waiting for Pod: mywebapp-1-k47tq to become ready
2019-01-13T17:12:51.231Z INFO Pod: mywebapp-1-k47tq is ready
2019-01-13T17:12:51.896Z INFO stdout: building file list ...

19 files to consider896Z INFO stdout:  0 files...

2019-01-13T17:12:51.899Z INFO stdout: ./

2019-01-13T17:12:51.899Z INFO stdout: .openshiftio/
public/
src/

2019-01-13T17:12:51.910Z INFO stdout:

2019-01-13T17:12:51.910Z INFO stdout: sent 439 bytes  received 44 bytes  966.00 bytes/sec
total size is 693522  speedup is 1435.86

2019-01-13T17:12:51.998Z ERROR stderr: rsync: failed to set permissions on "/opt/app-root/src/.": Operation not permitted (1)
rsync: failed to set permissions on "/opt/app-root/src/.openshiftio": Operation not permitted (1)
rsync: failed to set permissions on "/opt/app-root/src/public": Operation not permitted (1)
rsync: failed to set permissions on "/opt/app-root/src/src": Operation not permitted (1)
rsync error: some files could not be transferred (code 23) at /BuildRoot/Library/Caches/com.apple.xbs/Sources/rsync/rsync-52/rsync/main.c(996) [sender=2.6.9]

2019-01-13T17:12:51.998Z ERROR stderr: error: exit status 23

2019-01-13T17:12:52.004Z INFO 1 end
2019-01-13T17:12:52.004Z INFO complete

[lholmquist]

@lance i wonder if --no-perms=true should just be set, or should this be a nodeshift option that can be passed in. TBH, i don't think we need to get to fancy here, and could probably just set it

[lance]

Agree - I think we can just set it.