Skip to content

[PRA-264] Fix TIOBE workflow#212

Merged
Batalex merged 2 commits into3.5/edgefrom
pra-264-tiobe
Apr 23, 2026
Merged

[PRA-264] Fix TIOBE workflow#212
Batalex merged 2 commits into3.5/edgefrom
pra-264-tiobe

Conversation

@Batalex
Copy link
Copy Markdown
Contributor

@Batalex Batalex commented Apr 22, 2026

This PR fixes the broken TIOBE workflow that had gone unnoticed for a while.

The root cause was simple: we (and by "we", I mean "I") introduced a while ago a test using skopeo to check the compatibility between the various OCI resources involved.

Instead of installing the tool in the runner, I deselected this specific test since it does not contribute to the testing coverage and makes external network calls. I don't have the strongest of opinions here, so I'm happy to proceed the other way around and install skopeo on the runner.

You can check in the action tab that we now have a green run from the workflow (and even check the dashboard with a new quality measurement from today). Since I removed the commit in this PR running the workflow on pull request, the thing is a bit hidden.

@Batalex Batalex self-assigned this Apr 22, 2026
Copy link
Copy Markdown
Member

@theoctober19th theoctober19th left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

One way we can still keep the test and also not have to install skopeo in the runner machine is to install rockcraft snap in the runner, and use skopeo: str to provide rockcraft.skopeo value. I don't have strong opinion on keeping / removing the test from tox -e unit as long as it is possible to invoke this test somewhere separately in the CI (such that we don't leave gaps updating image with wrong version).

@Batalex Batalex requested a review from a team April 22, 2026 17:36
@Batalex
Copy link
Copy Markdown
Contributor Author

Batalex commented Apr 22, 2026

One way we can still keep the test and also not have to install skopeo in the runner machine is to install rockcraft snap in the runner, and use skopeo: str to provide rockcraft.skopeo value. I don't have strong opinion on keeping / removing the test from tox -e unit as long as it is possible to invoke this test somewhere separately in the CI (such that we don't leave gaps updating image with wrong version).

Oh yes, we definitely could install the tool in the runner, whether it be through the rockcraft snap or the vanilla tool itself. We could also just skip the test if skopeo is not found in the PATH, which is something I would prefer not to do.

@Batalex Batalex merged commit 83c97f5 into 3.5/edge Apr 23, 2026
43 checks passed
@Batalex Batalex deleted the pra-264-tiobe branch April 23, 2026 09:37
deusebio pushed a commit that referenced this pull request Apr 24, 2026
Batalex added a commit that referenced this pull request Apr 24, 2026
* ci: Exclude unit test requiring skopeo from tiobe workflow

* ci: Streamline tiobe workflow
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants