Skip to content

Fix air routes sample permission error on Podman#1588

Merged
kmcginnes merged 3 commits intoaws:mainfrom
kmcginnes:docs/fix-air-routes-sample
Mar 11, 2026
Merged

Fix air routes sample permission error on Podman#1588
kmcginnes merged 3 commits intoaws:mainfrom
kmcginnes:docs/fix-air-routes-sample

Conversation

@kmcginnes
Copy link
Collaborator

@kmcginnes kmcginnes commented Mar 11, 2026

Description

Fix the air routes Gremlin Server sample failing on Podman (and other rootless container runtimes) with a sed: Permission denied error.

The default entrypoint runs sed -i on the config file to replace the host binding, which fails when the container user cannot write to the bind-mounted volume. This bypasses the entrypoint by setting host: database directly in the config and launching gremlin-server.sh without the sed hack.

  • Set host: database in the Gremlin Server config so the sed substitution is unnecessary
  • Override the entrypoint to call gremlin-server.sh directly
  • Mount the sample volume as read-only since nothing needs to write to it

Validation

  • Tested with Podman — container starts successfully without permission errors

Check List

  • I confirm that my contribution is made under the terms of the Apache 2.0 license.
  • I have run pnpm checks to ensure code compiles and meets standards.
  • I have run pnpm test to check if all tests are passing.
  • I have covered new added functionality with unit tests if necessary.
  • I have added an entry in the Changelog.md.

@kmcginnes kmcginnes marked this pull request as ready for review March 11, 2026 20:40
@codecov
Copy link

codecov bot commented Mar 11, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 64.91%. Comparing base (c5affc5) to head (5cb93ae).
⚠️ Report is 99 commits behind head on main.

Additional details and impacted files
@@             Coverage Diff             @@
##             main    #1588       +/-   ##
===========================================
+ Coverage   47.81%   64.91%   +17.10%     
===========================================
  Files         382      372       -10     
  Lines        8525     8382      -143     
  Branches     3159     3123       -36     
===========================================
+ Hits         4076     5441     +1365     
+ Misses       3070     2086      -984     
+ Partials     1379      855      -524     
Flag Coverage Δ
unittests 64.91% <ø> (+17.10%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@kmcginnes kmcginnes merged commit 04612c0 into aws:main Mar 11, 2026
4 checks passed
@kmcginnes kmcginnes deleted the docs/fix-air-routes-sample branch March 11, 2026 22:02
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.

3 participants