Skip to content

CI warning is causing build to fail #50112

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

Closed
MarcoGorelli opened this issue Dec 7, 2022 · 3 comments · Fixed by #50113
Closed

CI warning is causing build to fail #50112

MarcoGorelli opened this issue Dec 7, 2022 · 3 comments · Fixed by #50113
Labels
CI Continuous Integration

Comments

@MarcoGorelli
Copy link
Member

The following shows up in some recent jobs: https://github.com/pandas-dev/pandas/actions/runs/3641110250/jobs/6146658226

2022-12-07T17:13:17.4282435Z pandas/_libs/tslibs/parsing.c: In function ‘__pyx_f_6pandas_5_libs_6tslibs_7parsing_dateutil_parse.constprop’:
2022-12-07T17:13:17.4286489Z pandas/_libs/tslibs/parsing.c:10634:22: error: array subscript ‘PyMethodObject[0]’ is partly outside array bounds of ‘PyObject[1]’ {aka ‘struct _object[1]’} [-Werror=array-bounds]
2022-12-07T17:13:17.4287232Z 10634 |           __pyx_t_10 = PyMethod_GET_SELF(__pyx_t_2);
2022-12-07T17:13:17.4288524Z In file included from /home/runner/micromamba/envs/test/include/python3.8/pytime.h:6,
2022-12-07T17:13:17.4289063Z                  from /home/runner/micromamba/envs/test/include/python3.8/Python.h:85,
2022-12-07T17:13:17.4289520Z                  from pandas/_libs/tslibs/parsing.c:46:
2022-12-07T17:13:17.4290230Z /home/runner/micromamba/envs/test/include/python3.8/object.h:560:22: note: while referencing ‘_Py_NoneStruct’
2022-12-07T17:13:17.4290904Z   560 | PyAPI_DATA(PyObject) _Py_NoneStruct; /* Don't use this directly */
2022-12-07T17:13:17.4291327Z       |                      ^~~~~~~~~~~~~~
2022-12-07T17:13:17.4299214Z pandas/_libs/tslibs/parsing.c:10636:23: error: array subscript ‘PyMethodObject[0]’ is partly outside array bounds of ‘PyObject[1]’ {aka ‘struct _object[1]’} [-Werror=array-bounds]
2022-12-07T17:13:17.4299878Z 10636 |             PyObject* function = PyMethod_GET_FUNCTION(__pyx_t_2);
2022-12-07T17:13:17.4300282Z       |                       ^~~~~~~~
2022-12-07T17:13:17.4300776Z In file included from /home/runner/micromamba/envs/test/include/python3.8/pytime.h:6,
2022-12-07T17:13:17.4301291Z                  from /home/runner/micromamba/envs/test/include/python3.8/Python.h:85,
2022-12-07T17:13:17.4301779Z                  from pandas/_libs/tslibs/parsing.c:46:
2022-12-07T17:13:17.4302447Z /home/runner/micromamba/envs/test/include/python3.8/object.h:560:22: note: while referencing ‘_Py_NoneStruct’
2022-12-07T17:13:17.4303287Z   560 | PyAPI_DATA(PyObject) _Py_NoneStruct; /* Don't use this directly */
2022-12-07T17:13:17.4303722Z       |                      ^~~~~~~~~~~~~~
2022-12-07T17:13:18.5474602Z cc1: all warnings being treated as errors

Example of green job: https://github.com/pandas-dev/pandas/actions/runs/3633476581/jobs/6130537543
Example of red job: https://github.com/pandas-dev/pandas/actions/runs/3641110250/jobs/6146658226

Can't tell what's caused this

Here's the difference between the environments:

                                                  Name Version_x Version_y
0    ──────────────────────────────────────────────...      None      None
59                                           db-dtypes     1.0.4     1.0.5
109                               google-cloud-storage     2.6.0     2.7.0
114                                             grpcio    1.49.1    1.51.1
115                                      grpcio-status    1.49.1    1.51.1
126                                importlib_resources    5.10.0    5.10.1
183                                            libgrpc    1.49.1    1.51.1
284                                     prompt-toolkit    3.0.33    3.0.36
@MarcoGorelli MarcoGorelli added the CI Continuous Integration label Dec 7, 2022
@MarcoGorelli
Copy link
Member Author

@jbrockmendel any idea what might be causing this?

@jbrockmendel
Copy link
Member

no idea cc @WillAyd ?

@MarcoGorelli
Copy link
Member Author

pinning ubuntu seems to have fixed it #50113

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI Continuous Integration
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants