[system_metrics] Update system.disk.time to system.disk.operation.time

[srikaaviya]

Description

This PR resolves a pending FIXME in the opentelemetry-instrumentation-system-metrics module by updating the deprecated system.disk.time semantic convention to the current standard system.disk.operation.time.

• Migrated system.disk.time internal configurations and metric generation callbacks to the new system.disk.operation.time nomenclature.
• Updated local unit test [test_system_metrics.py]

Type of change

• Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?

Ran the full instrumentation tox testing suite and linter, confirming both the runtime assertions and the style compliance passed perfectly on the updated module.

• tox -e py311-instrumentation-system-metrics
• tox -e lint-instrumentation-system-metrics

Does This PR Require a Core Repo Change?

• No.

Checklist:

• Followed the style guidelines of this project
• Unit tests have been updated

[tammy-baylis-swi]

I think this change snuck in from the other PR. Could we trim this PR down so it's only for the system-metrics?

[srikaaviya]

@tammy-baylis-swi Sorry for the PR mix. I've pushed a cleaned up branch that only contains the system metrics changes.

[tammy-baylis-swi]

Thanks @srikaaviya that's better! Please could you also add a line to Changelog.

[srikaaviya]

Thanks @srikaaviya that's better! Please could you also add a line to Changelog.

Done! Just pushed a changelog entry.

[tammy-baylis-swi]

This lgtm! The Maintainers will also look. The docker-test fails aren't related and are happening in other PRs.

I think technically this is a breaking change but not worth a special transition/opt-in.

[srikaaviya]

This lgtm! The Maintainers will also look. The docker-test fails aren't related and are happening in other PRs.

I think technically this is a breaking change but not worth a special transition/opt-in.

Thanks for the review!