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
SOLR-16635: Resolve crypto keys file path in MiniSolrCloudCluster #1312
Conversation
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.
Left a minor note, but overall LGTM.
@@ -89,6 +90,10 @@ | |||
public class MiniSolrCloudCluster { | |||
|
|||
private static final Logger log = LoggerFactory.getLogger(MethodHandles.lookup().lookupClass()); | |||
private static final URL PRE_GENERATED_PRIVATE_K_URL = |
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.
nit-pick, but can we use KEY
instead of K
i.e. PRE_GENERATED_PRIVATE_KEY_URL
@@ -89,6 +90,10 @@ | |||
public class MiniSolrCloudCluster { | |||
|
|||
private static final Logger log = LoggerFactory.getLogger(MethodHandles.lookup().lookupClass()); | |||
private static final URL PRE_GENERATED_PRIVATE_K_URL = | |||
MiniSolrCloudCluster.class.getClassLoader().getResource("cryptokeys/priv_key512_pkcs8.pem"); | |||
private static final URL PRE_GENERATED_PUBLIC_K_URL = |
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.
Same here
7458c0f
to
b5778a3
Compare
The failures I see in the test action are not reproducing and are common in Jenkins emails, so I assume they are unrelated to this change |
By resolving the full path of the pre-generated crypto keys in
MiniSolrCloudCluster
we make sure it also works for third party projects testing using Solr test framework