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.

Sign up for GitHub

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

Jump to bottom

Fix race condition when looking up reaper (ryuk) container #2508

Merged
merged 8 commits into from
Jun 10, 2024
Merged

Fix race condition when looking up reaper (ryuk) container #2508

merged 8 commits into from
Jun 10, 2024

Commits on May 21, 2024

  1. If the ryuk container has a health status, wait for a healthy contain… 

    …er before returning.
    @emetsger
    emetsger committed May 21, 2024
    Configuration menu
    Copy the full SHA
    4fee0d9 View commit details
    Browse the repository at this point in the history

  2. Use docker/types for health status, but also check for the zero value.

    @emetsger
    emetsger committed May 21, 2024
    Configuration menu
    Copy the full SHA
    b097968 View commit details
    Browse the repository at this point in the history

  3. Wait for the ryuk port to be available in case the container is still… 

    … being started by newReaper in a separate process.

A race between newReaper and lookUpReaperContainer occurs:
- newReaper creates the container with a ContainerRequest.WaitingFor = wait.ForListeningPort(listeningPort)
- newReaper starts the container
- lookUpReaperContainer obtains the container and returns.
- newReaper invokes the readiness hook wait.ForListeningPort(listeningPort).
    @emetsger
    emetsger committed May 21, 2024
    Configuration menu
    Copy the full SHA
    0c99285 View commit details
    Browse the repository at this point in the history

  4. Fix whitespace.

    @emetsger
    emetsger committed May 21, 2024
    Configuration menu
    Copy the full SHA
    f92db6e View commit details
    Browse the repository at this point in the history

Commits on Jun 10, 2024

  1. chore: simplify

    @mdelapenya
    mdelapenya committed Jun 10, 2024
    Configuration menu
    Copy the full SHA
    f9d0cf5 View commit details
    Browse the repository at this point in the history

  2. chore: make lint

    @mdelapenya
    mdelapenya committed Jun 10, 2024
    Configuration menu
    Copy the full SHA
    ab71f41 View commit details
    Browse the repository at this point in the history

  3. chore: change emoji in log output when waiting

    @mdelapenya
    mdelapenya committed Jun 10, 2024
    Configuration menu
    Copy the full SHA
    b835925 View commit details
    Browse the repository at this point in the history

  4. chore: move inside reuseReaper

    @mdelapenya
    mdelapenya committed Jun 10, 2024
    Configuration menu
    Copy the full SHA
    d7f1b71 View commit details
    Browse the repository at this point in the history