Skip to content

[DO NOT REVIEW] PR to test native sidecar#1588

Closed
aramase wants to merge 3 commits intoAzure:mainfrom
aramase:aramase/c/native_sidecar_test
Closed

[DO NOT REVIEW] PR to test native sidecar#1588
aramase wants to merge 3 commits intoAzure:mainfrom
aramase:aramase/c/native_sidecar_test

Conversation

@aramase
Copy link
Member

@aramase aramase commented Apr 8, 2025

No description provided.

devjoes and others added 3 commits April 8, 2025 14:13
Signed-off-by: Anish Ramasekar <anish.ramasekar@gmail.com>
Signed-off-by: Anish Ramasekar <anish.ramasekar@gmail.com>
Signed-off-by: Anish Ramasekar <anish.ramasekar@gmail.com>
@aramase
Copy link
Member Author

aramase commented Apr 8, 2025

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@aramase
Copy link
Member Author

aramase commented Apr 8, 2025

Tests failed as expected (simulating the failure by setting 1.33 as min version in webhook: 6a905c4)

• [FAILED] [0.071 seconds]
Proxy [LinuxOnly] [AKSSoakOnly] [It] should get a valid AAD token after injecting proxy init container and sidecar with no client_id in request
/mnt/vss/_work/1/s/test/e2e/proxy_test.go:79

  [FAILED] proxy sidecar is not injected to pod default-zj7zt
  Expected
      <*v1.Container | 0x0>: nil
  not to be nil
  In [It] at: /mnt/vss/_work/1/s/test/e2e/helpers.go:333 @ 04/08/25 21:33:41.228
Summarizing 2 Failures:
  [FAIL] Proxy [LinuxOnly] [AKSSoakOnly] [It] should get a valid AAD token after injecting proxy init container and sidecar with no client_id in request
  /mnt/vss/_work/1/s/test/e2e/helpers.go:333
  [FAIL] Proxy [LinuxOnly] [AKSSoakOnly] [It] should get a valid AAD token after injecting proxy init container and sidecar
  /mnt/vss/_work/1/s/test/e2e/helpers.go:333

Ran 12 of 12 Specs in 27.163 seconds
FAIL! -- 10 Passed | 2 Failed | 0 Pending | 0 Skipped

@aramase aramase closed this Apr 8, 2025
@aramase aramase deleted the aramase/c/native_sidecar_test branch April 8, 2025 21:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants