You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
## Description
The starlette framework tests currently pin sqlalchemy to 1.4.41 due to
an upstream issue which has now closed:
encode/databases#512. It looks like we can
unpin this dependency.
Note: this also indirectly fixes the failing starlette framework tests
due to this now closed upstream issue:
encode/databases#513, where sqlalchemy==1.4.42
added a new private field in the SQLAlchemy cursor class. While this job
uses the latest databases package that has accounted for this field,
this was causing attribute errors since we were using a previous version
of sqlalchemy that did not yet have that private field added.
<!-- If this is a breaking change, explain why it is necessary. Breaking
changes must append `!` after the type/scope. See
https://ddtrace.readthedocs.io/en/stable/contributing.html for more
details. -->
## Checklist
- [ ] Followed the [library release note
guidelines](https://ddtrace.readthedocs.io/en/stable/contributing.html#Release-Note-Guidelines)
when writing a release note.
- [ ] Add additional sections for `feat` and `fix` pull requests.
- [ ] [Library
documentation](https://github.com/DataDog/dd-trace-py/tree/1.x/docs)
and/or [Datadog's documentation
site](https://github.com/DataDog/documentation/) is updated. Link to doc
PR in description.
<!-- Copy and paste the relevant snippet based on the type of pull
request -->
<!-- START feat -->
## Motivation
<!-- Expand on why the change is required, include relevant context for
reviewers -->
## Design
<!-- Include benefits from the change as well as possible drawbacks and
trade-offs -->
## Testing strategy
<!-- Describe the automated tests and/or the steps for manual testing.
<!-- END feat -->
<!-- START fix -->
## Relevant issue(s)
<!-- Link the pull request to any issues related to the fix. Use
keywords for links to automate closing the issues once the pull request
is merged. -->
## Testing strategy
<!-- Describe any added regression tests and/or the manual testing
performed. -->
<!-- END fix -->
## Reviewer Checklist
- [x] Title is accurate.
- [x] Description motivates each change.
- [x] No unnecessary changes were introduced in this PR.
- [x] Avoid breaking
[API](https://ddtrace.readthedocs.io/en/stable/versioning.html#interfaces)
changes unless absolutely necessary.
- [x] Tests provided or description of manual testing performed is
included in the code or PR.
- [x] Release note has been added and follows the [library release note
guidelines](https://ddtrace.readthedocs.io/en/stable/contributing.html#Release-Note-Guidelines),
or else `changelog/no-changelog` label added.
- [x] All relevant GitHub issues are correctly linked.
- [x] Backports are identified and tagged with Mergifyio.
Copy file name to clipboardExpand all lines: .github/workflows/test_frameworks.yml
-3Lines changed: 0 additions & 3 deletions
Original file line number
Diff line number
Diff line change
@@ -252,9 +252,6 @@ jobs:
252
252
run: pip install ../ddtrace
253
253
- name: Install dependencies
254
254
run: scripts/install
255
-
- name: Pin SQLAlchemy
256
-
# https://github.com/encode/databases/issues/512
257
-
run: pip install "SQLAlchemy==1.4.41"
258
255
#Parameters for keyword expression skip 3 failing tests that are expected due to asserting on headers. The errors are because our context propagation headers are being added
259
256
#test_staticfiles_with_invalid_dir_permissions_returns_401 fails with and without ddtrace enabled
0 commit comments