Skip to content
This repository has been archived by the owner on Jul 21, 2023. It is now read-only.

build(deps): bump k8s-openapi from 0.12.0 to 0.13.1 in /facilitator #1028

Merged
merged 2 commits into from Jan 3, 2022

Conversation

dependabot[bot]
Copy link
Contributor

@dependabot dependabot bot commented on behalf of github Oct 11, 2021

Bumps k8s-openapi from 0.12.0 to 0.13.1.

Release notes

Sourced from k8s-openapi's releases.

v0.13.1 (2021-10-08)

k8s-openapi

  • BUGFIX: v0.13.0 added schemars::JsonSchema impls for resource types. For types like k8s_openapi::apimachinery::pkg::apis::meta::v1::FieldsV1 that are objects with no defined structure, the impl incorrectly emitted the schema as {} instead of { "type": "object" }. This has now been fixed.

Corresponding Kubernetes API server versions:

  • v1.11.10
  • v1.12.10
  • v1.13.12
  • v1.14.10
  • v1.15.12
  • v1.16.15
  • v1.17.17
  • v1.18.20
  • v1.19.15
  • v1.20.11
  • v1.21.5
  • v1.22.2

k8s-openapi-codegen-common

  • BUGFIX: See the bugfix mentioned above.

k8s-openapi-derive

  • No changes except to bump the k8s-openapi-codegen-common dependency to the new version.

v0.13.0 (2021-08-09)

k8s-openapi

  • BREAKING CHANGE: The change from v0.12.0 to make Option<Vec<T>> and Option<BTreeMap<K, V>> fields into Vec<T> and BTreeMap<K, V> fields has been reverted, because there is at least one case where a resource type needs to be serialized with an empty Vec field. See Arnavion/k8s-openapi#103

  • BREAKING CHANGE: Added support for Kubernetes 1.22 under the v1_22 feature.

  • BREAKING CHANGE: Fixed api::core::v1::Pod::connect_get_namespaced_pod_exec to take its optional command parameter as Option<&'a [String]> instead of Option<&'a str>

  • BREAKING CHANGE: Fixed api::core::v1::ServiceSpec's clusterIPs field to be correctly emitted as cluster_ips instead of cluster_i_ps

  • FEATURE: The new schemars crate feature enables impls of schemars::JsonSchema on all resource types.

Corresponding Kubernetes API server versions:

  • v1.11.10
  • v1.12.10
  • v1.13.12
  • v1.14.10
  • v1.15.12

... (truncated)

Changelog

Sourced from k8s-openapi's changelog.

v0.13.1 (2021-10-08)

k8s-openapi

  • BUGFIX: v0.13.0 added schemars::JsonSchema impls for resource types. For types like k8s_openapi::apimachinery::pkg::apis::meta::v1::FieldsV1 that are objects with no defined structure, the impl incorrectly emitted the schema as {} instead of { "type": "object" }. This has now been fixed.

Corresponding Kubernetes API server versions:

  • v1.11.10
  • v1.12.10
  • v1.13.12
  • v1.14.10
  • v1.15.12
  • v1.16.15
  • v1.17.17
  • v1.18.20
  • v1.19.15
  • v1.20.11
  • v1.21.5
  • v1.22.2

k8s-openapi-codegen-common

  • BUGFIX: See the bugfix mentioned above.

k8s-openapi-derive

  • No changes except to bump the k8s-openapi-codegen-common dependency to the new version.

v0.13.0 (2021-08-09)

k8s-openapi

  • BREAKING CHANGE: The change from v0.12.0 to make Option<Vec<T>> and Option<BTreeMap<K, V>> fields into Vec<T> and BTreeMap<K, V> fields has been reverted, because there is at least one case where a resource type needs to be serialized with an empty Vec field. See Arnavion/k8s-openapi#103

  • BREAKING CHANGE: Added support for Kubernetes 1.22 under the v1_22 feature.

  • BREAKING CHANGE: Fixed api::core::v1::Pod::connect_get_namespaced_pod_exec to take its optional command parameter as Option<&'a [String]> instead of Option<&'a str>

  • BREAKING CHANGE: Fixed api::core::v1::ServiceSpec's clusterIPs field to be correctly emitted as cluster_ips instead of cluster_i_ps

  • FEATURE: The new schemars crate feature enables impls of schemars::JsonSchema on all resource types.

