-
Notifications
You must be signed in to change notification settings - Fork 2.8k
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
Prepare for release v1.8.7 #15017
Prepare for release v1.8.7 #15017
Conversation
@@ -34,7 +34,7 @@ const ( | |||
// | |||
// Maintainers: Run ./Documentation/check-crd-compat-table.sh for each release | |||
// Developers: Bump patch for each change in the CRD schema. | |||
CustomResourceDefinitionSchemaVersion = "1.21.2" | |||
CustomResourceDefinitionSchemaVersion = "1.21.3" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
For this, I just ran check-compat-tables.h and did what it said without thought:
$ Documentation/check-crd-compat-ta
ble.sh v1.8
Current version for branch v1.8 should be 1.21.3, not 1.21.2, please run the following command to fix it:
git checkout origin/v1.8 && \
sed -i 's+1.21.2+1.21.3+' pkg/k8s/apis/cilium.io/v2/register.go
But I think this always increments for every release..? Is that intended @aanm ?
If not, how does someone actually know if the CRD changes? Are there specific file paths we are supposed to pay attention to?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't think we want to increment the version. Last release I had the same issue and Andre mentioned to ignore it for now until it's fixed.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah if CRDs were not changed then we shouldn't increment this version
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM besides the CRD schema version
2606949
to
e2cbc8f
Compare
Signed-off-by: Joe Stringer <joe@cilium.io>
e2cbc8f
to
ae05a55
Compare
Summary of Changes
Minor Changes:
Bugfixes:
when the sender sets the "don't fragment" (DF) bit. (Backport PR v1.8 backports 2021-01-26 #14740, Upstream PR pkg/node: Skip setting MTU on local node routes #14679, @aditighag)
k8s_event_lag_seconds
metric. (Backport PR v1.8 backports 2020-12-17 #14441, Upstream PR pkg/k8s: fix k8s_event_lag_seconds for negative time #14313, @aanm)Fix IPv6 connectivity to BPF HostPort when kube-proxy is installed (Backport PR v1.8 backports 2021-02-12 #14953, Upstream PR bpf: Do not bypass conntrack if running kube-proxy+vxlan+endpoint routes #14675, @pchaigno)
CI Changes:
Misc Changes:
Other Changes: