Re: [PR] SOLR-15857: Add AWS Secret Manager support for ZK ACL credentials [solr]

2023-11-28 Thread via GitHub
laminelam commented on code in PR #1994: URL: https://github.com/apache/solr/pull/1994#discussion_r1408273641 ## solr/server/scripts/cloud-scripts/zkcli.sh: ## @@ -13,6 +13,21 @@ log4j_config="file:$sdir/../../resources/log4j2-console.xml" solr_home="$sdir/../../solr" +# Ge

Re: [I] Failed to pull zookeeper image from private registry [solr-operator]

2023-11-28 Thread via GitHub
davidl789 commented on issue #621: URL: https://github.com/apache/solr-operator/issues/621#issuecomment-1830980343 No worries, thanks for looking into it. I run describe pod please see output below. I only see some mount point for the secret "/var/run/secrets/kubernetes.io/serviceaccount"

Re: [PR] SOLR-16995: Associate with each replica type a property for "numReplicas" [solr]

2023-11-28 Thread via GitHub
murblanc commented on code in PR #2039: URL: https://github.com/apache/solr/pull/2039#discussion_r1408434915 ## solr/core/src/java/org/apache/solr/cloud/api/collections/CreateCollectionCmd.java: ## @@ -97,9 +94,12 @@ public class CreateCollectionCmd implements CollApiCmds.Colle

Re: [PR] SOLR-16743: Auto reload keystore/truststore on change [solr]

2023-11-28 Thread via GitHub
tflobbe commented on code in PR #2100: URL: https://github.com/apache/solr/pull/2100#discussion_r1408524274 ## solr/core/src/java/org/apache/solr/handler/component/HttpShardHandlerFactory.java: ## @@ -279,12 +282,20 @@ public void init(PluginInfo info) { int soTimeout =

Re: [PR] SOLR-17036: Lazy create VersionBucket in the update log to free memory. [solr]

2023-11-28 Thread via GitHub
dsmiley commented on code in PR #2021: URL: https://github.com/apache/solr/pull/2021#discussion_r1408490861 ## solr/core/src/java/org/apache/solr/update/processor/DistributedUpdateProcessor.java: ## @@ -507,29 +504,19 @@ private boolean doVersionAdd( lastVer

Re: [PR] JWTAuthPluginIntegrationTest - dynamically generate the self-signed certificate using the localhost loopback address [solr]

2023-11-28 Thread via GitHub
jdyer1 commented on PR #2099: URL: https://github.com/apache/solr/pull/2099#issuecomment-1830832210 @janhoy On the plus side, all checks are now passing. On the minus side, I went ahead and mapped 127.0.0.1 to "localhost.localdomain" on my /etc/hosts file, and sadly the test still fails w

[jira] [Comment Edited] (SOLR-17060) CoreContainer#create may deadlock with concurrent requests for metrics

