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

chore: dependency updates #165

Merged
merged 2 commits into from
Sep 23, 2024
Merged

chore: dependency updates #165

merged 2 commits into from
Sep 23, 2024

Conversation

zach-robinson-dev
Copy link
Contributor

@zach-robinson-dev zach-robinson-dev commented Sep 20, 2024

Updates to the latest versions possible. Unfortunately, I couldn't update kube.rs, k8s-openapi, or kube-derive to latest versions due to compatibility issues with kubert, since its latest version is still using much older versions of these. Fortunately, I was still able to get new enough versions to pull in the bug fix need to close #159.

@zach-robinson-dev
Copy link
Contributor Author

As a side note, the new kubert version does include an extra optional crate that should let us expose the tokio runtime metrics to Prometheus pretty easily if we ever see any weird concurrency issues again.

@zach-robinson-dev zach-robinson-dev marked this pull request as ready for review September 20, 2024 18:25
@zach-robinson-dev zach-robinson-dev requested a review from a team September 20, 2024 18:25
Copy link
Contributor

@lukebond lukebond left a comment

Choose a reason for hiding this comment

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

i think you know this but just in case you don't: this component needs to be semi-manually deployed, i.e. update the image revision pin in jsonnet in k8s-idpe under apps/tubernetes/infra

@zach-robinson-dev zach-robinson-dev merged commit 2ada942 into main Sep 23, 2024
2 checks passed
@zach-robinson-dev zach-robinson-dev deleted the chore/dependency-updates branch September 23, 2024 18:57
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.

sinker occasionally stops reconciling some ResourceSyncs
2 participants