Skip to content
This repository has been archived by the owner on Nov 1, 2022. It is now read-only.

Release 1.25.4 #3641

Merged
merged 13 commits into from
Aug 30, 2022
Merged

Release 1.25.4 #3641

merged 13 commits into from
Aug 30, 2022

Conversation

kingdonb
Copy link
Member

This PR should be amended to include:

Once that one merges first, I will merge master to release/1.25.4 and this release should be ready to go.

Paulo Gomes and others added 10 commits July 27, 2022 12:04
Housekeeping: merge release branch 1.25.x into master
Signed-off-by: Paulo Gomes <paulo.gomes@weave.works>
We are removing all of the telemetry code as it is not needed, and being
on by default it may be contrary to CNCF policies around telemetry.

* remove check function

With no telemetry, there's no need for these function implementations or
this syscall in this package anymore.

We need to preserve the cadence of checkpoint.go to avoid disturbing any
thing that would constitute a breaking change, but basically all of the
underlying behavior of check can go away, since every Flux v1 version is
out of date every time, we don't need any remote service or lookup to
tell if an update is needed. (It's always needed)

Users should follow the link to upgrade (eg. the migration guide)

Signed-off-by: Kingdon Barrett <kingdon@weave.works>

Update pkg/checkpoint/checkpoint.go

Signed-off-by: Kingdon Barrett <kingdon@weave.works>
Co-authored-by: Stefan Prodan <stefan.prodan@gmail.com>
* to NOTES.txt
* to fluxctl install

Let new users know as soon as they start using Flux that this version is
going to be deprecated.

Signed-off-by: Kingdon Barrett <kingdon@weave.works>
Add deprecation warnings to Helm Chart and fluxctl
Signed-off-by: Kingdon Barrett <kingdon@weave.works>
Signed-off-by: Kingdon Barrett <kingdon@weave.works>
@kingdonb kingdonb mentioned this pull request Aug 29, 2022
@kingdonb kingdonb added this to the 1.25.4 milestone Aug 29, 2022
Copy link
Member

@pjbgf pjbgf left a comment

Choose a reason for hiding this comment

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

LGTM - pending the dependencies PR merge.

Kingdon Barrett and others added 3 commits August 30, 2022 06:49
* no need to upgrade Flux to yaml.v3

The dependencies which use yaml.v3 pinned a specific unreleased version
that was not secure, so we should replace it. (There's no CVE reported
against v2.4.0, the version that Flux itself uses internally.)

Co-authored-by: Paulo Gomes <paulo.gomes@weave.works>
Signed-off-by: Kingdon Barrett <kingdon@weave.works>
@kingdonb kingdonb marked this pull request as ready for review August 30, 2022 11:22
@kingdonb kingdonb merged commit 9549334 into release/1.25.x Aug 30, 2022
@kingdonb kingdonb deleted the release/1.25.4 branch August 30, 2022 11:23
@kingdonb kingdonb restored the release/1.25.4 branch August 30, 2022 11:23
@pjbgf pjbgf deleted the release/1.25.4 branch August 30, 2022 11:55
@pjbgf pjbgf restored the release/1.25.4 branch August 30, 2022 11:56
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants