Skip to content

Conversation

@akshaydeo
Copy link
Contributor

@akshaydeo akshaydeo commented Dec 15, 2025

Summary

Add AWS S3 bucket and Bedrock role ARN secrets to the core release workflow.

Changes

  • Added AWS_S3_BUCKET and AWS_BEDROCK_ROLE_ARN environment variables to the core release job
  • These secrets are now passed to the release-core.sh script during execution

Type of change

  • Chore/CI

Affected areas

  • Core (Go)

How to test

The changes can be verified by running a core release job and confirming that the AWS credentials are properly passed to the release script.

Breaking changes

  • No

Security considerations

This PR adds access to AWS secrets in the release pipeline. These secrets are properly handled through GitHub's secret management system and only exposed to the release script during execution.

Checklist

  • I verified builds succeed (Go and UI)
  • I verified the CI pipeline passes locally if applicable

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Dec 15, 2025

Caution

Review failed

The pull request is closed.

📝 Walkthrough

Summary by CodeRabbit

  • Chores
    • Updated release pipeline configuration to support enhanced deployment infrastructure capabilities.

✏️ Tip: You can customize this high-level summary in your review settings.

Walkthrough

A GitHub Actions workflow configuration update that passes two new AWS-related environment secrets (AWS_S3_BUCKET and AWS_BEDROCK_ROLE_ARN) to the core release step, expanding external system integration capabilities for the release pipeline.

Changes

Cohort / File(s) Change Summary
Release Pipeline Workflow Configuration
·github/workflows/release-pipeline.yml
Added AWS_S3_BUCKET and AWS_BEDROCK_ROLE_ARN environment secrets to the core release step

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐰 The workflow grows wings with secrets new,
AWS buckets and roles breakthrough!
A rabbit hops through pipelines so free,
Configuration changes light as can be! ✨

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch 12-16-fixes_release_pipeline_to_add_missing_env_variables

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between d959f9b and 9366ed0.

📒 Files selected for processing (1)
  • .github/workflows/release-pipeline.yml (1 hunks)

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Contributor Author

This stack of pull requests is managed by Graphite. Learn more about stacking.

@github-actions
Copy link
Contributor

🧪 Test Suite Available

This PR can be tested by a repository admin.

Run tests for PR #1099

@akshaydeo akshaydeo marked this pull request as ready for review December 15, 2025 19:46
@akshaydeo akshaydeo merged commit a322564 into main Dec 15, 2025
9 of 10 checks passed
@akshaydeo akshaydeo deleted the 12-16-fixes_release_pipeline_to_add_missing_env_variables branch December 15, 2025 19:46
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