2018-07-18 10:04:33 UTC - Matteo Merli: @Tomer Lev curl is not following the 
redirect to `<http://localhost:8080/metrics/>` (though prometheus follows that 
correctly). 

For ZK, the default HTTP port for metrics is 8080
----
2018-07-18 10:19:32 UTC - Tomer Lev: @Matteo Merli Thanks, For ZK, 8080 is not 
open as well
----
2018-07-18 13:58:03 UTC - Grant Wu: 
<https://pulsar.incubator.apache.org/docs/latest/admin-api/persistent-topics/#Resetcursor-t6e6o9>
 What are all of these `:destination` parameters?
----
2018-07-18 15:46:06 UTC - Sijie Guo: @Grant Wu the destination is the “topic”. 
in the example of “<persistent://test-tenant/ns1/tp1>", tenant is test-tenant, 
namespace is ns1 and destination is tp1
----
2018-07-18 15:50:44 UTC - Grant Wu: Uh, yes.
----
2018-07-18 15:50:52 UTC - Grant Wu: So should we change the terminology here to 
“topic”
----
2018-07-18 15:56:53 UTC - Sijie Guo: yes. it should be “topic”. I think in the 
code, we used “topic” all over the places. however somehow in the documentation 
we used “destination”, which is a bit strange. 

do you mind filing an issue for that? if you want to contribute a PR, that 
would be great as well :slightly_smiling_face:
----
2018-07-18 15:58:26 UTC - Grant Wu: Yeah, I mean, I already have one open PR, I 
might as well
----
2018-07-18 16:02:25 UTC - Grant Wu: Actually, @Sijie Guo this touches many of 
the same files my previous one touches
----
2018-07-18 16:02:49 UTC - Grant Wu: Should I add this onto my previous PR, or 
just wait
----
2018-07-18 16:04:33 UTC - Sijie Guo: it is better to do it in a separate PR
----
2018-07-18 16:04:59 UTC - Sijie Guo: I will check your status of your current 
PR. I think it is approved, just waiting for being merged.
----
2018-07-18 16:05:10 UTC - Grant Wu: I actually have an unresolved question
----
2018-07-18 16:06:02 UTC - Grant Wu: 
<https://github.com/apache/incubator-pulsar/pull/2174/files#r202833084>
----
2018-07-18 16:06:07 UTC - Grant Wu: actually, I can just branch off of my 
branch for now
----
2018-07-18 16:06:27 UTC - Sijie Guo: that’s great
----
2018-07-18 16:07:18 UTC - Grant Wu: Also in the future I’ll move this to 
<#C5ZSVEN4E|dev>
+1 : Sijie Guo, Matteo Merli
----
2018-07-18 16:22:06 UTC - Matteo Merli: @Grant Wu just replied. missed that 
yesterday
+1 : Grant Wu
----
2018-07-18 17:45:10 UTC - Adam Williams: Was wondering if anyone has seen 
similar errors when trying to grant permissions on a namespace? (version 1.22)
```
$ ./bin/pulsar-admin --admin-url <https://server:8443> namespaces 
grant-permission a/b/c --actions produce,consume --role admin
HTTP 500 Request failed.                                                        
               
                                                       
Reason: Some error occourred on the server
```
----
2018-07-18 17:46:58 UTC - Adam Williams: The server log contains a NPE, with 
this possibly relevant bit:
```
Caused by: java.lang.NullPointerException
        at 
org.apache.pulsar.broker.admin.Namespaces.grantPermissionOnNamespace(Namespaces.java:458)
```
----
2018-07-18 18:12:42 UTC - Matteo Merli: @Adam Williams which version is the 
broker running? (to match the line numbers)
----
2018-07-18 18:14:44 UTC - Adam Williams: @Matteo Merli The broker is running 
`v1.22.0-incubating`
----
2018-07-18 18:23:41 UTC - Matteo Merli: I see, the problem seems to be that 
authorization is disabled in the cluster (, and therefore the authorization 
plugin is not loaded.
(This should be improved to handle better this case and give proper error)
----
2018-07-18 18:24:14 UTC - Matteo Merli: in `broker.conf` : 
`authorizationEnabled=true`
----
2018-07-18 18:25:41 UTC - Adam Williams: Thanks @Matteo Merli, we'll look into 
that.
----

Reply via email to