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

report local time #26

Merged
merged 1 commit into from
Jun 19, 2024
Merged
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
7 changes: 7 additions & 0 deletions yang/ietf-schedule.yang
Original file line number Diff line number Diff line change
Expand Up @@ -744,6 +744,13 @@ module ietf-schedule {
description
"Indicates the schedule type.";
}
leaf local-time {
type yang:date-and-time;
config false;
description
"Reports the local time as used by the entity that
hosts the schedule.";
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
hosts the schedule.";
hosts the schedule. This parameter is only reported if schedule is expressed in local time format.";

Note that 3231 requires calendar schedules are always expressed in local time. I think if the schedule is using UTC time, no need to know the local time info.
I guess I don't really understand what time is reported by this parameter? The time when this value is retrieved? or the time when the schedule is managed?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You are right for calendar schedules, but other types do not have such restriction.

This will report the actual local-time as seen by the entity.

No need to restrict the attribute to non UTC but leave that the manager to infer whatever useful from the parameter.

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sure, I think I can live with that.

}
leaf last-update {
type yang:date-and-time;
config false;
Expand Down
Loading