-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
[minigraph.py] Enable telemetry service by default #3611
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please resolve new conflicts.
ece1b91
to
949baeb
Compare
Retest vs please |
@nikos-github: Even when loading from minigraph? This section of code will only write this section to the config_db.json when loading from minigraph. Otherwise, if there are no "on"/"enabled" entries in config_db.json, the default behavior will be "disabled". Is this still an issue for you? |
@jleveque Ok if this default enabling doesn't propagate to config_db.json, I have no issues. |
@nikos-github: It will only propagate to config_db.json if you are loading your config from a minigraph file. |
@jleveque Does the first install config of config_db.json load from minigraph? |
@nikos-github: No. Initial config is in JSON format. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ok for me then.
@pra-moh: Please resolve new conflicts. |
…to pramoh/enabletelemetry
- What I did
Enabling telemetry feature by default
- How I did it
on minigraph load adding new table in config db as 'features|telemetry' {'status':'on'}
- How to verify it
config load_minigraph
- Description for the changelog
- A picture of a cute animal (not mandatory but encouraged)