Sync the rpm manifest with the deb ones #197
Conversation
| @@ -5,7 +5,7 @@ | |||
|
|
|||
| Name: kubelet | |||
| Version: %{KUBE_VERSION} | |||
| Release: %{RPM_RELEASE} | |||
| Release: %{RPM_RELEASE}.1 | |||
There was a problem hiding this comment.
I would just bump the RPM_RELEASE counter here to signal a packaging change not an actual software change. Same for the .0 below.
| @@ -20,9 +20,14 @@ Source5: https://storage.googleapis.com/kubernetes-release/network-plugins/cni-a | |||
|
|
|||
| BuildRequires: curl | |||
| Requires: iptables >= 1.4.21 | |||
| Requires: kubernetes-cni >= 0.3.0.1 | |||
There was a problem hiding this comment.
Is this a required package? I thought that was optional.
…f kubelet or kubeadm. See kubernetes/release#197.
| Requires: socat | ||
| Requires: util-linux | ||
| Requires: ethtool | ||
| Requires: iproute2 |
There was a problem hiding this comment.
Have this been tested? AFAIK there is no package named iproute2 on RHEL and CentOS 7. This dependency can not be meet.
There was a problem hiding this comment.
You are correct, found and fixed recently in #212 but this hasn't made it to unstable yet.
There was a problem hiding this comment.
@dgoodwin any ETA on the unstable repo?
There was a problem hiding this comment.
Afraid I'm not sure, @mikedanese @luxas could we get a new unstable build going soon? It's currently quite busted on CentOS.
Fix link to 1.8 release team doc
@dgoodwin @mikedanese PTAL