Skip to content

fix(ci): pin importlib_metadata<5.0.0 #4257

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

Merged
merged 13 commits into from
Oct 6, 2022
Merged

Conversation

majorgreys
Copy link
Contributor

@majorgreys majorgreys commented Oct 3, 2022

CI jobs profile-windows-*, kombu and celery are failing due to the release of importlib_metadata==5.0.0 which removed deprecated shims, resulting in the following exception:

AttributeError: 'EntryPoints' object has no attribute 'get'

We fix this by pinning the dependency for kombu and celery. In the case of the Windows tests, we address it by upgrading the Windows image to a version that includes Python 3.10 rather than Python 3.7 which removes the need to install the dependency when installing tox.

@majorgreys majorgreys added the changelog/no-changelog A changelog entry is not required for this PR. label Oct 3, 2022
@majorgreys majorgreys requested a review from a team as a code owner October 3, 2022 16:59
Kyle-Verhoog
Kyle-Verhoog previously approved these changes Oct 3, 2022
Copy link
Member

@Kyle-Verhoog Kyle-Verhoog left a comment

Choose a reason for hiding this comment

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

Can we describe why we need to pin in the PR description?

@majorgreys
Copy link
Contributor Author

This PR or #4253 will need to be force merged to unblock #4252.

cc @Kyle-Verhoog @brettlangdon

mabdinur
mabdinur previously approved these changes Oct 3, 2022
Yun-Kim
Yun-Kim previously approved these changes Oct 3, 2022
@majorgreys
Copy link
Contributor Author

majorgreys commented Oct 3, 2022

@Mergifyio backport 1.5 1.4 0.61

@mergify
Copy link
Contributor

mergify bot commented Oct 3, 2022

backport 1.5 1.4 0.x 0.61

✅ Backports have been created

@Kyle-Verhoog
Copy link
Member

jd
jd previously approved these changes Oct 4, 2022
@jd
Copy link
Contributor

jd commented Oct 4, 2022

It'd be great to track the upstream issue (I imagine it's one of the dependencies we pull that breaks with 5.0.0, not our code).

@majorgreys
Copy link
Contributor Author

It'd be great to track the upstream issue (I imagine it's one of the dependencies we pull that breaks with 5.0.0, not our code).

Added open issue on the breaking change.

@mergify
Copy link
Contributor

mergify bot commented Oct 4, 2022

backport 1.5 1.4 0.61

✅ Backports have been created

@majorgreys majorgreys force-pushed the majorgreys/ci-pin-importlib branch from 47b6a59 to 21509fe Compare October 5, 2022 14:18
@majorgreys majorgreys force-pushed the majorgreys/ci-pin-importlib branch from 21509fe to 785976d Compare October 5, 2022 14:25
@majorgreys majorgreys dismissed stale reviews from jd, Yun-Kim, mabdinur, and Kyle-Verhoog via 5edda43 October 5, 2022 14:28
@majorgreys majorgreys force-pushed the majorgreys/ci-pin-importlib branch from 392f750 to 81e4de2 Compare October 5, 2022 17:35
@majorgreys majorgreys force-pushed the majorgreys/ci-pin-importlib branch from a29a762 to 1830fc6 Compare October 5, 2022 19:31
mabdinur
mabdinur previously approved these changes Oct 5, 2022
brettlangdon
brettlangdon previously approved these changes Oct 5, 2022
@majorgreys
Copy link
Contributor Author

As a follow-up, we can look into enabling the profile-windows jobs with the updated image.

cc @jd

Kyle-Verhoog
Kyle-Verhoog previously approved these changes Oct 5, 2022
Copy link
Member

@Kyle-Verhoog Kyle-Verhoog left a comment

Choose a reason for hiding this comment

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

🙌 for dealing with this mess @majorgreys!!

Co-authored-by: Kyle Verhoog <[email protected]>
@mergify mergify bot merged commit c2b416e into 1.x Oct 6, 2022
@mergify mergify bot deleted the majorgreys/ci-pin-importlib branch October 6, 2022 00:51
mergify bot pushed a commit that referenced this pull request Oct 6, 2022
CI jobs `profile-windows-*`, `kombu` and `celery` are failing due to the release of `importlib_metadata==5.0.0` which removed deprecated shims, resulting in the following exception:

```
AttributeError: 'EntryPoints' object has no attribute 'get'
```

We fix this by pinning the dependency for kombu and celery. In the case of the Windows tests, we address it by upgrading the Windows image to a version that includes Python 3.10 rather than Python 3.7 which removes the need to install the dependency when installing tox.

