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

ci: add support for injecting the latest ops when uv is used #1477

Merged

Conversation

tonyandrewmeyer
Copy link
Contributor

@tonyandrewmeyer tonyandrewmeyer commented Nov 29, 2024

When uv is used (as with self-signed-certificates-operator), the published-charms-tests workflow doesn't know how to inject the latest version of ops (there's no requirements.txt file).

This PR updates the workflow to look for uv.lock and if it exists then it uses uv to change the ops source in pyproject.toml.

self-signed-certificates-operator doesn't use uv to do the install in tox so we have to modify the raw sources list rather than have it be uv-specific. However, this should work even for projects that do, since the raw list is the originator of the dependencies - we might need to re-lock if other projects handle this differently, but we don't for now, so I've avoided that.

I've tested this manually, and have it manually against the branch (where it did install ops from my branch).

Copy link
Contributor

@james-garner-canonical james-garner-canonical left a comment

Choose a reason for hiding this comment

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

Comparing the failures on your manual run to the previous one, it looks like self-signed-certificates-operator passes now, and there are no new failures

@tonyandrewmeyer tonyandrewmeyer merged commit f1737bc into canonical:main Dec 2, 2024
34 checks passed
@tonyandrewmeyer tonyandrewmeyer deleted the published-charms-support-uv branch December 2, 2024 08:41
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