Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: make sure router gets built on start, for #5040 #5052

Merged
merged 2 commits into from
Jul 1, 2023

Conversation

rfay
Copy link
Member

@rfay rfay commented Jun 30, 2023

The Issue

We've been having trouble on upgrades with ddev-router not getting rebuilt; it turns out the docker-compose command didn't have --build in there.

How This PR Solves The Issue

Add --build so router gets rebuilt if it already existed.

Manual Testing Instructions

Automated Testing Overview

Related Issue Link(s)

Release/Deployment Notes

@github-actions
Copy link

@rfay
Copy link
Member Author

rfay commented Jun 30, 2023

This solved @rpkoller 's problem, this alone, was tested on misbehaving system.

@rfay rfay changed the title fix: make sure router gets built on start fix: make sure router gets built on start, fixes #5052 Jun 30, 2023
@rfay rfay marked this pull request as ready for review June 30, 2023 22:39
@rfay rfay requested a review from a team as a code owner June 30, 2023 22:39
@rfay rfay requested a review from gilbertsoft June 30, 2023 22:39
@rpkoller
Copy link
Collaborator

rpkoller commented Jun 30, 2023

I hope that is the correct bit @rfay asked for. but that is the output we had debugging the problem on my computer.

in ~/.ddev/router-build we did cat Dockerfile

#ddev-generated

ARG BASE_IMAGE
FROM $BASE_IMAGE


RUN apk add bash curl htop vim
WORKDIR /mnt/ddev-global-cache/traefik
COPY /traefik_healthcheck.sh /healthcheck.sh

and then the following command inspecting the router output:

$> docker inspect --format "{{json .State.Health }}" ddev-router
{"Status":"unhealthy","FailingStreak":1003,"Log":[{"Start":"2023-06-30T21:49:05.202976174Z","End":"2023-06-30T21:49:05.321276965Z","ExitCode":127,"Output":"/bin/sh: /healthcheck.sh: not found\n"},{"Start":"2023-06-30T21:49:06.328373341Z","End":"2023-06-30T21:49:06.445561549Z","ExitCode":127,"Output":"/bin/sh: /healthcheck.sh: not found\n"},{"Start":"2023-06-30T21:49:07.450499216Z","End":"2023-06-30T21:49:07.576819591Z","ExitCode":127,"Output":"/bin/sh: /healthcheck.sh: not found\n"},{"Start":"2023-06-30T21:49:08.582849508Z","End":"2023-06-30T21:49:08.688931134Z","ExitCode":127,"Output":"/bin/sh: /healthcheck.sh: not found\n"},{"Start":"2023-06-30T21:49:09.694490134Z","End":"2023-06-30T21:49:09.799587759Z","ExitCode":127,"Output":"/bin/sh: /healthcheck.sh: not found\n"}]}

If i should anything else please let me know. and again thanks for that quick fix and the spontaneous debugging session!

@gilbertsoft
Copy link
Member

@rpkoller can you please check if the binaries from #5052 (comment) work for you? Thanks!

@rfay
Copy link
Member Author

rfay commented Jul 1, 2023

@rpkoller had already tested earlier version; the complexity is once the problem is solved (once the new image gets built) the problem is solved. So he can't test without going back to earlier version (with traefik enabled), deleting all build cache, deleting -built image, and then upgrading.

@rfay
Copy link
Member Author

rfay commented Jul 1, 2023

Thanks for fixing in 8f409c7 and the refactor of the embeds is 💯

@rfay rfay changed the title fix: make sure router gets built on start, fixes #5052 fix: make sure router gets built on start, fixes #5040 Jul 1, 2023
@rfay rfay changed the title fix: make sure router gets built on start, fixes #5040 fix: make sure router gets built on start, for #5040 Jul 1, 2023
@rfay rfay merged commit 4e76a60 into ddev:master Jul 1, 2023
16 checks passed
@rfay rfay deleted the 20230630_router_build_on_up branch July 1, 2023 17:55
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.

None yet

3 participants