Doesn't work on Linux agent

[maxoris]

I'm trying to patch my xml file test.config with vso agent on Ubuntu 16.04, but there are several issues:

1. Problem with file name xmlPatch.js in package.
   Agent log:

2016-08-31T12:42:52.4259340Z
##[error]File not found: '/home/user/vsoagent/_work/_tasks/XmlPatch_acee093c-d84e-456e-b6f3-1d65969291bd/1.1.1000/xmlPatch.js
`

fixed with renaming xmlpatch.js to xmlPatch.js on agent machine
2.
Agent log:

2016-08-31T12:49:55.5736020Z ##[section]Starting: Patch files test.config
2016-08-31T12:49:56.7633110Z = /configuration/appSettings/add[@key=v1']/@value 0
2016-08-31T12:49:56.7633820Z = /configuration/appSettings/add[@key='v2']/@value
2016-08-31T12:49:56.7682220Z SyntaxError: Unexpected token u
2016-08-31T12:49:56.7777580Z ##[error]SyntaxError: Unexpected token u
2016-08-31T12:49:56.7883560Z ##[section]Finishing: Patch files test.config

Worker log:
[2016-08-31 12:49:55Z INFO StepsRunner] Processing step: DisplayName='Patch files test.config', AlwaysRun=False, ContinueOnError=False, Critical=False, Enabled=True, Finally=False
[2016-08-31 12:49:55Z INFO StepsRunner] Starting the step.
[2016-08-31 12:49:55Z INFO TaskManager] Loading task definition '/home/user/vsoagent/_work/_tasks/XmlPatch_acee093c-d84e-456e-b6f3-1d65969291bd/1.1.1000/task.json'.
[2016-08-31 12:49:55Z INFO TaskRunner] The original input is a rooted path, return absolute path: /home/user/vsoagent/_work/r1/a
[2016-08-31 12:49:55Z INFO ExtensionManager] Getting extensions for interface: 'Microsoft.VisualStudio.Services.Agent.Worker.IJobExtension'
[2016-08-31 12:49:55Z INFO ReleaseJobExtension] Artifact root path is system.artifactsDirectory: /home/user/vsoagent/_work/r1/a
[2016-08-31 12:49:55Z INFO ReleaseJobExtension] After prefix Artifact Path Root provide by JobExtension: /home/user/vsoagent/_work/r1/a
[2016-08-31 12:49:55Z INFO ReleaseJobExtension] Return absolute path after prefix ArtifactPathRoot: /home/user/vsoagent/_work/r1/a
[2016-08-31 12:49:55Z INFO TaskRunner] release JobExtension resolved a rooted path:: /home/user/vsoagent/_work/r1/a
[2016-08-31 12:49:55Z INFO ProcessInvoker] Starting process:
[2016-08-31 12:49:55Z INFO ProcessInvoker] File name: '/home/user/vsoagent/externals/node/bin/node'
[2016-08-31 12:49:55Z INFO ProcessInvoker] Arguments: '"/home/user/vsoagent/_work/_tasks/XmlPatch_acee093c-d84e-456e-b6f3-1d65969291bd/1.1.1000/xmlPatch.js"'
[2016-08-31 12:49:55Z INFO ProcessInvoker] Working directory: '/home/user/vsoagent/_work/r1/a'
[2016-08-31 12:49:55Z INFO ProcessInvoker] Require exit code zero: 'True'
[2016-08-31 12:49:55Z INFO ProcessInvoker] Encoding web name: ; code page: ''
[2016-08-31 12:49:55Z INFO ProcessInvoker] Process started with process id 29292, waiting for process exit.
[2016-08-31 12:49:55Z INFO JobServerQueue] Try to append 5 batches web console lines, success rate: 5/5.
[2016-08-31 12:49:56Z INFO JobServerQueue] Try to upload 1 log files or attachments, success rate: 1/1.
[2016-08-31 12:49:56Z INFO ProcessInvoker] Finished process with exit code 0, and elapsed time 00:00:01.2116654.
[2016-08-31 12:49:56Z INFO StepsRunner] Step result: Failed

I haven't found any way to fix it.

[asiffermann]

Hi @maxoris, and thank you for reporting those issues!

1. Oops, we will rename it ASAP! We are testing our tasks only on Windows and MacOS agents, so we weren't aware of case sensitivity... Nice to see an Ubuntu agent user!
2. There is something strange in your agent log, the following line seems to have a missing quote:
   2016-08-31T12:49:56.7633110Z = /configuration/appSettings/add[@key=v1']/@value 0
   Could you detail the parameters you are giving to the task? Particulary the "Patch Content" and the "Syntax type"?

[maxoris]

Hi @asiffermann, thank you for quick response.
There is no problem with quotes, I just replaced it for simplicity reasons and missed one quote, sorry.
My parameters are:
PathContent

= /configuration/appSettings/add[@key='v1']/@value 1
= /configuration/appSettings/add[@key='v2']/@value 2

Syntax type

Slick Syntax

[asiffermann]

I think your PathContent is missing the path / value separator.
As explained on the wiki, it should be formated like:

= /configuration/appSettings/add[@key='v1']/@value => 1
= /configuration/appSettings/add[@key='v2']/@value => 2

Could you test it again with this change?

[maxoris]

With separator I got:

2016-08-31T15:48:01.4701230Z ##[section]Starting: Patch files test.config
2016-08-31T15:48:02.1124610Z = /configuration/appSettings/add[@key='v1']/@value => 1
2016-08-31T15:48:02.1126370Z = /configuration/appSettings/add[@key='v2']/@value => 2
2016-08-31T15:48:02.1148000Z SyntaxError: Unexpected token =
2016-08-31T15:48:02.1238670Z ##[error]SyntaxError: Unexpected token =
2016-08-31T15:48:02.2053360Z ##[section]Finishing: Patch files test.config

[asiffermann]

Are your values numbers? Can you try escape them with quotes? Something like this:

= /configuration/appSettings/add[@key='v1']/@value => "1"
= /configuration/appSettings/add[@key='v2']/@value => "2"

[sandorfr]

@maxoris I fixed the casing issue, this might take a few minutes to propagate. This leaves the latter issue.

[sandorfr]

@maxoris I just tested the patch you provided and it works just fine on hosted agent. However I do have the same error as you do if I use Standard syntax (which assumes plain json). Are you sure you didn't forget to set the Syntax Type to Slick?

[maxoris]

@sandorfr Yes, you're right, I had to remove build step to keep my release definition working as for some reason disabling build step didn't have any effect and I forgot to set correct Syntax Type.
Now my Patch content is:

= /configuration/appSettings/add[@key='url']/@value => "$(TestUrl)"

And it works fine for me.
Thank you!

[asiffermann]

Great to know!

The casing issue is also deployed with the version 1.1.2001, can you confirm us it is solved for you too so we can close this issue?

[maxoris]

@asiffermann, yes I've just tried 1.1.2001, it works great.