Skip to content

fix: Use GitHub App token for prerelease workflow dispatch#913

Merged
Aaron ("AJ") Steers (aaronsteers) merged 1 commit intomainfrom
devin/1765672284-fix-prerelease-token
Dec 14, 2025
Merged

fix: Use GitHub App token for prerelease workflow dispatch#913
Aaron ("AJ") Steers (aaronsteers) merged 1 commit intomainfrom
devin/1765672284-fix-prerelease-token

Conversation

@aaronsteers
Copy link
Member

Summary

Fixes the /prerelease command which was failing with Error: Parameter token or opts.auth is required because GITHUB_CI_WORKFLOW_TRIGGER_PAT doesn't exist.

The fix uses the same GitHub App token pattern already used by other slash commands in this repo (/fix-pr, /test-pr): generate an installation token at runtime using actions/create-github-app-token with the Octavia Bot credentials.

Review & Testing Checklist for Human

  • Verify secrets exist: Confirm OCTAVIA_BOT_APP_ID and OCTAVIA_BOT_PRIVATE_KEY secrets are configured for this repo
  • Verify GitHub App permissions: The Octavia Bot app must have permission to trigger workflow_dispatch events on this repo
  • Test end-to-end after merge: Comment /prerelease on a test PR to verify the full flow works (this cannot be tested before merge)

Notes

This is the third iteration fixing the /prerelease command:

  1. First fix: Added missing permissions for nested workflow call
  2. Second fix: Changed from workflow_call to workflow_dispatch for OIDC attestation compatibility
  3. This fix: Use GitHub App token instead of non-existent PAT secret

Link to Devin run: https://app.devin.ai/sessions/c86d36be59664129af00617d0e66bc4d
Requested by: AJ Steers (Aaron ("AJ") Steers (@aaronsteers))

Co-Authored-By: AJ Steers <aj@airbyte.io>
@devin-ai-integration
Copy link
Contributor

Original prompt from AJ Steers
Received message in Slack channel #ask-devin-ai:

@Devin Please refresh my memory on this one. IIRC:
1. We may already have a PR but I don't recall why it wasn't merged.
2. The destination-side fix (probably in the CDK) is to not care if a namespace is provided as null. (Difference between _not_ being sent or being sent _as null_ is a JSON delivery nuance.)

Thread URL: https://airbytehq-team.slack.com/archives/C08BHPUMEPJ/p1765591320408679

Quote of conversation (https://airbytehq-team.slack.com/archives/C06FZ238P8W/p1765588766040519?thread_ts=1765588766.040519&cid=C06FZ238P8W):
> From Biplove Jaisi
> Hi everyone :wave:

> I’m trying to use PyAirbyte to move data from an S3 source to an Iceberg destination. The sync from the source to DuckDB cache works fine, but when loading into Iceberg, I’m getting an error saying that the namespace cannot be null.

> The issue is that the S3 source doesn’t seem to have any field or config option to populate the namespace, so I’m not sure how to handle this requirement on the destination side.

> Has anyone faced this before or found a workaround? Any guidance would be really appreciated. Thanks in advance! :pray:
> Posted on December 13, 2025 at 01:19 AM

@devin-ai-integration
Copy link
Contributor

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR. Add '(aside)' to your comment to have me ignore it.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment and CI monitoring

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Dec 14, 2025

Warning

Rate limit exceeded

Aaron ("AJ") Steers (@aaronsteers) has exceeded the limit for the number of commits or files that can be reviewed per hour. Please wait 6 minutes and 58 seconds before requesting another review.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

📥 Commits

Reviewing files that changed from the base of the PR and between cd1b390 and 752fd18.

📒 Files selected for processing (1)
  • .github/workflows/prerelease-command.yml (1 hunks)
✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch devin/1765672284-fix-prerelease-token

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

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

@github-actions
Copy link

👋 Greetings, Airbyte Team Member!

Here are some helpful tips and reminders for your convenience.

Testing This PyAirbyte Version

You can test this version of PyAirbyte using the following:

# Run PyAirbyte CLI from this branch:
uvx --from 'git+https://github.com/airbytehq/PyAirbyte.git@devin/1765672284-fix-prerelease-token' pyairbyte --help

# Install PyAirbyte from this branch for development:
pip install 'git+https://github.com/airbytehq/PyAirbyte.git@devin/1765672284-fix-prerelease-token'

Helpful Resources

PR Slash Commands

Airbyte Maintainers can execute the following slash commands on your PR:

  • /fix-pr - Fixes most formatting and linting issues
  • /poetry-lock - Updates poetry.lock file
  • /test-pr - Runs tests with the updated PyAirbyte
  • /prerelease - Builds and publishes a prerelease version to PyPI

Community Support

Questions? Join the #pyairbyte channel in our Slack workspace.

📝 Edit this welcome message.

@github-actions
Copy link

PyTest Results (Fast Tests Only, No Creds)

348 tests  ±0   348 ✅ ±0   6m 41s ⏱️ +47s
  1 suites ±0     0 💤 ±0 
  1 files   ±0     0 ❌ ±0 

Results for commit 752fd18. ± Comparison against base commit cd1b390.

@aaronsteers Aaron ("AJ") Steers (aaronsteers) merged commit 9f3c707 into main Dec 14, 2025
24 checks passed
@aaronsteers Aaron ("AJ") Steers (aaronsteers) deleted the devin/1765672284-fix-prerelease-token branch December 14, 2025 00:47
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.

1 participant