Skip to content

UID2-6321 Reenable aks e2e #2375

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
cYKatherine merged 3 commits into from
Feb 26, 2026
Merged

UID2-6321 Reenable aks e2e #2375

Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
0d28fb7
Add aks e2e test back
cYKatherine Feb 17, 2026
5e2b051
Use kcc-UID2-6321-reenable-aks-e2e branch for e2e test
cYKatherine Feb 17, 2026
044180d
Revert to v3
cYKatherine Feb 26, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
10 changes: 9 additions & 1 deletion .github/workflows/publish-azure-cc-enclave-docker.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -251,4 +251,12 @@
name: azure-aks-enclave-id-${{ needs.buildImage.outputs.jar_version }}
path: ${{ env.MANIFEST_OUTPUT_DIR }}
if-no-files-found: error


e2eAzureAks:
name: E2E Azure AKS
uses: ./.github/workflows/run-e2e-tests-on-operator.yaml
needs: [buildImage, azureAks]
with:
operator_type: aks
operator_image_version: ${{ needs.buildImage.outputs.image_tag }}
secrets: inherit
Comment on lines +256 to +262

Check warning

Code scanning / CodeQL

Workflow does not contain permissions Medium

Actions job or workflow does not limit the permissions of the GITHUB_TOKEN. Consider setting an explicit permissions block, using the following as a minimal starting point: {}

Copilot Autofix

AI 6 months ago

To fix the problem, explicitly declare permissions for the e2eAzureAks job so that the GITHUB_TOKEN does not default to potentially broad repository or organization settings. The minimal, behavior-preserving approach is to add permissions: {} to that job, which disables all default token permissions for this job and leaves any necessary scopes to be defined in the called reusable workflow (run-e2e-tests-on-operator.yaml).

Concretely, in .github/workflows/publish-azure-cc-enclave-docker.yaml, in the jobs: section where e2eAzureAks is defined (around lines 255–262), add a permissions: {} line alongside the other job-level keys (similar to the existing permissions: {} on azureCc and azureAks). The result will look like:

  e2eAzureAks:
    name: E2E Azure AKS
    permissions: {}
    uses: ./.github/workflows/run-e2e-tests-on-operator.yaml
    needs: [buildImage, azureAks]
    with:
      operator_type: aks
      operator_image_version: ${{ needs.buildImage.outputs.image_tag }}
    secrets: inherit

No additional imports, methods, or definitions are required.

Suggested changeset 1
.github/workflows/publish-azure-cc-enclave-docker.yaml

Autofix patch

Autofix patch
Run the following command in your local git repository to apply this patch
cat << 'EOF' | git apply
diff --git a/.github/workflows/publish-azure-cc-enclave-docker.yaml b/.github/workflows/publish-azure-cc-enclave-docker.yaml
--- a/.github/workflows/publish-azure-cc-enclave-docker.yaml
+++ b/.github/workflows/publish-azure-cc-enclave-docker.yaml
@@ -254,6 +254,7 @@
   
   e2eAzureAks:
     name: E2E Azure AKS
+    permissions: {}
     uses: ./.github/workflows/run-e2e-tests-on-operator.yaml
     needs: [buildImage, azureAks]
     with:
EOF
@@ -254,6 +254,7 @@

e2eAzureAks:
name: E2E Azure AKS
permissions: {}
uses: ./.github/workflows/run-e2e-tests-on-operator.yaml
needs: [buildImage, azureAks]
with:
Copilot is powered by AI and may make mistakes. Always verify output.

@lizk886 lizk886 Feb 20, 2026

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

maybe add
permissions:
contents: read
? Didn't spend time look into it want to make sure you have

Loading