Skip to content

Migrate to RWX for CI#862

Open
kylekthompson wants to merge 3 commits intopgdogdev:mainfrom
kylekthompson:kt/rwx
Open

Migrate to RWX for CI#862
kylekthompson wants to merge 3 commits intopgdogdev:mainfrom
kylekthompson:kt/rwx

Conversation

@kylekthompson
Copy link
Copy Markdown

@kylekthompson kylekthompson commented Apr 1, 2026

👋 This change migrates your .github/workflows/ci.yml configuration over to RWX. There are a few code changes in here as well to mitigate some flakiness that was prevalent on larger agent configurations.

@CLAassistant
Copy link
Copy Markdown

CLAassistant commented Apr 1, 2026

CLA assistant check
All committers have signed the CLA.

@kylekthompson kylekthompson changed the title Migrate to RWX for CI [wip] Migrate to RWX for CI Apr 1, 2026
Comment on lines +313 to +317
if [ "$status" -ne 0 ]; then
printf '%s\n' \
"pgdog-example-plugin unit tests failed with exit code $status" \
> "$RWX_WARNINGS/plugin-unit"
fi
Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

This and the similar one below are here to mirror the continue-on-error behavior in GitHub Actions.

They do seem to run successfully, though, so we could consider removing them

@kylekthompson kylekthompson changed the title [wip] Migrate to RWX for CI Migrate to RWX for CI Apr 2, 2026
@kylekthompson kylekthompson marked this pull request as ready for review April 2, 2026 19:39
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