- [Example log of failing job](https://app.circleci.com/pipelines/github/DataDog/dd-trace-py/21679/workflows/2a7606b5-9ca8-4212-9b02-053665de0198/jobs/1477147)
- python/importlib_metadata#409

(cherry picked from commit c2b416e)
mergify bot pushed a commit that referenced this pull request Oct 6, 2022
CI jobs `profile-windows-*`, `kombu` and `celery` are failing due to the release of `importlib_metadata==5.0.0` which removed deprecated shims, resulting in the following exception:

```
AttributeError: 'EntryPoints' object has no attribute 'get'
```

We fix this by pinning the dependency for kombu and celery. In the case of the Windows tests, we address it by upgrading the Windows image to a version that includes Python 3.10 rather than Python 3.7 which removes the need to install the dependency when installing tox.

- [Example log of failing job](https://app.circleci.com/pipelines/github/DataDog/dd-trace-py/21679/workflows/2a7606b5-9ca8-4212-9b02-053665de0198/jobs/1477147)
- python/importlib_metadata#409

(cherry picked from commit c2b416e)
mergify bot pushed a commit that referenced this pull request Oct 6, 2022
CI jobs `profile-windows-*`, `kombu` and `celery` are failing due to the release of `importlib_metadata==5.0.0` which removed deprecated shims, resulting in the following exception:

```
AttributeError: 'EntryPoints' object has no attribute 'get'
```

We fix this by pinning the dependency for kombu and celery. In the case of the Windows tests, we address it by upgrading the Windows image to a version that includes Python 3.10 rather than Python 3.7 which removes the need to install the dependency when installing tox.

- [Example log of failing job](https://app.circleci.com/pipelines/github/DataDog/dd-trace-py/21679/workflows/2a7606b5-9ca8-4212-9b02-053665de0198/jobs/1477147)
- python/importlib_metadata#409

(cherry picked from commit c2b416e)

# Conflicts:
#	.circleci/config.yml
@github-actions github-actions bot added this to the v1.6.0 milestone Oct 6, 2022
mergify bot pushed a commit that referenced this pull request Oct 6, 2022
CI jobs `profile-windows-*`, `kombu` and `celery` are failing due to the release of `importlib_metadata==5.0.0` which removed deprecated shims, resulting in the following exception:

```
AttributeError: 'EntryPoints' object has no attribute 'get'
```

We fix this by pinning the dependency for kombu and celery. In the case of the Windows tests, we address it by upgrading the Windows image to a version that includes Python 3.10 rather than Python 3.7 which removes the need to install the dependency when installing tox.

- [Example log of failing job](https://app.circleci.com/pipelines/github/DataDog/dd-trace-py/21679/workflows/2a7606b5-9ca8-4212-9b02-053665de0198/jobs/1477147)
- python/importlib_metadata#409

(cherry picked from commit c2b416e)

# Conflicts:
#	.circleci/config.yml
Kyle-Verhoog added a commit that referenced this pull request Oct 6, 2022
CI jobs `profile-windows-*`, `kombu` and `celery` are failing due to the release of `importlib_metadata==5.0.0` which removed deprecated shims, resulting in the following exception:

```
AttributeError: 'EntryPoints' object has no attribute 'get'
```

We fix this by pinning the dependency for kombu and celery. In the case of the Windows tests, we address it by upgrading the Windows image to a version that includes Python 3.10 rather than Python 3.7 which removes the need to install the dependency when installing tox.

- [Example log of failing job](https://app.circleci.com/pipelines/github/DataDog/dd-trace-py/21679/workflows/2a7606b5-9ca8-4212-9b02-053665de0198/jobs/1477147)
- python/importlib_metadata#409

(cherry picked from commit c2b416e)

Co-authored-by: Tahir H. Butt <[email protected]>
Co-authored-by: Kyle Verhoog <[email protected]>
Kyle-Verhoog pushed a commit that referenced this pull request Oct 12, 2022
CI jobs `profile-windows-*`, `kombu` and `celery` are failing due to the release of `importlib_metadata==5.0.0` which removed deprecated shims, resulting in the following exception:

```
AttributeError: 'EntryPoints' object has no attribute 'get'
```

We fix this by pinning the dependency for kombu and celery. In the case of the Windows tests, we address it by upgrading the Windows image to a version that includes Python 3.10 rather than Python 3.7 which removes the need to install the dependency when installing tox.

- [Example log of failing job](https://app.circleci.com/pipelines/github/DataDog/dd-trace-py/21679/workflows/2a7606b5-9ca8-4212-9b02-053665de0198/jobs/1477147)
- python/importlib_metadata#409

(cherry picked from commit c2b416e)

Co-authored-by: Tahir H. Butt <[email protected]>
Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
@mabdinur
Copy link
Contributor

@Mergifyio backport 0.61

@mergify
Copy link
Contributor

mergify bot commented Oct 19, 2022

backport 0.61

✅ Backports have been created

Kyle-Verhoog pushed a commit that referenced this pull request Oct 24, 2022
* fix(ci): pin importlib_metadata<5.0.0 (#4257)

CI jobs `profile-windows-*`, `kombu` and `celery` are failing due to the release of `importlib_metadata==5.0.0` which removed deprecated shims, resulting in the following exception:

```
AttributeError: 'EntryPoints' object has no attribute 'get'
```

We fix this by pinning the dependency for kombu and celery. In the case of the Windows tests, we address it by upgrading the Windows image to a version that includes Python 3.10 rather than Python 3.7 which removes the need to install the dependency when installing tox.

- [Example log of failing job](https://app.circleci.com/pipelines/github/DataDog/dd-trace-py/21679/workflows/2a7606b5-9ca8-4212-9b02-053665de0198/jobs/1477147)
- python/importlib_metadata#409

(cherry picked from commit c2b416e)

# Conflicts:
#	.circleci/config.yml

* fix conflicts

Co-authored-by: Tahir H. Butt <[email protected]>
Co-authored-by: Munir Abdinur <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
changelog/no-changelog A changelog entry is not required for this PR.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants