added checks for linux platform

__tests__/index.test.js

@@ -10,9 +10,17 @@ jest.mock('../src/main', () => ({
 }))
 
 describe('index', () => {
-  it('calls run when imported', async () => {
-    require('../src/index')
+  it('calls run when imported on linux', async () => {
+    const { init } = require('../src/index')
+    init('linux', 'x64')
 
     expect(run).toHaveBeenCalled()
   })
+
+  it('fails when imported on platform other than linux', async () => {
+    const { init } = require('../src/index')
+    init('darwin', 'x64')
+
+    expect(run).not.toHaveBeenCalled()
+  })
 })

src/index.js

@@ -4,16 +4,49 @@
 const { run } = require('./main')
 const { generateSummary } = require('./summary')
 const core = require('@actions/core')
+const os = require('os')
 
 const isPost = core.getState('isPost')
 const flag = isPost === 'true'
+const boltFailed = core.getState('boltFailed')
+const failedFlag = boltFailed === 'true'
 
-if (flag) {
-  // Post
-  generateSummary()
-} else {
-  if (!isPost) {
-    core.saveState('isPost', 'true')
+function init(platform, arch) {
+  if (flag) {
+    if (failedFlag) {
+      core.info('Skipping post action as bolt failed')
+      return
+    }
+    // Post
+    generateSummary()
+  } else {
+    if (!isPost) {
+      core.saveState('isPost', 'true')
+    }
+
+    // 'win32' | 'darwin' | 'linux' | 'freebsd' | 'openbsd' | 'android' | 'cygwin' | 'sunos'
+    if (['linux'].indexOf(platform) === -1) {
+      core.saveState('boltFailed', 'true')
+      core.setFailed(`This action is not supported on ${platform}`)
+      return
+    }
+    // Possible Archs
+    // 'x64' | 'arm' | 'arm64' | 'ia32' | 'mips' | 'mipsel' | 'ppc' | 'ppc64' | 'riscv64' | 's390' | 's390x'
+    const allowedArch = ['x64', 'arm64', 'arm']
+    if (allowedArch.indexOf(arch) === -1) {
+      core.saveState('boltFailed', 'true')
+      core.setFailed(`This action is not supported on ${arch}`)
+      return
+    }
+
+    run()
   }
-  run()
+}
+
+const platform = os.platform()
+const arch = os.arch()
+init(platform, arch)
+
+module.exports = {
+  init
 }

src/main.js

@@ -27,9 +27,17 @@ async function run() {
 
     // Changing boltUser will require changes in bolt.service and intercept.py
     const boltUser = 'bolt'
+    core.saveState('boltUser', boltUser)
+
+    const outputFile = 'output.log'
+    core.saveState('outputFile', outputFile)
+
+    const homeDir = `/home/${boltUser}`
+    core.saveState('homeDir', homeDir)
+
     const repoName = process.env.GITHUB_REPOSITORY; // e.g. koalalab-inc/bolt
     const repoOwner = repoName.split('/')[0]; // e.g. koalalab-inc
-    core.saveState('boltUser', boltUser)
+
 
     core.startGroup('create-bolt-user')
     core.info('Creating bolt user...')
@@ -196,6 +204,7 @@ async function run() {
     benchmark('setup-iptables-redirection')
   } catch (error) {
     // Fail the workflow run if an error occurs
+    core.saveState('boltFailed', 'true')
     core.setFailed(error.message)
   }
 }

src/summary.js

@@ -3,11 +3,7 @@ const { exec } = require('@actions/exec')
 const fs = require('fs')
 const YAML = require('yaml')
 
-async function generateTestResults(boltUser) {
-  const filePath = 'output.log'
-  await exec(`sudo cp /home/${boltUser}/${filePath} output.log`)
-  await exec(`sudo chown -R runner:docker ${filePath}`)
-
+async function generateTestResults(filePath) {
   try {
     // Read the entire file synchronously and split it into an array of lines
     const fileContent = fs.readFileSync(filePath, 'utf-8')
@@ -63,12 +59,26 @@ function resultToRow(result) {
 }
 
 async function generateSummary() {
+  const outputFile = core.getState('outputFile')
+  const homeDir = core.getState('homeDir')
   const boltUser = core.getState('boltUser')
   const mode = core.getInput('mode')
   const allowHTTP = core.getInput('allow_http')
   const defaultPolicy = core.getInput('default_policy')
   const egressRulesYAML = core.getInput('egress_rules')
   const trustedGithubAccountsYAML = core.getInput('trusted_github_accounts')
+
+  if (!outputFile || !boltUser || !homeDir) {
+    core.info(`Invalid Bold run. Missing required state variables`)
+    return
+  }
+  if (!fs.existsSync(`${homeDir}/${outputFile}`)) {
+    core.info(`Bolt output file not found`)
+    return
+  }
+
+  await exec(`sudo cp ${homeDir}/${outputFile} ${outputFile}`)
+
   // Verify that egress_rules_yaml is valid YAML
   let egressRules
   let trustedGithubAccounts
@@ -79,7 +89,7 @@ async function generateSummary() {
     core.info(`Invalid YAML: ${error.message}`)
   }
 
-  const results = await generateTestResults(boltUser)
+  const results = await generateTestResults(outputFile)
 
   const uniqueResults = getUniqueBy(results, ['destination', 'scheme'])
   // const uniqueResultRows = uniqueResults.map(resultToRow)