pulsar-helm-chart icon indicating copy to clipboard operation
pulsar-helm-chart copied to clipboard

Param pulsar_manager.admin.user/pulsar_manager.admin.password used in the wrong place

Open tabalt opened this issue 4 years ago • 4 comments

Describe the bug I am try to set pulsar_manager.admin.user/pulsar_manager.admin.password for login pulsar-manager ui, But it's actually used as the account/password for outside PostgreSQL instance.

To Reproduce helm upgrade --install pulsar charts/pulsar --set pulsar_manager.admin.user=pulsarpulsar --set pulsar_manager.admin.password=pulsarpulsar -n pulsar

org.postgresql.util.PSQLException: FATAL: role "pulsarpulsar" does not exist
	at org.postgresql.core.v3.QueryExecutorImpl.receiveErrorResponse(QueryExecutorImpl.java:2440) ~[postgresql-42.2.5.jar:42.2.5]
	at org.postgresql.core.v3.QueryExecutorImpl.readStartupMessages(QueryExecutorImpl.java:2559) ~[postgresql-42.2.5.jar:42.2.5]
	at org.postgresql.core.v3.QueryExecutorImpl.<init>(QueryExecutorImpl.java:133) ~[postgresql-42.2.5.jar:42.2.5]
	at org.postgresql.core.v3.ConnectionFactoryImpl.openConnectionImpl(ConnectionFactoryImpl.java:250) ~[postgresql-42.2.5.jar:42.2.5]
	at org.postgresql.core.ConnectionFactory.openConnection(ConnectionFactory.java:49) ~[postgresql-42.2.5.jar:42.2.5]
	at org.postgresql.jdbc.PgConnection.<init>(PgConnection.java:195) ~[postgresql-42.2.5.jar:42.2.5]
	at org.postgresql.Driver.makeConnection(Driver.java:454) ~[postgresql-42.2.5.jar:42.2.5]

tabalt avatar Jun 30 '20 09:06 tabalt

@tabalt I think the current pulsar-manager version doesn't support changing the login user and password very well. We are releasing pulsar-manager v0.2.0. It will support setting user and password.

sijie avatar Jul 02 '20 06:07 sijie

/cc @tuteng

sijie avatar Jul 02 '20 06:07 sijie

I notice v0.2.0 of pulsar-manager was released, and I tried it a couple of weeks ago. I don't think it supports this yet. The issue appears to be tracked here: https://github.com/apache/pulsar-manager/issues/297 . They do mention a workaround (which I haven't tried).

vkvm avatar Dec 12 '20 22:12 vkvm

Please review PR #212, it should fix this issue.

lhotari avatar Jan 26 '22 08:01 lhotari

Issue should be resolved by #457

lhotari avatar Feb 14 '24 18:02 lhotari