This was broken in:
commit 2e24fb76c84c8604da3606967f69a0a29b85f265 |
Author: Melissa Stone <mel...@puppet.com> |
Date: Fri Feb 15 15:39:58 2019 -0800 |
|
(PUP-9076) Do not push nil on server context |
|
and fail if none of the servers in `server_list` are reachable |
|
:040000 040000 85e374f3c6c205f0d591f57e8d5a52ab04bc116c 5cbdef33008c604f599eafa8337adcdbbb15a42e M lib |
:040000 040000 16c9a771f8f9f2a5f0cd56bd9c6e2a15e8516cac 0491538df3b80e5f96bdf00b36f17fd3a41a4d7f M spec
|
Previously if we failed to connect to a valid server, we pushed [nil, nil] on the context and then tried to connect to ":8140" for the node, catalog, etc requests, which of course failed. We then failed back to the cached catalog. After this commit, we raise if we can't resolve a valid server, never giving the agent a chance to use its cached catalog. |