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

Investigate why actions/setup-python@v5 fails sometimes in I2::CI::check_for_incorrect_images #4499

Closed
nxsaken opened this issue Apr 23, 2024 · 2 comments · Fixed by #4996
Closed
Labels
CI good first issue Good for newcomers iroha2-dev The re-implementation of a BFT hyperledger in RUST

Comments

@nxsaken
Copy link
Contributor

nxsaken commented Apr 23, 2024

The I2::CI::check_for_incorrect_images job fails sometimes with the following:

Error: The version '3.11' with architecture 'x64' was not found for this operating system.

Example log

@nxsaken nxsaken added iroha2-dev The re-implementation of a BFT hyperledger in RUST CI labels Apr 23, 2024
@nxsaken nxsaken changed the title Fix I2::CI::check_for_incorrect_images Investigate why actions/setup-python@v5 fails sometimes in I2::CI::check_for_incorrect_images May 14, 2024
@nxsaken nxsaken added the good first issue Good for newcomers label May 14, 2024
@RamilMus
Copy link
Contributor

RamilMus commented Aug 8, 2024

Checked and found some closed and open issues regarding setting up python via this action. The recent one is open.
And we have same log even though we use github runners with ubuntu-latest
Unable to locate executable file: lsb_release

  1. Do we need that workflow at all, is it useful for us?
    As I see it is intended to check that for main branch in all docker-compose files we will use iroha2-dev images, and for stable branch - iroha2:stable images, which is not actually true, we have different images in both branches. Guess it is not being useful anymore. Examples: for main, for stable

  2. If we will need it, we need to reconsider allowed images in those docker-compose files and then for action itself we can try adding some workarounds like setting this lsb_release or rerunning this action inside the job.

@nxsaken
Copy link
Contributor Author

nxsaken commented Aug 8, 2024

@RamilMus nowadays we do releases via tags, and each gets a separate image (e.g. image for pre-rc.22.0). If we're keeping this job, it needs to be modified.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI good first issue Good for newcomers iroha2-dev The re-implementation of a BFT hyperledger in RUST
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants