Skip to content
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

Handling removed default parameters #237

Open
jorgenfb opened this issue Jan 31, 2019 · 3 comments
Open

Handling removed default parameters #237

jorgenfb opened this issue Jan 31, 2019 · 3 comments

Comments

@jorgenfb
Copy link
Contributor

We have some parameters previously stored in default_path directory. Now we want to remove some of them from the config manager and instead set them in our launch files. The problem is that the config_manager still applies the old defaults on the parameters server.

How do I handle this properly assuming I don't have access to the database on all installations. The only workaround I can think of is renaming my parameters, but that will still leave the old ones on the parameter server, which may be very confusing.

@hawesie
Copy link
Member

hawesie commented Feb 1, 2019

A good question. Adding @Jailander as I think he's the one who bests understands this functionality. (Jaime, add someone else if not!)

That said, if your nodes can read the parameters from the store, you must have database access in some form, so it should be possible to script something to remove these old database entries.

@jorgenfb
Copy link
Contributor Author

jorgenfb commented Feb 1, 2019

I do have database access, I just want to have your opinion on how to handle removed defaults. If there is some missing functionality I might be able to provide it in an PR if we can agree on the desired behavior.

I think that at least the default value stored in the database should be ignored. I'm not sure what to do with any local override as there is possible to set local configs even for parameters without a default.

Why is defaults stored in the database anyway? Aren't they read from file anyway?

@jorgenfb
Copy link
Contributor Author

I would basically read all the defaults from the database and see if any of them is no longer read from default files, then delete the from the database itself.

I'm not really sure how to handle local config. I would probably still allow to have local config without a default. With #248 we already have a service for that can be used to reset the local values.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants