open-telemetry / opentelemetry-python-contrib

OpenTelemetry instrumentation for Python modules
https://opentelemetry.io
Apache License 2.0
728 stars 599 forks source link

CI: Fatal Python error: Aborted #1736

Open srikanthccv opened 1 year ago

srikanthccv commented 1 year ago

boto and botocore instrumentations fail with Fatal Python error: Aborted for pypy. Investigate and fix it if possible.

srikanthccv commented 1 year ago

Example run: https://github.com/open-telemetry/opentelemetry-python-contrib/actions/runs/4566389946/jobs/8058826592?pr=1735

xrmx commented 6 months ago

Logs are gone, locally this is failing ~because of no cryptography wheel is found (maybe too old pypy, they should be available) and~ because of regex dependency is not available for interpreters other than cpython and it segfaults. regex is a dependency of cfn-lint. Removing cfn-lint and regex from dependencies appear to still work.

There could be more requirements to remove, se pipdeptree output:

cfn-lint==0.85.2
├── aws-sam-translator [required: >=1.84.0, installed: 1.85.0]
│   ├── boto3 [required: >=1.19.5,==1.*, installed: 1.34.44]
│   │   ├── botocore [required: >=1.34.44,<1.35.0, installed: 1.34.44]
│   │   │   ├── jmespath [required: >=0.7.1,<2.0.0, installed: 1.0.1]
│   │   │   ├── python-dateutil [required: >=2.1,<3.0.0, installed: 2.8.2]
│   │   │   │   └── six [required: >=1.5, installed: 1.16.0]
│   │   │   └── urllib3 [required: >=1.25.4,<2.1, installed: 1.26.18]
│   │   ├── jmespath [required: >=0.7.1,<2.0.0, installed: 1.0.1]
│   │   └── s3transfer [required: >=0.10.0,<0.11.0, installed: 0.10.0]
│   │       └── botocore [required: >=1.33.2,<2.0a.0, installed: 1.34.44]
│   │           ├── jmespath [required: >=0.7.1,<2.0.0, installed: 1.0.1]
│   │           ├── python-dateutil [required: >=2.1,<3.0.0, installed: 2.8.2]
│   │           │   └── six [required: >=1.5, installed: 1.16.0]
│   │           └── urllib3 [required: >=1.25.4,<2.1, installed: 1.26.18]
│   ├── jsonschema [required: >=3.2,<5, installed: 4.21.1]
│   │   ├── attrs [required: >=22.2.0, installed: 23.2.0]
│   │   ├── jsonschema-specifications [required: >=2023.03.6, installed: 2023.12.1]
│   │   │   └── referencing [required: >=0.31.0, installed: 0.33.0]
│   │   │       ├── attrs [required: >=22.2.0, installed: 23.2.0]
│   │   │       └── rpds-py [required: >=0.7.0, installed: 0.18.0]
│   │   ├── referencing [required: >=0.28.4, installed: 0.33.0]
│   │   │   ├── attrs [required: >=22.2.0, installed: 23.2.0]
│   │   │   └── rpds-py [required: >=0.7.0, installed: 0.18.0]
│   │   └── rpds-py [required: >=0.7.1, installed: 0.18.0]
│   ├── pydantic [required: >=1.8,<3, installed: 2.6.1]
│   │   ├── annotated-types [required: >=0.4.0, installed: 0.6.0]
│   │   ├── pydantic_core [required: ==2.16.2, installed: 2.16.2]
│   │   │   └── typing_extensions [required: >=4.6.0,!=4.7.0, installed: 4.9.0]
│   │   └── typing_extensions [required: >=4.6.1, installed: 4.9.0]
│   └── typing_extensions [required: >=4.4, installed: 4.9.0]
xrmx commented 5 months ago

Would be interesting to see if this is reproducible with pypy 7.3.16 after #2559. That version is 3.9+ so cannot be used right now unfortunately.