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: Install ca-certificates package #1433

Merged
merged 2 commits into from
Feb 6, 2025
Merged

fix: Install ca-certificates package #1433

merged 2 commits into from
Feb 6, 2025

Conversation

akash1810
Copy link
Member

@akash1810 akash1810 commented Feb 6, 2025

What does this change?

It appears v6.15.0 of the CloudQuery CLI image no longer includes ca-certificates, which provides the update-ca-certificates command. Add it back!

The changes in #1405 didn't actually do anything as we have to manually publish a new image. I've added a note about this as an inline comment. We should automate this process in future though.

How has it been verified?

I've manually run the workflow to publish an image.

@akash1810 akash1810 requested review from a team as code owners February 6, 2025 09:32
It appears v6.15.0 of the CloudQuery CLI image no longer includes `ca-certificates`,
which provides the `update-ca-certificates` command.

Add it back!
Copy link
Member

@kelvin-chappell kelvin-chappell left a comment

Choose a reason for hiding this comment

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

👍

@akash1810 akash1810 merged commit 4567a55 into main Feb 6, 2025
7 of 8 checks passed
@akash1810 akash1810 deleted the aa/cq-docker branch February 6, 2025 11:56
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.

2 participants