-
Notifications
You must be signed in to change notification settings - Fork 17
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.
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
utm #405
base: develop
Are you sure you want to change the base?
utm #405
Conversation
Traceback (most recent call last): File "/opt/nats/comms_nats_controller.py", line 1498, in main_mdm results = await asyncio.gather(mdm.execute()) File "/opt/nats/comms_nats_controller.py", line 324, in execute self.setup_cbma() File "/opt/nats/comms_nats_controller.py", line 1193, in setup_cbma bridge_setting = self.__comms_ctrl.settings.bridge[0] IndexError: list index out of range with mdm_agent standalone run
@DaniilTroshkovTII, still in draft? |
Hello @DamienDeMayaTII I would prefer to merge this changes but I am not sure if @govindsi planning to merge it. In my view we can afford this small overhead of requesting MDM server for UTM config - for example if server do not support it (returns 405) MDM agent can just ignore it (make polling time s equal to OK polling time which is 10 mins). So this overhead request will only appears once in 10 minutes. From my view it will be easy to support this additional UTM functionality in MDM agent. |
@DaniilTroshkovTII , please check with Seb and kindly decide to merge or cancel. Thanks! |
@DaniilTroshkovTII , any update? |
This will not land in develop as this is meant for staging feature branch. After MVP, we will only have lte only MDM config for UTM specific use case. |
UTM config support