actions/runner-container-hooks
1
0
Fork 0
mirror of https://github.com/actions/runner-container-hooks.git synced 2025-12-14 08:36:45 +00:00
Code Issues Packages Projects Releases Wiki Activity

prepare job in k8s does not allow for no job container

Browse Source
This commit is contained in:
Nikola Jokic
2022-06-10 16:38:07 +02:00
parent 3e04b45585
commit d3ec1c0040
2 changed files with 6 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
packages/k8s/src/hooks/prepare-job.ts
View File

@@ -25,6 +25,10 @@ export async function prepareJob(
args: prepareJobArgs, args: prepareJobArgs,
responseFile responseFile
): Promise<void> { ): Promise<void> {
if (!args.container) {
throw new Error('Job Container is required.')
}
await prunePods() await prunePods()
if (!(await isAuthPermissionsOK())) { if (!(await isAuthPermissionsOK())) {
throw new Error( throw new Error(

4
packages/k8s/tests/prepare-job-test.ts
View File

@@ -69,10 +69,10 @@ describe('Prepare job', () => {
).rejects.toThrow() ).rejects.toThrow()
}) })
it('should run prepare job without the job container', async () => { it('should not run prepare job without the job container', async () => {
prepareJobData.args.container = undefined prepareJobData.args.container = undefined
await expect( await expect(
prepareJob(prepareJobData.args, prepareJobOutputFilePath) prepareJob(prepareJobData.args, prepareJobOutputFilePath)
).resolves.not.toThrow() ).rejects.toThrow()
}) })
}) })