-
Notifications
You must be signed in to change notification settings - Fork 24
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
Change the configuration format to include full location #751
Comments
I added some comments in the proposed code. Did you check with the config files from the properitery devices that we need the wireless_devices in this way? This issue is related to #370 |
Well, we need some way to know the band and we also need to know if it is a dual band device, so probably a good idea to just keep the syntax we already have and build on that. |
It would be cool if a config file for the ubiquiti devices were also generated. There isn't much to the ubnt config files and on a new device (or after reset) on the first landing page it's possible to upload a config file. |
thats the plan... ;-) |
Another aspect would be to move the gateway config over to our default format. I think it should be manageable for everything that is not gateway specific as we allready have a way of describing mesh, management and uplink interfaces. |
I added a |
EDIT: This was decided on our community day February 2024.
I talked with @FFHener about ways to further standardize the configuration format. We came up with the following ideas:
Doing so will allow us to move some parts of the configuration into the new model files:
Further it will allow us to remove some lines from the configurations:
In addition this will allow us to:
Here are examples of how a model file for a non OpenWRT device could look like:
The host configuration for such an device could look like this:
Channel assignments for non OpenWRT devices could be done alongside the other channel assignments:
We want to discuss this at the community day. Please let us know about any thoughs you have on this.
The text was updated successfully, but these errors were encountered: