[jitsi-dev] [jitsi-videobridge] Add documentation on how to connect the focus to Jitsi Videobridge for pubsub statistics (#107)


#1

Define xmpp message flow for connecting a focus to the Jitsi Videobridge to consume statistics stanzas.
You can view, comment on, or merge this pull request online at:

  https://github.com/jitsi/jitsi-videobridge/pull/107

-- Commit Summary --

  * Add documentation on how to connect the focus to the videobridge since it doesn't behave like a publisher as defined in XEP-0060

-- File Changes --

    M doc/using_statistics.md (107)

-- Patch Links --

https://github.com/jitsi/jitsi-videobridge/pull/107.patch
https://github.com/jitsi/jitsi-videobridge/pull/107.diff

···

---
Reply to this email directly or view it on GitHub:
https://github.com/jitsi/jitsi-videobridge/pull/107


#2

@@ -134,22 +136,93 @@ Server: Jetty(9.1.5.v20140505)
}
```

+**Connecting The Focus To The Publisher For PubSub Method**
+
+Upon start of the Jitsi Videobridge, it will send the following stanza to the focus

The stanza is sent to the pubsub component, not the focus. Great docs though.

···

---
Reply to this email directly or view it on GitHub:
https://github.com/jitsi/jitsi-videobridge/pull/107/files#r47513775


#3

@@ -134,22 +136,93 @@ Server: Jetty(9.1.5.v20140505)
}
```

+**Connecting The Focus To The Publisher For PubSub Method**
+
+Upon start of the Jitsi Videobridge, it will send the following stanza to the focus

Thanks, trying to save others the time I've spent digging for answers. What do you mean by component here? In our setup, the focus scales our Videobridge cluster by monitoring the pubsub stats messages so we want the Videobridge to publish to the focus.

···

---
Reply to this email directly or view it on GitHub:
https://github.com/jitsi/jitsi-videobridge/pull/107/files#r47519384


#4

@@ -134,22 +136,93 @@ Server: Jetty(9.1.5.v20140505)
}
```

+**Connecting The Focus To The Publisher For PubSub Method**
+
+Upon start of the Jitsi Videobridge, it will send the following stanza to the focus

The videobridge creates a pubsub node on the pubsub component and then publishes updates to that node. Then the focus subscribes to that pubsub node. The stats are not sent directly to the focus

···

---
Reply to this email directly or view it on GitHub:
https://github.com/jitsi/jitsi-videobridge/pull/107/files#r47525569


#5

@@ -134,22 +136,93 @@ Server: Jetty(9.1.5.v20140505)
}
```

+**Connecting The Focus To The Publisher For PubSub Method**
+
+Upon start of the Jitsi Videobridge, it will send the following stanza to the focus

Ah, our component and focus are kind of jumbled together. I'll update the wording to be more clear. Thank you for the feedback.

···

---
Reply to this email directly or view it on GitHub:
https://github.com/jitsi/jitsi-videobridge/pull/107/files#r47530504


#6

@bgrozev thank you for the feedback, would it make sense to change it to "Third, the client must implement the client-to-mixer audio level RTP header extension (RFC6464) and the focus must forward it to the video bridge"

···

---
Reply to this email directly or view it on GitHub:
https://github.com/jitsi/jitsi-videobridge/pull/107#issuecomment-164542539


#7

LGTM

···

---
Reply to this email directly or view it on GitHub:
https://github.com/jitsi/jitsi-videobridge/pull/107#issuecomment-164548716


#8

Are there any further changes you would like to see to this documentation?

···

---
Reply to this email directly or view it on GitHub:
https://github.com/jitsi/jitsi-videobridge/pull/107#issuecomment-165231669


#9

Merged #107.

···

---
Reply to this email directly or view it on GitHub:
https://github.com/jitsi/jitsi-videobridge/pull/107#event-493955386


#10

No, it looks good. Thank you!

···

---
Reply to this email directly or view it on GitHub:
https://github.com/jitsi/jitsi-videobridge/pull/107#issuecomment-165238913