Skip to content
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.

Sign up for GitHub

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

Jump to bottom

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

Closed
tabalt opened this issue Jun 30, 2020 · 5 comments · Fixed by #219
Closed

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

tabalt opened this issue Jun 30, 2020 · 5 comments · Fixed by #219

Comments

@tabalt
Copy link
Contributor

tabalt commented Jun 30, 2020 •
edited
Loading

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 tabalt changed the title param pulsar_manager.admin.user/pulsar_manager.admin.password used in the wrong place Param pulsar_manager.admin.user/pulsar_manager.admin.password used in the wrong place Jun 30, 2020
@sijie
Copy link
Member

sijie commented Jul 2, 2020

@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
Copy link
Member

sijie commented Jul 2, 2020

/cc @tuteng

@vkvm
Copy link

vkvm commented Dec 12, 2020

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: apache/pulsar-manager#297 . They do mention a workaround (which I haven't tried).

This was referenced Mar 16, 2021
Closed
Closed
lhotari added a commit to lhotari/pulsar-helm-chart that referenced this issue Jan 26, 2022
@lhotari
Fix setting pulsar manager credentials and disable pulsar manager by …
2b9e5b2 
…default

Fixes apache#31
Fixes apache#108

- disable pulsar_manager by default because of security reasons

- set properties compatible with pulsar-manager v0.2.0,
  https://github.com/apache/pulsar-manager/blob/v0.2.0/src/main/resources/application.properties#L87-L91

- remove obsolete environment variables
@lhotari lhotari mentioned this issue Jan 26, 2022
Closed
@lhotari
Copy link
Member

lhotari commented Jan 26, 2022

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

@csthomas1 csthomas1 mentioned this issue Jan 27, 2022
Merged
1 task
pgier pushed a commit to pgier/pulsar-helm-chart that referenced this issue Apr 22, 2022
@michaeljmarshall
Fix retreival of extraAuthProvider (apache#31)
79371af
rdhabalia added a commit to rdhabalia/pulsar-helm-chart that referenced this issue Feb 2, 2023
@rdhabalia
Fix: healthcheck metrics with broker/bookie and expose to prometheus (a…
1defe00 
…pache#31)

Fix: healthcheck metrics with broker and expose to prometheus

Co-authored-by: rajan <[email protected]>
@lhotari
Copy link
Member

lhotari commented Feb 14, 2024

Issue should be resolved by #457

@lhotari lhotari closed this as completed Feb 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
4 participants
@lhotari @sijie @tabalt @vkvm