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

connect: use deterministic injected dynamic exposed port label #10492

Merged
merged 1 commit into from
May 3, 2021

Commits on Apr 30, 2021

  1. connect: use deterministic injected dynamic exposed port

    This PR uses the checksum of the check for which a dynamic exposed
    port is being generated (instead of a UUID prefix) so that the
    generated port label is deterministic.
    
    This fixes 2 bugs:
     - 'job plan' output is now idempotent for jobs making use of injected ports
     - tasks will no longer be destructively updated when jobs making use of
       injected ports are re-run without changing any user specified part of
       job config.
    
    Closes: #10099
    shoenig committed Apr 30, 2021
    Configuration menu
    Copy the full SHA
    08a766a View commit details
    Browse the repository at this point in the history