Skip to content

test: verify Docker image builds and starts successfully with custom port#269

Open
icebear0828 wants to merge 1 commit intomasterfrom
add-custom-port-smoke-test-4386442151084696089
Open

test: verify Docker image builds and starts successfully with custom port#269
icebear0828 wants to merge 1 commit intomasterfrom
add-custom-port-smoke-test-4386442151084696089

Conversation

@icebear0828
Copy link
Copy Markdown
Owner

Fixes #120 by expanding the ci-docker.yml workflow to perform the required smoke testing on the custom port 8090 using a generated default.yaml configuration.


PR created automatically by Jules for task 4386442151084696089 started by @icebear0828

…port

Added tests to `.github/workflows/ci-docker.yml` to ensure the Docker container correctly parses the custom server port setting via `default.yaml`. Tests were added to generate a custom configuration and confirm the new port is respected and verified by both the container healthcheck and the external `/health` endpoint.

Co-authored-by: icebear0828 <63769498+icebear0828@users.noreply.github.com>
@google-labs-jules
Copy link
Copy Markdown

👋 Jules, reporting for duty! I'm here to lend a hand with this pull request.

When you start a review, I'll add a 👀 emoji to each comment to let you know I've read it. I'll focus on feedback directed at me and will do my best to stay out of conversations between you and other bots or reviewers to keep the noise down.

I'll push a commit with your requested changes shortly after. Please note there might be a delay between these steps, but rest assured I'm on the job!

For more direct control, you can switch me to Reactive Mode. When this mode is on, I will only act on comments where you specifically mention me with @jules. You can find this option in the Pull Request section of your global Jules UI settings. You can always switch back!

New to Jules? Learn more at jules.google/docs.


For security, I will only act on instructions from the user who triggered this task.

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.

test: verify Docker image builds and starts successfully

1 participant