Merge branch 'feature/AddCancellationRequestedState' of github.com:Ma…

…ttMcL4475/task-execution-schemas into MattMcL4475-feature/AddCancellationRequestedState

.github/workflows/lint-validate-openapi.yml

@@ -0,0 +1,51 @@
+name: Lint and validate OpenAPI specs
+
+on:
+  - push
+  - pull_request
+
+jobs:
+
+  lint:
+    name: Lint OpenAPI definition
+    runs-on: ubuntu-latest
+    steps:
+      - name: Check out head branch
+        uses: actions/checkout@v2
+      - name: Run OpenAPI Lint Action
+        uses: nwestfall/openapi-action@v1.0.2
+        with:
+          github_token: ${{ secrets.GITHUB_TOKEN }}
+          file: openapi/task_execution_service.openapi.yaml
+
+  diff:
+    name: Show OpenAPI differences relative to target branch
+    runs-on: ubuntu-latest
+    if: ${{ github.event_name == 'pull_request' }}
+    steps:
+      - name: Check out head branch
+        uses: actions/checkout@v2
+        with:
+          ref: ${{ github.head_ref }}
+          path: head
+      - name: Check out base branch
+        uses: actions/checkout@v2
+        with:
+          ref: ${{ github.base_ref }}
+          path: base
+      - name: Run OpenAPI Diff Action
+        uses: mvegter/openapi-diff-action@v0.23.5
+        with:
+          head-spec: head/openapi/task_execution_service.openapi.yaml
+          base-spec: base/openapi/task_execution_service.openapi.yaml
+
+  validate:
+    name: Validate OpenAPI definition
+    runs-on: ubuntu-latest
+    steps:
+      - name: Check out head branch
+        uses: actions/checkout@v2
+      - name: Run OpenAPI Validate Action
+        uses: char0n/swagger-editor-validate@v1
+        with:
+          definition-file: openapi/task_execution_service.openapi.yaml

openapi/task_execution_service.openapi.yaml

@@ -643,7 +643,9 @@ components:
          - `SYSTEM_ERROR`: The task was stopped due to a system error, but not from an Executor,
         for example an upload failed due to network issues, the worker's ran out
         of disk space, etc.
-         - `CANCELED`: The task was canceled by the user.
+         - `CANCELED`: The task was canceled by the user, and downstream resources have been deleted.
+         - `CANCELING`: The task was canceled by the user,
+        but the downstream resources are still awaiting deletion.
          - `PREEMPTED`: The task is stopped (preempted) by the system. The reasons for this would be tied to
         the specific system running the job. Generally, this means that the system reclaimed the compute
         capacity for reallocation.
@@ -660,6 +662,7 @@ components:
       - SYSTEM_ERROR
       - CANCELED
       - PREEMPTED
+      - CANCELING
     tesTask:
       required:
       - executors