Fix clickhouse-driver integration spans

[sentrivana]

General

• change POTelSpan._get_attribute to public POTelSpan.get_attribute

clickhouse-driver integration

• Change set_data to set_attribute
• Span attr db.params is now a string
• params are now stored on the connection temporarily. Previously, they were saved on the span, but since we're setting them once and then reading them and appending to them in another place, span attrs no longer work for this nicely -- we'd need to serialize them to str, then deserialize to add stuff, and serialize again. So I'm changing this to keep track of them elsewhere to avoid this dance.
• Span attr db.query.text has been added. Previously, we were storing query on the span temporarily, but then were getting rid of it. This PR preserves it and gives it an OTel-friendly name. (This also means the query text is now stored on the span 3x -- in the description, in sentry.name and in db.query.text.)
• We were checking same_process_as_parent in the tests, now we're not.
• In OTel, the spans now have a status attribute, we're also not checking this in the tests.

Caveats

• Breadcrumbs will not work yet -- this needs Move subprocess breadcrumbs from maybe_create_breadcrumbs_from_span to integration. #3637

[codecov]

❌ 2933 Tests Failed:

Tests completed	Failed	Passed	Skipped
20072	2933	17139	4443

View the top 3 failed tests by shortest run time

tests.integrations.litestar.test_litestar test_middleware_spans

Stack Traces | 0s run time

No failure message available

tests.integrations.celery.test_update_celery_task_headers test_monitor_beat_tasks[False]

Stack Traces | 0.001s run time

.../integrations/celery/test_update_celery_task_headers.py:38: in test_monitor_beat_tasks
    assert "sentry-monitor-start-timestamp-s" not in outgoing_headers["headers"]
E   KeyError: 'headers'

tests.test_scope test_with_isolation_scope_data

Stack Traces | 0.001s run time

tests/test_scope.py:387: in test_with_isolation_scope_data
    assert scope._tags == {"before_isolation_scope": 1}
E   AssertionError: assert {} == {'before_isolation_scope': 1}
E     Right contains 1 more item:
E     {'before_isolation_scope': 1}
E     Full diff:
E     - {'before_isolation_scope': 1}
E     + {}

To view individual test run time comparison to the main branch, go to the Test Analytics Dashboard

[sentrivana]

https://opentelemetry.io/docs/specs/semconv/database/database-spans/#common-attributes

[sl0thentr0py]

If we need to access attributes outside, let's officially expose get_attributes or a @property called attributes. Accessing underscore methods outside the implementation is an anti-pattern and we should stop doing that.