Skip to content

Commit

Permalink
Change chmod from 777 to 755 (#19141)
Browse files Browse the repository at this point in the history
-- Changed chmod for the tasks from 777 to 755s
-- Changed Tasks: ContainerBuildV0, ContainerStructureTestV0, DuffleInstallerV0, FuncToolsInstallerV0, OpenPolicyAgentInstallerV0
  • Loading branch information
DmitriiBobreshev authored Oct 19, 2023
1 parent 9864b6c commit 42319fa
Show file tree
Hide file tree
Showing 15 changed files with 17 additions and 17 deletions.
2 changes: 1 addition & 1 deletion Tasks/ContainerBuildV0/src/utils.ts
Original file line number Diff line number Diff line change
Expand Up @@ -65,7 +65,7 @@ export async function downloadBuildctl(version: string): Promise<string> {

tl.debug('Buildctl path: ' + buildctlpath);

fs.chmodSync(buildctlpath, "777");
fs.chmodSync(buildctlpath, "755");
return buildctlpath;
}

Expand Down
2 changes: 1 addition & 1 deletion Tasks/ContainerBuildV0/task.json
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@
"author": "Microsoft Corporation",
"version": {
"Major": 0,
"Minor": 225,
"Minor": 230,
"Patch": 0
},
"demands": [],
Expand Down
2 changes: 1 addition & 1 deletion Tasks/ContainerBuildV0/task.loc.json
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@
"author": "Microsoft Corporation",
"version": {
"Major": 0,
"Minor": 225,
"Minor": 230,
"Patch": 0
},
"demands": [],
Expand Down
2 changes: 1 addition & 1 deletion Tasks/ContainerStructureTestV0/task.json
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@
"author": "Microsoft Corporation",
"version": {
"Major": 0,
"Minor": 219,
"Minor": 230,
"Patch": 0
},
"preview": true,
Expand Down
2 changes: 1 addition & 1 deletion Tasks/ContainerStructureTestV0/task.loc.json
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@
"author": "Microsoft Corporation",
"version": {
"Major": 0,
"Minor": 219,
"Minor": 230,
"Patch": 0
},
"preview": true,
Expand Down
2 changes: 1 addition & 1 deletion Tasks/ContainerStructureTestV0/testrunner.ts
Original file line number Diff line number Diff line change
Expand Up @@ -37,7 +37,7 @@ export class TestRunner {
throw new Error(`Download or caching of tool(${runnerPath}) failed`);
}

chmodSync(runnerPath, "777");
chmodSync(runnerPath, "755");
var start = new Date().getTime();
const output: string = this.runContainerStructureTest(runnerPath, this.testFilePath, this.imageName);
var end = new Date().getTime();
Expand Down
2 changes: 1 addition & 1 deletion Tasks/DuffleInstallerV0/src/duffleinstaller.ts
Original file line number Diff line number Diff line change
Expand Up @@ -39,7 +39,7 @@ async function downloadDuffle(version: string): Promise<string> {
}

const dufflePath = path.join(cachedToolPath, DuffleToolName + getExecutableExtension());
fs.chmodSync(dufflePath, '777');
fs.chmodSync(dufflePath, '755');
return dufflePath;
}

Expand Down
2 changes: 1 addition & 1 deletion Tasks/DuffleInstallerV0/task.json
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@
"author": "Microsoft Corporation",
"version": {
"Major": 0,
"Minor": 228,
"Minor": 230,
"Patch": 0
},
"demands": [],
Expand Down
2 changes: 1 addition & 1 deletion Tasks/DuffleInstallerV0/task.loc.json
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@
"author": "Microsoft Corporation",
"version": {
"Major": 0,
"Minor": 228,
"Minor": 230,
"Patch": 0
},
"demands": [],
Expand Down
4 changes: 2 additions & 2 deletions Tasks/FuncToolsInstallerV0/src/utils.ts
Original file line number Diff line number Diff line change
Expand Up @@ -75,10 +75,10 @@ async function downloadFuncToolsInternal(version: string): Promise<string> {
}

const funcPath = path.join(cachedToolpath, funcToolName + getExecutableExtension());
fs.chmodSync(funcPath, '777');
fs.chmodSync(funcPath, '755');
const gozipPath = path.join(cachedToolpath, 'gozip' + getExecutableExtension());
if (fs.existsSync(gozipPath)) {
fs.chmodSync(gozipPath, '777');
fs.chmodSync(gozipPath, '755');
}

return funcPath;
Expand Down
2 changes: 1 addition & 1 deletion Tasks/FuncToolsInstallerV0/task.json
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@
"author": "Microsoft Corporation",
"version": {
"Major": 0,
"Minor": 224,
"Minor": 230,
"Patch": 0
},
"preview": true,
Expand Down
2 changes: 1 addition & 1 deletion Tasks/FuncToolsInstallerV0/task.loc.json
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@
"author": "Microsoft Corporation",
"version": {
"Major": 0,
"Minor": 224,
"Minor": 230,
"Patch": 0
},
"preview": true,
Expand Down
4 changes: 2 additions & 2 deletions Tasks/OpenPolicyAgentInstallerV0/src/utils.ts
Original file line number Diff line number Diff line change
Expand Up @@ -83,11 +83,11 @@ export async function downloadOpa(version: string): Promise<string> {
if (!cachedToolpath || !fs.existsSync(opaPath)) {
const opaPathTmp = path.join(getTempDirectory(), opaToolName + getExecutableExtension());
taskLib.cp(opaDownloadPath, opaPathTmp, '-f');
fs.chmodSync(opaPathTmp, '777');
fs.chmodSync(opaPathTmp, '755');
return opaPathTmp;
}

fs.chmodSync(opaPath, '777');
fs.chmodSync(opaPath, '755');
return opaPath;
}

Expand Down
2 changes: 1 addition & 1 deletion Tasks/OpenPolicyAgentInstallerV0/task.json
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@
"author": "Microsoft Corporation",
"version": {
"Major": 0,
"Minor": 225,
"Minor": 230,
"Patch": 0
},
"minimumAgentVersion": "2.144.0",
Expand Down
2 changes: 1 addition & 1 deletion Tasks/OpenPolicyAgentInstallerV0/task.loc.json
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@
"author": "Microsoft Corporation",
"version": {
"Major": 0,
"Minor": 225,
"Minor": 230,
"Patch": 0
},
"minimumAgentVersion": "2.144.0",
Expand Down

0 comments on commit 42319fa

Please sign in to comment.