search

openwisp / docker-openwisp

OpenWISP in docker. For production usage we recommend using the ansible-openwisp2 role.
https://openwisp.io/docs/dev/docker/
BSD 3-Clause "New" or "Revised" License
149 stars 75 forks source link

[change!] Changed the command to delete_old_radiusbatch_users #294 #295

Closed praptisharma28 closed 4 months ago

praptisharma28 commented 5 months ago

Updated the command. Related to openwisp/openwisp-radius#502 .This change ensures proper functionality and maintains consistency.

Closes #294

praptisharma28 commented 5 months ago

Screenshot from 2024-03-09 20-13-02

Screenshot from 2024-03-09 20-12-50 Attached the screenshots too @nemesifier @pandafy

praptisharma28 commented 5 months ago

@nemesifier what corrections are needed to be done here now?

nemesifier commented 5 months ago

@nemesifier what corrections are needed to be done here now?

Nothing here, just the build on master seems failing, we'll need to resolve that before merging.

pandafy commented 5 months ago

@praptisharma28 From the initial impression it looks like a python dependency issue. Look a this line in the logs https://github.com/openwisp/docker-openwisp/pull/295/checks#step:6:1391

praptisharma28 commented 5 months ago

@praptisharma28 From the initial impression it looks like a python dependency issue. Look a this line in the logs https://github.com/openwisp/docker-openwisp/pull/295/checks#step:6:1391

Yeah, u r right the error message indicates that the need for netaddr~=0.8 or netaddr~=0.8.0 is not fulfilled, but idk how to resolve it. @pandafy