2023-11-28 Thread Alex Deparvu (Jira)
[ https://issues.apache.org/jira/browse/SOLR-17060?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17790775#comment-17790775 ] Alex Deparvu edited comment on SOLR-17060 at 11/28/23 10:02 PM: ---

[jira] [Commented] (SOLR-17060) CoreContainer#create may deadlock with concurrent requests for metrics

2023-11-28 Thread Alex Deparvu (Jira)
[ https://issues.apache.org/jira/browse/SOLR-17060?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17790775#comment-17790775 ] Alex Deparvu commented on SOLR-17060: - I took a stab at a proposal for a fix https://

[PR] SOLR-17060 CoreContainer#create may deadlock with concurrent requests for metrics [solr]

2023-11-28 Thread via GitHub
stillalex opened a new pull request, #2101: URL: https://github.com/apache/solr/pull/2101 https://issues.apache.org/jira/browse/SOLR-17060 # Description Posting a proposal for SOLR-17060, I am not entirely sure this will work correctly. The approach is to identif

[jira] [Commented] (SOLR-11629) CloudSolrClient.Builder should accept a zk host

2023-11-28 Thread Dario Viva (Jira)
[ https://issues.apache.org/jira/browse/SOLR-11629?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17790764#comment-17790764 ] Dario Viva commented on SOLR-11629: --- [~varun] I know I am kind of late to the party, bu

Re: [PR] SOLR-17084: Don't return full list of zombie in Exception [solr]

2023-11-28 Thread via GitHub
dsmiley commented on PR #2097: URL: https://github.com/apache/solr/pull/2097#issuecomment-1830713415 Cool; can you please add a CHANGES.txt entry to the 9.5 section... I suppose as a Bug but I could also see this as an Improvement or Optimization if you prefer. It's sort of a "scale bug" b

[jira] [Resolved] (SOLR-17078) Update all Python scripts to Python3

2023-11-28 Thread Jason Gerlowski (Jira)
[ https://issues.apache.org/jira/browse/SOLR-17078?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Jason Gerlowski resolved SOLR-17078. Fix Version/s: main (10.0) 9.5 Resolution: Fixed > Update all Pyt

[jira] [Commented] (SOLR-17078) Update all Python scripts to Python3

2023-11-28 Thread ASF subversion and git services (Jira)

Re: [PR] SOLR-16743: Auto reload keystore/truststore on change [solr]

2023-11-28 Thread via GitHub
HoustonPutman commented on code in PR #2100: URL: https://github.com/apache/solr/pull/2100#discussion_r1408369327 ## solr/core/src/java/org/apache/solr/handler/component/HttpShardHandlerFactory.java: ## @@ -279,12 +282,20 @@ public void init(PluginInfo info) { int soTimeout

[jira] [Commented] (SOLR-17078) Update all Python scripts to Python3

2023-11-28 Thread ASF subversion and git services (Jira)
[ https://issues.apache.org/jira/browse/SOLR-17078?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17790737#comment-17790737 ] ASF subversion and git services commented on SOLR-17078: Commit 3

Re: [PR] SOLR-17078: Upgrade ltr script to use Python3 [solr]

2023-11-28 Thread via GitHub
gerlowskija merged PR #2077: URL: https://github.com/apache/solr/pull/2077 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: issues-unsubscr...@solr.apach

[PR] SOLR-16743: Auto reload keystore/truststore on change [solr]

2023-11-28 Thread via GitHub
tflobbe opened a new pull request, #2100: URL: https://github.com/apache/solr/pull/2100 Marking this as a draft for now, since it's missing the documentation changes. Also, I wasn't able to test the Windows start script changes -- This is an automated message from the Apache Git Service.

Re: [PR] SOLR-15857: Add AWS Secret Manager support for ZK ACL credentials [solr]

2023-11-28 Thread via GitHub
laminelam commented on code in PR #1994: URL: https://github.com/apache/solr/pull/1994#discussion_r1408284705 ## solr/server/scripts/cloud-scripts/zkcli.sh: ## @@ -23,6 +38,10 @@ solr_home="$sdir/../../solr" #... # -DzkDigestCredentialsFile=/path/to/zkDigestCredentialsFile.p

Re: [PR] SOLR-15857: Add AWS Secret Manager support for ZK ACL credentials [solr]

2023-11-28 Thread via GitHub
laminelam commented on code in PR #1994: URL: https://github.com/apache/solr/pull/1994#discussion_r1408273641 ## solr/server/scripts/cloud-scripts/zkcli.sh: ## @@ -13,6 +13,21 @@ log4j_config="file:$sdir/../../resources/log4j2-console.xml" solr_home="$sdir/../../solr" +# Ge

Re: [PR] SOLR-15857: Add AWS Secret Manager support for ZK ACL credentials [solr]

2023-11-28 Thread via GitHub
laminelam commented on code in PR #1994: URL: https://github.com/apache/solr/pull/1994#discussion_r1408273641 ## solr/server/scripts/cloud-scripts/zkcli.sh: ## @@ -13,6 +13,21 @@ log4j_config="file:$sdir/../../resources/log4j2-console.xml" solr_home="$sdir/../../solr" +# Ge

[jira] [Updated] (SOLR-17088) TestPrepRecovery.testLeaderNotResponding fails much more lately

2023-11-28 Thread David Smiley (Jira)
[ https://issues.apache.org/jira/browse/SOLR-17088?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] David Smiley updated SOLR-17088: Attachment: 2023-11-27 fail.log.txt Status: Open (was: Open) > TestPrepRecovery.testLeader

Re: [PR] SOLR-17083: Make CoreSorter class configurable [solr]

2023-11-28 Thread via GitHub
dsmiley commented on PR #2088: URL: https://github.com/apache/solr/pull/2088#issuecomment-1830417075 I filed https://issues.apache.org/jira/browse/SOLR-17087 for the kerberos test flapping failure. I filed https://issues.apache.org/jira/browse/SOLR-17088 for the TestPrepRecovery test fla

Re: [PR] SOLR-17058: Request param to disable distributed stats request at query time [solr]

2023-11-28 Thread via GitHub
weiwang19 commented on code in PR #2046: URL: https://github.com/apache/solr/pull/2046#discussion_r1408158000 ## solr/solr-ref-guide/modules/deployment-guide/pages/solrcloud-distributed-requests.adoc: ## @@ -445,6 +445,21 @@ For example, the following line makes Solr use the `E

Re: [PR] SOLR-17083: Make CoreSorter class configurable [solr]

2023-11-28 Thread via GitHub
pvcnt commented on PR #2088: URL: https://github.com/apache/solr/pull/2088#issuecomment-1830384213 @dsmiley I was thinking: do you think that a `` element should be preferred? On the one hand it might be more consistent with how most plugins, but on the other hand overriding this class migh

Re: [PR] JWTAuthPluginIntegrationTest - dynamically generate the self-signed certificate using the localhost loopback address [solr]

2023-11-28 Thread via GitHub
janhoy commented on PR #2099: URL: https://github.com/apache/solr/pull/2099#issuecomment-1830366870 The other test failure looks like some Java security manager thing; ``` 2> 3803 INFO (SUITE-JWTAuthPluginIntegrationTest-seed#[F234B11BA9F49CDF]-worker) [n: c: s: r: x: t:] o.a.s

Re: [PR] JWTAuthPluginIntegrationTest - dynamically generate the self-signed certificate using the localhost loopback address [solr]

2023-11-28 Thread via GitHub
janhoy commented on PR #2099: URL: https://github.com/apache/solr/pull/2099#issuecomment-1830360446 Regarding the `usedUndeclaredArtifacts` you can add a special entry in grade file for it, look in dev-docs/ for details. -- This is an automated message from the Apache Git Service. To resp

Re: [PR] JWTAuthPluginIntegrationTest - dynamically generate the self-signed certificate using the localhost loopback address [solr]

2023-11-28 Thread via GitHub
janhoy commented on code in PR #2099: URL: https://github.com/apache/solr/pull/2099#discussion_r1408138396 ## solr/modules/jwt-auth/src/test/org/apache/solr/security/jwt/JWTAuthPluginIntegrationTest.java: ## @@ -116,6 +122,12 @@ public static void afterClass() throws Exception {

Re: [PR] SOLR-17084: Don't return full list of zombie in Exception [solr]

2023-11-28 Thread via GitHub
gbellaton commented on PR #2097: URL: https://github.com/apache/solr/pull/2097#issuecomment-1830354595 I agree. I changed the PR accordingly. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the s

Re: [PR] SOLR-17058: Request param to disable distributed stats request at query time [solr]

2023-11-28 Thread via GitHub
weiwang19 commented on code in PR #2046: URL: https://github.com/apache/solr/pull/2046#discussion_r1408099532 ## solr/core/src/test/org/apache/solr/search/stats/TestDistribIDF.java: ## @@ -263,4 +267,51 @@ private void addDocsRandomly() throws IOException, SolrServerException {

Re: [PR] SOLR-17058: Request param to disable distributed stats request at query time [solr]

2023-11-28 Thread via GitHub
weiwang19 commented on code in PR #2046: URL: https://github.com/apache/solr/pull/2046#discussion_r1408097780 ## solr/core/src/test/org/apache/solr/search/stats/TestDistribIDF.java: ## @@ -263,4 +267,51 @@ private void addDocsRandomly() throws IOException, SolrServerException {

Re: [PR] SOLR-17058: Request param to disable distributed stats request at query time [solr]

2023-11-28 Thread via GitHub
weiwang19 commented on code in PR #2046: URL: https://github.com/apache/solr/pull/2046#discussion_r1408098257 ## solr/core/src/java/org/apache/solr/handler/component/QueryComponent.java: ## @@ -730,8 +732,11 @@ protected void regularFinishStage(ResponseBuilder rb) { protect

Re: [PR] SOLR-17058: Request param to disable distributed stats request at query time [solr]

2023-11-28 Thread via GitHub
weiwang19 commented on code in PR #2046: URL: https://github.com/apache/solr/pull/2046#discussion_r1408096714 ## solr/solrj/src/java/org/apache/solr/common/params/CommonParams.java: ## @@ -289,6 +289,14 @@ public static EchoParamStyle get(String v) { @Deprecated(since = "9.4"

Re: [PR] SOLR-15857: Add AWS Secret Manager support for ZK ACL credentials [solr]

2023-11-28 Thread via GitHub
epugh commented on code in PR #1994: URL: https://github.com/apache/solr/pull/1994#discussion_r1408096593 ## solr/server/scripts/cloud-scripts/zkcli.sh: ## @@ -13,6 +13,21 @@ log4j_config="file:$sdir/../../resources/log4j2-console.xml" solr_home="$sdir/../../solr" +# Get so

Re: [I] Solr operator going in CrashLoopBackOff state [solr-operator]

2023-11-28 Thread via GitHub
HoustonPutman closed issue #41: Solr operator going in CrashLoopBackOff state URL: https://github.com/apache/solr-operator/issues/41 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comme

Re: [I] Solr operator going in CrashLoopBackOff state [solr-operator]

2023-11-28 Thread via GitHub
HoustonPutman commented on issue #41: URL: https://github.com/apache/solr-operator/issues/41#issuecomment-1830296859 Please go to https://solr.apache.org/operator/resources.html for more information. I don't know where that error is coming from, but it's not ours. We have other discus

Re: [I] Solr operator going in CrashLoopBackOff state [solr-operator]

2023-11-28 Thread via GitHub
HoustonPutman closed issue #41: Solr operator going in CrashLoopBackOff state URL: https://github.com/apache/solr-operator/issues/41 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comme

Re: [PR] JWTAuthPluginIntegrationTest - dynamically generate the self-signed certificate using the localhost loopback address [solr]

2023-11-28 Thread via GitHub
jdyer1 commented on PR #2099: URL: https://github.com/apache/solr/pull/2099#issuecomment-1830290767 @epugh Thank you for your review. I expected IntelliJ to automatically use the project's formatting preferences but it does not seem to be the case. But `gradlew tidy` did the trick.

Re: [I] Full Solr Cloud Initialization (ConfigSet, Collections, Alias, ...) [solr-operator]

2023-11-28 Thread via GitHub
HoustonPutman commented on issue #587: URL: https://github.com/apache/solr-operator/issues/587#issuecomment-1830291570 So the only real way to deal with this, if you can only use `lifecycle.podstart`, is to add a wait in there. The solr operator will set up the pods such that the `readiness

Re: [I] Solr operator going in CrashLoopBackOff state [solr-operator]

2023-11-28 Thread via GitHub
chaman53 commented on issue #41: URL: https://github.com/apache/solr-operator/issues/41#issuecomment-1830283682 Hi Team, I tried with below repository: apache/solr-operator # solr.image.tag -- Tag for the SOLR image tag: v0.8.0 and getting error exec /scripts/setup.s

Re: [I] Can not use the collections API to backup to S3 [solr-operator]

2023-11-28 Thread via GitHub
HoustonPutman commented on issue #404: URL: https://github.com/apache/solr-operator/issues/404#issuecomment-1830279963 This is a new feature, as Solr does not support this (and is unrelated to the Solr Operator). Please use the [Solr Users Mailing list](https://solr.apache.org/community.htm

Re: [I] Adding support to dynamic value in values.yaml [solr-operator]

2023-11-28 Thread via GitHub
HoustonPutman commented on issue #661: URL: https://github.com/apache/solr-operator/issues/661#issuecomment-1830259188 Is this common for other Helm charts? I understand how it could be useful, but it would add a lot of complexity to the already fairly complex helm chart. -- This i

Re: [I] Solr operator going in CrashLoopBackOff state [solr-operator]

2023-11-28 Thread via GitHub
chaman53 commented on issue #41: URL: https://github.com/apache/solr-operator/issues/41#issuecomment-1830255772 Hi Apache/Solr-Operator I am new on this please suggest which repository should I use and what would be the tag version On Tue, 28 Nov 2023 at 11:10 AM, Houston Put

Re: [I] Failed to pull zookeeper image from private registry [solr-operator]

2023-11-28 Thread via GitHub
HoustonPutman commented on issue #621: URL: https://github.com/apache/solr-operator/issues/621#issuecomment-1830252947 Sorry for the late response here. When you do this, and look at the `ZookeeperCluster` or `Pod`. (i.e. `kubectl describe zookeepercluster`), does it show an imagePullSecret

Re: [I] Solr startup Probe failure - Node IP used [solr-operator]

2023-11-28 Thread via GitHub
HoustonPutman commented on issue #623: URL: https://github.com/apache/solr-operator/issues/623#issuecomment-1830243306 Sorry for the late response. The solr operator didn't set an ip address for the pod startup probes, so it must have been a kubernetes thing. If you are still seeing t

Re: [I] Unable to add additional PVCs to Solr Pods [solr-operator]

2023-11-28 Thread via GitHub
HoustonPutman commented on issue #622: URL: https://github.com/apache/solr-operator/issues/622#issuecomment-1830234865 Hmmm, Yeah I don't think we support more than one PVC right now. You can try adding it, PRs are always welcome. Why are you looking to store logs on persistent volume

Re: [I] Release for new solr-operator with latest ZooKeeper dependency [solr-operator]

2023-11-28 Thread via GitHub
HoustonPutman commented on issue #657: URL: https://github.com/apache/solr-operator/issues/657#issuecomment-1830230445 I'm not sure where banzaicloud's zookeeper operator comes from, but the one that Solr relies on is https://github.com/pravega/zookeeper-operator. The latest release of the

Re: [I] Solr operator going in CrashLoopBackOff state [solr-operator]

2023-11-28 Thread via GitHub
HoustonPutman commented on issue #41: URL: https://github.com/apache/solr-operator/issues/41#issuecomment-1830170979 This is a long closed issue, and had to do with the **solr operator** crashing. You are using the bitnami Solr helm chart, completely unrelated to this project and an unoffic

Re: [PR] SOLR-15857: Add AWS Secret Manager support for ZK ACL credentials [solr]

2023-11-28 Thread via GitHub
HoustonPutman commented on code in PR #1994: URL: https://github.com/apache/solr/pull/1994#discussion_r1408013410 ## solr/modules/aws-secret-provider/src/java/org/apache/solr/secret/zk/AWSSecretManagerCredentialsInjector.java: ## @@ -0,0 +1,144 @@ +/* + * Licensed to the Apache

Re: [I] Solr operator going in CrashLoopBackOff state [solr-operator]

2023-11-28 Thread via GitHub
chaman53 commented on issue #41: URL: https://github.com/apache/solr-operator/issues/41#issuecomment-1830144990 solr 15:53:01.84 solr 15:53:01.84 Welcome to the Bitnami solr container solr 15:53:01.84 Subscribe to project updates by watching https://github.com/bitnami/bitnami-docker-s

[jira] [Created] (SOLR-17087) TestSolrCloudWithKerberosAlt.testBasics fails because address in use

2023-11-28 Thread David Smiley (Jira)
David Smiley created SOLR-17087: --- Summary: TestSolrCloudWithKerberosAlt.testBasics fails because address in use Key: SOLR-17087 URL: https://issues.apache.org/jira/browse/SOLR-17087 Project: Solr

Re: [PR] JWTAuthPluginIntegrationTest - dynamically generate the self-signed certificate using the localhost loopback address [solr]

2023-11-28 Thread via GitHub
epugh commented on code in PR #2099: URL: https://github.com/apache/solr/pull/2099#discussion_r1407862073 ## solr/modules/jwt-auth/src/test/org/apache/solr/security/jwt/JWTAuthPluginIntegrationTest.java: ## @@ -25,16 +25,14 @@ import java.io.InputStreamReader; import java.io.O

Re: [PR] SOLR-17036: Lazy create VersionBucket in the update log to free memory. [solr]

2023-11-28 Thread via GitHub
bruno-roustant commented on PR #2021: URL: https://github.com/apache/solr/pull/2021#issuecomment-1829889063 I'm going to merge this in 2 days. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the

Re: [PR] Add support for optional DBQ expansion. [solr-sandbox]

2023-11-28 Thread via GitHub
sigram merged PR #91: URL: https://github.com/apache/solr-sandbox/pull/91 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: issues-unsubscr...@solr.apache

[jira] [Updated] (SOLR-17086) v2 Backup and Replica creation: HTTP 500 error messages

2023-11-28 Thread Jira
[ https://issues.apache.org/jira/browse/SOLR-17086?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Jan Høydahl updated SOLR-17086: --- Component/s: Backup/Restore > v2 Backup and Replica creation: HTTP 500 error messages > -

[I] Adding support to dynamic value in values.yaml [solr-operator]

2023-11-28 Thread via GitHub
idolaman opened a new issue, #661: URL: https://github.com/apache/solr-operator/issues/661 First of all this project is amazing! I encountered an issue when i tried to use this chart as an dependency to my chart. in my use case i want to be able to give a dynamic value in value

[jira] [Created] (SOLR-17086) v2 Backup and Replica creation: HTTP 500 error messages

2023-11-28 Thread Jira
Hakan Özler created SOLR-17086: -- Summary: v2 Backup and Replica creation: HTTP 500 error messages Key: SOLR-17086 URL: https://issues.apache.org/jira/browse/SOLR-17086 Project: Solr Issue Type:

Re: [PR] JWTAuthPluginIntegrationTest - dynamically generate the self-signed certificate using the localhost loopback address [solr]

2023-11-28 Thread via GitHub
janhoy commented on code in PR #2099: URL: https://github.com/apache/solr/pull/2099#discussion_r1407459703 ## solr/modules/jwt-auth/src/test/org/apache/solr/security/jwt/JWTAuthPluginIntegrationTest.java: ## @@ -116,6 +122,12 @@ public static void afterClass() throws Exception {