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

Fix plain emit with Anonymous Credentials #692 #693

Closed
wants to merge 2 commits into from

Conversation

daaniam
Copy link

@daaniam daaniam commented Sep 17, 2024

No description provided.

@Tommaso-FLYR
Copy link

thank you! This broke our CI :/

@daaniam
Copy link
Author

daaniam commented Sep 18, 2024

@Tommaso-FLYR, do you have an estimated timeframe for when it might be fixed? We use it without a pinned version in a few Dockerfiles as a UV tool. Thank you.

@Tommaso-FLYR
Copy link

@Tommaso-FLYR, do you have an estimated timeframe for when it might be fixed? We use it without a pinned version in a few Dockerfiles as a UV tool. Thank you.

well I'm not a maintainer here so I can't answer to that. However we probably have the same setup (dockerfile + uv)

This is how I fixed it:

- RUN uv pip install keyring keyrings.google-artifactregistry-auth && \
+ RUN uv pip install keyring==25.3.0 keyrings.google-artifactregistry-auth && \
    uv pip install --keyring-provider subprocess -r requirements.txt && \
    uv pip uninstall keyring keyrings.google-artifactregistry-auth

@jaraco
Copy link
Owner

jaraco commented Sep 20, 2024

Per #694, the issue also affects _emit_json. Since #695 addresses both concerns, I'm going to work from that one. Regardless, thanks for the contrib, as it re-enforces the approach proposed there.

@jaraco jaraco closed this Sep 20, 2024
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