-
Notifications
You must be signed in to change notification settings - Fork 655
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
Clarify last-reboot-time is updated during a power-on event #1179
base: master
Are you sure you want to change the base?
Conversation
No major YANG version changes in commit 3edd9fd |
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.
As discussed at the OC operators meeting, the simplest change to clarify the description defining what this leaf is intended to represent is the least disruptive. Pedantically the leaf name last-reboot-time
remains slightly misleading, but that could always be another PR if anyone feels strongly enough about it?
@rgwilton @earies @LimeHat @nandanarista for any comments |
looks good to me. |
With |
@dplore Wouldn't it be better to add a new leaf called boot-time/component-boot-time indicating the boot time of the component? openconfig-system boot-time could indicate the boot-time of active controller card in a modular system or chassis in case of a fixed system. |
Few generic comments/questions
|
Fixes #1171