docker-jitsi-meet icon indicating copy to clipboard operation
docker-jitsi-meet copied to clipboard

relayId coming as 0.0.0.0 from after stable-7287-2 jvb images

Open shuhaib3 opened this issue 2 years ago • 6 comments

Hi @saghul ,

After jvb release 7287-2 ie, in stable-7439 and stable-7439-2. the relayid shown in jicofo logs is as 0.0.0.0. image I think the relay id is being picked up from here in the last two releases. when I change the bind address the relay id in jicofo is changing. Any way to change the relayid to JVB_OCTO_PUBLIC_ADDRESS?

shuhaib3 avatar Jul 22 '22 17:07 shuhaib3

I honestly have no idea :-/ @bgrozev can you enlighten us here?

saghul avatar Jul 22 '22 17:07 saghul

@bgrozev is this might be the issue https://github.com/jitsi/jitsi-videobridge/commit/26085c6fa0f16de558e2b66f3a82603dcde7b62b

shuhaib3 avatar Jul 22 '22 20:07 shuhaib3

@bgrozev even if relayid is there it is getting a fallback to bind address

shuhaib3 avatar Jul 22 '22 20:07 shuhaib3

Sorry, I'm not sure what breaks, but it's best to simply update to the latest config scheme. Something like:

{{ if $ENABLE_OCTO -}}
relay {
  enabled = true
  relay-id="<RELAY-ID>"
  region = "{{ .Env.JVB_OCTO_REGION | default "europe" }}"
}
{{ end -}}

I'm not sure what the best way to set the ID in a docker setup is. It just needs to be unique in the set of jvb's connected to jicofo. Taking $JVB_MUC_NICKNAME might work, or whatever was used to set bind-address.

We can get rid of .Env.JVB_OCTO_BIND_ADDRESS, .Env.JVB_OCTO_PUBLIC_ADDRESS, and .Env.JVB_OCTO_BIND_PORT.

bgrozev avatar Jul 22 '22 20:07 bgrozev

image ok, but here what is happening is that in last two releases the bind address is taken as relayid. hence the default value is 0.0.0.0 it is not getting unique. previously it used to take the relay id from public-address = "{{ .Env.JVB_OCTO_PUBLIC_ADDRESS }}" @saghul do we need a change here https://github.com/jitsi/docker-jitsi-meet/blob/master/jvb/rootfs/defaults/jvb.conf

shuhaib3 avatar Jul 23 '22 04:07 shuhaib3

Sorry, I'm not sure what breaks, but it's best to simply update to the latest config scheme. Something like:


{{ if $ENABLE_OCTO -}}

relay {

  enabled = true

  relay-id="<RELAY-ID>"

  region = "{{ .Env.JVB_OCTO_REGION | default "europe" }}"

}

{{ end -}}

I'm not sure what the best way to set the ID in a docker setup is. It just needs to be unique in the set of jvb's connected to jicofo. Taking $JVB_MUC_NICKNAME might work, or whatever was used to set bind-address.

We can get rid of .Env.JVB_OCTO_BIND_ADDRESS, .Env.JVB_OCTO_PUBLIC_ADDRESS, and .Env.JVB_OCTO_BIND_PORT.

Thanks I'll update the config!

saghul avatar Jul 24 '22 05:07 saghul

Continues on https://github.com/jitsi/docker-jitsi-meet/pull/1371

saghul avatar Aug 18 '22 12:08 saghul

@saghul relay id is still coming as 0.0.0.0 in latest jitsi/jvb:stable-7648-4 image version=2.2.22-g42bc1b99, relayId=0.0.0.0, region=europe, stress=0.01] with stress=0.01358 for participantRegion=ap-south-1

shuhaib3 avatar Sep 14 '22 20:09 shuhaib3

Did you set JVB_OCTO_RELAY_ID ?

saghul avatar Sep 15 '22 05:09 saghul

@shuhaib3 @saghul I don't believe these changes are included in any releases yet, I had to manually apply them for my deployment.

yoerayo avatar Sep 15 '22 05:09 yoerayo

Right, my bad! I just noticed the PR is not merged, I'll do that now.

saghul avatar Sep 15 '22 05:09 saghul