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

OCPBUGS-39175: [release-4.14] Leap event 4.14 #356

Open
wants to merge 9 commits into
base: release-4.14
Choose a base branch
from

Conversation

vitus133
Copy link
Contributor

@vitus133 vitus133 commented Aug 29, 2024

This is a manual cherry-pick of #326, #342 and #358

/cc @aneeshkp @jzding @josephdrichard @nishant-parekh

Vitaly Grinberg and others added 8 commits July 10, 2024 08:10
This adds automatic leap file updates from GPS data.

Signed-off-by: Vitaly Grinberg <[email protected]>
This fixes a bug causing leap file expiration date
to be off by 70 years and update date to be outdated

Signed-off-by: Vitaly Grinberg <[email protected]>
Signed-off-by: Aneesh Puttur <[email protected]>
This commit fixes various leap file corner cases and
adds missing functionality:
1. Rehash the leapfile after setting the expiration time to 2036
2. Fix corner cases of leap event appearing in a different ways in
the leap indication in the past or in the future + add unit tests
3. General cleanup and refactor for better testability
4. Add unit tests

Signed-off-by: Vitaly Grinberg <[email protected]>
This PR fixes a bug where the daemon wrote uninitialized values to the time properties dataset, which led to corrupting the ptp time scale, current UTC offset and other dataset properties.
The fix is done by always initializing the time properties according to the current UTC offset and a predefined set of flags.
Leap manager was changed to provide current UTC offset to external packages.

Signed-off-by: Vitaly Grinberg <[email protected]>
@openshift-ci-robot openshift-ci-robot added jira/valid-reference Indicates that this PR references a valid Jira ticket of any type. jira/invalid-bug Indicates that a referenced Jira bug is invalid for the branch this PR is targeting. labels Aug 29, 2024
@openshift-ci-robot
Copy link
Contributor

@vitus133: This pull request references Jira Issue OCPBUGS-39175, which is invalid:

  • expected dependent Jira Issue OCPBUGS-39174 to be in one of the following states: VERIFIED, RELEASE PENDING, CLOSED (ERRATA), CLOSED (CURRENT RELEASE), CLOSED (DONE), CLOSED (DONE-ERRATA), but it is New instead

Comment /jira refresh to re-evaluate validity if changes to the Jira bug are made, or edit the title of this pull request to link to a different bug.

The bug has been updated to refer to the pull request using the external bug tracker.

In response to this:

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the openshift-eng/jira-lifecycle-plugin repository.

Copy link
Contributor

openshift-ci bot commented Aug 29, 2024

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: vitus133

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@openshift-ci openshift-ci bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Aug 29, 2024
@vitus133 vitus133 force-pushed the leap-event-4.14 branch 3 times, most recently from 92f20bc to 2d52f38 Compare September 1, 2024 15:56
this commit adds leap event handling that includes following:
1. Determine that current time is within -12h...1m from the leap
2. Send PMC command (once) to indicate leap61 or leap 59 to the
downstream architecture and
3. Handle ts2phc time ambiguity

Signed-off-by: Vitaly Grinberg <[email protected]>
Copy link
Contributor

openshift-ci bot commented Sep 2, 2024

@vitus133: all tests passed!

Full PR test history. Your PR dashboard.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.

@aneeshkp
Copy link
Contributor

aneeshkp commented Sep 3, 2024

/label backport-risk-assessed

@openshift-ci openshift-ci bot added the backport-risk-assessed Indicates a PR to a release branch has been evaluated and considered safe to accept. label Sep 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved Indicates a PR has been approved by an approver from all required OWNERS files. backport-risk-assessed Indicates a PR to a release branch has been evaluated and considered safe to accept. jira/invalid-bug Indicates that a referenced Jira bug is invalid for the branch this PR is targeting. jira/valid-reference Indicates that this PR references a valid Jira ticket of any type.
Projects
None yet
Development

Successfully merging this pull request may close these issues.