-
-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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: allow override of artifact server bind address #1560
Conversation
* Prior to this change, the artifact server always binds to the detected "outbound IP", breaks functionality when that IP is unroutable. For example, Zscaler assigns the host a local CGNAT address, 100.64.0.1, which is unreachable from Docker Desktop. * Add the `--artifact-server-addr` flag to allow override of the address to which the artifact server binds, defaulting to the existing behaviour. Fixes: #1559
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good for me. Thanks for your contribution.
@isometry this pull request has failed checks 🛠 |
Codecov Report
@@ Coverage Diff @@
## master #1560 +/- ##
==========================================
+ Coverage 61.22% 61.44% +0.22%
==========================================
Files 46 46
Lines 7141 7156 +15
==========================================
+ Hits 4372 4397 +25
+ Misses 2462 2452 -10
Partials 307 307
📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
@KnisterPeter : anything else I need to do to get this merged? |
More approvals from maintainers are required. Just wait a bit longer. |
--artifact-server-addr
flag to allow override of the address to which the artifact server binds, defaulting to the existing behaviour.Fixes #1559