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

Add topo timestamp api #172

Merged
merged 3 commits into from
Jan 8, 2025
Merged

Add topo timestamp api #172

merged 3 commits into from
Jan 8, 2025

Conversation

congwang09
Copy link
Contributor

@congwang09 congwang09 commented Jan 8, 2025

Resolves: #161

Add an API to query the timestamp for latest topology pulling from OXP.
When updating the latest topology, LC will save the timestamp to the db. The timestamp is in seconds, e.g., 1734837838.

@congwang09 congwang09 requested review from sajith and YufengXin January 8, 2025 15:23
@coveralls
Copy link

coveralls commented Jan 8, 2025

Pull Request Test Coverage Report for Build 12673669336

Details

  • 1 of 5 (20.0%) changed or added relevant lines in 1 file are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage decreased (-0.2%) to 81.019%

Changes Missing Coverage Covered Lines Changed/Added Lines %
sdx_lc/controllers/topology_controller.py 1 5 20.0%
Totals Coverage Status
Change from base Build 12240686689: -0.2%
Covered Lines: 1208
Relevant Lines: 1491

💛 - Coveralls

@congwang09 congwang09 merged commit e2ad332 into main Jan 8, 2025
11 checks passed
@congwang09 congwang09 deleted the add-topo-timestamp-api branch January 8, 2025 16:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add API for querying the latest time that a topology was pulled from OXP
3 participants