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

Release 1.23.2 #3521

Merged
merged 7 commits into from
Aug 4, 2021
Merged

Release 1.23.2 #3521

merged 7 commits into from
Aug 4, 2021

Conversation

kingdonb
Copy link
Member

@kingdonb kingdonb commented Jul 29, 2021

This release only changes one thing, a bug fix:

Reviewer, please see also the Chart update PR in:

Kingdon Barrett added 5 commits July 22, 2021 22:57
Signed-off-by: Kingdon Barrett <kingdon@weave.works>
Signed-off-by: Kingdon Barrett <kingdon@weave.works>
@kingdonb
Copy link
Member Author

kingdonb commented Jul 30, 2021

We can consider adding

in this release as well, but if there are issues with approving either of those straight away...

I would rather release this version from this PR as-is, because it has already been confirmed as a fix for current users who are experiencing the issue described from #3015 in the wild.

(Let's merge these two PRs in another patch release later in this series, I will open a 1.23.3 milestone if there is agreement.)

@stefanprodan stefanprodan requested review from squaremo and removed request for stefanprodan August 1, 2021 11:20
@kingdonb
Copy link
Member Author

kingdonb commented Aug 1, 2021

Based on the roundabout steps I had to take to attempt to triage #3523 this morning (which is probably the same issue fixed by this PR, #3485), I think this release should probably not be published without including some specific new documentation.

Perhaps adding an FAQ entry or two would make it better, about ECR and the common modes of failure we have seen, and the workarounds that are possible. The workaround is really small, but the fact that it requires a docs update and users to change their behavior in order to take advantage, suggests to me this change may be a MINOR rather than PATCH update.

Users have to know, for example, that in many common cases the Flux daemon's ECR pre-flight checks will not succeed, and they will need to provide region information manually to enable scanning of ECRs, even if they are properly permitted by IAM.

@kingdonb
Copy link
Member Author

kingdonb commented Aug 4, 2021

We have agreement in today's Flux Dev meeting that this is a PATCH release, as it fixes a bug.

If you specify the region, then Flux has no reason to try to reach the Instance Metadata API, and therefore it is a bug. So I now think this release should go out as-is. There is no new required reading, I will update the changelog to reflect the actual release date and get this released soon.

@kingdonb
Copy link
Member Author

kingdonb commented Aug 4, 2021

Approved, via #3522

Kingdon Barrett added 2 commits August 4, 2021 09:51
Signed-off-by: Kingdon Barrett <kingdon@weave.works>
Signed-off-by: Kingdon Barrett <kingdon@weave.works>
@kingdonb kingdonb merged commit c2d39c3 into release/1.23.x Aug 4, 2021
@kingdonb kingdonb added this to the 1.23.2 milestone Aug 4, 2021
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