add set-workspace command

* Update dotnet install script (#155)

* root search pattern for hashfiles and allow forward slash on windows (#151)

docs/contribute.md

@@ -1,12 +1,12 @@
-# Contribution guide for developers
+# Contribute (Dev)
 
-## Required Dev Dependencies
+## Dev Dependencies
 
 ![Win](res/win_sm.png) Git for Windows [Install Here](https://git-scm.com/downloads) (needed for dev sh script)
 
-## To Build, Test, Layout 
+## Build, Test, Layout 
 
-Navigate to the `src` directory and run the following command:
+From src:
 
 ![Win](res/win_sm.png) `dev {command}`  
 
@@ -14,12 +14,13 @@ Navigate to the `src` directory and run the following command:
   
 **Commands:**  
 
-* `layout` (`l`):  Run first time to create a full agent layout in `{root}/_layout`
+`layout` (`l`):  Run first time to create a full agent layout in {root}/_layout  
-* `build` (`b`):   Build everything and update agent layout folder
-* `test` (`t`):    Build agent binaries and run unit tests
 
-Sample developer flow:
+`build` (`b`):   build everything and update agent layout folder  
 
+`test` (`t`):    build agent binaries and run unit tests  
+
+Normal dev flow:
 ```bash
 git clone https://github.com/actions/runner
 cd ./src
@@ -36,5 +37,5 @@ cd ./src
 
 ## Styling
 
-We use the .NET Foundation and CoreCLR style guidelines [located here](
+We use the dotnet foundation and CoreCLR style guidelines [located here](
 https://github.com/dotnet/corefx/blob/master/Documentation/coding-guidelines/coding-style.md)

src/Misc/externals.sh

@@ -146,9 +146,3 @@ fi
 if [[ "$PACKAGERUNTIME" == "linux-arm" ]]; then
     acquireExternalTool "$NODE_URL/v${NODE12_VERSION}/node-v${NODE12_VERSION}-linux-armv7l.tar.gz" node12 fix_nested_dir
 fi
-
-if [[ "$PACKAGERUNTIME" == "linux-arm64" ]]; then
-    acquireExternalTool "$NODE_URL/v${NODE12_VERSION}/node-v${NODE12_VERSION}-linux-arm64.tar.gz" node12 fix_nested_dir
-
-    ## TODO: alpine arm64 version node.js
-fi

src/Runner.Common/ExtensionManager.cs

@@ -42,6 +42,7 @@ namespace GitHub.Runner.Common
                 // Action command extensions.
                 case "GitHub.Runner.Worker.IActionCommandExtension":
                     Add<T>(extensions, "GitHub.Runner.Worker.InternalPluginSetRepoPathCommandExtension, Runner.Worker");
+                    Add<T>(extensions, "GitHub.Runner.Worker.SetWorkspaceCommandExtension, Runner.Worker");
                     Add<T>(extensions, "GitHub.Runner.Worker.SetEnvCommandExtension, Runner.Worker");
                     Add<T>(extensions, "GitHub.Runner.Worker.SetOutputCommandExtension, Runner.Worker");
                     Add<T>(extensions, "GitHub.Runner.Worker.SaveStateCommandExtension, Runner.Worker");

src/Runner.Common/Runner.Common.csproj

@@ -3,7 +3,7 @@
   <PropertyGroup>
     <TargetFramework>netcoreapp3.0</TargetFramework>
     <OutputType>Library</OutputType>
-    <RuntimeIdentifiers>win-x64;win-x86;linux-x64;linux-arm;linux-arm64;rhel.6-x64;osx-x64</RuntimeIdentifiers>
+    <RuntimeIdentifiers>win-x64;win-x86;linux-x64;linux-arm;rhel.6-x64;osx-x64</RuntimeIdentifiers>
     <TargetLatestRuntimePatch>true</TargetLatestRuntimePatch>
     <AssetTargetFallback>portable-net45+win8</AssetTargetFallback>
     <NoWarn>NU1701;NU1603</NoWarn>
@@ -57,9 +57,6 @@
   <PropertyGroup Condition="'$([System.Runtime.InteropServices.RuntimeInformation]::IsOSPlatform($([System.Runtime.InteropServices.OSPlatform]::Linux)))' == 'true' AND '$(PackageRuntime)' == 'linux-arm'">
     <DefineConstants>OS_LINUX;ARM;TRACE</DefineConstants>
   </PropertyGroup>
-  <PropertyGroup Condition="'$([System.Runtime.InteropServices.RuntimeInformation]::IsOSPlatform($([System.Runtime.InteropServices.OSPlatform]::Linux)))' == 'true' AND '$(PackageRuntime)' == 'linux-arm64'">
-    <DefineConstants>OS_LINUX;ARM64;TRACE</DefineConstants>
-  </PropertyGroup>
   <PropertyGroup Condition="'$([System.Runtime.InteropServices.RuntimeInformation]::IsOSPlatform($([System.Runtime.InteropServices.OSPlatform]::Linux)))' == 'true' AND '$(Configuration)' == 'Debug' AND '$(PackageRuntime)' == 'linux-x64'">
     <DefineConstants>OS_LINUX;X64;DEBUG;TRACE</DefineConstants>
   </PropertyGroup>
@@ -69,7 +66,4 @@
   <PropertyGroup Condition="'$([System.Runtime.InteropServices.RuntimeInformation]::IsOSPlatform($([System.Runtime.InteropServices.OSPlatform]::Linux)))' == 'true' AND '$(Configuration)' == 'Debug' AND '$(PackageRuntime)' == 'linux-arm'">
     <DefineConstants>OS_LINUX;ARM;DEBUG;TRACE</DefineConstants>
   </PropertyGroup>
-  <PropertyGroup Condition="'$([System.Runtime.InteropServices.RuntimeInformation]::IsOSPlatform($([System.Runtime.InteropServices.OSPlatform]::Linux)))' == 'true' AND '$(Configuration)' == 'Debug' AND '$(PackageRuntime)' == 'linux-arm64'">
-    <DefineConstants>OS_LINUX;ARM64;DEBUG;TRACE</DefineConstants>
-  </PropertyGroup>
 </Project>

src/Runner.Common/Util/VarUtil.cs

@@ -54,8 +54,6 @@ namespace GitHub.Runner.Common.Util
                         return "X64";
                     case Constants.Architecture.Arm:
                         return "ARM";
-                    case Constants.Architecture.Arm64:
-                        return "ARM64";    
                     default:
                         throw new NotSupportedException(); // Should never reach here.
                 }

src/Runner.Listener/Runner.Listener.csproj

@@ -3,7 +3,7 @@
   <PropertyGroup>
     <TargetFramework>netcoreapp3.0</TargetFramework>
     <OutputType>Exe</OutputType>
-    <RuntimeIdentifiers>win-x64;win-x86;linux-x64;linux-arm;linux-arm64;rhel.6-x64;osx-x64</RuntimeIdentifiers>
+    <RuntimeIdentifiers>win-x64;win-x86;linux-x64;linux-arm;rhel.6-x64;osx-x64</RuntimeIdentifiers>
     <TargetLatestRuntimePatch>true</TargetLatestRuntimePatch>
     <AssetTargetFallback>portable-net45+win8</AssetTargetFallback>
     <NoWarn>NU1701;NU1603</NoWarn>
@@ -60,9 +60,6 @@
   <PropertyGroup Condition="'$([System.Runtime.InteropServices.RuntimeInformation]::IsOSPlatform($([System.Runtime.InteropServices.OSPlatform]::Linux)))' == 'true' AND '$(PackageRuntime)' == 'linux-arm'">
     <DefineConstants>OS_LINUX;ARM;TRACE</DefineConstants>
   </PropertyGroup>
-  <PropertyGroup Condition="'$([System.Runtime.InteropServices.RuntimeInformation]::IsOSPlatform($([System.Runtime.InteropServices.OSPlatform]::Linux)))' == 'true' AND '$(PackageRuntime)' == 'linux-arm64'">
-    <DefineConstants>OS_LINUX;ARM64;TRACE</DefineConstants>
-  </PropertyGroup>
   <PropertyGroup Condition="'$([System.Runtime.InteropServices.RuntimeInformation]::IsOSPlatform($([System.Runtime.InteropServices.OSPlatform]::Linux)))' == 'true' AND '$(Configuration)' == 'Debug' AND '$(PackageRuntime)' == 'linux-x64'">
     <DefineConstants>OS_LINUX;X64;DEBUG;TRACE</DefineConstants>
   </PropertyGroup>
@@ -72,7 +69,4 @@
   <PropertyGroup Condition="'$([System.Runtime.InteropServices.RuntimeInformation]::IsOSPlatform($([System.Runtime.InteropServices.OSPlatform]::Linux)))' == 'true' AND '$(Configuration)' == 'Debug' AND '$(PackageRuntime)' == 'linux-arm'">
     <DefineConstants>OS_LINUX;ARM;DEBUG;TRACE</DefineConstants>
   </PropertyGroup>
-  <PropertyGroup Condition="'$([System.Runtime.InteropServices.RuntimeInformation]::IsOSPlatform($([System.Runtime.InteropServices.OSPlatform]::Linux)))' == 'true' AND '$(Configuration)' == 'Debug' AND '$(PackageRuntime)' == 'linux-arm64'">
-    <DefineConstants>OS_LINUX;ARM64;DEBUG;TRACE</DefineConstants>
-  </PropertyGroup>
 </Project>

src/Runner.PluginHost/Runner.PluginHost.csproj

@@ -3,7 +3,7 @@
   <PropertyGroup>
     <TargetFramework>netcoreapp3.0</TargetFramework>
     <OutputType>Exe</OutputType>
-    <RuntimeIdentifiers>win-x64;win-x86;linux-x64;linux-arm;linux-arm64;rhel.6-x64;osx-x64</RuntimeIdentifiers>
+    <RuntimeIdentifiers>win-x64;win-x86;linux-x64;linux-arm;rhel.6-x64;osx-x64</RuntimeIdentifiers>
     <TargetLatestRuntimePatch>true</TargetLatestRuntimePatch>
     <AssetTargetFallback>portable-net45+win8</AssetTargetFallback>
     <NoWarn>NU1701;NU1603</NoWarn>
@@ -53,9 +53,6 @@
   <PropertyGroup Condition="'$([System.Runtime.InteropServices.RuntimeInformation]::IsOSPlatform($([System.Runtime.InteropServices.OSPlatform]::Linux)))' == 'true' AND '$(PackageRuntime)' == 'linux-arm'">
     <DefineConstants>OS_LINUX;ARM;TRACE</DefineConstants>
   </PropertyGroup>
-  <PropertyGroup Condition="'$([System.Runtime.InteropServices.RuntimeInformation]::IsOSPlatform($([System.Runtime.InteropServices.OSPlatform]::Linux)))' == 'true' AND '$(PackageRuntime)' == 'linux-arm64'">
-    <DefineConstants>OS_LINUX;ARM64;TRACE</DefineConstants>
-  </PropertyGroup>
   <PropertyGroup Condition="'$([System.Runtime.InteropServices.RuntimeInformation]::IsOSPlatform($([System.Runtime.InteropServices.OSPlatform]::Linux)))' == 'true' AND '$(Configuration)' == 'Debug' AND '$(PackageRuntime)' == 'linux-x64'">
     <DefineConstants>OS_LINUX;X64;DEBUG;TRACE</DefineConstants>
   </PropertyGroup>
@@ -65,7 +62,4 @@
   <PropertyGroup Condition="'$([System.Runtime.InteropServices.RuntimeInformation]::IsOSPlatform($([System.Runtime.InteropServices.OSPlatform]::Linux)))' == 'true' AND '$(Configuration)' == 'Debug' AND '$(PackageRuntime)' == 'linux-arm'">
     <DefineConstants>OS_LINUX;ARM;DEBUG;TRACE</DefineConstants>
   </PropertyGroup>
-  <PropertyGroup Condition="'$([System.Runtime.InteropServices.RuntimeInformation]::IsOSPlatform($([System.Runtime.InteropServices.OSPlatform]::Linux)))' == 'true' AND '$(Configuration)' == 'Debug' AND '$(PackageRuntime)' == 'linux-arm64'">
-    <DefineConstants>OS_LINUX;ARM64;DEBUG;TRACE</DefineConstants>
-  </PropertyGroup>
 </Project>

src/Runner.Plugins/Runner.Plugins.csproj

@@ -3,7 +3,7 @@
   <PropertyGroup>
     <TargetFramework>netcoreapp3.0</TargetFramework>
     <OutputType>Library</OutputType>
-    <RuntimeIdentifiers>win-x64;win-x86;linux-x64;linux-arm;linux-arm64;rhel.6-x64;osx-x64</RuntimeIdentifiers>
+    <RuntimeIdentifiers>win-x64;win-x86;linux-x64;linux-arm;rhel.6-x64;osx-x64</RuntimeIdentifiers>
     <TargetLatestRuntimePatch>true</TargetLatestRuntimePatch>
     <AssetTargetFallback>portable-net45+win8</AssetTargetFallback>
     <NoWarn>NU1701;NU1603</NoWarn>
@@ -49,9 +49,6 @@
   <PropertyGroup Condition="'$([System.Runtime.InteropServices.RuntimeInformation]::IsOSPlatform($([System.Runtime.InteropServices.OSPlatform]::Linux)))' == 'true' AND '$(PackageRuntime)' == 'linux-arm'">
     <DefineConstants>OS_LINUX;ARM;TRACE</DefineConstants>
   </PropertyGroup>
-  <PropertyGroup Condition="'$([System.Runtime.InteropServices.RuntimeInformation]::IsOSPlatform($([System.Runtime.InteropServices.OSPlatform]::Linux)))' == 'true' AND '$(PackageRuntime)' == 'linux-arm64'">
-    <DefineConstants>OS_LINUX;ARM64;TRACE</DefineConstants>
-  </PropertyGroup>
   <PropertyGroup Condition="'$([System.Runtime.InteropServices.RuntimeInformation]::IsOSPlatform($([System.Runtime.InteropServices.OSPlatform]::Linux)))' == 'true' AND '$(Configuration)' == 'Debug' AND '$(PackageRuntime)' == 'linux-x64'">
     <DefineConstants>OS_LINUX;X64;DEBUG;TRACE</DefineConstants>
   </PropertyGroup>
@@ -61,7 +58,4 @@
   <PropertyGroup Condition="'$([System.Runtime.InteropServices.RuntimeInformation]::IsOSPlatform($([System.Runtime.InteropServices.OSPlatform]::Linux)))' == 'true' AND '$(Configuration)' == 'Debug' AND '$(PackageRuntime)' == 'linux-arm'">
     <DefineConstants>OS_LINUX;ARM;DEBUG;TRACE</DefineConstants>
   </PropertyGroup>
-  <PropertyGroup Condition="'$([System.Runtime.InteropServices.RuntimeInformation]::IsOSPlatform($([System.Runtime.InteropServices.OSPlatform]::Linux)))' == 'true' AND '$(Configuration)' == 'Debug' AND '$(PackageRuntime)' == 'linux-arm64'">
-    <DefineConstants>OS_LINUX;ARM64;DEBUG;TRACE</DefineConstants>
-  </PropertyGroup>
 </Project>

src/Runner.Sdk/Runner.Sdk.csproj

@@ -3,7 +3,7 @@
   <PropertyGroup>
     <TargetFramework>netcoreapp3.0</TargetFramework>
     <OutputType>Library</OutputType>
-    <RuntimeIdentifiers>win-x64;win-x86;linux-x64;linux-arm;linux-arm64;rhel.6-x64;osx-x64</RuntimeIdentifiers>
+    <RuntimeIdentifiers>win-x64;win-x86;linux-x64;linux-arm;rhel.6-x64;osx-x64</RuntimeIdentifiers>
     <TargetLatestRuntimePatch>true</TargetLatestRuntimePatch>
     <AssetTargetFallback>portable-net45+win8</AssetTargetFallback>
     <NoWarn>NU1701;NU1603</NoWarn>
@@ -54,9 +54,6 @@
   <PropertyGroup Condition="'$([System.Runtime.InteropServices.RuntimeInformation]::IsOSPlatform($([System.Runtime.InteropServices.OSPlatform]::Linux)))' == 'true' AND '$(PackageRuntime)' == 'linux-arm'">
     <DefineConstants>OS_LINUX;ARM;TRACE</DefineConstants>
   </PropertyGroup>
-  <PropertyGroup Condition="'$([System.Runtime.InteropServices.RuntimeInformation]::IsOSPlatform($([System.Runtime.InteropServices.OSPlatform]::Linux)))' == 'true' AND '$(PackageRuntime)' == 'linux-arm64'">
-    <DefineConstants>OS_LINUX;ARM64;TRACE</DefineConstants>
-  </PropertyGroup>
   <PropertyGroup Condition="'$([System.Runtime.InteropServices.RuntimeInformation]::IsOSPlatform($([System.Runtime.InteropServices.OSPlatform]::Linux)))' == 'true' AND '$(Configuration)' == 'Debug' AND '$(PackageRuntime)' == 'linux-x64'">
     <DefineConstants>OS_LINUX;X64;DEBUG;TRACE</DefineConstants>
   </PropertyGroup>
@@ -66,7 +63,4 @@
   <PropertyGroup Condition="'$([System.Runtime.InteropServices.RuntimeInformation]::IsOSPlatform($([System.Runtime.InteropServices.OSPlatform]::Linux)))' == 'true' AND '$(Configuration)' == 'Debug' AND '$(PackageRuntime)' == 'linux-arm'">
     <DefineConstants>OS_LINUX;ARM;DEBUG;TRACE</DefineConstants>
   </PropertyGroup>
-  <PropertyGroup Condition="'$([System.Runtime.InteropServices.RuntimeInformation]::IsOSPlatform($([System.Runtime.InteropServices.OSPlatform]::Linux)))' == 'true' AND '$(Configuration)' == 'Debug' AND '$(PackageRuntime)' == 'linux-arm64'">
-    <DefineConstants>OS_LINUX;ARM64;DEBUG;TRACE</DefineConstants>
-  </PropertyGroup>
 </Project>

src/Runner.Worker/ActionCommandManager.cs

@@ -184,6 +184,40 @@ namespace GitHub.Runner.Worker
         }
     }
 
+    public sealed class SetWorkspaceCommandExtension : RunnerService, IActionCommandExtension
+    {
+        public string Command => "set-workspace";
+
+        public Type ExtensionType => typeof(IActionCommandExtension);
+
+        public void ProcessCommand(IExecutionContext context, string line, ActionCommand command)
+        {
+            string path = command.Data;
+            if (string.IsNullOrWhiteSpace(path))
+            {
+                throw new Exception("Path not specified");
+            }
+
+            if (!Path.IsPathRooted(path))
+            {
+                throw new Exception("Expected a rooted path");
+            }
+
+            string runnerWorkspace = context.GetRunnerContext("workspace");
+            ArgUtil.Directory(runnerWorkspace, nameof(runnerWorkspace));
+
+            // Must be under runner workspace
+            path = Path.GetFullPath(path); // remove relative pathing and normalize slashes
+            if (!path.StartsWith(runnerWorkspace + Path.DirectorySeparatorChar, IOUtil.FilePathStringComparison))
+            {
+                throw new Exception($"Expected path to be under {runnerWorkspace}");
+            }
+
+            Trace.Info($"Setting GitHub workspace to '{path}'");
+            context.SetGitHubContext("workspace", path);
+        }
+    }
+
     public sealed class SetEnvCommandExtension : RunnerService, IActionCommandExtension
     {
         public string Command => "set-env";
@@ -541,6 +575,7 @@ namespace GitHub.Runner.Worker
                     break;
                 default:
                     throw new Exception($"Invalid echo command value. Possible values can be: 'on', 'off'. Current value is: '{command.Data}'.");
+                    break;
             }
         }
     }

src/Runner.Worker/ActionRunner.cs

@@ -94,21 +94,7 @@ namespace GitHub.Runner.Worker
                 {
                     postDisplayName = $"Post {this.DisplayName}";
                 }
-
+                ExecutionContext.RegisterPostJobAction(postDisplayName, handlerData.CleanupCondition, Action);
-                var repositoryReference = Action.Reference as RepositoryPathReference;
-                var pathString = string.IsNullOrEmpty(repositoryReference.Path) ? string.Empty : $"/{repositoryReference.Path}";
-                var repoString = string.IsNullOrEmpty(repositoryReference.Ref) ? $"{repositoryReference.Name}{pathString}" :
-                    $"{repositoryReference.Name}{pathString}@{repositoryReference.Ref}";
-
-                ExecutionContext.Debug($"Register post job cleanup for action: {repoString}");
-
-                var actionRunner = HostContext.CreateService<IActionRunner>();
-                actionRunner.Action = Action;
-                actionRunner.Stage = ActionRunStage.Post;
-                actionRunner.Condition = handlerData.CleanupCondition;
-                actionRunner.DisplayName = postDisplayName;
-
-                ExecutionContext.RegisterPostJobStep($"{actionRunner.Action.Name}_post", actionRunner);
             }
 
             IStepHost stepHost = HostContext.CreateService<IDefaultStepHost>();

src/Runner.Worker/Container/DockerCommandManager.cs

@@ -276,9 +276,7 @@ namespace GitHub.Runner.Worker.Container
             return await ExecuteDockerCommandAsync(context, "exec", $"{options} {containerId} {command}", context.CancellationToken);
         }
 
-#pragma warning disable CS1998 // Async method lacks 'await' operators and will run synchronously (method has async logic on only certain platforms)
         public async Task<int> DockerExec(IExecutionContext context, string containerId, string options, string command, List<string> output)
-#pragma warning restore CS1998 // Async method lacks 'await' operators and will run synchronously
         {
             ArgUtil.NotNull(output, nameof(output));
 
@@ -339,9 +337,7 @@ namespace GitHub.Runner.Worker.Container
             return ExecuteDockerCommandAsync(context, command, options, null, cancellationToken);
         }
 
-#pragma warning disable CS1998 // Async method lacks 'await' operators and will run synchronously (method has async logic on only certain platforms)
         private async Task<int> ExecuteDockerCommandAsync(IExecutionContext context, string command, string options, IDictionary<string, string> environment, EventHandler<ProcessDataReceivedEventArgs> stdoutDataReceived, EventHandler<ProcessDataReceivedEventArgs> stderrDataReceived, CancellationToken cancellationToken = default(CancellationToken))
-#pragma warning restore CS1998 // Async method lacks 'await' operators and will run synchronously
         {
             string arg = $"{command} {options}".Trim();
             context.Command($"{DockerPath} {arg}");
@@ -366,9 +362,7 @@ namespace GitHub.Runner.Worker.Container
 #endif
         }
 
-#pragma warning disable CS1998 // Async method lacks 'await' operators and will run synchronously (method has async logic on only certain platforms)
         private async Task<int> ExecuteDockerCommandAsync(IExecutionContext context, string command, string options, string workingDirectory, CancellationToken cancellationToken = default(CancellationToken))
-#pragma warning restore CS1998 // Async method lacks 'await' operators and will run synchronously
         {
             string arg = $"{command} {options}".Trim();
             context.Command($"{DockerPath} {arg}");

src/Runner.Worker/ContainerOperationProvider.cs

@@ -11,7 +11,6 @@ using GitHub.Runner.Common;
 using GitHub.Runner.Sdk;
 using GitHub.DistributedTask.Pipelines.ContextData;
 using Microsoft.Win32;
-using GitHub.DistributedTask.Pipelines.ObjectTemplating;
 
 namespace GitHub.Runner.Worker
 {
@@ -39,14 +38,6 @@ namespace GitHub.Runner.Worker
             List<ContainerInfo> containers = data as List<ContainerInfo>;
             ArgUtil.NotNull(containers, nameof(containers));
 
-            var postJobStep = new JobExtensionRunner(runAsync: this.StopContainersAsync,
-                                                condition: $"{PipelineTemplateConstants.Always}()",
-                                                displayName: "Stop containers",
-                                                data: data);
-            
-            executionContext.Debug($"Register post job cleanup for stoping/deleting containers.");
-            executionContext.RegisterPostJobStep(nameof(StopContainersAsync), postJobStep);
-
             // Check whether we are inside a container.
             // Our container feature requires to map working directory from host to the container.
             // If we are already inside a container, we will not able to find out the real working direcotry path on the host.

src/Runner.Worker/DiagnosticLogManager.cs

@@ -31,9 +31,7 @@ namespace GitHub.Runner.Worker
     public sealed class DiagnosticLogManager : RunnerService, IDiagnosticLogManager
     {
         private static string DateTimeFormat = "yyyyMMdd-HHmmss";
-#pragma warning disable CS1998 // Async method lacks 'await' operators and will run synchronously (method has async logic on only certain platforms)
         public async Task UploadDiagnosticLogsAsync(IExecutionContext executionContext,
-#pragma warning restore CS1998 // Async method lacks 'await' operators and will run synchronously
                                                 IExecutionContext parentContext, 
                                                 Pipelines.AgentJobRequestMessage message,
                                                 DateTime jobStartTimeUtc)

src/Runner.Worker/ExecutionContext.cs

@@ -78,6 +78,7 @@ namespace GitHub.Runner.Worker
         void Start(string currentOperation = null);
         TaskResult Complete(TaskResult? result = null, string currentOperation = null, string resultCode = null);
         void SetEnvContext(string name, string value);
+        string GetRunnerContext(string name);
         void SetRunnerContext(string name, string value);
         string GetGitHubContext(string name);
         void SetGitHubContext(string name, string value);
@@ -98,7 +99,7 @@ namespace GitHub.Runner.Worker
 
         // others
         void ForceTaskComplete();
-        void RegisterPostJobStep(string refName, IStep step);
+        void RegisterPostJobAction(string displayName, string condition, Pipelines.ActionStep action);
     }
 
     public sealed class ExecutionContext : RunnerService, IExecutionContext
@@ -240,10 +241,27 @@ namespace GitHub.Runner.Worker
             });
         }
 
-        public void RegisterPostJobStep(string refName, IStep step)
+        public void RegisterPostJobAction(string displayName, string condition, Pipelines.ActionStep action)
         {
-            step.ExecutionContext = Root.CreatePostChild(step.DisplayName, refName, IntraActionState);
+            if (action.Reference.Type != ActionSourceType.Repository)
-            Root.PostJobSteps.Push(step);
+            {
+                throw new NotSupportedException("Only action that has `action.yml` can define post job execution.");
+            }
+
+            var repositoryReference = action.Reference as RepositoryPathReference;
+            var pathString = string.IsNullOrEmpty(repositoryReference.Path) ? string.Empty : $"/{repositoryReference.Path}";
+            var repoString = string.IsNullOrEmpty(repositoryReference.Ref) ? $"{repositoryReference.Name}{pathString}" :
+                $"{repositoryReference.Name}{pathString}@{repositoryReference.Ref}";
+
+            this.Debug($"Register post job cleanup for action: {repoString}");
+
+            var actionRunner = HostContext.CreateService<IActionRunner>();
+            actionRunner.Action = action;
+            actionRunner.Stage = ActionRunStage.Post;
+            actionRunner.Condition = condition;
+            actionRunner.DisplayName = displayName;
+            actionRunner.ExecutionContext = Root.CreatePostChild(displayName, $"{actionRunner.Action.Name}_post", IntraActionState);
+            Root.PostJobSteps.Push(actionRunner);
         }
 
         public IExecutionContext CreateChild(Guid recordId, string displayName, string refName, string scopeName, string contextName, Dictionary<string, string> intraActionState = null, int? recordOrder = null)
@@ -348,6 +366,13 @@ namespace GitHub.Runner.Worker
             return Result.Value;
         }
 
+        public string GetRunnerContext(string name)
+        {
+            ArgUtil.NotNullOrEmpty(name, nameof(name));
+            var runnerContext = ExpressionValues["runner"] as RunnerContext;
+            return runnerContext[name].AssertString($"runner[{name}]");
+        }
+
         public void SetRunnerContext(string name, string value)
         {
             ArgUtil.NotNullOrEmpty(name, nameof(name));

src/Runner.Worker/Handlers/ContainerActionHandler.cs

@@ -22,9 +22,7 @@ namespace GitHub.Runner.Worker.Handlers
     {
         public ContainerActionExecutionData Data { get; set; }
 
-#pragma warning disable CS1998 // Async method lacks 'await' operators and will run synchronously (method has async logic on only certain platforms)
         public async Task RunAsync(ActionRunStage stage)
-#pragma warning restore CS1998 // Async method lacks 'await' operators and will run synchronously
         {
             // Validate args.
             Trace.Entering();

src/Runner.Worker/JobExtension.cs

@@ -110,7 +110,9 @@ namespace GitHub.Runner.Worker
                         }
                     }
 
-                    // Build up 2 lists of steps, pre-job, job
+                    // Build up 3 lists of steps, pre-job, job, post-job
+                    var postJobStepsBuilder = new Stack<IStep>();
+
                     // Download actions not already in the cache
                     Trace.Info("Downloading actions");
                     var actionManager = HostContext.GetService<IActionManager>();
@@ -132,6 +134,10 @@ namespace GitHub.Runner.Worker
                                                                           condition: $"{PipelineTemplateConstants.Success}()",
                                                                           displayName: "Initialize containers",
                                                                           data: (object)containers));
+                        postJobStepsBuilder.Push(new JobExtensionRunner(runAsync: containerProvider.StopContainersAsync,
+                                                                        condition: $"{PipelineTemplateConstants.Always}()",
+                                                                        displayName: "Stop containers",
+                                                                        data: (object)containers));
                     }
 
                     // Add action steps
@@ -181,9 +187,33 @@ namespace GitHub.Runner.Worker
                         }
                     }
 
+                    // Add post-job steps
+                    Trace.Info("Adding post-job steps");
+                    while (postJobStepsBuilder.Count > 0)
+                    {
+                        postJobSteps.Add(postJobStepsBuilder.Pop());
+                    }
+
+                    // Create execution context for post-job steps
+                    foreach (var step in postJobSteps)
+                    {
+                        if (step is JobExtensionRunner)
+                        {
+                            JobExtensionRunner extensionStep = step as JobExtensionRunner;
+                            ArgUtil.NotNull(extensionStep, extensionStep.DisplayName);
+                            Guid stepId = Guid.NewGuid();
+                            extensionStep.ExecutionContext = jobContext.CreateChild(stepId, extensionStep.DisplayName, stepId.ToString("N"), null, null);
+                        }
+                    }
+
                     List<IStep> steps = new List<IStep>();
                     steps.AddRange(preJobSteps);
                     steps.AddRange(jobSteps);
+                    steps.AddRange(postJobSteps);
+
+                    // Start agent log plugin host process
+                    // var logPlugin = HostContext.GetService<IAgentLogPlugin>();
+                    // await logPlugin.StartAsync(context, steps, jobContext.CancellationToken);
 
                     // Prepare for orphan process cleanup
                     _processCleanup = jobContext.Variables.GetBoolean("process.clean") ?? true;

src/Runner.Worker/Runner.Worker.csproj

@@ -3,7 +3,7 @@
   <PropertyGroup>
     <TargetFramework>netcoreapp3.0</TargetFramework>
     <OutputType>Exe</OutputType>
-    <RuntimeIdentifiers>win-x64;win-x86;linux-x64;linux-arm;linux-arm64;rhel.6-x64;osx-x64</RuntimeIdentifiers>
+    <RuntimeIdentifiers>win-x64;win-x86;linux-x64;linux-arm;rhel.6-x64;osx-x64</RuntimeIdentifiers>
     <TargetLatestRuntimePatch>true</TargetLatestRuntimePatch>
     <AssetTargetFallback>portable-net45+win8</AssetTargetFallback>
     <NoWarn>NU1701;NU1603</NoWarn>
@@ -64,9 +64,6 @@
   <PropertyGroup Condition="'$([System.Runtime.InteropServices.RuntimeInformation]::IsOSPlatform($([System.Runtime.InteropServices.OSPlatform]::Linux)))' == 'true' AND '$(PackageRuntime)' == 'linux-arm'">
     <DefineConstants>OS_LINUX;ARM;TRACE</DefineConstants>
   </PropertyGroup>
-  <PropertyGroup Condition="'$([System.Runtime.InteropServices.RuntimeInformation]::IsOSPlatform($([System.Runtime.InteropServices.OSPlatform]::Linux)))' == 'true' AND '$(PackageRuntime)' == 'linux-arm64'">
-    <DefineConstants>OS_LINUX;ARM64;TRACE</DefineConstants>
-  </PropertyGroup>
   <PropertyGroup Condition="'$([System.Runtime.InteropServices.RuntimeInformation]::IsOSPlatform($([System.Runtime.InteropServices.OSPlatform]::Linux)))' == 'true' AND '$(Configuration)' == 'Debug' AND '$(PackageRuntime)' == 'linux-x64'">
     <DefineConstants>OS_LINUX;X64;DEBUG;TRACE</DefineConstants>
   </PropertyGroup>
@@ -76,7 +73,4 @@
   <PropertyGroup Condition="'$([System.Runtime.InteropServices.RuntimeInformation]::IsOSPlatform($([System.Runtime.InteropServices.OSPlatform]::Linux)))' == 'true' AND '$(Configuration)' == 'Debug' AND '$(PackageRuntime)' == 'linux-arm'">
     <DefineConstants>OS_LINUX;ARM;DEBUG;TRACE</DefineConstants>
   </PropertyGroup>
-  <PropertyGroup Condition="'$([System.Runtime.InteropServices.RuntimeInformation]::IsOSPlatform($([System.Runtime.InteropServices.OSPlatform]::Linux)))' == 'true' AND '$(Configuration)' == 'Debug' AND '$(PackageRuntime)' == 'linux-arm64'">
-    <DefineConstants>OS_LINUX;ARM64;DEBUG;TRACE</DefineConstants>
-  </PropertyGroup>
 </Project>

src/Sdk/Sdk.csproj

@@ -3,7 +3,7 @@
     <PropertyGroup>
         <TargetFramework>netcoreapp3.0</TargetFramework>
         <OutputType>Library</OutputType>
-        <RuntimeIdentifiers>win-x64;win-x86;linux-x64;linux-arm;linux-arm64;rhel.6-x64;osx-x64</RuntimeIdentifiers>
+        <RuntimeIdentifiers>win-x64;win-x86;linux-x64;linux-arm;rhel.6-x64;osx-x64</RuntimeIdentifiers>
         <TargetLatestRuntimePatch>true</TargetLatestRuntimePatch>
         <AssetTargetFallback>portable-net45+win8</AssetTargetFallback>
         <NoWarn>NU1701;NU1603</NoWarn>

src/Test/L0/CommandLineParserL0.cs

@@ -47,8 +47,8 @@ namespace GitHub.Runner.Common.Tests
                 });
 
                 // Assert.
-                Assert.Equal("***", hc.SecretMasker.MaskSecrets("secret value 1"));
+                Assert.Equal(hc.SecretMasker.MaskSecrets("secret value 1"), "***");
-                Assert.Equal("***", hc.SecretMasker.MaskSecrets("secret value 2"));
+                Assert.Equal(hc.SecretMasker.MaskSecrets("secret value 2"), "***");
             }
         }
 
@@ -90,9 +90,9 @@ namespace GitHub.Runner.Common.Tests
                 trace.Info("Args: {0}", clp.Args.Count);
                 Assert.True(clp.Args.Count == 2);
                 Assert.True(clp.Args.ContainsKey("arg1"));
-                Assert.Equal("arg1val", clp.Args["arg1"]);
+                Assert.Equal(clp.Args["arg1"], "arg1val");
                 Assert.True(clp.Args.ContainsKey("arg2"));
-                Assert.Equal("arg2val", clp.Args["arg2"]);
+                Assert.Equal(clp.Args["arg2"], "arg2val");
             }
         }
 
@@ -113,8 +113,8 @@ namespace GitHub.Runner.Common.Tests
 
                 trace.Info("Args: {0}", clp.Flags.Count);
                 Assert.True(clp.Flags.Count == 2);
-                Assert.Contains("flag1", clp.Flags);
+                Assert.True(clp.Flags.Contains("flag1"));
-                Assert.Contains("flag2", clp.Flags);
+                Assert.True(clp.Flags.Contains("flag2"));
             }
         }
 

src/Test/L0/ConstantGenerationL0.cs

@@ -18,7 +18,6 @@ namespace GitHub.Runner.Common.Tests
                 "win-x86",
                 "linux-x64",
                 "linux-arm",
-                "linux-arm64",
                 "rhel.6-x64",
                 "osx-x64"
             };

src/Test/L0/Container/DockerUtilL0.cs

@@ -36,10 +36,10 @@ namespace GitHub.Runner.Common.Tests.Worker.Container
 
             // Assert
             Assert.NotNull(result0);
-            Assert.Equal(0, result0.Count);
+            Assert.Equal(result0.Count, 0);
 
             Assert.NotNull(result1);
-            Assert.Equal(1, result1.Count);
+            Assert.Equal(result1.Count, 1);
             var result1Port80Mapping = result1.Find(pm =>
                 string.Equals(pm.ContainerPort, "80") &&
                 string.Equals(pm.HostPort, "32881") &&
@@ -48,10 +48,10 @@ namespace GitHub.Runner.Common.Tests.Worker.Container
             Assert.NotNull(result1Port80Mapping);
 
             Assert.NotNull(result1Empty);
-            Assert.Equal(0, result1Empty.Count);
+            Assert.Equal(result1Empty.Count, 0);
 
             Assert.NotNull(result2);
-            Assert.Equal(2, result2.Count);
+            Assert.Equal(result2.Count, 2);
             var result2Port80Mapping = result2.Find(pm =>
                 string.Equals(pm.ContainerPort, "80") &&
                 string.Equals(pm.HostPort, "32881") &&

src/Test/L0/HostContextL0.cs

@@ -107,7 +107,7 @@ namespace GitHub.Runner.Common.Tests
             }
         }
 
-        private void Setup([CallerMemberName] string testName = "")
+        public void Setup([CallerMemberName] string testName = "")
         {
             _tokenSource = new CancellationTokenSource();
             _hc = new HostContext(
@@ -115,7 +115,7 @@ namespace GitHub.Runner.Common.Tests
                 logFile: Path.Combine(Path.GetDirectoryName(Assembly.GetEntryAssembly().Location), $"trace_{nameof(HostContextL0)}_{testName}.log"));
         }
 
-        private void Teardown()
+        public void Teardown()
         {
             _hc?.Dispose();
             _tokenSource?.Dispose();

src/Test/L0/Listener/CommandSettingsL0.cs

@@ -52,7 +52,7 @@ namespace GitHub.Runner.Common.Tests
                     // Assert.
                     Assert.Equal("some agent", actual);
                     Assert.Equal(string.Empty, Environment.GetEnvironmentVariable("ACTIONS_RUNNER_INPUT_AGENT") ?? string.Empty); // Should remove.
-                    Assert.Equal("some agent", hc.SecretMasker.MaskSecrets("some agent"));
+                    Assert.Equal(hc.SecretMasker.MaskSecrets("some agent"), "some agent");
                 }
                 finally
                 {
@@ -80,7 +80,7 @@ namespace GitHub.Runner.Common.Tests
                     // Assert.
                     Assert.Equal("some secret token value", actual);
                     Assert.Equal(string.Empty, Environment.GetEnvironmentVariable("ACTIONS_RUNNER_INPUT_TOKEN") ?? string.Empty); // Should remove.
-                    Assert.Equal("***", hc.SecretMasker.MaskSecrets("some secret token value"));
+                    Assert.Equal(hc.SecretMasker.MaskSecrets("some secret token value"), "***");
                 }
                 finally
                 {
@@ -250,7 +250,7 @@ namespace GitHub.Runner.Common.Tests
                     bool actual = command.Unattended;
 
                     // Assert.
-                    Assert.True(actual);
+                    Assert.Equal(true, actual);
                     Assert.Equal(string.Empty, Environment.GetEnvironmentVariable("ACTIONS_RUNNER_INPUT_UNATTENDED") ?? string.Empty); // Should remove.
                 }
                 finally
@@ -720,7 +720,7 @@ namespace GitHub.Runner.Common.Tests
                 var command = new CommandSettings(hc, args: new string[] { "badcommand" });
 
                 // Assert.
-                Assert.Contains("badcommand", command.Validate());
+                Assert.True(command.Validate().Contains("badcommand"));
             }
         }
 
@@ -735,7 +735,7 @@ namespace GitHub.Runner.Common.Tests
                 var command = new CommandSettings(hc, args: new string[] { "--badflag" });
 
                 // Assert.
-                Assert.Contains("badflag", command.Validate());
+                Assert.True(command.Validate().Contains("badflag"));
             }
         }
 
@@ -750,7 +750,7 @@ namespace GitHub.Runner.Common.Tests
                 var command = new CommandSettings(hc, args: new string[] { "--badargname", "bad arg value" });
 
                 // Assert.
-                Assert.Contains("badargname", command.Validate());
+                Assert.True(command.Validate().Contains("badargname"));
             }
         }
 

src/Test/L0/Listener/Configuration/ConfigurationManagerL0.cs

@@ -43,6 +43,10 @@ namespace GitHub.Runner.Common.Tests.Listener.Configuration
         private string _expectedServerUrl = "https://localhost";
         private string _expectedAgentName = "expectedAgentName";
         private string _expectedPoolName = "poolName";
+        private string _expectedCollectionName = "testCollectionName";
+        private string _expectedProjectName = "testProjectName";
+        private string _expectedProjectId = "edf3f94e-d251-49df-bfce-602d6c967409";
+        private string _expectedMachineGroupName = "testMachineGroupName";
         private string _expectedAuthType = "pat";
         private string _expectedWorkFolder = "_work";
         private int _expectedPoolId = 1;

src/Test/L0/PagingLoggerL0.cs

@@ -64,7 +64,7 @@ namespace GitHub.Runner.Common.Tests.Listener
                                 string line;
                                 while ((line = freader.ReadLine()) != null)
                                 {
-                                    Assert.EndsWith(LogData, line);
+                                    Assert.True(line.EndsWith(LogData));
                                     bytesWritten += logDataSize;
                                 }
                             }

src/Test/L0/Util/VssUtilL0.cs

@@ -31,8 +31,8 @@ namespace GitHub.Runner.Common.Tests.Util
                     var connect = VssUtil.CreateConnection(new Uri("https://github.com/actions/runner"), new VssCredentials());
 
                     // Assert.
-                    Assert.Equal("10", connect.Settings.MaxRetryRequest.ToString());
+                    Assert.Equal(connect.Settings.MaxRetryRequest.ToString(), "10");
-                    Assert.Equal("360", connect.Settings.SendTimeout.TotalSeconds.ToString());
+                    Assert.Equal(connect.Settings.SendTimeout.TotalSeconds.ToString(), "360");
 
                     trace.Info("Set httpretry to 100.");
                     Environment.SetEnvironmentVariable("GITHUB_ACTIONS_RUNNER_HTTP_RETRY", "100");
@@ -42,8 +42,8 @@ namespace GitHub.Runner.Common.Tests.Util
                     connect = VssUtil.CreateConnection(new Uri("https://github.com/actions/runner"), new VssCredentials());
 
                     // Assert.
-                    Assert.Equal("10", connect.Settings.MaxRetryRequest.ToString());
+                    Assert.Equal(connect.Settings.MaxRetryRequest.ToString(), "10");
-                    Assert.Equal("1200", connect.Settings.SendTimeout.TotalSeconds.ToString());
+                    Assert.Equal(connect.Settings.SendTimeout.TotalSeconds.ToString(), "1200");
                 }
                 finally
                 {

src/Test/L0/Worker/ActionManifestManagerL0.cs

@@ -37,25 +37,25 @@ namespace GitHub.Runner.Common.Tests.Worker
 
                 //Assert
 
-                Assert.Equal("Hello World", result.Name);
+                Assert.Equal(result.Name, "Hello World");
-                Assert.Equal("Greet the world and record the time", result.Description);
+                Assert.Equal(result.Description, "Greet the world and record the time");
-                Assert.Equal(2, result.Inputs.Count);
+                Assert.Equal(result.Inputs.Count, 2);
-                Assert.Equal("greeting", result.Inputs[0].Key.AssertString("key").Value);
+                Assert.Equal(result.Inputs[0].Key.AssertString("key").Value, "greeting");
-                Assert.Equal("Hello", result.Inputs[0].Value.AssertString("value").Value);
+                Assert.Equal(result.Inputs[0].Value.AssertString("value").Value, "Hello");
-                Assert.Equal("entryPoint", result.Inputs[1].Key.AssertString("key").Value);
+                Assert.Equal(result.Inputs[1].Key.AssertString("key").Value, "entryPoint");
-                Assert.Equal("", result.Inputs[1].Value.AssertString("value").Value);
+                Assert.Equal(result.Inputs[1].Value.AssertString("value").Value, "");
 
-                Assert.Equal(ActionExecutionType.Container, result.Execution.ExecutionType);
+                Assert.Equal(result.Execution.ExecutionType, ActionExecutionType.Container);
 
                 var containerAction = result.Execution as ContainerActionExecutionData;
 
-                Assert.Equal("Dockerfile", containerAction.Image);
+                Assert.Equal(containerAction.Image, "Dockerfile");
-                Assert.Equal("main.sh", containerAction.EntryPoint);
+                Assert.Equal(containerAction.EntryPoint, "main.sh");
-                Assert.Equal("bzz", containerAction.Arguments[0].ToString());
+                Assert.Equal(containerAction.Arguments[0].ToString(), "bzz");
-                Assert.Equal("Token", containerAction.Environment[0].Key.ToString());
+                Assert.Equal(containerAction.Environment[0].Key.ToString(), "Token");
-                Assert.Equal("foo", containerAction.Environment[0].Value.ToString());
+                Assert.Equal(containerAction.Environment[0].Value.ToString(), "foo");
-                Assert.Equal("Url", containerAction.Environment[1].Key.ToString());
+                Assert.Equal(containerAction.Environment[1].Key.ToString(), "Url");
-                Assert.Equal("bar", containerAction.Environment[1].Value.ToString());
+                Assert.Equal(containerAction.Environment[1].Value.ToString(), "bar");
             }
             finally
             {
@@ -81,27 +81,27 @@ namespace GitHub.Runner.Common.Tests.Worker
 
                 //Assert
 
-                Assert.Equal("Hello World", result.Name);
+                Assert.Equal(result.Name, "Hello World");
-                Assert.Equal("Greet the world and record the time", result.Description);
+                Assert.Equal(result.Description, "Greet the world and record the time");
-                Assert.Equal(2, result.Inputs.Count);
+                Assert.Equal(result.Inputs.Count, 2);
-                Assert.Equal("greeting", result.Inputs[0].Key.AssertString("key").Value);
+                Assert.Equal(result.Inputs[0].Key.AssertString("key").Value, "greeting");
-                Assert.Equal("Hello", result.Inputs[0].Value.AssertString("value").Value);
+                Assert.Equal(result.Inputs[0].Value.AssertString("value").Value, "Hello");
-                Assert.Equal("entryPoint", result.Inputs[1].Key.AssertString("key").Value);
+                Assert.Equal(result.Inputs[1].Key.AssertString("key").Value, "entryPoint");
-                Assert.Equal("", result.Inputs[1].Value.AssertString("value").Value);
+                Assert.Equal(result.Inputs[1].Value.AssertString("value").Value, "");
 
-                Assert.Equal(ActionExecutionType.Container, result.Execution.ExecutionType);
+                Assert.Equal(result.Execution.ExecutionType, ActionExecutionType.Container);
 
                 var containerAction = result.Execution as ContainerActionExecutionData;
 
-                Assert.Equal("Dockerfile", containerAction.Image);
+                Assert.Equal(containerAction.Image, "Dockerfile");
-                Assert.Equal("main.sh", containerAction.EntryPoint);
+                Assert.Equal(containerAction.EntryPoint, "main.sh");
-                Assert.Equal("cleanup.sh", containerAction.Cleanup);
+                Assert.Equal(containerAction.Cleanup, "cleanup.sh");
-                Assert.Equal("failure()", containerAction.CleanupCondition);
+                Assert.Equal(containerAction.CleanupCondition, "failure()");
-                Assert.Equal("bzz", containerAction.Arguments[0].ToString());
+                Assert.Equal(containerAction.Arguments[0].ToString(), "bzz");
-                Assert.Equal("Token", containerAction.Environment[0].Key.ToString());
+                Assert.Equal(containerAction.Environment[0].Key.ToString(), "Token");
-                Assert.Equal("foo", containerAction.Environment[0].Value.ToString());
+                Assert.Equal(containerAction.Environment[0].Value.ToString(), "foo");
-                Assert.Equal("Url", containerAction.Environment[1].Key.ToString());
+                Assert.Equal(containerAction.Environment[1].Key.ToString(), "Url");
-                Assert.Equal("bar", containerAction.Environment[1].Value.ToString());
+                Assert.Equal(containerAction.Environment[1].Value.ToString(), "bar");
             }
             finally
             {
@@ -126,19 +126,19 @@ namespace GitHub.Runner.Common.Tests.Worker
                 var result = actionManifest.Load(_ec.Object, Path.Combine(TestUtil.GetTestDataPath(), "dockerfileaction_noargs_noenv_noentrypoint.yml"));
 
                 //Assert
-                Assert.Equal("Hello World", result.Name);
+                Assert.Equal(result.Name, "Hello World");
-                Assert.Equal("Greet the world and record the time", result.Description);
+                Assert.Equal(result.Description, "Greet the world and record the time");
-                Assert.Equal(2, result.Inputs.Count);
+                Assert.Equal(result.Inputs.Count, 2);
-                Assert.Equal("greeting", result.Inputs[0].Key.AssertString("key").Value);
+                Assert.Equal(result.Inputs[0].Key.AssertString("key").Value, "greeting");
-                Assert.Equal("Hello", result.Inputs[0].Value.AssertString("value").Value);
+                Assert.Equal(result.Inputs[0].Value.AssertString("value").Value, "Hello");
-                Assert.Equal("entryPoint", result.Inputs[1].Key.AssertString("key").Value);
+                Assert.Equal(result.Inputs[1].Key.AssertString("key").Value, "entryPoint");
-                Assert.Equal("", result.Inputs[1].Value.AssertString("value").Value);
+                Assert.Equal(result.Inputs[1].Value.AssertString("value").Value, "");
 
-                Assert.Equal(ActionExecutionType.Container, result.Execution.ExecutionType);
+                Assert.Equal(result.Execution.ExecutionType, ActionExecutionType.Container);
 
                 var containerAction = result.Execution as ContainerActionExecutionData;
 
-                Assert.Equal("Dockerfile", containerAction.Image);
+                Assert.Equal(containerAction.Image, "Dockerfile");
             }
             finally
             {
@@ -164,25 +164,25 @@ namespace GitHub.Runner.Common.Tests.Worker
 
                 //Assert
 
-                Assert.Equal("Hello World", result.Name);
+                Assert.Equal(result.Name, "Hello World");
-                Assert.Equal("Greet the world and record the time", result.Description);
+                Assert.Equal(result.Description, "Greet the world and record the time");
-                Assert.Equal(2, result.Inputs.Count);
+                Assert.Equal(result.Inputs.Count, 2);
-                Assert.Equal("greeting", result.Inputs[0].Key.AssertString("key").Value);
+                Assert.Equal(result.Inputs[0].Key.AssertString("key").Value, "greeting");
-                Assert.Equal("Hello", result.Inputs[0].Value.AssertString("value").Value);
+                Assert.Equal(result.Inputs[0].Value.AssertString("value").Value, "Hello");
-                Assert.Equal("entryPoint", result.Inputs[1].Key.AssertString("key").Value);
+                Assert.Equal(result.Inputs[1].Key.AssertString("key").Value, "entryPoint");
-                Assert.Equal("", result.Inputs[1].Value.AssertString("value").Value);
+                Assert.Equal(result.Inputs[1].Value.AssertString("value").Value, "");
 
-                Assert.Equal(ActionExecutionType.Container, result.Execution.ExecutionType);
+                Assert.Equal(result.Execution.ExecutionType, ActionExecutionType.Container);
 
                 var containerAction = result.Execution as ContainerActionExecutionData;
 
-                Assert.Equal("Dockerfile", containerAction.Image);
+                Assert.Equal(containerAction.Image, "Dockerfile");
-                Assert.Equal("main.sh", containerAction.EntryPoint);
+                Assert.Equal(containerAction.EntryPoint, "main.sh");
-                Assert.Equal("${{ inputs.greeting }}", containerAction.Arguments[0].ToString());
+                Assert.Equal(containerAction.Arguments[0].ToString(), "${{ inputs.greeting }}");
-                Assert.Equal("Token", containerAction.Environment[0].Key.ToString());
+                Assert.Equal(containerAction.Environment[0].Key.ToString(), "Token");
-                Assert.Equal("foo", containerAction.Environment[0].Value.ToString());
+                Assert.Equal(containerAction.Environment[0].Value.ToString(), "foo");
-                Assert.Equal("Url", containerAction.Environment[1].Key.ToString());
+                Assert.Equal(containerAction.Environment[1].Key.ToString(), "Url");
-                Assert.Equal("${{ inputs.entryPoint }}", containerAction.Environment[1].Value.ToString());
+                Assert.Equal(containerAction.Environment[1].Value.ToString(), "${{ inputs.entryPoint }}");
             }
             finally
             {
@@ -207,25 +207,25 @@ namespace GitHub.Runner.Common.Tests.Worker
                 var result = actionManifest.Load(_ec.Object, Path.Combine(TestUtil.GetTestDataPath(), "dockerhubaction.yml"));
 
                 //Assert
-                Assert.Equal("Hello World", result.Name);
+                Assert.Equal(result.Name, "Hello World");
-                Assert.Equal("Greet the world and record the time", result.Description);
+                Assert.Equal(result.Description, "Greet the world and record the time");
-                Assert.Equal(2, result.Inputs.Count);
+                Assert.Equal(result.Inputs.Count, 2);
-                Assert.Equal("greeting", result.Inputs[0].Key.AssertString("key").Value);
+                Assert.Equal(result.Inputs[0].Key.AssertString("key").Value, "greeting");
-                Assert.Equal("Hello", result.Inputs[0].Value.AssertString("value").Value);
+                Assert.Equal(result.Inputs[0].Value.AssertString("value").Value, "Hello");
-                Assert.Equal("entryPoint", result.Inputs[1].Key.AssertString("key").Value);
+                Assert.Equal(result.Inputs[1].Key.AssertString("key").Value, "entryPoint");
-                Assert.Equal("", result.Inputs[1].Value.AssertString("value").Value);
+                Assert.Equal(result.Inputs[1].Value.AssertString("value").Value, "");
 
-                Assert.Equal(ActionExecutionType.Container, result.Execution.ExecutionType);
+                Assert.Equal(result.Execution.ExecutionType, ActionExecutionType.Container);
 
                 var containerAction = result.Execution as ContainerActionExecutionData;
 
-                Assert.Equal("docker://ubuntu:18.04", containerAction.Image);
+                Assert.Equal(containerAction.Image, "docker://ubuntu:18.04");
-                Assert.Equal("main.sh", containerAction.EntryPoint);
+                Assert.Equal(containerAction.EntryPoint, "main.sh");
-                Assert.Equal("bzz", containerAction.Arguments[0].ToString());
+                Assert.Equal(containerAction.Arguments[0].ToString(), "bzz");
-                Assert.Equal("Token", containerAction.Environment[0].Key.ToString());
+                Assert.Equal(containerAction.Environment[0].Key.ToString(), "Token");
-                Assert.Equal("foo", containerAction.Environment[0].Value.ToString());
+                Assert.Equal(containerAction.Environment[0].Value.ToString(), "foo");
-                Assert.Equal("Url", containerAction.Environment[1].Key.ToString());
+                Assert.Equal(containerAction.Environment[1].Key.ToString(), "Url");
-                Assert.Equal("bar", containerAction.Environment[1].Value.ToString());
+                Assert.Equal(containerAction.Environment[1].Value.ToString(), "bar");
             }
             finally
             {
@@ -250,24 +250,24 @@ namespace GitHub.Runner.Common.Tests.Worker
                 var result = actionManifest.Load(_ec.Object, Path.Combine(TestUtil.GetTestDataPath(), "nodeaction.yml"));
 
                 //Assert
-                Assert.Equal("Hello World", result.Name);
+                Assert.Equal(result.Name, "Hello World");
-                Assert.Equal("Greet the world and record the time", result.Description);
+                Assert.Equal(result.Description, "Greet the world and record the time");
-                Assert.Equal(2, result.Inputs.Count);
+                Assert.Equal(result.Inputs.Count, 2);
-                Assert.Equal("greeting", result.Inputs[0].Key.AssertString("key").Value);
+                Assert.Equal(result.Inputs[0].Key.AssertString("key").Value, "greeting");
-                Assert.Equal("Hello", result.Inputs[0].Value.AssertString("value").Value);
+                Assert.Equal(result.Inputs[0].Value.AssertString("value").Value, "Hello");
-                Assert.Equal("entryPoint", result.Inputs[1].Key.AssertString("key").Value);
+                Assert.Equal(result.Inputs[1].Key.AssertString("key").Value, "entryPoint");
-                Assert.Equal("", result.Inputs[1].Value.AssertString("value").Value);
+                Assert.Equal(result.Inputs[1].Value.AssertString("value").Value, "");
-                Assert.Equal(1, result.Deprecated.Count);
+                Assert.Equal(result.Deprecated.Count, 1);
 
                 Assert.True(result.Deprecated.ContainsKey("greeting"));
                 result.Deprecated.TryGetValue("greeting", out string value);
-                Assert.Equal("This property has been deprecated", value);
+                Assert.Equal(value, "This property has been deprecated");
 
-                Assert.Equal(ActionExecutionType.NodeJS, result.Execution.ExecutionType);
+                Assert.Equal(result.Execution.ExecutionType, ActionExecutionType.NodeJS);
 
                 var nodeAction = result.Execution as NodeJSActionExecutionData;
 
-                Assert.Equal("main.js", nodeAction.Script);
+                Assert.Equal(nodeAction.Script, "main.js");
             }
             finally
             {
@@ -292,26 +292,26 @@ namespace GitHub.Runner.Common.Tests.Worker
                 var result = actionManifest.Load(_ec.Object, Path.Combine(TestUtil.GetTestDataPath(), "nodeaction_cleanup.yml"));
 
                 //Assert
-                Assert.Equal("Hello World", result.Name);
+                Assert.Equal(result.Name, "Hello World");
-                Assert.Equal("Greet the world and record the time", result.Description);
+                Assert.Equal(result.Description, "Greet the world and record the time");
-                Assert.Equal(2, result.Inputs.Count);
+                Assert.Equal(result.Inputs.Count, 2);
-                Assert.Equal("greeting", result.Inputs[0].Key.AssertString("key").Value);
+                Assert.Equal(result.Inputs[0].Key.AssertString("key").Value, "greeting");
-                Assert.Equal("Hello", result.Inputs[0].Value.AssertString("value").Value);
+                Assert.Equal(result.Inputs[0].Value.AssertString("value").Value, "Hello");
-                Assert.Equal("entryPoint", result.Inputs[1].Key.AssertString("key").Value);
+                Assert.Equal(result.Inputs[1].Key.AssertString("key").Value, "entryPoint");
-                Assert.Equal("", result.Inputs[1].Value.AssertString("value").Value);
+                Assert.Equal(result.Inputs[1].Value.AssertString("value").Value, "");
-                Assert.Equal(1, result.Deprecated.Count);
+                Assert.Equal(result.Deprecated.Count, 1);
 
                 Assert.True(result.Deprecated.ContainsKey("greeting"));
                 result.Deprecated.TryGetValue("greeting", out string value);
-                Assert.Equal("This property has been deprecated", value);
+                Assert.Equal(value, "This property has been deprecated");
 
-                Assert.Equal(ActionExecutionType.NodeJS, result.Execution.ExecutionType);
+                Assert.Equal(result.Execution.ExecutionType, ActionExecutionType.NodeJS);
 
                 var nodeAction = result.Execution as NodeJSActionExecutionData;
 
-                Assert.Equal("main.js", nodeAction.Script);
+                Assert.Equal(nodeAction.Script, "main.js");
-                Assert.Equal("cleanup.js", nodeAction.Cleanup);
+                Assert.Equal(nodeAction.Cleanup, "cleanup.js");
-                Assert.Equal("cancelled()", nodeAction.CleanupCondition);
+                Assert.Equal(nodeAction.CleanupCondition, "cancelled()");
             }
             finally
             {
@@ -336,19 +336,19 @@ namespace GitHub.Runner.Common.Tests.Worker
                 var result = actionManifest.Load(_ec.Object, Path.Combine(TestUtil.GetTestDataPath(), "pluginaction.yml"));
 
                 //Assert
-                Assert.Equal("Hello World", result.Name);
+                Assert.Equal(result.Name, "Hello World");
-                Assert.Equal("Greet the world and record the time", result.Description);
+                Assert.Equal(result.Description, "Greet the world and record the time");
-                Assert.Equal(2, result.Inputs.Count);
+                Assert.Equal(result.Inputs.Count, 2);
-                Assert.Equal("greeting", result.Inputs[0].Key.AssertString("key").Value);
+                Assert.Equal(result.Inputs[0].Key.AssertString("key").Value, "greeting");
-                Assert.Equal("Hello", result.Inputs[0].Value.AssertString("value").Value);
+                Assert.Equal(result.Inputs[0].Value.AssertString("value").Value, "Hello");
-                Assert.Equal("entryPoint", result.Inputs[1].Key.AssertString("key").Value);
+                Assert.Equal(result.Inputs[1].Key.AssertString("key").Value, "entryPoint");
-                Assert.Equal("", result.Inputs[1].Value.AssertString("value").Value);
+                Assert.Equal(result.Inputs[1].Value.AssertString("value").Value, "");
 
-                Assert.Equal(ActionExecutionType.Plugin, result.Execution.ExecutionType);
+                Assert.Equal(result.Execution.ExecutionType, ActionExecutionType.Plugin);
 
                 var pluginAction = result.Execution as PluginActionExecutionData;
 
-                Assert.Equal("someplugin", pluginAction.Plugin);
+                Assert.Equal(pluginAction.Plugin, "someplugin");
             }
             finally
             {
@@ -383,9 +383,9 @@ namespace GitHub.Runner.Common.Tests.Worker
                 var result = actionManifest.EvaluateContainerArguments(_ec.Object, arguments, evaluateContext);
 
                 //Assert
-                Assert.Equal("hello", result[0]);
+                Assert.Equal(result[0], "hello");
-                Assert.Equal("test", result[1]);
+                Assert.Equal(result[1], "test");
-                Assert.Equal(2, result.Count);
+                Assert.Equal(result.Count, 2);
             }
             finally
             {
@@ -420,9 +420,9 @@ namespace GitHub.Runner.Common.Tests.Worker
                 var result = actionManifest.EvaluateContainerEnvironment(_ec.Object, environment, evaluateContext);
 
                 //Assert
-                Assert.Equal("hello", result["hello"]);
+                Assert.Equal(result["hello"], "hello");
-                Assert.Equal("test", result["test"]);
+                Assert.Equal(result["test"], "test");
-                Assert.Equal(2, result.Count);
+                Assert.Equal(result.Count, 2);
             }
             finally
             {
@@ -459,13 +459,13 @@ namespace GitHub.Runner.Common.Tests.Worker
                 var result = actionManifest.EvaluateDefaultInput(_ec.Object, "testInput", new StringToken(null, null, null, "defaultValue"), evaluateContext);
 
                 //Assert
-                Assert.Equal("defaultValue", result);
+                Assert.Equal(result, "defaultValue");
 
                 //Act
                 result = actionManifest.EvaluateDefaultInput(_ec.Object, "testInput", new BasicExpressionToken(null, null, null, "github.ref"), evaluateContext);
 
                 //Assert
-                Assert.Equal("refs/heads/master", result);
+                Assert.Equal(result, "refs/heads/master");
             }
             finally
             {

src/Test/L0/Worker/ActionRunnerL0.cs

@@ -31,6 +31,7 @@ namespace GitHub.Runner.Common.Tests.Worker
         private TestHostContext _hc;
         private ActionRunner _actionRunner;
         private IActionManifestManager _actionManifestManager;
+        private string _workFolder;
         private DictionaryContextData _context = new DictionaryContextData();
 
         [Fact]
@@ -74,9 +75,9 @@ namespace GitHub.Runner.Common.Tests.Worker
             }
 
             //Assert
-            Assert.Equal("test1", finialInputs["input1"]);
+            Assert.Equal(finialInputs["input1"], "test1");
-            Assert.Equal("test2", finialInputs["input2"]);
+            Assert.Equal(finialInputs["input2"], "test2");
-            Assert.Equal("github", finialInputs["input3"]);
+            Assert.Equal(finialInputs["input3"], "github");
         }
 
         [Fact]
@@ -277,6 +278,24 @@ namespace GitHub.Runner.Common.Tests.Worker
             Assert.Equal("${{ matrix.node }}", _actionRunner.DisplayName);
         }
 
+        private void CreateAction(string yamlContent, out Pipelines.ActionStep instance, out string directory)
+        {
+            directory = Path.Combine(_workFolder, Constants.Path.ActionsDirectory, "GitHub/actions".Replace(Path.AltDirectorySeparatorChar, Path.DirectorySeparatorChar), "master");
+            string file = Path.Combine(directory, Constants.Path.ActionManifestFile);
+            Directory.CreateDirectory(Path.GetDirectoryName(file));
+            File.WriteAllText(file, yamlContent);
+            instance = new Pipelines.ActionStep()
+            {
+                Id = Guid.NewGuid(),
+                Reference = new Pipelines.RepositoryPathReference()
+                {
+                    Name = "GitHub/actions",
+                    Ref = "master",
+                    RepositoryType = Pipelines.RepositoryTypes.GitHub
+                }
+            };
+        }
+
         private void Setup([CallerMemberName] string name = "")
         {
             _ecTokenSource?.Dispose();

src/Test/L0/Worker/ExecutionContextL0.cs

@@ -206,22 +206,8 @@ namespace GitHub.Runner.Common.Tests.Worker
                 var action2 = jobContext.CreateChild(Guid.NewGuid(), "action_2", "action_2", null, null);
                 action2.IntraActionState["state"] = "2";
 
-
+                action1.RegisterPostJobAction("post1", "always()", new Pipelines.ActionStep() { Name = "post1", DisplayName = "Test 1", Reference = new Pipelines.RepositoryPathReference() { Name = "actions/action" } });
-                var postRunner1 = hc.CreateService<IActionRunner>();
+                action2.RegisterPostJobAction("post2", "always()", new Pipelines.ActionStep() { Name = "post2", DisplayName = "Test 2", Reference = new Pipelines.RepositoryPathReference() { Name = "actions/action" } });
-                postRunner1.Action = new Pipelines.ActionStep() { Name = "post1", DisplayName = "Test 1", Reference = new Pipelines.RepositoryPathReference() { Name = "actions/action" } };
-                postRunner1.Stage = ActionRunStage.Post;
-                postRunner1.Condition = "always()";
-                postRunner1.DisplayName = "post1";
-
-
-                var postRunner2 = hc.CreateService<IActionRunner>();
-                postRunner2.Action = new Pipelines.ActionStep() { Name = "post2", DisplayName = "Test 2", Reference = new Pipelines.RepositoryPathReference() { Name = "actions/action" } };
-                postRunner2.Stage = ActionRunStage.Post;
-                postRunner2.Condition = "always()";
-                postRunner2.DisplayName = "post2";
-
-                action1.RegisterPostJobStep("post1", postRunner1);
-                action2.RegisterPostJobStep("post2", postRunner2);
 
                 Assert.NotNull(jobContext.JobSteps);
                 Assert.NotNull(jobContext.PostJobSteps);

src/Test/L0/Worker/VariablesL0.cs

@@ -150,7 +150,7 @@ namespace GitHub.Runner.Common.Tests.Worker
                 string actual = variables.Get("no such");
 
                 // Assert.
-                Assert.Null(actual);
+                Assert.Equal(null, actual);
             }
         }
 

src/Test/Test.csproj

@@ -1,7 +1,7 @@
 <Project Sdk="Microsoft.NET.Sdk">
     <PropertyGroup>
         <TargetFramework>netcoreapp3.0</TargetFramework>
-        <RuntimeIdentifiers>win-x64;win-x86;linux-x64;linux-arm;linux-arm64;rhel.6-x64;osx-x64</RuntimeIdentifiers>
+        <RuntimeIdentifiers>win-x64;win-x86;linux-x64;linux-arm;rhel.6-x64;osx-x64</RuntimeIdentifiers>
         <TargetLatestRuntimePatch>true</TargetLatestRuntimePatch>
         <AssetTargetFallback>portable-net45+win8</AssetTargetFallback>
         <NoWarn>NU1701;NU1603;NU1603;xUnit2013;</NoWarn>
@@ -58,9 +58,6 @@
     <PropertyGroup Condition="'$([System.Runtime.InteropServices.RuntimeInformation]::IsOSPlatform($([System.Runtime.InteropServices.OSPlatform]::Linux)))' == 'true' AND '$(PackageRuntime)' == 'linux-arm'">
         <DefineConstants>OS_LINUX;ARM;TRACE</DefineConstants>
     </PropertyGroup>
-    <PropertyGroup Condition="'$([System.Runtime.InteropServices.RuntimeInformation]::IsOSPlatform($([System.Runtime.InteropServices.OSPlatform]::Linux)))' == 'true' AND '$(PackageRuntime)' == 'linux-arm64'">
-        <DefineConstants>OS_LINUX;ARM64;TRACE</DefineConstants>
-    </PropertyGroup>
     <PropertyGroup Condition="'$([System.Runtime.InteropServices.RuntimeInformation]::IsOSPlatform($([System.Runtime.InteropServices.OSPlatform]::Linux)))' == 'true' AND '$(Configuration)' == 'Debug' AND '$(PackageRuntime)' == 'linux-x64'">
         <DefineConstants>OS_LINUX;X64;DEBUG;TRACE</DefineConstants>
     </PropertyGroup>
@@ -70,7 +67,4 @@
     <PropertyGroup Condition="'$([System.Runtime.InteropServices.RuntimeInformation]::IsOSPlatform($([System.Runtime.InteropServices.OSPlatform]::Linux)))' == 'true' AND '$(Configuration)' == 'Debug' AND '$(PackageRuntime)' == 'linux-arm'">
         <DefineConstants>OS_LINUX;ARM;DEBUG;TRACE</DefineConstants>
     </PropertyGroup>
-    <PropertyGroup Condition="'$([System.Runtime.InteropServices.RuntimeInformation]::IsOSPlatform($([System.Runtime.InteropServices.OSPlatform]::Linux)))' == 'true' AND '$(Configuration)' == 'Debug' AND '$(PackageRuntime)' == 'linux-arm64'">
-        <DefineConstants>OS_LINUX;ARM64;DEBUG;TRACE</DefineConstants>
-    </PropertyGroup>
 </Project>

src/dev.sh

@@ -43,7 +43,7 @@ elif [[ "$CURRENT_PLATFORM" == 'linux' ]]; then
       CPU_NAME=$(uname -m)
       case $CPU_NAME in
          armv7l) RUNTIME_ID="linux-arm";;
-         aarch64) RUNTIME_ID="linux-arm64";;
+         aarch64) RUNTIME_ID="linux-arm";;
       esac
    fi
    
@@ -60,7 +60,7 @@ fi
 
 # Make sure current platform support publish the dotnet runtime
 # Windows can publish win-x86/x64
-# Linux can publish linux-x64/arm/arm64/rhel.6-x64
+# Linux can publish linux-x64/arm/rhel.6-x64
 # OSX can publish osx-x64
 if [[ "$CURRENT_PLATFORM" == 'windows' ]]; then
    if [[ ("$RUNTIME_ID" != 'win-x86') && ("$RUNTIME_ID" != 'win-x64') ]]; then
@@ -68,7 +68,7 @@ if [[ "$CURRENT_PLATFORM" == 'windows' ]]; then
       exit 1
    fi
 elif [[ "$CURRENT_PLATFORM" == 'linux' ]]; then
-   if [[ ("$RUNTIME_ID" != 'linux-x64') && ("$RUNTIME_ID" != 'linux-arm') && ("$RUNTIME_ID" != 'linux-arm64') && ("$RUNTIME_ID" != 'rhel.6-x64') ]]; then
+   if [[ ("$RUNTIME_ID" != 'linux-x64') && ("$RUNTIME_ID" != 'linux-arm') && ("$RUNTIME_ID" != 'rhel.6-x64') ]]; then
       echo "Failed: Can't build $RUNTIME_ID package $CURRENT_PLATFORM" >&2
       exit 1
    fi
@@ -208,6 +208,8 @@ fi
 echo "Prepend ${DOTNETSDK_INSTALLDIR} to %PATH%"
 export PATH=${DOTNETSDK_INSTALLDIR}:$PATH
 
+heading "Github Dreamlifter Runner"
+
 heading "Dotnet SDK Version"
 dotnet --version