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

topology.conf parser error when using extended patterns #420

Open
thiell opened this issue Sep 13, 2019 · 2 comments
Open

topology.conf parser error when using extended patterns #420

thiell opened this issue Sep 13, 2019 · 2 comments
Labels
Milestone

Comments

@thiell
Copy link
Collaborator

thiell commented Sep 13, 2019

[routes]
sh-hn01: @service
@sherlock:1&@service: @sherlock:1&@compute
@sherlock:2&@service: @sherlock:2&@compute

results in:

clush: Unknown group source: "service"

Quotes don't help:

[routes]
"sh-hn01": "@service"
"@sherlock:1&@service": "@sherlock:1&@compute"
"@sherlock:2&@service": "@sherlock:2&@compute"
clush: Unknown group source: "service""
@thiell thiell changed the title topology.conf parser error when extended patterns topology.conf parser error when using extended patterns Sep 13, 2019
@thiell thiell added the bug label Sep 13, 2019
@thiell
Copy link
Collaborator Author

thiell commented Sep 29, 2019

Unfortunately, it is due to a limitation of ConfigParser.

https://stackoverflow.com/questions/17947319/python-configparser-with-colon-in-the-key
https://bugs.python.org/issue16374

Instead of trying to hack this, I think we need to get rid of configparser and switch to a (simple) yaml file for topology.conf.

@thiell thiell added this to the 1.9 milestone Sep 29, 2019
@thiell
Copy link
Collaborator Author

thiell commented Sep 29, 2019

A possible workaround (for now) is to define groups in the default group source that can be used as key (with NO ":") in topology.conf, something like that in the default yaml group file:

'gateways_sh1':   '@sherlock:1&@service'
'gateways_sh2':   '@sherlock:2&@service'

@thiell thiell modified the milestones: 1.9, 2.0 Jun 26, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant