Skip to content

docs(enterprise): recommend cloud workload identity for CI traces storage#11442

Merged
mergify[bot] merged 1 commit into
mainfrom
devs/sileht/onpremise-iam-role/recommend-cloud-workload-identity-ci-traces--db1bb865
May 12, 2026
Merged

docs(enterprise): recommend cloud workload identity for CI traces storage#11442
mergify[bot] merged 1 commit into
mainfrom
devs/sileht/onpremise-iam-role/recommend-cloud-workload-identity-ci-traces--db1bb865

Conversation

@sileht
Copy link
Copy Markdown
Member

@sileht sileht commented May 8, 2026

Lead with IAM role discovery / Application Default Credentials in both the
S3 and GCS sections so self-hosted operators land on the path that avoids
manual key rotation. Move AWS_ACCOUNT_ID/AWS_REGION and the
MERGIFYENGINE_AWS_* deprecation note into the access-key option, since
neither is needed when boto3 resolves credentials from the workload.

Co-Authored-By: Claude Opus 4.7 (1M context) noreply@anthropic.com

Copilot AI review requested due to automatic review settings May 8, 2026 19:42
@mergify mergify Bot had a problem deploying to Mergify Merge Protections May 8, 2026 19:42 Failure
@mergify
Copy link
Copy Markdown
Contributor

mergify Bot commented May 8, 2026

Merge Protections

Your pull request matches the following merge protections and will not be merged until they are valid.

🟢 🤖 Continuous Integration

Wonderful, this rule succeeded.
  • all of:
    • check-success = build
    • check-success = lint
    • check-success = test
    • any of:
      • check-success = test-broken-links
      • label = ignore-broken-links
    • any of:
      • check-success=Cloudflare Pages
      • -head-repo-full-name~=^Mergifyio/

🟢 👀 Review Requirements

Wonderful, this rule succeeded.
  • any of:
    • #approved-reviews-by >= 2
    • author = dependabot[bot]
    • author = mergify-ci-bot

🟢 Enforce conventional commit

Wonderful, this rule succeeded.

Make sure that we follow https://www.conventionalcommits.org/en/v1.0.0/

  • title ~= ^(fix|feat|docs|style|refactor|perf|test|build|ci|chore|revert|ui)(?:\(.+\))?:

🟢 🔎 Reviews

Wonderful, this rule succeeded.
  • #changes-requested-reviews-by = 0
  • #review-requested = 0
  • #review-threads-unresolved = 0

🟢 📕 PR description

Wonderful, this rule succeeded.
  • body ~= (?ms:.{48,})

@sileht sileht marked this pull request as ready for review May 8, 2026 19:43
@mergify mergify Bot requested a review from a team May 8, 2026 19:44
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates the Enterprise “Advanced Features” documentation to emphasize using cloud-native identity/credential discovery (ADC on GCP, IAM role-based discovery on AWS) for CI traces object storage, reducing reliance on long-lived static keys and manual rotation.

Changes:

  • Reordered GCS auth options to lead with Application Default Credentials (recommended), moving JSON key usage to a secondary option.
  • Reworked S3 auth guidance to lead with IAM role discovery (recommended), moving access-key details (and related notes) into the access-key option.
  • Adjusted the introduction copy to present workload-identity discovery as the primary mode across providers.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread src/content/docs/enterprise/advanced-features.mdx
Comment thread src/content/docs/enterprise/advanced-features.mdx Outdated
Comment thread src/content/docs/enterprise/advanced-features.mdx
@sileht sileht force-pushed the devs/sileht/onpremise-iam-role/recommend-cloud-workload-identity-ci-traces--db1bb865 branch from 7aa7ddf to 450013c Compare May 8, 2026 20:15
@sileht
Copy link
Copy Markdown
Member Author

sileht commented May 8, 2026

Revision history

# Type Changes Reason Date
1 initial 7aa7ddf 2026-05-08 20:15 UTC
2 content 7aa7ddf → 450013c (raw) 2026-05-08 20:15 UTC

@mergify mergify Bot had a problem deploying to Mergify Merge Protections May 8, 2026 20:15 Failure
JulianMaurin
JulianMaurin previously approved these changes May 11, 2026
…rage

Lead with IAM role discovery / Application Default Credentials in both the
S3 and GCS sections so self-hosted operators land on the path that avoids
manual key rotation. Move AWS_ACCOUNT_ID/AWS_REGION and the
MERGIFYENGINE_AWS_* deprecation note into the access-key option, since
neither is needed when boto3 resolves credentials from the workload.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Change-Id: Idb1bb865a7b671c6e002eeeffb4f322f11dadb6c
@jd jd force-pushed the devs/sileht/onpremise-iam-role/recommend-cloud-workload-identity-ci-traces--db1bb865 branch from 450013c to f9c9bb8 Compare May 12, 2026 07:09
@mergify mergify Bot dismissed JulianMaurin’s stale review May 12, 2026 07:09

Pull request has been modified.

@mergify mergify Bot deployed to Mergify Merge Protections May 12, 2026 07:09 Active
@mergify mergify Bot requested a review from a team May 12, 2026 07:11
@mergify
Copy link
Copy Markdown
Contributor

mergify Bot commented May 12, 2026

Merge Queue Status

This pull request spent 3 minutes 7 seconds in the queue, including 2 minutes 41 seconds running CI.

Required conditions to merge

mergify Bot added a commit that referenced this pull request May 12, 2026
@mergify mergify Bot added the queued label May 12, 2026
@mergify mergify Bot merged commit ad5668f into main May 12, 2026
8 checks passed
@mergify mergify Bot deleted the devs/sileht/onpremise-iam-role/recommend-cloud-workload-identity-ci-traces--db1bb865 branch May 12, 2026 07:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Development

Successfully merging this pull request may close these issues.

4 participants