Corresponding Kubernetes API server versions:

  • v1.11.10
  • v1.12.10
  • v1.13.12

... (truncated)

Commits
  • 0e5de07 v0.13.1
  • d612c83 Make the root crate a cargo workspace of all the other crates.
  • 5c08ac9 Update dirs to v4
  • a2473c8 Fix schemars::JsonSchema of "object" type schemas without additional proper...
  • 5c1a3e2 Update to v1.21.5, v1.22.2
  • 2059b37 Fix new clippy lint in 1.55.0 about unnecessarily .collect()ing an Iterator.
  • 7bfa986 Update to v1.19.15, v1.20.11
  • 10325aa Update to v1.22.1
  • e099020 Update to v1.19.14, v1.20.10, v1.21.4
  • 8b9ee50 v0.13.0
  • Additional commits viewable in compare view

Dependabot compatibility score

Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting @dependabot rebase.


Dependabot commands and options

You can trigger Dependabot actions by commenting on this PR:

  • @dependabot rebase will rebase this PR
  • @dependabot recreate will recreate this PR, overwriting any edits that have been made to it
  • @dependabot merge will merge this PR after your CI passes on it
  • @dependabot squash and merge will squash and merge this PR after your CI passes on it
  • @dependabot cancel merge will cancel a previously requested merge and block automerging
  • @dependabot reopen will reopen this PR if it is closed
  • @dependabot close will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually
  • @dependabot ignore this major version will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this minor version will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this dependency will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)

@dependabot dependabot bot added dependencies Pull requests that update a dependency file rust Pull requests that update Rust code labels Oct 11, 2021
@codecov-commenter
Copy link

codecov-commenter commented Oct 11, 2021

Codecov Report

Merging #1028 (5a58067) into main (c228a23) will not change coverage.
The diff coverage is 0.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##             main    #1028   +/-   ##
=======================================
  Coverage   58.23%   58.23%           
=======================================
  Files          39       39           
  Lines        7725     7725           
=======================================
  Hits         4499     4499           
  Misses       3103     3103           
  Partials      123      123           
Flag Coverage Δ
deploy_tool_tests 40.19% <ø> (ø)
facilitator_tests 61.01% <0.00%> (ø)
key_rotator_tests 57.58% <ø> (ø)
workflow_manager_tests 38.50% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
facilitator/src/bin/facilitator.rs 0.11% <0.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update c228a23...5a58067. Read the comment docs.

@dependabot dependabot bot force-pushed the dependabot/cargo/facilitator/k8s-openapi-0.13.1 branch from ff34bb3 to d167572 Compare November 11, 2021 12:30
Bumps [k8s-openapi](https://github.com/Arnavion/k8s-openapi) from 0.12.0 to 0.13.1.
- [Release notes](https://github.com/Arnavion/k8s-openapi/releases)
- [Changelog](https://github.com/Arnavion/k8s-openapi/blob/master/CHANGELOG.md)
- [Commits](Arnavion/k8s-openapi@v0.12.0...v0.13.1)

---
updated-dependencies:
- dependency-name: k8s-openapi
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
@dependabot dependabot bot force-pushed the dependabot/cargo/facilitator/k8s-openapi-0.13.1 branch from d167572 to a4d3ad0 Compare December 17, 2021 00:55
To update `k8s-openapi` past 0.13, we also need to move to a newer
`kube`. Going past `kube` 0.59 _should_ let us drop our dependency on
OpenSSL. We also don't need to depend on `kube-runtime` via
`Cargo.toml` directly.

Resolves #451
@tgeoghegan
Copy link
Collaborator

To get this working, I had to dust off my changes from #451. However now I need to test this in a real Kubernetes cluster to make sure it actually works.

@tgeoghegan
Copy link
Collaborator

This has been running happily in my dev env over the weekend, in both AWS and GCP, so I think it is OK to merge. Sadly our facilitator images don't get any smaller for having dropped the OpenSSL dependency: we were only installing openssl-dev in the builder container, and it seems the alpine-3.15 image contains libssl no matter what. Still, we are better off for using rustls everywhere.

@tgeoghegan tgeoghegan merged commit b8726bf into main Jan 3, 2022
@tgeoghegan tgeoghegan deleted the dependabot/cargo/facilitator/k8s-openapi-0.13.1 branch January 3, 2022 18:31
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
dependencies Pull requests that update a dependency file rust Pull requests that update Rust code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants