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

Retrieval of content file(s) failing #395

Closed
jhlegarreta opened this issue Aug 22, 2022 · 2 comments
Closed

Retrieval of content file(s) failing #395

jhlegarreta opened this issue Aug 22, 2022 · 2 comments
Labels
type:Bug Inconsistencies or issues which will cause an incorrect result under some or all circumstances type:Data Changes to example data (usually displayed images) type:Infrastructure Infrastructure/ecosystem related changes, such as CMake or buildbots type:Testing Ensure that the purpose of a class is met/the results on a wide set of test cases are correct

Comments

@jhlegarreta
Copy link
Member

Description

build-test-python-superbuild and build-test-documentation CI builds are consistently failing since some time.
https://github.com/InsightSoftwareConsortium/ITKSphinxExamples/actions/workflows/build-test-publish.yml

It looks like there is a SHA512 content file that cannot be retrieved properly
https://github.com/InsightSoftwareConsortium/ITKSphinxExamples/runs/7933591657?check_suite_focus=true#step:8:8011

from https://insightsoftwareconsortium.github.io/ITKTestingData/SHA512

This results in a compiler error:
https://github.com/InsightSoftwareConsortium/ITKSphinxExamples/runs/7933591657?check_suite_focus=true#step:8:8029

However, the file does exist in https://data.kitware.com:443/api/v1/file/hashsum/SHA512/ (opening the link with the SHA512 checksum downloads the file).

Additional Information

It is unclear to me the example to which the file corresponds (it is apparently named OutputMeshes.png), but the content link of the only file named as such
https://github.com/InsightSoftwareConsortium/ITKSphinxExamples/blob/master/src/Filtering/QuadEdgeMeshFiltering/ComputePlanarParameterizationOfAMesh/OutputMeshes.png.sha512

Also, it looks like the failing content file is not always the same:
#393 (comment)

This is affecting the master branch and any PR that has been submitted lately.

@jhlegarreta jhlegarreta added type:Bug Inconsistencies or issues which will cause an incorrect result under some or all circumstances type:Infrastructure Infrastructure/ecosystem related changes, such as CMake or buildbots type:Testing Ensure that the purpose of a class is met/the results on a wide set of test cases are correct type:Data Changes to example data (usually displayed images) labels Aug 22, 2022
@thewtex
Copy link
Member

thewtex commented Nov 7, 2022

Addressed in #398

thewtex added a commit to thewtex/ITKSphinxExamples that referenced this issue Nov 11, 2022
This improves reliability, addressing InsightSoftwareConsortium#395, but also the contributor
experience, maintainability, and sustainability.
thewtex added a commit to thewtex/ITKSphinxExamples that referenced this issue Nov 11, 2022
This improves reliability, addressing InsightSoftwareConsortium#395, but also the contributor
experience, maintainability, and sustainability.
@jhlegarreta
Copy link
Member Author

Closing as #398 has been merged.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type:Bug Inconsistencies or issues which will cause an incorrect result under some or all circumstances type:Data Changes to example data (usually displayed images) type:Infrastructure Infrastructure/ecosystem related changes, such as CMake or buildbots type:Testing Ensure that the purpose of a class is met/the results on a wide set of test cases are correct
Projects
None yet
Development

No branches or pull requests

2 participants