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(windows-agent): Delegates agent.WaitReady to the daemon #982

Merged
merged 4 commits into from
Dec 4, 2024

Commits on Nov 26, 2024

  1. Revert "Temp workaround for flaky behaviour shown by TestLogs"

    This reverts commit eb6c321.
    It's just one hunk, but I think reverting a commit is more meaningful.
    CarlosNihelton committed Nov 26, 2024
    Configuration menu
    Copy the full SHA
    d4a1671 View commit details
    Browse the repository at this point in the history
  2. Delay closing daemon.serving

    Till the latest moment possible.
    CarlosNihelton committed Nov 26, 2024
    Configuration menu
    Copy the full SHA
    45193ca View commit details
    Browse the repository at this point in the history
  3. Let the daemon advise when ready

    The deamon knows best when is the most likely moment to
    be serving.
    CarlosNihelton committed Nov 26, 2024
    Configuration menu
    Copy the full SHA
    6d385e8 View commit details
    Browse the repository at this point in the history
  4. Agent defers to daemon to advise when ready

    The a.ready channel is still useful to synchronize the access to a.daemon,
    preventing nil pointer access when the deamon is not yet initialized.
    Checking for nil is still needed, as the a.ready channel may be closed without ever serving.
    CarlosNihelton committed Nov 26, 2024
    Configuration menu
    Copy the full SHA
    8cb8854 View commit details
    Browse the repository at this point in the history