Skip to content

fix(ci): restore AS checks and stabilize gateway build - #226

Merged
jialez0 merged 3 commits into
mainfrom
agent/fix-artifact-policy-ci-tests
Aug 20, 2026
Merged

fix(ci): restore AS checks and stabilize gateway build#226
jialez0 merged 3 commits into
mainfrom
agent/fix-artifact-policy-ci-tests

Conversation

@jialez0

@jialez0 jialez0 commented Aug 20, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • initialize the challenge key rotation test config through Config::default so new optional AS fields do not break the integration test
  • add policy-to-Artifact-Server tests for the request wire shape, resolved/denied results, and infrastructure error propagation
  • exercise no-default and isolated policy-artifact-server feature builds in Rust CI
  • make the Gateway GOPROXY configurable and use proxy.golang.org in GitHub Actions while retaining the existing Aliyun default for local builds

Root cause

After PR #220 was merged, the main branch exposed two failures: challenge_key_rotation.rs no longer initialized every Config field, and the Aliyun Go proxy repeatedly returned invalid module zip files to the GitHub runner.

Validation

  • cargo +1.76 test -p attestation-service -p verifier
  • cargo +1.76 test -p attestation-service --lib --no-default-features
  • cargo +1.76 test -p attestation-service --lib --no-default-features --features policy-artifact-server
  • cargo +1.76 test -p attestation-service --lib --no-default-features --features policy-rvps
  • cargo +1.76.0 clippy -p attestation-service -p verifier -- -D warnings -A clippy::derive_partial_eq_without_eq
  • cargo +1.76 fmt -p attestation-service -p verifier --check
  • go test ./... under trustee-gateway
  • actionlint .github/workflows/docker-build.yml .github/workflows/rust-check.yml

Signed-off-by: Jiale Zhang <zhangjiale@linux.alibaba.com>
Signed-off-by: Jiale Zhang <zhangjiale@linux.alibaba.com>
Signed-off-by: Jiale Zhang <zhangjiale@linux.alibaba.com>
@ostest-bot

Copy link
Copy Markdown

@jialez0 ,您好,您的请求已接收,请耐心等待结果。

@ostest-bot

Copy link
Copy Markdown

@jialez0 ,您好,未检测到有镜像需要构建,如需重新检测请评论 /start

@jialez0
jialez0 merged commit 531bcb0 into main Aug 20, 2026
22 checks passed
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.

2 participants