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

deprecate uptime apis #3226

Merged
merged 8 commits into from
Aug 15, 2024
Merged

deprecate uptime apis #3226

merged 8 commits into from
Aug 15, 2024

Conversation

ceyonur
Copy link
Contributor

@ceyonur ceyonur commented Jul 24, 2024

We're going to remove subnet uptime tracking and recording in P-chain state. This PR puts an deprecation message for required APIs and places.

Copy link
Contributor

@abi87 abi87 left a comment

Choose a reason for hiding this comment

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

If this is going to happen following a fork activation, I'd explicitly mention it in the deprecation message

@StephenButtolph StephenButtolph added this to the v1.11.11 milestone Jul 25, 2024
@StephenButtolph StephenButtolph added the documentation Improvements or additions to documentation or examples label Jul 25, 2024
api/info/service.go Outdated Show resolved Hide resolved
vms/platformvm/client_permissionless_validator.go Outdated Show resolved Hide resolved
@ceyonur
Copy link
Contributor Author

ceyonur commented Jul 26, 2024

If this is going to happen following a fork activation, I'd explicitly mention it in the deprecation message

These will be removed prior to E-Upgrade

@StephenButtolph StephenButtolph added this pull request to the merge queue Aug 15, 2024
Merged via the queue into master with commit 9e77ecc Aug 15, 2024
21 checks passed
@StephenButtolph StephenButtolph deleted the deprecate-uptime-apis branch August 15, 2024 14:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation or examples Uptime Tracking
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

6 participants