Skip to content
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

The nightly build job failed on Tuesday (2024-07-09) #747

Closed
github-actions bot opened this issue Jul 10, 2024 · 44 comments · Fixed by #756
Closed

The nightly build job failed on Tuesday (2024-07-09) #747

github-actions bot opened this issue Jul 10, 2024 · 44 comments · Fixed by #756

Comments

@github-actions
Copy link

The nightly build job failed on Tuesday (2024-07-09) in run 9867467789

Copy link
Author

The nightly build job failed on Wednesday (2024-07-10) in run 9867467789

@jdblischak
Copy link
Collaborator

Nightly failure caused by removal of dump functions in TileDB-Inc/TileDB#5026

Other nightlies are also failing for the same reason (TileDB-Inc/conda-forge-nightly-controller#108, https://github.com/jdblischak/centralized-tiledb-nightlies/issues/14)

Copy link
Author

The nightly build job failed on Wednesday (2024-07-10) in run 9884769007

@jdblischak
Copy link
Collaborator

Still failing due to the dump error

@jdblischak
Copy link
Collaborator

Fix pending in TileDB-Inc/TileDB#5179

Copy link
Author

The nightly build job failed on Thursday (2024-07-11) in run 9901719476

Copy link
Author

The nightly build job failed on Friday (2024-07-12) in run 9916791171

Copy link
Author

The nightly build job failed on Saturday (2024-07-13) in run 9924855664

Copy link
Author

The nightly build job failed on Monday (2024-07-15) in run 9933055676

Copy link
Author

The nightly build job failed on Tuesday (2024-07-16) in run 9950025355

Copy link
Author

The nightly build job failed on Wednesday (2024-07-17) in run 9967445139

@jdblischak
Copy link
Collaborator

There is a new error now. I think it was caused by the renaming of set_buffer to set_data_buffer in TileDB-Inc/TileDB#5146

/home/runner/work/TileDB-VCF/TileDB-VCF/TileDB-VCF/libtiledbvcf/src/dataset/attribute_buffer_set.cc:705:16: error: ‘class tiledb::Query’ has no member named ‘set_buffer’
  705 |         query->set_buffer(name, buff->data<void>(), 0);
      |                ^~~~~~~~~~

@gspowley
Copy link
Member

Yes, VCF is using deprecated APIs that are now being removed from libtiledb.

Copy link
Author

The nightly build job failed on Wednesday (2024-07-17) in run 9984800099

Copy link
Author

The nightly build job failed on Friday (2024-07-19) in run 10001849608

Copy link
Author

The nightly build job failed on Friday (2024-07-19) in run 10017037575

Copy link
Author

The nightly build job failed on Saturday (2024-07-20) in run 10025129401

Copy link
Author

The nightly build job failed on Sunday (2024-07-21) in run 10033892381

Copy link
Author

The nightly build job failed on Monday (2024-07-22) in run 10051950114

Copy link
Author

The nightly build job failed on Tuesday (2024-07-23) in run 10069751383

Copy link
Author

The nightly build job failed on Wednesday (2024-07-24) in run 10087413480

@jdblischak
Copy link
Collaborator

VCF is using deprecated APIs that are now being removed from libtiledb.

Is there a Shortcut Story to track this?

@gspowley
Copy link
Member

https://app.shortcut.com/tiledb-inc/story/45536/vcf-migrate-away-from-deprecated-tiledb-c-apis

Copy link
Author

The nightly build job failed on Thursday (2024-07-25) in run 10104974819

Copy link
Author

The nightly build job failed on Friday (2024-07-26) in run 10120341663

Copy link
Author

The nightly build job failed on Sunday (2024-07-28) in run 10128443495

Copy link
Author

The nightly build job failed on Sunday (2024-07-28) in run 10137339331

Copy link
Author

The nightly build job failed on Monday (2024-07-29) in run 10155388427

Copy link
Author

The nightly build job failed on Tuesday (2024-07-30) in run 10173302227

Copy link
Author

github-actions bot commented Aug 1, 2024

The nightly build job failed on Thursday (2024-08-01) in run 10191214289

Copy link
Author

github-actions bot commented Aug 2, 2024

The nightly build job failed on Thursday (2024-08-01) in run 10209259984

Copy link
Author

github-actions bot commented Aug 3, 2024

The nightly build job failed on Friday (2024-08-02) in run 10224770255

Copy link
Author

github-actions bot commented Aug 4, 2024

The nightly build job failed on Saturday (2024-08-03) in run 10232985196

Copy link
Author

github-actions bot commented Aug 5, 2024

The nightly build job failed on Sunday (2024-08-04) in run 10241833692

Copy link
Author

github-actions bot commented Aug 6, 2024

The nightly build job failed on Monday (2024-08-05) in run 10259517615

Copy link
Author

github-actions bot commented Aug 7, 2024

The nightly build job failed on Tuesday (2024-08-06) in run 10277377168

Copy link
Author

github-actions bot commented Aug 7, 2024

The nightly build job failed on Wednesday (2024-08-07) in run 10277377168

@gspowley
Copy link
Member

gspowley commented Aug 7, 2024

The libtiledb deprecated APIs are resolved. The latest failure happens when building tiledb-py from source.

@jdblischak have you seen this error in other builds?

Copy link
Author

github-actions bot commented Aug 8, 2024

The nightly build job failed on Thursday (2024-08-08) in run 10295190928

@jdblischak
Copy link
Collaborator

The libtiledb deprecated APIs are resolved.

Awesome! Thanks @awenocur!

xref: #753, #755

The latest failure happens when building tiledb-py from source. have you seen this error in other builds?

@gspowley Yes, this is a known issue, but there is a workaround. I'll investigate. Also, TileDB-Py has migrated to using scikit-build-core, so the nightly script needs updating anyways.

@dudoslav
Copy link
Contributor

dudoslav commented Aug 8, 2024

AFAIK this kind of issue happens when you run any code that imports tiledb when you have a folder tiledb in you current directory. I don't understand why it started happening now, but it was an issue even before scikit-build-core rework.

 Traceback (most recent call last):
  File "<string>", line 1, in <module>
  File "/Users/runner/work/TileDB-VCF/TileDB-VCF/TileDB-Py/tiledb/__init__.py", line 22, in <module>
    from .array_schema import ArraySchema
  File "/Users/runner/work/TileDB-VCF/TileDB-VCF/TileDB-Py/tiledb/array_schema.py", line 10, in <module>
    from .attribute import Attr
  File "/Users/runner/work/TileDB-VCF/TileDB-VCF/TileDB-Py/tiledb/attribute.py", line 9, in <module>
    from .ctx import Ctx, CtxMixin
  File "/Users/runner/work/TileDB-VCF/TileDB-VCF/TileDB-Py/tiledb/ctx.py", line 16, in <module>
    class Config(lt.Config):
AttributeError: module 'tiledb.cc' has no attribute 'Config'

Copy link
Author

github-actions bot commented Aug 9, 2024

The nightly build job failed on Thursday (2024-08-08) in run 10312916306

@jdblischak
Copy link
Collaborator

AFAIK this kind of issue happens when you run any code that imports tiledb when you have a folder tiledb in you current directory.

@dudoslav thanks! I keep running into this again and again

I don't understand why it started happening now, but it was an issue even before scikit-build-core rework.

Previously I had been using python setup.py develop, which performs an editable install. When I switched to scikit-build-core, I decided to not do an editable install, so that is why the error started happening now.

Copy link
Author

The nightly build job failed on Friday (2024-08-09) in run 10328445394

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

Successfully merging a pull request may close this issue.

6 participants