Remove input private key from config shared with shipper#2096
Remove input private key from config shared with shipper#2096faec wants to merge 3 commits intoelastic:mainfrom
Conversation
|
This pull request does not have a backport label. Could you fix it @faec? 🙏
NOTE: |
💔 Tests Failed
Expand to view the summary
Build stats
Test stats 🧪
Test errors
Expand to view the tests failures
|
blakerouse
left a comment
There was a problem hiding this comment.
I am good with this change. Possible this broke the fake shipper tests.
|
@blakerouse Could you take a closer look? The fix for the current error isn't obvious to me, and seems like a bug in the configuration testing: The error arises in But that's where I lose the thread, because the initialization for fake units seems considerably different than for real ones and I'm not sure where it's diverging. Simply removing the |
|
/package |
|
i think i remember something like this with ast maps, it was something with references, that object where key map got injected got niled when it exited a block if i recall correctly |
|
@faec do we still need this one with the new shipper plan? |
|
Closing this one for now as we will need to rewrite things following the new Shipper plan. |
What does this PR do?
The shipper is currently given all key data from the input units, including the private key, but only the input unit itself should have the private key.